@bongos/core 1.19.491 → 1.19.493

package/modules/hall-ui/public/copy-desk.js

@@ -34,6 +34,10 @@
   let queue = null;
   // The composer's current target, set when a "Flag" button is pressed.
   let target = null;
+  // The proposal composer's target (task 1003114 / R03), set from a "Propose
+  // wording" button. Kept apart from `target` because the two dialogs can carry
+  // different things: a flag can name a whole surface, a proposal never can.
+  let proposing = null;
   let closing = null;
   let dupePager = null;
 
@@ -123,8 +127,22 @@
     const note = f.status !== 'open' && f.resolution_note
       ? `<p class="cd-closed"><b>${escapeHtml(f.status)}</b> — ${escapeHtml(f.resolution_note)}</p>`
       : '';
-    const actions = f.status === 'open' && canClose(f)
-      ? `<div class="ov-row__actions"><button class="btn-ghost cd-close-btn" data-flag="${escapeHtml(String(f.id))}">Close…</button></div>`
+    // The queue is where the artist actually works, so the answer to a flag is
+    // offered right on it — but only for an open flag whose target is still
+    // there. A flag whose string has since been reworded (target_state 'gone')
+    // gets no propose button: there is nothing to patch, and the honest next
+    // step is a person deciding whether the flag is still live.
+    const proposable = f.status === 'open' && f.scope === 'string' && f.current
+      ? proposeButton({
+        id: f.string_id, surface: f.surface, text: f.current.text,
+        file: f.current.file, line: f.current.line, flag: f.id,
+      })
+      : '';
+    const closeable = f.status === 'open' && canClose(f)
+      ? `<button class="btn-ghost cd-close-btn" data-flag="${escapeHtml(String(f.id))}">Close…</button>`
+      : '';
+    const actions = proposable || closeable
+      ? `<div class="ov-row__actions">${proposable}${closeable}</div>`
       : '';
 
     return `<li class="ov-row" data-flag="${escapeHtml(String(f.id))}">
@@ -147,6 +165,20 @@
       data-id="${escapeHtml(d.id)}" data-surface="${escapeHtml(d.surface)}" data-text="${escapeHtml(d.text)}">Flag this…</button></div>`;
   }
 
+  // The propose button (task 1003114 / R03), offered ONLY where the row names one
+  // occurrence — file and line. That is not a UI shortcut: a proposal patches a
+  // specific place in a specific file, and a row that cannot say which place
+  // would be asking the artist to write a change nobody can land. The duplicates
+  // list is exactly that case, and it keeps just "Flag this…" — a sentence said
+  // in seven places usually wants saying once, not rewording seven times.
+  function proposeButton(d) {
+    if (!d.id || !d.file || !Number.isInteger(d.line)) return '';
+    return `<button class="btn-ghost cd-propose-btn"
+      data-id="${escapeHtml(d.id)}" data-surface="${escapeHtml(d.surface)}" data-text="${escapeHtml(d.text)}"
+      data-file="${escapeHtml(d.file)}" data-line="${escapeHtml(String(d.line))}"
+      ${d.flag ? `data-flag="${escapeHtml(String(d.flag))}"` : ''}>Propose wording…</button>`;
+  }
+
   // The duplicate's state. Colour marks the EXCEPTION here, the inverse of the
   // queue: most of this list is untouched work, so the untouched row is the
   // plain pill and colour goes to the few already in the queue or looked at.
@@ -182,7 +214,13 @@
           ${e.origin ? `<span class="ov-fact">${escapeHtml(e.origin)}</span>` : ''}
         </div>
       </div>
-      <div class="ov-row__right">${e.flagged ? pill('fact-pill--open', 'already flagged') : flagButton(e)}</div>
+      <div class="ov-row__right">
+        <div class="ov-row__actions">
+          ${e.flagged ? pill('fact-pill--open', 'already flagged') : `<button class="btn-ghost cd-flag-btn"
+            data-id="${escapeHtml(e.id)}" data-surface="${escapeHtml(e.surface)}" data-text="${escapeHtml(e.text)}">Flag this…</button>`}
+          ${proposeButton(e)}
+        </div>
+      </div>
     </li>`;
   }
 
@@ -355,6 +393,85 @@
     if ($('#cd-search').value.trim()) await runSearch();
   }
 
+  // ---------------------------------------------------------------------------
+  // the proposal composer (task 1003114 / R03)
+  //
+  // The affordance the goal asked for, shaped so it does not become the thing
+  // the goal forbids. Nothing here writes a string: it fills in a change, the
+  // server turns it into a task, and the wording reaches a reader only after
+  // somebody claims that task and ships the diff.
+  // ---------------------------------------------------------------------------
+
+  const HOLE = '{…}';
+  const holes = (s) => String(s || '').split(HOLE).length - 1;
+
+  function openProposer(t) {
+    proposing = t;
+    $('#cd-propose-target').innerHTML = `${quote(t.text)}<br>${where(t.file, t.line)}`;
+    // PREFILLED with the current wording. An artist is editing a sentence, not
+    // composing one from nothing, and retyping it from the quote above is how a
+    // proposal accidentally changes a word nobody meant to change.
+    $('#cd-propose-text').value = t.text;
+    $('#cd-propose-why').value = '';
+    $('#cd-propose-err').textContent = '';
+
+    const n = holes(t.text);
+    const hint = $('#cd-propose-holes');
+    hint.hidden = n === 0;
+    if (n > 0) {
+      hint.textContent = `This sentence has ${n} ${HOLE} where a value is dropped in when it is shown. `
+        + `Keep all ${n} — move them if the new sentence needs them elsewhere, but do not remove one: `
+        + 'the values come from the code, and dropping one is a code change.';
+    }
+    $('#cd-propose-dialog').showModal();
+  }
+
+  async function submitProposal() {
+    const proposed = $('#cd-propose-text').value.replace(/\s+/g, ' ').trim();
+    const why = $('#cd-propose-why').value.trim();
+    const err = $('#cd-propose-err');
+
+    // The server and its floors are the real gates; these mirror them so the
+    // common mistakes answer instantly instead of costing a round trip.
+    if (!proposed) { err.textContent = 'Write the wording you want instead.'; return; }
+    if (proposed === String(proposing.text).replace(/\s+/g, ' ').trim()) {
+      err.textContent = 'That is the wording it already has. Change something, or file a flag instead if you only want to mark it.';
+      return;
+    }
+    if (holes(proposed) !== holes(proposing.text)) {
+      err.textContent = `Keep all ${holes(proposing.text)} ${HOLE} — they are the values the code drops in, and the change cannot land without them.`;
+      return;
+    }
+    if (why.replace(/\s+/g, ' ').length < 10) {
+      err.textContent = 'Say why it is better. Whoever reviews this decides it from your reason, without asking you.';
+      return;
+    }
+
+    const body = {
+      string_id: proposing.string_id,
+      proposed_text: proposed,
+      why,
+      file: proposing.file,
+      line: proposing.line,
+    };
+    if (proposing.flag_id) body.flag_id = Number(proposing.flag_id);
+
+    const res = await api.request('POST', `${API}/copy-desk/proposals`, { body });
+    if (!res.ok) {
+      const e = (res.data && res.data.error) || {};
+      const d = e.details || {};
+      err.textContent = d.what_this_means || d.why || e.message || 'Could not file the change.';
+      return;
+    }
+    $('#cd-propose-dialog').close();
+    const task = (res.data && res.data.task) || {};
+    toast(res.data && res.data.created === false
+      ? `Already filed — task #${task.id} carries this exact change.`
+      : `Filed as task #${task.id}. It lands when someone claims it.`);
+    await loadQueue();
+    if ($('#cd-search').value.trim()) await runSearch();
+  }
+
   function openCloser(id) {
     closing = queue.flags.find((f) => String(f.id) === String(id));
     if (!closing) return;
@@ -427,12 +544,26 @@
       });
       return;
     }
+    const proposeBtn = ev.target.closest('.cd-propose-btn');
+    if (proposeBtn) {
+      openProposer({
+        string_id: proposeBtn.dataset.id,
+        surface: proposeBtn.dataset.surface,
+        text: proposeBtn.dataset.text,
+        file: proposeBtn.dataset.file,
+        line: Number(proposeBtn.dataset.line),
+        flag_id: proposeBtn.dataset.flag || null,
+      });
+      return;
+    }
     const closeBtn = ev.target.closest('.cd-close-btn');
     if (closeBtn) openCloser(closeBtn.dataset.flag);
   });
 
   $('#cd-cancel').addEventListener('click', () => $('#cd-dialog').close());
   $('#cd-submit').addEventListener('click', submitFlag);
+  $('#cd-propose-cancel').addEventListener('click', () => $('#cd-propose-dialog').close());
+  $('#cd-propose-submit').addEventListener('click', submitProposal);
   $('#cd-close-cancel').addEventListener('click', () => $('#cd-close-dialog').close());
   $('#cd-close-submit').addEventListener('click', submitClose);
 

package/modules/hall-ui/public/copy-desk.states.json

@@ -1,5 +1,5 @@
 {
-  "_": "Copy desk — the one-voice review queue. Its two dialogs are real <dialog>s, so their ::backdrop is the sheet's single sanctioned literal (black with alpha as a scrim). Written with the re-skin (task 1003307) so this page is MEASURED at three widths in both modes rather than looked at once; the redesign (task 1003471) put a queue, ten duplicates and a search's hits in the harness so the rows and both dialogs can be looked at.",
+  "_": "Copy desk — the one-voice review queue. Its three dialogs are real <dialog>s, so their ::backdrop is the sheet's single sanctioned literal (black with alpha as a scrim). Written with the re-skin (task 1003307) so this page is MEASURED at three widths in both modes rather than looked at once; the redesign (task 1003471) put a queue, ten duplicates and a search's hits in the harness so the rows and the dialogs can be looked at; task 1003114 (R03) added the proposal composer and the `proposer` state that opens it.",
   "page": "/builders/copy-desk",
   "surface": "hall-ui",
   "stub": {
@@ -102,6 +102,23 @@
           250
         ]
       ]
+    },
+    "proposer": {
+      "auth": true,
+      "actions": [
+        [
+          "wait",
+          700
+        ],
+        [
+          "clickIfVisible",
+          ".cd-propose-btn"
+        ],
+        [
+          "wait",
+          250
+        ]
+      ]
     }
   },
   "reducedMotion": {

package/modules/hall-ui/public/task.css

@@ -1,6 +1,7 @@
 /* task.css — what the TASK record has that the idea record does not
    (task 1003305, goal 1000081). The shared composition is record.css; this is
-   the page's own two shapes.
+   the page's own shapes: the lifecycle rail and the ship visual (1003305), the
+   grade panel and the session log (task 1003437).
 
    Promoted out of task.html's inline <style>, which was the last substantive
    inline block in the hall and sat at its tests/hall_audit.mjs ratchet with zero
@@ -69,3 +70,72 @@
   .tk-rail { grid-template-columns: 1fr; }
   .tk-step { padding: 12px 0 0; }
 }
+
+/* ---- the grade panel (task 1003437) ------------------------------------- */
+/* The four workers on hairlines, then the score at display weight with the
+   verdict pill beside it and the panel crumb after. Verification state wears
+   the fixed vocabulary through .rec-pill--ok/warn/danger (record.css) — never
+   the accent: a verdict is a fact, and the accent means pressable-or-live. */
+.tk-axes {
+  display: grid;
+  grid-template-columns: repeat(4, minmax(0, 1fr));
+  border-top: var(--line) solid var(--rule-soft);
+  margin-top: var(--sp-md);
+}
+.tk-axis { padding: 16px 18px 18px; border-left: var(--line) solid var(--rule-soft); }
+.tk-axis:first-child { border-left: 0; padding-left: 0; }
+.tk-axis__n { display: block; font-family: var(--font-mono); font-size: 12px; color: var(--ink-soft); margin-bottom: 11px; }
+.tk-score { display: flex; align-items: baseline; gap: 18px; flex-wrap: wrap; margin: 22px 0 0; }
+.tk-score__v {
+  font-family: var(--font-display);
+  font-weight: 300;
+  font-size: clamp(32px, 3.8vw, 50px);
+  line-height: 1;
+  letter-spacing: -.035em;
+  color: var(--ink);
+  font-variant-numeric: tabular-nums;
+}
+.tk-score__of { font-size: max(.34em, 12px); font-family: var(--font-mono); color: var(--ink-faint); margin-left: .3em; }
+.tk-crumb {
+  font-family: var(--label-face);
+  font-size: var(--label-size);
+  letter-spacing: var(--label-space);
+  text-transform: var(--label-case);
+  color: var(--ink-faint);
+  font-variant-numeric: tabular-nums;
+}
+
+/* ---- the session log (task 1003437) -------------------------------------- */
+/* One small key/value ledger per session, newest first; a second session is
+   separated by a heavier rule. The handoff sits behind a disclosure so a long
+   note cannot push the actions off the page; the summary is a real control. */
+.tk-ledger { display: grid; margin-top: var(--sp-md); }
+.tk-ledger + .tk-ledger { margin-top: var(--sp-lg); border-top: 2px solid var(--rule-soft); padding-top: var(--sp-md); }
+.tk-lrow {
+  display: grid;
+  grid-template-columns: 170px minmax(0, 1fr);
+  gap: 22px;
+  border-top: var(--line) solid var(--rule-soft);
+  padding: 13px 0;
+  align-items: baseline;
+}
+.tk-lrow:first-child { border-top: 0; padding-top: 0; }
+.tk-lk {
+  font-family: var(--label-face);
+  font-size: var(--label-size);
+  letter-spacing: var(--label-space);
+  text-transform: var(--label-case);
+  color: var(--ink-faint);
+}
+.tk-lv { font-size: 15px; color: var(--ink); font-variant-numeric: tabular-nums; overflow-wrap: anywhere; }
+.tk-handoff > summary { cursor: pointer; min-height: 24px; font-size: 15px; color: var(--ink-soft); }
+.tk-handoff .rec-md { margin-top: 10px; }
+
+/* the claim ledger's earlier claims (task 1003437) — same list shape as Touches */
+.tk-claims li { font-size: 13.5px; color: var(--ink-soft); }
+
+@media (max-width: 640px) {
+  .tk-axes { grid-template-columns: repeat(2, minmax(0, 1fr)); }
+  .tk-axis:nth-child(3) { border-left: 0; padding-left: 0; }
+  .tk-lrow { grid-template-columns: 1fr; gap: 6px; }
+}

package/modules/hall-ui/public/task.js

@@ -330,7 +330,107 @@
       `Task description:\n${desc}`;
   }
 
-  function renderTask(t, lastClaim, viewer, goal) {
+  // ---- the grade panel (task 1003437) ------------------------------------
+  // The adversarial panel's verdict on THIS task, on the task's own page rather
+  // than only on Standing's #recent-grades. Verification state wears the fixed
+  // vocabulary (ok / warn / danger / info) through .rec-pill, never the accent —
+  // a verdict is a fact about the work, not a control. The four workers ride
+  // grade.signals.workers[] (task_grades has no columns for them); score,
+  // verdict and threshold are the row's own; panel version and model come from
+  // the signals. A task with no grade row draws NOTHING (the ship-visual
+  // precedent — no placeholder, no "no grade yet" line); a row whose panel could
+  // not run is data, not absence, and says so in one line.
+  const WORKER_ORDER = ['narc', 'quality', 'hacker', 'efficiency'];
+  const VERDICT_TONE = { pass: 'ok', concerns: 'warn', fail: 'danger', error: 'danger' };
+  function verdictPill(verdict, small) {
+    const v = String(verdict || 'error').toLowerCase();
+    const tone = VERDICT_TONE[v] || 'info';
+    return `<span class="rec-pill rec-pill--${tone}${small ? ' rec-pill--sm' : ''}">${escapeHtml(v)}</span>`;
+  }
+  function gradePanelHtml(grade) {
+    if (!grade || typeof grade !== 'object') return '';
+    const sig = grade.signals && typeof grade.signals === 'object' ? grade.signals : {};
+    const head = `<section class="rec-sec" id="task-grade"><p class="rec-sec__h">Grade</p>`;
+    if (sig.panel_outcome === 'unavailable') {
+      return `${head}<p class="rec-body">The grading panel could not run when this task was shipped, so no quality signal was recorded — an outage, not a verdict.</p></section>`;
+    }
+    const workers = Array.isArray(sig.workers) ? sig.workers : [];
+    const kindOf = (w) => String((w && w.kind) || '').toLowerCase();
+    const known = WORKER_ORDER.filter((k) => workers.some((w) => kindOf(w) === k));
+    const extra = workers.map(kindOf).filter((k) => k && !WORKER_ORDER.includes(k));
+    const axes = known.concat(extra).map((k) => {
+      const w = workers.find((x) => kindOf(x) === k);
+      return `<div class="tk-axis"><span class="tk-axis__n">${escapeHtml(k)}</span>${verdictPill(w.error ? 'error' : w.verdict, true)}</div>`;
+    }).join('');
+    const score = grade.score == null || !Number.isFinite(Number(grade.score)) ? null : Number(grade.score);
+    const verdict = grade.passed === true ? 'pass' : grade.passed === false ? 'fail' : String(sig.panel_outcome || 'pending');
+    const models = [...new Set(workers.map((w) => w && w.model).filter(Boolean))];
+    const crumb = [
+      grade.threshold != null && Number.isFinite(Number(grade.threshold)) ? `threshold ${Number(grade.threshold)}` : '',
+      grade.grader_version ? `panel ${String(grade.grader_version)}` : '',
+      models.join(', '),
+    ].filter(Boolean).map(escapeHtml).join(' · ');
+    return `${head}
+      <p class="rec-body">The adversarial panel. Each worker reports pass, concerns or fail on its own; the gating workers must all pass before confirmed becomes ${escapeHtml(curLabel())}, and one concerns falls the task through to the manual confirm path.</p>
+      ${axes ? `<div class="tk-axes">${axes}</div>` : ''}
+      <div class="tk-score">
+        ${score == null ? '' : `<span class="tk-score__v">${escapeHtml(String(score))}<small class="tk-score__of">/10</small></span>`}
+        <span class="rec-pill rec-pill--${VERDICT_TONE[verdict] || 'info'}">verdict ${escapeHtml(verdict)}</span>
+        ${crumb ? `<span class="tk-crumb">${crumb}</span>` : ''}
+      </div>
+    </section>`;
+  }
+
+  // ---- the session log (task 1003437) ----------------------------------------
+  // Every session that worked THIS task — a claim seals one session_logs row at
+  // ship or release — on the task's own page rather than only on /sessions.
+  // Newest first; each is a small ledger: when and who, how long, the outcome,
+  // what the task paid, the handoff. The handoff is the template markdown and
+  // renders through the shared reader behind a disclosure, so a long note does
+  // not push the actions off the page. No sessions → nothing, like the visual.
+  // /sessions has no per-session address today, so "Full log" opens the room.
+  const OUTCOME_TONE = { shipped: 'ok', partial: 'warn', abandoned: 'danger' };
+  function fmtDuration(startIso, endIso) {
+    const ms = Date.parse(endIso) - Date.parse(startIso);
+    if (!Number.isFinite(ms) || ms < 0) return '';
+    const min = Math.round(ms / 60000);
+    if (min < 60) return `${min}m`;
+    const h = Math.floor(min / 60);
+    const m = min % 60;
+    return m ? `${h}h ${String(m).padStart(2, '0')}m` : `${h}h`;
+  }
+  function builderLink(row) {
+    const who = (row && (row.display_name || row.github_login)) || 'a builder';
+    if (!row || row.builder_id == null) return escapeHtml(who);
+    return `<a class="rec-dep" href="${hallBase()}/profile?id=${encodeURIComponent(String(row.builder_id))}">${escapeHtml(who)}</a>`;
+  }
+  function sessionLogHtml(logs, t) {
+    if (!Array.isArray(logs) || !logs.length) return '';
+    const label = curLabel();
+    const paidLabel = label.charAt(0).toUpperCase() + label.slice(1);
+    const blocks = logs.map((s) => {
+      const when = s.ended_at ? fmtDate(s.ended_at) : '';
+      const dur = fmtDuration(s.started_at, s.ended_at);
+      const outcome = String(s.outcome || 'unknown').toLowerCase();
+      const rows = [
+        `<div class="tk-lrow"><span class="tk-lk">Session</span><span class="tk-lv">${[escapeHtml(when), builderLink(s), escapeHtml(dur)].filter(Boolean).join(', ')}</span></div>`,
+        `<div class="tk-lrow"><span class="tk-lk">Outcome</span><span class="tk-lv"><span class="rec-pill rec-pill--sm rec-pill--${OUTCOME_TONE[outcome] || 'info'}">${escapeHtml(outcome)}</span></span></div>`,
+      ];
+      if (outcome === 'shipped' && t && t.credits_awarded != null) {
+        rows.push(`<div class="tk-lrow"><span class="tk-lk">${escapeHtml(paidLabel)} awarded</span><span class="tk-lv">${escapeHtml(String(t.credits_awarded))}</span></div>`);
+      }
+      if (s.notes_md) {
+        rows.push(`<div class="tk-lrow"><span class="tk-lk">Handoff</span><details class="tk-handoff"><summary>Read the handoff notes</summary><div class="rec-desc rec-md">${descriptionHtml(s.notes_md)}</div></details></div>`);
+      }
+      rows.push(`<div class="tk-lrow"><span class="tk-lk">Full log</span><span class="tk-lv"><a class="rec-dep" href="${hallBase()}/sessions">Open in Sessions</a></span></div>`);
+      return `<div class="tk-ledger">${rows.join('')}</div>`;
+    });
+    const count = logs.length > 1 ? ` <em>${logs.length} sessions</em>` : '';
+    return `<section class="rec-sec" id="task-sessions"><p class="rec-sec__h">Session log${count}</p>${blocks.join('')}</section>`;
+  }
+
+  function renderTask(t, lastClaim, viewer, goal, extras) {
+    const { grade = null, sessionLogs = null, claims = null } = extras || {};
     document.title = `#${t.id} — ${t.title || 'Task'} — ${document.title.split(' — ').pop()}`;
     const sub = $('#task-sub');
     if (sub) sub.textContent = `Task ${t.id}`;
@@ -382,17 +482,26 @@
     ]));
     parts.push(lifecycleRail(t, creditVal));
 
-    // Claim history (task 751 — best-effort: the API surfaces the most recent
-    // claim holder, not the full ledger of every past claim).
-    if (lastClaim) {
-      const who = escapeHtml(lastClaim.github_login || 'a builder');
-      const when = lastClaim.claimed_at ? fmtDate(lastClaim.claimed_at, { time: true }) : '';
-      const state = lastClaim.released_at
-        ? (lastClaim.outcome ? `released, outcome ${escapeHtml(lastClaim.outcome)}` : 'released')
+    // Claim history (task 751). The most recent holder leads; since task 1003437
+    // the read also carries the FULL claim ledger (?include=claims), so a task
+    // worked across several claims lists the earlier ones beneath. The old
+    // "most recent only" caveat is gone because it is no longer true.
+    const ledger = Array.isArray(claims) ? claims : [];
+    if (lastClaim || ledger.length) {
+      const head = lastClaim || ledger[0];
+      const who = escapeHtml(head.github_login || head.display_name || 'a builder');
+      const when = head.claimed_at ? fmtDate(head.claimed_at, { time: true }) : '';
+      const state = head.released_at
+        ? (head.outcome ? `released, outcome ${escapeHtml(head.outcome)}` : 'released')
         : 'holding this claim';
-      parts.push(`<section class="rec-sec"><p class="rec-sec__h">Claim</p>` +
-        `<p class="rec-body">Last claimed by <b>${who}</b>${when ? ' on ' + escapeHtml(when) : ''}. ${state}.` +
-        ` The task feed carries the most recent claim only, never the full ledger.</p></section>`);
+      const prior = ledger.slice(1).map((c) => {
+        const cWhen = c.claimed_at ? fmtDate(c.claimed_at, { time: true }) : '';
+        const cState = c.released_at ? escapeHtml(c.outcome || 'released') : 'holding';
+        return `<li>${builderLink(c)}${cWhen ? ' · ' + escapeHtml(cWhen) : ''} · ${cState}</li>`;
+      }).join('');
+      parts.push(`<section class="rec-sec"><p class="rec-sec__h">Claim${ledger.length > 1 ? ` <em>${ledger.length} claims</em>` : ''}</p>` +
+        `<p class="rec-body">Last claimed by <b>${who}</b>${when ? ' on ' + escapeHtml(when) : ''}. ${state}.</p>` +
+        (prior ? `<ul class="rec-files tk-claims">${prior}</ul>` : '') + '</section>');
     }
 
     if (t.value_summary) {
@@ -429,6 +538,12 @@
         `<p class="rec-body">Shipped ${escapeHtml(fmtDate(t.shipped_at))}${by}, ${escapeHtml(fmtRel(t.shipped_at))}.</p></section>`);
     }
 
+    // task 1003437: the grade panel and the session log — the owner's two net-new
+    // regions, between the record and its actions as the mocks compose them.
+    // Each draws nothing when it has nothing.
+    parts.push(gradePanelHtml(grade));
+    parts.push(sessionLogHtml(sessionLogs, t));
+
     // task 759: lifecycle action from the browser (no /builder-* CLI). The server
     // enforces every rank + ownership gate (ADR 0016); we only surface the control
     // the caller is actually allowed to use, and map any refusal to a friendly
@@ -529,7 +644,7 @@
     const id = taskIdFromPath();
     if (!id) { renderNotFound(''); return; }
     try {
-      const d = await getJSON(`${API}/tasks/${encodeURIComponent(id)}?include=last_claim`, { softAuth: true });
+      const d = await getJSON(`${API}/tasks/${encodeURIComponent(id)}?include=last_claim,grade,session_logs,claims`, { softAuth: true });
       if (!d || !d.task) { renderNotFound(id); return; }
       // Best-effort viewer probe so we know which lifecycle control (if any) the
       // caller is allowed to see. A failure just means no action buttons render —
@@ -556,7 +671,11 @@
         if (ri && typeof ri.blob_base === 'string') repoBlobBase = ri.blob_base;
         if (ri && typeof ri.tree_base === 'string') repoTreeBase = ri.tree_base;
       } catch (_) { /* links degrade to the original href */ }
-      renderTask(d.task, d.last_claim || null, viewer, goal);
+      renderTask(d.task, d.last_claim || null, viewer, goal, {
+        grade: d.grade || null,
+        sessionLogs: Array.isArray(d.session_logs) ? d.session_logs : null,
+        claims: Array.isArray(d.claims) ? d.claims : null,
+      });
     } catch (err) {
       if (err && err.code === AUTH_NEEDED) { toAuth(); return; }
       if (err && err.status === 404) { renderNotFound(id); return; }

package/modules/hall-ui/public/task.states.json

@@ -7,7 +7,7 @@
   "ignoreRequests": ["/api/(gds|bongos)/"],
   "ignoreConsole": ["/api/(gds|bongos)/", "\b401\b", "\b404\b", "Unauthorized", "Not Found", "load failed"],
   "states": {
-    "record": { "auth": true, "actions": [["wait", 600]], "expect": { "visible": ["#task-body", "#copy-prompt-btn"] } },
+    "record": { "auth": true, "actions": [["wait", 600]], "expect": { "visible": ["#task-body", "#task-grade", "#task-sessions", "#copy-prompt-btn"] } },
     "from-goal": { "auth": true, "url": "/task/2166?from=%2Fgoals%23%2Fgoal%2F1000081", "actions": [["wait", 600]], "expect": { "visible": ["#task-back", "#copy-prompt-btn"], "attr": [["#task-back", "href", "/goals#/goal/1000081"]] } },
     "missing": { "auth": true, "url": "/task/1003999", "actions": [["wait", 600]], "expect": { "visible": ["#task-h"] } }
   },

package/modules/lifecycle/db-claim-reads.js

@@ -168,11 +168,51 @@ async function getTaskGrade(taskId) {
   return { ...row, issues: Array.isArray(fromSignals) ? fromSignals : [] };
 }
 
+// getTaskClaims — the FULL claim ledger for a task, newest first (task 1003437).
+// getLastClaimHolder is LIMIT 1 of this same read; the hall's task page shows
+// the whole ledger so a task worked by several builders across several claims
+// says so, instead of naming only the last hand. Same columns as the last-holder
+// read plus the claim id, the worktree and the display name; like that read,
+// deliberately NOT the holder's rank (#678). Pure read, builder-agnostic.
+async function getTaskClaims(taskId) {
+  const { rows } = await pool.query(
+    `SELECT c.id AS claim_id, c.builder_id, c.claimed_at, c.released_at, c.outcome,
+            c.worktree_name, b.github_login, b.display_name
+       FROM claims c
+       JOIN builders b ON b.id = c.builder_id
+      WHERE c.task_id = $1
+      ORDER BY c.claimed_at DESC, c.id DESC`,
+    [taskId]
+  );
+  return rows;
+}
+
+// getTaskSessionLogs — every sealed session on a task, newest first (task
+// 1003437). session_logs is keyed by task_id directly (a claim seals one row at
+// ship or release), so there is no claim hop; joined to the builder for the
+// name. notes_md is the handoff-template markdown the hall renders through its
+// reader. Until this read the table had no API surface at all — write-only.
+async function getTaskSessionLogs(taskId) {
+  const { rows } = await pool.query(
+    `SELECT s.id, s.builder_id, s.claim_id, s.worktree_name, s.started_at, s.ended_at,
+            s.outcome, s.notes_md, s.confidence,
+            b.github_login, b.display_name
+       FROM session_logs s
+       JOIN builders b ON b.id = s.builder_id
+      WHERE s.task_id = $1
+      ORDER BY s.ended_at DESC, s.id DESC`,
+    [taskId]
+  );
+  return rows;
+}
+
 module.exports = {
   activeClaimCountForBuilder,
   getActiveClaimsForBuilder,
   getActiveClaimsForSession,
   getLastClaimHolder,
+  getTaskClaims,
   getTaskGrade,
+  getTaskSessionLogs,
   moduleScopeForActiveClaims,
 };

package/modules/lifecycle/db.js

@@ -190,7 +190,9 @@ const {
   getActiveClaimsForBuilder,
   getActiveClaimsForSession,
   getLastClaimHolder,
+  getTaskClaims,
   getTaskGrade,
+  getTaskSessionLogs,
   moduleScopeForActiveClaims,
 } = require('./db-claim-reads.js');
 const {
@@ -330,6 +332,8 @@ module.exports = {
   getActiveClaimsForSession,
   getLastClaimHolder, // --regrade ownership guard (#678)
   getTaskGrade,       // V4.R08 — diff-hash reuse check before grading
+  getTaskClaims,      // task 1003437 — the task page's claim ledger (?include=claims)
+  getTaskSessionLogs, // task 1003437 — the task page's session log (?include=session_logs)
   getBuildersRoster,
   getActiveClaimsForBuilderWithStale,
   releaseClaimOnBehalf,

package/modules/lifecycle/lifecycle.js

@@ -53,6 +53,12 @@ module.exports = {
   // nobody surfaces is merely stored.
   openTaskRecommendationsFor: (...a) => taskRecommendations.countOpenForRecipient(...a),
   getTask: (...a) => db.getTask(...a),
+  // findTaskBySourceRef: the create-vector idempotency read (task 1003114 / R03
+  // of goal 1000074). copy-desk turns an artist's copy proposal into a task and
+  // needs the same "this (source, source_ref) already minted a task" check the
+  // core POST /tasks vector does, so a double-submitted proposal resolves to the
+  // task it already made instead of littering the board with twins.
+  findTaskBySourceRef: (...a) => db.findTaskBySourceRef(...a),
   listVersions: (...a) => db.listVersions(...a),
   versionProgress: (...a) => db.versionProgress(...a),
   recentShippedTasks: (...a) => db.recentShippedTasks(...a),

package/modules/lifecycle/routes/tasks.js

@@ -254,15 +254,26 @@ module.exports = function buildTasksRouter() {
     const includes = String(req.query.include || '').split(',');
     const wantLastClaim = includes.includes('last_claim');
     const wantGrade = includes.includes('grade');
-    const [dependencies, dependents, lastClaim, grade] = await Promise.all([
+    // task 1003437: the hall's task page draws a grade panel, a session log and
+    // the full claim ledger — two more opt-in siblings on this same read (the
+    // grade already rode it), so the hot path still pays for nothing it was not
+    // asked for. Each is ABSENT when not asked and an array (maybe empty) when
+    // asked, so a page can tell "none" from "not requested".
+    const wantSessionLogs = includes.includes('session_logs');
+    const wantClaims = includes.includes('claims');
+    const [dependencies, dependents, lastClaim, grade, sessionLogs, claims] = await Promise.all([
       db.getTaskDependencies(id),
       db.getTaskDependents(id),
       wantLastClaim ? db.getLastClaimHolder(id) : Promise.resolve(null),
       wantGrade ? db.getTaskGrade(id) : Promise.resolve(null),
+      wantSessionLogs ? db.getTaskSessionLogs(id) : Promise.resolve(null),
+      wantClaims ? db.getTaskClaims(id) : Promise.resolve(null),
     ]);
     const body = { task: { ...task, dependencies, dependents } };
     if (wantLastClaim) body.last_claim = lastClaim;
     if (wantGrade) body.grade = grade; // null when no grade exists yet
+    if (wantSessionLogs) body.session_logs = sessionLogs;
+    if (wantClaims) body.claims = claims;
     res.json(body);
   });
 

package/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.491",
+  "version": "1.19.493",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bongos/core",
-      "version": "1.19.491",
+      "version": "1.19.493",
       "license": "AGPL-3.0-or-later",
       "dependencies": {
         "express": "^4.21.2",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.491",
+  "version": "1.19.493",
   "description": "Cloud Bongos — the AI-first build platform core (GDS + platform surfaces + module system), installed as a versioned dependency (ADR 0108).",
   "license": "AGPL-3.0-or-later",
   "main": "src/platform-server.js",