@bongos/core 1.21.173 → 1.21.175

package/modules/hall-ui/public/goal-inbox.js

@@ -46,37 +46,36 @@
 
   // The goal-detail hash route on the Goals page (matches goals.js's row link).
   const goalHref = (goalId) => `/goals#/goal/${encodeURIComponent(goalId)}`;
-  const noteHtml = (note) => (note ? `<p class="ginbox__note">“${escapeHtml(note)}”</p>` : '');
 
   function invitationRow(inv) {
     const who = inv.invited_by_name || inv.invited_by_login || 'someone';
-    return `<li class="ginbox__item" data-goal="${escapeHtml(String(inv.goal_id))}">
-        <div class="ginbox__body">
-          <a class="ginbox__goal" href="${goalHref(inv.goal_id)}" title="Open ${escapeHtml(inv.goal_title || 'goal')}">${escapeHtml(inv.goal_title || 'a goal')}</a>
-          <span class="ginbox__sub">invited by ${escapeHtml(who)}</span>
-          ${noteHtml(inv.note)}
-        </div>
-        <div class="ginbox__actions">
-          <button class="btn-ghost" type="button" data-accept-inv="${escapeHtml(String(inv.id))}">Accept</button>
-          <button class="btn-ghost" type="button" data-decline-inv="${escapeHtml(String(inv.id))}">Decline</button>
-        </div>
-      </li>`;
+    const id = String(inv.id);
+    return window.OTBAsk.html({
+      cls: 'ginbox__item', attrs: { 'data-goal': String(inv.goal_id) },
+      where: `Invited by ${who}`,
+      q: `Join ${inv.goal_title || 'a goal'}?`, qHref: goalHref(inv.goal_id),
+      said: inv.note ? `“${inv.note}”` : '', row: true,
+      opts: [
+        { label: 'Join', result: 'You can claim its work', rec: true, attrs: { 'data-accept-inv': id } },
+        { label: 'No thanks', result: 'The invitation closes', attrs: { 'data-decline-inv': id } },
+      ],
+    });
   }
 
   function requestRow(r) {
     const who = r.display_name || r.github_login || 'a builder';
-    const rank = r.rank ? ` · ${escapeHtml(rankLabel(r.rank, { empty: '' }))}` : '';
-    return `<li class="ginbox__item" data-goal="${escapeHtml(String(r.goal_id))}">
-        <div class="ginbox__body">
-          <a class="ginbox__goal" href="${goalHref(r.goal_id)}" title="Open ${escapeHtml(r.goal_title || 'goal')}">${escapeHtml(r.goal_title || 'a goal')}</a>
-          <span class="ginbox__sub">${escapeHtml(who)}${rank} asks to join</span>
-          ${noteHtml(r.note)}
-        </div>
-        <div class="ginbox__actions">
-          <button class="btn-ghost" type="button" data-approve-req="${escapeHtml(String(r.id))}">Approve</button>
-          <button class="btn-ghost" type="button" data-deny-req="${escapeHtml(String(r.id))}">Deny</button>
-        </div>
-      </li>`;
+    const rank = r.rank ? ` · ${rankLabel(r.rank, { empty: '' })}` : '';
+    const id = String(r.id);
+    return window.OTBAsk.html({
+      cls: 'ginbox__item', attrs: { 'data-goal': String(r.goal_id) },
+      where: `${r.goal_title || 'A goal'}${rank}`,
+      q: `Let ${who} join?`, qHref: goalHref(r.goal_id),
+      said: r.note ? `“${r.note}”` : '', row: true,
+      opts: [
+        { label: 'Let them in', result: 'They can claim its work', rec: true, attrs: { 'data-approve-req': id } },
+        { label: 'Not now', result: 'The request closes', attrs: { 'data-deny-req': id } },
+      ],
+    });
   }
 
   function pendingLine(myPending) {
@@ -289,20 +288,16 @@
   // non-triager's fetch 403s and the slice renders nothing. One gate, on the
   // server, beats a client check that can disagree with it.
   function nodRowHtml(r) {
-    const title = escapeHtml(r.title || `task #${r.task_id}`);
-    const kind = r.kind ? `<span class="ginbox__sub">${escapeHtml(r.kind)}</span>` : '';
-    const who = r.filed_by_login ? ` · filed by ${escapeHtml(r.filed_by_login)}` : '';
-    const goal = r.goal_title ? escapeHtml(r.goal_title) : `goal ${escapeHtml(String(r.goal_id))}`;
-    return `<li class="ginbox__item" data-goal="${escapeHtml(String(r.goal_id))}">
-        <div class="ginbox__body">
-          <a class="ginbox__goal" href="/task/${encodeURIComponent(r.task_id)}">${title}</a>
-          <span class="ginbox__sub">${goal}${who}</span>
-        </div>
-        ${kind}
-        <div class="ginbox__actions">
-          <button class="btn-ghost" type="button" data-nod-task="${escapeHtml(String(r.task_id))}">Give the nod</button>
-        </div>
-      </li>`;
+    const goal = r.goal_title || `goal ${r.goal_id}`;
+    const who = r.filed_by_login ? ` · filed by ${r.filed_by_login}` : '';
+    return window.OTBAsk.html({
+      cls: 'ginbox__item', attrs: { 'data-goal': String(r.goal_id) },
+      where: `${goal}${who}`,
+      q: r.title || `Task ${r.task_id}`, qHref: `/task/${encodeURIComponent(r.task_id)}`,
+      opts: [
+        { label: 'Give the nod', result: 'It opens for anyone to claim', rec: true, attrs: { 'data-nod-task': String(r.task_id) } },
+      ],
+    });
   }
 
   function renderNodQueue(data) {
@@ -366,23 +361,18 @@
   // One word each brings it back into that band; the sentence each answer means
   // rides the `title`, and the head already says what the question is.
   function removalRowHtml(r) {
-    const title = escapeHtml(r.title || `task #${r.proposal_task_id}`);
-    const from = r.ship_task_id
-      ? `deleted by task #${escapeHtml(String(r.ship_task_id))}`
-      : 'from a ship whose id could not be read';
-    const goal = r.goal_title ? ` · ${escapeHtml(r.goal_title)}` : '';
-    return `<li class="ginbox__item ginbox__item--stacked">
-        <div class="ginbox__body">
-          <a class="ginbox__goal" href="/task/${encodeURIComponent(r.proposal_task_id)}">${title}</a>
-          <span class="ginbox__sub">${from}${goal} — open it to see what went</span>
-        </div>
-        <div class="ginbox__actions">
-          <button class="btn-ghost" type="button" data-removal-task="${escapeHtml(String(r.proposal_task_id))}"
-            data-removal-verdict="confirmed" title="That work is genuinely gone">Confirm</button>
-          <button class="btn-ghost" type="button" data-removal-task="${escapeHtml(String(r.proposal_task_id))}"
-            data-removal-verdict="dismissed" title="A refactor, a move, or work already superseded">Dismiss</button>
-        </div>
-      </li>`;
+    const from = r.ship_task_id ? `Deleted by task ${r.ship_task_id}` : 'Deleted by a ship';
+    const id = String(r.proposal_task_id);
+    return window.OTBAsk.html({
+      cls: 'ginbox__item',
+      where: `${from}${r.goal_title ? ' · ' + r.goal_title : ''}`,
+      q: r.title || `Task ${r.proposal_task_id}`, qHref: `/task/${encodeURIComponent(r.proposal_task_id)}`,
+      row: true,
+      opts: [
+        { label: 'It is gone', result: 'That work is really removed', attrs: { 'data-removal-task': id, 'data-removal-verdict': 'confirmed' } },
+        { label: 'Not removed', result: 'It moved or was replaced', attrs: { 'data-removal-task': id, 'data-removal-verdict': 'dismissed' } },
+      ],
+    });
   }
 
   function renderRemovals(data) {
@@ -437,29 +427,23 @@
   }
 
   function rotRowHtml(r) {
-    const href = r.tier === 'goal'
-      ? goalHref(r.id)
-      : `/task/${encodeURIComponent(r.id)}`;
-    const where = r.tier === 'goal'
-      ? 'goal'
-      : (r.goal_title ? escapeHtml(r.goal_title) : `goal ${escapeHtml(String(r.goal_id || '—'))}`);
-    // `watered` is worth showing: a row that is back DESPITE having been watered is
-    // a different message from one nobody has ever looked at, and hiding that would
-    // make the second watering feel like the first one did nothing.
+    const href = r.tier === 'goal' ? goalHref(r.id) : `/task/${encodeURIComponent(r.id)}`;
+    const where = r.tier === 'goal' ? 'Goal' : (r.goal_title || `goal ${r.goal_id || '—'}`);
+    // a row back DESPITE a watering says something a first-timer does not
     const again = r.ever_watered ? ' · watered before' : '';
-    const buttons = (Array.isArray(r.verbs) ? r.verbs : []).map((v) => {
-      const path = String(v.path || '').replace(':id', encodeURIComponent(r.id));
-      return `<button class="btn-ghost" type="button" data-rot-path="${escapeHtml(path)}"`
-        + ` data-rot-verb="${escapeHtml(String(v.verb || ''))}"`
-        + ` title="${escapeHtml(String(v.hint || ''))}">${escapeHtml(String(v.label || v.verb || '?'))}</button>`;
-    }).join('');
-    return `<li class="ginbox__item" data-rot-tier="${escapeHtml(String(r.tier))}">
-        <div class="ginbox__body">
-          <a class="ginbox__goal" href="${escapeHtml(href)}">${escapeHtml(r.title || `${r.tier} #${r.id}`)}</a>
-          <span class="ginbox__sub">${escapeHtml(String(r.status || ''))} · ${where} · ${escapeHtml(untouchedFor(r.last_touched_at))}${again}</span>
-        </div>
-        <div class="ginbox__actions">${buttons}</div>
-      </li>`;
+    const opts = (Array.isArray(r.verbs) ? r.verbs : []).map((v) => ({
+      label: String(v.label || v.verb || '?'),
+      result: String(v.hint || ''),
+      attrs: {
+        'data-rot-path': String(v.path || '').replace(':id', encodeURIComponent(r.id)),
+        'data-rot-verb': String(v.verb || ''),
+      },
+    }));
+    return window.OTBAsk.html({
+      cls: 'ginbox__item', attrs: { 'data-rot-tier': String(r.tier) },
+      where: `${where} · ${untouchedFor(r.last_touched_at)}${again}`,
+      q: r.title || `${r.tier} ${r.id}`, qHref: href, opts,
+    });
   }
 
   // What each verb says when it lands. Keyed by the SERVER's verb name so a new verb
@@ -531,20 +515,19 @@
       titleClass: 'ginbox__goal',
       noteClass: 'ginbox__note',
       whenClass: 'ginbox__sub',
-      // The SAME overlay task-detail.js has rendered since task 1003066's
-      // predecessor — this slice is what gives it its first door in.
       esc: window.OTBKit.esc,
       fmtDate: window.OTBKit.fmtAbsDate,
       blockerHref: (id) => `#/blocker/${encodeURIComponent(id)}`,
       taskHref: (id) => (window.OTBTaskDetail ? window.OTBTaskDetail.href(id) : `/task/${encodeURIComponent(id)}`),
     });
-    return `<li class="ginbox__item">
-        <div class="ginbox__body">
-          ${facts.titleHtml}
-          ${facts.factsHtml}
-          ${facts.noteHtml}
-        </div>
-      </li>`;
+    // Was a link only; now it answers in place: the overlay is where it resolves.
+    return window.OTBAsk.html({
+      cls: 'ginbox__item',
+      where: 'Waiting on the project owner',
+      q: b.title || `Blocker ${b.id}`, qHref: `#/blocker/${encodeURIComponent(b.id)}`,
+      saidHtml: facts.factsHtml + facts.noteHtml,
+      opts: [{ label: 'Resolve it', result: 'Say what settled it', rec: true, href: `#/blocker/${encodeURIComponent(b.id)}` }],
+    });
   }
 
   function renderBlockers(data) {

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

@@ -0,0 +1,57 @@
+/* modules/hall-ui/public/hall-ask.css — THE CITY'S ASK CARD, IN THE HALL
+   (task 1004820, criterion wa6-hall-reads-like-home).
+   The hall's actions are drawn as the city's ask card (street-deck.css `.ask`):
+   where it sits, the question large and light, what was said, then each answer
+   as a one-tap option chip carrying its result line. The hall keeps its own
+   prefix (`hask`) so a page that loads both sheets never cascades one into the
+   other; the tokens are the street's glass chip recipe on the hall's own
+   ground, light first and then twice for dark (the hall's switch). */
+
+body {
+  --hask-chip: color-mix(in srgb, var(--ink) 4%, transparent);
+  --hask-chip-pick: color-mix(in srgb, var(--bg-card) 80%, var(--bg));
+  --hask-edge: color-mix(in srgb, var(--ink) 9%, transparent);
+  --hask-edge-pick: color-mix(in srgb, var(--ink) 34%, transparent);
+}
+html[data-theme="dark"] body {
+  --hask-chip: color-mix(in srgb, var(--ink) 5%, transparent);
+  --hask-chip-pick: color-mix(in srgb, var(--ink) 11%, transparent);
+  --hask-edge: color-mix(in srgb, var(--ink) 11%, transparent);
+  --hask-edge-pick: color-mix(in srgb, var(--ink) 42%, transparent);
+}
+@media (prefers-color-scheme: dark) {
+  html:not([data-theme]) body {
+    --hask-chip: color-mix(in srgb, var(--ink) 5%, transparent);
+    --hask-chip-pick: color-mix(in srgb, var(--ink) 11%, transparent);
+    --hask-edge: color-mix(in srgb, var(--ink) 11%, transparent);
+    --hask-edge-pick: color-mix(in srgb, var(--ink) 42%, transparent);
+  }
+}
+
+/* the card: one item that waits on the reader */
+.hask { list-style: none; display: flex; flex-direction: column; gap: 12px; padding: 16px 16px 14px; margin: 0 0 10px; box-sizing: border-box; border-radius: 22px; background: var(--hask-chip); border: 1px solid var(--hask-edge); color: var(--ink); }
+.hask:last-child { margin-bottom: 0; }
+.hask__where { display: flex; align-items: center; gap: 8px; min-height: 24px; font-size: 13px; color: var(--ink-soft); min-width: 0; }
+.hask__dot { flex: none; width: 7px; height: 7px; border-radius: 50%; background: var(--accent); }
+.hask__where-t { min-width: 0; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
+.hask__q { margin: 0; font-size: 21px; font-weight: 300; letter-spacing: -0.02em; line-height: 1.2; overflow-wrap: anywhere; }
+.hask__q a { color: inherit; text-decoration: none; }
+.hask__q a:hover { text-decoration: underline; text-underline-offset: 4px; text-decoration-color: var(--hask-edge-pick); }
+.hask__said { margin: -4px 0 0; font-size: 14px; line-height: 1.45; color: var(--ink-soft); }
+.hask__opts { display: flex; flex-direction: column; gap: 8px; }
+.hask__opt { display: grid; grid-template-columns: minmax(0, 1fr); row-gap: 2px; align-items: center; min-height: 52px; width: 100%; padding: 9px 16px; box-sizing: border-box; border-radius: 15px; border: 1px solid var(--hask-edge); background: transparent; color: var(--ink); font-family: var(--font-body); text-align: left; text-decoration: none; cursor: pointer; }
+.hask__opt:hover { background: var(--hask-chip-pick); }
+.hask__opt--rec { background: var(--hask-chip-pick); border-color: var(--hask-edge-pick); }
+.hask__opt[disabled] { cursor: progress; opacity: .6; }
+.hask__opt:focus-visible { outline: 2px solid var(--accent); outline-offset: 3px; }
+.hask__label { font-size: 15px; font-weight: 500; min-width: 0; }
+.hask__result { font-size: 13px; line-height: 1.35; color: var(--ink-soft); min-width: 0; }
+@media (min-width: 761px) {
+  .hask__opts--row { flex-direction: row; }
+  .hask__opts--row .hask__opt { flex: 1 1 0; }
+}
+
+/* an inbox row drawn as the card keeps .ginbox__item (the count and the
+   card's visibility read it); the card's own box wins over the ledger row */
+.ginbox > .hask.ginbox__item { display: flex; flex-direction: column; align-items: stretch; gap: 12px; padding: 16px 16px 14px; border: 1px solid var(--hask-edge); margin: 0 0 10px; }
+.ginbox > .hask.ginbox__item:first-child { border-top: 1px solid var(--hask-edge); }

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

@@ -0,0 +1,50 @@
+// modules/hall-ui/public/hall-ask.js — THE CITY'S ASK CARD, IN THE HALL
+// (task 1004820, criterion wa6-hall-reads-like-home).
+//
+// One builder for every hall action that waits on the reader: where it sits,
+// the question, what was said, and each answer as a one-tap option chip with
+// its result line (the city's ask card, street-deck.js, in hall-ask.css).
+//
+// It draws markup only. An option carries the SAME data-* attributes the
+// page's delegated click handler already reads, so no action gains a second
+// path: switching a row to the card changes how it looks, never what it calls.
+// An option with `href` is a link (an act that needs its own panel).
+//
+//   OTBAsk.html({ where, q, qHref, said, saidHtml, tag, cls, attrs,
+//                 row: bool, opts: [{ label, result, rec, href, attrs }] })
+//
+// Every string is escaped here; `saidHtml` is the one pre-built slot, for a
+// caller whose facts are already escaped markup (blockerFacts).
+(function () {
+  const esc = (s) => String(s == null ? '' : s)
+    .replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;')
+    .replace(/"/g, '&quot;').replace(/'/g, '&#39;');
+  const attrsOf = (a) => Object.keys(a || {})
+    .filter((k) => a[k] != null && a[k] !== false)
+    .map((k) => ` ${k}="${esc(a[k])}"`).join('');
+
+  function optHtml(o) {
+    const cls = 'hask__opt' + (o.rec ? ' hask__opt--rec' : '');
+    const inner = `<span class="hask__label">${esc(o.label)}</span>`
+      + (o.result ? `<span class="hask__result">${esc(o.result)}</span>` : '');
+    return o.href
+      ? `<a class="${cls}" href="${esc(o.href)}"${attrsOf(o.attrs)}>${inner}</a>`
+      : `<button class="${cls}" type="button"${attrsOf(o.attrs)}>${inner}</button>`;
+  }
+
+  function html(c) {
+    const tag = c.tag || 'li';
+    const opts = (c.opts || []).filter(Boolean);
+    const q = c.qHref ? `<a href="${esc(c.qHref)}">${esc(c.q)}</a>` : esc(c.q);
+    return `<${tag} class="hask${c.cls ? ' ' + esc(c.cls) : ''}"${attrsOf(c.attrs)}>`
+      + (c.where ? `<div class="hask__where"><span class="hask__dot" aria-hidden="true"></span><span class="hask__where-t">${esc(c.where)}</span></div>` : '')
+      + `<h4 class="hask__q">${q}</h4>`
+      + (c.saidHtml ? `<div class="hask__said">${c.saidHtml}</div>` : (c.said ? `<p class="hask__said">${esc(c.said)}</p>` : ''))
+      + (opts.length ? `<div class="hask__opts${c.row ? ' hask__opts--row' : ''}">${opts.map(optHtml).join('')}</div>` : '')
+      + `</${tag}>`;
+  }
+
+  const api = { html, esc };
+  if (typeof window !== 'undefined') window.OTBAsk = api;
+  if (typeof module !== 'undefined' && module.exports) module.exports = api;
+})();

package/modules/hall-ui/public/index.html

@@ -19,6 +19,7 @@
   <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-02-shell1">
+  <link rel="stylesheet" href="/builders/hall-ask.css">
 </head>
 <body data-page="home">
 
@@ -136,6 +137,8 @@
 <!-- task 1003455: the shared blocker-row renderer (window.OTBBlockersLib), read by
      both goal-inbox.js's attention-inbox slice below and the /blockers page.
      Loaded BEFORE goal-inbox.js, which calls it while rendering. -->
+<!-- task 1004820: the city ask card, drawn for every inbox action -->
+<script src="/builders/hall-ask.js" defer></script>
 <script src="/builders/blockers-lib.js" defer></script>
 <!-- ADR 0112 (task 1954): the home-page goal-membership inbox — invitations to
      accept + join-requests to approve. Self-registers; hidden when empty. -->

package/modules/hall-ui/records/hall-ask-card.md

@@ -0,0 +1,24 @@
+# The city's ask card, in the hall — every action a builder answers in place (task 1004820)
+
+Criterion `wa6-hall-reads-like-home` of goal 1000095 (owner ask 2026-10-23): the buttons a builder acts
+on inside the hall use the card template the city view built (`street-deck.css` `.ask`, task 1004702).
+
+- **One builder, one sheet.** `hall-ask.js` (`window.OTBAsk.html`) draws the card — where it sits, the
+  question, what was said, each answer as a one-tap option chip with its result line — and
+  `hall-ask.css` styles it under its own `hask` prefix on the hall's tokens, light and twice dark. The
+  city keeps `.ask`; a page loading both never cascades one into the other.
+- **No second action path.** An option carries the same `data-*` the page's delegated handler already
+  read, so switching a row to the card changes how it looks, never what it calls. An option with `href`
+  is a link, for an act that needs its own panel (resolving a blocker opens its overlay).
+- **Where it is applied.** Home's attention inbox (join invitations, join requests, the nod, "was this
+  removed?", untouched work, and the blocker slice — which had NO action and now has "Resolve it"),
+  the Blockers page's closed reviewer rows (Resolve it / Link a task; the open forms are unchanged), and
+  the Board Room's vote (Vote yes / Object as the card's chips).
+- **Rows keep `.ginbox__item`**, because `OTBAttention.refresh()` counts them; `hall-ask.css` makes the
+  card's own box win over the ledger row.
+- **Wording follows the Home rebuild:** a question, not a label ("Join X?", "Let Ana join?"), and every
+  answer says what it does.
+
+Left for the page tasks of the same criterion (1004821–1004829), which each re-lay a whole page: the
+task page's lifecycle buttons, Goals' row claim/release, collab, gate, profile connections and the
+government forms.

package/modules/lifecycle/artist-review-goal.js

@@ -0,0 +1,103 @@
+// modules/lifecycle/artist-review-goal.js — every version's ARTIST REVIEW goal
+// (task 1004819). The home for copywriting and Artist Studio work: page tweak
+// rounds, the engineer tasks a round leaves behind, and copy proposals.
+//
+// Owner direction (2026-10-23): all of that work sits under one ongoing goal, and
+// every version carries one, with two standing done-when criteria (below). Before
+// this the studio's create paths passed allowCatchAll, and a version with no
+// "<version> — general" goal (BONGOS-V2) filed every round with no goal at all.
+//
+// ensureMaintenanceGoal's shape (db-goals.js), with two differences worth knowing:
+//   - it is matched by TITLE, not a flag. Nothing decides anything from this goal
+//     being the artist one except where new studio work lands, so a column (and a
+//     migration) would buy nothing a hand-titled goal could abuse;
+//   - it is NOT exempt from the version-close count. Its criteria are meant to
+//     hold the version open until every page has been reviewed.
+// A 'planning' version takes one too, so createVersion can give a new version
+// its goal at birth; a closed version never does.
+//
+// Runs on the caller's client (deps.client) when given, so the goal, its criteria
+// and the task that needed them commit together or not at all; otherwise in a
+// transaction of its own.
+
+const api = require('../../src/module-api');
+
+const { pool, withTx } = api;
+
+const OPEN_TO_WORK = new Set(['planning', 'building']);
+
+const ARTIST_REVIEW_CRITERIA = Object.freeze([
+  Object.freeze({
+    criterion_id: 'artist-every-page-reviewed',
+    criterion_md: 'Every page in the page inventory has had at least one artist review (a shipped tweak round) of its words before the version closes.',
+  }),
+  Object.freeze({
+    criterion_id: 'artist-pages-simple-to-use',
+    criterion_md: 'Every page is simple for a user to read and use: plain words, one clear next step, nothing a newcomer has to decode. An artist signs this off per page.',
+  }),
+]);
+
+function artistReviewGoalTitle(versionId) {
+  return `${versionId} — artist review`;
+}
+
+async function ensureArtistReviewGoal(versionId, deps = {}) {
+  if (versionId == null) return null;
+  if (deps.client) return findOrCreate(versionId, deps.client);
+  // No caller transaction (a copy proposal): open one, or the advisory lock below
+  // would release the moment it is taken and two writers could both insert.
+  const txPool = (deps.pool && process.env.NODE_ENV === 'test') ? deps.pool : pool;
+  return withTx((client) => findOrCreate(versionId, client), { pool: txPool });
+}
+
+async function findOrCreate(versionId, exec) {
+  const title = artistReviewGoalTitle(versionId);
+
+  const { rows: vRows } = await exec.query(`SELECT status FROM versions WHERE id = $1`, [versionId]);
+  if (!vRows[0] || !OPEN_TO_WORK.has(vRows[0].status)) return null;
+
+  // No unique constraint on (version_id, title), so two studio writes in the same
+  // second would both insert without this. Released at COMMIT/ROLLBACK.
+  await exec.query(`SELECT pg_advisory_xact_lock(hashtext($1))`, [title]);
+
+  const { rows: found } = await exec.query(
+    `SELECT id, version_id, title FROM goals WHERE version_id = $1 AND title = $2 ORDER BY id LIMIT 1`,
+    [versionId, title]
+  );
+  if (found[0]) {
+    // A goal of this title made by hand (POST /goals) before the first studio
+    // write still gets the two standing criteria; ones already there are kept.
+    await seedCriteria(exec, versionId, found[0].id);
+    return found[0];
+  }
+
+  const { rows: made } = await exec.query(
+    `INSERT INTO goals (version_id, title, description, scope_modules, status, created_by, sort_order)
+     SELECT $1, $2, $3, $4, 'open', NULL, COALESCE(MAX(sort_order) + 1, 0)
+       FROM goals WHERE version_id = $1
+     RETURNING id, version_id, title`,
+    [versionId, title,
+      'The ongoing home for copywriting and Artist Studio work in this version: page tweak rounds, ' +
+      'the engineer tasks they leave, and copy proposals. Every version carries one (task 1004819).',
+      []]
+  );
+  const goal = made[0] || null;
+  if (!goal) return null;
+  await seedCriteria(exec, versionId, goal.id);
+  return goal;
+}
+
+async function seedCriteria(exec, versionId, goalId) {
+  let order = 0;
+  for (const c of ARTIST_REVIEW_CRITERIA) {
+    order += 1;
+    await exec.query(
+      `INSERT INTO done_when_criteria (version_id, criterion_id, criterion_md, goal_id, sort_order)
+       SELECT $1, $2, $3, $4, COALESCE(MAX(sort_order), 0) + $5 FROM done_when_criteria WHERE version_id = $1
+       ON CONFLICT (version_id, criterion_id) DO NOTHING`,
+      [versionId, c.criterion_id, c.criterion_md, goalId, order]
+    );
+  }
+}
+
+module.exports = { ensureArtistReviewGoal, artistReviewGoalTitle, ARTIST_REVIEW_CRITERIA };

package/modules/lifecycle/db-tasks.js

@@ -21,6 +21,7 @@ const { executorFor } = require('./db-executor.js');
 const { isReservedCriterionSlug } = require('./criterion-ref.js');
 const { resolveWriteDiscipline, WRITABLE_DISCIPLINES } = require('./task-classifier.js');
 const { gateOnSpecTask } = require('./goal-spec-task.js');
+const { ensureArtistReviewGoal } = require('./artist-review-goal.js');
 
 // -------------------------------------------------------------------------
 // Tasks
@@ -158,6 +159,11 @@ async function createTask({
   // copy-desk. Grep for it before adding a third: the answer is almost always
   // to name a goal instead.
   allowCatchAll = false,
+  // task 1004819: copywriting / Artist Studio work. With no goalId it lands in
+  // the version's artist review goal (created on first use, artist-review-goal.js)
+  // rather than the catch-all; the catch-all is only the fallback for a version
+  // that cannot take one.
+  artistReview = false,
 }, deps = {}) { // deps.client enlists this INSERT in the CALLER'S transaction (task 1003065, BV1.R01); the argument for it is at modules/ideas/inbox.js routeIdeaToGoal
   // task 1677 / ADR 0096 (2026-06-28): creation is now the PRIMARY reward-assignment
   // point — every task is born with a fair drachmae value, not just at promote/claim.
@@ -196,8 +202,18 @@ async function createTask({
   // opt-ins can name a real goal. Owner's call, recorded here so the remaining
   // gap is not mistaken for an oversight.
   let goalIdEffective = goalId;
+  if (goalIdEffective == null && artistReview === true && !deps.client) {
+    // No caller transaction (a copy proposal): run the whole create in one, so the
+    // goal it may create and the task commit together or not at all.
+    return api.withTx((client) => createTask(arguments[0], { ...deps, client }),
+      { pool: (deps.pool && process.env.NODE_ENV === 'test') ? deps.pool : pool });
+  }
+  if (goalIdEffective == null && artistReview === true) {
+    const artistGoal = await ensureArtistReviewGoal(versionId, { client: deps.client });
+    if (artistGoal) goalIdEffective = artistGoal.id;
+  }
   if (goalIdEffective == null) {
-    if (allowCatchAll !== true) {
+    if (allowCatchAll !== true && artistReview !== true) {
       throw new Error(
         'createTask: goalId is required — every task belongs to a goal (BV1.R11). ' +
         'A vector that genuinely cannot name one yet must pass allowCatchAll: true, ' +

package/modules/lifecycle/db-versions.js

@@ -20,6 +20,7 @@ const { executorFor, inTransaction } = require('./db-executor.js');
 // (fitness pins the count). createVersion needs it to seed a version's FIRST goal
 // in the same transaction — task 1003701.
 const { createGoal } = require('./db-goals.js');
+const { ensureArtistReviewGoal } = require('./artist-review-goal.js');
 const { cancelPendingMembershipRequests } = require('./goal-close-sweep.js');
 
 async function listVersions() {
@@ -214,7 +215,10 @@ async function createVersion({ id, name, status = 'planning', scopeDocPath = nul
       if (cr[0]) createdCriteria.push(cr[0]);
       i++;
     }
-    return { version, goal: goal || null, criteria: createdCriteria };
+    // task 1004819: every version is born with its artist review goal and its two
+    // standing criteria, on the same client, so they roll back with the version.
+    const artistReviewGoal = await ensureArtistReviewGoal(id, { client });
+    return { version, goal: goal || null, criteria: createdCriteria, artistReviewGoal };
   }, { pool: txPool });
 }
 

package/modules/lifecycle/page-tweak-claim.js

@@ -156,9 +156,9 @@ async function webClaimPageTweak({ pageId, builderId, versionId = null, round },
     const n = Math.max(rounds.length, ...rounds.map((t) => roundNumber(t.source_ref))) + 1;
     const task = await createTask({
       versionId,
-      // ADR 0341 D2: a round lands in the building version's catch-all goal,
-      // exactly as a copy proposal does. Declared out loud (BV1.R11).
-      allowCatchAll: true,
+      // ADR 0341 D2, task 1004819: a round lands in the building version's
+      // artist review goal, exactly as a copy proposal does.
+      artistReview: true,
       title: round.title(n),
       description: round.description(n),
       status: 'ready',

package/modules/lifecycle/page-tweak-write.js

@@ -35,6 +35,7 @@
 'use strict';
 
 const api = require('../../src/module-api');
+const { ensureArtistReviewGoal } = require('./artist-review-goal.js');
 const db = require('./db');
 
 function activePoolFor(deps) {
@@ -139,14 +140,21 @@ async function submitPageTweak({ taskId, builderId, versionId = null, freeze },
       );
       for (const f of found) if (!existing.has(f.source_ref)) existing.set(f.source_ref, String(f.id));
     }
+    // The version's artist review goal, resolved ONCE for the whole loop (task
+    // 1004819) rather than once per line: the version cannot change mid-submit.
+    const roundVersionId = versionId != null ? versionId : held.task.version_id;
+    const artistGoal = wanted.some((u) => !existing.has(u.sourceRef))
+      ? await ensureArtistReviewGoal(roundVersionId, { client }) : null;
     const filed = [];
     for (const u of wanted) {
       if (existing.has(u.sourceRef)) { filed.push({ id: existing.get(u.sourceRef), key: u.key, created: false }); continue; }
       const t = await createTask({
-        versionId: versionId != null ? versionId : held.task.version_id,
-        // The building version's catch-all goal, declared out loud (BV1.R11),
-        // exactly as the round itself and a copy proposal land.
-        allowCatchAll: true,
+        versionId: roundVersionId,
+        // The version's artist review goal (task 1004819), exactly as the
+        // round itself and a copy proposal land; artistReview is the fallback
+        // should the goal above not resolve.
+        goalId: artistGoal ? artistGoal.id : null,
+        artistReview: true,
         title: u.title,
         description: u.description,
         // BACKLOG (ADR 0341 D4): an engineer's triage decides when, and the

package/package-lock.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.21.173",
+  "version": "1.21.175",
   "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/release-notes.json

@@ -10120,5 +10120,21 @@
       "id": "1003808",
       "text": "The module store now ranks the best-scoring modules first, uses price only to break ties (free first), and lets buyers hide anything above a price they set."
     }
+  ],
+  "1.21.174": [
+    {
+      "id": "1004791",
+      "text": "The city's cards no longer cut off their words. Long questions show in full, answer buttons say their whole answer, and a \"Full text\" link on each card opens everything the item says when there is more to read. Long goal names"
+    },
+    {
+      "id": "1004819",
+      "text": "Every version now has an artist review goal, holding all copywriting and Artist Studio work, with two standing checks: every page reviewed by an artist before the version closes, and every page simple to read and use. New page"
+    }
+  ],
+  "1.21.175": [
+    {
+      "id": "1004820",
+      "text": "Actions in the hall now use the city's ask card: a question, then one big button per answer that says what it does. Home's inbox (join invites and requests, the nod, removals, stale work) and the Blockers page and Board Room v"
+    }
   ]
 }

package/src/module-api.js

@@ -75,7 +75,7 @@ const { responsibilityFor, ROLE_RESPONSIBILITIES } = require('./role-responsibil
 // MAJOR (see allowBoxScope below): passes the request through untouched.
 function deprecatedNoopMiddleware(_req, _res, next) { next(); }
 
-const CORE_VERSION = '1.21.173'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.21.175'; // 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('discord'); log.info('mounted');