npm - @bongos/core - Versions diffs - 1.19.448 → 1.19.449 - Mend

@bongos/core 1.19.448 → 1.19.449

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 (25) hide show

package/.bongos-core.json +55 -25
package/docs/copy-inventory.md +60 -49
package/docs/copy-registry.json +249 -117
package/docs/module-api-changelog.md +2 -0
package/modules/hall-ui/public/idea.css +138 -71
package/modules/hall-ui/public/idea.html +13 -3
package/modules/hall-ui/public/idea.js +17 -4
package/modules/hall-ui/public/idea.states.json +14 -0
package/modules/hall-ui/public/ideas.css +148 -107
package/modules/hall-ui/public/ideas.html +15 -5
package/modules/hall-ui/public/ideas.states.json +15 -0
package/modules/hall-ui/public/record.css +324 -0
package/modules/hall-ui/public/style.css +173 -39
package/modules/hall-ui/public/task.css +71 -0
package/modules/hall-ui/public/task.html +15 -32
package/modules/hall-ui/public/task.js +142 -39
package/modules/hall-ui/public/task.states.json +14 -0
package/modules/hall-ui/public/work.css +4 -15
package/modules/ui-design/kit/serve.js +7 -1
package/package-lock.json +2 -2
package/package.json +1 -1
package/scripts/hall-preview/server.js +5 -0
package/src/module-api.js +1 -1
package/tests/hall_audit.mjs +4 -1
package/tests/hall_record_world.mjs +412 -0

package/modules/hall-ui/public/task.html CHANGED Viewed

@@ -9,35 +9,11 @@
   <link rel="preconnect" href="https://fonts.googleapis.com">
   <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
   <link rel="stylesheet" href="/builders/style.css?v=2026-07-03-roadmap1">
-  <style>
-    /* Page-local styles for the task-detail page (task 751) — the full-page
-       replacement for the old #/task/N overlay (task-detail.js). Reuses the
-       overlay's .deeplink-card__*/.td-* content classes (style.css) since the
-       body markup is the same shape, just not wrapped in a modal panel. */
-    .task-detail__back { font-size: 13px; color: var(--ink-soft); text-decoration: none; display: inline-block; margin-bottom: 10px; }
-    .task-detail__back:hover { color: var(--ink); text-decoration: underline; }
-    .task-status-pill {
-      display: inline-flex; align-items: center; height: 20px; padding: 0 10px;
-      border-radius: 999px; font-size: 12px; font-weight: 600;
-      background: var(--surface-2); border: 1px solid var(--border); color: var(--ink-soft);
-      text-transform: capitalize;
-    }
-    .task-status-pill--ready,
-    .task-status-pill--shipped,
-    .task-status-pill--confirmed { background: var(--ok-weak); color: var(--ok); border-color: color-mix(in srgb, var(--ok) 30%, transparent); }
-    .task-status-pill--active,
-    .task-status-pill--completed { background: var(--accent-weak); color: var(--accent-text); border-color: color-mix(in srgb, var(--accent) 30%, transparent); }
-    .task-status-pill--blocked,
-    .task-status-pill--backlog { background: var(--warn-weak); color: var(--warn); border-color: color-mix(in srgb, var(--warn) 30%, transparent); }
-    .task-detail__claim {
-      font-size: 13px; color: var(--ink-soft); margin: 0 0 12px;
-      display: flex; flex-wrap: wrap; gap: 4px 8px; align-items: baseline;
-    }
-    .task-detail__claim strong { color: var(--ink); font-weight: 600; }
-  </style>
+  <!-- The record archetype (shared with idea.html) and this page's own two
+       shapes. Promoted out of an inline <style> by task 1003305 — the block was
+       the hall's last substantive one and sat at its hall_audit ratchet. -->
+  <link rel="stylesheet" href="/builders/record.css">
+  <link rel="stylesheet" href="/builders/task.css">
 </head>
 <body data-page="task" data-page-title="Task">
@@ -51,14 +27,21 @@
         <!-- task 2175: the breadcrumb sits in the page head, above the card, so a
              deep-linked visitor (this page is deliberately not in the nav) has a
              way back before the task body has loaded. -->
-        <a class="task-detail__back" href="/work">← Back to Tasks</a>
-        <p class="page-head__sub" id="task-sub">Loading task…</p>
+        <div class="rec-crumb">
+          <a class="rec-back" href="/work">
+            <svg class="rec-back__ic" viewBox="0 0 16 16" aria-hidden="true"><path d="M9.4 3.6 5 8l4.4 4.4"/><path d="M5 8h7.2"/></svg>Back to tasks</a>
+          <p class="rec-sub" id="task-sub">Loading task</p>
+        </div>
       </div>
       <main id="hall-main">
         <section class="scroll" id="task-scroll" aria-labelledby="task-h">
           <div id="task-body">
-            <p class="deeplink-card__title" id="task-h">Loading…</p>
+            <!-- The h1 is present in EVERY state, including this one: the
+                 one-header-main-h1 floor measures the rendered DOM, and a page
+                 whose heading only appears once a fetch lands has none for the
+                 whole of the load, the not-found and the error renders. -->
+            <h1 class="rec-big" id="task-h">Loading the task</h1>
           </div>
         </section>
       </main>

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

@@ -141,12 +141,77 @@
     return `${Math.floor(sec / 86400)}d ago`;
   }
+  // The status vocabulary, which is the ONE sanctioned exception to the world's
+  // Fifteen Rule (DESIGN.md §Reuse) — a lifecycle state is verification state, so
+  // it keeps green/amber/red and never borrows the accent. The accent is for what
+  // is pressable or live, and a status pill is neither.
+  //
+  // `backlog` is DELIBERATELY absent, and that is a change: the retired inline
+  // block painted it amber, identically to `blocked`. Those are not the same
+  // fact. Amber says "something is wrong here", which is true of a task blocked
+  // behind an unshipped dependency and false of one simply not scheduled yet, and
+  // conflating them made every unscheduled task read as a problem. Backlog takes
+  // the neutral pill; the lifecycle rail's first beat carries the distinction in
+  // words ("not ready to be claimed yet" vs "waiting on another task").
+  const PILL_TONE = {
+    shipped: 'ok', confirmed: 'ok',
+    active: 'info', completed: 'info', ready: 'info',
+    blocked: 'warn',
+  };
+  function statusPill(status, extraClass) {
+    const s = String(status || 'unknown');
+    const tone = PILL_TONE[s];
+    return `<span class="rec-pill${tone ? ` rec-pill--${tone}` : ''}${extraClass ? ` ${extraClass}` : ''}">${escapeHtml(s)}</span>`;
+  }
   function depList(kindWord, items) {
     const rows = items.map((d) => {
-      const status = d.status ? `<span class="td-dep-status">${escapeHtml(d.status)}</span>` : '';
-      return `<li><a class="td-dep" href="${hallBase()}/task/${escapeHtml(String(d.id))}">#${escapeHtml(String(d.id))} — ${escapeHtml(d.title || 'task')}</a>${status}</li>`;
+      const status = d.status ? statusPill(d.status, 'rec-pill--sm') : '';
+      return `<li><a class="rec-dep" href="${hallBase()}/task/${escapeHtml(String(d.id))}">` +
+        `<b class="rec-dep__id">${escapeHtml(String(d.id))}</b>${escapeHtml(d.title || 'task')}</a>${status}</li>`;
     }).join('');
-    return `<div class="td-deps"><p class="td-section-h">${escapeHtml(kindWord)}</p><ul class="td-dep-list">${rows}</ul></div>`;
+    return `<section><p class="rec-sec__h">${escapeHtml(kindWord)}</p><ul class="rec-deplist">${rows}</ul></section>`;
+  }
+  // The lifecycle rail: five beats, exactly one lit. Six GDS statuses fold into
+  // five because backlog / blocked / ready all mean "no builder holds it yet";
+  // the first beat's caption is what says WHICH of the three it is, so the rail
+  // never implies a claim that has not happened.
+  const RAIL_AT = { active: 1, completed: 2, confirmed: 3, shipped: 4 };
+  const CLAIM_CAPTION = {
+    backlog: 'not ready to be claimed yet',
+    blocked: 'waiting on another task',
+    ready: 'waiting for a builder',
+  };
+  function lifecycleRail(t, credits) {
+    const at = RAIL_AT[t.status] ?? 0;
+    const landed = t.status === 'shipped' && credits != null;
+    const beats = [
+      ['Claim', CLAIM_CAPTION[t.status] || 'one builder, one worktree'],
+      ['Build', 'the code and the docs'],
+      ['Grade', 'the panel reads the diff'],
+      ['Ship', 'merged, then deployed'],
+      ['Credit', landed ? `${credits} ${window.OTBBranding?.taskCreditLabel?.(t) ?? curLabel()} landed` : 'credits land on the merge'],
+    ];
+    const steps = beats.map(([name, caption], i) =>
+      `<li class="tk-step"${i === at ? ' aria-current="step"' : ''}>` +
+      `<div class="tk-step__dot"></div><span class="tk-step__n">${escapeHtml(name)}</span>` +
+      `<span class="tk-step__c">${escapeHtml(caption)}</span></li>`).join('');
+    return `<ol class="tk-rail" aria-label="Lifecycle">${steps}</ol>`;
+  }
+  // The figure band — v2's stat-tile grid, the reference for any figure row the
+  // hall renders. Only figures the record actually CARRIES are drawn: most tasks
+  // have no recorded actual, and a band of dashes reads as a broken record
+  // rather than an honest one. Under two figures there is no band at all.
+  function figureBand(figs) {
+    const present = figs.filter((f) => f && f[1] != null && f[1] !== '');
+    if (present.length < 2) return '';
+    const cells = present.map(([label, value, unit]) =>
+      `<div class="rec-fig"><span class="rec-fig__l">${escapeHtml(label)}</span>` +
+      `<span class="rec-fig__v">${escapeHtml(String(value))}` +
+      `${unit ? `<u class="rec-fig__u">${escapeHtml(unit)}</u>` : ''}</span></div>`).join('');
+    return `<div class="rec-figs">${cells}</div>`;
   }
   // task 1003109 (goal 1000072, "Human legibility") — the ship-time visual.
@@ -164,11 +229,11 @@
     if (!/^\/api\/(gds|bongos)\/task-visuals\/task-\d+-[0-9a-f]{16}\.(png|jpg|gif|webp)$/.test(url)) return '';
     const alt = String(t.visual_alt || `Visual for task ${t.id}`);
     const caption = t.visual_alt
-      ? `<figcaption class="td-visual__caption">${escapeHtml(t.visual_alt)}</figcaption>`
+      ? `<figcaption class="tk-visual__caption">${escapeHtml(t.visual_alt)}</figcaption>`
       : '';
-    return `<figure class="td-visual">` +
+    return `<figure class="tk-visual">` +
       `<a href="${escapeHtml(url)}" target="_blank" rel="noopener">` +
-      `<img class="td-visual__img" src="${escapeHtml(url)}" alt="${escapeHtml(alt)}" loading="lazy">` +
+      `<img class="tk-visual__img" src="${escapeHtml(url)}" alt="${escapeHtml(alt)}" loading="lazy">` +
       `</a>${caption}</figure>`;
   }
@@ -196,29 +261,33 @@
   function renderTask(t, lastClaim, viewer, goal) {
     document.title = `#${t.id} — ${t.title || 'Task'} — ${document.title.split(' — ').pop()}`;
     const sub = $('#task-sub');
-    if (sub) sub.textContent = `Task #${t.id}`;
+    if (sub) sub.textContent = `Task ${t.id}`;
     const parts = [];
-    parts.push(`<p class="deeplink-card__kind">TASK <span class="task-status-pill task-status-pill--${escapeHtml(t.status || '')}">${escapeHtml(t.status || 'unknown')}</span></p>`);
-    parts.push(`<h1 class="deeplink-card__title" id="task-h">#${t.id}${t.title ? ' — ' + escapeHtml(t.title) : ''}</h1>`);
+    parts.push(`<p class="rec-kind">Task ${statusPill(t.status)}</p>`);
+    parts.push(`<h1 class="rec-title" id="task-h"><span class="rec-title__id">${escapeHtml(String(t.id))}</span>${escapeHtml(t.title || 'Untitled task')}</h1>`);
+    // The facts that place the task, in one line: what it belongs to, what KIND
+    // of work it is, and who is allowed to take it. The figures moved out of this
+    // run into the band below — a credit total set as one more comma-separated
+    // word is the thing the round's stat-tile reference exists to fix.
     const meta = [];
-    if (t.version_id) meta.push(escapeHtml(t.version_id));
+    if (t.version_id) meta.push(`<b>${escapeHtml(t.version_id)}</b>`);
     if (t.kind) meta.push(escapeHtml(t.kind));
     if (t.discipline && t.discipline !== 'unclassified') meta.push(escapeHtml(t.discipline));
-    if (t.priority != null) meta.push('P' + escapeHtml(String(t.priority)));
-    meta.push('min: ' + escapeHtml(rankText(t.requires_rank)));
-    const creditVal = window.OTBBranding?.taskCreditValue?.(t) ?? t.credits_reward;
-    if (creditVal != null) meta.push(`${creditVal} ${window.OTBBranding?.taskCreditLabel?.(t) ?? curLabel()}`);
-    if (t.est_minutes) meta.push(`${t.est_minutes}m`);
-    parts.push(`<p class="deeplink-card__meta">${meta.join(' · ')}</p>`);
+    // rankText() answers "any rank" for an open task, which read as
+    // "min rank any rank" once the label was prefixed. An open floor is not a
+    // minimum, so it is said as the absence of one.
+    const floor = rankText(t.requires_rank);
+    meta.push(escapeHtml(floor === 'any rank' ? 'open to any rank' : `min rank ${floor}`));
+    parts.push(`<p class="rec-meta">${meta.join(' &middot; ')}</p>`);
     // task 1002955: when was this written? The panel showed a shipped date but
     // never a created one, so an untouched task carried no age at all. Absolute
     // date first (the panel has the room the card chip doesn't), age after it.
     if (t.created_at) {
       const age = fmtRel(t.created_at);
-      parts.push(`<p class="deeplink-card__note">Created ${escapeHtml(fmtDate(t.created_at))}${age ? ` — ${escapeHtml(age)}` : ''}.</p>`);
+      parts.push(`<p class="rec-note">Created ${escapeHtml(fmtDate(t.created_at))}${age ? `, ${escapeHtml(age)}` : ''}.</p>`);
     }
     if (t.goal_id) {
@@ -227,24 +296,35 @@
       // read "Goal #1000072", which tells a person nothing they didn't already
       // know, and the chip answers "what kind of work is this" without opening
       // the goal. Marked inherited because a task carries no category of its own.
-      const goalName = goal && goal.title ? ` — ${escapeHtml(goal.title)}` : '';
+      const goalName = goal && goal.title ? `, ${escapeHtml(goal.title)}` : '';
       const catChip = window.OTB.categoryChip(goal && goal.category, { inherited: true });
-      parts.push(`<p class="td-goal">Goal: <a href="/goals#/goal/${encodeURIComponent(t.goal_id)}">Goal #${escapeHtml(String(t.goal_id))}</a>${goalName}${catChip}</p>`);
+      parts.push(`<p class="rec-goal">Goal <a href="/goals#/goal/${encodeURIComponent(t.goal_id)}">${escapeHtml(String(t.goal_id))}${goalName}</a>${catChip}</p>`);
     }
+    const creditVal = window.OTBBranding?.taskCreditValue?.(t) ?? t.credits_reward;
+    parts.push(figureBand([
+      [window.OTBBranding?.taskCreditLabel?.(t) ?? curLabel(), creditVal],
+      ['Estimated', t.est_minutes, 'm'],
+      ['Actual', t.actual_minutes, 'm'],
+      ['Priority', t.priority == null ? null : 'P' + t.priority],
+    ]));
+    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 (${escapeHtml(lastClaim.outcome)})` : 'released')
+        ? (lastClaim.outcome ? `released, outcome ${escapeHtml(lastClaim.outcome)}` : 'released')
         : 'holding this claim';
-      parts.push(`<p class="task-detail__claim">Last claimed by <strong>${who}</strong> ${when ? 'on ' + escapeHtml(when) : ''} — ${state}.</p>`);
+      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>`);
     }
     if (t.value_summary) {
-      parts.push(`<p class="td-goal"><strong>Value:</strong> ${escapeHtml(t.value_summary)}</p>`);
+      parts.push(`<section class="rec-sec"><p class="rec-sec__h">Value</p><p class="rec-body">${escapeHtml(t.value_summary)}</p></section>`);
     }
     // task 1003109: the ship-time VISUAL — the same claim as the value summary in
@@ -252,23 +332,29 @@
     // isn't. Absent on almost every task, and that is not an omission: a task with
     // no visual must never look incomplete, so nothing is drawn at all when there
     // is none — no placeholder, no empty frame, no "no visual" line.
-    if (t.visual_url) parts.push(taskVisualHtml(t));
+    if (t.visual_url) parts.push(`<section class="rec-sec">${taskVisualHtml(t)}</section>`);
     if (t.description) {
-      parts.push(`<div class="td-desc">${escapeHtml(t.description)}</div>`);
+      parts.push(`<section class="rec-sec"><p class="rec-sec__h">Description</p><p class="rec-desc">${escapeHtml(t.description)}</p></section>`);
     }
     const touches = Array.isArray(t.touches) ? t.touches : [];
-    if (touches.length) parts.push(`<p class="td-touches">✎ ${touches.map(escapeHtml).join(', ')}</p>`);
+    if (touches.length) {
+      const files = touches.map((f) => `<li>${escapeHtml(f)}</li>`).join('');
+      parts.push(`<section class="rec-sec"><p class="rec-sec__h">Touches <em>${touches.length} ${touches.length === 1 ? 'file' : 'files'}</em></p>` +
+        `<ul class="rec-files">${files}</ul></section>`);
+    }
     const deps = Array.isArray(t.dependencies) ? t.dependencies : [];
     const dependents = Array.isArray(t.dependents) ? t.dependents : [];
-    if (deps.length) parts.push(depList('Blocked by', deps));
-    if (dependents.length) parts.push(depList('Blocks', dependents));
+    if (deps.length || dependents.length) {
+      parts.push(`<div class="rec-sec rec-depcols">${deps.length ? depList('Blocked by', deps) : ''}${dependents.length ? depList('Blocks', dependents) : ''}</div>`);
+    }
     if (t.shipped_at) {
-      const by = t.shipped_by ? ` by builder #${escapeHtml(String(t.shipped_by))}` : '';
-      parts.push(`<p class="deeplink-card__note">Shipped ${escapeHtml(fmtRel(t.shipped_at))}${by}.</p>`);
+      const by = t.shipped_by ? ` by builder ${escapeHtml(String(t.shipped_by))}` : '';
+      parts.push(`<section class="rec-sec"><p class="rec-sec__h">Shipped</p>` +
+        `<p class="rec-body">Shipped ${escapeHtml(fmtDate(t.shipped_at))}${by}, ${escapeHtml(fmtRel(t.shipped_at))}.</p></section>`);
     }
     // task 759: lifecycle action from the browser (no /builder-* CLI). The server
@@ -287,17 +373,25 @@
     } else if (t.status === 'completed') {
       lifecycle = rank === 'archon'
         ? `<button class="td-btn td-btn--primary" type="button" id="confirm-btn">Confirm</button>`
-        : `<p class="td-action-note">Awaiting the grader / a confirmation before it can ship.</p>`;
+        : `<p class="rec-actnote">Awaiting the grader, or a confirmation, before it can ship.</p>`;
+    } else if (t.status === 'shipped') {
+      // A shipped task has no lifecycle control left, so the row says so in words
+      // rather than leaving a gap a reader has to interpret. Nothing here is
+      // pressable-or-live, so nothing here is warm — the One Warm Thing Rule.
+      lifecycle = `<p class="rec-actnote">This task has shipped and its credits have landed. Neither Ship it nor Confirm is available.</p>`;
     }
     // Copy prompt (the "ship link") — the same claim/build/ship prompt the Work
     // Board hands an agent. Purely client-side (clipboard write, no server call).
     parts.push(`
-      <div class="td-actions">
-        ${lifecycle}
-        <button class="td-btn" type="button" id="copy-prompt-btn">Copy prompt</button>
-        <a class="td-btn" href="${hallBase()}/work">Open the Work Board</a>
-      </div>
+      <section class="rec-sec">
+        <p class="rec-sec__h">Actions</p>
+        <div class="rec-actions">
+          ${lifecycle}
+          <button class="td-btn" type="button" id="copy-prompt-btn">Copy prompt</button>
+          <a class="td-btn" href="${hallBase()}/work">Open the work board</a>
+        </div>
+      </section>
     `);
     const body = $('#task-body');
@@ -331,11 +425,16 @@
     }
   }
+  // The two honest failures. Both keep the <h1> — the empty and error renders are
+  // renders like any other, and a page whose only heading lives on the happy path
+  // has none in the states a reader is most likely to hit.
   function renderNotFound(id) {
     const body = $('#task-body');
     if (body) body.innerHTML = `
-      <p class="deeplink-card__title" id="task-h">#${escapeHtml(String(id))} — no such task</p>
-      <p class="deeplink-card__note">Tasks are numbered separately from GitHub pull requests, so this may be a PR rather than a GDS task.</p>
+      <p class="rec-kind">Task ${statusPill('no such task')}</p>
+      <h1 class="rec-big" id="task-h">${escapeHtml(String(id))}, no such task</h1>
+      <p class="rec-body">Tasks in the ledger are numbered separately from GitHub pull requests, so this may be a PR number rather than a task. The work board lists everything that does exist.</p>
+      <div class="rec-actions rec-sec"><a class="td-btn" href="${hallBase()}/work">Open the work board</a></div>
     `;
     const sub = $('#task-sub');
     if (sub) sub.textContent = 'Not found';
@@ -344,9 +443,13 @@
   function renderError(id) {
     const body = $('#task-body');
     if (body) body.innerHTML = `
-      <p class="deeplink-card__title" id="task-h">Could not load #${escapeHtml(String(id))}</p>
-      <p class="deeplink-card__note">Try reloading in a moment.</p>
+      <p class="rec-kind">Task ${statusPill('feed unavailable')}</p>
+      <h1 class="rec-big" id="task-h">Could not load ${escapeHtml(String(id))}</h1>
+      <p class="rec-body">The task feed did not answer. Nothing on the record has changed, only the read failed. Try again in a moment, or open the work board.</p>
+      <div class="rec-actions rec-sec"><a class="td-btn" href="${hallBase()}/work">Open the work board</a></div>
     `;
+    const sub = $('#task-sub');
+    if (sub) sub.textContent = 'Could not load';
   }
   async function load() {

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

@@ -0,0 +1,14 @@
+{
+  "_": "The hall's TASK RECORD through modules/ui-design/kit/render.js (task 1003305). The record's own controls — the dependency links, the lifecycle buttons, the figure band, the way back — only exist once a task has LOADED, so the floors here are run against the hall-preview harness rather than the kit's own stub, which carries the public-landing feeds and not the hall's:\n\n  node scripts/hall-preview/server.js --port 4612 --fixture-me\n  node modules/ui-design/kit/render.js --states modules/hall-ui/public/task.states.json --base-in http://builders.localhost:4612 --base-out http://builders.localhost:4612\n\nRun against the kit's own stubs instead and every state renders `no such task`, which is a real state and is the `missing` one below — just not the one the re-skin changed.",
+  "page": "/task/2166",
+  "surface": "hall-ui",
+  "stub": { "prefix": "/builders/" },
+  "modeQuery": false,
+  "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"] } },
+    "missing": { "auth": true, "url": "/task/1003999", "actions": [["wait", 600]], "expect": { "visible": ["#task-h"] } }
+  },
+  "reducedMotion": { "auth": true, "actions": [["wait", 600]] }
+}

package/modules/hall-ui/public/work.css CHANGED Viewed

@@ -16,21 +16,10 @@
    goals.html LOADS THIS SHEET TOO — a change here lands on both pages. */
-/* The board's page head, in v3's LABEL voice: the display face is saved for
-   section headings, so the page's own name is quiet (mock: .greeting). It
-   overrides the pre-shell .page-head h1 compat rule in style.css, which is
-   22px/650 and belongs to the pages nobody has re-skinned yet. */
-.page-head--board .page-head__h {
-  margin: 0 0 6px;
-  font-family: var(--label-face);
-  font-size: 14px;
-  font-weight: 400;
-  letter-spacing: var(--label-space);
-  text-transform: var(--label-case);
-  color: var(--ink);
-  text-shadow: var(--type-halo);
-}
-.page-head--board .page-head__sub { font-size: 14px; max-width: var(--measure); }
+/* `.page-head--board` MOVED to style.css (task 1003305) when the ideas composer
+   became its third consumer: a modifier on the shell's own .page-head is shell
+   chrome, and a page sheet that only work.html and goals.html load cannot serve
+   a page that loads neither. Same rule, one home. */
 .kbd {
   display: inline-block;

package/modules/ui-design/kit/serve.js CHANGED Viewed

@@ -288,7 +288,13 @@ const server = http.createServer((req, res) => {
   const under = p.startsWith(PREFIX) ? p.slice(PREFIX.length) : (PREFIX !== '/' && (p === PREFIX.slice(0, -1)) ? '' : p.replace(/^\//, ''));
   const asset = fileUnder(DIR, under);
   if (asset && !asset.endsWith('.html')) return sendFile(asset, res);
-  const name = under === '' ? 'index' : under.replace(/\/$/, '').replace(/\.html$/, '');
+  const under2 = under.replace(/\/$/, '').replace(/\.html$/, '');
+  // A record page is `<page>/<id>` — the hall serves /builders/task/2166 and
+  // /builders/idea/12 from task.html and idea.html (serve-internal.js's two
+  // TASK_DETAIL_PAGE_RE / IDEA_DETAIL_PAGE_RE routes). Without this the states
+  // for those pages could only be rendered at a URL production never serves, and
+  // a page script that reads its id out of location.pathname would see none.
+  const name = under2 === '' ? 'index' : (fs.existsSync(path.join(DIR, under2 + '.html')) ? under2 : under2.replace(/\/[^/]+$/, ''));
   if (!name.includes('/') && fs.existsSync(path.join(DIR, name + '.html'))) {
     try { res.writeHead(200, { 'content-type': MIME['.html'], 'cache-control': 'no-store' }); return res.end(page(name + '.html').html); }
     catch (e) { res.writeHead(500, { 'content-type': 'text/plain' }); return res.end('render failed: ' + e.stack); }

package/package-lock.json CHANGED Viewed

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

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.448",
+  "version": "1.19.449",
   "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",

package/scripts/hall-preview/server.js CHANGED Viewed

@@ -222,7 +222,12 @@ function sendPage(res, name) {
 }
 app.get('/', (req, res, next) => { if (!sendPage(res, 'index.html')) next(); });
+// The two RECORD pages are the hall's only two-segment page routes
+// (serve-internal.js TASK_DETAIL_PAGE_RE / IDEA_DETAIL_PAGE_RE). /idea/:id was
+// missing here, so the idea record could not be previewed at all — the harness
+// fell through to the real gate and 302'd to sign-in (task 1003305).
 app.get(/^\/task\/\d+$/, (req, res, next) => { if (!sendPage(res, 'task.html')) next(); });
+app.get(/^\/idea\/\d+$/, (req, res, next) => { if (!sendPage(res, 'idea.html')) next(); });
 app.get(/^\/([a-z-]+)$/, (req, res, next) => {
   if (!sendPage(res, `${req.params[0]}.html`)) next();
 });

package/src/module-api.js CHANGED Viewed

@@ -33,7 +33,7 @@ const { validateOrRespond, LIMITS, parseId, asyncHandler, corsPublicGet, parsePa
 // there. scripts/gds/bump-version.js still rewrites the literal below; it appends
 // the entry to that file. Look for a version's history there, not here.
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.448'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.19.449'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
 // A namespaced logger so a module's log lines are attributable + consistent.
 // Usage: const log = api.logger('dev-box'); log.info('mounted');

package/tests/hall_audit.mjs CHANGED Viewed

@@ -29,7 +29,10 @@ const pages = () => fs.readdirSync(HALL).filter((f) => f.endsWith('.html'));
 // rather than "predates the program" — knowing WHY is what tells the next
 // reader whether promoting it would actually fix anything.
 const INLINE_STYLE_BUDGET = {
-  'task.html': 27,        // the task-751 detail-page block, deliberate
+  // (task.html's 27-line task-751 block was the last substantive inline style in
+  // the hall. Task 1003305 promoted it to task.css + record.css when the re-skin
+  // needed room the ratchet would not give, so the exception is gone with it —
+  // task.html now carries none and passes on the default.)
   // (my-projects + scouting were carried here as standalone surfaces until task
   // 1002868 adopted them into the shell; their exceptions are gone with them.)
   // Genuinely page-local, and already token-derived. One line over the default;