@travetto/compiler 4.0.9 → 4.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@travetto/compiler",
-  "version": "4.0.9",
+  "version": "4.1.1",
   "description": "The compiler infrastructure for the Travetto framework",
   "keywords": [
     "compiler",
@@ -31,12 +31,12 @@
   },
   "dependencies": {
     "@parcel/watcher": "^2.4.1",
-    "@travetto/manifest": "^4.0.3",
-    "@travetto/transformer": "^4.0.4",
-    "@types/node": "^20.11.25"
+    "@travetto/manifest": "^4.1.0",
+    "@travetto/transformer": "^4.1.1",
+    "@types/node": "^20.14.2"
   },
   "peerDependencies": {
-    "@travetto/cli": "^4.0.7"
+    "@travetto/cli": "^4.1.1"
   },
   "peerDependenciesMeta": {
     "@travetto/cli": {

package/src/compiler.ts

@@ -67,6 +67,7 @@ export class Compiler {
       case 'error': {
         process.exitCode = 1;
         if (err) {
+          EventUtil.sendEvent('log', { level: 'error', message: err.toString(), time: Date.now() });
           log.error('Shutting down due to failure', err.message);
         }
         break;
@@ -127,7 +128,7 @@ export class Compiler {
     EventUtil.sendEvent('state', { state: 'init', extra: { pid: process.pid } });
 
     const emitter = await this.getCompiler();
-    let failed = false;
+    let failure: Error | undefined;
 
     log.debug('Compiler loaded');
 
@@ -136,17 +137,16 @@ export class Compiler {
     if (this.#dirtyFiles.length) {
       for await (const ev of this.emit(this.#dirtyFiles, emitter)) {
         if (ev.err) {
-          failed = true;
           const compileError = CompilerUtil.buildTranspileError(ev.file, ev.err);
+          failure ??= compileError;
           EventUtil.sendEvent('log', { level: 'error', message: compileError.toString(), time: Date.now() });
         }
       }
       if (this.#signal.aborted) {
         log.debug('Compilation aborted');
-      } else if (failed) {
+      } else if (failure) {
         log.debug('Compilation failed');
-        process.exitCode = 1;
-        return;
+        return this.#shutdown('error', failure);
       } else {
         log.debug('Compilation succeeded');
       }

package/src/state.ts

@@ -63,14 +63,15 @@ export class CompilerState implements ts.CompilerHost {
 
     // Register all inputs
     for (const x of this.#modules) {
+      const base = x?.files ?? {};
       const files = [
-        ...x.files.bin ?? [],
-        ...x.files.src ?? [],
-        ...x.files.support ?? [],
-        ...x.files.doc ?? [],
-        ...x.files.test ?? [],
-        ...x.files.$index ?? [],
-        ...x.files.$package ?? []
+        ...base.bin ?? [],
+        ...base.src ?? [],
+        ...base.support ?? [],
+        ...base.doc ?? [],
+        ...base.test ?? [],
+        ...base.$index ?? [],
+        ...base.$package ?? []
       ];
       for (const [file, type] of files) {
         if (CompilerUtil.validFile(type) || type === 'typings') {
@@ -103,7 +104,11 @@ export class CompilerState implements ts.CompilerHost {
   }
 
   getArbitraryInputFile(): string {
-    return this.getBySource(this.#manifestIndex.getModule('@travetto/manifest')!.files.src[0].sourceFile)!.inputFile;
+    const randomSource = this.#manifestIndex.getWorkspaceModules()
+      .filter(x => x.files.src?.length)[0]
+      .files.src[0].sourceFile;
+
+    return this.getBySource(randomSource)!.inputFile;
   }
 
   async createProgram(force = false): Promise<ts.Program> {

package/src/util.ts

@@ -105,7 +105,6 @@ export class CompilerUtil {
     }
 
     const errors: string[] = diagnostics.slice(0, 5).map(diag => {
-      // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
       const message = ts.flattenDiagnosticMessageText(diag.messageText, '\n');
       if (diag.file) {
         const { line, character } = diag.file.getLineAndCharacterOfPosition(diag.start!);

package/src/watch.ts

@@ -30,6 +30,10 @@ export class CompilerWatcher {
     this.#signal = signal;
   }
 
+  #reset(ev: WatchEvent): never {
+    throw new Error('RESET', { cause: `${ev.action}:${ev.file}` });
+  }
+
   #getIgnores(): string[] {
     // TODO: Read .gitignore?
     let ignores = PackageUtil.readPackage(this.#state.manifest.workspace.path)?.travetto?.build?.watchIgnores;
@@ -101,7 +105,11 @@ export class CompilerWatcher {
 
     for (const file of allFiles) {
       for (const parent of parents.get(file.mod)!) {
-        moduleToFiles.get(parent)!.files.push(file);
+        const mod = moduleToFiles.get(parent);
+        if (!mod || !mod.files) {
+          this.#reset({ action: file.action, file: file.moduleFile });
+        }
+        mod.files.push(file);
       }
     }
 
@@ -157,7 +165,7 @@ export class CompilerWatcher {
           sourceFile === ROOT_PKG ||
           (action === 'delete' && (sourceFile === OUTPUT_PATH || sourceFile === COMPILER_PATH))
         ) {
-          throw new Error('RESET', { cause: `${action}:${sourceFile}` });
+          this.#reset(ev);
         }
 
         const fileType = ManifestModuleUtil.getFileType(sourceFile);