glimmerjs · ef4 · Mar 17, 2025 · Mar 17, 2025 · Apr 29, 2025 · Apr 29, 2025
diff --git a/index.html b/index.html
@@ -7,6 +7,7 @@
 
   <body>
     <script type="module">
+      import "qunit/qunit/qunit.js";
       import { runTests } from "@glimmer-workspace/integration-tests";
       await runTests(
         import.meta.glob("./packages/{@glimmer,@glimmer-workspace}/*/test/**/*-test.ts"),

diff --git a/package.json b/package.json
@@ -51,7 +51,7 @@
     "@glimmer-workspace/eslint-plugin": "workspace:*",
     "@glimmer-workspace/integration-tests": "workspace:*",
     "@glimmer-workspace/repo-metadata": "workspace:*",
-    "@glimmer/env": "0.1.7",
+    "@glimmer/env": "workspace:*",
     "@rollup/plugin-sucrase": "^5.0.2",
     "@rollup/plugin-terser": "^0.4.4",
     "@tsconfig/strictest": "^2.0.5",
@@ -135,13 +135,13 @@
     },
     "overrides": {
       "@glimmer/syntax": "workspace:*",
+      "@oclif/plugin-warn-if-update-available": "^3.1.42",
       "@rollup/pluginutils": "^5.0.2",
       "@types/node": "$@types/node",
-      "typescript": "$typescript",
-      "@oclif/plugin-warn-if-update-available": "^3.1.42",
       "d3-color": "^3.1.0",
       "esbuild": "^0.25.0",
-      "got": "^11.8.5"
+      "got": "^11.8.5",
+      "typescript": "$typescript"
     },
     "peerDependencyRules": {
       "allowAny": [

diff --git a/packages/@glimmer-workspace/integration-tests/lib/setup-harness.ts b/packages/@glimmer-workspace/integration-tests/lib/setup-harness.ts
@@ -2,8 +2,6 @@
 import type { Expand } from '@glimmer/interfaces';
 import type { Runner } from 'js-reporters';
 import { debug } from '@glimmer/validator';
-import { autoRegister } from 'js-reporters';
-import { default as QUnit } from 'qunit';
 
 const SMOKE_TEST_FILE = './packages/@glimmer-workspace/integration-tests/test/smoke-test.ts';
 
@@ -31,7 +29,7 @@ export async function bootQunit(
 }
 
 export async function setupQunit() {
-  const qunitLib: QUnit = await import('qunit');
+  const qunitLib: QUnit = QUnit;
   await import('qunit/qunit/qunit.css');
 
   const testing = Testing.withConfig(
@@ -72,10 +70,11 @@ export async function setupQunit() {
     }
   );
 
-  const runner = autoRegister();
-
   testing.begin(() => {
     if (testing.config.ci) {
+      // @ts-expect-error TODO: js-reporters not available
+      const runner = autoRegister();
+
       // @ts-expect-error add reporters.tap to the types
 
       const tap = qunitLib.reporters.tap as {

diff --git a/packages/@glimmer-workspace/integration-tests/lib/setup.ts b/packages/@glimmer-workspace/integration-tests/lib/setup.ts
@@ -3,7 +3,6 @@ import type { IteratorDelegate } from '@glimmer/reference';
 import type { TestBase } from 'qunit';
 import setGlobalContext from '@glimmer/global-context';
 import { consumeTag, dirtyTagFor, tagFor } from '@glimmer/validator';
-import QUnit from 'qunit';
 
 import { scheduleDidDestroy, scheduleWillDestroy } from './base-env';
 import { NativeIteratorDelegate } from './modes/env';

diff --git a/packages/@glimmer-workspace/integration-tests/lib/test-helpers/tracked.ts b/packages/@glimmer-workspace/integration-tests/lib/test-helpers/tracked.ts
@@ -1,15 +1,2 @@
-import { trackedData } from '@glimmer/validator';
-
-export function tracked(target: object, key: string) {
-  let { getter, setter } = trackedData<any, any>(key);
-
-  Object.defineProperty(target, key, {
-    get() {
-      return getter(this);
-    },
-
-    set(value: unknown) {
-      setter(this, value);
-    },
-  });
-}
+// TODO: lets point everybody who uses this directly at @glimmer/tracking
+export { tracked } from '@glimmer/tracking';
diff --git a/packages/@glimmer-workspace/integration-tests/package.json b/packages/@glimmer-workspace/integration-tests/package.json
@@ -27,6 +27,7 @@
     "@glimmer/reference": "workspace:*",
     "@glimmer/runtime": "workspace:*",
     "@glimmer/syntax": "workspace:*",
+    "@glimmer/tracking": "workspace:*",
     "@glimmer/util": "workspace:*",
     "@glimmer/validator": "workspace:*",
     "@glimmer/vm": "workspace:*",

diff --git a/packages/@glimmer/compiler/lib/compiler.ts b/packages/@glimmer/compiler/lib/compiler.ts
@@ -28,11 +28,13 @@ interface Crypto {
 }
 
 export const defaultId: TemplateIdFn = (() => {
+  /* prettier-ignore */
   const req = (
+    // @ts-expect-error using node API here, but only here
     typeof module === 'object' && typeof module.require === 'function'
-      ? // eslint-disable-next-line @typescript-eslint/unbound-method
+      ? // @ts-expect-error using node API here, but only here
         module.require
-      : globalThis.require
+      : (globalThis as any).require
   ) as typeof require | undefined;
 
   if (req) {

diff --git a/packages/@glimmer/compiler/package.json b/packages/@glimmer/compiler/package.json
@@ -45,8 +45,6 @@
     "@glimmer/wire-format": "workspace:*"
   },
   "devDependencies": {
-    "@glimmer-workspace/build-support": "workspace:*",
-    "@glimmer-workspace/env": "workspace:*",
     "@glimmer/constants": "workspace:*",
     "@glimmer/debug": "workspace:*",
     "@glimmer/debug-util": "workspace:*",

diff --git a/packages/@glimmer/constants/test/package.json b/packages/@glimmer/constants/test/package.json
@@ -4,7 +4,7 @@
   "version": "0.92.0",
   "scripts": {},
   "dependencies": {
-    "@glimmer/env": "0.1.7",
+    "@glimmer/env": "workspace:*",
     "@glimmer/util": "workspace:*"
   },
   "devDependencies": {

diff --git a/packages/@glimmer/debug-util/lib/debug-to-string.ts b/packages/@glimmer/debug-util/lib/debug-to-string.ts
@@ -2,7 +2,7 @@ import type { AnyFn } from '@glimmer/interfaces';
 
 let debugToString: undefined | ((value: unknown) => string);
 
-if (import.meta.env.DEV) {
+if (import.meta.env?.DEV) {
   let getFunctionName = (fn: AnyFn) => {
     let functionName = fn.name;
 

diff --git a/packages/@glimmer/debug-util/test/package.json b/packages/@glimmer/debug-util/test/package.json
@@ -4,7 +4,7 @@
   "version": "0.92.0",
   "scripts": {},
   "dependencies": {
-    "@glimmer/env": "0.1.7",
+    "@glimmer/env": "workspace:*",
     "@glimmer/util": "workspace:*"
   },
   "devDependencies": {

diff --git a/packages/@glimmer/debug/index.ts b/packages/@glimmer/debug/index.ts
@@ -1,6 +1,5 @@
 export type { DebugOp, SomeDisassembledOperand } from './lib/debug';
-export { debugOp, describeOpcode, logOpcodeSlice } from './lib/debug';
-export { describeOp } from './lib/dism/opcode';
+export { debugOp, describeOpcode, logOpcodeSlice, describeOp } from './lib/debug';
 export {
   buildEnum,
   buildMetas,

diff --git a/packages/@glimmer/debug/lib/debug.ts b/packages/@glimmer/debug/lib/debug.ts
@@ -23,11 +23,28 @@ import { $fp, $pc, $ra, $s0, $s1, $sp, $t0, $t1, $v0 } from '@glimmer/vm';
 import type { Primitive, RegisterName } from './dism/dism';
 import type { NormalizedOperand, OperandType, ShorthandOperand } from './dism/operand-types';
 
-import { describeOp } from './dism/opcode';
 import { OPERANDS } from './dism/operands';
 import { opcodeMetadata } from './opcode-metadata';
-import { frag } from './render/fragment';
+import { frag, type Fragment, as, type IntoFragment } from './render/fragment';
 import { DebugLogger } from './render/logger';
+import { SerializeBlockContext } from './dism/opcode';
+import { join } from './render/basic';
+
+export function describeOp(
+  op: RuntimeOp,
+  program: Program,
+  meta: Nullable<BlockMetadata>
+): Fragment {
+  const { name, params } = debugOp(program, op, meta);
+
+  const block = new SerializeBlockContext(meta?.symbols ?? null);
+
+  let args: IntoFragment[] = Object.entries(params).map(
+    ([p, v]) => frag`${as.attrName(p)}=${block.serialize(v)}`
+  );
+
+  return frag`(${join([as.kw(name), ...args], ' ')})`;
+}
 
 export function logOpcodeSlice(context: CompilationContext, start: number, end: number) {
   if (LOCAL_TRACE_LOGGING) {
@@ -405,48 +422,3 @@ function fromRaw(operands: Dict<RawDisassembledOperand>): Dict<SomeDisassembledO
     Object.entries(operands).map(([name, raw]) => [name, DisassembledOperand.of(raw)])
   );
 }
-
-export function decodeCurry(curry: number): 'component' | 'helper' | 'modifier' {
-  switch (curry) {
-    case CURRIED_COMPONENT:
-      return 'component';
-    case CURRIED_HELPER:
-      return 'helper';
-    case CURRIED_MODIFIER:
-      return 'modifier';
-    default:
-      throw Error(`Unexpected curry value: ${curry}`);
-  }
-}
-
-export function decodeRegister(register: number): RegisterName {
-  switch (register) {
-    case $pc:
-      return '$pc';
-    case $ra:
-      return '$ra';
-    case $fp:
-      return '$fp';
-    case $sp:
-      return '$sp';
-    case $s0:
-      return '$s0';
-    case $s1:
-      return '$s1';
-    case $t0:
-      return '$t0';
-    case $t1:
-      return '$t1';
-    case $v0:
-      return '$v0';
-    default:
-      return `$bug${register}`;
-  }
-}
-
-export function decodePrimitive(primitive: number, constants: ProgramConstants): Primitive {
-  if (primitive >= 0) {
-    return constants.getValue(decodeHandle(primitive));
-  }
-  return decodeImmediate(primitive);
-}
diff --git a/packages/@glimmer/debug/lib/decoders.ts b/packages/@glimmer/debug/lib/decoders.ts
@@ -0,0 +1,56 @@
+import type { ProgramConstants } from '@glimmer/interfaces';
+import {
+  CURRIED_COMPONENT,
+  CURRIED_HELPER,
+  CURRIED_MODIFIER,
+  decodeHandle,
+  decodeImmediate,
+} from '@glimmer/constants';
+import { $fp, $pc, $ra, $s0, $s1, $sp, $t0, $t1, $v0 } from '@glimmer/vm';
+
+import type { Primitive, RegisterName } from './dism/dism';
+
+export function decodeCurry(curry: number): 'component' | 'helper' | 'modifier' {
+  switch (curry) {
+    case CURRIED_COMPONENT:
+      return 'component';
+    case CURRIED_HELPER:
+      return 'helper';
+    case CURRIED_MODIFIER:
+      return 'modifier';
+    default:
+      throw Error(`Unexpected curry value: ${curry}`);
+  }
+}
+
+export function decodeRegister(register: number): RegisterName {
+  switch (register) {
+    case $pc:
+      return '$pc';
+    case $ra:
+      return '$ra';
+    case $fp:
+      return '$fp';
+    case $sp:
+      return '$sp';
+    case $s0:
+      return '$s0';
+    case $s1:
+      return '$s1';
+    case $t0:
+      return '$t0';
+    case $t1:
+      return '$t1';
+    case $v0:
+      return '$v0';
+    default:
+      return `$bug${register}`;
+  }
+}
+
+export function decodePrimitive(primitive: number, constants: ProgramConstants): Primitive {
+  if (primitive >= 0) {
+    return constants.getValue(decodeHandle(primitive));
+  }
+  return decodeImmediate(primitive);
+}
diff --git a/packages/@glimmer/debug/lib/dism/opcode.ts b/packages/@glimmer/debug/lib/dism/opcode.ts
@@ -1,14 +1,11 @@
 import type { ClassifiedLocalDebug, ClassifiedLocalDebugFor } from '@glimmer/debug-util';
 import type {
   AppendingBlock,
-  BlockMetadata,
   BlockSymbolNames,
   Cursor,
   NamedArguments,
   Nullable,
   PositionalArguments,
-  Program,
-  RuntimeOp,
   VMArguments,
 } from '@glimmer/interfaces';
 import { dev, exhausted, getLocalDebugType } from '@glimmer/debug-util';
@@ -19,27 +16,10 @@ import type { ValueRefOptions } from '../render/basic';
 import type { IntoFragment } from '../render/fragment';
 import type { RegisterName, SomeDisassembledOperand } from './dism';
 
-import { debugOp } from '../debug';
 import { empty, join, unknownValue, value } from '../render/basic';
 import { array } from '../render/combinators';
 import { as, frag, Fragment } from '../render/fragment';
 
-export function describeOp(
-  op: RuntimeOp,
-  program: Program,
-  meta: Nullable<BlockMetadata>
-): Fragment {
-  const { name, params } = debugOp(program, op, meta);
-
-  const block = new SerializeBlockContext(meta?.symbols ?? null);
-
-  let args: IntoFragment[] = Object.entries(params).map(
-    ([p, v]) => frag`${as.attrName(p)}=${block.serialize(v)}`
-  );
-
-  return frag`(${join([as.kw(name), ...args], ' ')})`;
-}
-
 export class SerializeBlockContext {
   readonly #symbols: Nullable<BlockSymbolNames>;
 

diff --git a/packages/@glimmer/debug/lib/dism/operands.ts b/packages/@glimmer/debug/lib/dism/operands.ts
@@ -9,7 +9,7 @@ import type {
   OperandType,
 } from './operand-types';
 
-import { decodeCurry, decodePrimitive, decodeRegister } from '../debug';
+import { decodeCurry, decodePrimitive, decodeRegister } from '../decoders';
 
 interface DisassemblyState {
   readonly offset: number;

diff --git a/packages/@glimmer/debug/lib/stack-check.ts b/packages/@glimmer/debug/lib/stack-check.ts
@@ -415,7 +415,7 @@ function defaultMessage(value: unknown, expected: string): string {
   return `Got ${value}, expected:\n${expected}`;
 }
 
-/*@__PURE__*/
+/*@__NO_SIDE_EFFECTS__*/
 export function check<T>(
   value: unknown,
   checker: Checker<T>,

diff --git a/packages/@glimmer/debug/lib/vm/snapshot.ts b/packages/@glimmer/debug/lib/vm/snapshot.ts
@@ -15,7 +15,7 @@ import { $fp, $pc } from '@glimmer/vm';
 
 import type { Fragment } from '../render/fragment';
 
-import { decodeRegister } from '../debug';
+import { decodeRegister } from '../decoders';
 import { value } from '../render/basic';
 import { array } from '../render/combinators';
 import { as, frag } from '../render/fragment';

diff --git a/packages/@glimmer/destroyable/package.json b/packages/@glimmer/destroyable/package.json
@@ -38,8 +38,6 @@
     "@glimmer/interfaces": "workspace:*"
   },
   "devDependencies": {
-    "@glimmer-workspace/build-support": "workspace:*",
-    "@glimmer-workspace/env": "workspace:*",
     "@glimmer/debug-util": "workspace:*",
     "eslint": "^9.20.1",
     "publint": "^0.3.2",

diff --git a/packages/@glimmer/encoder/package.json b/packages/@glimmer/encoder/package.json
@@ -37,8 +37,6 @@
     "@glimmer/vm": "workspace:*"
   },
   "devDependencies": {
-    "@glimmer-workspace/build-support": "workspace:*",
-    "@glimmer-workspace/env": "workspace:*",
     "eslint": "^9.20.1",
     "publint": "^0.3.2",
     "rollup": "^4.34.8",