Use import.meta.env?.DEV

This will require a dev environment to enable (or manually setting
import.meta.env.DEV = true.
Otherwise prod behavior will be received.

Author: NullVoxPopuli

CONTRIBUTING.md

@@ -276,9 +276,7 @@ There are helpers for many of these functions, which will resolve this for you:
 If your tests can't or aren't covered by a helper, one common solution is the use of `DEBUG` flag. Wrapping the debug-only dependent test in a check of this flag will cause that test to not be run in the prod test suite:
 
 ```javascript
-import { DEBUG } from '@glimmer/env';
-
-if (DEBUG) {
+if (import.meta.env?.DEV ?? true) {
   // Development-only test goes here
 }
 ```

package.json

@@ -390,4 +390,4 @@
     }
   },
   "packageManager": "[email protected]"
-}
+}

packages/@ember/-internals/container/lib/container.ts

@@ -9,7 +9,7 @@ import type {
 import { setOwner } from '@ember/-internals/owner';
 import { dictionary } from '@ember/-internals/utils';
 import { assert } from '@ember/debug';
-import { DEBUG } from '@glimmer/env';
+
 import type { DebugRegistry } from './registry';
 import type Registry from './registry';
 
@@ -24,7 +24,7 @@ declare const gc: undefined | (() => void);
 
 let leakTracking: LeakTracking;
 let containers: WeakSet<Container>;
-if (DEBUG) {
+if (import.meta.env?.DEV) {
   // requires v8
   // chrome --js-flags="--allow-natives-syntax --expose-gc"
   // node --allow-natives-syntax --expose-gc
@@ -92,7 +92,7 @@ export default class Container {
     this.isDestroyed = false;
     this.isDestroying = false;
 
-    if (DEBUG) {
+    if (import.meta.env?.DEV) {
       this.validationCache = dictionary(options.validationCache || null);
       if (containers !== undefined) {
         containers.add(this);
@@ -232,7 +232,7 @@ export default class Container {
   }
 }
 
-if (DEBUG) {
+if (import.meta.env?.DEV) {
   Container._leakTracking = leakTracking!;
 }
 
@@ -312,13 +312,13 @@ function factoryFor(
     return;
   }
 
-  if (DEBUG && factory && typeof factory._onLookup === 'function') {
+  if (import.meta.env?.DEV && factory && typeof factory._onLookup === 'function') {
     factory._onLookup(fullName);
   }
 
   let manager = new InternalFactoryManager(container, factory, fullName, normalizedName);
 
-  if (DEBUG) {
+  if (import.meta.env?.DEV) {
     manager = wrapManagerInDeprecationProxy(manager);
   }
 
@@ -541,7 +541,7 @@ export class InternalFactoryManager<
     setOwner(props, container.owner!);
     setFactoryFor(props, this);
 
-    if (DEBUG) {
+    if (import.meta.env?.DEV) {
       let lazyInjections;
       let validationCache = this.container.validationCache;
       // Ensure that all lazy injections are valid at instantiation time

packages/@ember/-internals/container/lib/registry.ts

@@ -10,7 +10,7 @@ import type {
 import { dictionary, intern } from '@ember/-internals/utils';
 import { assert } from '@ember/debug';
 import type { set } from '@ember/object';
-import { DEBUG } from '@glimmer/env';
+
 import type { ContainerOptions, LazyInjection } from './container';
 import Container from './container';
 
@@ -468,7 +468,7 @@ export declare class DebugRegistry extends Registry {
   validateInjections(injections: Injection[]): void;
 }
 
-if (DEBUG) {
+if (import.meta.env?.DEV) {
   const proto = Registry.prototype as DebugRegistry;
   proto.normalizeInjectionsHash = function (hash: { [key: string]: LazyInjection }) {
     let injections: Injection[] = [];

packages/@ember/-internals/container/tests/container_test.js

@@ -1,6 +1,6 @@
 import { getOwner } from '@ember/-internals/owner';
 import Service from '@ember/service';
-import { DEBUG } from '@glimmer/env';
+
 import { Registry } from '..';
 import { factory, moduleFor, AbstractTestCase, runTask } from 'internal-test-helpers';
 
@@ -447,7 +447,7 @@ moduleFor(
     }
 
     ['@test Lazy injection validations are cached'](assert) {
-      if (!DEBUG) {
+      if (!import.meta.env?.DEV) {
         assert.expect(0);
         return;
       }

packages/@ember/-internals/environment/lib/env.ts

@@ -1,4 +1,3 @@
-import { DEBUG } from '@glimmer/env';
 import global from './global';
 
 /**
@@ -91,7 +90,7 @@ export const ENV = {
     @default false
     @private
   */
-  _DEBUG_RENDER_TREE: DEBUG,
+  _DEBUG_RENDER_TREE: import.meta.env?.DEV,
 
   /**
    Whether to force all deprecations to be enabled. This is used internally by
@@ -200,7 +199,7 @@ export const ENV = {
     }
   }
 
-  if (DEBUG) {
+  if (import.meta.env?.DEV) {
     ENV._DEBUG_RENDER_TREE = true;
   }
 })(global.EmberENV);

packages/@ember/-internals/glimmer/lib/component-managers/curly.ts

@@ -9,7 +9,7 @@ import { addChildView, setElementView, setViewElement } from '@ember/-internals/
 import type { Nullable } from '@ember/-internals/utility-types';
 import { assert, debugFreeze } from '@ember/debug';
 import { _instrumentStart } from '@ember/instrumentation';
-import { DEBUG } from '@glimmer/env';
+
 import type {
   Bounds,
   CapturedArguments,
@@ -337,7 +337,7 @@ export default class CurlyComponentManager
       bucket.classRef = args.named.get('class');
     }
 
-    if (DEBUG) {
+    if (import.meta.env?.DEV) {
       processComponentInitializationAssertions(component, props);
     }
 

packages/@ember/-internals/glimmer/lib/component-managers/root.ts

@@ -1,7 +1,7 @@
 import { getFactoryFor } from '@ember/-internals/container';
 import { assert } from '@ember/debug';
 import { _instrumentStart } from '@ember/instrumentation';
-import { DEBUG } from '@glimmer/env';
+
 import type {
   ComponentDefinition,
   Environment,
@@ -57,7 +57,7 @@ class RootComponentManager extends CurlyComponentManager {
       }
     }
 
-    if (DEBUG) {
+    if (import.meta.env?.DEV) {
       processComponentInitializationAssertions(component, {});
     }
 

packages/@ember/-internals/glimmer/lib/component-managers/route-template.ts

@@ -14,7 +14,7 @@ import type {
   WithCustomDebugRenderTree,
 } from '@glimmer/interfaces';
 import type { Nullable } from '@ember/-internals/utility-types';
-import { DEBUG } from '@glimmer/env';
+
 import { capabilityFlagsFrom } from '@glimmer/manager';
 import type { Reference } from '@glimmer/reference';
 import { createDebugAliasRef, valueForRef } from '@glimmer/reference';
@@ -61,7 +61,7 @@ class RouteTemplateManager
   ): RouteTemplateInstanceState {
     let self = args.named.get('controller');
 
-    if (DEBUG) {
+    if (import.meta.env?.DEV) {
       self = createDebugAliasRef!('this', self);
     }
 

packages/@ember/-internals/glimmer/lib/component.ts

@@ -19,7 +19,7 @@ import {
 } from '@ember/-internals/views';
 import { guidFor } from '@ember/-internals/utils';
 import { assert } from '@ember/debug';
-import { DEBUG } from '@glimmer/env';
+
 import type { Environment, Template, TemplateFactory } from '@glimmer/interfaces';
 import { setInternalComponentManager } from '@glimmer/manager';
 import { isUpdatableRef, updateRef } from '@glimmer/reference';
@@ -939,7 +939,12 @@ class Component<S = unknown>
       }
     }
 
-    if (DEBUG && eventDispatcher && this.renderer._isInteractive && this.tagName === '') {
+    if (
+      import.meta.env?.DEV &&
+      eventDispatcher &&
+      this.renderer._isInteractive &&
+      this.tagName === ''
+    ) {
       let eventNames = [];
       let events = eventDispatcher.finalEventNameMapping;