@jphutchins/code-review 0.1.0-alpha.27 → 0.1.0-alpha.28

package/dist/index.js

@@ -9,6 +9,7 @@ import { Ajv2020 } from 'ajv/dist/2020.js';
 import _addFormats from 'ajv-formats';
 import * as t from 'io-ts';
 import { execFile } from 'child_process';
+import { performance } from 'perf_hooks';
 import { PathReporter } from 'io-ts/lib/PathReporter.js';
 
 // src/cost.ts
@@ -1829,15 +1830,21 @@ var RunCodec = t.type({
   conclusion: t.union([t.string, t.null]),
   run_number: t.number
 });
-var RunsCodec = t.type({ workflow_runs: t.array(RunCodec) });
+var RUN_JQ = ".workflow_runs[] | {id: .id, name: .name, status: .status, conclusion: .conclusion, run_number: .run_number}";
 var resolveCiRun = async (repo, headSha, workflowName, ghApi) => {
-  const stdout = await ghApi([`repos/${repo}/actions/runs?head_sha=${headSha}&per_page=100`]);
-  const decoded = RunsCodec.decode(JSON.parse(stdout));
-  if (decoded._tag === "Left")
-    throw new Error(
-      `workflow runs for ${headSha} did not match the expected shape: ${PathReporter.report(decoded).join("; ")}`
+  const endpoint = `repos/${repo}/actions/runs?head_sha=${headSha}&per_page=100`;
+  const rows = parseJsonl(await ghApi([endpoint, "--paginate", "--jq", RUN_JQ]));
+  const decoded = rows.map((row) => RunCodec.decode(row));
+  const runs = decoded.flatMap((d) => d._tag === "Right" ? [d.right] : []);
+  const dropped = decoded.length - runs.length;
+  if (dropped > 0) {
+    const firstDrift = decoded.find((d) => d._tag === "Left");
+    const detail = firstDrift === void 0 ? "" : ` (${PathReporter.report(firstDrift).join("; ")})`;
+    process.stderr.write(
+      `Warning: ${String(dropped)} of ${String(rows.length)} workflow-run row(s) from ${endpoint} failed to decode${detail} \u2014 excluded from the lookup
+`
     );
-  const runs = decoded.right.workflow_runs;
+  }
   const latest = runs.filter((r) => r.name === workflowName).reduce(
     (best, r) => best === null || r.run_number > best.run_number ? r : best,
     null
@@ -1848,21 +1855,34 @@ var resolveCiRun = async (repo, headSha, workflowName, ghApi) => {
   };
 };
 var awaitCiConclusion = async (repo, headSha, options, deps = { ghApi: runGhApi, sleep: defaultSleep, elapsedMs: monotonicElapsed() }) => {
-  const poll = async () => {
-    const { run, seenNames } = await resolveCiRun(repo, headSha, options.workflowName, deps.ghApi);
-    if (run !== null && run.status === "completed")
-      return { kind: "concluded", conclusion: run.conclusion ?? "unknown", runId: run.id };
+  const safeResolve = async () => {
+    try {
+      return await resolveCiRun(repo, headSha, options.workflowName, deps.ghApi);
+    } catch (err) {
+      process.stderr.write(
+        `Warning: CI-run lookup for ${headSha} failed (${errMsg(err)}) \u2014 retrying until the timeout
+`
+      );
+      return { run: null, seenNames: [] };
+    }
+  };
+  const poll = async (lastSeenNames, lastRunId) => {
+    const { run, seenNames } = await safeResolve();
+    if (run !== null && run.status === "completed" && run.conclusion !== null)
+      return { kind: "concluded", conclusion: run.conclusion, runId: run.id };
+    const runId = run === null ? lastRunId : run.id;
+    const names = seenNames.length > 0 ? seenNames : lastSeenNames;
     if (deps.elapsedMs() >= options.timeoutMs)
-      return { kind: "timed-out", runId: run === null ? null : run.id, seenNames };
+      return { kind: "timed-out", runId, seenNames: names };
     await deps.sleep(options.pollIntervalMs);
-    return poll();
+    return poll(names, runId);
   };
-  return poll();
+  return poll([], null);
 };
 var defaultSleep = (ms) => new Promise((resolvePromise) => setTimeout(resolvePromise, ms));
 var monotonicElapsed = () => {
-  const start = Date.now();
-  return () => Date.now() - start;
+  const start = performance.now();
+  return () => performance.now() - start;
 };
 var renderCiOutputs = (outcome) => outcome.kind === "concluded" ? `ci_settled=true
 ci_conclusion=${outcome.conclusion}