npm - @agentsdance/codejury - Versions diffs - 0.1.0 → 0.1.2 - Mend

@agentsdance/codejury 0.1.0 → 0.1.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (10) hide show

package/web/index.html CHANGED Viewed

@@ -1,5 +1,5 @@
 <!doctype html>
-<title>Code Jury — Convergence Console</title>
+<title>Code Jury — Review Console</title>
 <style>
   :root {
     --ground:#F4F6F8; --surface:#FFFFFF; --surface-alt:#EDF0F4;
@@ -237,7 +237,7 @@
   .io { background:var(--t-bg); border:1px solid var(--t-line); border-radius:6px;
         overflow:hidden; box-shadow:var(--shadow); font-family:var(--mono); }
   .io-body { padding:13px 16px 18px; font-size:12.6px; line-height:1.6; color:var(--t-fg);
-             max-height:640px; overflow:auto; }
+             overflow-x:auto; }
   /* One column per reviewer. They are side by side to make the independence
      legible at a glance; below the breakpoint they stack, which is the same
@@ -258,7 +258,6 @@
   .convo-switch .pr-chip[aria-pressed="true"]:hover { color:#fff; }
   .convo-switch .pr-chip[aria-pressed="false"] { opacity:.7; }
   .convo-note { margin:0 0 10px; font-size:12.5px; color:var(--muted); }
-  .convo .io-body { max-height:none; }
   /* A long unbroken report must scroll inside its own column rather than
      widening the grid and pushing the other conversation off-screen. */
   .convo .stream { white-space:pre-wrap; overflow-wrap:anywhere; }
@@ -362,12 +361,12 @@
   .convo > header .d { width:8px; height:8px; border-radius:50%; background:var(--ok); flex:none; }
   .convo > header .d[data-live="1"] { background:var(--warn); animation:blip 1.4s ease-in-out infinite; }
   @keyframes blip { 0%,100%{opacity:1} 50%{opacity:.25} }
-  /* The thread is what this page exists to show, so it gets real height: tall
-     enough to read in, with a floor so a short conversation still reads as the
-     main surface rather than a strip under the timeline. */
+  /* Completed threads grow with their content. The page owns vertical
+     scrolling; neither the thread nor an individual message hides findings in
+     a nested scroll region. */
   .msgs { padding:14px 16px; display:flex; flex-direction:column; gap:10px;
-    overflow-y:auto; max-height:82vh; min-height:min(34em, 58vh); }
-  .convo-grid[data-stack="1"] .msgs { max-height:56vh; min-height:0; }
+    min-height:min(34em, 58vh); }
+  .convo-grid[data-stack="1"] .msgs { min-height:0; }
   /* Wider than before: bubbles now carry the reviewer's report in full, and a
      narrow column turns a report into a ribbon. */
   .msg { max-width:min(88%, 940px); display:flex; flex-direction:column; gap:4px; min-width:0; }
@@ -437,9 +436,8 @@
   .bub[data-k="streaming"]::after {
     content:"▍"; animation:blip 1s steps(2) infinite; color:var(--accent);
   }
-  /* A long message keeps all of its text and gives up height instead. The
-     inset shadows are the affordance: they say "there is more above/below"
-     without a control to click. */
+  /* Only a live transcript is compact. Once the report completes, it is
+     rendered directly in the conversation at full height. */
   .bub .tall { max-height:26em; overflow-y:auto; overscroll-behavior:contain;
     scrollbar-width:thin; border-radius:3px;
     background:
@@ -522,7 +520,8 @@ let RUN = {
     url: "https://github.com/acme/worker-pool/pull/128",
     title: "Grow the CreateSandbox retry wait on repeated failures",
     branch: "fix/sandbox-manager-create-backoff", trunk: "master",
-    diffstat: "2 files · +418 −13", state: "merged", stateNote: "merged · converged on ae473ab",
+    judge: "claude",
+    diffstat: "2 files · +418 −13", state: "merged", stateNote: "Merged", reviewedCommit: "ae473ab",
     summary: "Two reviewers — codex and droid — read each pushed commit independently. The main agent triages what they find, fixes what survives verification, answers every finding including the ones it rejects, and pushes again. Three rounds until both reported nothing new."
   },
   totalMin: 100,
@@ -594,7 +593,7 @@ let RUN = {
         { who:"codex", ts:"round 1 · T+51m", body:"Accepted. The producer reservation code is byte-for-byte outside this commit's diff, so the issue is pre-existing. The new backoff does not materially widen the concurrent-producer window; cancellation-aware worker exit arguably reduces old/new manager overlap. I agree it belongs in a separate PR." } ] }
   ],
   ship: [
-    { n:"3",  k:"rounds",       s:"to convergence" },
+    { n:"3",  k:"rounds",       s:"until complete" },
     { n:"5",  k:"exchanges",    s:"3 fixed · 1 deferred · 1 rejected" },
     { n:"11", k:"turns",        s:"every finding answered" },
     { n:"0",  k:"prod bugs",    s:"found in these 3 rounds" },
@@ -760,7 +759,7 @@ function draw(){
       ${(x.turns ?? []).map(tt => `
         <div class="ln" data-who="${tt.who}">
           <span class="at">${when(tt)}</span>
-          <span class="nm">${icon(tt.who)}${esc(tt.who)}</span>
+          <span class="nm"${tt.who === judgeName() ? ' style="color:var(--t-claude)"' : ""}>${icon(tt.who)}${esc(tt.who)}</span>
           <span class="say">${rich(tt.body)}
             ${tt.quote ? `<div class="q">${esc(tt.quote)}</div>` : ""}
             ${tt.ask ? `<div class="ask">${esc(tt.ask)}</div>` : ""}
@@ -796,7 +795,7 @@ function draw(){
         return `
           <div class="xgroup">
             <div class="rule" data-who="${esc(agent)}">
-              <span class="lbl">${icon(agent)}claude ⇄ ${esc(agent)}</span><i></i>
+              <span class="lbl">${icon(agent)}${esc(judgeName())} ⇄ ${esc(agent)}</span><i></i>
               <span class="rt">${mine.length} of ${shown} finding(s)</span>
             </div>
             ${mine.map(card).join("")}
@@ -845,9 +844,11 @@ function renderHeader(){
   const branch = t.branch ? `<span>${esc(t.branch)} → ${esc(t.trunk ?? "master")}</span>` : "";
   const stat = t.diffstat ? `<span>${esc(t.diffstat)}</span>` : "";
   const cls = BADGE[t.state] ?? "badge-mute";
+  const commit = reviewedCommit(t);
   const badge = t.state
-    ? `<span class="${cls}"><span class="d"></span>${esc(t.stateNote || t.state)}</span>` : "";
-  $("t-meta").innerHTML = [link, branch, stat, badge].filter(Boolean).join("");
+    ? `<span class="${cls}"><span class="d"></span>${esc(statusLabel(t))}</span>` : "";
+  const reviewed = commit ? `<span>Reviewed commit <code>${esc(commit)}</code></span>` : "";
+  $("t-meta").innerHTML = [link, branch, stat, badge, reviewed].filter(Boolean).join("");
   // Only worth showing when there is something to switch between.
   //
@@ -934,7 +935,7 @@ function runSummary(r) {
   const t = r.target ?? {};
   if (isLive(r) && t.stateNote) return t.stateNote;
   const n = (r.exchanges ?? []).length;
-  if (t.state === "converged") return "converged";
+  if (t.state === "converged") return "Review complete";
   const found = n ? `${n} finding${n === 1 ? "" : "s"}` : "";
   // A stale "review" is a run that stopped without finishing. Saying "review"
   // would repeat the claim that misled the grouping in the first place.
@@ -942,6 +943,17 @@ function runSummary(r) {
   return found || t.stateNote || t.state || "";
 }
+function reviewedCommit(t) {
+  if (t?.reviewedCommit) return t.reviewedCommit;
+  return (t?.stateNote?.match(/\bconverged on ([0-9a-f]{7,40})\b/i)?.[1] ?? "").toLowerCase();
+}
+function statusLabel(t) {
+  if (t?.state === "converged") return "Review complete";
+  if (t?.state === "merged" && /converged/i.test(t.stateNote ?? "")) return "Merged";
+  return t?.stateNote || t?.state || "";
+}
 function drawSwitcher() {
   const el = $("switcher");
   if (!el) return;
@@ -1125,14 +1137,15 @@ function render(){
    folded run; a conversation needs the events in the order they were spoken,
    and folding throws that order away.
-   Side is fixed by speaker, never by turn order: claude left, reviewer right.
+   Side is fixed by speaker, never by turn order: judge left, reviewer right.
    A reader should be able to tell who is talking from across the room. */
 let THREADS = [];        // [{agent, turns:[…]}]
 let CONVO_STACK = false; // one thread full width, or every thread stacked
 let CONVO_PICK = null;   // which thread, when not stacked
 const CONVO_SHUT = new Set(); // collapsed threads, by agent, while stacked
-const SIDE = (who) => (who === "claude" ? "l" : "r");
+const judgeName = () => RUN.target?.judge ?? "claude";
+const SIDE = (who) => (who === judgeName() ? "l" : "r");
 // A stable colour per agent. codex and droid have tokens already; anything else
 // is hashed to a hue so a five-reviewer run does not render five identical
@@ -1155,22 +1168,12 @@ const VERDICT_TEXT = {
   superseded: "Superseded",
 };
-// Long reports collapse. A twenty-minute reviewer turn can be thousands of
-// lines, and an un-collapsed one buries every exchange after it.
+// A live transcript can be thousands of lines before it becomes a completed
+// report. Keep only that in-progress state compact; completed messages expand.
 const CLAMP = 700;
-/* The collapsing half of a bubble, on its own so that bubbles which lead with a
-   header — a verdict, a named claim — can clamp their body without clamping the
-   header along with it. */
 /**
- * Every message in full, always. Nothing is truncated and nothing hides behind
- * a button: reading a review should not mean clicking through it.
- *
- * The reason a clamp existed is still real — evidence arrives verbatim from
- * `jury finding reproduce --evidence` and is routinely thousands of lines, and
- * one such turn inlined at full height buries every turn after it. So a long
- * message scrolls WITHIN ITS OWN BOX instead of being cut: the whole text is in
- * the DOM, selectable and searchable with the browser's own find, while the
- * thread below it stays reachable. Bounded height, not bounded content.
+ * Compact an in-progress transcript without truncating its DOM content.
+ * Completed reports bypass this helper and expand to their full height.
  */
 function clamped(text) {
   const t = text ?? "";
@@ -1179,7 +1182,7 @@ function clamped(text) {
     aria-label="long message, ${bytes(t.length)}, scrollable">${esc(t)}</div>`;
 }
 function bubble(text, k, attrs = "") {
-  return `<div class="bub" data-k="${k}" ${attrs}>${clamped(text)}</div>`;
+  return `<div class="bub" data-k="${k}" ${attrs}>${esc(text ?? "")}</div>`;
 }
 /** m:ss, or h:mm:ss once a reviewer has been going that long. */
 function hms(sec) {
@@ -1235,19 +1238,17 @@ function stampFor(t) {
 function turnHTML(t, agent) {
   const side = SIDE(t.who);
-  const nm = t.who === "claude" ? "claude" : agent;
+  const nm = t.who === judgeName() ? judgeName() : agent;
   let body = "";
   switch (t.kind) {
     case "finding":
       body = `<div class="bub" data-k="finding"><span class="hd">${esc(t.claim ?? "")}</span
-        >${t.loc ? `<span class="loc">${esc(t.loc)}</span>\n` : ""}${esc((t.body ?? "").slice(0, CLAMP))}</div>`;
+        >${t.loc ? `<span class="loc">${esc(t.loc)}</span>\n` : ""}${esc(t.body ?? "")}</div>`;
       break;
     case "reproduced":
-      // Evidence is whatever the reviewer passed to `finding reproduce
-      // --evidence`, which is routinely a wall of captured command output. It
-      // collapses, or one reproduction buries every turn below it.
+      // Evidence is shown inline in full once reproduction has completed.
       body = `<div class="bub" data-k="reproduced"
-        >${t.claim ? `<span class="re">${esc(t.claim)}</span>\n` : ""}Reproduced. ${clamped(t.text ?? "")}</div>`;
+        >${t.claim ? `<span class="re">${esc(t.claim)}</span>\n` : ""}Reproduced. ${esc(t.text ?? "")}</div>`;
       break;
     case "verdict":
       // The claim is named here because the findings are no longer bubbles of
@@ -1262,7 +1263,7 @@ function turnHTML(t, agent) {
       // being re-argued, so it names the claim and leaves the words to the
       // report. Claude's turn in the thread does carry text, and shows it.
       body = `<div class="bub" data-k="rebuttal"
-        ><span class="hd">${t.who === "claude" ? "still open" : "not convinced"}</span
+        ><span class="hd">${t.who === judgeName() ? "still open" : "not convinced"}</span
         >${t.claim ? `<span class="re">${esc(t.claim)}</span>` : ""}${t.text ? `\n${esc(t.text)}` : ""}</div>`;
       break;
     case "commit":
@@ -1299,7 +1300,7 @@ function turnHTML(t, agent) {
       body = bubble(t.text ?? "", esc(t.kind ?? "turn"));
   }
   return `<div class="msg" data-side="${side}"><span class="nm"
-    ><span class="ic" style="color:${nm === "claude" ? "var(--main-hue)" : hueFor(nm)}">${icon(nm)}</span>${esc(nm)}${stampFor(t)}</span>${body}</div>`;
+    ><span class="ic" style="color:${nm === judgeName() ? "var(--main-hue)" : hueFor(nm)}">${icon(nm)}</span>${esc(nm)}${stampFor(t)}</span>${body}</div>`;
 }
 /**
@@ -1380,7 +1381,7 @@ function drawConvo() {
     return `<div class="convo" data-a="${esc(t.agent)}" data-open="${open ? 1 : 0}">
       <header data-fold-agent="${esc(t.agent)}"><span class="tw">${open ? "▾" : "▸"}</span
         ><span class="d" data-live="${l ? 1 : 0}"></span>
-        <span class="who"><span class="ic" style="color:var(--main-hue)">${icon("claude")}</span>claude
+        <span class="who"><span class="ic" style="color:var(--main-hue)">${icon(judgeName())}</span>${esc(judgeName())}
           <span class="sw">⇄</span>
           <span class="ic" style="color:${hueFor(t.agent)}">${icon(t.agent)}</span>${esc(t.agent)}</span>
         <span class="st">${t.turns.length} turn${t.turns.length === 1 ? "" : "s"}${l ? " · live" : ""}</span>
@@ -1502,8 +1503,12 @@ function foldConversation(events) {
   };
   const owner = new Map();
   const claimOf = new Map();
+  let judge = "claude";
   for (const e of events) {
     switch (e.t) {
+      case "target":
+        judge = e.target?.judge ?? judge;
+        break;
       case "finding.raised":
         owner.set(e.id, e.agent);
         // Deliberately NOT a turn of its own. The reviewer already said this in
@@ -1518,12 +1523,12 @@ function foldConversation(events) {
         thread(e.agent);
         break;
       case "finding.reproduced":
-        thread(owner.get(e.id) ?? "?").turns.push({ who: "claude", kind: "reproduced",
+        thread(owner.get(e.id) ?? "?").turns.push({ who: e.who ?? judge, kind: "reproduced",
           id: e.id, claim: claimOf.get(e.id) ?? "",
           text: typeof e.evidence === "string" ? e.evidence : "", ts: e.ts });
         break;
       case "finding.resolved":
-        thread(owner.get(e.id) ?? "?").turns.push({ who: "claude", kind: "verdict",
+        thread(owner.get(e.id) ?? "?").turns.push({ who: e.who ?? judge, kind: "verdict",
           id: e.id, claim: claimOf.get(e.id) ?? "",
           verdict: e.verdict, text: e.reason ?? "", test: e.test ?? "", ts: e.ts });
         break;
@@ -1532,12 +1537,12 @@ function foldConversation(events) {
         // and that reply is already rendered whole as its `answer` turn — so the
         // text is deliberately dropped, exactly as a raised finding's is. What
         // survives is the linkage: which claim is being re-argued, and by whom.
-        // Claude's own turn is the exception: it is not an excerpt of anything
+        // The judge's own turn is the exception: it is not an excerpt of anything
         // shown elsewhere, so dropping its text would lose it entirely.
         const who = e.who ?? "?";
         const turn = { who, kind: "rebuttal", id: e.id,
           claim: claimOf.get(e.id) ?? "", ts: e.ts };
-        if (who === "claude") turn.text = e.text ?? "";
+        if (who === judge) turn.text = e.text ?? "";
         thread(owner.get(e.id) ?? e.agent ?? "?").turns.push(turn);
         break;
       }
@@ -1589,7 +1594,7 @@ function foldConversation(events) {
         break;
       }
       case "reply.sent":
-        thread(e.agent).turns.push({ who: "claude", kind: "reply", text: e.text ?? "", ts: e.ts });
+        thread(e.agent).turns.push({ who: e.who ?? judge, kind: "reply", text: e.text ?? "", ts: e.ts });
         break;
       case "reply.answered":
         thread(e.agent).turns.push({ who: e.agent, kind: "answer",
@@ -1597,7 +1602,7 @@ function foldConversation(events) {
         break;
       case "commit.pushed":
         for (const t of threads.values()) {
-          t.turns.push({ who: "claude", kind: "commit", sha: e.sha, text: e.subject ?? "", ts: e.ts });
+          t.turns.push({ who: e.who ?? judge, kind: "commit", sha: e.sha, text: e.subject ?? "", ts: e.ts });
         }
         break;
     }