@travetto/cli 3.4.1 → 3.4.2

package/package.json

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

package/src/module.ts

@@ -11,19 +11,20 @@ export class CliModuleUtil {
 
   /**
    * Find modules that changed, and the dependent modules
-   * @param hash
+   * @param fromHash
+   * @param toHash
    * @param transitive
    * @returns
    */
-  static async findChangedModulesRecursive(hash?: string, transitive = true): Promise<IndexedModule[]> {
-    hash ??= await CliScmUtil.findLastRelease();
+  static async findChangedModulesRecursive(fromHash?: string, toHash?: string, transitive = true): Promise<IndexedModule[]> {
+    fromHash ??= await CliScmUtil.findLastRelease();
 
-    if (!hash) {
+    if (!fromHash) {
       return RootIndex.getLocalModules();
     }
 
     const out = new Map<string, IndexedModule>();
-    for (const mod of await CliScmUtil.findChangedModulesSince(hash)) {
+    for (const mod of await CliScmUtil.findChangedModules(fromHash, toHash)) {
       out.set(mod.name, mod);
       if (transitive) {
         for (const sub of await RootIndex.getDependentModules(mod)) {
@@ -42,9 +43,9 @@ export class CliModuleUtil {
    * @param transitive
    * @returns
    */
-  static async findModules(mode: 'all' | 'changed', sinceHash?: string): Promise<IndexedModule[]> {
+  static async findModules(mode: 'all' | 'changed', fromHash?: string, toHash?: string): Promise<IndexedModule[]> {
     return (mode === 'changed' ?
-      await this.findChangedModulesRecursive(sinceHash) :
+      await this.findChangedModulesRecursive(fromHash, toHash) :
       [...RootIndex.getModuleList('all')].map(x => RootIndex.getModule(x)!)
     ).filter(x => x.sourcePath !== RootIndex.manifest.workspacePath);
   }

package/src/scm.ts

@@ -41,13 +41,13 @@ export class CliScmUtil {
   }
 
   /**
-   * Find all source files that changed since hash
-   * @param hash
+   * Find all source files that changed between from and to hashes
+   * @param fromHash
    * @returns
    */
-  static async findChangedFilesSince(hash: string): Promise<string[]> {
+  static async findChangedFiles(fromHash: string, toHash: string = 'HEAD'): Promise<string[]> {
     const ws = RootIndex.manifest.workspacePath;
-    const res = await ExecUtil.spawn('git', ['diff', '--name-only', `HEAD..${hash}`, ':!**/DOC.*', ':!**/README.*'], { cwd: ws }).result;
+    const res = await ExecUtil.spawn('git', ['diff', '--name-only', `${fromHash}..${toHash}`, ':!**/DOC.*', ':!**/README.*'], { cwd: ws }).result;
     const out = new Set<string>();
     for (const line of res.stdout.split(/\n/g)) {
       const entry = RootIndex.getEntry(path.resolve(ws, line));
@@ -59,12 +59,13 @@ export class CliScmUtil {
   }
 
   /**
-   * Find all modules that changed since hash
-   * @param hash
+   * Find all modules that changed between from and to hashes
+   * @param fromHash
+   * @param toHash
    * @returns
    */
-  static async findChangedModulesSince(hash: string): Promise<IndexedModule[]> {
-    const files = await this.findChangedFilesSince(hash);
+  static async findChangedModules(fromHash: string, toHash?: string): Promise<IndexedModule[]> {
+    const files = await this.findChangedFiles(fromHash, toHash);
     const mods = files
       .map(x => RootIndex.getFromSource(x))
       .filter((x): x is IndexedFile => !!x)