@jphutchins/code-review 0.1.0-alpha.50 → 0.1.0-alpha.52

package/README.md

@@ -37,7 +37,7 @@ npx @jphutchins/code-review <subcommand>
 
 | Subcommand | What it does |
 | --- | --- |
-| `post` | Post a complete review (inline comments + sticky summary) from findings + envelope + diff — the one-call path |
+| `post` | Post a complete review from findings + envelope + diff — the one-call path. The findings are listed in the sticky summary; `--inline` renders the in-diff ones as comments on the diff instead. Also renders the review into `$GITHUB_STEP_SUMMARY`: the sticky is overwritten each round, the run summary keeps each round as it stood |
 | `gather` | Resolve the PR from the CI head SHA and gather the review inputs (diff with git-diff fallback, PR context, prior bot review, the prior review's answered-findings registry, failing-job logs) into the workspace for the agent |
 | `parse-command` | Resolve a PR's head from its number and parse a ChatOps trigger comment (`/code-review [24m] [$1.00] <instructions>`) into review overrides — the on-demand comment trigger |
 | `react` | Add/remove a GitHub comment reaction — the ChatOps acknowledgement (👀 on receipt, 🚀 on completion) |
@@ -96,8 +96,10 @@ links to:
 
 - **`code-review-findings`** — the findings JSON + result envelope the comment job renders. The
   sticky comment embeds this same JSON directly, base64-encoded, in an
-  `<!-- code-review:findings-json;base64 <base64> -->` HTML comment — a reviewing agent (or any
-  downstream tool) SHOULD base64-decode and parse that marker rather than parse the comment's prose.
+  `<!-- code-review:findings-json;base64 <base64> -->` HTML comment. The comment usually renders that
+  same review as prose, which is the cheaper read; decode the marker where the prose is not the
+  review, where the prose carries only part of it, or where the prose omits a field you need. The directive that rides ahead of the
+  marker states that rule.
   Embedding in the comment (rather than only linking the artifact) keeps the pointer from expiring
   with artifact retention; when the encoded findings are too large to embed, the sticky falls back to
   a `<!-- code-review:findings-json <url> -->` link marker instead. The shared serializer is
@@ -152,7 +154,7 @@ links to:
   mechanism frequencies carried in `convergence.rounds` (each entry's `codes` map), from which the
   re-review seed re-derives the advisory `scope_metastasis` entry it hands the next-round agent. Each
   inline review comment embeds only its own finding (a `schema_version` + one-finding fragment), and the
-  review-object body only links the sticky — so the **sticky is the sole documented decode surface** for
+  review-object body links the sticky and the workflow run — so the **sticky is the sole documented decode surface** for
   the whole-document marker; a decoding agent reads it there. The review body is written only after the
   sticky exists (a failed sticky write aborts the run first), so it never carries the blob itself.
 - **`code-review-transcript`** — the full Claude Code session transcripts for the triage and review

package/dist/index.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import { defineCommand, runMain } from 'citty';
-import { readFileSync, writeFileSync, copyFileSync, statSync, readdirSync } from 'fs';
+import { readFileSync, writeFileSync, copyFileSync, statSync, readdirSync, appendFileSync } from 'fs';
 import { randomBytes } from 'crypto';
 import { resolve as resolve$1, join, dirname, basename, extname } from 'path';
 import { Eta } from 'eta';
@@ -282,9 +282,16 @@ var NonEmptyPriceSlots = t.refinement(
   (a) => a.length >= 1,
   "NonEmptyPriceSlots"
 );
+var SlottedRequired = t.type({ slots: NonEmptyPriceSlots });
+var SlottedOptional = t.partial({ weekend_slots: NonEmptyPriceSlots });
+var SlottedShape = t.intersection([SlottedRequired, SlottedOptional]);
+var SLOTTED_KEYS = /* @__PURE__ */ new Set([
+  ...Object.keys(SlottedRequired.props),
+  ...Object.keys(SlottedOptional.props)
+]);
 var SlottedModelPricesStrict = t.refinement(
-  t.type({ slots: NonEmptyPriceSlots }),
-  (s) => Object.keys(s).every((k) => k === "slots"),
+  SlottedShape,
+  (s) => Object.keys(s).every((k) => SLOTTED_KEYS.has(k)),
   "SlottedModelPricesStrict"
 );
 var SlottedModelPricesCodec = t.exact(SlottedModelPricesStrict);
@@ -339,6 +346,11 @@ var slotCovers = (slot, minute) => {
   const to = hhmmToMinutes(slot.utc_to);
   return from < to ? minute >= from && minute < to : minute >= from || minute < to;
 };
+var BEIJING_OFFSET_MINUTES = 8 * 60;
+var isBeijingWeekend = (instant) => {
+  const day = new Date(instant.getTime() + BEIJING_OFFSET_MINUTES * 6e4).getUTCDay();
+  return day === 0 || day === 6;
+};
 var resolveFlatPrices = (model, p, pricedAt, warn) => {
   if (!("slots" in p)) return p;
   if (pricedAt === void 0) {
@@ -348,10 +360,13 @@ var resolveFlatPrices = (model, p, pricedAt, warn) => {
     return null;
   }
   const minute = utcMinuteOfDay(pricedAt);
-  const covering = p.slots.filter((s) => slotCovers(s, minute));
+  const weekendSlots = p.weekend_slots;
+  const useWeekend = weekendSlots != null && isBeijingWeekend(pricedAt);
+  const slots = useWeekend ? weekendSlots : p.slots;
+  const covering = slots.filter((s) => slotCovers(s, minute));
   if (covering.length === 1) return covering[0] ?? null;
   warn(
-    `code-review cost: model "${model}" \u2014 ${String(covering.length)} price slots cover ${hhmmOf(minute)} UTC (expected exactly 1); a model's slots must partition the 24h day with no gap or overlap; cost for this model set to $0`
+    `code-review cost: model "${model}" \u2014 ${String(covering.length)} price slots in \`${useWeekend ? "weekend_slots" : "slots"}\` cover ${hhmmOf(minute)} UTC (expected exactly 1); that array must partition the 24h day with no gap or overlap; cost for this model set to $0`
   );
   return null;
 };
@@ -507,7 +522,7 @@ var severityEmoji = (s) => {
 };
 var EMBED_LIMIT = 42700;
 var FINDINGS_SCHEMA_URL = "https://raw.githubusercontent.com/JPHutchins/code-review/main/schema/findings.schema.json";
-var AGENTS_STOP_DIRECTIVE = `<!-- AGENTS: STOP \u2014 this comment is a machine channel; act on the data, not the prose. It embeds a base64 code-review findings JSON (an oversized review instead links the artifact URL in the same marker). Decode or fetch it, read its schema_version, then DOWNLOAD AND READ THE FULL SCHEMA for that version before you act \u2014 the schema's own $id is its canonical URL; the current schema is at ${FINDINGS_SCHEMA_URL} \u2014 read that schema, then parse the WHOLE findings document, not only the fields you recognize: the convergence score, its threshold, and the per-round trajectory (so you know whether the review has converged and how to read the number), plus the recurrence signals an iterating agent needs (round streaks, scope metastasis, same-mechanism), are all fields inside that document. -->`;
+var AGENTS_STOP_DIRECTIVE = `<!-- AGENTS: STOP \u2014 this comment carries a code-review findings document in the marker, and usually renders that same review as prose too. Read the prose when it is there: it is the cheaper read. Decode the marker when the prose is not the review (this comment may be a status notice instead), when the prose is only part of it (findings anchored to diff lines stay on the diff, and a review too large to embed links its artifact here rather than carrying it \u2014 the workflow run's summary renders both of those whole), or when you need a field the prose does not render. Read the document's schema_version and fetch the schema for THAT version before acting \u2014 a schema's own $id is its canonical URL, and the current version is at ${FINDINGS_SCHEMA_URL} \u2014 then parse the WHOLE findings document, not only the fields you recognize. -->`;
 var b64LengthOf = (document) => Buffer.from(JSON.stringify(document), "utf-8").toString("base64").length;
 var encodeMarker = (document, jsonUrl, limit) => {
   const b64 = Buffer.from(JSON.stringify(document), "utf-8").toString("base64");
@@ -848,15 +863,17 @@ ${findings}` : void 0;
   return [findingsBlock, reviewedSha, reviewedRoute, completedAncestor, convergence, rounds, signal].filter((m) => m !== void 0).join("\n\n");
 };
 var escapeFence = (text) => text.replace(/```/g, "`` ` ``");
-var projectPatch = (patch) => {
+var projectPatch = (patch, surface) => {
   if (patch === void 0) return { kind: "none" };
-  const lowered = patchToSuggestion(patch);
+  const lowered = surface === "diff-anchored" ? patchToSuggestion(patch) : void 0;
   return typeof lowered === "string" ? { kind: "suggestion", text: escapeFence(lowered) } : { kind: "patch", raw: escapeFence(patch) };
 };
 var formatConfidence = (n) => n.toFixed(2);
-var reviewBodyPointer = (headSha, stickyUrl) => {
+var reviewBodyPointer = (headSha, stickyUrl, runUrl, runHasSummary) => {
   const sha7 = headSha.slice(0, 7);
-  return stickyUrl ? `\u{1F916} Automated code review for \`${sha7}\` \u2014 see the [summary comment](${stickyUrl}) for the verdict, walkthrough, and cost.` : `\u{1F916} Automated code review for \`${sha7}\` \u2014 see the summary comment for the verdict, walkthrough, and cost.`;
+  const summary = stickyUrl ? `the [summary comment](${stickyUrl})` : "the summary comment";
+  const run = runUrl ? ` See the [workflow run](${runUrl}) for ${runHasSummary ? "this round's review in full, its job log," : "the job log"} and the findings artifact.` : "";
+  return `\u{1F916} Automated code review for \`${sha7}\` \u2014 see ${summary} for the verdict, walkthrough, and cost.${run}`;
 };
 var asRecord = (u) => typeof u === "object" && u !== null && !Array.isArray(u) ? u : null;
 var errMsg = (e) => e instanceof Error ? e.message : String(e);
@@ -1195,7 +1212,7 @@ var sanitizeFinding = (f, answeredNotes) => {
     ...f,
     title: escapePipes(f.title),
     path: escapeCodeBackticks(f.path),
-    patchProjection: projectPatch(f.patch),
+    patchProjection: projectPatch(f.patch, "comment-body"),
     answeredNote: answeredNotes !== void 0 && Object.prototype.hasOwnProperty.call(answeredNotes, key2) ? answeredNotes[key2] ?? "" : ""
   };
 };
@@ -1267,6 +1284,7 @@ var render = (input) => {
     systemic: (input.findings.systemic_problems ?? []).map(sanitizeSystemic),
     unanchoredCount: input.unanchoredCount ?? 0,
     inlineDisposition: input.inlineDisposition ?? null,
+    unverifiedNoLogs: input.unverifiedNoLogs === true,
     runUrl: input.runUrl ?? null,
     jsonUrl: input.jsonUrl ?? null,
     // The blob is the agent's complete document with the pipeline-stamped convergence field inside it
@@ -1365,7 +1383,7 @@ var renderCommentBody = (f, eta, template, modelsText, jsonUrl, pointer, sameRoo
   // eslint-disable-next-line @typescript-eslint/no-unnecessary-type-assertion
   eta.renderString(template, {
     ...f,
-    patchProjection: projectPatch(f.patch),
+    patchProjection: projectPatch(f.patch, "diff-anchored"),
     severityEmoji,
     formatConfidence,
     modelsText,
@@ -2273,31 +2291,43 @@ var commentPayload = (c) => ({
   ...c.start_line !== void 0 && c.start_side !== void 0 ? { start_line: c.start_line, start_side: c.start_side } : {},
   body: formatMarkdown(c.body)
 });
-var postInlineReview = async (repo, prNumber, headSha, comments, inDiff, stickyUrl, ghApi) => {
-  const pointer = reviewBodyPointer(headSha, stickyUrl);
+var postInlineReview = async (pr, comments, inDiff, ghApi) => {
+  const pointer = reviewBodyPointer(
+    pr.headSha,
+    pr.stickyUrl,
+    pr.runUrl,
+    (process.env["GITHUB_STEP_SUMMARY"] ?? "") !== ""
+  );
   const reviewBody = (withComments) => JSON.stringify({
     body: pointer,
-    commit_id: headSha,
+    commit_id: pr.headSha,
     event: "COMMENT",
     comments: withComments ? comments.map(commentPayload) : []
   });
-  const reviewsEndpoint = [`repos/${repo}/pulls/${String(prNumber)}/reviews`, "--input", "-"];
+  const reviewsEndpoint = [`repos/${pr.repo}/pulls/${String(pr.prNumber)}/reviews`, "--input", "-"];
   try {
     const stdout = await ghApi(reviewsEndpoint, reviewBody(true));
     return { url: parseHtmlUrl(stdout), inlinePosted: comments.length, unposted: [] };
   } catch (err) {
     if (comments.length === 0) throw err;
     process.stderr.write(
-      `Warning: the batched inline review on PR #${String(prNumber)} was rejected (${errMsg(err)}) \u2014 posting the review body-only, then each comment individually to keep the ones GitHub accepts (issue #57)
+      `Warning: the batched inline review on PR #${String(pr.prNumber)} was rejected (${errMsg(err)}) \u2014 posting the review body-only, then each comment individually to keep the ones GitHub accepts (issue #57)
 `
     );
     const url = parseHtmlUrl(await ghApi(reviewsEndpoint, reviewBody(false)));
-    const commentsEndpoint = [`repos/${repo}/pulls/${String(prNumber)}/comments`, "--input", "-"];
+    const commentsEndpoint = [
+      `repos/${pr.repo}/pulls/${String(pr.prNumber)}/comments`,
+      "--input",
+      "-"
+    ];
     const unposted = [];
     let inlinePosted = 0;
     for (const [i, c] of comments.entries()) {
       try {
-        await ghApi(commentsEndpoint, JSON.stringify({ commit_id: headSha, ...commentPayload(c) }));
+        await ghApi(
+          commentsEndpoint,
+          JSON.stringify({ commit_id: pr.headSha, ...commentPayload(c) })
+        );
         inlinePosted += 1;
       } catch (e) {
         const finding = inDiff[i];
@@ -2351,6 +2381,18 @@ var postComment = async (repo, prNumber, body, ghApi) => {
   );
   return parseCommentRef(stdout);
 };
+var appendRunSummary = (summaryPath, body) => {
+  if (summaryPath === void 0 || summaryPath === "") return;
+  const rendered = body();
+  try {
+    appendFileSync(summaryPath, `
+${rendered}
+`);
+  } catch (err) {
+    process.stderr.write(`Warning: could not write the run summary: ${errMsg(err)}
+`);
+  }
+};
 var upsertSticky = async (repo, prNumber, existing, body, ghApi) => {
   if (existing !== null) {
     const patched = await patchComment(repo, existing.id, body, ghApi);
@@ -2388,6 +2430,7 @@ var dismissReviews = async (repo, prNumber, ids, ghApi) => {
           "--input",
           "-"
         ],
+        // GitHub caps a dismissal message at 140 chars.
         JSON.stringify({ message: "Superseded by a new review for an updated commit." })
       );
     } catch (err) {
@@ -2545,7 +2588,7 @@ ${conv}`;
       noticeBody(
         `${DEFAULT_MARKER}
 
-\u26A0\uFE0F **CI-fix pass completed with no findings** for \`${input.headSha.slice(0, 7)}\` \u2014 the completed full review of \`${priorSha ? priorSha.slice(0, 7) : "an earlier commit"}\` is preserved below.${dropNote ? `
+\u26A0\uFE0F **CI-fix pass completed with no findings** for \`${input.headSha.slice(0, 7)}\`${input.unverifiedNoLogs === true ? ' \u2014 **but no failing-job logs were available**, so it had only the diff to work from and "no findings" is not evidence of none' : ""} \u2014 the completed full review of \`${priorSha ? priorSha.slice(0, 7) : "an earlier commit"}\` is preserved below.${dropNote ? `
 
 ${dropNote}` : ""}`,
         sticky.body
@@ -2560,7 +2603,7 @@ ${dropNote}` : ""}`,
     throw new Error(`Price map at ${input.pricesPath} does not match the expected shape`);
   }
   const template = readFileSync(input.templatePath, "utf-8");
-  const inlineTemplate = readFileSync(input.inlineTemplatePath, "utf-8");
+  const inlineRequested = input.inline === true;
   const renderNotice = (message) => {
     const findings2 = stampConvergence(incompleteFindings(`### \u26A0\uFE0F ${message}`), priorConv);
     return formatMarkdown(
@@ -2677,19 +2720,27 @@ ${dropNote}` : ""}`,
         convergenceRound: false,
         testReport,
         clocDiff,
-        inlineDisposition: { kind: "no-envelope" },
+        inlineDisposition: inlineRequested ? { kind: "no-envelope" } : { kind: "disabled" },
         runUrl: input.runUrl,
+        unverifiedNoLogs: input.unverifiedNoLogs,
         jsonUrl: input.jsonUrl,
         findingsPointer: findingsBlob(stampedFindings2),
         postedAt: input.postedAt
       })
     );
     await upsertSticky(input.repo, prNumber, existingSticky, body, ghApi);
+    appendRunSummary(process.env["GITHUB_STEP_SUMMARY"], () => body);
+    if (inlineRequested) {
+      process.stderr.write(
+        "Warning: inline: true was requested, but the result envelope is missing \u2014 inline comments cannot be built; the findings are in the sticky and the run summary instead\n"
+      );
+    }
     process.stderr.write(
-      "Result envelope missing or malformed \u2014 posted sticky summary without usage/cost data; no inline review\n"
+      "Result envelope missing or malformed \u2014 posted sticky summary without usage/cost data\n"
     );
     process.exit(0);
   }
+  const inlineTemplate = inlineRequested ? readFileSync(input.inlineTemplatePath, "utf-8") : "";
   const thisIncomplete = envelope.incomplete === true || isIncompleteFindings(findings);
   if (wouldBuryCompleted(thisIncomplete)) {
     logAnsweredDrops();
@@ -2703,14 +2754,14 @@ ${dropNote}` : ""}`,
     comments: rawComments,
     strays,
     inDiff
-  } = buildInlineComments(visibleFindings, diff, {
+  } = inlineRequested ? buildInlineComments(visibleFindings, diff, {
     inlineTemplate,
     models: envelope.models.map((m) => m.model),
     findings,
     jsonUrl: input.jsonUrl,
     sameRootNotes,
     answeredNotes: reRaisedNotes
-  });
+  }) : { comments: [], strays: visibleFindings, inDiff: [] };
   const { comments, longFiles } = checkLongSuggestions(rawComments);
   for (const wf of longFiles) {
     process.stderr.write(
@@ -2719,7 +2770,7 @@ ${dropNote}` : ""}`,
     );
   }
   const botReviews = await fetchBotReviews(input.repo, prNumber, input.botLogin, ghApi);
-  const initialDisposition = comments.length === 0 && strays.length > 0 ? { kind: "none-in-diff" } : void 0;
+  const initialDisposition = !inlineRequested ? { kind: "disabled" } : comments.length === 0 && strays.length > 0 ? { kind: "none-in-diff" } : void 0;
   const currentCounts = computeSeverityCounts(findings.findings);
   const currentCodes = computeCodeCounts(findings.findings, findings.systemic_problems ?? []);
   const roundNumber = priorRoundCount + 1;
@@ -2767,6 +2818,7 @@ ${dropNote}` : ""}`,
     strays,
     suppressedNits,
     runUrl: input.runUrl,
+    unverifiedNoLogs: input.unverifiedNoLogs,
     jsonUrl: input.jsonUrl,
     findingsPointer: findingsMarker,
     postedAt: input.postedAt,
@@ -2805,16 +2857,20 @@ ${dropNote}` : ""}`,
     inlinePosted,
     unposted
   } = await postInlineReview(
-    input.repo,
-    prNumber,
-    input.headSha,
+    {
+      repo: input.repo,
+      prNumber,
+      headSha: input.headSha,
+      stickyUrl: stickyRef?.url,
+      runUrl: input.runUrl
+    },
     comments,
     inDiff,
-    stickyRef?.url,
     ghApi
   );
   process.stderr.write(
-    `Posted a review with ${String(inlinePosted)} inline comment(s) on PR #${String(prNumber)}
+    inlineRequested ? `Posted a review with ${String(inlinePosted)} inline comment(s) on PR #${String(prNumber)}
+` : `Posted a body-only review on PR #${String(prNumber)}; the findings are in the sticky
 `
   );
   const priorReviewIds = botReviews.map((r) => r.id);
@@ -2844,6 +2900,14 @@ ${dropNote}` : ""}`,
       );
     }
   }
+  appendRunSummary(
+    process.env["GITHUB_STEP_SUMMARY"],
+    () => renderBody(
+      { kind: "whole-document", inlineCount: inlinePosted, rejectedCount: unanchoredCount },
+      reviewUrl,
+      visibleFindings
+    )
+  );
 };
 var noticeBody = (lead, existingBody) => {
   const carried = existingBody ? carryForwardMarkers(existingBody) : "";
@@ -3216,6 +3280,8 @@ conclusion=${result.conclusion}
 diff_size=${String(result.diffSize)}
 stacked=${String(result.stacked)}
 base_sha=${result.baseSha}
+staged_job_logs=${String(result.stagedJobLogs)}
+failing_jobs=${String(result.failingJobs)}
 `;
   }
 };
@@ -3244,7 +3310,7 @@ var IssueCommentCodec = t.intersection([
   })
 ]);
 var JobCodec = t.type({ id: t.number, conclusion: t.union([t.string, t.null]) });
-var JobsResponseCodec = t.type({ jobs: t.array(JobCodec) });
+var JOBS_JQ = ".jobs[] | {id: .id, conclusion: .conclusion}";
 var fetchPrMeta = async (repo, prNumber, ghApi) => {
   const stdout = await ghApi([
     `repos/${repo}/pulls/${String(prNumber)}`,
@@ -3391,16 +3457,23 @@ var reviewsFrom = (reviews, botLogin) => boundedHuman(reviews, botLogin, "review
   state: r.state ?? null,
   body: clip(r.body)
 }));
+var MAX_STAGED_JOB_LOGS = 20;
 var downloadFailingJobLogs = async (repo, runId, outDir, ghApi) => {
-  const stdout = await ghApi([`repos/${repo}/actions/runs/${runId}/jobs`]);
-  const decoded = JobsResponseCodec.decode(JSON.parse(stdout));
+  const rows = parseJsonl(
+    await ghApi([`repos/${repo}/actions/runs/${runId}/jobs`, "--paginate", "--jq", JOBS_JQ])
+  );
+  const decoded = t.array(JobCodec).decode(rows);
   if (decoded._tag === "Left") {
     throw new Error(`Jobs list for run ${runId} did not match the expected shape`);
   }
-  for (const job of decoded.right.jobs.filter((j) => j.conclusion === "failure")) {
+  const failing = decoded.right.filter((j) => j.conclusion === "failure");
+  const selected = failing.slice(0, MAX_STAGED_JOB_LOGS);
+  let staged = 0;
+  for (const job of selected) {
     try {
       const log = await ghApi([`repos/${repo}/actions/jobs/${String(job.id)}/logs`]);
       writeFileSync(join(outDir, `job_${String(job.id)}.log`), log);
+      staged += 1;
     } catch (err) {
       process.stderr.write(
         `Warning: failed to download logs for job ${String(job.id)}: ${errMsg(err)} \u2014 continuing with the logs retrieved so far
@@ -3408,6 +3481,19 @@ var downloadFailingJobLogs = async (repo, runId, outDir, ghApi) => {
       );
     }
   }
+  if (failing.length > selected.length) {
+    process.stderr.write(
+      `::warning::${annotationSafe(`${String(failing.length)} failing job(s) in run ${runId}; staged ${String(staged)} of the first ${String(selected.length)} log(s) \u2014 the review does not see the rest`)}
+`
+    );
+  }
+  if (staged === 0) {
+    process.stderr.write(
+      `::warning::${annotationSafe(`No failing-job logs could be staged for run ${runId} (${String(failing.length)} failing job(s) reported) \u2014 the review has only the diff to work from`)}
+`
+    );
+  }
+  return { staged, failing: failing.length };
 };
 var gather = async (input, ghApi = runGhApi, gitRun = runGit) => {
   const candidates = await fetchPrCandidates(input.repo, input.headSha, ghApi);
@@ -3477,16 +3563,16 @@ var gather = async (input, ghApi = runGhApi, gitRun = runGit) => {
       reviews: reviews === null ? [] : reviewsFrom(reviews, input.botLogin)
     })
   );
-  if (input.conclusion === "failure") {
-    await downloadFailingJobLogs(input.repo, input.runId, input.outDir, ghApi);
-  }
+  const jobLogs = input.conclusion === "failure" ? await downloadFailingJobLogs(input.repo, input.runId, input.outDir, ghApi) : { staged: 0, failing: 0 };
   return {
     kind: "gathered",
     pr: prNumber,
     conclusion: input.conclusion,
     diffSize: Buffer.byteLength(prDiff, "utf8"),
     stacked,
-    baseSha: meta.base_sha
+    baseSha: meta.base_sha,
+    stagedJobLogs: jobLogs.staged,
+    failingJobs: jobLogs.failing
   };
 };
 
@@ -5146,7 +5232,7 @@ var postCmd = defineCommand({
     },
     "run-url": {
       type: "string",
-      description: "Workflow run URL (transcript/traces), rendered as a link in the LLM Disclosure aside"
+      description: "Workflow run URL (transcript/traces), rendered as a link in the LLM Disclosure aside and in the review-object body"
     },
     "json-url": {
       type: "string",
@@ -5159,6 +5245,14 @@ var postCmd = defineCommand({
     "nit-visibility-floor": {
       type: "string",
       description: NIT_VISIBILITY_FLOOR_DESCRIPTION
+    },
+    inline: {
+      type: "boolean",
+      description: "Also render findings as inline review comments on the diff. Off by default: an inline thread cannot be revised by a later round, so stale threads accumulate. The review object is posted either way; with this off the sticky lists the findings instead"
+    },
+    "unverified-no-logs": {
+      type: "boolean",
+      description: "Mark the review unverified: the fast-fix route ran with no failing-job logs staged, so its findings came from the diff alone. The caller decides this \u2014 the logs are staged in the review job, not here"
     }
   },
   run: async ({ args }) => {
@@ -5182,6 +5276,8 @@ var postCmd = defineCommand({
       jsonUrl: args["json-url"],
       convergenceThreshold: parseConvergenceThreshold(args["convergence-threshold"]),
       nitVisibilityFloor: parseNitVisibilityFloor(args["nit-visibility-floor"]),
+      inline: args.inline,
+      unverifiedNoLogs: args["unverified-no-logs"],
       postedAt: formatUtc(/* @__PURE__ */ new Date()),
       pricedAt: /* @__PURE__ */ new Date()
     });