Update Glimmer VM to v0.42.0

packages/@glimmer/app-compiler/package.json

@@ -13,9 +13,9 @@
   "dependencies": {
     "@glimmer/application": "^0.14.0-alpha.13",
     "@glimmer/compiler-delegates": "^0.14.0-alpha.13",
-    "@glimmer/bundle-compiler": "^0.41.0",
-    "@glimmer/interfaces": "^0.41.0",
-    "@glimmer/opcode-compiler": "^0.41.0",
+    "@glimmer/bundle-compiler": "^0.42.0",
+    "@glimmer/interfaces": "^0.42.0",
+    "@glimmer/opcode-compiler": "^0.42.0",
     "broccoli-plugin": "^1.3.0",
     "debug": "^3.1.0",
     "walk-sync": "^0.3.2"

packages/@glimmer/application-test-helpers/package.json

@@ -15,14 +15,14 @@
   "module": "dist/modules/es2017/index.js",
   "types": "dist/types/index.d.ts",
   "dependencies": {
-    "@glimmer/compiler": "^0.41.0",
+    "@glimmer/compiler": "^0.42.0",
     "@glimmer/di": "^0.1.9",
     "@glimmer/env": "^0.1.7",
-    "@glimmer/interfaces": "^0.41.0",
-    "@glimmer/reference": "^0.41.0",
+    "@glimmer/interfaces": "^0.42.0",
+    "@glimmer/reference": "^0.42.0",
     "@glimmer/resolver": "^0.3.0",
-    "@glimmer/runtime": "^0.41.0",
-    "@glimmer/util": "^0.41.0",
-    "@glimmer/wire-format": "^0.41.0"
+    "@glimmer/runtime": "^0.42.0",
+    "@glimmer/util": "^0.42.0",
+    "@glimmer/wire-format": "^0.42.0"
   }
 }

packages/@glimmer/application/package.json

@@ -15,23 +15,23 @@
     "dist"
   ],
   "dependencies": {
-    "@glimmer/compiler": "^0.41.0",
+    "@glimmer/compiler": "^0.42.0",
     "@glimmer/component": "^0.14.0-alpha.13",
     "@glimmer/di": "^0.1.9",
     "@glimmer/env": "^0.1.7",
-    "@glimmer/interfaces": "^0.41.0",
-    "@glimmer/opcode-compiler": "^0.41.0",
-    "@glimmer/reference": "^0.41.0",
+    "@glimmer/interfaces": "^0.42.0",
+    "@glimmer/opcode-compiler": "^0.42.0",
+    "@glimmer/reference": "^0.42.0",
     "@glimmer/resolver": "^0.3.0",
-    "@glimmer/runtime": "^0.41.0",
-    "@glimmer/util": "^0.41.0",
+    "@glimmer/runtime": "^0.42.0",
+    "@glimmer/util": "^0.42.0",
     "@simple-dom/interface": "^1.4.0"
   },
   "devDependencies": {
     "@glimmer/application-test-helpers": "^0.14.0-alpha.13",
-    "@glimmer/bundle-compiler": "^0.41.0",
+    "@glimmer/bundle-compiler": "^0.42.0",
     "@glimmer/component": "^0.14.0-alpha.13",
-    "@glimmer/wire-format": "^0.41.0",
+    "@glimmer/wire-format": "^0.42.0",
     "@simple-dom/document": "^1.4.0",
     "@simple-dom/serializer": "^1.4.0",
     "@simple-dom/void-map": "^1.4.0"

packages/@glimmer/application/src/helpers/user-helper.ts

@@ -1,4 +1,4 @@
-import { TagWrapper, RevisionTag } from '@glimmer/reference';
+import { Tag } from '@glimmer/reference';
 
 import { Dict, VMArguments, CapturedArguments, Helper as GlimmerHelper } from '@glimmer/interfaces';
 
@@ -11,7 +11,7 @@ export default function buildUserHelper(helperFunc: UserHelper): GlimmerHelper {
 }
 
 export class HelperReference extends CachedReference<unknown> {
-  public tag: TagWrapper<RevisionTag | null>;
+  public tag: Tag;
   private args: CapturedArguments;
 
   constructor(private helper: UserHelper, args: VMArguments) {

packages/@glimmer/compiler-delegates/package.json

@@ -12,13 +12,13 @@
   "types": "dist/types/index.d.ts",
   "dependencies": {
     "@glimmer/application": "^0.14.0-alpha.13",
-    "@glimmer/bundle-compiler": "^0.41.0",
+    "@glimmer/bundle-compiler": "^0.42.0",
     "@glimmer/component": "^0.14.0-alpha.13",
-    "@glimmer/interfaces": "^0.41.0",
-    "@glimmer/opcode-compiler": "^0.41.0",
-    "@glimmer/program": "^0.41.0",
-    "@glimmer/syntax": "^0.41.0",
-    "@glimmer/util": "^0.41.0",
+    "@glimmer/interfaces": "^0.42.0",
+    "@glimmer/opcode-compiler": "^0.42.0",
+    "@glimmer/program": "^0.42.0",
+    "@glimmer/syntax": "^0.42.0",
+    "@glimmer/util": "^0.42.0",
     "@types/node": "^8.0.46",
     "debug": "^3.1.0",
     "glimmer-analyzer": "^0.3.3",

packages/@glimmer/component/package.json

@@ -30,10 +30,10 @@
   "dependencies": {
     "@glimmer/di": "^0.1.9",
     "@glimmer/env": "^0.1.7",
-    "@glimmer/reference": "^0.41.0",
-    "@glimmer/runtime": "^0.41.0",
+    "@glimmer/reference": "^0.42.0",
+    "@glimmer/runtime": "^0.42.0",
     "@glimmer/tracking": "^0.14.0-alpha.13",
-    "@glimmer/util": "^0.41.0",
+    "@glimmer/util": "^0.42.0",
     "broccoli-file-creator": "^2.1.1",
     "broccoli-merge-trees": "^3.0.2",
     "ember-cli-babel": "^7.7.3",
@@ -48,10 +48,10 @@
   "devDependencies": {
     "@ember/optional-features": "^0.6.1",
     "@glimmer/application-test-helpers": "^0.14.0-alpha.13",
-    "@glimmer/compiler": "^0.41.0",
-    "@glimmer/interfaces": "^0.41.0",
+    "@glimmer/compiler": "^0.42.0",
+    "@glimmer/interfaces": "^0.42.0",
     "@glimmer/resolver": "^0.3.0",
-    "@glimmer/wire-format": "^0.41.0",
+    "@glimmer/wire-format": "^0.42.0",
     "@types/ember": "~3.0.29",
     "@types/ember-qunit": "~3.4.3",
     "@types/ember-test-helpers": "~1.0.4",

packages/@glimmer/component/src/component.ts

@@ -1,5 +1,5 @@
 import { metaFor, trackedGet } from '@glimmer/tracking';
-import { CURRENT_TAG } from '@glimmer/reference';
+import { CURRENT_TAG, update } from '@glimmer/reference';
 
 import GlimmerComponent from '../addon/-private/component';
 import { assert } from '@glimmer/util';
@@ -17,9 +17,9 @@ export default class Component<T extends object = object> extends GlimmerCompone
 
   set args(args) {
     this.__args__ = args;
-    metaFor(this)
-      .updatableTagFor('args')
-      .inner.update(CURRENT_TAG);
+    update(metaFor(this)
+      .updatableTagFor('args'),
+      CURRENT_TAG);
   }
 
   /** @private

packages/@glimmer/component/src/references.ts

@@ -1,14 +1,18 @@
 import { dict } from '@glimmer/util';
 import {
   PathReference,
-  CONSTANT_TAG,
   ConstReference,
   DirtyableTag,
   UpdatableTag,
   combine,
   isConst,
   Tag,
-  TagWrapper,
+  validate,
+  value,
+  createUpdatableTag,
+  update,
+  createTag,
+  dirty
 } from '@glimmer/reference';
 import {
   ConditionalReference as GlimmerConditionalReference,
@@ -37,9 +41,9 @@ export abstract class CachedReference<T> extends ComponentPathReference<T> {
   value() {
     let { tag, _lastRevision, _lastValue } = this;
 
-    if (!_lastRevision || !tag.validate(_lastRevision)) {
+    if (!_lastRevision || !validate(tag, _lastRevision)) {
       _lastValue = this._lastValue = this.compute();
-      this._lastRevision = tag.value();
+      this._lastRevision = value(tag);
     }
 
     return _lastValue;
@@ -95,14 +99,14 @@ export class RootPropertyReference extends PropertyReference {
 export class NestedPropertyReference extends PropertyReference {
   public tag: Tag;
   private _parentReference: PathReference<any>;
-  private _parentObjectTag: TagWrapper<UpdatableTag>;
+  private _parentObjectTag: UpdatableTag;
   private _propertyKey: string;
 
   constructor(parentReference: PathReference<any>, propertyKey: string) {
     super();
 
     let parentReferenceTag = parentReference.tag;
-    let parentObjectTag = UpdatableTag.create(CONSTANT_TAG);
+    let parentObjectTag = createUpdatableTag();
 
     this._parentReference = parentReference;
     this._parentObjectTag = parentObjectTag;
@@ -116,7 +120,7 @@ export class NestedPropertyReference extends PropertyReference {
 
     let parentValue = _parentReference.value();
 
-    _parentObjectTag.inner.update(tagForProperty(parentValue, _propertyKey));
+    update(_parentObjectTag, tagForProperty(parentValue, _propertyKey));
 
     if (typeof parentValue === 'string' && _propertyKey === 'length') {
       return parentValue.length;
@@ -131,13 +135,13 @@ export class NestedPropertyReference extends PropertyReference {
 }
 
 export class UpdatableReference<T> extends ComponentPathReference<T> {
-  public tag: TagWrapper<DirtyableTag>;
+  public tag: DirtyableTag;
   private _value: T;
 
   constructor(value: T) {
     super();
 
-    this.tag = DirtyableTag.create();
+    this.tag = createTag();
     this._value = value;
   }
 
@@ -149,7 +153,7 @@ export class UpdatableReference<T> extends ComponentPathReference<T> {
     let { _value } = this;
 
     if (value !== _value) {
-      this.tag.inner.dirty();
+      dirty(this.tag);
       this._value = value;
     }
   }

packages/@glimmer/ssr/package.json

@@ -14,10 +14,10 @@
     "@glimmer/application": "^0.14.0-alpha.13",
     "@glimmer/component": "^0.14.0-alpha.13",
     "@glimmer/di": "^0.1.9",
-    "@glimmer/node": "^0.41.0",
-    "@glimmer/reference": "^0.41.0",
-    "@glimmer/runtime": "^0.41.0",
-    "@glimmer/util": "^0.41.0",
+    "@glimmer/node": "^0.42.0",
+    "@glimmer/reference": "^0.42.0",
+    "@glimmer/runtime": "^0.42.0",
+    "@glimmer/util": "^0.42.0",
     "@simple-dom/document": "^1.4.0",
     "@simple-dom/serializer": "^1.4.0",
     "@simple-dom/void-map": "^1.4.0"

packages/@glimmer/tracking/package.json

@@ -17,15 +17,15 @@
   "dependencies": {
     "@glimmer/di": "^0.1.9",
     "@glimmer/env": "^0.1.7",
-    "@glimmer/reference": "^0.41.0",
-    "@glimmer/runtime": "^0.41.0",
-    "@glimmer/util": "^0.41.0"
+    "@glimmer/reference": "^0.42.0",
+    "@glimmer/runtime": "^0.42.0",
+    "@glimmer/util": "^0.42.0"
   },
   "devDependencies": {
     "@glimmer/application-test-helpers": "^0.14.0-alpha.13",
-    "@glimmer/compiler": "^0.41.0",
-    "@glimmer/interfaces": "^0.41.0",
+    "@glimmer/compiler": "^0.42.0",
+    "@glimmer/interfaces": "^0.42.0",
     "@glimmer/resolver": "^0.3.0",
-    "@glimmer/wire-format": "^0.41.0"
+    "@glimmer/wire-format": "^0.42.0"
   }
 }

packages/@glimmer/tracking/src/tracked.ts

@@ -3,10 +3,13 @@ import {
   Tag,
   DirtyableTag,
   UpdatableTag,
-  TagWrapper,
   combine,
   CONSTANT_TAG,
-  bump
+  bump,
+  dirty,
+  createTag,
+  createUpdatableTag,
+  update,
 } from '@glimmer/reference';
 import { dict, assert } from '@glimmer/util';
 import { Option, Dict } from '@glimmer/interfaces';
@@ -147,7 +150,7 @@ function descriptorForField(
 ): DecoratorPropertyDescriptor {
   assert(
     !desc || (!desc.value && !desc.get && !desc.set),
-    `You attempted to use @tracked on ${key}, but that element is not a class field. @tracked is only usable on class fields. Native getters and setters will autotrack add any tracked fields they encounter, so there is no need mark getters and setters with @tracked.`,
+    `You attempted to use @tracked on ${key}, but that element is not a class field. @tracked is only usable on class fields. Native getters and setters will autotrack add any tracked fields they encounter, so there is no need mark getters and setters with @tracked.`
   );
 
   const meta = metaFor(target);
@@ -178,9 +181,7 @@ function descriptorForField(
     },
 
     set(newValue: any): void {
-      (metaFor(this)
-        .tagFor(key)
-        .inner as any as DirtyableTag).dirty();
+      dirty(metaFor(this).tagFor(key) as DirtyableTag);
 
       values.set(this, newValue);
       propertyDidChange();
@@ -217,20 +218,17 @@ function descriptorForTrackedComputedProperty(
     if (CURRENT_TRACKER) CURRENT_TRACKER.add(tag);
     // Update the UpdatableTag for this property with the tag for all of the
     // consumed dependencies.
-    metaFor(this)
-      .updatableTagFor(key)
-      .inner.update(tag);
+    update(metaFor(this).updatableTagFor(key), tag);
 
     return ret;
   }
 
   function setter(this: object) {
-    bump();
+    let tag = createTag();
+    dirty(tag);
 
     // Mark the UpdatableTag for this property with the current tag.
-    metaFor(this)
-      .updatableTagFor(key)
-      .inner.update(DirtyableTag.create());
+    update(metaFor(this).updatableTagFor(key), tag);
     set.apply(this, arguments);
   }
 
@@ -264,13 +262,13 @@ export function trackedGet(obj: Object, key: string) {
  */
 export default class Meta {
   tags: Dict<Tag>;
-  computedPropertyTags: Dict<TagWrapper<UpdatableTag>>;
+  computedPropertyTags: Dict<UpdatableTag>;
   trackedProperties: Dict<boolean>;
   trackedComputedProperties: Dict<boolean>;
 
   constructor(parent: Meta) {
     this.tags = dict<Tag>();
-    this.computedPropertyTags = dict<TagWrapper<UpdatableTag>>();
+    this.computedPropertyTags = dict<UpdatableTag>();
     this.trackedProperties = parent ? Object.create(parent.trackedProperties) : dict<boolean>();
     this.trackedComputedProperties = parent
       ? Object.create(parent.trackedComputedProperties)
@@ -293,10 +291,10 @@ export default class Meta {
     }
 
     if (this.trackedComputedProperties[key]) {
-      return (this.tags[key] = UpdatableTag.create(CONSTANT_TAG));
+      return (this.tags[key] = createUpdatableTag());
     }
 
-    return (this.tags[key] = DirtyableTag.create());
+    return (this.tags[key] = createTag());
   }
 
   /**
@@ -305,8 +303,8 @@ export default class Meta {
    * For computed properties, it is the UpdatableTag used as one of the tags in
    * the tag combinator of the CP and its dependencies.
    */
-  updatableTagFor(key: Key): TagWrapper<UpdatableTag> {
-    return this.tags[key] as TagWrapper<UpdatableTag> || (this.tags[key] = UpdatableTag.create(CONSTANT_TAG));
+  updatableTagFor(key: Key): UpdatableTag {
+    return (this.tags[key] as UpdatableTag) || (this.tags[key] = createUpdatableTag());
   }
 }
 

packages/@glimmer/tracking/test/browser/helpers/tags.ts

@@ -1,10 +1,10 @@
-import { bump, Tag } from '@glimmer/reference';
+import { bump, Tag, validate } from '@glimmer/reference';
 
 export function assertValidAfterUnrelatedBump(tag: Tag, snapshot: number) {
   bump();
 
   QUnit.assert.strictEqual(
-    tag.validate(snapshot),
+    validate(tag, snapshot),
     true,
     'tag is still valid after an unrelated bump'
   );