@travetto/cli 4.1.1 → 4.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@travetto/cli",
-  "version": "4.1.1",
+  "version": "4.1.2",
   "description": "CLI infrastructure for Travetto framework",
   "keywords": [
     "cli",

package/src/module.ts

@@ -43,9 +43,9 @@ export class CliModuleUtil {
    * @param transitive
    * @returns
    */
-  static async findModules(mode: 'all' | 'changed', fromHash?: string, toHash?: string): Promise<IndexedModule[]> {
+  static async findModules(mode: 'all' | 'changed', fromHash?: string, toHash?: string, emptyOnFail = false): Promise<IndexedModule[]> {
     return (mode === 'changed' ?
-      await this.findChangedModulesRecursive(fromHash, toHash) :
+      await this.findChangedModulesRecursive(fromHash, toHash, emptyOnFail) :
       [...RuntimeIndex.getModuleList('all')].map(x => RuntimeIndex.getModule(x)!)
     ).filter(x => x.sourcePath !== RuntimeContext.workspace.path);
   }

package/src/parse.ts

@@ -210,7 +210,7 @@ export class CliParseUtil {
       inputs,
       all: out,
       unknown: out.filter(x => x.type === 'unknown').map(x => x.input),
-      flags: out.filter((x): x is ParsedInput & { type: 'flag' } => x.type === 'flag')
+      flags: out.filter(x => x.type === 'flag')
     };
   }
 }

package/src/scm.ts

@@ -2,7 +2,7 @@ import { spawn } from 'node:child_process';
 import fs from 'node:fs/promises';
 
 import { ExecUtil } from '@travetto/base';
-import { IndexedFile, IndexedModule, RuntimeIndex, RuntimeContext, path } from '@travetto/manifest';
+import { IndexedModule, RuntimeIndex, RuntimeContext, path } from '@travetto/manifest';
 
 export class CliScmUtil {
   /**
@@ -45,9 +45,13 @@ export class CliScmUtil {
    * @param fromHash
    * @returns
    */
-  static async findChangedFiles(fromHash: string, toHash: string = 'HEAD'): Promise<string[]> {
+  static async findChangedFiles(fromHash: string, toHash: string = 'HEAD', emptyOnFail = false): Promise<string[]> {
     const ws = RuntimeContext.workspace.path;
-    const res = await ExecUtil.getResult(spawn('git', ['diff', '--name-only', `${fromHash}..${toHash}`, ':!**/DOC.*', ':!**/README.*'], { cwd: ws }));
+    const res = await ExecUtil.getResult(spawn('git', ['diff', '--name-only', `${fromHash}..${toHash}`, ':!**/DOC.*', ':!**/README.*'], { cwd: ws }), { catch: true });
+    if (!res.valid && emptyOnFail) {
+      console.warn('Unable to detect changes between', fromHash, toHash, 'with', (res.stderr || res.stdout));
+      return [];
+    }
     const out = new Set<string>();
     for (const line of res.stdout.split(/\n/g)) {
       const entry = RuntimeIndex.getEntry(path.resolve(ws, line));
@@ -68,9 +72,9 @@ export class CliScmUtil {
     const files = await this.findChangedFiles(fromHash, toHash);
     const mods = files
       .map(x => RuntimeIndex.getFromSource(x))
-      .filter((x): x is IndexedFile => !!x)
+      .filter(x => !!x)
       .map(x => RuntimeIndex.getModule(x.module))
-      .filter((x): x is IndexedModule => !!x);
+      .filter(x => !!x);
 
     return [...new Set(mods)]
       .sort((a, b) => a.name.localeCompare(b.name));