Fix a few things that were broken on the error recovery branch

glimmerjs · Nov 22, 2023 · 63d4ebf · 63d4ebf
1 parent c6f163d
commit 63d4ebf
Show file tree

Hide file tree

Showing 6 changed files with 523 additions and 283 deletions.
diff --git a/packages/@glimmer-workspace/build/lib/config.js b/packages/@glimmer-workspace/build/lib/config.js
@@ -64,6 +64,7 @@ export function tsconfig(updates) {
     declaration: true,
     declarationMap: true,
     verbatimModuleSyntax: true,
+    isolatedModules: true,
     module: ModuleKind.NodeNext,
     moduleResolution: ModuleResolutionKind.NodeNext,
     ...updates,
@@ -254,6 +255,8 @@ export class Package {
    * @property {boolean} [ esm ] enabled by default
    * @property {boolean} [ cjs ] enabled by default until eslint-plugin-ember and ember-source no longer need it
    *
+   * @param {Formats} [formats]
+   *
    * @returns {import("rollup").RollupOptions[] | import("rollup").RollupOptions}
    */
   config(formats = {}) {

diff --git a/packages/@glimmer-workspace/build/lib/replace.js b/packages/@glimmer-workspace/build/lib/replace.js
@@ -70,6 +70,8 @@ export function createReplacePlugin(test, replacements, sourcemap) {
         /** @type {TransformResult} */
         const result = { code: s.toString() };
         if (sourcemap) {
+          // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+          // @ts-ignore
           result.map = s.generateMap({ hires: true });
         }
         return result;

diff --git a/packages/@glimmer-workspace/integration-tests/lib/setup-harness.ts b/packages/@glimmer-workspace/integration-tests/lib/setup-harness.ts
@@ -50,6 +50,8 @@ export async function setupQunit() {
 
   testing.begin(() => {
     if (testing.config.ci) {
+      // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+      // @ts-ignore
       const tap = qunitLib.reporters.tap;
       tap.init(runner, { log: console.info });
     }

diff --git a/packages/@glimmer/runtime/lib/vm/low-level.ts b/packages/@glimmer/runtime/lib/vm/low-level.ts
@@ -203,7 +203,7 @@ export interface Externs {
   debug: VM;
 }
 
-export interface VmDebugState {
+export interface LowLevelVmDebugState {
   readonly registers: Registers;
   readonly currentPc: number;
   readonly stack: DebugStack;
@@ -292,7 +292,7 @@ export class LowLevelVM {
   /**
    * @mutable
    */
-  get debug(): VmDebugState {
+  get debug(): LowLevelVmDebugState {
     return {
       currentPc: this.#registers.pc - this.#currentOpSize,
       registers: this.#registers,