@bongos/core 1.21.108 → 1.21.110

package/.bongos-core.json

@@ -2,22 +2,22 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/gds/package-core.js",
-  "core_version": "1.21.108",
-  "core_contract": "1.21.108",
-  "source_commit": "4744afee212fe2a5c5bac890ee578bbaae7e3c5b",
+  "core_version": "1.21.110",
+  "core_contract": "1.21.110",
+  "source_commit": "4df16a5c3d2eee37ad6572bb1be32a106bea9fd1",
   "source_ref": "HEAD",
-  "built_at": "2026-10-09T16:36:02.296Z",
+  "built_at": "2026-10-09T17:01:53.765Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 592,
     "agent_docs_stubbed": 28,
-    "functional_verbatim": 3027,
+    "functional_verbatim": 3034,
     "rules": 3,
     "gate_literals": 3,
     "gate": "passed"
   },
-  "file_count": 3648,
-  "tree_sha256": "62c2521f5de99bd666286fed934e1c0e7269d9b22a9953b772b54b1fa3a738e2",
+  "file_count": 3655,
+  "tree_sha256": "b5f9cc8c8673642d60dd789484f12cf32347f5ebad634f319e34770a77620c72",
   "files": [
     {
       "path": ".claude/skills/ask-for-help/SKILL.md",
@@ -152,7 +152,7 @@
     {
       "path": ".claude/skills/goal-create/SKILL.md",
       "mode": "0000644",
-      "sha256": "5fb0e00c6d6d8f1d1cc87136dd0a42cfc9379ffc55b8b5bdbbd736541e50c299"
+      "sha256": "11f9e820c5e37818de14b701945f8606562572c4381ba90ded31364005edf0fb"
     },
     {
       "path": ".claude/skills/goal-review/SKILL.md",
@@ -207,7 +207,7 @@
     {
       "path": ".claude/skills/planning-session/SKILL.md",
       "mode": "0000644",
-      "sha256": "07383a95bda58d86c833d963c421a9792b5fb879fa7fc75aba7b18b318e59b6e"
+      "sha256": "0d3ad6a9a7f3d0ee4bbdc0a8539705bdc5e9ab492151f3886120c15e102758a6"
     },
     {
       "path": ".claude/skills/priority-session/SKILL.md",
@@ -2357,12 +2357,12 @@
     {
       "path": "docs/copy-inventory.md",
       "mode": "0000644",
-      "sha256": "1d38789dcf8ea53dfc188e72896d346c1324494ab9f80aec4fc2aeef3907766e"
+      "sha256": "1fb4f9a78241bcfc0aa998f66bb96cab2e63b1a07c82f3ed54307728b9a3a856"
     },
     {
       "path": "docs/copy-registry.json",
       "mode": "0000644",
-      "sha256": "a93000df72e34c83bdf0ab43817a754f7db6f4d7dcdd6f436422426a2cc57a1f"
+      "sha256": "70ca4ad9a7a6dfcbb294c3b4e0f4f80f779ac9b89b24e845c5c602e287df528c"
     },
     {
       "path": "docs/design/apex-pass-2-direction.md",
@@ -2872,7 +2872,7 @@
     {
       "path": "docs/module-api-changelog.md",
       "mode": "0000644",
-      "sha256": "591ca6f09789a658cec2c0442d76fff5b80c3af4c620fb6b197499aa276d21fc"
+      "sha256": "0ee9660d1757ad8c73fd71e37250ff001b89a9ca894f8e75a01fc97e5db4e759"
     },
     {
       "path": "docs/modules-contract.md",
@@ -2962,7 +2962,7 @@
     {
       "path": "docs/page-readings.json",
       "mode": "0000644",
-      "sha256": "b23bcbca4f4689129a3c589c867c7f9b6191ca855d9b458f34155f41531f47ce"
+      "sha256": "c052d5258d274ab5a04793af96b52751f3233a7772b8406a2242a365ecce88d6"
     },
     {
       "path": "docs/project-context.template.md",
@@ -4432,7 +4432,7 @@
     {
       "path": "modules/copy-desk/skills/tweak/SKILL.md",
       "mode": "0000644",
-      "sha256": "21c1447d883837a53caf2b0cd34fda2f557422b97d91464832fa279b95ca8ed1"
+      "sha256": "c5f631d9490fdb2a6bfa68a5e8a206aefd24c29a30d826c90344eb3f325a0e41"
     },
     {
       "path": "modules/copy-desk/tests/copy_docx.mjs",
@@ -5557,7 +5557,17 @@
     {
       "path": "modules/hall-ui/public/craft-mode.js",
       "mode": "0000644",
-      "sha256": "d0eb466ce91917bda3ecb01450feb53b2741bec8b581c8f8d5f4245006cf8bfc"
+      "sha256": "ad340a50871425731439ca84abbbe28de067216e1d2e267559800d331560a023"
+    },
+    {
+      "path": "modules/hall-ui/public/craft-review.css",
+      "mode": "0000644",
+      "sha256": "2b720902eb865e9abda43e26b523f81daf3413fb67d14c0fe1e74b48cbbbf1cf"
+    },
+    {
+      "path": "modules/hall-ui/public/craft-review.js",
+      "mode": "0000644",
+      "sha256": "2da84dcf050187748e80a49636b4048e982a43b3b913e64f23f8d00d9500f92f"
     },
     {
       "path": "modules/hall-ui/public/criterion-uat-ui.js",
@@ -6352,7 +6362,7 @@
     {
       "path": "modules/hall-ui/public/style.css",
       "mode": "0000644",
-      "sha256": "ed3c2a242d279e61bfe11e8c4218f72fac2227ced9b5bdc1211b21e2720d860f"
+      "sha256": "9626e3d3781ae6b0b137105d04628a599723b50808d35f9ceaf99221d40b113c"
     },
     {
       "path": "modules/hall-ui/public/task-detail.js",
@@ -6512,7 +6522,7 @@
     {
       "path": "modules/hall-ui/records/craft-mode.md",
       "mode": "0000644",
-      "sha256": "1d4499e1dde2b7f4d88e621a47b164e913ab99878ccf73c7d3398f231cf70bfe"
+      "sha256": "3978525f0e27fced2e88642d285f389427bee552aa61740ecdb57cc0fdafe9e4"
     },
     {
       "path": "modules/hall-ui/records/filter-views.md",
@@ -6829,6 +6839,11 @@
       "mode": "0000644",
       "sha256": "e7c7db8a6faf50f3b75cb26ac33caa8357c462293e64318212ace69b0b654507"
     },
+    {
+      "path": "modules/lifecycle/autopickup-files.js",
+      "mode": "0000644",
+      "sha256": "a6caa4c76fbaeab89e861019609274b79062dac6b7d07bb0d0202cddedc6147f"
+    },
     {
       "path": "modules/lifecycle/autopickup.js",
       "mode": "0000644",
@@ -6952,7 +6967,7 @@
     {
       "path": "modules/lifecycle/db-ship.js",
       "mode": "0000644",
-      "sha256": "982d65a4e74063ba914a48ace9e95b595bd1540ea27b1c9f47a842c82f8c9ff6"
+      "sha256": "831ad96d2bd970d3b0828ba39339c596149a90be4aeef25e261ac328f0c9043e"
     },
     {
       "path": "modules/lifecycle/db-tasks.js",
@@ -7417,7 +7432,7 @@
     {
       "path": "modules/lifecycle/routes/tasks.js",
       "mode": "0000644",
-      "sha256": "1ced2b26ca9ce2e755e7ac1ce5b56130785041119a9373e3290250ee4b790ff4"
+      "sha256": "d6ee7c06be0c26423a88dd7e3e92537563cd4a3a110728ea7607f594bd8fe151"
     },
     {
       "path": "modules/lifecycle/routes/version-route-authz.js",
@@ -10157,12 +10172,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "6a7262e960db14a5a541c0735d0f41d9e5f436e0c434a6fd77dd01e177e6fb34"
+      "sha256": "e8c0e689fad7269139da1c9dd103cd1ea1c3058cffeda9799a45dcb9a79851e0"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "bd46dfa1e2c3169155d64d561b31e229137c85c66271ac2f0003ab47bedffdcd"
+      "sha256": "e7ea8bae12ce6058d01dfc32e6c52c39b69d4ca1285f1246bec58b862fe190cf"
     },
     {
       "path": "public-docs/index.html",
@@ -10182,7 +10197,7 @@
     {
       "path": "release-notes.json",
       "mode": "0000644",
-      "sha256": "71cdec4456ab1476dce44818b9d3f10cee7c0a8a8d67ad70c1adfc96debcf47b"
+      "sha256": "c78d78610bcdd0f13313e14da1dfabfb88480bc4edd1c501b6dd4d778a79766b"
     },
     {
       "path": "scripts/cron/style_guide_audit.sh",
@@ -10537,7 +10552,7 @@
     {
       "path": "scripts/gds/copy-apply.js",
       "mode": "0000644",
-      "sha256": "8c51a0f5e714202a26bbb23b717e9ae77ecc3708376c9319b76bb3372f280800"
+      "sha256": "02486308ff472de00169f256816edd98e40441dcab41a72ab36409a2c9da0a6f"
     },
     {
       "path": "scripts/gds/copy-inventory.js",
@@ -11864,6 +11879,11 @@
       "mode": "0000644",
       "sha256": "cc6e8640376ada7ceff6a896007b3257c8ea4d943a900eae28433cd41bdd139d"
     },
+    {
+      "path": "scripts/gds/tweak-halt.js",
+      "mode": "0000644",
+      "sha256": "df5fa45f622f59d7867295afc559d334807a31e732b16a1c869305d3033fb9a3"
+    },
     {
       "path": "scripts/gds/tweak-renders.js",
       "mode": "0000644",
@@ -11982,7 +12002,7 @@
     {
       "path": "scripts/hall-preview/server.js",
       "mode": "0000644",
-      "sha256": "73f90a45a8dbe04de03bdca1ad42f61c6dfe6e75982c3df8f311c8f1fa9eeed9"
+      "sha256": "5819bcbb67a9940a05ee7d874d29edbad366d2bdae64f40d0f9d5a4ea8e32376"
     },
     {
       "path": "scripts/hall-preview/task-visual.js",
@@ -12457,7 +12477,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "f208a278069baf39f28e39ddc85193af5cda7d17095dfe1edbbb89f2e7568350"
+      "sha256": "a6eddfed735bb10f8d6e43b148e56c100c177cc6f98313f142401d7649ff18f1"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -13534,6 +13554,11 @@
       "mode": "0000644",
       "sha256": "4c7c2de3e7e6e5157b2e16da06ef8fbb4e7e1360d34c3649007fb3a9debe8e9c"
     },
+    {
+      "path": "tests/craft_review_mode.mjs",
+      "mode": "0000644",
+      "sha256": "0e350eb3d5c1edd79bb854880dc326cc4fb47e59512e5c6f16e99d9e3623db6c"
+    },
     {
       "path": "tests/credential_presence.mjs",
       "mode": "0000644",
@@ -14894,6 +14919,11 @@
       "mode": "0000644",
       "sha256": "14e0bfda70dac75dc29bd784739ade11fe2cb986214bd0a1e8859f398b52dafb"
     },
+    {
+      "path": "tests/hall_preview_encoded_paths.mjs",
+      "mode": "0000644",
+      "sha256": "a4a0141c0743557d2344e2fbb0c573112b6c787e41332fdd04d5b656b17b65ba"
+    },
     {
       "path": "tests/hall_preview_governor_fixtures.mjs",
       "mode": "0000644",
@@ -16162,7 +16192,7 @@
     {
       "path": "tests/page_notes_panel.mjs",
       "mode": "0000644",
-      "sha256": "0be8365d3f45e42d98817aef289ab95880fc4c0ff0101ed98ea4a0722d432426"
+      "sha256": "56b4940c7b0757a0f7601b511eabbc98b81b9691e1b78ab5319a2243b399f01d"
     },
     {
       "path": "tests/page_reader.mjs",
@@ -16252,7 +16282,7 @@
     {
       "path": "tests/planning_session_skill.mjs",
       "mode": "0000644",
-      "sha256": "05bf5a11c0cb334d5da1b14f976f9e8ed8459ffec609f418c752ad72d87b7345"
+      "sha256": "5fa3820f0c2a9b718cc3bcb513264ffae24c25c91d1a7039f0459a2befb36a6b"
     },
     {
       "path": "tests/platform_boot.mjs",
@@ -17952,13 +17982,18 @@
     {
       "path": "tests/tweak_batch_apply.mjs",
       "mode": "0000644",
-      "sha256": "c9ee80f594f60ca39c8c706490dc69c3acc8ad053e2084feb1d2415caf6b0288"
+      "sha256": "db9ec48449abd88929a0b40df203e63b510318b4db0ddfaab6fce547f23dce0a"
     },
     {
       "path": "tests/tweak_replaces_artist_review.mjs",
       "mode": "0000644",
       "sha256": "16e4eb3977e7b5c2f09a402740b92ebb376a91199766034fa93f6ef34833b8a2"
     },
+    {
+      "path": "tests/tweak_unattended.mjs",
+      "mode": "0000644",
+      "sha256": "004e0fff732ebda78788ad64405b8f69afb64bdd5fb03263510ceb50a1efe10b"
+    },
     {
       "path": "tests/ui_design_kit.mjs",
       "mode": "0000644",

package/.claude/skills/goal-create/SKILL.md

@@ -12,7 +12,7 @@ cost: >-
 
 You are running the **goal-create** session for this Cloud Bongos instance. You produce **one goal**: a title, a scope wall, two to four done-when criteria, and a handful of seed tasks — written to Bongos in one sitting.
 
-> **Not `/planning-session`, not `/goal-review`.** `/planning-session` scopes a whole **version**: 5–9 criteria, a written spec under `docs/specs/`, a multi-round interview, a seed script, a migration. This skill produces **one goal**, with no spec file and no migration. `/goal-review` is the other end of the same lifecycle — it *confirms* criteria and auto-achieves goals. If the ask is "plan the next version", stop and run `/planning-session` instead.
+> **Not `/planning-session`, not `/goal-review`.** `/planning-session` scopes a whole **version**: 5–9 criteria, a written spec under `docs/specs/`, a multi-round interview, a seed script, a migration. This skill produces **one goal**, with no spec file and no migration. `/goal-review` is the other end of the same lifecycle — it *confirms* criteria and auto-achieves goals. If the ask is "plan the next version", stop and run `/planning-session` instead. **The same goes for a big design-driven ask** — the owner has a design in mind (a mockup, a named model, "it should feel like…") and it would take several tasks to build: route it to `/planning-session` before seeding anything, because only there is each criterion tied to the agreed design and each task told where that design lives (task 1004675). Seeding it here skips that binding, and the grader — which never reads the design — cannot notice the build drifting from it.
 
 ## Rank gate — Metic+ only
 

package/.claude/skills/planning-session/SKILL.md

@@ -21,6 +21,7 @@ The first planning session — V3 Planning Session #1, 2026-05-12 — is the tem
 - The user says "let's plan V4" / "plan the next version" / "open a planning session"
 - A version is in `status='planning'` and needs scope locked before tasks start landing in earnest
 - A previous planning session left proposals in `idea_inbox` tagged `kind='criterion-proposal'` that need ratification
+- **A big design-driven ask arrives — route it here BEFORE any task is seeded (task 1004675).** The tell: the owner has a design in mind (a mockup, a named model, "it should feel like…"), and the work would span several tasks. Seeding those tasks straight from chat — through `/goal-create`, `/idea-triage`, or a hand-written `POST /tasks` — skips the spec, the interview and the design binding in Phase 2, which is exactly how a build drifts from what was agreed. A single small task, or a goal whose shape is already settled and has no design to honour, does not need this.
 
 **Run an ideation pass first when the goal's SHAPE is not yet settled.** An ideator session (the [Ideator pack](../../../docs/packs/ideator.md)) inverts this skill's stance: there the human drives and you are the sounding board, so *their* framing sets the shape of the thing. Carry its output into Phase 1 and the Phase-4 interview is left with genuine residue instead of first-pass design.
 
@@ -106,12 +107,14 @@ This is where the user sees what they're shaping. Do not skip this — criteria
 
    **A row WITHOUT the pointer still needs its description read**, because the marker only exists from the day someone used merge instead of abandon-with-a-note, and the older rows are exactly the ones a rescope trips over. **Verify against the repo, not the row.** On 2026-08-16 a session read task 1003005 ("Rename modules/governance to modules/government") as `abandoned` and nearly put a false edge in a new goal's graph — the folder move had shipped under task 1003004, and only the prose said so. One `git log` over the paths the task named would have settled it.
 
-### Phase 1.5 — Map the mechanism (process-change sessions only) — task 1003075
+### Phase 1.5 — Map the mechanism (the default whenever an existing surface is touched) — task 1003075, task 1004675
 
-**Run this phase when the session is scoped to CHANGING how an existing mechanism already works** (a redesign, not a net-new feature with nothing live to replace) — skip it for a purely additive version with no prior mechanism in play. The precedent is the 2026-08-17 idea-routing session (goal 1000071): a subagent mapping the LIVE capture→triage→promote path before Phase 2 turned "route bugs to goals" into the capture-time-promotion design and resolved the bugs-vs-features question almost by itself — the session's own retro named it the highest-value step, and until now the skill only implied it.
+**Run this phase by default for any session that touches an existing surface** — a process-change redesign, an extension of a live feature, a new view over data that already exists. **Skipping it is the thing that needs a stated reason**, written into the spec's *Design decisions*: e.g. "purely additive, no live mechanism in play — nothing to map". Do not skip it because the owner arrived with a confident model; that is exactly the case where an unmapped assumption costs the most.
+
+The precedent is the 2026-08-17 idea-routing session (goal 1000071): a subagent mapping the LIVE capture→triage→promote path before Phase 2 turned "route bugs to goals" into the capture-time-promotion design and resolved the bugs-vs-features question almost by itself — the session's own retro named it the highest-value step. The 2026-09-19 ideator-surface session (meta task 1004076) was the second in a row: mapping moved the scope three times before any criterion was written — the owner's "size = karma" was not computable (karma rows carry no task or idea key), task parentage could not carry a descent (one level, no cross-version parents), and nothing recorded that shipped work was later removed, so half the design rested on a signal that did not exist.
 
 1. **Input.** Name the subsystem(s) the version is about to change, and the entry points you already know (a route file, a CLI script, a module's public function) as the subagent's starting point.
-2. **Spawn a read-only mapping subagent** (`Explore` or `general-purpose`, `model: sonnet` or higher per CLAUDE.md's subagent routing — this feeds a design decision, not plumbing) with a prompt asking it to trace the CURRENT live mechanism end-to-end: every entry point that touches it, the chokepoint(s) data already funnels through, and any function/route already doing part of what the new design needs. Ask for file:line references, not prose summaries — the output has to be citable in the spec.
+2. **Spawn a read-only mapping subagent** (`Explore` or `general-purpose`, `model: sonnet` or higher per CLAUDE.md's subagent routing — this feeds a design decision, not plumbing) with a prompt asking it to trace the CURRENT live mechanism end-to-end: every entry point that touches it, the chokepoint(s) data already funnels through, and any function/route already doing part of what the new design needs. Ask for file:line references, not prose summaries — the output has to be citable in the spec. **Always ask it one standard question as well: "Name any quantity this design assumes is computable, and say whether it actually is — which table or column carries it, or why nothing does."**
 3. **Output artifact — the mechanics map.** A short structured list: each chokepoint or reusable function/route, with a file:line and a one-line note on what it does today. Write it down (in the session, or straight into the spec draft) rather than letting it live only in the subagent's reply.
 4. **Consumed by:**
    - **Phase 2 (Criteria)** — a chokepoint the map surfaces (e.g. "every capture vector funnels through one function") often turns directly into a criterion, or reframes one that would otherwise be written as a task list.
@@ -129,6 +132,12 @@ For each criterion:
 
 **Apply the strength test BEFORE presenting each criterion.** A criterion passes the test if it plausibly anchors at least 5 tasks across multiple `kind` values (feature + bug + infra, not just feature). If it only anchors 1-3 specific tasks, or reads like an implementation step, rewrite it as the resulting outcome before showing it to the user.
 
+**Tie every criterion to the agreed design — outcomes alone let the build stray (task 1004675).** An outcome criterion is right, but it is only half the job. The grader reads the **task description and the diff**; it does **not** read `docs/specs/`. So an outcome criterion ("a thought is never lost") plus a vague task ("make the outcome persist") can be satisfied by a build that has nothing to do with the design the owner agreed, and nothing in the pipeline objects — the spec is the design of record and sits outside the loop that judges the work. The owner caught this in the 2026-09-19 ideator-surface session: "none of this criteria is easily indicative of the initial scope and designs we made". Close the gap three ways:
+
+1. **Split the settled design into MODEL and RENDERING.** The model is the durable decision — the named states, the independent channels, the rules (that session: five named states, two independent channels, a decay rule). It goes **into the criterion text**. The rendering — how it is drawn (that session: as a sky) — stays in the spec, free to change. A criterion that names the rendering breaks the day the look changes; a criterion that omits the model lets any build pass.
+2. **Give each criterion a proof whose scenario a wrong build fails.** Not "the page renders" but "a thought that caused 180 credits of work, since removed, displays at full size and zero brightness". Write that scenario into the proof task's **description** (Phase 3), because the description is where the grader actually looks.
+3. **Cite the design of record in every task description, and make sure the builder can open it.** Name the spec path (`docs/specs/<ver>-<goal-slug>.md`, committed in Phase 5) and the section the task implements. A private artifact link or a chat-only mockup as the design of record is decorative — whoever builds cannot open it.
+
 **Number every criterion** (`1.`, `2.`, …) in the presented list. The user will reference them by number when editing — e.g. "edit 4, remove 5, add a new one about X" — so they don't have to copy/paste the criterion titles.
 
 **Ask "what's missing?" before finalizing.** Once you've presented N criteria and worked through edits, explicitly prompt: "What concerns you that I haven't named? What's missing from this list?"
@@ -179,9 +188,9 @@ Don't tunnel-vision. Surface lateral options the user might not have thought of:
 
 **Proactively propose needs-exploration tasks** for any criterion where the implementation path isn't clear. Don't wait for the user to flag it. Frame them as "this one needs an architecture pass first — output an ADR + sub-task list before we build."
 
-**For each criterion, explicitly propose a proof-task** — the single live demonstration that proves the criterion is met. Examples: "Builder #2 onboards in ≤2h" for an onboarding criterion, "five concurrent builders shipping without collisions on staging" for a parallel-safety criterion. Without a proof-task, the criterion may technically ship without anyone knowing it works.
+**For each criterion, explicitly propose a proof-task** — the single live demonstration that proves the criterion is met. Examples: "Builder #2 onboards in ≤2h" for an onboarding criterion, "five concurrent builders shipping without collisions on staging" for a parallel-safety criterion. Without a proof-task, the criterion may technically ship without anyone knowing it works. **The proof's scenario must be one a wrong build fails** — state the concrete input and the exact observable result the agreed MODEL implies (Phase 2), so a build that ignored the design cannot pass it.
 
-**Encourage short task descriptions with cross-refs.** Each task's description should focus on done-when + touches; reference dependency tasks by their R## rather than restating context. Long standalone descriptions are content-debt at scale (80 tasks × 200 words = a lot of redundant context).
+**Encourage short task descriptions with cross-refs.** Each task's description should focus on done-when + touches; reference dependency tasks by their R## rather than restating context. **But every description cites the design of record** — the spec path and section it implements (Phase 2, "Tie every criterion to the agreed design") — because the grader never opens the spec on its own. Long standalone descriptions are content-debt at scale (80 tasks × 200 words = a lot of redundant context).
 
 > **`touches[]` is an advisory hint, not a hard fence ([ADR 0049](../../../docs/adr/0049-split-parallel-safety-contract.md)).** Parallel-safety is a *split contract*: for interactive single-human work, `git merge` is the authoritative collision detector and the claim-time overlap signal only warns; only the autonomous `--parallel` runner treats footprint overlap as a hard gate. So don't agonize over predicting every file at planning time — a rough blast-radius is enough, and a wrong prediction is no longer ship-blocking. (Step 7/8 — task [#882](https://example.com/builders#/task/882) — makes the per-task `touches` field optional and backfills it from the committed diff at ship.)
 
@@ -213,6 +222,8 @@ Tell the user the spec is written and where, then move straight to the interview
 
 **Ask only NON-OBVIOUS questions.** A question earns its place only if: the spec is genuinely underspecified on it, reasonable experts would disagree, AND the answer changes what gets built or how it's broken into tasks. Span dimensions across rounds — **data model & schema, API surface & contracts, UI/UX & information architecture, permissions/authz, privacy/consent, performance & scale, failure modes & edge cases, migration/rollout/backfill, naming the user will see, and scope boundaries.**
 
+**Naming questions: show the options side by side before recommending one (task 1004675).** A name the user will see — a page, a state, a role, a button — is a question where a lone recommendation anchors the owner before they have compared anything. Lay out three or four candidates together, each with a one-line note on how it reads to the person who will see it, and only then say which you would pick and why. In an AskUserQuestion round, that means each candidate is its own option (use the option `preview` for the side-by-side), with your pick listed first and marked recommended — never one name with "or something else?".
+
 **Do NOT ask** anything answerable by reading the codebase, anything the criteria already settle, yes/no rubber-stamps, or "do you want it to be good?" filler. If you catch yourself about to ask something with one obvious answer, decide it yourself and record it in *Design decisions* instead.
 
 > **This interview is the deliberate exception to the usual "decide implementation details yourself" default (see Tone, and CLAUDE.md §2).** The owner has explicitly asked to be consulted in depth here — so probe technical implementation, UX, tradeoffs, and concerns, not just outcomes.

package/docs/copy-inventory.md

@@ -4,13 +4,13 @@
 > Deterministic on a given tree — regenerate and commit rather than hand-editing.
 > Machine-readable twin: [`copy-registry.json`](copy-registry.json).
 
-**2633 strings** across 3 surfaces · 2236 distinct pieces of copy · 2633 certain, 0 heuristic · 235 shown in more than one place.
+**2637 strings** across 3 surfaces · 2240 distinct pieces of copy · 2637 certain, 0 heuristic · 235 shown in more than one place.
 
 ## Surfaces
 
 | surface | lives at | files | strings |
 |---|---|---|---|
-| `builders-hall` | `modules/hall-ui/public` | 134 | 1860 |
+| `builders-hall` | `modules/hall-ui/public` | 135 | 1864 |
 | `status-dashboard` | `modules/status-ui/public` | 4 | 100 |
 | `landing` | `modules/public-landing/public` | 8 | 673 |
 
@@ -78,7 +78,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `f6f68ba322be` | 3x | Open {…} | modules/hall-ui/public/goal-inbox.js:53, modules/hall-ui/public/goal-inbox.js:69, modules/hall-ui/public/goals.js:67 |
 | … | | _195 more in the JSON registry_ | |
 
-## `builders-hall` — 1860 strings
+## `builders-hall` — 1864 strings
 
 | id | conf | text | where |
 |---|---|---|---|
@@ -187,12 +187,12 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `d9dd46194f35` | certain | Argue from mechanism, not vibes. Always name the assumption a number rests on. | `modules/hall-ui/public/settings.html:270` |
 | `4d3b1bcae3eb` | certain | Artist Review Status | `modules/hall-ui/public/studio.html:131` |
 | `c05047322118` | certain | Artist review | `modules/hall-ui/public/paintbrush.js:148` |
-| `4a9385eeab33` | certain | Artists start on Artistry. | `modules/hall-ui/public/craft-mode.js:487` |
+| `4a9385eeab33` | certain | Artists start on Artistry. | `modules/hall-ui/public/craft-mode.js:736` |
 | `5fcfdf8975cf` | certain | As configuration: | `modules/hall-ui/public/government.js:998` |
 | `01a0f29ff406` | certain | As first filed | `modules/hall-ui/public/idea.js:467` |
 | `1ac8ed08e067` | certain | Ask for this upgrade | `modules/hall-ui/public/settings-software-update.js:102` |
 | `1ac8ed08e067` | certain | Ask for this upgrade | `modules/hall-ui/public/settings-software-update.js:236` |
-| `fa86bf403775` | certain | Ask me again next session | `modules/hall-ui/public/craft-mode.js:486` |
+| `fa86bf403775` | certain | Ask me again next session | `modules/hall-ui/public/craft-mode.js:735` |
 | `9b02ba5e6d23` | certain | Asked of others | `modules/hall-ui/public/collab.html:131` |
 | `688cc6dd38ee` | certain | Asked of you | `modules/hall-ui/public/collab.html:48` |
 | `688cc6dd38ee` | certain | Asked of you | `modules/hall-ui/public/collab.html:122` |
@@ -231,7 +231,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `da910fee01d7` | certain | Browse goals → | `modules/hall-ui/public/goals.js:83` |
 | `25b4ab94728a` | certain | Build from your own machine — get a copy of the project, sign in once, then work in Claude. | `modules/hall-ui/public/hall-render.js:810` |
 | `08bfcc9198ba` | certain | Builder id | `modules/hall-ui/public/government.html:109` |
-| `fd1903586e8e` | certain | Builders hall · this session | `modules/hall-ui/public/craft-mode.js:481` |
+| `fd1903586e8e` | certain | Builders hall · this session | `modules/hall-ui/public/craft-mode.js:730` |
 | `306bc316184d` | certain | Builders you may know | `modules/hall-ui/public/profile-connections.js:158` |
 | `60b248791dc3` | certain | Builders — {{worldName}} | `modules/hall-ui/public/roster.html:6` |
 | `60b248791dc3` | certain | Builders — {{worldName}} | `modules/hall-ui/public/roster.html:6` |
@@ -277,7 +277,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `6bf684ebc1c3` | certain | Choose the skills for {…} | `modules/hall-ui/public/settings-specialities.js:180` |
 | `2b504633779f` | certain | Choose which goals your own runner works while you are away. You can only choose from the goals the project al | `modules/hall-ui/public/settings.html:182` |
 | `6c88cdf417f1` | certain | Choose which sound plays for each build event — one when Claude waits for your input, another when a task ship | `modules/hall-ui/public/settings.html:403` |
-| `f7d8c25af75d` | certain | Choose your mode | `modules/hall-ui/public/craft-mode.js:400` |
+| `f7d8c25af75d` | certain | Choose your mode | `modules/hall-ui/public/craft-mode.js:649` |
 | `b7d5648b66db` | certain | Claim it | `modules/hall-ui/public/task.js:714` |
 | `b7d5648b66db` | certain | Claim it | `modules/hall-ui/public/task.js:843` |
 | `c10806b50c61` | certain | Claim task {…} | `modules/hall-ui/public/builders.js:973` |
@@ -295,7 +295,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `d12a2fb5730d` | certain | Close this flag | `modules/hall-ui/public/copy-desk.html:116` |
 | `9b9c2abeae90` | certain | Close this stage | `modules/hall-ui/public/genesis-home.js:220` |
 | `e6f1842afde1` | certain | Close transcript | `modules/hall-ui/public/sessions.html:99` |
-| `29e8c5e30095` | certain | Close without choosing | `modules/hall-ui/public/craft-mode.js:479` |
+| `29e8c5e30095` | certain | Close without choosing | `modules/hall-ui/public/craft-mode.js:728` |
 | `3d148e19e03a` | certain | Closed. | `modules/hall-ui/public/copy-desk.js:510` |
 | `2654dc3ded21` | certain | Closing is a judgement too — say what you decided, so the next person does not re-litigate it. | `modules/hall-ui/public/copy-desk.js:495` |
 | `e5548861ad79` | certain | Coach the grader | `modules/hall-ui/public/studio.js:211` |
@@ -920,7 +920,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `3ce534eb733f` | certain | Pick Auto or Manual. | `modules/hall-ui/public/project-settings.js:198` |
 | `6bc03d97457b` | certain | Pick a priority first. | `modules/hall-ui/public/task.js:900` |
 | `fcf996d77050` | certain | Pick a rank to see what it may do. The four ladder ranks ship with the platform; a custom rank is yours to sha | `modules/hall-ui/public/government.html:76` |
-| `4f87d5b17b54` | certain | Pick one. You can switch any time from the top bar. | `modules/hall-ui/public/craft-mode.js:483` |
+| `4f87d5b17b54` | certain | Pick one. You can switch any time from the top bar. | `modules/hall-ui/public/craft-mode.js:732` |
 | `45378ea261f6` | certain | Pick one… | `modules/hall-ui/public/settings.html:257` |
 | `25815a34d246` | certain | Plain speech | `modules/hall-ui/public/sessions.html:78` |
 | `727ef20b71df` | certain | Plan with Claude | `modules/hall-ui/public/goals-render.js:965` |
@@ -1036,6 +1036,8 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `deafad2284f4` | certain | Retire this task… | `modules/hall-ui/public/task.js:799` |
 | `908a0d5d0685` | certain | Review how it judged this work, and teach it what good looked like. | `modules/hall-ui/public/studio.js:211` |
 | `f9e58942b121` | certain | Review how this page looks too? | `modules/hall-ui/public/paintbrush.js:180` |
+| `3a3cce04a536` | certain | Review mode | `modules/hall-ui/public/craft-mode.js:418` |
+| `82f542b587f2` | certain | Review mode: see the hall as another mode | `modules/hall-ui/public/craft-mode.js:511` |
 | `f1b2b158c3b1` | certain | Review quality | `modules/hall-ui/public/watch.html:85` |
 | `4e062d4f8d69` | certain | Reviews by builder | `modules/hall-ui/public/watch.html:78` |
 | `5c233879ce4e` | certain | Reviews from before Tweak Mode | `modules/hall-ui/public/studio.html:165` |
@@ -1165,6 +1167,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `92f87ca38190` | certain | Still splitting. It will appear in this page's docs when it is done. | `modules/hall-ui/public/page-notes.js:255` |
 | `490a3cbb8c1c` | certain | Still working on core {…}. It keeps going without this page — reload to see where it got to. | `modules/hall-ui/public/deploy.js:764` |
 | `752ee4373182` | certain | Stop editing | `modules/hall-ui/public/paintbrush.js:147` |
+| `c358b0abfdfb` | certain | Stop reviewing | `modules/hall-ui/public/craft-mode.js:428` |
 | `9a4ced1afb6c` | certain | Stop waiting on task {…} | `modules/hall-ui/public/task.js:250` |
 | `b0861c874a2f` | certain | Stopped using it. | `modules/hall-ui/public/settings-specialities.js:457` |
 | `6c6544f5c011` | certain | Stored encrypted; never shown again after saving — only its last four digits. | `modules/hall-ui/public/settings.html:207` |
@@ -1419,7 +1422,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `c1fcfa9b48c2` | certain | Watch — {{worldName}} | `modules/hall-ui/public/watch.html:6` |
 | `0658f0358ffd` | certain | Welcome back, {…}. | `modules/hall-ui/public/hall-render.js:600` |
 | `04ef2f1bc4c4` | certain | Welcome, {…}. Claim and release tasks and follow work in progress — no terminal needed. | `modules/hall-ui/public/work.js:941` |
-| `60853319f290` | certain | What are you doing now? | `modules/hall-ui/public/craft-mode.js:482` |
+| `60853319f290` | certain | What are you doing now? | `modules/hall-ui/public/craft-mode.js:731` |
 | `d6bae87dc29b` | certain | What builders will see | `modules/hall-ui/public/government.html:86` |
 | `c077272746d4` | certain | What could this page become? | `modules/hall-ui/public/lightbulb.js:74` |
 | `f3d93a872a38` | certain | What counts | `modules/hall-ui/public/drachmae.js:107` |
@@ -1484,6 +1487,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `52e5fe3c6075` | certain | Write a Full Idea | `modules/hall-ui/public/ideas.html:145` |
 | `ad1e1ab1ecac` | certain | Write a reply… | `modules/hall-ui/public/collab.js:175` |
 | `a79ca1e2463b` | certain | Write another | `modules/hall-ui/public/page-notes.js:166` |
+| `22893267beca` | certain | Write feedback on this page | `modules/hall-ui/public/craft-mode.js:417` |
 | `08f28ecf123f` | certain | Write more | `modules/hall-ui/public/lightbulb.js:108` |
 | `47f61e576454` | certain | Write the wording you want instead. | `modules/hall-ui/public/copy-desk.js:436` |
 | `da6de9f79bcc` | certain | Write what this page made you think of. Each separate point becomes its own idea. | `modules/hall-ui/public/lightbulb.js:83` |

package/docs/copy-registry.json

@@ -3,17 +3,17 @@
   "generator": "scripts/gds/copy-inventory.js",
   "goal": 1000074,
   "counts": {
-    "strings": 2633,
-    "unique_ids": 2236,
-    "certain": 2633,
+    "strings": 2637,
+    "unique_ids": 2240,
+    "certain": 2637,
     "heuristic": 0,
     "duplicate_ids": 235
   },
   "surfaces": {
     "builders-hall": {
       "at": "modules/hall-ui/public",
-      "files": 134,
-      "strings": 1860
+      "files": 135,
+      "strings": 1864
     },
     "status-dashboard": {
       "at": "modules/status-ui/public",
@@ -3502,7 +3502,7 @@
       "surface": "builders-hall",
       "text": "Artists start on Artistry.",
       "file": "modules/hall-ui/public/craft-mode.js",
-      "line": 487,
+      "line": 736,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -3547,7 +3547,7 @@
       "surface": "builders-hall",
       "text": "Ask me again next session",
       "file": "modules/hall-ui/public/craft-mode.js",
-      "line": 486,
+      "line": 735,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -3898,7 +3898,7 @@
       "surface": "builders-hall",
       "text": "Builders hall · this session",
       "file": "modules/hall-ui/public/craft-mode.js",
-      "line": 481,
+      "line": 730,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -4312,7 +4312,7 @@
       "surface": "builders-hall",
       "text": "Choose your mode",
       "file": "modules/hall-ui/public/craft-mode.js",
-      "line": 400,
+      "line": 649,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -4474,7 +4474,7 @@
       "surface": "builders-hall",
       "text": "Close without choosing",
       "file": "modules/hall-ui/public/craft-mode.js",
-      "line": 479,
+      "line": 728,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -10099,7 +10099,7 @@
       "surface": "builders-hall",
       "text": "Pick one. You can switch any time from the top bar.",
       "file": "modules/hall-ui/public/craft-mode.js",
-      "line": 483,
+      "line": 732,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -11138,6 +11138,24 @@
       "origin": "js-markup",
       "confidence": "certain"
     },
+    {
+      "id": "3a3cce04a536",
+      "surface": "builders-hall",
+      "text": "Review mode",
+      "file": "modules/hall-ui/public/craft-mode.js",
+      "line": 418,
+      "origin": "js-markup",
+      "confidence": "certain"
+    },
+    {
+      "id": "82f542b587f2",
+      "surface": "builders-hall",
+      "text": "Review mode: see the hall as another mode",
+      "file": "modules/hall-ui/public/craft-mode.js",
+      "line": 511,
+      "origin": "js-text-assign",
+      "confidence": "certain"
+    },
     {
       "id": "f1b2b158c3b1",
       "surface": "builders-hall",
@@ -12299,6 +12317,15 @@
       "origin": "js-markup",
       "confidence": "certain"
     },
+    {
+      "id": "c358b0abfdfb",
+      "surface": "builders-hall",
+      "text": "Stop reviewing",
+      "file": "modules/hall-ui/public/craft-mode.js",
+      "line": 428,
+      "origin": "js-markup",
+      "confidence": "certain"
+    },
     {
       "id": "9a4ced1afb6c",
       "surface": "builders-hall",
@@ -14590,7 +14617,7 @@
       "surface": "builders-hall",
       "text": "What are you doing now?",
       "file": "modules/hall-ui/public/craft-mode.js",
-      "line": 482,
+      "line": 731,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -15170,6 +15197,15 @@
       "origin": "js-markup",
       "confidence": "certain"
     },
+    {
+      "id": "22893267beca",
+      "surface": "builders-hall",
+      "text": "Write feedback on this page",
+      "file": "modules/hall-ui/public/craft-mode.js",
+      "line": 417,
+      "origin": "js-markup",
+      "confidence": "certain"
+    },
     {
       "id": "08f28ecf123f",
       "surface": "builders-hall",

package/docs/module-api-changelog.md

@@ -3053,5 +3053,9 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.21.106 with no explicit bump. run 37957919746. (task 1002620)
   1.21.108 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         landed since 1.21.107 with no explicit bump. run 37960186494. (task 1002620)
+  1.21.109 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.21.108 with no explicit bump. run 37962089344. (task 1002620)
+  1.21.110 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.21.109 with no explicit bump. run 37963285770. (task 1002620)
 ---------------------------------------------------------------------------
 ```