@jphutchins/code-review 0.1.0-alpha.46 → 0.1.0-alpha.47

package/dist/index.js

@@ -477,13 +477,7 @@ var normalizeCodeCounts = (codes, priorCodes) => {
   if (typeof codes !== "object" || codes === null || Array.isArray(codes)) return void 0;
   const entries = Object.entries(codes).filter(
     (e) => typeof e[1] === "number" && Number.isSafeInteger(e[1]) && e[1] > 0
-  ).sort((a, b) => {
-    if (b[1] !== a[1]) return b[1] - a[1];
-    const aPrior = hasCode(priorCodes, a[0]) ? 1 : 0;
-    const bPrior = hasCode(priorCodes, b[0]) ? 1 : 0;
-    if (aPrior !== bPrior) return bPrior - aPrior;
-    return a[0] < b[0] ? -1 : a[0] > b[0] ? 1 : 0;
-  });
+  );
   if (entries.length === 0) return void 0;
   const sorted = entries.sort((a, b) => {
     if (b[1] !== a[1]) return b[1] - a[1];
@@ -666,6 +660,14 @@ var priorBelowFloorNits = (priorDoc, floor = DEFAULT_NIT_VISIBILITY_FLOOR) => {
 };
 var convergenceBadge = (c) => c.converged ? `**Convergence** \u{1F3C1} ${formatScore(c.score)} \u2264 ${String(c.threshold)} \u2014 converged` : `**Convergence** \u{1F504} ${formatScore(c.score)} > ${String(c.threshold)} \u2014 iterating`;
 var convergenceSummary = (doc, threshold = DEFAULT_CONVERGENCE_THRESHOLD) => convergenceBadge(convergenceSignal(doc, threshold));
+var nextRoundNumber = (priorTraj, priorConvRounds) => {
+  const last = (rounds) => rounds.length > 0 ? rounds[rounds.length - 1]?.round ?? rounds.length : 0;
+  return Math.max(last(priorTraj), last(priorConvRounds)) + 1;
+};
+var inProgressConvergence = (prior, runningRound) => {
+  const rounds = prior.rounds ?? [];
+  return rounds.length === 0 ? "" : `${roundsSummary(rounds, runningRound)} \u2192 \u23F3`;
+};
 var SURFACE_SCHEMA_VERSION = "0.8.0";
 var convergenceSignal = (doc, threshold = DEFAULT_CONVERGENCE_THRESHOLD) => {
   const score = convergenceScore(doc, threshold);
@@ -1170,7 +1172,6 @@ var render = (input) => {
     // (issue #174) — no separate signal or rounds marker rides beside it. post always supplies the
     // precomputed marker; the standalone `render` command falls back to encoding the doc here.
     findingsPointer: input.findingsPointer ?? findingsPointer(input.findings, input.jsonUrl),
-    roundsMarker: "",
     roundsSummary: roundsSummary(trajectory, input.roundCount),
     metastasisNote: advisoryAllowed ? metastasisNote(trajectory) : "",
     sameRootNotes: advisoryAllowed ? sameRootNotes : {},
@@ -2378,8 +2379,7 @@ var post = async (input, ghApi = runGhApi) => {
   const priorBody = existingSticky?.body ?? "";
   const priorTraj = priorTrajectory(priorDoc, priorBody);
   const priorConv = carriedConvergence(priorDoc, priorBody);
-  const lastRound = (rounds) => rounds.length > 0 ? rounds[rounds.length - 1]?.round ?? rounds.length : 0;
-  const priorRoundCount = Math.max(lastRound(priorTraj), lastRound(priorConv?.rounds ?? []));
+  const priorRoundCount = nextRoundNumber(priorTraj, priorConv?.rounds ?? []) - 1;
   const stampConvergence = (doc, conv) => ({
     ...doc,
     convergence: conv ?? void 0
@@ -2724,12 +2724,22 @@ var bodyRefsRun = (body, runUrl) => {
   const runId = runIdFromUrl(runUrl);
   return runId === null ? body.includes(runUrl) : new RegExp(`/actions/runs/${runId}(?!\\d)`).test(body);
 };
-var announceBody = (headSha, runUrl, existingBody) => noticeBody(
-  `${DEFAULT_MARKER}
+var announceBody = (headSha, runUrl, existingBody) => {
+  const priorDoc = existingBody !== void 0 ? parseFindingsMarker(existingBody) : null;
+  const prior = existingBody !== void 0 ? carriedConvergence(priorDoc, existingBody) : null;
+  const progress = prior !== null ? inProgressConvergence(
+    prior,
+    nextRoundNumber(priorTrajectory(priorDoc, existingBody ?? ""), prior.rounds ?? [])
+  ) : "";
+  return noticeBody(
+    `${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.`,
-  existingBody
-);
+\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.${progress ? `
+
+${progress}` : ""}`,
+    existingBody
+  );
+};
 var announce = async (input, ghApi = runGhApi) => {
   const candidates = await fetchPrCandidates(input.repo, input.headSha, ghApi);
   const resolution = resolvePr(candidates, input.headBranch);