ember-source 6.12.0-beta.3 → 7.0.0-beta.1

package/dist/{packages → dev/packages}/@ember/routing/history-location.js

@@ -1,10 +1,12 @@
 import EmberObject from '../object/index.js';
 import '../debug/index.js';
 import { getHash } from './lib/location-utils.js';
-import { isDevelopingApp } from '@embroider/macros';
 import { assert } from '../debug/lib/assert.js';
 
-let popstateFired = false;
+/**
+@module @ember/routing/history-location
+*/
+
 function _uuid() {
   return 'xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx'.replace(/[xy]/g, function (c) {
     let r, v;
@@ -58,7 +60,6 @@ class HistoryLocation extends EmberObject {
   // SAFETY: both of these properties initialized via `init`.
 
   history;
-  _previousURL;
   _popstateHandler;
 
   /**
@@ -101,11 +102,7 @@ class HistoryLocation extends EmberObject {
       state
     } = history;
     let path = this.formatURL(this.getURL());
-    if (state && state.path === path) {
-      // preserve existing state
-      // used for webkit workaround, since there will be no initial popstate event
-      this._previousURL = this.getURL();
-    } else {
+    if (!state || state.path !== path) {
       this.replaceState(path);
     }
   }
@@ -143,7 +140,7 @@ class HistoryLocation extends EmberObject {
     @param path {String}
   */
   setURL(path) {
-    (isDevelopingApp() && !(this.history) && assert('HistoryLocation.history is unexpectedly missing', this.history));
+    (!(this.history) && assert('HistoryLocation.history is unexpectedly missing', this.history));
     let {
       state
     } = this.history;
@@ -161,7 +158,7 @@ class HistoryLocation extends EmberObject {
     @param path {String}
   */
   replaceURL(path) {
-    (isDevelopingApp() && !(this.history) && assert('HistoryLocation.history is unexpectedly missing', this.history));
+    (!(this.history) && assert('HistoryLocation.history is unexpectedly missing', this.history));
     let {
       state
     } = this.history;
@@ -182,11 +179,8 @@ class HistoryLocation extends EmberObject {
       path,
       uuid: _uuid()
     };
-    (isDevelopingApp() && !(this.history) && assert('HistoryLocation.history is unexpectedly missing', this.history));
+    (!(this.history) && assert('HistoryLocation.history is unexpectedly missing', this.history));
     this.history.pushState(state, '', path);
-
-    // used for webkit workaround
-    this._previousURL = this.getURL();
   }
 
   /**
@@ -200,11 +194,8 @@ class HistoryLocation extends EmberObject {
       path,
       uuid: _uuid()
     };
-    (isDevelopingApp() && !(this.history) && assert('HistoryLocation.history is unexpectedly missing', this.history));
+    (!(this.history) && assert('HistoryLocation.history is unexpectedly missing', this.history));
     this.history.replaceState(state, '', path);
-
-    // used for webkit workaround
-    this._previousURL = this.getURL();
   }
 
   /**
@@ -217,13 +208,6 @@ class HistoryLocation extends EmberObject {
   onUpdateURL(callback) {
     this._removeEventListener();
     this._popstateHandler = () => {
-      // Ignore initial page load popstate event in Chrome
-      if (!popstateFired) {
-        popstateFired = true;
-        if (this.getURL() === this._previousURL) {
-          return;
-        }
-      }
       callback(this.getURL());
     };
     window.addEventListener('popstate', this._popstateHandler);

package/dist/dev/packages/@ember/routing/index.js

@@ -0,0 +1,14 @@
+import '../../shared-chunks/fragment-Cc5k9Oy4.js';
+import '../../shared-chunks/debug-to-string-CFb7h0lY.js';
+import '../../@glimmer/global-context/index.js';
+import '../../@glimmer/validator/index.js';
+import '../../shared-chunks/reference-C3TKDRnP.js';
+import '../../shared-chunks/index-Q7JnrdBn.js';
+import '../../@glimmer/destroyable/index.js';
+import '../../shared-chunks/capabilities-O_xc7Yqk.js';
+export { L as LinkTo } from '../../shared-chunks/index-PxU6E7q8.js';
+import '../-internals/container/index.js';
+import '../debug/index.js';
+import '../../shared-chunks/dynamic-CFg3dljk.js';
+import '../../shared-chunks/constants-eoaL3OJQ.js';
+import '../runloop/index.js';

package/dist/{packages → dev/packages}/@ember/routing/lib/dsl.js

@@ -1,5 +1,4 @@
 import '../../debug/index.js';
-import { isDevelopingApp } from '@embroider/macros';
 import { assert } from '../../debug/lib/assert.js';
 
 let uuid = 0;
@@ -27,18 +26,18 @@ class DSLImpl {
     let callback = null;
     let dummyErrorRoute = `/_unused_dummy_error_path_route_${name}/:error`;
     if (isCallback(_options)) {
-      (isDevelopingApp() && !(arguments.length === 2) && assert('Unexpected arguments', arguments.length === 2));
+      (!(arguments.length === 2) && assert('Unexpected arguments', arguments.length === 2));
       options = {};
       callback = _options;
     } else if (isCallback(_callback)) {
-      (isDevelopingApp() && !(arguments.length === 3) && assert('Unexpected arguments', arguments.length === 3));
-      (isDevelopingApp() && !(isOptions(_options)) && assert('Unexpected arguments', isOptions(_options)));
+      (!(arguments.length === 3) && assert('Unexpected arguments', arguments.length === 3));
+      (!(isOptions(_options)) && assert('Unexpected arguments', isOptions(_options)));
       options = _options;
       callback = _callback;
     } else {
       options = _options || {};
     }
-    (isDevelopingApp() && !((() => {
+    (!((() => {
       if (options.overrideNameAssertion === true) {
         return true;
       }
@@ -49,7 +48,7 @@ class DSLImpl {
       }
       return ['basic', 'application'].indexOf(name) === -1;
     })()));
-    (isDevelopingApp() && !(name.indexOf(':') === -1) && assert(`'${name}' is not a valid route name. It cannot contain a ':'. You may want to use the 'path' option instead.`, name.indexOf(':') === -1));
+    (!(name.indexOf(':') === -1) && assert(`'${name}' is not a valid route name. It cannot contain a ':'. You may want to use the 'path' option instead.`, name.indexOf(':') === -1));
     if (this.enableLoadingSubstates) {
       createRoute(this, `${name}_loading`, {
         resetNamespace: options.resetNamespace

package/dist/{packages → dev/packages}/@ember/routing/lib/generate_controller.js

@@ -1,22 +1,34 @@
 import '../../-internals/meta/lib/meta.js';
-import '../../../shared-chunks/mandatory-setter-DLKyVs4Q.js';
-import { isDevelopingApp } from '@embroider/macros';
+import '../../../shared-chunks/mandatory-setter-DHZe7-kW.js';
 import { info } from '../../debug/index.js';
 import '../../../@glimmer/destroyable/index.js';
 import '../../../@glimmer/validator/index.js';
-import '../../../shared-chunks/debug-to-string-BsFOvUtQ.js';
+import '../../../shared-chunks/debug-to-string-CFb7h0lY.js';
 import '../../../@glimmer/global-context/index.js';
-import '../../../shared-chunks/reference-B6HMX4y0.js';
-import '../../../shared-chunks/capabilities-DHiXCCuB.js';
-import { g as get } from '../../../shared-chunks/observers-DhgQ6ba5.js';
-import '../../../shared-chunks/env-DEd6hPbj.js';
+import '../../../shared-chunks/reference-C3TKDRnP.js';
+import '../../../shared-chunks/capabilities-O_xc7Yqk.js';
+import { g as get } from '../../../shared-chunks/observers-Bj9qLVau.js';
+import '../../-internals/environment/index.js';
 import Controller from '../../controller/index.js';
 import { assert } from '../../debug/lib/assert.js';
 
+/**
+ @module @ember/routing
+*/
+
+/**
+  Generates a controller factory
+
+  @for Ember
+  @method generateControllerFactory
+  @private
+*/
+
+/* eslint-disable @typescript-eslint/no-empty-object-type */
 function generateControllerFactory(owner, controllerName) {
   /* eslint-enable @typescript-eslint/no-empty-object-type */
   let factoryManager = owner.factoryFor('controller:basic');
-  (isDevelopingApp() && !(factoryManager !== undefined) && assert('[BUG] unexpectedly missing a factoryManager for `controller:basic`', factoryManager !== undefined)); // SAFETY: This is *not* safe, and the cast should be removed in favor of the
+  (!(factoryManager !== undefined) && assert('[BUG] unexpectedly missing a factoryManager for `controller:basic`', factoryManager !== undefined)); // SAFETY: This is *not* safe, and the cast should be removed in favor of the
   // `assert()` below after altering *tests*. It is left in this state for the
   // moment in the interest of keeping type-only changes separate from changes
   // to the runtime behavior of the system, even for tests.
@@ -51,8 +63,8 @@ function generateController(owner, controllerName) {
   generateControllerFactory(owner, controllerName);
   let fullName = `controller:${controllerName}`;
   let instance = owner.lookup(fullName);
-  (isDevelopingApp() && !(instance instanceof Controller) && assert('Expected an instance of controller', instance instanceof Controller));
-  if (isDevelopingApp()) {
+  (!(instance instanceof Controller) && assert('Expected an instance of controller', instance instanceof Controller));
+  {
     if (get(instance, 'namespace.LOG_ACTIVE_GENERATION')) {
       info(`generated -> ${fullName}`, {
         fullName

package/dist/{packages → dev/packages}/@ember/routing/lib/location-utils.js

@@ -1,11 +1,10 @@
 /**
   @private
 
-  Returns the current `location.pathname`, normalized for IE inconsistencies.
+  Returns the current `location.pathname`, ensuring it has a leading slash.
 */
 function getPath(location) {
   let pathname = location.pathname;
-  // Various versions of IE/Opera don't always return a leading slash
   if (pathname[0] !== '/') {
     pathname = `/${pathname}`;
   }
@@ -35,18 +34,6 @@ function getHash(location) {
 function getFullPath(location) {
   return getPath(location) + getQuery(location) + getHash(location);
 }
-function getOrigin(location) {
-  let origin = location.origin;
-
-  // Older browsers, especially IE, don't have origin
-  if (!origin) {
-    origin = `${location.protocol}//${location.hostname}`;
-    if (location.port) {
-      origin += `:${location.port}`;
-    }
-  }
-  return origin;
-}
 
 /**
   Replaces the current location, making sure we explicitly include the origin
@@ -55,7 +42,7 @@ function getOrigin(location) {
   @private
 */
 function replacePath(location, path) {
-  location.replace(getOrigin(location) + path);
+  location.replace(location.origin + path);
 }
 
-export { getFullPath, getHash, getOrigin, getPath, getQuery, replacePath };
+export { getFullPath, getHash, getPath, getQuery, replacePath };

package/dist/{packages → dev/packages}/@ember/routing/lib/routing-service.js

@@ -1,16 +1,15 @@
 import { getOwner } from '../../-internals/owner/index.js';
 import '../../debug/index.js';
 import '../../-internals/meta/lib/meta.js';
-import '../../../shared-chunks/mandatory-setter-DLKyVs4Q.js';
-import { isDevelopingApp } from '@embroider/macros';
+import '../../../shared-chunks/mandatory-setter-DHZe7-kW.js';
 import '../../../@glimmer/destroyable/index.js';
 import '../../../@glimmer/validator/index.js';
-import '../../../shared-chunks/debug-to-string-BsFOvUtQ.js';
+import '../../../shared-chunks/debug-to-string-CFb7h0lY.js';
 import '../../../@glimmer/global-context/index.js';
-import '../../../shared-chunks/reference-B6HMX4y0.js';
-import '../../../shared-chunks/capabilities-DHiXCCuB.js';
-import '../../../shared-chunks/observers-DhgQ6ba5.js';
-import '../../../shared-chunks/env-DEd6hPbj.js';
+import '../../../shared-chunks/reference-C3TKDRnP.js';
+import '../../../shared-chunks/capabilities-O_xc7Yqk.js';
+import '../../../shared-chunks/observers-Bj9qLVau.js';
+import '../../-internals/environment/index.js';
 import { readOnly } from '../../object/lib/computed/computed_macros.js';
 import '../../object/index.js';
 import '../../object/core.js';
@@ -32,6 +31,17 @@ import { assert } from '../../debug/lib/assert.js';
 @module ember
 */
 
+
+/**
+  The Routing service is used by LinkTo, and provides facilities for
+  the component/view layer to interact with the router.
+
+  This is a private service for internal usage only. For public usage,
+  refer to the `Router` service.
+
+  @private
+  @class RoutingService
+*/
 class RoutingService extends Service {
   [ROUTER];
   get router() {
@@ -40,9 +50,9 @@ class RoutingService extends Service {
       return router;
     }
     let owner = getOwner(this);
-    (isDevelopingApp() && !(owner) && assert('RoutingService is unexpectedly missing an owner', owner));
+    (!(owner) && assert('RoutingService is unexpectedly missing an owner', owner));
     let _router = owner.lookup('router:main');
-    (isDevelopingApp() && !(_router instanceof EmberRouter) && assert('ROUTING SERVICE BUG: Expected router to be an instance of EmberRouter', _router instanceof EmberRouter));
+    (!(_router instanceof EmberRouter) && assert('ROUTING SERVICE BUG: Expected router to be an instance of EmberRouter', _router instanceof EmberRouter));
     _router.setupRouter();
     return this[ROUTER] = _router;
   }

package/dist/{packages → dev/packages}/@ember/routing/lib/utils.js

@@ -1,20 +1,19 @@
 import '../../-internals/meta/lib/meta.js';
-import '../../../shared-chunks/mandatory-setter-DLKyVs4Q.js';
-import { isDevelopingApp } from '@embroider/macros';
+import '../../../shared-chunks/mandatory-setter-DHZe7-kW.js';
 import '../../debug/index.js';
 import '../../../@glimmer/destroyable/index.js';
 import '../../../@glimmer/validator/index.js';
-import '../../../shared-chunks/debug-to-string-BsFOvUtQ.js';
+import '../../../shared-chunks/debug-to-string-CFb7h0lY.js';
 import '../../../@glimmer/global-context/index.js';
-import '../../../shared-chunks/reference-B6HMX4y0.js';
-import '../../../shared-chunks/capabilities-DHiXCCuB.js';
-import { g as get } from '../../../shared-chunks/observers-DhgQ6ba5.js';
-import '../../../shared-chunks/env-DEd6hPbj.js';
+import '../../../shared-chunks/reference-C3TKDRnP.js';
+import '../../../shared-chunks/capabilities-O_xc7Yqk.js';
+import { g as get } from '../../../shared-chunks/observers-Bj9qLVau.js';
+import '../../-internals/environment/index.js';
 import { getOwner } from '../../-internals/owner/index.js';
 import EngineInstance from '../../engine/instance.js';
 import '../../../route-recognizer/index.js';
 import '../../../shared-chunks/rsvp-CnCSY930.js';
-import { S as STATE_SYMBOL } from '../../../shared-chunks/unrecognized-url-error-BBMMZhBN.js';
+import { S as STATE_SYMBOL } from '../../../shared-chunks/unrecognized-url-error-B3wUTorp.js';
 import { assert } from '../../debug/lib/assert.js';
 
 const ALL_PERIODS_REGEX = /\./g;
@@ -33,7 +32,7 @@ function extractRouteArgs(args) {
   if (typeof args[0] === 'string') {
     routeName = args.shift();
     // We just checked this!
-    (isDevelopingApp() && !(typeof routeName === 'string') && assert('routeName is a string', typeof routeName === 'string'));
+    (!(typeof routeName === 'string') && assert('routeName is a string', typeof routeName === 'string'));
   }
 
   // SAFTEY: We removed the name and options if they existed, only models left.
@@ -47,7 +46,7 @@ function extractRouteArgs(args) {
 function getActiveTargetName(router) {
   let routeInfos = router.activeTransition ? router.activeTransition[STATE_SYMBOL].routeInfos : router.state.routeInfos;
   let lastRouteInfo = routeInfos[routeInfos.length - 1];
-  (isDevelopingApp() && !(lastRouteInfo) && assert('has last route info', lastRouteInfo));
+  (!(lastRouteInfo) && assert('has last route info', lastRouteInfo));
   return lastRouteInfo.name;
 }
 function stashParamNames(router, routeInfos) {
@@ -60,13 +59,13 @@ function stashParamNames(router, routeInfos) {
   // on whether a URL transition or named transition is happening.
   // Hopefully we can remove this in the future.
   let routeInfo = routeInfos[routeInfos.length - 1];
-  (isDevelopingApp() && !(routeInfo) && assert('has route info', routeInfo));
+  (!(routeInfo) && assert('has route info', routeInfo));
   let targetRouteName = routeInfo.name;
   let recogHandlers = router._routerMicrolib.recognizer.handlersFor(targetRouteName);
   let dynamicParent;
   for (let i = 0; i < routeInfos.length; ++i) {
     let routeInfo = routeInfos[i];
-    (isDevelopingApp() && !(routeInfo) && assert('has route info', routeInfo));
+    (!(routeInfo) && assert('has route info', routeInfo));
     let names = recogHandlers[i].names;
     if (names.length) {
       dynamicParent = routeInfo;
@@ -203,7 +202,7 @@ function resemblesURL(str) {
 function prefixRouteNameArg(route, args) {
   let routeName;
   let owner = getOwner(route);
-  (isDevelopingApp() && !(owner instanceof EngineInstance) && assert('Expected route to have EngineInstance as owner', owner instanceof EngineInstance));
+  (!(owner instanceof EngineInstance) && assert('Expected route to have EngineInstance as owner', owner instanceof EngineInstance));
   let prefix = owner.mountPoint;
 
   // only alter the routeName if it's actually referencing a route.

package/dist/{packages → dev/packages}/@ember/routing/none-location.js

@@ -1,6 +1,5 @@
 import EmberObject from '../object/index.js';
 import '../debug/index.js';
-import { isDevelopingApp } from '@embroider/macros';
 import { assert } from '../debug/lib/assert.js';
 
 /**
@@ -14,8 +13,7 @@ import { assert } from '../debug/lib/assert.js';
   application in a larger page).
 
   Using `NoneLocation` causes Ember to not store the applications URL state
-  in the actual URL. This is generally used for testing purposes, and is one
-  of the changes made when calling `App.setupForTesting()`.
+  in the actual URL. This is generally used for testing purposes.
 
   @class NoneLocation
   @extends EmberObject
@@ -43,7 +41,7 @@ class NoneLocation extends EmberObject {
     // This assert doesn't have anything to do with state initialization,
     // but we're hijacking this method since it's called after the route has
     // set the rootURL property on its Location instance.
-    (isDevelopingApp() && !(rootURL.charAt(rootURL.length - 1) === '/') && assert('rootURL must end with a trailing forward slash e.g. "/app/"', rootURL.charAt(rootURL.length - 1) === '/'));
+    (!(rootURL.charAt(rootURL.length - 1) === '/') && assert('rootURL must end with a trailing forward slash e.g. "/app/"', rootURL.charAt(rootURL.length - 1) === '/'));
   }
 
   /**

package/dist/{packages → dev/packages}/@ember/routing/route.js

@@ -1,20 +1,19 @@
-import { p as privatize } from '../../shared-chunks/registry-XY5cfmbH.js';
-import { g as get, I as flushAsyncObservers, b as descriptorForProperty, d as defineProperty, x as addObserver, c as computed } from '../../shared-chunks/observers-DhgQ6ba5.js';
+import { privatize } from '../-internals/container/index.js';
+import { g as get, H as flushAsyncObservers, b as descriptorForProperty, d as defineProperty, u as addObserver, c as computed } from '../../shared-chunks/observers-Bj9qLVau.js';
 import '../-internals/meta/lib/meta.js';
 import { info } from '../debug/index.js';
 import '../../@glimmer/validator/index.js';
-import { l as lookupDescriptor } from '../../shared-chunks/mandatory-setter-DLKyVs4Q.js';
-import { isDevelopingApp } from '@embroider/macros';
-import { i as isProxy } from '../../shared-chunks/is_proxy-C2q5rUMp.js';
+import { l as lookupDescriptor } from '../../shared-chunks/mandatory-setter-DHZe7-kW.js';
+import { i as isProxy } from '../../shared-chunks/is_proxy-B0smdQy8.js';
 import '../../@glimmer/destroyable/index.js';
-import { c as hasInternalComponentManager } from '../../shared-chunks/api-CkUl6KyJ.js';
+import { b as hasInternalComponentManager } from '../../shared-chunks/api-BqXkkT0p.js';
 import '../../@glimmer/global-context/index.js';
-import '../../shared-chunks/debug-to-string-BsFOvUtQ.js';
-import '../../shared-chunks/reference-B6HMX4y0.js';
-import '../../shared-chunks/capabilities-DHiXCCuB.js';
-import { s as set } from '../../shared-chunks/property_set-DrZnfGQ7.js';
-import { g as getProperties, s as setProperties } from '../../shared-chunks/set_properties-Dag9Xz6u.js';
-import '../../shared-chunks/env-DEd6hPbj.js';
+import '../../shared-chunks/debug-to-string-CFb7h0lY.js';
+import '../../shared-chunks/reference-C3TKDRnP.js';
+import '../../shared-chunks/capabilities-O_xc7Yqk.js';
+import { s as set } from '../../shared-chunks/property_set-DaoZXGM5.js';
+import { g as getProperties, s as setProperties } from '../../shared-chunks/set_properties-kVGzZL_a.js';
+import '../-internals/environment/index.js';
 import { getOwner } from '../-internals/owner/index.js';
 import EmberObject from '../object/index.js';
 import Evented from '../object/evented.js';
@@ -34,7 +33,7 @@ import { dependentKeyCompat } from '../object/compat.js';
 import { once } from '../runloop/index.js';
 import '../../route-recognizer/index.js';
 import '../../shared-chunks/rsvp-CnCSY930.js';
-import { S as STATE_SYMBOL, P as PARAMS_SYMBOL } from '../../shared-chunks/unrecognized-url-error-BBMMZhBN.js';
+import { S as STATE_SYMBOL, P as PARAMS_SYMBOL } from '../../shared-chunks/unrecognized-url-error-B3wUTorp.js';
 import generateController from './lib/generate_controller.js';
 import { prefixRouteNameArg, stashParamNames, calculateCacheKey, normalizeControllerQueryParams } from './lib/utils.js';
 import { a as decorateMethodV2 } from '../../shared-chunks/chunk-3SQBS3Y5-Cj4eryg1.js';
@@ -132,7 +131,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
     let object = {};
     if (params.length === 1) {
       let [name] = params;
-      (isDevelopingApp() && !(name) && assert('has name', name));
+      (!(name) && assert('has name', name));
       if (typeof model === 'object' && name in model) {
         object[name] = get(model, name);
       } else if (/_id$/.test(name)) {
@@ -289,7 +288,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
   _setRouteName(name) {
     this.routeName = name;
     let owner = getOwner(this);
-    (isDevelopingApp() && !(owner instanceof EngineInstance) && assert('Expected route to have EngineInstance as owner', owner instanceof EngineInstance));
+    (!(owner instanceof EngineInstance) && assert('Expected route to have EngineInstance as owner', owner instanceof EngineInstance));
     this.fullRouteName = getEngineRouteName(owner, name);
   }
 
@@ -381,7 +380,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
   */
   paramsFor(name) {
     let owner = getOwner(this);
-    (isDevelopingApp() && !(owner) && assert('Route is unexpectedly missing an owner', owner));
+    (!(owner) && assert('Route is unexpectedly missing an owner', owner));
     let route = owner.lookup(`route:${name}`);
     if (route === undefined) {
       return {};
@@ -394,7 +393,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
     };
     let queryParams = getQueryParamsFor(route, state);
     return Object.entries(queryParams).reduce((params, [key, value]) => {
-      (isDevelopingApp() && !(!params[key]) && assert(`The route '${this.routeName}' has both a dynamic segment and query param with name '${key}'. Please rename one to avoid collisions.`, !params[key]));
+      (!(!params[key]) && assert(`The route '${this.routeName}' has both a dynamic segment and query param with name '${key}'. Please rename one to avoid collisions.`, !params[key]));
       params[key] = value;
       return params;
     }, params);
@@ -636,7 +635,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
       let allParams = queryParams.propertyNames;
       allParams.forEach(prop => {
         let aQp = queryParams.map[prop];
-        (isDevelopingApp() && !(aQp) && assert('expected aQp', aQp));
+        (!(aQp) && assert('expected aQp', aQp));
         aQp.values = params;
         let cacheKey = calculateCacheKey(aQp.route.fullRouteName, aQp.parts, aQp.values);
         let value = cache.lookup(cacheKey, prop, aQp.undecoratedDefaultValue);
@@ -954,7 +953,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
 
   controllerFor(name, _skipAssert = false) {
     let owner = getOwner(this);
-    (isDevelopingApp() && !(owner) && assert('Route is unexpectedly missing an owner', owner));
+    (!(owner) && assert('Route is unexpectedly missing an owner', owner));
     let route = owner.lookup(`route:${name}`);
     if (route && route.controllerName) {
       name = route.controllerName;
@@ -964,8 +963,8 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
     // NOTE: We're specifically checking that skipAssert is true, because according
     //   to the old API the second parameter was model. We do not want people who
     //   passed a model to skip the assertion.
-    (isDevelopingApp() && !(controller !== undefined || _skipAssert === true) && assert(`The controller named '${name}' could not be found. Make sure that this route exists and has already been entered at least once. If you are accessing a controller not associated with a route, make sure the controller class is explicitly defined.`, controller !== undefined || _skipAssert === true));
-    (isDevelopingApp() && !(controller === undefined || controller instanceof Controller) && assert(`Expected controller:${name} to be an instance of Controller`, controller === undefined || controller instanceof Controller));
+    (!(controller !== undefined || _skipAssert === true) && assert(`The controller named '${name}' could not be found. Make sure that this route exists and has already been entered at least once. If you are accessing a controller not associated with a route, make sure the controller class is explicitly defined.`, controller !== undefined || _skipAssert === true));
+    (!(controller === undefined || controller instanceof Controller) && assert(`Expected controller:${name} to be an instance of Controller`, controller === undefined || controller instanceof Controller));
     return controller;
   }
 
@@ -987,7 +986,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
   */
   generateController(name) {
     let owner = getOwner(this);
-    (isDevelopingApp() && !(owner) && assert('Route is unexpectedly missing an owner', owner));
+    (!(owner) && assert('Route is unexpectedly missing an owner', owner));
     return generateController(owner, name);
   }
 
@@ -1028,7 +1027,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
   modelFor(_name) {
     let name;
     let owner = getOwner(this);
-    (isDevelopingApp() && !(owner instanceof EngineInstance) && assert('Expected router owner to be an EngineInstance', owner instanceof EngineInstance));
+    (!(owner instanceof EngineInstance) && assert('Expected router owner to be an EngineInstance', owner instanceof EngineInstance));
     let transition = this._router && this._router._routerMicrolib ? this._router._routerMicrolib.activeTransition : undefined;
 
     // Only change the route name when there is an active transition.
@@ -1120,17 +1119,17 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
   /** @deprecated Manually define your own store, such as with `@service store` */
   get _store() {
     const owner = getOwner(this);
-    (isDevelopingApp() && !(owner) && assert('Route is unexpectedly missing an owner', owner));
+    (!(owner) && assert('Route is unexpectedly missing an owner', owner));
     let routeName = this.routeName;
     return {
       find(name, value) {
         let modelClass = owner.factoryFor(`model:${name}`);
-        (isDevelopingApp() && !(Boolean(modelClass)) && assert(`You used the dynamic segment \`${name}_id\` in your route ` + `\`${routeName}\` for which Ember requires you provide a ` + `data-loading implementation. Commonly, that is done by ` + `adding a model hook implementation on the route ` + `(\`model({${name}_id}) {\`) or by injecting an implemention of ` + `a data store: \`@service store;\`.`, Boolean(modelClass)));
+        (!(Boolean(modelClass)) && assert(`You used the dynamic segment \`${name}_id\` in your route ` + `\`${routeName}\` for which Ember requires you provide a ` + `data-loading implementation. Commonly, that is done by ` + `adding a model hook implementation on the route ` + `(\`model({${name}_id}) {\`) or by injecting an implemention of ` + `a data store: \`@service store;\`.`, Boolean(modelClass)));
         if (!modelClass) {
           return;
         }
         modelClass = modelClass.class;
-        (isDevelopingApp() && !(typeof modelClass.find === 'function') && assert(`You used the dynamic segment \`${name}_id\` in your route ` + `\`${routeName}\` for which Ember requires you provide a ` + `data-loading implementation. Commonly, that is done by ` + `adding a model hook implementation on the route ` + `(\`model({${name}_id}) {\`) or by injecting an implemention of ` + `a data store: \`@service store;\`.\n\n` + `Rarely, applications may attempt to use a legacy behavior where ` + `the model class (in this case \`${name}\`) is resolved and the ` + `\`find\` method on that class is invoked to load data. In this ` + `application, a model of \`${name}\` was found but it did not ` + `provide a \`find\` method. You should not add a \`find\` ` + `method to your model. Instead, please implement an appropriate ` + `\`model\` hook on the \`${routeName}\` route.`, typeof modelClass.find === 'function'));
+        (!(typeof modelClass.find === 'function') && assert(`You used the dynamic segment \`${name}_id\` in your route ` + `\`${routeName}\` for which Ember requires you provide a ` + `data-loading implementation. Commonly, that is done by ` + `adding a model hook implementation on the route ` + `(\`model({${name}_id}) {\`) or by injecting an implemention of ` + `a data store: \`@service store;\`.\n\n` + `Rarely, applications may attempt to use a legacy behavior where ` + `the model class (in this case \`${name}\`) is resolved and the ` + `\`find\` method on that class is invoked to load data. In this ` + `application, a model of \`${name}\` was found but it did not ` + `provide a \`find\` method. You should not add a \`find\` ` + `method to your model. Instead, please implement an appropriate ` + `\`model\` hook on the \`${routeName}\` route.`, typeof modelClass.find === 'function'));
         return modelClass.find(value);
       }
     };
@@ -1147,12 +1146,12 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
     let combinedQueryParameterConfiguration = {};
     let controllerName = this.controllerName || this.routeName;
     let owner = getOwner(this);
-    (isDevelopingApp() && !(owner) && assert('Route is unexpectedly missing an owner', owner));
+    (!(owner) && assert('Route is unexpectedly missing an owner', owner));
     let controller = owner.lookup(`controller:${controllerName}`);
     let queryParameterConfiguraton = get(this, 'queryParams');
     let hasRouterDefinedQueryParams = Object.keys(queryParameterConfiguraton).length > 0;
     if (controller) {
-      (isDevelopingApp() && !(controller instanceof Controller) && assert('Expected an instance of controller', controller instanceof Controller)); // the developer has authored a controller class in their application for
+      (!(controller instanceof Controller) && assert('Expected an instance of controller', controller instanceof Controller)); // the developer has authored a controller class in their application for
       // this route find its query params and normalize their object shape them
       // merge in the query params for the route. As a mergedProperty,
       // Route#queryParams is always at least `{}`
@@ -1180,7 +1179,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
         continue;
       }
       let desc = combinedQueryParameterConfiguration[propName];
-      (isDevelopingApp() && !(desc) && assert(`[BUG] missing query parameter configuration for ${propName}`, desc));
+      (!(desc) && assert(`[BUG] missing query parameter configuration for ${propName}`, desc));
       let scope = desc.scope || 'model';
       let parts = undefined;
       if (scope === 'controller') {
@@ -1225,7 +1224,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
         */
         inactive: (prop, value) => {
           let qp = map[prop];
-          (isDevelopingApp() && !(qp) && assert('expected inactive callback to only be called for registered qps', qp));
+          (!(qp) && assert('expected inactive callback to only be called for registered qps', qp));
           this._qpChanged(prop, value, qp);
         },
         /*
@@ -1235,7 +1234,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
         */
         active: (prop, value) => {
           let qp = map[prop];
-          (isDevelopingApp() && !(qp) && assert('expected active callback to only be called for registered qps', qp));
+          (!(qp) && assert('expected active callback to only be called for registered qps', qp));
           this._qpChanged(prop, value, qp);
           return this._activeQPChanged(qp, value);
         },
@@ -1245,7 +1244,7 @@ class Route extends EmberObject.extend(ActionHandler, Evented) {
         */
         allowOverrides: (prop, value) => {
           let qp = map[prop];
-          (isDevelopingApp() && !(qp) && assert('expected allowOverrides callback to only be called for registered qps', qp));
+          (!(qp) && assert('expected allowOverrides callback to only be called for registered qps', qp));
           this._qpChanged(prop, value, qp);
           return this._updatingQPChanged(qp);
         }
@@ -1303,10 +1302,10 @@ function getRenderState(route) {
 }
 function buildRenderState(route) {
   let owner = getOwner(route);
-  (isDevelopingApp() && !(owner) && assert('Route is unexpectedly missing an owner', owner));
+  (!(owner) && assert('Route is unexpectedly missing an owner', owner));
   let name = route.routeName;
   let controller = owner.lookup(`controller:${route.controllerName || name}`);
-  (isDevelopingApp() && !(controller instanceof Controller) && assert('Expected an instance of controller', controller instanceof Controller));
+  (!(controller instanceof Controller) && assert('Expected an instance of controller', controller instanceof Controller));
   let model = route.currentModel;
   let templateFactoryOrComponent = owner.lookup(`template:${route.templateName || name}`);
 
@@ -1330,14 +1329,14 @@ function buildRenderState(route) {
     if (hasInternalComponentManager(templateFactoryOrComponent)) {
       template = templateFactoryOrComponent;
     } else {
-      if (isDevelopingApp() && typeof templateFactoryOrComponent !== 'function') {
+      if (typeof templateFactoryOrComponent !== 'function') {
         let label;
         try {
           label = `\`${String(templateFactoryOrComponent)}\``;
         } catch {
           label = 'an unknown object';
         }
-        (isDevelopingApp() && true && assert(`Failed to render the ${name} route, expected ` + `\`template:${route.templateName || name}\` to resolve into ` + `a component or a \`TemplateFactory\`, got: ${label}. ` + `Most likely an improperly defined class or an invalid module export.`));
+        (assert(`Failed to render the ${name} route, expected ` + `\`template:${route.templateName || name}\` to resolve into ` + `a component or a \`TemplateFactory\`, got: ${label}. ` + `Most likely an improperly defined class or an invalid module export.`));
       }
       template = templateFactoryOrComponent(owner);
     }
@@ -1352,7 +1351,7 @@ function buildRenderState(route) {
     model,
     template
   };
-  if (isDevelopingApp()) {
+  {
     let LOG_VIEW_LOOKUPS = get(route._router, 'namespace.LOG_VIEW_LOOKUPS');
     // This is covered by tests and the existing code was deliberately
     // targeting the value prior to normalization, but is this message actually
@@ -1495,7 +1494,7 @@ Route.reopen({
   templateName: null,
   controllerName: null,
   send(...args) {
-    (isDevelopingApp() && !(!this.isDestroying && !this.isDestroyed) && assert(`Attempted to call .send() with the action '${args[0]}' on the destroyed route '${this.routeName}'.`, !this.isDestroying && !this.isDestroyed));
+    (!(!this.isDestroying && !this.isDestroyed) && assert(`Attempted to call .send() with the action '${args[0]}' on the destroyed route '${this.routeName}'.`, !this.isDestroying && !this.isDestroyed));
     if (this._router && this._router._routerMicrolib || !isTesting()) {
       this._router.send(...args);
     } else {
@@ -1551,7 +1550,7 @@ Route.reopen({
         let qp = qpMap[change];
         if (qp) {
           let options = this._optionsForQueryParam(qp);
-          (isDevelopingApp() && !(options && typeof options === 'object') && assert('options exists', options && typeof options === 'object'));
+          (!(options && typeof options === 'object') && assert('options exists', options && typeof options === 'object'));
           if (get(options, 'refreshModel') && this._router.currentState) {
             this.refresh();
             break;