Fix: Only seeing "in progress" status

.gitignore

@@ -1,2 +1,4 @@
 node_modules
-.DS_Store
+.DS_Store
+yarn.lock
+package-lock.json

src/post.ts

@@ -11,12 +11,12 @@ try {
     const workflowRunStatus = await getWorkflowRunStatus();
     if (
       (showCardOnExit && !showCardOnFailure) ||
-      (showCardOnFailure && workflowRunStatus.conclusion !== "success")
+      (showCardOnFailure && workflowRunStatus?.conclusion !== "success")
     ) {
       formatAndNotify(
         "exit",
-        workflowRunStatus.conclusion,
-        workflowRunStatus.elapsedSeconds
+        workflowRunStatus?.conclusion,
+        workflowRunStatus?.elapsedSeconds
       );
     } else {
       info("Configured to not show card upon job exit.");

src/utils.ts

@@ -8,7 +8,6 @@ import { WebhookBody, PotentialAction } from "./models";
 import { formatCompactLayout } from "./layouts/compact";
 import { formatCozyLayout } from "./layouts/cozy";
 import { formatCompleteLayout } from "./layouts/complete";
-import { CONCLUSION_THEMES } from "./constants";
 
 export function escapeMarkdownTokens(text: string) {
   return text
@@ -96,34 +95,47 @@ export async function getWorkflowRunStatus() {
     run_id: parseInt(runInfo.runId || "1"),
   });
 
-  const job = workflowJobs.data.jobs.find(
-    (job: Octokit.ActionsListJobsForWorkflowRunResponseJobsItem) =>
-      job.name === process.env.GITHUB_JOB
-  );
-
-  let lastStep;
-  const stoppedStep = job?.steps.find(
-    (step: Octokit.ActionsListJobsForWorkflowRunResponseJobsItemStepsItem) =>
-      step.conclusion === "failure" ||
-      step.conclusion === "timed_out" ||
-      step.conclusion === "cancelled" ||
-      step.conclusion === "action_required"
-  );
-
-  if (stoppedStep) {
-    lastStep = stoppedStep;
-  } else {
-    lastStep = job?.steps
-      .reverse()
-      .find(
-        (
-          step: Octokit.ActionsListJobsForWorkflowRunResponseJobsItemStepsItem
-        ) => step.status === "completed"
-      );
-  }
+  let lastStep = {} as Octokit.ActionsListJobsForWorkflowRunResponseJobsItemStepsItem
+  let jobStartDate
+
+  /**
+   * We have to verify all jobs steps. We don't know
+   * if users are using multiple jobs or not. Btw,
+   * we don't need to check if GITHUB_JOB env is the 
+   * same of the Octokit job name, because it is different.
+   * 
+   * @note We are using a quadratic way to search all steps.
+   * But we have just a few elements, so this is not 
+   * a performance issue
+   * 
+   * The conclusion steps, according to the documentation, are:
+   * <success>, <cancelled>, <failure> and <skipped>
+   */
+  let abort = false
+  for(let job of workflowJobs.data.jobs) {
+    for(let step of job.steps) {
+      // check if current step still running
+      if (step.completed_at !== null) {
+        lastStep = step
+        jobStartDate = job.started_at
+        // Some step/job has failed. Get out from here.
+        if (step?.conclusion !== "success" && step?.conclusion !== "skipped") {
+            abort = true
+            break
+        }
+       /**  
+        * If nothing has failed, so we have a success scenario
+        * @note ignoring skipped cases. 
+        */
+        lastStep.conclusion = "success"
+      }
+    }
+    // // Some step/job has failed. Get out from here.
+    if (abort) break
+   }
+  const startTime = moment(jobStartDate, moment.ISO_8601);
+  const endTime = moment(lastStep?.completed_at, moment.ISO_8601); 
 
-  const startTime = moment(job?.started_at, moment.ISO_8601);
-  const endTime = moment(lastStep?.completed_at, moment.ISO_8601);
   return {
     elapsedSeconds: endTime.diff(startTime, "seconds"),
     conclusion: lastStep?.conclusion,
@@ -167,4 +179,4 @@ export function renderActions(statusUrl: string, diffUrl: string) {
     }
   }
   return actions;
-}
+}