Handle new job status, Add retry for api requests (#98)

New job status in GitHub Actions API are now handled by the action

@Octokit/plugin-retry is added to retry the API requests in case of failure

Closes #93
Closes #94

lib/github/jobs.test.ts

@@ -6,11 +6,14 @@ import { getCurrentJobs } from "./jobs";
 const listJobsForWorkflowRunAttempt = jest.fn();
 
 jest.mock("@octokit/rest", () => ({
-    Octokit: jest.fn().mockImplementation(() => ({
-        actions: {
-            listJobsForWorkflowRunAttempt
-        }
-    }))
+    Octokit: {
+        plugin: () =>
+            jest.fn().mockImplementation(() => ({
+                actions: {
+                    listJobsForWorkflowRunAttempt
+                }
+            }))
+    }
 }));
 
 beforeEach(() => {

lib/github/jobs.ts

@@ -1,9 +1,12 @@
 import { debug } from "@actions/core";
 import { components } from "@octokit/openapi-types/types.d";
 import { Octokit } from "@octokit/rest";
+import { retry } from "@octokit/plugin-retry";
 
 import { Context } from "./context";
 
+const MyOctokit = Octokit.plugin(retry);
+
 /**
  * Jobs of perceptual run
  */
@@ -30,7 +33,7 @@ export async function getCurrentJobs(token: string): Promise<Jobs> {
     // prettier-ignore
     const { runId: run_id, runAttempt: attempt_number, repo: { owner, repo } } = new Context();
     // prettier-ignore
-    const { actions: { listJobsForWorkflowRunAttempt } } = new Octokit({ auth: token });
+    const { actions: { listJobsForWorkflowRunAttempt } } = new MyOctokit({ auth: token });
     debug(`fetching jobs for /repos/${owner}/${repo}/actions/runs/${run_id}/attempts/${attempt_number}/jobs`);
     const { status, data } = await listJobsForWorkflowRunAttempt({
         attempt_number,

package.json

@@ -32,7 +32,9 @@
     "@actions/artifact": "^2.1.7",
     "@actions/core": "^1.10.1",
     "@actions/github": "^6.0.0",
-    "@octokit/rest": "^20.0.2"
+    "@octokit/plugin-retry": "^6.0.1",
+    "@octokit/rest": "^20.1.0",
+    "@octokit/core": "^5.2.0"
   },
   "devDependencies": {
     "@octokit/openapi-types": "^22.2.0",

src/Dependency.ts

@@ -37,8 +37,13 @@ export class Dependency {
                 info(`job "${name}" in progress ⌛ current step "${stepName}"`);
                 return false;
             case "queued":
+            case "requested":
                 info(`job "${name}" not started yet 👀`);
                 return false;
+            case "waiting":
+            case "pending":
+                info(`job "${name}" waiting for resources 🤔`);
+                return false;
             default:
                 // this should never happen
                 throw new Error(`error: unknown status "${status}" of job "${name}"`);