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

package/dist/index.js

@@ -195,6 +195,13 @@ var parseFindingsMarker = (body) => {
     return null;
   }
 };
+var carryForwardMarkers = (body) => {
+  const findings = /<!-- code-review:findings-json[^>]*-->/.exec(body)?.[0];
+  const reviewedSha = /<!-- reviewed-sha: [0-9a-fA-F]{40} -->/.exec(body)?.[0];
+  const findingsBlock = findings ? `${AGENTS_STOP_DIRECTIVE}
+${findings}` : void 0;
+  return [findingsBlock, reviewedSha].filter((m) => m !== void 0).join("\n\n");
+};
 var escapeFence = (text) => text.replace(/```/g, "`` ` ``");
 var projectPatch = (patch) => {
   if (patch === void 0) return { kind: "none" };
@@ -1055,18 +1062,27 @@ var runGhApi = (args, stdin, env) => new Promise((resolve3, reject) => {
 });
 
 // src/pr.ts
+var CANDIDATE_JQ = ".[] | {number: .number, state: .state, headRef: .head.ref, headSha: .head.sha}";
+var parseCandidates = (stdout) => parseJsonl(stdout);
 var fetchPrCandidates = async (repo, headSha, ghApi) => {
-  const stdout = await ghApi([
-    `repos/${repo}/commits/${headSha}/pulls`,
-    "--jq",
-    ".[] | {number: .number, state: .state, headRef: .head.ref}"
-  ]);
-  return stdout.trim().split("\n").filter(Boolean).map((line) => JSON.parse(line));
+  const direct = parseCandidates(
+    await ghApi([`repos/${repo}/commits/${headSha}/pulls`, "--jq", CANDIDATE_JQ])
+  );
+  if (direct.length > 0) return direct;
+  const open = parseCandidates(
+    await ghApi([
+      `repos/${repo}/pulls?state=open&per_page=100`,
+      "--paginate",
+      "--jq",
+      CANDIDATE_JQ
+    ])
+  );
+  return open.filter((c) => c.headSha === headSha);
 };
 var resolvePr = (candidates, headBranch) => {
   if (candidates.length === 0) return { kind: "none" };
   const scoped = candidates.length > 1 && headBranch ? candidates.filter((c) => c.headRef === headBranch) : candidates;
-  const chosen = scoped[0] ?? candidates[0];
+  const chosen = scoped.find((c) => c.state === "open") ?? scoped[0] ?? candidates[0];
   if (chosen === void 0) return { kind: "none" };
   return chosen.state === "open" ? { kind: "open", prNumber: chosen.number } : { kind: "not-open", prNumber: chosen.number, state: chosen.state };
 };
@@ -1581,6 +1597,47 @@ var post = async (input, ghApi = runGhApi) => {
     }
   }
 };
+var announceBody = (headSha, runUrl, existingBody) => {
+  const notice = `${DEFAULT_MARKER}
+
+\u{1F504} **Code review in progress** for \`${headSha.slice(0, 7)}\` \u2014 see the [workflow run](${runUrl}) for progress; this comment is updated with the review when it completes.`;
+  const carried = existingBody ? carryForwardMarkers(existingBody) : "";
+  return carried ? `${notice}
+
+${carried}` : notice;
+};
+var announce = async (input, ghApi = runGhApi) => {
+  const candidates = await fetchPrCandidates(input.repo, input.headSha, ghApi);
+  const resolution = resolvePr(candidates, input.headBranch);
+  if (resolution.kind !== "open") {
+    process.stderr.write(
+      `No open PR for ${input.headSha} \u2014 nothing to announce (${resolution.kind})
+`
+    );
+    return;
+  }
+  const existing = await findBotComment(
+    input.repo,
+    resolution.prNumber,
+    input.botLogin,
+    DEFAULT_MARKER,
+    ghApi
+  );
+  if (existing !== null && parseReviewedSha(existing.body) === input.headSha.toLowerCase()) {
+    process.stderr.write(
+      `Sticky already reflects a completed review of ${input.headSha} \u2014 leaving it in place
+`
+    );
+    return;
+  }
+  await upsertSticky(
+    input.repo,
+    resolution.prNumber,
+    existing,
+    announceBody(input.headSha, input.runUrl, existing?.body),
+    ghApi
+  );
+};
 var DURATION_RE = /^(\d+)(h|m|s)$/;
 var USD_RE = /^\$(\d+(?:\.\d+)?)$/;
 var toSeconds = (n, unit) => unit === "h" ? n * 3600 : unit === "m" ? n * 60 : n;
@@ -1846,12 +1903,17 @@ var PrMetaCodec = t.type({
   title: t.string,
   body: t.union([t.string, t.null])
 });
-var IssueCommentCodec = t.type({
-  id: t.number,
-  body: t.union([t.string, t.null]),
-  user: t.type({ login: t.string })
-});
-var IssueCommentsCodec = t.array(IssueCommentCodec);
+var IssueCommentCodec = t.intersection([
+  t.type({
+    id: t.number,
+    body: t.union([t.string, t.null]),
+    user: t.type({ login: t.string })
+  }),
+  t.partial({
+    created_at: t.union([t.string, t.null]),
+    author_association: t.union([t.string, t.null])
+  })
+]);
 var JobCodec = t.type({ id: t.number, conclusion: t.union([t.string, t.null]) });
 var JobsResponseCodec = t.type({ jobs: t.array(JobCodec) });
 var fetchPrMeta = async (repo, prNumber, ghApi) => {
@@ -1873,18 +1935,92 @@ var fetchApiDiff = async (repo, prNumber, ghApi) => {
     return null;
   }
 };
-var fetchPriorReview = async (repo, prNumber, botLogin, ghApi) => {
+var COMMENT_JQ = ".[] | {id: .id, body: .body, user: {login: .user.login}, created_at: .created_at, author_association: .author_association}";
+var REVIEW_COMMENT_JQ = ".[] | {body: .body, user: {login: .user.login}, created_at: .created_at, author_association: .author_association, path: .path, line: .line}";
+var REVIEW_JQ = ".[] | {body: .body, user: {login: .user.login}, submitted_at: .submitted_at, author_association: .author_association, state: .state}";
+var ReviewCommentCodec = t.intersection([
+  t.type({ body: t.union([t.string, t.null]), user: t.type({ login: t.string }) }),
+  t.partial({
+    created_at: t.union([t.string, t.null]),
+    author_association: t.union([t.string, t.null]),
+    path: t.union([t.string, t.null]),
+    line: t.union([t.number, t.null])
+  })
+]);
+var ReviewCodec = t.intersection([
+  t.type({ body: t.union([t.string, t.null]), user: t.type({ login: t.string }) }),
+  t.partial({
+    submitted_at: t.union([t.string, t.null]),
+    author_association: t.union([t.string, t.null]),
+    state: t.union([t.string, t.null])
+  })
+]);
+var fetchJsonlRows = async (ghApi, endpoint, jq) => {
   try {
-    const stdout = await ghApi([`repos/${repo}/issues/${String(prNumber)}/comments`, "--paginate"]);
-    const decoded = IssueCommentsCodec.decode(JSON.parse(stdout || "[]"));
-    if (decoded._tag === "Left") return null;
-    const byBot = decoded.right.filter((c) => c.user.login === botLogin);
-    const last = byBot[byBot.length - 1];
-    return last ? { id: last.id, body: last.body } : null;
-  } catch {
+    return parseJsonl(await ghApi([endpoint, "--paginate", "--jq", jq]));
+  } catch (err) {
+    process.stderr.write(
+      `Warning: could not fetch ${endpoint} (${errMsg(err)}) \u2014 omitting it from the review context
+`
+    );
     return null;
   }
 };
+var decodeArrayOrNull = (codec, rows) => {
+  if (rows === null) return null;
+  return rows.flatMap((row) => {
+    const decoded = codec.decode(row);
+    return decoded._tag === "Right" ? [decoded.right] : [];
+  });
+};
+var priorReviewFrom = (comments, botLogin) => {
+  const byBot = comments.filter((c) => c.user.login === botLogin);
+  const last = byBot[byBot.length - 1];
+  return last ? { id: last.id, body: last.body } : null;
+};
+var MAX_CONVERSATION_COMMENTS = 50;
+var MAX_CONVERSATION_BODY_CHARS = 4e3;
+var clip = (body) => {
+  if (body.length <= MAX_CONVERSATION_BODY_CHARS) return body;
+  const cut = body.slice(0, MAX_CONVERSATION_BODY_CHARS);
+  const safe = /[\uD800-\uDBFF]$/.test(cut) ? cut.slice(0, -1) : cut;
+  return `${safe}
+\u2026 [truncated]`;
+};
+var boundedHuman = (items, botLogin, label, project) => {
+  const human = items.filter(
+    (a) => a.user.login !== botLogin && typeof a.body === "string" && a.body.trim() !== ""
+  );
+  const kept = human.slice(-MAX_CONVERSATION_COMMENTS);
+  if (kept.length < human.length) {
+    process.stderr.write(
+      `Note: PR has ${String(human.length)} ${label} \u2014 feeding the review the most recent ${String(MAX_CONVERSATION_COMMENTS)}
+`
+    );
+  }
+  return kept.map(project);
+};
+var issueCommentsFrom = (comments, botLogin) => boundedHuman(comments, botLogin, "discussion comments", (c) => ({
+  author: c.user.login,
+  author_association: c.author_association ?? null,
+  created_at: c.created_at ?? null,
+  body: clip(c.body)
+}));
+var reviewCommentsFrom = (comments, botLogin) => boundedHuman(comments, botLogin, "inline review comments", (c) => ({
+  author: c.user.login,
+  author_association: c.author_association ?? null,
+  created_at: c.created_at ?? null,
+  path: c.path ?? null,
+  line: c.line ?? null,
+  body: clip(c.body)
+}));
+var reviewsFrom = (reviews, botLogin) => boundedHuman(reviews, botLogin, "review submissions", (r) => ({
+  author: r.user.login,
+  author_association: r.author_association ?? null,
+  submitted_at: r.submitted_at ?? null,
+  state: r.state ?? null,
+  body: clip(r.body)
+}));
 var downloadFailingJobLogs = async (repo, runId, outDir, ghApi) => {
   const stdout = await ghApi([`repos/${repo}/actions/runs/${runId}/jobs`]);
   const decoded = JobsResponseCodec.decode(JSON.parse(stdout));
@@ -1934,11 +2070,31 @@ var gather = async (input, ghApi = runGhApi, gitRun = runGit) => {
     join(input.outDir, "pr_context.json"),
     JSON.stringify({ title: meta.title, body: meta.body })
   );
-  const prior = await fetchPriorReview(input.repo, prNumber, input.botLogin, ghApi);
+  const [issueRows, reviewCommentRows, reviewRows] = await Promise.all([
+    fetchJsonlRows(ghApi, `repos/${input.repo}/issues/${String(prNumber)}/comments`, COMMENT_JQ),
+    fetchJsonlRows(
+      ghApi,
+      `repos/${input.repo}/pulls/${String(prNumber)}/comments`,
+      REVIEW_COMMENT_JQ
+    ),
+    fetchJsonlRows(ghApi, `repos/${input.repo}/pulls/${String(prNumber)}/reviews`, REVIEW_JQ)
+  ]);
+  const issueComments = decodeArrayOrNull(IssueCommentCodec, issueRows);
+  const reviewComments = decodeArrayOrNull(ReviewCommentCodec, reviewCommentRows);
+  const reviews = decodeArrayOrNull(ReviewCodec, reviewRows);
+  const prior = issueComments === null ? null : priorReviewFrom(issueComments, input.botLogin);
   writeFileSync(
     join(input.outDir, "prior_review.json"),
     prior === null ? "null" : JSON.stringify(prior)
   );
+  writeFileSync(
+    join(input.outDir, "pr_conversation.json"),
+    JSON.stringify({
+      issue_comments: issueComments === null ? [] : issueCommentsFrom(issueComments, input.botLogin),
+      review_comments: reviewComments === null ? [] : reviewCommentsFrom(reviewComments, input.botLogin),
+      reviews: reviews === null ? [] : reviewsFrom(reviews, input.botLogin)
+    })
+  );
   if (input.conclusion === "failure") {
     await downloadFailingJobLogs(input.repo, input.runId, input.outDir, ghApi);
   }
@@ -3195,7 +3351,7 @@ var stopGateCmd = defineCommand({
 var gatherCmd = defineCommand({
   meta: {
     name: "gather",
-    description: "Resolve the PR from the CI head SHA and gather review inputs (diff with git-diff fallback, PR context, prior bot review, failing-job logs) as files for the review agent"
+    description: "Resolve the PR from the CI head SHA and gather review inputs (diff with git-diff fallback, PR context, prior bot review, untrusted PR conversation to triage, failing-job logs) as files for the review agent"
   },
   args: {
     repo: { type: "string", description: "Repository (owner/name)", required: true },
@@ -3329,6 +3485,51 @@ var postCmd = defineCommand({
     });
   }
 });
+var announceCmd = defineCommand({
+  meta: {
+    name: "announce",
+    description: "Post (or update) the sticky the moment a review starts \u2014 an in-progress placeholder linking the run \u2014 so a workflow_run review, which runs from the default branch and is otherwise invisible on the PR, is visibly under way. Preserves a prior sticky's embedded findings + reviewed-sha markers so the re-review seed survives the swap."
+  },
+  args: {
+    "head-sha": {
+      type: "string",
+      description: "Trusted head SHA to resolve the PR (from workflow_run.head_sha)",
+      required: true
+    },
+    repo: {
+      type: "string",
+      description: "Repository (owner/name)",
+      required: true
+    },
+    "run-url": {
+      type: "string",
+      description: "Workflow run URL the placeholder links to",
+      required: true
+    },
+    "bot-login": {
+      type: "string",
+      description: "Bot login to trust for the sticky comment upsert (default: github-actions[bot])"
+    },
+    "head-branch": {
+      type: "string",
+      description: "Head branch to disambiguate the PR when multiple share a commit"
+    }
+  },
+  run: async ({ args }) => {
+    await announce({
+      repo: args.repo,
+      headSha: args["head-sha"],
+      botLogin: args["bot-login"] || "github-actions[bot]",
+      runUrl: args["run-url"],
+      headBranch: args["head-branch"]
+    }).catch(
+      (err) => process.stderr.write(
+        `code-review announce: could not post the in-progress sticky (${errMsg(err)}) \u2014 continuing (cosmetic)
+`
+      )
+    );
+  }
+});
 var requireCeilingSec = (raw) => {
   if (raw === void 0) return null;
   const ms = parseWallMs(raw);
@@ -3501,6 +3702,7 @@ var main = defineCommand({
     render: renderCmd,
     inline: inlineCmd,
     post: postCmd,
+    announce: announceCmd,
     cost: costCmd,
     "check-cost": checkCostCmd,
     validate: validateCmd,