ember-source 4.0.0-beta.8 → 4.0.0-beta.9

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Ember Changelog
 
+### v4.0.0-beta.9 (November 20, 2021)
+
+- [#19749](https://github.com/emberjs/ember.js/pull/19749) [CLEANUP] Remove `deprecate-router-events` support code
+- [#19762](https://github.com/emberjs/ember.js/pull/19762) [CLEANUP] Update GlimmerVM to 0.81
+  - removes deprecation of mutations during helper compute 
+  - removes deprecation of mutations during unknownProperty
+  - `@glimmer/integration-tests`, `@glimmer/manager`, `@glimmer/validator`
+    * [#1330](https://github.com/glimmerjs/glimmer-vm/pull/1330) Remove deprecated support for mutation after consumption during certain manager hooks ([@snewcomer](https://github.com/snewcomer))
+  - `@glimmer/manager`
+    * [#1328](https://github.com/glimmerjs/glimmer-vm/pull/1328) Remove deprecated Component Manager version 3.4 ([@nlfurniss](https://github.com/nlfurniss))
+  - `@glimmer/integration-tests`, `@glimmer/manager`
+    * [#1329](https://github.com/glimmerjs/glimmer-vm/pull/1329) Remove deprecated Modifier Manager version 3.13 ([@nlfurniss](https://github.com/nlfurniss))
+- [#19806](https://github.com/emberjs/ember.js/pull/19806) [CLEANUP] Drop export of built-ins, remove legacy components 
+
 ### v4.0.0-beta.8 (November 5, 2021)
 
 - [#19823](https://github.com/emberjs/ember.js/pull/19823) / [#19828](https://github.com/emberjs/ember.js/pull/19828) [BUGFIX] Fix deprecation `until` and link for Component.reopenClass and Component.reopen

package/build-metadata.json

@@ -1,6 +1,6 @@
 {
-  "version": "4.0.0-beta.8",
+  "version": "4.0.0-beta.9",
   "buildType": "tag",
-  "SHA": "78f8fbf3bebc724e39dd0d6806f28c8021764866",
-  "assetPath": "/tag/shas/78f8fbf3bebc724e39dd0d6806f28c8021764866.tgz"
+  "SHA": "c4be3a86a7c03860bde7f4928ff6fc54a3216e24",
+  "assetPath": "/tag/shas/c4be3a86a7c03860bde7f4928ff6fc54a3216e24.tgz"
 }

package/dist/dependencies/@glimmer/manager.js

@@ -1,9 +1,8 @@
 import { DEBUG } from '@glimmer/env';
-import { debugToString, _WeakSet, HAS_NATIVE_PROXY, assign, dict } from '@glimmer/util';
+import { debugToString, _WeakSet, HAS_NATIVE_PROXY } from '@glimmer/util';
 import { valueForRef, createConstRef, createComputeRef, UNDEFINED_REFERENCE } from '@glimmer/reference';
-import { track, deprecateMutationsInTrackingTransaction, createUpdatableTag, untrack } from '@glimmer/validator';
+import { track, createUpdatableTag, untrack } from '@glimmer/validator';
 import { registerDestructor, associateDestroyableChild } from '@glimmer/destroyable';
-import { setOwner } from '@glimmer/owner';
 
 const COMPONENT_MANAGERS = new WeakMap();
 const MODIFIER_MANAGERS = new WeakMap();
@@ -382,16 +381,11 @@ const CAPABILITIES = {
   hasSubOwner: false
 };
 function componentCapabilities(managerAPI, options = {}) {
-  if (DEBUG && managerAPI !== '3.4' && managerAPI !== '3.13') {
+  if (DEBUG && managerAPI !== '3.13') {
     throw new Error('Invalid component manager compatibility specified');
   }
 
-  let updateHook = true;
-
-  if (managerAPI === '3.13') {
-    updateHook = Boolean(options.updateHook);
-  }
-
+  let updateHook = Boolean(options.updateHook);
   return buildCapabilities({
     asyncLifeCycleCallbacks: Boolean(options.asyncLifecycleCallbacks),
     destructor: Boolean(options.destructor),
@@ -456,7 +450,7 @@ class CustomComponentManager {
 
       if (DEBUG && !FROM_CAPABILITIES.has(delegate.capabilities)) {
         // TODO: This error message should make sense in both Ember and Glimmer https://github.com/glimmerjs/glimmer-vm/issues/1200
-        throw new Error(`Custom component managers must have a \`capabilities\` property that is the result of calling the \`capabilities('3.4' | '3.13')\` (imported via \`import { capabilities } from '@ember/component';\`). Received: \`${JSON.stringify(delegate.capabilities)}\` for: \`${delegate}\``);
+        throw new Error(`Custom component managers must have a \`capabilities\` property that is the result of calling the \`capabilities('3.13')\` (imported via \`import { capabilities } from '@ember/component';\`). Received: \`${JSON.stringify(delegate.capabilities)}\` for: \`${delegate}\``);
       }
 
       componentManagerDelegates.set(owner, delegate);
@@ -468,16 +462,7 @@ class CustomComponentManager {
   create(owner, definition, vmArgs) {
     let delegate = this.getDelegateFor(owner);
     let args = argsProxyFor(vmArgs.capture(), 'component');
-    let component;
-
-    if (DEBUG && deprecateMutationsInTrackingTransaction !== undefined) {
-      deprecateMutationsInTrackingTransaction(() => {
-        component = delegate.createComponent(definition, args);
-      });
-    } else {
-      component = delegate.createComponent(definition, args);
-    }
-
+    let component = delegate.createComponent(definition, args);
     return new CustomComponentState(component, delegate, args);
   }
 
@@ -563,16 +548,12 @@ class CustomComponentState {
 }
 
 function modifierCapabilities(managerAPI, optionalFeatures = {}) {
-  if (DEBUG && managerAPI !== '3.13' && managerAPI !== '3.22') {
+  if (DEBUG && managerAPI !== '3.22') {
     throw new Error('Invalid modifier manager compatibility specified');
   }
 
   return buildCapabilities({
-    disableAutoTracking: Boolean(optionalFeatures.disableAutoTracking),
-    useArgsProxy: managerAPI === '3.13' ? false : true,
-    // This capability is used in Ember, exclusively in resolution mode. See the
-    // Ember glimmer resolver for details.
-    passFactoryToCreate: managerAPI === '3.13'
+    disableAutoTracking: Boolean(optionalFeatures.disableAutoTracking)
   });
 }
 /**
@@ -620,7 +601,7 @@ class CustomModifierManager {
 
       if (DEBUG && !FROM_CAPABILITIES.has(delegate.capabilities)) {
         // TODO: This error message should make sense in both Ember and Glimmer https://github.com/glimmerjs/glimmer-vm/issues/1200
-        throw new Error(`Custom modifier managers must have a \`capabilities\` property that is the result of calling the \`capabilities('3.13' | '3.22')\` (imported via \`import { capabilities } from '@ember/modifier';\`). Received: \`${JSON.stringify(delegate.capabilities)}\` for: \`${delegate}\``);
+        throw new Error(`Custom modifier managers must have a \`capabilities\` property that is the result of calling the \`capabilities('3.22')\` (imported via \`import { capabilities } from '@ember/modifier';\`). Received: \`${JSON.stringify(delegate.capabilities)}\` for: \`${delegate}\``);
       }
 
       componentManagerDelegates.set(owner, delegate);
@@ -631,67 +612,23 @@ class CustomModifierManager {
 
   create(owner, element, definition, capturedArgs) {
     let delegate = this.getDelegateFor(owner);
-    let {
-      useArgsProxy,
-      passFactoryToCreate
-    } = delegate.capabilities;
-    let argsProxy = argsProxyFor(capturedArgs, 'modifier');
-    let args = useArgsProxy ? argsProxy : reifyArgs(capturedArgs);
-    let instance;
-    let factoryOrDefinition = definition;
-
-    if (passFactoryToCreate) {
-      // Make a fake factory. While not perfect, this should generally prevent
-      // breakage in users of older modifier capabilities.
-      factoryOrDefinition = {
-        create(args) {
-          let params = assign({}, args);
-          setOwner(params, owner);
-          return definition.create(args);
-        },
-
-        class: definition
-      };
-    }
-
-    if (DEBUG && deprecateMutationsInTrackingTransaction !== undefined) {
-      deprecateMutationsInTrackingTransaction(() => {
-        instance = delegate.createModifier(factoryOrDefinition, args);
-      });
-    } else {
-      instance = delegate.createModifier(factoryOrDefinition, args);
-    }
-
+    let args = argsProxyFor(capturedArgs, 'modifier');
+    let instance = delegate.createModifier(definition, args);
     let tag = createUpdatableTag();
     let state;
-
-    if (useArgsProxy) {
-      state = {
-        tag,
-        element,
-        delegate,
-        args,
-        modifier: instance
-      };
-    } else {
-      state = {
-        tag,
-        element,
-        modifier: instance,
-        delegate,
-
-        get args() {
-          return reifyArgs(capturedArgs);
-        }
-
-      };
-    }
+    state = {
+      tag,
+      element,
+      delegate,
+      args,
+      modifier: instance
+    };
 
     if (DEBUG) {
       state.debugName = typeof definition === 'function' ? definition.name : definition.toString();
     }
 
-    registerDestructor(state, () => delegate.destroyModifier(instance, argsProxy));
+    registerDestructor(state, () => delegate.destroyModifier(instance, args));
     return state;
   }
 
@@ -745,22 +682,6 @@ class CustomModifierManager {
   }
 
 }
-function reifyArgs({
-  named,
-  positional
-}) {
-  let reifiedNamed = dict();
-
-  for (let key in named) {
-    reifiedNamed[key] = valueForRef(named[key]);
-  }
-
-  let reifiedPositional = positional.map(valueForRef);
-  return {
-    named: reifiedNamed,
-    positional: reifiedPositional
-  };
-}
 
 function helperCapabilities(managerAPI, options = {}) {
   if (DEBUG && managerAPI !== '3.23') {

package/dist/dependencies/@glimmer/opcode-compiler.js

@@ -1,4 +1,4 @@
-import { dict, assign, isSmallInt, debugToString, EMPTY_ARRAY, EMPTY_STRING_ARRAY, encodeImmediate, Stack, encodeHandle, unwrapTemplate } from '@glimmer/util';
+import { dict, assign, isSmallInt, debugToString, EMPTY_ARRAY, EMPTY_STRING_ARRAY, encodeImmediate, Stack, encodeHandle } from '@glimmer/util';
 import { DEBUG } from '@glimmer/env';
 import { $v0, $fp, $s0, $s1, $sp, isMachineOp } from '@glimmer/vm';
 import { assert, deprecate } from '@glimmer/global-context';
@@ -353,8 +353,7 @@ function resolveComponentOrHelper(resolver, constants, meta, [, expr, {
  */
 
 function resolveOptionalHelper(resolver, constants, meta, [, expr, {
-  ifHelper,
-  ifFallback
+  ifHelper
 }]) {
   let {
     upvars,
@@ -363,9 +362,7 @@ function resolveOptionalHelper(resolver, constants, meta, [, expr, {
   let name = upvars[expr[1]];
   let helper = resolver.lookupHelper(name, owner);
 
-  if (helper === null) {
-    ifFallback(name, meta.moduleName);
-  } else {
+  if (helper) {
     ifHelper(constants.helper(helper, name), name, meta.moduleName);
   }
 }
@@ -376,8 +373,7 @@ function resolveOptionalHelper(resolver, constants, meta, [, expr, {
 function resolveOptionalComponentOrHelper(resolver, constants, meta, [, expr, {
   ifComponent,
   ifHelper,
-  ifValue,
-  ifFallback
+  ifValue
 }]) {
   let type = expr[0];
 
@@ -432,10 +428,7 @@ function resolveOptionalComponentOrHelper(resolver, constants, meta, [, expr, {
 
     if (helper !== null) {
       ifHelper(constants.helper(helper, name));
-      return;
     }
-
-    ifFallback(name);
   }
 }
 
@@ -533,28 +526,6 @@ EXPRESSIONS.add(31
   , sym, _handle => {// TODO: Implement in strict mode
   });
 });
-EXPRESSIONS.add(33
-/* GetFreeAsFallback */
-, (op, [, freeVar, path]) => {
-  op(1010
-  /* ResolveLocal */
-  , freeVar, (name, moduleName) => {
-    if (DEBUG) {
-      let propertyPath = path ? [name, ...path].join('.') : name;
-      deprecate(`The \`${propertyPath}\` property path was used in the \`${moduleName}\` template without using \`this\`. This fallback behavior has been deprecated, all properties must be looked up on \`this\` when used in the template: {{this.${propertyPath}}}`, false, {
-        id: 'this-property-fallback'
-      });
-    }
-
-    op(21
-    /* GetVariable */
-    , 0);
-    op(22
-    /* GetProperty */
-    , name);
-  });
-  withPath(op, path);
-});
 EXPRESSIONS.add(34
 /* GetFreeAsComponentOrHelperHeadOrThisFallback */
 , () => {
@@ -576,17 +547,6 @@ EXPRESSIONS.add(36
     , expr$$1, {
       ifHelper: handle => {
         Call(op, handle, null, null);
-      },
-      ifFallback: (name, moduleName) => {
-        deprecate(`The \`${name}\` property was used in the \`${moduleName}\` template without using \`this\`. This fallback behavior has been deprecated, all properties must be looked up on \`this\` when used in the template: {{this.${name}}}`, false, {
-          id: 'this-property-fallback'
-        });
-        op(21
-        /* GetVariable */
-        , 0);
-        op(22
-        /* GetProperty */
-        , name);
       }
     });
   });
@@ -608,17 +568,6 @@ EXPRESSIONS.add(99
           id: 'argument-less-helper-paren-less-invocation'
         });
         Call(op, handle, null, null);
-      },
-      ifFallback: (name, moduleName) => {
-        deprecate(`The \`${name}\` property was used in the \`${moduleName}\` template without using \`this\`. This fallback behavior has been deprecated, all properties must be looked up on \`this\` when used in the template: {{this.${name}}}`, false, {
-          id: 'this-property-fallback'
-        });
-        op(21
-        /* GetVariable */
-        , 0);
-        op(22
-        /* GetProperty */
-        , name);
       }
     });
   });
@@ -801,7 +750,6 @@ function meta(layout) {
 
   let [, symbols,, upvars] = layout.block;
   return {
-    asPartial: layout.asPartial || false,
     evalSymbols: evalSymbols(layout),
     upvars: upvars,
     scopeValues: (_b = (_a = layout.scope) === null || _a === void 0 ? void 0 : _a.call(layout)) !== null && _b !== void 0 ? _b : null,
@@ -2038,27 +1986,6 @@ STATEMENTS.add(8
     InvokeDynamicComponent(op, expr$$1, elementBlock, null, named, blocks, true, true);
   }
 });
-STATEMENTS.add(19
-/* Partial */
-, (op, [, name, evalInfo]) => {
-  ReplayableIf(op, () => {
-    expr(op, name);
-    op(33
-    /* Dup */
-    , $sp, 0);
-    return 2;
-  }, () => {
-    op(101
-    /* InvokePartial */
-    , evalSymbolsOperand(), evalInfo);
-    op(40
-    /* PopScope */
-    );
-    op(1
-    /* PopFrame */
-    );
-  });
-});
 STATEMENTS.add(18
 /* Yield */
 , (op, [, to, params]) => YieldBlock(op, to, params));
@@ -2112,31 +2039,6 @@ STATEMENTS.add(1
         op(1
         /* PopFrame */
         );
-      },
-
-      ifFallback(_name) {
-        op(0
-        /* PushFrame */
-        );
-        op(1010
-        /* ResolveLocal */
-        , value[1], (name, moduleName) => {
-          deprecate(`The \`${name}\` property was used in the \`${moduleName}\` template without using \`this\`. This fallback behavior has been deprecated, all properties must be looked up on \`this\` when used in the template: {{this.${name}}}`, false, {
-            id: 'this-property-fallback'
-          });
-          op(21
-          /* GetVariable */
-          , 0);
-          op(22
-          /* GetProperty */
-          , name);
-        });
-        op(3
-        /* InvokeStatic */
-        , stdlibOperand('cautious-append'));
-        op(1
-        /* PopFrame */
-        );
       }
 
     });
@@ -2567,16 +2469,8 @@ function encodeOp(encoder, constants, resolver, meta, op) {
       :
         let freeVar = op[1];
         let name = meta.upvars[freeVar];
-
-        if (meta.asPartial === true) {
-          encoder.push(constants, 102
-          /* ResolveMaybeLocal */
-          , name);
-        } else {
-          let then = op[2];
-          then(name, meta.moduleName);
-        }
-
+        let andThen = op[2];
+        andThen(name, meta.moduleName);
         break;
 
       case 1011
@@ -2860,7 +2754,6 @@ function compileStd(context) {
   return new StdLib(mainHandle, trustingGuardedDynamicAppend, cautiousGuardedDynamicAppend, trustingGuardedNonDynamicAppend, cautiousGuardedNonDynamicAppend);
 }
 const STDLIB_META = {
-  asPartial: false,
   evalSymbols: null,
   upvars: null,
   moduleName: 'stdlib',
@@ -2938,24 +2831,6 @@ const MINIMAL_CAPABILITIES = {
   hasSubOwner: false
 };
 
-class PartialDefinitionImpl {
-  constructor(name, // for debugging
-  template) {
-    this.name = name;
-    this.template = template;
-  }
-
-  getPartial(context) {
-    let partial = unwrapTemplate(this.template).asPartial();
-    let handle = partial.compile(context);
-    return {
-      symbolTable: partial.symbolTable,
-      handle
-    };
-  }
-
-}
-
 class WrappedBuilder {
   constructor(layout, moduleName) {
     this.layout = layout;
@@ -3094,7 +2969,6 @@ class TemplateImpl {
     this.parsedLayout = parsedLayout;
     this.result = 'ok';
     this.layout = null;
-    this.partial = null;
     this.wrappedLayout = null;
   }
 
@@ -3117,25 +2991,14 @@ class TemplateImpl {
 
   asLayout() {
     if (this.layout) return this.layout;
-    return this.layout = compilable(assign({}, this.parsedLayout, {
-      asPartial: false
-    }), this.moduleName);
-  }
-
-  asPartial() {
-    if (this.partial) return this.partial;
-    return this.partial = compilable(assign({}, this.parsedLayout, {
-      asPartial: true
-    }), this.moduleName);
+    return this.layout = compilable(assign({}, this.parsedLayout), this.moduleName);
   }
 
   asWrappedLayout() {
     if (this.wrappedLayout) return this.wrappedLayout;
-    return this.wrappedLayout = new WrappedBuilder(assign({}, this.parsedLayout, {
-      asPartial: false
-    }), this.moduleName);
+    return this.wrappedLayout = new WrappedBuilder(assign({}, this.parsedLayout), this.moduleName);
   }
 
 }
 
-export { debugCompiler, compileStatements, compilable, InvokeStaticBlockWithStack as invokeStaticBlockWithStack, InvokeStaticBlock as invokeStaticBlock, compileStd, meta, StdLib, PartialDefinitionImpl, templateFactory, templateCacheCounters, WrappedBuilder, EMPTY_BLOCKS, CompileTimeCompilationContextImpl, programCompilationContext, templateCompilationContext, DEFAULT_CAPABILITIES, MINIMAL_CAPABILITIES };
+export { debugCompiler, compileStatements, compilable, InvokeStaticBlockWithStack as invokeStaticBlockWithStack, InvokeStaticBlock as invokeStaticBlock, compileStd, meta, StdLib, templateFactory, templateCacheCounters, WrappedBuilder, EMPTY_BLOCKS, CompileTimeCompilationContextImpl, programCompilationContext, templateCompilationContext, DEFAULT_CAPABILITIES, MINIMAL_CAPABILITIES };

package/dist/dependencies/@glimmer/runtime.js

@@ -1,4 +1,4 @@
-import { assign, symbol, Stack, fillNulls, _WeakSet, isObject, dict, emptyArray, EMPTY_STRING_ARRAY, debugToString, decodeHandle, decodeImmediate, isHandle, unwrapTemplate, unwrapHandle, clearElement, buildUntouchableThis, HAS_NATIVE_PROXY, isDict } from '@glimmer/util';
+import { assign, symbol, Stack, fillNulls, _WeakSet, isObject, dict, emptyArray, EMPTY_STRING_ARRAY, debugToString, decodeHandle, decodeImmediate, isHandle, unwrapTemplate, clearElement, unwrapHandle, buildUntouchableThis, HAS_NATIVE_PROXY, isDict } from '@glimmer/util';
 import { UNDEFINED_REFERENCE, valueForRef, createComputeRef, createDebugAliasRef, childRefFor, TRUE_REFERENCE, FALSE_REFERENCE, isConstRef, createPrimitiveRef, NULL_REFERENCE, createConstRef, createIteratorRef, updateRef, createIteratorItemRef, isInvokableRef } from '@glimmer/reference';
 import { warnIfStyleNotTrusted, toBool, assertGlobalContextWasSet, deprecate, getPath, setPath } from '@glimmer/global-context';
 import { DEBUG } from '@glimmer/env';
@@ -3903,60 +3903,6 @@ APPEND_OPCODES.add(103
   callback(valueForRef(vm.getSelf()), path => valueForRef(inspector.get(path)));
 });
 
-APPEND_OPCODES.add(101
-/* InvokePartial */
-, (vm, {
-  op1: _symbols,
-  op2: _evalInfo
-}) => {
-  let {
-    [CONSTANTS]: constants,
-    stack
-  } = vm;
-  let name = valueForRef(stack.pop());
-  let outerScope = vm.scope();
-  let owner = outerScope.owner;
-  let outerSymbols = constants.getArray(_symbols);
-  let evalInfo = constants.getArray(decodeHandle(_evalInfo));
-  let definition = vm.runtime.resolver.lookupPartial(name, owner);
-  let {
-    symbolTable,
-    handle: vmHandle
-  } = definition.getPartial(vm.context);
-  {
-    let partialSymbols = symbolTable.symbols;
-    let partialScope = vm.pushRootScope(partialSymbols.length, owner);
-    let evalScope = outerScope.getEvalScope();
-    partialScope.bindEvalScope(evalScope);
-    partialScope.bindSelf(outerScope.getSelf());
-    let locals = Object.create(outerScope.getPartialMap());
-
-    for (let i = 0; i < evalInfo.length; i++) {
-      let slot = evalInfo[i];
-
-      if (slot !== -1) {
-        let name = outerSymbols[slot - 1];
-        let ref = outerScope.getSymbol(slot);
-        locals[name] = ref;
-      }
-    }
-
-    if (evalScope) {
-      for (let i = 0; i < partialSymbols.length; i++) {
-        let name = partialSymbols[i];
-        let symbol$$1 = i + 1;
-        let value = evalScope[name];
-        if (value !== undefined) partialScope.bind(symbol$$1, value);
-      }
-    }
-
-    partialScope.bindPartialMap(locals);
-    vm.pushFrame(); // sp += 2
-
-    vm.call(unwrapHandle(vmHandle));
-  }
-});
-
 APPEND_OPCODES.add(72
 /* EnterList */
 , (vm, {

package/dist/dependencies/@glimmer/validator.js

@@ -1,5 +1,5 @@
 import { DEBUG } from '@glimmer/env';
-import { deprecate, assert, scheduleRevalidate } from '@glimmer/global-context';
+import { assert, scheduleRevalidate } from '@glimmer/global-context';
 
 // eslint-disable-next-line @typescript-eslint/ban-types
 function indexable(input) {
@@ -28,7 +28,6 @@ function unwrap(val) {
 let beginTrackingTransaction;
 let endTrackingTransaction;
 let runInTrackingTransaction;
-let deprecateMutationsInTrackingTransaction;
 let resetTrackingTransaction;
 let setTrackingTransactionEnv;
 let assertTagNotConsumed;
@@ -62,14 +61,13 @@ if (DEBUG) {
 
   setTrackingTransactionEnv = env => Object.assign(TRANSACTION_ENV, env);
 
-  beginTrackingTransaction = (_debugLabel, deprecate$$1 = false) => {
+  beginTrackingTransaction = _debugLabel => {
     CONSUMED_TAGS = CONSUMED_TAGS || new WeakMap();
     let debugLabel = _debugLabel || undefined;
     let parent = TRANSACTION_STACK[TRANSACTION_STACK.length - 1] || null;
     TRANSACTION_STACK.push({
       parent,
-      debugLabel,
-      deprecate: deprecate$$1
+      debugLabel
     });
   };
 
@@ -122,28 +120,6 @@ if (DEBUG) {
       }
     }
   };
-  /**
-   * Switches to deprecating within an autotracking transaction, if one exists.
-   * If `runInAutotrackingTransaction` is called within the callback of this
-   * method, it switches back to throwing an error, allowing zebra-striping of
-   * the types of errors that are thrown.
-   *
-   * Does not start an autotracking transaction.
-   *
-   * NOTE: For Ember usage only, in general you should assert that these
-   * invariants are true.
-   */
-
-
-  deprecateMutationsInTrackingTransaction = (fn, debugLabel) => {
-    beginTrackingTransaction(debugLabel, true);
-
-    try {
-      fn();
-    } finally {
-      endTrackingTransaction();
-    }
-  };
 
   let nthIndex = (str, pattern, n, startingPos = -1) => {
     let i = startingPos;
@@ -201,32 +177,24 @@ if (DEBUG) {
   assertTagNotConsumed = (tag, obj, keyName) => {
     if (CONSUMED_TAGS === null) return;
     let transaction = CONSUMED_TAGS.get(tag);
-    if (!transaction) return;
-    let currentTransaction = TRANSACTION_STACK[TRANSACTION_STACK.length - 1];
+    if (!transaction) return; // This hack makes the assertion message nicer, we can cut off the first
+    // few lines of the stack trace and let users know where the actual error
+    // occurred.
 
-    if (currentTransaction.deprecate) {
-      deprecate(makeTrackingErrorMessage(transaction, obj, keyName), false, {
-        id: 'autotracking.mutation-after-consumption'
-      });
-    } else {
-      // This hack makes the assertion message nicer, we can cut off the first
-      // few lines of the stack trace and let users know where the actual error
-      // occurred.
-      try {
-        assert(false, makeTrackingErrorMessage(transaction, obj, keyName));
-      } catch (e) {
-        if (e.stack) {
-          let updateStackBegin = e.stack.indexOf('Stack trace for the update:');
-
-          if (updateStackBegin !== -1) {
-            let start = nthIndex(e.stack, '\n', 1, updateStackBegin);
-            let end = nthIndex(e.stack, '\n', 4, updateStackBegin);
-            e.stack = e.stack.substr(0, start) + e.stack.substr(end);
-          }
+    try {
+      assert(false, makeTrackingErrorMessage(transaction, obj, keyName));
+    } catch (e) {
+      if (e.stack) {
+        let updateStackBegin = e.stack.indexOf('Stack trace for the update:');
+
+        if (updateStackBegin !== -1) {
+          let start = nthIndex(e.stack, '\n', 1, updateStackBegin);
+          let end = nthIndex(e.stack, '\n', 4, updateStackBegin);
+          e.stack = e.stack.substr(0, start) + e.stack.substr(end);
         }
-
-        throw e;
       }
+
+      throw e;
     }
   };
 }
@@ -765,4 +733,4 @@ if (globalObj[GLIMMER_VALIDATOR_REGISTRATION] === true) {
 
 globalObj[GLIMMER_VALIDATOR_REGISTRATION] = true;
 
-export { ALLOW_CYCLES, bump, combine, COMPUTE, CONSTANT_TAG, CONSTANT, createTag, createUpdatableTag, CurrentTag, CURRENT_TAG, DIRTY_TAG as dirtyTag, INITIAL, isConstTag, UPDATE_TAG as updateTag, validateTag, valueForTag, VolatileTag, VOLATILE_TAG, VOLATILE, dirtyTagFor, tagFor, tagMetaFor, beginTrackFrame, endTrackFrame, beginUntrackFrame, endUntrackFrame, resetTracking, consumeTag, isTracking, track, untrack, createCache, isConst, getValue, trackedData, logTrackingStack, setTrackingTransactionEnv, runInTrackingTransaction, beginTrackingTransaction, endTrackingTransaction, deprecateMutationsInTrackingTransaction };
+export { ALLOW_CYCLES, bump, combine, COMPUTE, CONSTANT_TAG, CONSTANT, createTag, createUpdatableTag, CurrentTag, CURRENT_TAG, DIRTY_TAG as dirtyTag, INITIAL, isConstTag, UPDATE_TAG as updateTag, validateTag, valueForTag, VolatileTag, VOLATILE_TAG, VOLATILE, dirtyTagFor, tagFor, tagMetaFor, beginTrackFrame, endTrackFrame, beginUntrackFrame, endUntrackFrame, resetTracking, consumeTag, isTracking, track, untrack, createCache, isConst, getValue, trackedData, logTrackingStack, setTrackingTransactionEnv, runInTrackingTransaction, beginTrackingTransaction, endTrackingTransaction };