Merge pull request #78 from dorny/issue-77-tag-as-base-broken

Fix change detection when base is a tag

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## v2.9.3
+- [Fix change detection when base is a tag](https://github.com/dorny/paths-filter/pull/78)
+
+## v2.9.2
+- [Fix fetching git history](https://github.com/dorny/paths-filter/pull/75)
+
 ## v2.9.1
 - [Fix fetching git history + fallback to unshallow repo](https://github.com/dorny/paths-filter/pull/74)
 

dist/index.js

@@ -3865,51 +3865,60 @@ async function getChangesOnHead() {
     return parseGitDiffOutput(output);
 }
 exports.getChangesOnHead = getChangesOnHead;
-async function getChangesSinceMergeBase(baseRef, ref, initialFetchDepth) {
+async function getChangesSinceMergeBase(base, ref, initialFetchDepth) {
+    let baseRef;
     async function hasMergeBase() {
-        return (await exec_1.default('git', ['merge-base', baseRef, ref], { ignoreReturnCode: true })).code === 0;
+        return (baseRef !== undefined && (await exec_1.default('git', ['merge-base', baseRef, ref], { ignoreReturnCode: true })).code === 0);
     }
     let noMergeBase = false;
-    core.startGroup(`Searching for merge-base ${baseRef}...${ref}`);
+    core.startGroup(`Searching for merge-base ${base}...${ref}`);
     try {
-        let init = true;
-        let lastCommitCount = await getCommitCount();
-        let depth = Math.max(lastCommitCount * 2, initialFetchDepth);
-        while (!(await hasMergeBase())) {
-            if (init) {
-                await exec_1.default('git', ['fetch', `--depth=${depth}`, 'origin', `${baseRef}:${baseRef}`, `${ref}`]);
-                init = false;
-            }
-            else {
-                await exec_1.default('git', ['fetch', `--deepen=${depth}`, 'origin', baseRef, ref]);
-            }
-            const commitCount = await getCommitCount();
-            if (commitCount === lastCommitCount) {
-                core.info('No more commits were fetched');
-                core.info('Last attempt will be to fetch full history');
-                await exec_1.default('git', ['fetch', '--unshallow']);
-                if (!(await hasMergeBase())) {
-                    noMergeBase = true;
+        baseRef = await getFullRef(base);
+        if (!(await hasMergeBase())) {
+            await exec_1.default('git', ['fetch', '--no-tags', `--depth=${initialFetchDepth}`, 'origin', base, ref]);
+            if (baseRef === undefined) {
+                baseRef = await getFullRef(base);
+                if (baseRef === undefined) {
+                    await exec_1.default('git', ['fetch', '--tags', `--depth=1`, 'origin', base, ref]);
+                    baseRef = await getFullRef(base);
+                    if (baseRef === undefined) {
+                        throw new Error(`Could not determine what is ${base} - fetch works but it's not a branch or tag`);
+                    }
                 }
-                break;
             }
-            depth = Math.min(depth * 2, Number.MAX_SAFE_INTEGER);
-            lastCommitCount = commitCount;
+            let depth = initialFetchDepth;
+            let lastCommitCount = await getCommitCount();
+            while (!(await hasMergeBase())) {
+                depth = Math.min(depth * 2, Number.MAX_SAFE_INTEGER);
+                await exec_1.default('git', ['fetch', `--deepen=${depth}`, 'origin', base, ref]);
+                const commitCount = await getCommitCount();
+                if (commitCount === lastCommitCount) {
+                    core.info('No more commits were fetched');
+                    core.info('Last attempt will be to fetch full history');
+                    await exec_1.default('git', ['fetch']);
+                    if (!(await hasMergeBase())) {
+                        noMergeBase = true;
+                    }
+                    break;
+                }
+                lastCommitCount = commitCount;
+            }
         }
     }
     finally {
         core.endGroup();
     }
+    let diffArg = `${baseRef}...${ref}`;
     if (noMergeBase) {
-        core.warning('No merge base found - all files will be listed as added');
-        return await listAllFilesAsAdded();
+        core.warning('No merge base found - change detection will use direct <commit>..<commit> comparison');
+        diffArg = `${baseRef}..${ref}`;
     }
-    // Get changes introduced on HEAD compared to ref
-    core.startGroup(`Change detection ${baseRef}...${ref}`);
+    // Get changes introduced on ref compared to base
+    core.startGroup(`Change detection ${diffArg}`);
     let output = '';
     try {
         // Three dots '...' change detection - finds merge-base and compares against it
-        output = (await exec_1.default('git', ['diff', '--no-renames', '--name-status', '-z', `${baseRef}...${ref}`])).stdout;
+        output = (await exec_1.default('git', ['diff', '--no-renames', '--name-status', '-z', diffArg])).stdout;
     }
     finally {
         fixStdOutNullTermination();
@@ -3997,6 +4006,24 @@ async function getCommitCount() {
     const count = parseInt(output);
     return isNaN(count) ? 0 : count;
 }
+async function getFullRef(shortName) {
+    if (isGitSha(shortName)) {
+        return shortName;
+    }
+    const output = (await exec_1.default('git', ['show-ref', shortName], { ignoreReturnCode: true })).stdout;
+    const refs = output
+        .split(/\r?\n/g)
+        .map(l => { var _a, _b; return (_b = (_a = l.match(/refs\/.*$/)) === null || _a === void 0 ? void 0 : _a[0]) !== null && _b !== void 0 ? _b : ''; })
+        .filter(l => l !== '');
+    if (refs.length === 0) {
+        return undefined;
+    }
+    const remoteRef = refs.find(ref => ref.startsWith('refs/remotes/origin/'));
+    if (remoteRef) {
+        return remoteRef;
+    }
+    return refs[0];
+}
 function fixStdOutNullTermination() {
     // Previous command uses NULL as delimiters and output is printed to stdout.
     // We have to make sure next thing written to stdout will start on new line.
@@ -4746,7 +4773,7 @@ async function getChangedFilesFromGit(base, initialFetchDepth) {
         return await git.getChanges(baseSha);
     }
     // Changes introduced by current branch against the base branch
-    core.info(`Changes will be detected against the branch ${baseRef}`);
+    core.info(`Changes will be detected against ${baseRef}`);
     return await git.getChangesSinceMergeBase(baseRef, ref, initialFetchDepth);
 }
 // Uses github REST api to get list of files changed in PR

src/git.ts

@@ -54,56 +54,65 @@ export async function getChangesOnHead(): Promise<File[]> {
   return parseGitDiffOutput(output)
 }
 
-export async function getChangesSinceMergeBase(
-  baseRef: string,
-  ref: string,
-  initialFetchDepth: number
-): Promise<File[]> {
+export async function getChangesSinceMergeBase(base: string, ref: string, initialFetchDepth: number): Promise<File[]> {
+  let baseRef: string | undefined
   async function hasMergeBase(): Promise<boolean> {
-    return (await exec('git', ['merge-base', baseRef, ref], {ignoreReturnCode: true})).code === 0
+    return (
+      baseRef !== undefined && (await exec('git', ['merge-base', baseRef, ref], {ignoreReturnCode: true})).code === 0
+    )
   }
 
   let noMergeBase = false
-  core.startGroup(`Searching for merge-base ${baseRef}...${ref}`)
+  core.startGroup(`Searching for merge-base ${base}...${ref}`)
   try {
-    let init = true
-    let lastCommitCount = await getCommitCount()
-    let depth = Math.max(lastCommitCount * 2, initialFetchDepth)
-    while (!(await hasMergeBase())) {
-      if (init) {
-        await exec('git', ['fetch', `--depth=${depth}`, 'origin', `${baseRef}:${baseRef}`, `${ref}`])
-        init = false
-      } else {
-        await exec('git', ['fetch', `--deepen=${depth}`, 'origin', baseRef, ref])
-      }
-      const commitCount = await getCommitCount()
-      if (commitCount === lastCommitCount) {
-        core.info('No more commits were fetched')
-        core.info('Last attempt will be to fetch full history')
-        await exec('git', ['fetch', '--unshallow'])
-        if (!(await hasMergeBase())) {
-          noMergeBase = true
+    baseRef = await getFullRef(base)
+    if (!(await hasMergeBase())) {
+      await exec('git', ['fetch', '--no-tags', `--depth=${initialFetchDepth}`, 'origin', base, ref])
+      if (baseRef === undefined) {
+        baseRef = await getFullRef(base)
+        if (baseRef === undefined) {
+          await exec('git', ['fetch', '--tags', `--depth=1`, 'origin', base, ref])
+          baseRef = await getFullRef(base)
+          if (baseRef === undefined) {
+            throw new Error(`Could not determine what is ${base} - fetch works but it's not a branch or tag`)
+          }
         }
-        break
       }
-      depth = Math.min(depth * 2, Number.MAX_SAFE_INTEGER)
-      lastCommitCount = commitCount
+
+      let depth = initialFetchDepth
+      let lastCommitCount = await getCommitCount()
+      while (!(await hasMergeBase())) {
+        depth = Math.min(depth * 2, Number.MAX_SAFE_INTEGER)
+        await exec('git', ['fetch', `--deepen=${depth}`, 'origin', base, ref])
+        const commitCount = await getCommitCount()
+        if (commitCount === lastCommitCount) {
+          core.info('No more commits were fetched')
+          core.info('Last attempt will be to fetch full history')
+          await exec('git', ['fetch'])
+          if (!(await hasMergeBase())) {
+            noMergeBase = true
+          }
+          break
+        }
+        lastCommitCount = commitCount
+      }
     }
   } finally {
     core.endGroup()
   }
 
+  let diffArg = `${baseRef}...${ref}`
   if (noMergeBase) {
-    core.warning('No merge base found - all files will be listed as added')
-    return await listAllFilesAsAdded()
+    core.warning('No merge base found - change detection will use direct <commit>..<commit> comparison')
+    diffArg = `${baseRef}..${ref}`
   }
 
-  // Get changes introduced on HEAD compared to ref
-  core.startGroup(`Change detection ${baseRef}...${ref}`)
+  // Get changes introduced on ref compared to base
+  core.startGroup(`Change detection ${diffArg}`)
   let output = ''
   try {
     // Three dots '...' change detection - finds merge-base and compares against it
-    output = (await exec('git', ['diff', '--no-renames', '--name-status', '-z', `${baseRef}...${ref}`])).stdout
+    output = (await exec('git', ['diff', '--no-renames', '--name-status', '-z', diffArg])).stdout
   } finally {
     fixStdOutNullTermination()
     core.endGroup()
@@ -193,6 +202,29 @@ async function getCommitCount(): Promise<number> {
   return isNaN(count) ? 0 : count
 }
 
+async function getFullRef(shortName: string): Promise<string | undefined> {
+  if (isGitSha(shortName)) {
+    return shortName
+  }
+
+  const output = (await exec('git', ['show-ref', shortName], {ignoreReturnCode: true})).stdout
+  const refs = output
+    .split(/\r?\n/g)
+    .map(l => l.match(/refs\/.*$/)?.[0] ?? '')
+    .filter(l => l !== '')
+
+  if (refs.length === 0) {
+    return undefined
+  }
+
+  const remoteRef = refs.find(ref => ref.startsWith('refs/remotes/origin/'))
+  if (remoteRef) {
+    return remoteRef
+  }
+
+  return refs[0]
+}
+
 function fixStdOutNullTermination(): void {
   // Previous command uses NULL as delimiters and output is printed to stdout.
   // We have to make sure next thing written to stdout will start on new line.

src/main.ts

@@ -121,7 +121,7 @@ async function getChangedFilesFromGit(base: string, initialFetchDepth: number):
   }
 
   // Changes introduced by current branch against the base branch
-  core.info(`Changes will be detected against the branch ${baseRef}`)
+  core.info(`Changes will be detected against ${baseRef}`)
   return await git.getChangesSinceMergeBase(baseRef, ref, initialFetchDepth)
 }