@bongos/core 1.21.173 → 1.21.174

package/.bongos-core.json

@@ -2,22 +2,22 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/gds/package-core.js",
-  "core_version": "1.21.173",
-  "core_contract": "1.21.173",
-  "source_commit": "fb74ac25169bf2ab96aceb07fb4efd45efd018b2",
+  "core_version": "1.21.174",
+  "core_contract": "1.21.174",
+  "source_commit": "adfd7c27e23529361ee07bbaf4ef49077e4111d1",
   "source_ref": "HEAD",
-  "built_at": "2026-10-23T18:08:17.265Z",
+  "built_at": "2026-10-23T18:25:19.708Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 609,
     "agent_docs_stubbed": 28,
-    "functional_verbatim": 3194,
+    "functional_verbatim": 3196,
     "rules": 3,
     "gate_literals": 3,
     "gate": "passed"
   },
-  "file_count": 3832,
-  "tree_sha256": "526ec0b1a53ccf907d7663008fe0804338804fe2018aaca9eb10ef323fd162bb",
+  "file_count": 3834,
+  "tree_sha256": "caab1743d1725c9a840734341335be349f8c978cf2d458f1ba9effab69975d30",
   "files": [
     {
       "path": ".claude/skills/ask-for-help/SKILL.md",
@@ -2412,12 +2412,12 @@
     {
       "path": "docs/copy-inventory.md",
       "mode": "0000644",
-      "sha256": "5aac1b2e54ab9a0ef16cdb1eeef22c797933d08ee7bd40f21c052bc415b21f73"
+      "sha256": "c75fc40e23e292f5f95de5218c44861ea53ef2d0f691afbd2e8c3383c3e77a09"
     },
     {
       "path": "docs/copy-registry.json",
       "mode": "0000644",
-      "sha256": "5171c48da53a029424585fa32da23cb78fc8ee99f17a07df8bf9186c73fd6b4f"
+      "sha256": "16a9f7fc63e3c592067ed0286a9faaf52fc7c07fb88d0eb5b1e9ef966822b741"
     },
     {
       "path": "docs/design/apex-pass-2-direction.md",
@@ -2927,7 +2927,7 @@
     {
       "path": "docs/module-api-changelog.md",
       "mode": "0000644",
-      "sha256": "5373b4a333b0d0ce97ed2516d107a395b532cedce60e682e0d9be33bb3eb9e3b"
+      "sha256": "8c7c1a18bb466eb86c4e6ed6d7b39b433111e99562b04f444671c455bb96ac62"
     },
     {
       "path": "docs/modules-contract.md",
@@ -3017,7 +3017,7 @@
     {
       "path": "docs/page-readings.json",
       "mode": "0000644",
-      "sha256": "a2c08ff05348d995ef43ec6a24da479c28f8a1e19343e4d1176e1f7f6cca3929"
+      "sha256": "9ec7d9542b85b8232622692dbe007a852da947e6f674172b2aa51f961acf17d4"
     },
     {
       "path": "docs/project-context.template.md",
@@ -4517,7 +4517,7 @@
     {
       "path": "modules/copy-desk/routes/copy-desk.js",
       "mode": "0000644",
-      "sha256": "2f88b12c9a8ce90f5a153407cb492d342dd6beda43f8b21c1e15a42cb2d8f899"
+      "sha256": "c29c4015b09ed16ee0ae929077f077f4490b33e5a8055f9d331e097e1aabcf97"
     },
     {
       "path": "modules/copy-desk/skills/tweak/SKILL.md",
@@ -5652,7 +5652,7 @@
     {
       "path": "modules/hall-ui/public/city-decide.js",
       "mode": "0000644",
-      "sha256": "dff181b388393cca09bc7ab4c2949eff95c63c2795fef0f8348f49a3387b5c59"
+      "sha256": "de7085b284e4ed6bfd8f5703074934ec962697ae1d9cf9d7e31ec3ef81952fc2"
     },
     {
       "path": "modules/hall-ui/public/city-draw.js",
@@ -5667,12 +5667,12 @@
     {
       "path": "modules/hall-ui/public/city.css",
       "mode": "0000644",
-      "sha256": "2b4bebeeef82ec53a21bbcff83925b185e81afa356a889722ab8bd2206c162bd"
+      "sha256": "bc2b6e59862ada8af0fc125ccff4f875c916ca5624d46d5fb1493a8d849c154c"
     },
     {
       "path": "modules/hall-ui/public/city.html",
       "mode": "0000644",
-      "sha256": "e20c167df8c5f7e630ba394ab5e39f274de8aed428f3d41f2fa987c8b340047d"
+      "sha256": "48d8eacaf02f0f6da2959da09f0d56245d718ef26efe2753012dee0f0b64bcb1"
     },
     {
       "path": "modules/hall-ui/public/city.js",
@@ -7154,6 +7154,11 @@
       "mode": "0000644",
       "sha256": "e7c7db8a6faf50f3b75cb26ac33caa8357c462293e64318212ace69b0b654507"
     },
+    {
+      "path": "modules/lifecycle/artist-review-goal.js",
+      "mode": "0000644",
+      "sha256": "e2e9318d76c337b6339b0685cd2585c9d58cd13c7ec1bac8186e614a60a31be4"
+    },
     {
       "path": "modules/lifecycle/autopickup-files.js",
       "mode": "0000644",
@@ -7292,12 +7297,12 @@
     {
       "path": "modules/lifecycle/db-tasks.js",
       "mode": "0000644",
-      "sha256": "710e4c97c0b67a3b258d6997ba9dc485e42bb57a0988777c0f10f8391b952489"
+      "sha256": "29b4ced94d5fc5721dcb0c17ecbd9171b90e5444f0c526f3d8498f6e99b22fc6"
     },
     {
       "path": "modules/lifecycle/db-versions.js",
       "mode": "0000644",
-      "sha256": "7ffd6ca56a6621a933abf854589e836260cd749c6148a278542ab4ae89eb6b13"
+      "sha256": "e5328df4be96e980fd22486f441f0f2a47abe5572a1b34ae47caed96965dadc9"
     },
     {
       "path": "modules/lifecycle/db.js",
@@ -7577,7 +7582,7 @@
     {
       "path": "modules/lifecycle/page-tweak-claim.js",
       "mode": "0000644",
-      "sha256": "4b9f9c23161a961381857a01fc810ce1b960e2b89b74e616fa17d34a403b592f"
+      "sha256": "a3adde93c8bdef716f68aa09b9c60a81fa55f65520980d9686447db95647eecf"
     },
     {
       "path": "modules/lifecycle/page-tweak-hold.js",
@@ -7592,7 +7597,7 @@
     {
       "path": "modules/lifecycle/page-tweak-write.js",
       "mode": "0000644",
-      "sha256": "a13c01e96c5ae09837b9d599a66b23765f6d6f16cc29b11ca897da4f497d0b0d"
+      "sha256": "c6b5cb868778d1043029c0f17640caa4e2124e13783319e33cbdb56b0135ba45"
     },
     {
       "path": "modules/lifecycle/pg-constraint-fail.js",
@@ -10602,12 +10607,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "b223fa39f36866db4ba70f3dc764eb58397f8fbbe42172c15836fb775bf8ff9e"
+      "sha256": "0a8aad85242c815483d937f464a70e0635a6f82f3e0bbd1d53e0007c4f391599"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "6bcb32ea5c5595fdedd41e360311f9afebbbc1b15977e2336d28e56ff07d3e10"
+      "sha256": "d875bd84ad023a7e1a802a045ae88ef86e835f82ebd527db9ec1a82134a72e31"
     },
     {
       "path": "public-docs/index.html",
@@ -10627,7 +10632,7 @@
     {
       "path": "release-notes.json",
       "mode": "0000644",
-      "sha256": "3a1a03d0f50336977ef9dadfcf5a39dd261a2c526b85bc08da58e61b0ad54c84"
+      "sha256": "23c78c80dcbd8687b941dbae2186b92c9c5842b381dbbc7549cb8780b6cf7cb5"
     },
     {
       "path": "scripts/cron/style_guide_audit.sh",
@@ -13017,7 +13022,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "8be0183cf9a8f427c7df2f5cd015730dab2ae93dd5fe1965e71101902a27e0fb"
+      "sha256": "2a6426c050bd5aad4518c4ff3613fac041993deb0eebc6cab7d79184ff120159"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -13374,6 +13379,11 @@
       "mode": "0000644",
       "sha256": "370b54d4e323e144ef8f9e382b873a4ff803b7727844c0e2d6b9e5cdfeb0fbe1"
     },
+    {
+      "path": "tests/artist_review_goal.mjs",
+      "mode": "0000644",
+      "sha256": "ce9b9035338a7dbfb316277ac3d881bb7c36560f93d50dd811ea8f6b2ce2ec7c"
+    },
     {
       "path": "tests/asset_version.mjs",
       "mode": "0000644",
@@ -13992,7 +14002,7 @@
     {
       "path": "tests/copy_desk_page_claim.mjs",
       "mode": "0000644",
-      "sha256": "af6cdac77b90aad511a603757e12d7ffefaa7fdbcdd14aefd70ad49e3cd193fe"
+      "sha256": "f8bf52b346895367eca47dc9b0d169439296f908c504e7b3dc25f3e76a62250b"
     },
     {
       "path": "tests/copy_desk_page_docx.mjs",
@@ -14017,7 +14027,7 @@
     {
       "path": "tests/copy_desk_page_submit.mjs",
       "mode": "0000644",
-      "sha256": "7f5d5b9b9054b5dc6a066253af93864e8ea1332ca32b8300f217832d6640cabf"
+      "sha256": "c21159c1d8dcf82128120362232f72d4c7d6cf0bdc8726517788f5c6b18f5bd2"
     },
     {
       "path": "tests/copy_guarded_lines.mjs",
@@ -14642,7 +14652,7 @@
     {
       "path": "tests/fixtures/page-ledger.mjs",
       "mode": "0000644",
-      "sha256": "b36096f0bc3f6ea645c5e29e8bc1c769d7fb78fa8cd6d4985ab573a5a3cb0e88"
+      "sha256": "b0a8f277883957ea314ad6efd16fc3109729726a75c28136bdd890c1570749b1"
     },
     {
       "path": "tests/fixtures/session-506-digest.json",
@@ -15557,7 +15567,7 @@
     {
       "path": "tests/hall_city_rewording.mjs",
       "mode": "0000644",
-      "sha256": "93fc45e48ecf65029f5eaae33b001474bf4fc147becc3df3a7fcd56288d350d5"
+      "sha256": "88dc75df0b62ed23ff0aa110d6fa3eb362ccc05e8ac6c0999dcc064c315e6962"
     },
     {
       "path": "tests/hall_city_switchboard.mjs",
@@ -18717,7 +18727,7 @@
     {
       "path": "tests/task_goal_required.mjs",
       "mode": "0000644",
-      "sha256": "073581d9a97095666ee2ed7ebb0a6b280f3e8ea1efc53015c2f77eb5d08c4975"
+      "sha256": "5bf5deb47453f3ec5a64251137d731f5974c712ada11b9dbedde91469421c6a4"
     },
     {
       "path": "tests/task_lib.mjs",

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).
 
-**2796 strings** across 3 surfaces · 2377 distinct pieces of copy · 2796 certain, 0 heuristic · 243 shown in more than one place.
+**2797 strings** across 3 surfaces · 2378 distinct pieces of copy · 2797 certain, 0 heuristic · 243 shown in more than one place.
 
 ## Surfaces
 
 | surface | lives at | files | strings |
 |---|---|---|---|
-| `builders-hall` | `modules/hall-ui/public` | 150 | 1974 |
+| `builders-hall` | `modules/hall-ui/public` | 150 | 1975 |
 | `status-dashboard` | `modules/status-ui/public` | 4 | 100 |
 | `landing` | `modules/public-landing/public` | 9 | 722 |
 
@@ -78,7 +78,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `8226c979db4d` | 3x | Not now | modules/hall-ui/public/collab.js:668, modules/hall-ui/public/paintbrush.js:119, modules/hall-ui/public/paintbrush.js:183 |
 | … | | _203 more in the JSON registry_ | |
 
-## `builders-hall` — 1974 strings
+## `builders-hall` — 1975 strings
 
 | id | conf | text | where |
 |---|---|---|---|
@@ -563,6 +563,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `d32c5e058d1e` | certain | From the system | `modules/hall-ui/public/hall-render.js:721` |
 | `fc6fdfa639a7` | certain | Full log | `modules/hall-ui/public/task.js:464` |
 | `11fc0483c149` | certain | Full screen — press Esc, or the button in the corner, to bring the bars back. | `modules/hall-ui/public/shell.js:595` |
+| `7537fe1f3792` | certain | Full text | `modules/hall-ui/public/city-decide.js:253` |
 | `ecae3916bf01` | certain | Gate — {{worldName}} | `modules/hall-ui/public/gate.html:6` |
 | `ecae3916bf01` | certain | Gate — {{worldName}} | `modules/hall-ui/public/gate.html:6` |
 | `c93912bbad76` | certain | Gathering the reviews and flagged copy that wait for you. | `modules/hall-ui/public/studio.html:172` |

package/docs/copy-registry.json

@@ -3,9 +3,9 @@
   "generator": "scripts/gds/copy-inventory.js",
   "goal": 1000074,
   "counts": {
-    "strings": 2796,
-    "unique_ids": 2377,
-    "certain": 2796,
+    "strings": 2797,
+    "unique_ids": 2378,
+    "certain": 2797,
     "heuristic": 0,
     "duplicate_ids": 243
   },
@@ -13,7 +13,7 @@
     "builders-hall": {
       "at": "modules/hall-ui/public",
       "files": 150,
-      "strings": 1974
+      "strings": 1975
     },
     "status-dashboard": {
       "at": "modules/status-ui/public",
@@ -6975,6 +6975,15 @@
       "origin": "js-text-assign",
       "confidence": "certain"
     },
+    {
+      "id": "7537fe1f3792",
+      "surface": "builders-hall",
+      "text": "Full text",
+      "file": "modules/hall-ui/public/city-decide.js",
+      "line": 253,
+      "origin": "js-text-assign",
+      "confidence": "certain"
+    },
     {
       "id": "ecae3916bf01",
       "surface": "builders-hall",

package/docs/module-api-changelog.md

@@ -3183,5 +3183,7 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.21.171 with no explicit bump. run 39998870016. (task 1002620)
   1.21.173 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         landed since 1.21.172 with no explicit bump. run 40000711332. (task 1002620)
+  1.21.174 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.21.173 with no explicit bump. run 40003024305. (task 1002620)
 ---------------------------------------------------------------------------
 ```

package/docs/page-readings.json

@@ -5,7 +5,7 @@
   "adr": "0341 D3",
   "reread": "node scripts/gds/page-reader.js --stale",
   "how": ["Generated by node scripts/gds/page-reader.js (BV2.TW04, task 1004315; ADR 0341 D3). Do not hand-edit.","One entry per page of docs/page-inventory.json. Lines are in page order: the default state first, and a line only another state shows sits where that state shows it, with its state named.","placement: placed = a docs/copy-registry.json row renders it (file, line, string_id); shared = shell text every page of the surface carries (top bar, sidebar, footer), with its row when the registry has one; unplaced = the registry cannot place it (a rewrite becomes an engineer task, ADR 0341 D4).","text keeps {…} where the registry row has a hole, or where a value the page read from its API was filled in. A string that is only such data is counted under omitted.data, not listed.","reading_hash is over the non-shared lines (key, section, text); files_hash is over the page's inventory files. --check compares files_hash only and never opens a browser.","Rendered through the hall-preview harness (hall) and the ui-design kit stub (landing, status), at 1280x800 with the clock pinned, so a reading is the fixture world's page."],
-  "counts": {"pages":46,"lines":5404,"placed":1697,"shared":399,"unplaced":3308,"omitted_data":3498},
+  "counts": {"pages":46,"lines":5405,"placed":1698,"shared":399,"unplaced":3308,"omitted_data":3498},
   "pages": [
     {
       "id": "builders:atlas",
@@ -145,9 +145,9 @@
       "surface": "builders",
       "title": "The city",
       "states_read": ["city","fullscreen","switched","focus","pins","living","card","card-ask"],
-      "files_hash": "793c4d2505f9afb1",
-      "reading_hash": "e6b8d4e8f991a975",
-      "counts": {"lines":42,"placed":9,"shared":8,"unplaced":25},
+      "files_hash": "1e4c5ab40907996d",
+      "reading_hash": "77470c4eb3047a05",
+      "counts": {"lines":43,"placed":10,"shared":8,"unplaced":25},
       "omitted": {"data":17},
       "notes": [],
       "lines": [
@@ -165,34 +165,35 @@
         {"key":"L0012","section":"page top","text":"Total 1 /","placement":"unplaced","state":"card-ask"},
         {"key":"L0013","section":"page top","text":"Yours 1","placement":"unplaced","state":"card-ask"},
         {"key":"L0014","section":"{…}","text":"I'd {…} $2350. {…}","placement":"unplaced","state":"card-ask"},
-        {"key":"L0015","section":"{…}","text":"{…} and {…}","placement":"placed","file":"modules/hall-ui/public/profile-nudge.js","line":77,"string_id":"56b00d773232","state":"card-ask","occurrences":2},
-        {"key":"L0016","section":"{…}","text":"my pick","placement":"unplaced","state":"card-ask"},
-        {"key":"L0017","section":"{…}","text":"{…} it","placement":"unplaced","state":"card-ask"},
-        {"key":"L0018","section":"{…}","text":"esc","placement":"unplaced","state":"card-ask"},
-        {"key":"L0019","section":"{…}","text":"{…} 37 in a row","placement":"unplaced","state":"card-ask"},
-        {"key":"L0020","section":"page top","text":"Total 9 /","placement":"unplaced","state":"card"},
-        {"key":"L0021","section":"page top","text":"Yours 9","placement":"unplaced","state":"card"},
-        {"key":"L0022","section":"{…} of {…}","text":"{…} of {…}","placement":"placed","file":"modules/hall-ui/public/sky-panel.js","line":120,"string_id":"73c1372b43d9","state":"card","occurrences":2},
-        {"key":"L0023","section":"{…} of {…}","text":"It holds work until someone decides.","placement":"unplaced","state":"card"},
-        {"key":"L0024","section":"{…} of {…}","text":"Tab · ↑ ↓ items ← → buildings","placement":"unplaced","state":"card"},
-        {"key":"L0025","section":"{…} of {…}","text":"{…} all 9","placement":"unplaced","state":"card"},
-        {"key":"L0026","section":"{…} of {…}","text":"esc","placement":"unplaced","state":"card"},
-        {"key":"L0027","section":"{…} of {…}","text":"{…} 37 in a row","placement":"unplaced","state":"card"},
-        {"key":"L0028","section":"page top","text":"{…} 37 in a row","placement":"unplaced"},
-        {"key":"L0029","section":"37 to do · 5 {…} working","text":"37 to do · 5 {…} working","placement":"unplaced"},
-        {"key":"L0030","section":"37 to do · 5 {…} working","text":"Archived:","placement":"unplaced"},
-        {"key":"L0031","section":"37 to do · 5 {…} working","text":"Street","placement":"unplaced"},
-        {"key":"L0032","section":"37 to do · 5 {…} working","text":"Building Level Views","placement":"placed","file":"modules/hall-ui/public/city.html","line":103,"string_id":"5df2aa54f042"},
-        {"key":"L0033","section":"37 to do · 5 {…} working","text":"Full screen — press Esc, or the button in the corner, to bring the bars back.","placement":"placed","file":"modules/hall-ui/public/shell.js","line":595,"string_id":"11fc0483c149","state":"fullscreen"},
-        {"key":"L0034","section":"37 to do · 5 {…} working","text":"Builders hall · this session","placement":"placed","file":"modules/hall-ui/public/craft-mode.js","line":674,"string_id":"fd1903586e8e"},
-        {"key":"L0035","section":"What are you doing now?","text":"What are you doing now?","placement":"placed","file":"modules/hall-ui/public/craft-mode.js","line":675,"string_id":"60853319f290"},
-        {"key":"L0036","section":"What are you doing now?","text":"Pick one. You can switch any time from the top bar.","placement":"placed","file":"modules/hall-ui/public/craft-mode.js","line":676,"string_id":"4f87d5b17b54"},
-        {"key":"L0037","section":"What are you doing now?","text":"Building","placement":"unplaced"},
-        {"key":"L0038","section":"What are you doing now?","text":"{…} and {…}","placement":"placed","file":"modules/hall-ui/public/profile-nudge.js","line":77,"string_id":"56b00d773232","occurrences":3},
-        {"key":"L0039","section":"What are you doing now?","text":"Artistry","placement":"unplaced"},
-        {"key":"L0040","section":"What are you doing now?","text":"Ideating","placement":"unplaced"},
-        {"key":"L0041","section":"What are you doing now?","text":"Turn thoughts into {…}.","placement":"unplaced"},
-        {"key":"L0042","section":"What are you doing now?","text":"Ask me again next session","placement":"placed","file":"modules/hall-ui/public/craft-mode.js","line":679,"string_id":"fa86bf403775"}
+        {"key":"L0015","section":"{…}","text":"Full text","placement":"placed","file":"modules/hall-ui/public/city-decide.js","line":253,"string_id":"7537fe1f3792","state":"card-ask"},
+        {"key":"L0016","section":"{…}","text":"{…} and {…}","placement":"placed","file":"modules/hall-ui/public/profile-nudge.js","line":77,"string_id":"56b00d773232","state":"card-ask","occurrences":2},
+        {"key":"L0017","section":"{…}","text":"my pick","placement":"unplaced","state":"card-ask"},
+        {"key":"L0018","section":"{…}","text":"{…} it","placement":"unplaced","state":"card-ask"},
+        {"key":"L0019","section":"{…}","text":"esc","placement":"unplaced","state":"card-ask"},
+        {"key":"L0020","section":"{…}","text":"{…} 37 in a row","placement":"unplaced","state":"card-ask"},
+        {"key":"L0021","section":"page top","text":"Total 9 /","placement":"unplaced","state":"card"},
+        {"key":"L0022","section":"page top","text":"Yours 9","placement":"unplaced","state":"card"},
+        {"key":"L0023","section":"{…} of {…}","text":"{…} of {…}","placement":"placed","file":"modules/hall-ui/public/sky-panel.js","line":120,"string_id":"73c1372b43d9","state":"card","occurrences":2},
+        {"key":"L0024","section":"{…} of {…}","text":"It holds work until someone decides.","placement":"unplaced","state":"card"},
+        {"key":"L0025","section":"{…} of {…}","text":"Tab · ↑ ↓ items ← → buildings","placement":"unplaced","state":"card"},
+        {"key":"L0026","section":"{…} of {…}","text":"{…} all 9","placement":"unplaced","state":"card"},
+        {"key":"L0027","section":"{…} of {…}","text":"esc","placement":"unplaced","state":"card"},
+        {"key":"L0028","section":"{…} of {…}","text":"{…} 37 in a row","placement":"unplaced","state":"card"},
+        {"key":"L0029","section":"page top","text":"{…} 37 in a row","placement":"unplaced"},
+        {"key":"L0030","section":"37 to do · 5 {…} working","text":"37 to do · 5 {…} working","placement":"unplaced"},
+        {"key":"L0031","section":"37 to do · 5 {…} working","text":"Archived:","placement":"unplaced"},
+        {"key":"L0032","section":"37 to do · 5 {…} working","text":"Street","placement":"unplaced"},
+        {"key":"L0033","section":"37 to do · 5 {…} working","text":"Building Level Views","placement":"placed","file":"modules/hall-ui/public/city.html","line":103,"string_id":"5df2aa54f042"},
+        {"key":"L0034","section":"37 to do · 5 {…} working","text":"Full screen — press Esc, or the button in the corner, to bring the bars back.","placement":"placed","file":"modules/hall-ui/public/shell.js","line":595,"string_id":"11fc0483c149","state":"fullscreen"},
+        {"key":"L0035","section":"37 to do · 5 {…} working","text":"Builders hall · this session","placement":"placed","file":"modules/hall-ui/public/craft-mode.js","line":674,"string_id":"fd1903586e8e"},
+        {"key":"L0036","section":"What are you doing now?","text":"What are you doing now?","placement":"placed","file":"modules/hall-ui/public/craft-mode.js","line":675,"string_id":"60853319f290"},
+        {"key":"L0037","section":"What are you doing now?","text":"Pick one. You can switch any time from the top bar.","placement":"placed","file":"modules/hall-ui/public/craft-mode.js","line":676,"string_id":"4f87d5b17b54"},
+        {"key":"L0038","section":"What are you doing now?","text":"Building","placement":"unplaced"},
+        {"key":"L0039","section":"What are you doing now?","text":"{…} and {…}","placement":"placed","file":"modules/hall-ui/public/profile-nudge.js","line":77,"string_id":"56b00d773232","occurrences":3},
+        {"key":"L0040","section":"What are you doing now?","text":"Artistry","placement":"unplaced"},
+        {"key":"L0041","section":"What are you doing now?","text":"Ideating","placement":"unplaced"},
+        {"key":"L0042","section":"What are you doing now?","text":"Turn thoughts into {…}.","placement":"unplaced"},
+        {"key":"L0043","section":"What are you doing now?","text":"Ask me again next session","placement":"placed","file":"modules/hall-ui/public/craft-mode.js","line":679,"string_id":"fa86bf403775"}
       ]
     },
     {

package/modules/copy-desk/routes/copy-desk.js

@@ -690,10 +690,10 @@ module.exports = function buildCopyDeskRouter() {
       const body = proposals.proposalTaskBody(patch);
       const task = await lifecycle.createTask({
         versionId: building.id,
-        // BV1.R11 (task 1003598): no goal to name until the per-version maintenance
-        // goal exists (task 1003605, R18). Declared out loud rather than relying on
-        // a silent default — see createTask.
-        allowCatchAll: true,
+        // task 1004819: a copy proposal is studio work, so it lands in the version's
+        // artist review goal (created on first use; the catch-all only when the
+        // version cannot take one) — see createTask.
+        artistReview: true,
         title: proposals.proposalTaskTitle(patch),
         description: flagRef
           ? `${body}\n\nRaised from copy-desk flag ${flagRef}.`

package/modules/hall-ui/public/city-decide.js

@@ -46,7 +46,7 @@
       if (choice && choice.options && choice.options.length) {
         var bare = a.key === 'decide' || a.key === 'answer';
         choice.options.forEach(function (o) {
-          opts.push({ key: a.key + ':' + str(o.key), label: bare ? str(o.label) : str(a.label) + ': ' + str(o.label), result: '', act: a.key, rec: false, plan: plan(o.input), off: false, why: '' });
+          opts.push({ key: a.key + ':' + str(o.key), label: (function (w) { return bare ? w : str(a.label) + ': ' + w; }(str(o.input && o.input.label) || str(o.label))), result: '', act: a.key, rec: false, plan: plan(o.input), off: false, why: '' });
         });
       } else {
         opts.push({ key: a.key, label: str(a.label), result: '', act: a.key, rec: false, plan: plan({}), off: false, why: '' });
@@ -221,10 +221,44 @@
     return b;
   }
 
+  function matterOf(docket, id) {
+    var ms = docket && Array.isArray(docket.matters) ? docket.matters : [];
+    for (var i = 0; i < ms.length; i++) if (ms[i] && ms[i].id === id) return ms[i];
+    return null;
+  }
+  // The matter's whole text, when it says more than the card already shows.
+  function fullText(matter, shown) {
+    if (!matter) return '';
+    var parts = [];
+    var title = str(matter.title).trim();
+    if (title && title !== str(shown.question).trim()) parts.push(title);
+    var detail = str((matter.ask && matter.ask.detail) || matter.detail).trim();
+    if (detail && detail !== title) parts.push(detail);
+    return parts.join('\n\n');
+  }
+
   // The deck's card, answerable; a typed answer gets its own box and send.
   function drawCard(doc, K, cv, on, o) {
     var shown = answerable(cv);
+    var matter = matterOf(on.docket && on.docket(), cv.id);
+    // the question in full: a matter with no stored card is titled by its own words, cut to fit the deck
+    if (matter && /…$/.test(str(shown.question)) && matter.title) shown.question = str(matter.title).trim();
     var art = K.drawAsk(doc, shown, on, o);
+    // and everything else it says, one tap away rather than cut
+    var full = fullText(matter, shown);
+    if (full) {
+      var more = doc.createElement('details');
+      more.className = 'ask-full';
+      var sum = doc.createElement('summary');
+      sum.textContent = 'Full text';
+      more.appendChild(sum);
+      var body = doc.createElement('p');
+      body.className = 'ask-full__text';
+      body.textContent = full;
+      more.appendChild(body);
+      var opts = art.querySelector ? art.querySelector('.ask__opts') : null;
+      if (opts && typeof art.insertBefore === 'function') art.insertBefore(more, opts); else art.appendChild(more);
+    }
     // the crab: open this one in Claude to help decide it
     var q = art.querySelector ? art.querySelector('.ask__q') : null;
     if (q && q.parentNode && typeof q.parentNode.insertBefore === 'function') {
@@ -638,5 +672,5 @@
     return { open: open, close: close, render: render, isOpen: function () { return st.open; } };
   }
 
-  window.GovCityDecide = { make: make, answerable: answerable, drawCard: drawCard, flash: flash, armKey: armKey, rewordingIds: rewordingIds, rewordingCount: rewordingCount, withoutReworded: withoutReworded, paintArmed: paintArmed, promptOf: promptOf, CLAUDE_URL: CLAUDE_URL };
+  window.GovCityDecide = { make: make, answerable: answerable, drawCard: drawCard, flash: flash, fullText: fullText, armKey: armKey, rewordingIds: rewordingIds, rewordingCount: rewordingCount, withoutReworded: withoutReworded, paintArmed: paintArmed, promptOf: promptOf, CLAUDE_URL: CLAUDE_URL };
 }());

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

@@ -394,8 +394,8 @@ html[data-fullscreen="on"] .city-top { right: 184px; }
 .swb__row { display: flex; align-items: center; gap: 10px; min-height: 40px; border-top: 1px solid var(--rule-soft); padding-top: 6px; }
 .swb__row:first-of-type { border-top: 0; padding-top: 0; }
 .swb__label { flex: 1; min-width: 0; display: flex; flex-direction: column; }
-.swb__name { font-size: 14px; color: var(--ink); overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
-.swb__fact { font-size: 12px; color: var(--ink-soft); overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
+.swb__name { font-size: 14px; line-height: 1.35; color: var(--ink); overflow-wrap: anywhere; }
+.swb__fact { font-size: 12px; color: var(--ink-soft); overflow-wrap: anywhere; }
 .swb__toggle { flex: none; position: relative; display: inline-flex; align-items: center; width: 64px; height: 30px; padding: 0 8px; box-sizing: border-box; border-radius: 999px; border: 1px solid var(--gov-chip-edge); background: var(--gov-chip-pick); color: var(--ink-soft); font-family: var(--font-body); font-size: 11.5px; cursor: pointer; transition: background 160ms ease; }
 .swb__knob { position: absolute; top: 3px; left: 3px; width: 22px; height: 22px; border-radius: 50%; background: var(--ink-soft); transition: left 160ms ease, background 160ms ease; }
 .swb__state { margin-left: auto; }
@@ -451,3 +451,9 @@ html[data-fullscreen="on"] .city-top { right: 184px; }
 .ask-reword__go:focus-visible, .ask-reword__no:focus-visible, .ask-reword__note:focus-visible { outline: 2px solid var(--focus); outline-offset: 2px; }
 /* only drawn when something is out for a rewrite; a quiet footnote to the counts */
 .city-card__rewording, .decide__rewording { display: block; margin-top: 1px; font-size: 10.5px; color: var(--ink-faint); text-align: right; }
+
+/* a card's whole text, one tap away instead of cut (city-decide.js) */
+.ask-full { font-size: 12.5px; color: var(--ink-soft); }
+.ask-full summary { display: inline-flex; align-items: center; min-height: 24px; cursor: pointer; color: var(--ink); text-decoration: underline; text-underline-offset: 3px; text-decoration-color: var(--gov-chip-edge-pick); }
+.ask-full__text { margin: 6px 0 0; max-height: 240px; overflow-y: auto; white-space: pre-wrap; line-height: 1.5; overflow-wrap: anywhere; }
+.ask-full summary:focus-visible { outline: 2px solid var(--focus); outline-offset: 2px; }

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

@@ -12,7 +12,7 @@
        (tests/hall_tokens.mjs: no hall page hardcodes a web-font stylesheet).
        The prototype carried its own Google Fonts link for the same two faces. -->
   <link rel="stylesheet" href="/builders/style.css?v=2026-07-02-shell1">
-  <link rel="stylesheet" href="/builders/city.css?v=2026-10-23-decide11">
+  <link rel="stylesheet" href="/builders/city.css?v=2026-10-23-decide12">
   <!-- the ask card in the glass language, shared with the street's deck (task 1004702) -->
   <link rel="stylesheet" href="/builders/street-deck.css?v=2026-10-17-tidy">
 </head>
@@ -115,14 +115,14 @@
 <!-- a building's card answers its first ask in place through the deck's own
      route (task 1004702): the desk rows (street-desk.js over the street's kit)
      and the deck's card and sender (street-deck.js) -->
-<script src="/builders/hall-kit.js?v=2026-10-23-decide11"></script>
+<script src="/builders/hall-kit.js?v=2026-10-23-decide12"></script>
 <script src="/builders/street-draw.js?v=2026-10-09-living3"></script>
 <script src="/builders/street-card.js?v=2026-10-04-record1"></script>
 <script src="/builders/street-inside.js?v=2026-10-09-living3"></script>
 <script src="/builders/street-desk.js?v=2026-10-04-record1"></script>
 <script src="/builders/street-deck.js?v=2026-10-13-glass1"></script>
-<script src="/builders/city-decide.js?v=2026-10-23-decide11"></script>
-<script src="/builders/city-switchboard.js?v=2026-10-23-board4"></script>
-<script src="/builders/city.js?v=2026-10-23-decide11"></script>
+<script src="/builders/city-decide.js?v=2026-10-23-decide12"></script>
+<script src="/builders/city-switchboard.js?v=2026-10-23-board5"></script>
+<script src="/builders/city.js?v=2026-10-23-decide12"></script>
 </body>
 </html>

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.174",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bongos/core",
-      "version": "1.21.173",
+      "version": "1.21.174",
       "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.174",
   "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,15 @@
       "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"
+    }
   ]
 }

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.174'; // 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');

package/tests/artist_review_goal.mjs

@@ -0,0 +1,119 @@
+// tests/artist_review_goal.mjs
+//
+// Every version's ARTIST REVIEW goal (task 1004819): the find-or-create behind
+// createTask's `artistReview` switch and createVersion. DB-free: a recording fake
+// client stands in for Postgres, and the statement order is the contract — the
+// version gate before anything else, the advisory lock before the read, and the
+// two standing criteria written only when the goal is new.
+//
+// Run:  node tests/artist_review_goal.mjs
+
+import { strict as assert } from 'node:assert';
+import { createRequire } from 'node:module';
+import { makeRunner } from './helpers.mjs';
+
+process.env.NODE_ENV = 'test';
+
+const require = createRequire(import.meta.url);
+const { ensureArtistReviewGoal, artistReviewGoalTitle, ARTIST_REVIEW_CRITERIA } = require('../modules/lifecycle/artist-review-goal.js');
+const { createTask } = require('../modules/lifecycle/db.js');
+
+const { test, summary } = makeRunner();
+const norm = (s) => String(s).replace(/\s+/g, ' ').trim();
+
+// One version, at most one goal: the fake answers like the tables would.
+function fakeClient({ status = 'building', goal = null } = {}) {
+  const queries = [];
+  const state = { goal, criteria: [], tasks: [] };
+  const client = {
+    queries, state,
+    async query(sql, params = []) {
+      const q = norm(sql);
+      queries.push({ sql: q, params });
+      if (/^(BEGIN|COMMIT|ROLLBACK)$/.test(q)) return { rows: [] };
+      if (/^SELECT status FROM versions WHERE id = \$1/.test(q)) return { rows: status ? [{ status }] : [] };
+      if (/pg_advisory_xact_lock/.test(q)) return { rows: [] };
+      if (/^SELECT id, version_id, title FROM goals WHERE version_id = \$1 AND title = \$2/.test(q)) {
+        return { rows: state.goal && state.goal.title === params[1] ? [state.goal] : [] };
+      }
+      if (/^INSERT INTO goals/.test(q)) { state.goal = { id: 91, version_id: params[0], title: params[1] }; return { rows: [state.goal] }; }
+      if (/^INSERT INTO done_when_criteria/.test(q)) { state.criteria.push({ id: params[1], goal: params[3] }); return { rows: [] }; }
+      if (/^SELECT scope_modules FROM goals WHERE id = \$1/.test(q)) return { rows: [{ scope_modules: [] }] };
+      if (/^INSERT INTO tasks/.test(q)) { const t = { id: 1, goal_id: params[22] }; state.tasks.push(t); return { rows: [t] }; }
+      return { rows: [] };
+    },
+    release() {},
+  };
+  return client;
+}
+
+await test('the title is the version plus " — artist review" (em dash)', () => {
+  assert.equal(artistReviewGoalTitle('BONGOS-V3'), 'BONGOS-V3 — artist review');
+  assert.deepEqual(ARTIST_REVIEW_CRITERIA.map((c) => c.criterion_id), ['artist-every-page-reviewed', 'artist-pages-simple-to-use']);
+});
+
+await test('a new goal is made with its two criteria, after the version gate and the lock', async () => {
+  const c = fakeClient();
+  const g = await ensureArtistReviewGoal('BONGOS-V3', { client: c });
+  assert.equal(g.title, 'BONGOS-V3 — artist review');
+  const order = c.queries.map((q) => q.sql.split(' ').slice(0, 3).join(' '));
+  const at = (re) => c.queries.findIndex((q) => re.test(q.sql));
+  assert.ok(at(/FROM versions/) < at(/pg_advisory_xact_lock/), 'the version is checked first');
+  assert.ok(at(/pg_advisory_xact_lock/) < at(/FROM goals WHERE version_id/), 'the lock is taken before the read');
+  assert.equal(c.queries.find((q) => /pg_advisory_xact_lock/.test(q.sql)).params[0], 'BONGOS-V3 — artist review', 'locked on the title');
+  assert.deepEqual(c.state.criteria, [{ id: 'artist-every-page-reviewed', goal: 91 }, { id: 'artist-pages-simple-to-use', goal: 91 }]);
+  assert.ok(/ON CONFLICT \(version_id, criterion_id\) DO NOTHING/.test(c.queries.find((q) => /INSERT INTO done_when_criteria/.test(q.sql)).sql),
+    'a criterion already on the version is left alone', order);
+});
+
+await test('an existing goal is reused, never duplicated, and still carries the two criteria', async () => {
+  // A goal of this title made by hand before the first studio write is found,
+  // and its criteria are seeded (ON CONFLICT DO NOTHING keeps any already there).
+  const c = fakeClient({ goal: { id: 1000134, version_id: 'BONGOS-V2', title: 'BONGOS-V2 — artist review' } });
+  const g = await ensureArtistReviewGoal('BONGOS-V2', { client: c });
+  assert.equal(g.id, 1000134);
+  assert.equal(c.queries.some((q) => /^INSERT INTO goals/.test(q.sql)), false, 'no second goal');
+  assert.deepEqual(c.state.criteria, [{ id: 'artist-every-page-reviewed', goal: 1000134 }, { id: 'artist-pages-simple-to-use', goal: 1000134 }]);
+});
+
+await test('a closed or missing version gets no goal, and nothing is locked or written', async () => {
+  for (const status of ['shipped', null]) {
+    const c = fakeClient({ status });
+    assert.equal(await ensureArtistReviewGoal('V0', { client: c }), null);
+    assert.equal(c.queries.length, 1, 'only the version read ran');
+  }
+  const planning = fakeClient({ status: 'planning' });
+  assert.ok(await ensureArtistReviewGoal('V4', { client: planning }), 'a planning version takes one, so a new version is born with it');
+});
+
+await test('with no caller transaction it opens its own, so the lock means something', async () => {
+  const c = fakeClient();
+  const pool = { connect: async () => c, query: (...a) => c.query(...a) };
+  await ensureArtistReviewGoal('BONGOS-V3', { pool });
+  const sqls = c.queries.map((q) => q.sql);
+  assert.equal(sqls[0], 'BEGIN');
+  assert.equal(sqls.at(-1), 'COMMIT');
+});
+
+await test('createTask with artistReview lands in the goal; without one the catch-all still answers', async () => {
+  const c = fakeClient();
+  await createTask({ versionId: 'BONGOS-V3', title: 'Tweak: Home (builders:home)', discipline: 'artist', artistReview: true }, { client: c });
+  assert.equal(c.state.tasks[0].goal_id, 91, 'the studio task names the artist review goal');
+  await assert.rejects(
+    () => createTask({ versionId: 'BONGOS-V3', title: 'plain', discipline: 'engineer' }, { client: fakeClient() }),
+    /goalId is required/, 'an ordinary goal-less create still throws');
+});
+
+await test('a create with no caller transaction (a copy proposal) puts goal and task in ONE transaction', async () => {
+  const c = fakeClient();
+  const pool = { connect: async () => c, query: () => { throw new Error('nothing may run outside the transaction'); } };
+  await createTask({ versionId: 'BONGOS-V3', title: 'Reword the Home heading', discipline: 'artist', artistReview: true }, { pool });
+  const sqls = c.queries.map((q) => q.sql);
+  const begins = sqls.filter((q) => q === 'BEGIN').length;
+  assert.equal(begins, 1, 'exactly one transaction');
+  assert.ok(sqls.indexOf('BEGIN') < sqls.findIndex((q) => /^INSERT INTO goals/.test(q)), 'the goal is made inside it');
+  assert.ok(sqls.findIndex((q) => /^INSERT INTO tasks/.test(q)) < sqls.lastIndexOf('COMMIT'), 'and the task too, before the one COMMIT');
+  assert.equal(c.state.tasks[0].goal_id, 91);
+});
+
+summary();

package/tests/copy_desk_page_claim.mjs

@@ -86,8 +86,13 @@ try {
     assert.equal(db.tasks.length, 1);
     assert.deepEqual(
       [t.source, t.source_ref, t.title, t.kind, t.discipline, t.credits_reward, t.version_id, t.goal_id, t.status],
-      ['page-tweak', 'page-tweak/builders:studio/r1', 'Tweak: Studio (builders:studio)', 'feature', 'artist', 30, 'V9', '77', 'active']
+      ['page-tweak', 'page-tweak/builders:studio/r1', 'Tweak: Studio (builders:studio)', 'feature', 'artist', 30, 'V9', '88', 'active']
     );
+    // task 1004819: the round lands in the version's artist review goal, made on
+    // first use with its two standing criteria — never the catch-all.
+    assert.equal(db.artistGoal.title, 'V9 — artist review');
+    assert.deepEqual(db.criteria.map((x) => [x.criterion_id, x.goal_id]),
+      [['artist-every-page-reviewed', '88'], ['artist-pages-simple-to-use', '88']]);
     assert.deepEqual(t.touches, F.STUDIO_FILES, 'touches are the inventory page files');
     const c = db.claims[0];
     assert.deepEqual([c.builder_id, c.worktree_name, c.creator_session_id], [ALICE, null, null], 'a web claim: no worktree, no session');

package/tests/copy_desk_page_submit.mjs

@@ -208,7 +208,7 @@ try {
       `page-tweak-unplaced/${studioTask().id}/L0005`,
     ]);
     for (const t of eng) {
-      assert.deepEqual([t.status, t.discipline, t.kind, t.version_id, t.goal_id, t.created_by], ['backlog', 'engineer', 'feature', 'V9', '77', ALICE]);
+      assert.deepEqual([t.status, t.discipline, t.kind, t.version_id, t.goal_id, t.created_by], ['backlog', 'engineer', 'feature', 'V9', '88', ALICE], 'an unplaced line lands in the artist review goal (task 1004819)');
       assert.match(t.description, /builders:studio/);
     }
     const quiet = eng.find((t) => t.source_ref.endsWith('/L0005'));

package/tests/fixtures/page-ledger.mjs

@@ -36,6 +36,7 @@ export function installPageLedger(api, { builders }) {
   // first commits or rolls back.
   const locks = new Map();
   async function lock(key, owner) {
+    if (locks.has(key) && locks.get(key).owner === owner) return; // re-entrant, as Postgres is for one session
     while (locks.has(key)) await locks.get(key).done;
     let release;
     const done = new Promise((r) => { release = r; });
@@ -109,6 +110,18 @@ export function installPageLedger(api, { builders }) {
     if (/FROM goals g JOIN versions v ON v\.id = g\.version_id WHERE g\.version_id = \$1 AND g\.title = \$2/.test(q)) {
       return { rows: [{ id: '77', version_status: 'building' }] };
     }
+    // task 1004819: studio work lands in the version's artist review goal, found
+    // (or made, with its two criteria) on the same transaction client.
+    if (/^SELECT status FROM versions WHERE id = \$1/.test(q)) return { rows: [{ status: 'building' }] };
+    if (/^SELECT id, version_id, title FROM goals WHERE version_id = \$1 AND title = \$2/.test(q)) {
+      assert.ok(owner, 'the artist review goal must be found on the transaction client');
+      return { rows: db.artistGoal ? [{ ...db.artistGoal }] : [] };
+    }
+    if (/^INSERT INTO goals \(version_id, title, description, scope_modules, status, created_by, sort_order\)/.test(q)) {
+      db.artistGoal = { id: '88', version_id: p[0], title: p[1] };
+      return { rows: [{ ...db.artistGoal }] };
+    }
+    if (/^INSERT INTO done_when_criteria/.test(q)) { (db.criteria = db.criteria || []).push({ version_id: p[0], criterion_id: p[1], goal_id: p[3] }); return { rows: [] }; }
     if (/^SELECT scope_modules FROM goals WHERE id = \$1/.test(q)) return { rows: [{ scope_modules: [] }] };
     if (/^INSERT INTO tasks \(version_id, title, description, status, touches,/.test(q)) {
       assert.ok(owner, 'the round must be created inside the claim transaction');

package/tests/hall_city_rewording.mjs

@@ -49,3 +49,14 @@ test('the "?" asks before it sends: the send lives behind a confirm button', ()
   assert.match(fn, /Send this to Claude to rewrite\?/);
   assert.match(fn, /Nothing is decided or changed/);
 });
+
+test('a card never cuts what it says: the whole text is one tap away, and only when there is more', () => {
+  const G = boot();
+  const m = { id: 'override_requests:1', title: 'Override: a long title', detail: 'The whole story, word for word.' };
+  assert.equal(G.fullText(m, { question: 'Release it?' }), 'Override: a long title\n\nThe whole story, word for word.');
+  assert.equal(G.fullText(m, { question: 'Override: a long title' }), 'The whole story, word for word.', 'the question is not repeated');
+  assert.equal(G.fullText({ id: 'x:1', title: 'Same' }, { question: 'Same' }), '', 'nothing more to show, no opener');
+  assert.equal(G.fullText(null, { question: 'Q?' }), '');
+  const src = fs.readFileSync(path.join(PUB, 'city-decide.js'), 'utf8');
+  assert.match(src, /str\(o\.input && o\.input\.label\) \|\| str\(o\.label\)/, 'an answer uses its full words, not the desk chip cut to fit');
+});

package/tests/task_goal_required.mjs

@@ -113,12 +113,11 @@ test('POST /tasks does NOT pass the escape hatch — this door always has a goal
 
 const OPT_INS = [
   ['modules/onboarding/newcomer-restock.js', 'the beginner queue, stamped from templates'],
-  ['modules/copy-desk/routes/copy-desk.js', 'copy-change proposals'],
   ['modules/lifecycle/kickoff-seed.js', "the founder's first board, on an instance with no goals at all"],
   ['modules/lifecycle/cascade.js', 'a follow-up off a parent task that is itself goal-less'],
 ];
 
-test('exactly four vectors opt in, and each says why', () => {
+test('exactly three vectors opt in, and each says why', () => {
   for (const [file, why] of OPT_INS) {
     const body = src(file);
     assert.match(body, /allowCatchAll/, `${file} should opt in (${why})`);
@@ -127,6 +126,17 @@ test('exactly four vectors opt in, and each says why', () => {
   }
 });
 
+// task 1004819: studio work names its home instead of the catch-all. Each of the
+// three studio create vectors passes artistReview, which lands it in the
+// version's artist review goal; none of them opts into the catch-all any more.
+test('the three studio vectors land in the artist review goal, not the catch-all', () => {
+  for (const file of ['modules/copy-desk/routes/copy-desk.js', 'modules/lifecycle/page-tweak-claim.js', 'modules/lifecycle/page-tweak-write.js']) {
+    const body = src(file);
+    assert.match(body, /artistReview: true/, `${file} should name the artist review goal`);
+    assert.equal(/allowCatchAll/.test(body), false, `${file} must not opt into the catch-all`);
+  }
+});
+
 test('cascade opts in ONLY when inheritance actually failed', () => {
   // The others opt in unconditionally because they have no goal to name at all.
   // A cascade normally inherits its parent's goal and must keep doing so — the