fix: ignore inconsistent runs that have a status set to 'in_progress' but conclusion is 'success' (#59)

* Ignore inconsistent runs that have a status set to 'in_progress' but conclusion is 'success'

* Bump action node version to 16

* Upgrade dependencies, run npm dedupe

* run fmt and build

Signed-off-by: Rui Chen <[email protected]>

---------

Signed-off-by: Rui Chen <[email protected]>
Co-authored-by: gustaff-weldon <[email protected]>

Author: chenrui333

dist/index.js

@@ -8,9 +8,8 @@ async function run() {
   try {
     const input = parseInput(env);
     debug(
-      `Parsed inputs (w/o token): ${(({ githubToken, ...inputs }) => inputs)(
-        input,
-      )}`,
+      `Parsed inputs (w/o token): ${(({ githubToken, ...inputs }) =>
+        JSON.stringify(inputs))(input)}`,
     );
     const github = new OctokitGitHub(input.githubToken);
     debug(`Fetching workflows for ${input.owner}/${input.repo}...`);

src/main.ts

@@ -57,6 +57,17 @@ export class Waiter implements Wait {
     this.debug(`Found ${runs.length} ${this.workflowId} runs`);
     const previousRuns = runs
       .filter((run) => run.id < this.input.runId)
+      .filter((run) => {
+        const isSuccessful: boolean = run.conclusion === "success";
+
+        if (isSuccessful) {
+          this.debug(
+            `Skipping run ${run.id}, status: ${run.status}, conclusion: ${run.conclusion}`,
+          );
+        }
+
+        return !isSuccessful;
+      })
       .sort((a, b) => b.id - a.id);
     if (!previousRuns || !previousRuns.length) {
       setOutput("force_continued", "");