@bakers-build/addon-forge-github 0.9.1-next.8 → 0.10.0

package/dist/src/index.js

@@ -3,7 +3,7 @@
 // shortly before it expires; or, with `auth: 'gh'`, the conductor's own `gh`
 // login. The repository comes from config, never from a clone's `origin`.
 import { createSign } from 'node:crypto';
-import { cliToken, requestJson, requestText, tail, } from '@bakers-build/addon-forge/backend';
+import { cliToken, requestJson, } from '@bakers-build/addon-forge/backend';
 const API = 'https://api.github.com';
 const REFRESH_MS = 60_000;
 function settingsOf(raw) {
@@ -64,7 +64,6 @@ function stateStatus(state) {
         return 'success';
     return state === 'pending' ? 'pending' : 'failure';
 }
-const NO_RUN = { run_id: null, job_id: null, run_started_at: null };
 const basic = (token) => Buffer.from(`x-access-token:${token}`).toString('base64');
 const factory = async (raw, context) => {
     const s = settingsOf(raw);
@@ -102,7 +101,7 @@ const factory = async (raw, context) => {
         };
         return out.token;
     };
-    const api = async (path, init = {}) => (init.text ? requestText : requestJson)(doFetch, `${repo}${path}`, {
+    const api = async (path, init = {}) => requestJson(doFetch, `${repo}${path}`, {
         method: init.method ?? 'GET',
         headers: {
             authorization: `Bearer ${await accessToken()}`,
@@ -136,11 +135,11 @@ const factory = async (raw, context) => {
         openPrsWithBase: async (base) => (await api(`/pulls?state=open&base=${encodeURIComponent(base)}&per_page=100`))
             .filter((p) => p.base.ref === base)
             .map(prOf),
-        checks: async (ref, base, opts = {}) => {
+        checks: async (ref, base) => {
             const at = encodeURIComponent(ref);
             const b = encodeURIComponent(base);
             const [runs, status, rules, branch] = await Promise.all([
-                api(`/commits/${at}/check-runs?filter=all&per_page=100`),
+                api(`/commits/${at}/check-runs?per_page=100`),
                 api(`/commits/${at}/status`),
                 api(`/rules/branches/${b}`, { allow404: true }),
                 api(`/branches/${b}`, { allow404: true }),
@@ -152,67 +151,21 @@ const factory = async (raw, context) => {
                     .map((c) => c.context),
                 ...(branch?.protection?.required_status_checks?.contexts ?? []),
             ]);
-            // shortcut: one page only; a partial list could hide a newer attempt, so refuse it; paginate when this trips.
-            if ((runs.total_count ?? 0) > runs.check_runs.length)
-                throw new Error(`${ref} has ${runs.total_count} check runs, more than the ${runs.check_runs.length} one page holds; the result would be partial`);
-            // `ref` may be a branch: the runs lookup needs the commit the check runs report.
-            const sha = runs.check_runs[0]?.head_sha;
-            const workflowRuns = sha
-                ? (await api(`/actions/runs?head_sha=${encodeURIComponent(sha)}&per_page=100`)).workflow_runs
-                : [];
-            const bySuite = new Map(workflowRuns.map((w) => [w.check_suite_id, w]));
-            const newest = new Map();
-            for (const w of workflowRuns) {
-                const kept = newest.get(w.workflow_id);
-                if (!kept ||
-                    w.run_started_at > kept.run_started_at ||
-                    (w.run_started_at === kept.run_started_at && w.id > kept.id))
-                    newest.set(w.workflow_id, w);
-            }
-            let jobs = runs.check_runs.map((r) => ({
-                r,
-                w: r.check_suite ? bySuite.get(r.check_suite.id) : undefined,
-            }));
-            if (!opts.allRuns) {
-                // An Actions check run without a listed run belongs to an earlier attempt.
-                jobs = jobs.filter(({ r, w }) => w
-                    ? newest.get(w.workflow_id) === w
-                    : r.app?.slug !== 'github-actions');
-                // A re-run attempt is a newer check run of the same name in the same run.
-                const latest = new Map();
-                const key = (j) => `${j.w?.id ?? ''}\0${j.r.name}`;
-                for (const j of jobs)
-                    latest.set(key(j), Math.max(latest.get(key(j)) ?? 0, j.r.id));
-                jobs = jobs.filter((j) => latest.get(key(j)) === j.r.id);
-            }
             const rows = [
-                ...jobs.map(({ r, w }) => ({
+                ...runs.check_runs.map((r) => ({
                     name: r.name,
                     status: runStatus(r),
                     required: required.has(r.name),
-                    conclusion: r.conclusion,
-                    run_id: w?.id ?? null,
-                    // An Actions job's id is its check run's id.
-                    job_id: w ? r.id : null,
-                    run_started_at: w?.run_started_at ?? null,
                 })),
                 ...status.statuses.map((st) => ({
                     name: st.context,
                     status: stateStatus(st.state),
                     required: required.has(st.context),
-                    conclusion: st.state,
-                    ...NO_RUN,
                 })),
             ];
             for (const name of required)
                 if (!rows.some((r) => r.name === name))
-                    rows.push({
-                        name,
-                        status: 'missing',
-                        required: true,
-                        conclusion: null,
-                        ...NO_RUN,
-                    });
+                    rows.push({ name, status: 'missing', required: true });
             return rows;
         },
         mergeBlocker: async (pr) => {
@@ -237,16 +190,6 @@ const factory = async (raw, context) => {
                 });
         },
         isMerged: async (pr) => (await pull(pr.number)).merged === true,
-        // The API redirects to a signed download URL; fetch follows it.
-        jobLog: async (jobId, maxBytes) => {
-            const log = (await api(`/actions/jobs/${jobId}/logs`, {
-                allow404: true,
-                text: true,
-            }));
-            if (log === undefined)
-                throw new Error(`job ${jobId} is not a job of ${s.repository}`);
-            return tail(log, maxBytes);
-        },
     };
     return backend;
 };

package/options.schema.json

@@ -5,7 +5,9 @@
   "description": "The forge entry's with.backend.with: a GitHub App, or the conductor's gh login with auth 'gh'.",
   "type": "object",
   "additionalProperties": false,
-  "required": ["repository"],
+  "required": [
+    "repository"
+  ],
   "properties": {
     "repository": {
       "type": "string",
@@ -17,12 +19,18 @@
       "description": "Use the conductor's own gh login instead of a GitHub App."
     },
     "app_id": {
-      "type": ["integer", "string"],
+      "type": [
+        "integer",
+        "string"
+      ],
       "minLength": 1,
       "description": "The GitHub App's ID; required unless `auth` is set."
     },
     "installation_id": {
-      "type": ["integer", "string"],
+      "type": [
+        "integer",
+        "string"
+      ],
       "minLength": 1,
       "description": "The App's installation on the repository; required unless `auth` is set."
     },
@@ -38,9 +46,14 @@
     }
   },
   "if": {
-    "required": ["auth"]
+    "required": [
+      "auth"
+    ]
   },
   "else": {
-    "required": ["app_id", "installation_id"]
+    "required": [
+      "app_id",
+      "installation_id"
+    ]
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bakers-build/addon-forge-github",
-  "version": "0.9.1-next.8",
+  "version": "0.10.0",
   "description": "bakers.build forge backend for GitHub: a GitHub App installation token scoped to one repository. Implements @bakers-build/addon-forge's ForgeBackend contract.",
   "type": "module",
   "license": "MIT",
@@ -21,11 +21,11 @@
     "directory": "bakers-cli/addons/forge-github"
   },
   "peerDependencies": {
-    "@bakers-build/addon-forge": "^0.9.1-next.8",
-    "@bakers-build/core": "^0.9.1-next.8"
+    "@bakers-build/addon-forge": "^0.10.0",
+    "@bakers-build/core": "^0.10.0"
   },
   "bakersBuild": {
-    "commit": "41eb5631e4ddf107b0b7a5ce2ffe2ed2a6052b5e",
-    "channel": "next"
+    "commit": "5857b00008bc0926fd52fdc8d9366d6d52444e35",
+    "channel": "latest"
   }
 }