@bongos/core 1.19.322 → 1.19.323

package/.bongos-core.json

@@ -2,22 +2,22 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/gds/package-core.js",
-  "core_version": "1.19.322",
-  "core_contract": "1.19.322",
-  "source_commit": "2fd56f3db9adacbb215ffdc56bba734266ef8e89",
+  "core_version": "1.19.323",
+  "core_contract": "1.19.323",
+  "source_commit": "6b1fb52bfb9b8d899ffc44c0b8a22fd08cab9379",
   "source_ref": "HEAD",
-  "built_at": "2026-08-24T01:23:28.434Z",
+  "built_at": "2026-08-24T01:33:26.263Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 324,
     "agent_docs_stubbed": 22,
-    "functional_verbatim": 1455,
+    "functional_verbatim": 1457,
     "rules": 3,
     "gate_literals": 3,
     "gate": "passed"
   },
-  "file_count": 1801,
-  "tree_sha256": "6cd259fb90abf1a7df74cc5e7dffead0cfa9c5d0f636032c51b94e715d21b31d",
+  "file_count": 1803,
+  "tree_sha256": "6534c0e9c341ea8d670e2f26984693052bc41041c70f99e671d5abd7ca3448dc",
   "files": [
     {
       "path": ".claude/skills/blocker-review/SKILL.md",
@@ -367,7 +367,7 @@
     {
       "path": "clients/bongos-client/index.d.ts",
       "mode": "0000644",
-      "sha256": "85041b30fb2113ca5b3684ae961e67dc2bd5dfdb152064b151af3f0a93a29f31"
+      "sha256": "356ef6ee04671774868423f5312cedd2ae194f1f4abcdcaaac01828b044c1c14"
     },
     {
       "path": "clients/bongos-client/index.mjs",
@@ -1422,12 +1422,12 @@
     {
       "path": "docs/api-reference.md",
       "mode": "0000644",
-      "sha256": "a654d9def4780d3210c66031193198f58fd0d7d84f5088c4ce7e6d0a58682193"
+      "sha256": "25a1978be80e817a5a9b54609aaa52dc0e172c941110183b9cfc93d193430f0b"
     },
     {
       "path": "docs/api/openapi.json",
       "mode": "0000644",
-      "sha256": "e0b531f19187a3c00a54917bff4e2827a75b250f1aaa761229660a974a4f6217"
+      "sha256": "1cbc9af7fe828c5c81e6348a1a219b53ab484f9df67ec9ccb41e6f6cdef1693e"
     },
     {
       "path": "docs/architecture.md",
@@ -1452,12 +1452,12 @@
     {
       "path": "docs/copy-inventory.md",
       "mode": "0000644",
-      "sha256": "265f46593fe663dc38aa7980b0eacde8e6ab001b75301c5d0e38f1c1887d0e11"
+      "sha256": "9b667d64d98c00a5a0d5991f3f69a783d82f8368719f50610d7514fff4d8ebe7"
     },
     {
       "path": "docs/copy-registry.json",
       "mode": "0000644",
-      "sha256": "e2e2af87ad6c8023e6b7399caac2372f08a87f41239873e83dbff39ac2fe1855"
+      "sha256": "039628f57cb1bbda153b4450053cb9ca10e9eb7bad9c950854250139ac810c4a"
     },
     {
       "path": "docs/design-contract.md",
@@ -2459,6 +2459,11 @@
       "mode": "0000644",
       "sha256": "8fc38c0353811699e19d861321d4fca8fe8fd5cca1f66ff96312d7f44fb0290e"
     },
+    {
+      "path": "migrations/core_224_goal_subtitle.sql",
+      "mode": "0000644",
+      "sha256": "a379e9f926274559d8c57c9127024463e38b99e3f73962022b9f5d0f4e156a57"
+    },
     {
       "path": "modules/autonomy/CLAUDE.md",
       "mode": "0000644",
@@ -3417,22 +3422,22 @@
     {
       "path": "modules/hall-ui/public/goals-lib.js",
       "mode": "0000644",
-      "sha256": "b899d1245f1138bf0e82115b80664eaa68ed2601c940bb0848ce2b784866d3e3"
+      "sha256": "a7a0385c3c8026e6b21ac5793bdf5b48fa31968f176ebe891c43ad5c867df1c8"
     },
     {
       "path": "modules/hall-ui/public/goals-page.js",
       "mode": "0000644",
-      "sha256": "472483a2a5c8d5a2d033e630de2c24025dcbc7d7fc40017f64ea0d18f830e3a0"
+      "sha256": "969aee56c6ec869a04d6e01752b24109114bd10e061c4a16fb8c7fa050671f47"
     },
     {
       "path": "modules/hall-ui/public/goals.css",
       "mode": "0000644",
-      "sha256": "ef69c360f46f508b1e987d8e9e523ed8708b7c74f48b6069103fae14ccc710f9"
+      "sha256": "4ebae2d1b12a4a6a196e510dc034e2534ca0429e516292babdcb04917188c004"
     },
     {
       "path": "modules/hall-ui/public/goals.html",
       "mode": "0000644",
-      "sha256": "a9a6a23f60abba6072114db729691714819fefd316d42441462653e44eeab27b"
+      "sha256": "f3656a4952d581c902c91c267a576193a91e60617b45745c879157e9cc8e702e"
     },
     {
       "path": "modules/hall-ui/public/goals.js",
@@ -3852,7 +3857,7 @@
     {
       "path": "modules/lifecycle/db-goals.js",
       "mode": "0000644",
-      "sha256": "29b950000d31618de168c6128d9b78c03daad7e68b9a9b6d3e83aa40abd3d9fb"
+      "sha256": "b9d1768f8e271ad5c3595728f2433c27f44c841ab1c6be7d24086d1f6c8c7691"
     },
     {
       "path": "modules/lifecycle/db-grade.js",
@@ -3932,7 +3937,7 @@
     {
       "path": "modules/lifecycle/goal-edits.js",
       "mode": "0000644",
-      "sha256": "0367f88aefa890d8d7f2472dd847467d632150a93db8a96b85717b5fb1cc8c9e"
+      "sha256": "7115a27272fe2594b65e2e56a796f5bf97600cb94ba301bc21599c6bc4e60175"
     },
     {
       "path": "modules/lifecycle/goal-suggest.js",
@@ -4007,7 +4012,7 @@
     {
       "path": "modules/lifecycle/routes/goals.js",
       "mode": "0000644",
-      "sha256": "3fcc0339927bcc7f426e93c7a5762b45496ecf4e9b841f905fb5f1c468b5f8c0"
+      "sha256": "a58f01647120ce0b1143314f5edab2501bc18b8a748073dac788c92aaa0484ca"
     },
     {
       "path": "modules/lifecycle/routes/lifecycle.js",
@@ -4647,12 +4652,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "c9390686abc47c1954f212e7e4ebbce0320cfd21d9acc1fd2fe79d4f9667ab79"
+      "sha256": "670e665f41fbb4d1c4154b2e2a08dcb64383583a8162188101dd0807ae849caa"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "290a425b20d5c5f87bc9a1eaeb154d02f8ca95e8147dd7a6f5835ea443d25ec4"
+      "sha256": "27ab832e2bac7f7baac4016571d5536ac884b4b0f125a4b6f6a2dbae9b73112f"
     },
     {
       "path": "public-docs/index.html",
@@ -6422,7 +6427,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "d808c8f826309876d6ce3bf99e47ca874b5afa0efe108fcbda629328c45d9b00"
+      "sha256": "8624156a5587eaca3430325ef66420fb96b9567ed19af27dce361dc8af08611f"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -7404,6 +7409,11 @@
       "mode": "0000644",
       "sha256": "447e6376ae5f5e2bedf1a80b3355604544a5e349da87d110fc0135cc3cd98fac"
     },
+    {
+      "path": "tests/goal_subtitle.mjs",
+      "mode": "0000644",
+      "sha256": "4505dbeaafba7e7ca716d693864bea3246b4fe8767386bf0c7f3d48c41030db1"
+    },
     {
       "path": "tests/goal_suggest.mjs",
       "mode": "0000644",
@@ -7412,7 +7422,7 @@
     {
       "path": "tests/goal_tier.mjs",
       "mode": "0000644",
-      "sha256": "00539cd1b188def53c67aab06301971132fbbe0bcd47818c19763ed760aaccda"
+      "sha256": "e6a3114339989123d20f42f120f9fed52690fbc0230b10a99fa1fd0453acbfd2"
     },
     {
       "path": "tests/goal_wall_proof.mjs",

package/clients/bongos-client/index.d.ts

@@ -238,7 +238,7 @@ export interface PostGoalsIdTransferRequest { builder_id?: StringifiableId }
 export interface PostGoalsIdTransferResponse { ok: boolean; goal: unknown; changed: boolean }
 export interface PostGoalsReorderRequest { version_id: string; ordered_ids: number[] }
 export interface PostGoalsReorderResponse { ok: boolean; goals: unknown }
-export interface PostGoalsRequest { version_id: string; title: string; description?: string; scope_modules?: string[]; succeeds_goal_id?: number }
+export interface PostGoalsRequest { version_id: string; title: string; subtitle?: string; description?: string; scope_modules?: string[]; succeeds_goal_id?: number }
 export interface PostGoalsResponse { ok: boolean; goal: unknown; succeeds_goal_id: unknown }
 export interface PostGovernmentAssignmentsRequest { builder_id: number; rank_key: string }
 export interface PostGovernmentBoardItemsItemIdVotesRequest { direction: string; reason_md?: string; section_key?: string }

package/docs/api/openapi.json

@@ -4316,7 +4316,7 @@
           "goals"
         ],
         "summary": "PATCH /goals/:id",
-        "description": "PATCH /goals/:id — correct a goal's own title/description (task 1003102). Handler + rationale + rank note: ../goal-edits.js. Pinned in route-rank-check.\n\n**Rank:** `any-builder` — Any authenticated builder (row-level ownership enforced in-handler).",
+        "description": "PATCH /goals/:id — correct a goal's own title/subtitle/description (tasks 1003102, 1003224). Handler + rationale + rank note: ../goal-edits.js. Pinned in route-rank-check.\n\n**Rank:** `any-builder` — Any authenticated builder (row-level ownership enforced in-handler).",
         "x-rank": "any-builder",
         "x-source": "modules/lifecycle/routes/goals.js",
         "parameters": [
@@ -16983,6 +16983,10 @@
             "minLength": 1,
             "maxLength": 200
           },
+          "subtitle": {
+            "type": "string",
+            "maxLength": 200
+          },
           "description": {
             "type": "string",
             "maxLength": 50000

package/docs/api-reference.md

@@ -202,9 +202,9 @@ Base path: `/api/bongos`. Ranks (enforced server-side, [ADR 0016](adr/<redacted>
 | Method | Path | Rank | Body | Description |
 |---|---|---|---|---|
 | GET | `/api/bongos/goals` | `any-builder` | — | GET /goals?version=&status= — list goals (read). |
-| POST | `/api/bongos/goals` | `metic+archon` | `version_id`, `title`, `description`, `scope_modules`, `succeeds_goal_id` | POST /goals — create a goal. |
+| POST | `/api/bongos/goals` | `metic+archon` | `version_id`, `title`, `subtitle`, `description`, `scope_modules`, `succeeds_goal_id` | POST /goals — create a goal. |
 | GET | `/api/bongos/goals/:id` | `any-builder` | — | GET /goals/:id — one goal with its members + linked criteria (read). |
-| PATCH | `/api/bongos/goals/:id` | `any-builder` | — | PATCH /goals/:id — correct a goal's own title/description (task 1003102). |
+| PATCH | `/api/bongos/goals/:id` | `any-builder` | — | PATCH /goals/:id — correct a goal's own title/subtitle/description (tasks 1003102, 1003224). |
 | POST | `/api/bongos/goals/:id/archive` | `metic+archon` | — | POST /goals/:id/archive — retire a goal (open\|achieved → archived). |
 | GET | `/api/bongos/goals/:id/conflicts` | `any-builder` | — | GET /goals/:id/conflicts — the server-side task-conflict view (task 1736). |
 | POST | `/api/bongos/goals/:id/conflicts/resolve` | `any-builder` | `action`, `note`, `touches`, `first_task_id`, `then_task_id`, `task_id` | POST /goals/:id/conflicts/resolve — the owner/manager resolution actions for a detected conflict (task 1736; authority model ADR 0106): s… |

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).
 
-**1379 strings** across 3 surfaces · 1224 distinct pieces of copy · 1379 certain, 0 heuristic · 125 shown in more than one place.
+**1380 strings** across 3 surfaces · 1225 distinct pieces of copy · 1380 certain, 0 heuristic · 125 shown in more than one place.
 
 ## Surfaces
 
 | surface | lives at | files | strings |
 |---|---|---|---|
-| `builders-hall` | `modules/hall-ui/public` | 65 | 1095 |
+| `builders-hall` | `modules/hall-ui/public` | 65 | 1096 |
 | `status-dashboard` | `modules/status-ui/public` | 4 | 98 |
 | `landing` | `public-landing` | 2 | 186 |
 
@@ -52,7 +52,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `df38941b25a6` | 3x | Sign in with GitHub | public-landing/projects.html:525, public-landing/projects.html:583, public-landing/projects.html:692 |
 | `b2b399fdd6a4` | 3x | in progress | modules/hall-ui/public/goals-page.js:423, modules/hall-ui/public/roadmap.js:108, modules/hall-ui/public/roadmap.js:194 |
 | `cc5106af4b35` | 3x | {…} of {…} criteria satisfied | modules/hall-ui/public/goals-lib.js:30, modules/hall-ui/public/goals-page.js:1016, modules/hall-ui/public/goals.js:45 |
-| `7926e30955d2` | 3x | ← Back to goal | modules/hall-ui/public/goals.html:152, modules/hall-ui/public/goals.html:163, modules/hall-ui/public/goals.html:175 |
+| `7926e30955d2` | 3x | ← Back to goal | modules/hall-ui/public/goals.html:160, modules/hall-ui/public/goals.html:171, modules/hall-ui/public/goals.html:183 |
 | `a8813c34d20d` | 2x | “Host denied” | modules/hall-ui/public/box-panel.js:160, modules/hall-ui/public/box-panel.js:340 |
 | `66cea7cefa4e` | 2x | '; for (const row of bodyRows) { html += ' | modules/hall-ui/public/md-reader.js:188, modules/hall-ui/public/primer.js:278 |
 | `87fa099f5068` | 2x | '; } html += ' | modules/hall-ui/public/md-reader.js:192, modules/hall-ui/public/primer.js:282 |
@@ -64,8 +64,8 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `490b73a0c6e8` | 2x | Approve your Dev Box app — {{worldName}} | modules/hall-ui/public/pair.html:6, modules/hall-ui/public/pair.html:6 |
 | `b71a26e6fb37` | 2x | Archons only | modules/hall-ui/public/sessions.html:28, modules/hall-ui/public/watch.html:31 |
 | `08afb1257b71` | 2x | Back to home | modules/hall-ui/public/primer.html:68, modules/hall-ui/public/primer.js:516 |
-| `563d29ed86d8` | 2x | Bulk actions | modules/hall-ui/public/goals.html:140, modules/hall-ui/public/work.html:118 |
-| `46b33550d52b` | 2x | Bulk claim | modules/hall-ui/public/goals.html:143, modules/hall-ui/public/work.html:121 |
+| `563d29ed86d8` | 2x | Bulk actions | modules/hall-ui/public/goals.html:148, modules/hall-ui/public/work.html:118 |
+| `46b33550d52b` | 2x | Bulk claim | modules/hall-ui/public/goals.html:151, modules/hall-ui/public/work.html:121 |
 | `c10806b50c61` | 2x | Claim task {…} | modules/hall-ui/public/goals-page.js:551, modules/hall-ui/public/work.js:356 |
 | `a832f32b5597` | 2x | Claimed {…} tasks. Open a terminal to start work. | modules/hall-ui/public/goals-page.js:647, modules/hall-ui/public/work.js:754 |
 | `cfe0f4c4fcc2` | 2x | Closing your box… (a few seconds) | modules/hall-ui/public/box-panel.js:265, modules/hall-ui/public/box-panel.js:370 |
@@ -78,7 +78,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `7775fec0fa85` | 2x | Could not load sessions. | modules/hall-ui/public/sessions.js:168, modules/hall-ui/public/sessions.js:207 |
 | … | | _85 more in the JSON registry_ | |
 
-## `builders-hall` — 1095 strings
+## `builders-hall` — 1096 strings
 
 | id | conf | text | where |
 |---|---|---|---|
@@ -184,9 +184,9 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `a78477a24a48` | certain | Builders' Hall | `modules/hall-ui/public/index.html:12` |
 | `a78477a24a48` | certain | Builders' Hall | `modules/hall-ui/public/index.html:38` |
 | `b51edc826cbe` | certain | Built by this project | `modules/hall-ui/public/modules.html:47` |
-| `563d29ed86d8` | certain | Bulk actions | `modules/hall-ui/public/goals.html:140` |
+| `563d29ed86d8` | certain | Bulk actions | `modules/hall-ui/public/goals.html:148` |
 | `563d29ed86d8` | certain | Bulk actions | `modules/hall-ui/public/work.html:118` |
-| `46b33550d52b` | certain | Bulk claim | `modules/hall-ui/public/goals.html:143` |
+| `46b33550d52b` | certain | Bulk claim | `modules/hall-ui/public/goals.html:151` |
 | `46b33550d52b` | certain | Bulk claim | `modules/hall-ui/public/work.html:121` |
 | `6d6d626fc518` | certain | Bulk release | `modules/hall-ui/public/work.html:122` |
 | `181cfa50b127` | certain | By the numbers | `modules/hall-ui/public/profile.html:37` |
@@ -215,6 +215,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `b9c64fe023a9` | certain | Claude Desktop app | `modules/hall-ui/public/builders.js:984` |
 | `9edebf179aa3` | certain | Claude Online | `modules/hall-ui/public/builders.js:984` |
 | `f2c734539bb8` | certain | Claude starts automatically when you open the terminal. | `modules/hall-ui/public/builders.js:984` |
+| `5b3b75ee0596` | certain | Claude's notes on this goal | `modules/hall-ui/public/goals.html:67` |
 | `593dda3178eb` | certain | Click to copy | `modules/hall-ui/public/copy-on-click.js:137` |
 | `87789c3dc97c` | certain | Click to copy {…} | `modules/hall-ui/public/box-panel.js:137` |
 | `e712dd69a5f6` | certain | Clone the project to your machine, or open your hosted dev box (where it is already cloned). The build command | `modules/hall-ui/public/builders.js:1154` |
@@ -381,7 +382,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `c15d1f0ddfb6` | certain | Estimation accuracy | `modules/hall-ui/public/watch.html:133` |
 | `44f928d13699` | certain | Event sounds | `modules/hall-ui/public/settings.html:282` |
 | `8690c6424585` | certain | Every builder in {{worldName}} — rank, {{currency}}, karma, and active claims. | `modules/hall-ui/public/watch.html:61` |
-| `2ae1eb35611e` | certain | Every done-when criterion for this goal, with its full detail. | `modules/hall-ui/public/goals.html:178` |
+| `2ae1eb35611e` | certain | Every done-when criterion for this goal, with its full detail. | `modules/hall-ui/public/goals.html:186` |
 | `1e05b7fe42c8` | certain | Every project you build on across the platform. | `modules/hall-ui/public/my-projects.html:24` |
 | `efafe5ce29ed` | certain | Every task linked to this criterion has shipped — a Metic can confirm it | `modules/hall-ui/public/goals-page.js:1042` |
 | `ddebbccd4b37` | certain | Everyone building across hosted projects, ranked by track record. Descriptive only — cross-project numbers nev | `modules/hall-ui/public/people.html:23` |
@@ -419,7 +420,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `13acaeb4b0bf` | certain | Goal #{…} | `modules/hall-ui/public/task.js:202` |
 | `9e70797b7e97` | certain | Goal id | `modules/hall-ui/public/goals-page.js:1213` |
 | `529460e82058` | certain | Goal invitations to you | `modules/hall-ui/public/goal-inbox.js:113` |
-| `e2a9e33abe60` | certain | Goal tasks | `modules/hall-ui/public/goals.html:104` |
+| `e2a9e33abe60` | certain | Goal tasks | `modules/hall-ui/public/goals.html:112` |
 | `e0bc28d10785` | certain | Goals could not be loaded. Refresh to try again. | `modules/hall-ui/public/goals-page.js:1799` |
 | `4517f2253d0f` | certain | Goals couldn’t be loaded just now. Refresh to try again. | `modules/hall-ui/public/goals.js:112` |
 | `2ed2cb8fbffd` | certain | Goals page | `modules/hall-ui/public/goals.js:94` |
@@ -616,8 +617,8 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `855277dff074` | certain | Open task #{…} | `modules/hall-ui/public/work.js:963` |
 | `e0810149de64` | certain | Open task details | `modules/hall-ui/public/goals-page.js:559` |
 | `e0810149de64` | certain | Open task details | `modules/hall-ui/public/work.js:341` |
-| `99c630009152` | certain | Open tasks whose declared files overlap — sequence these before two builders collide. | `modules/hall-ui/public/goals.html:90` |
-| `99c630009152` | certain | Open tasks whose declared files overlap — sequence these before two builders collide. | `modules/hall-ui/public/goals.html:155` |
+| `99c630009152` | certain | Open tasks whose declared files overlap — sequence these before two builders collide. | `modules/hall-ui/public/goals.html:98` |
+| `99c630009152` | certain | Open tasks whose declared files overlap — sequence these before two builders collide. | `modules/hall-ui/public/goals.html:163` |
 | `29ff278e96a8` | certain | Open the Work Board | `modules/hall-ui/public/task.js:258` |
 | `e706a4836ce8` | certain | Open the pairing page | `modules/hall-ui/public/settings.html:201` |
 | `2cad724171e6` | certain | Open your online environment. | `modules/hall-ui/public/builders.js:984` |
@@ -907,7 +908,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `d3a8b2b5b93f` | certain | Where {{currency}} and karma come from and where they go. | `modules/hall-ui/public/diagrams.html:76` |
 | `8f5954c06daa` | certain | Whether your session cards — what | `modules/hall-ui/public/settings.html:229` |
 | `f9ab9eb870a6` | certain | While you wait: install the Claude desktop app at | `modules/hall-ui/public/builders.js:640` |
-| `045c629729b7` | certain | Who’s on this goal, and what they’ve contributed to it. | `modules/hall-ui/public/goals.html:166` |
+| `045c629729b7` | certain | Who’s on this goal, and what they’ve contributed to it. | `modules/hall-ui/public/goals.html:174` |
 | `6e8a3151ec76` | certain | Why \u2014 the reason the author will act on | `modules/hall-ui/public/government.js:741` |
 | `a3ac116b499a` | certain | Why? | `modules/hall-ui/public/copy-desk.html:99` |
 | `6108c16edece` | certain | Window: last {…} days. | `modules/hall-ui/public/watch.js:628` |
@@ -1160,9 +1161,9 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `7d29dc9e0391` | certain | ← Back to Home | `modules/hall-ui/public/not-ready.html:35` |
 | `9e7f83400f9a` | certain | ← Back to Ideas | `modules/hall-ui/public/idea.html:27` |
 | `a71cd6fa8d57` | certain | ← Back to Tasks | `modules/hall-ui/public/task.html:55` |
-| `7926e30955d2` | certain | ← Back to goal | `modules/hall-ui/public/goals.html:152` |
-| `7926e30955d2` | certain | ← Back to goal | `modules/hall-ui/public/goals.html:163` |
-| `7926e30955d2` | certain | ← Back to goal | `modules/hall-ui/public/goals.html:175` |
+| `7926e30955d2` | certain | ← Back to goal | `modules/hall-ui/public/goals.html:160` |
+| `7926e30955d2` | certain | ← Back to goal | `modules/hall-ui/public/goals.html:171` |
+| `7926e30955d2` | certain | ← Back to goal | `modules/hall-ui/public/goals.html:183` |
 | `1e7cf7b1e64e` | certain | ← Newer | `modules/hall-ui/public/sessions.html:56` |
 | `2180e0ecf138` | certain | → no idea # | `modules/hall-ui/public/task-detail.js:479` |
 | `d5d76f1c0dfe` | certain | ○ reserved | `modules/hall-ui/public/diagrams.html:53` |

package/docs/copy-registry.json

@@ -3,9 +3,9 @@
   "generator": "scripts/gds/copy-inventory.js",
   "goal": 1000074,
   "counts": {
-    "strings": 1379,
-    "unique_ids": 1224,
-    "certain": 1379,
+    "strings": 1380,
+    "unique_ids": 1225,
+    "certain": 1380,
     "heuristic": 0,
     "duplicate_ids": 125
   },
@@ -13,7 +13,7 @@
     "builders-hall": {
       "at": "modules/hall-ui/public",
       "files": 65,
-      "strings": 1095
+      "strings": 1096
     },
     "status-dashboard": {
       "at": "modules/status-ui/public",
@@ -233,9 +233,9 @@
       "text": "← Back to goal",
       "occurrences": 3,
       "at": [
-        "modules/hall-ui/public/goals.html:152",
-        "modules/hall-ui/public/goals.html:163",
-        "modules/hall-ui/public/goals.html:175"
+        "modules/hall-ui/public/goals.html:160",
+        "modules/hall-ui/public/goals.html:171",
+        "modules/hall-ui/public/goals.html:183"
       ]
     },
     {
@@ -354,7 +354,7 @@
       "text": "Bulk actions",
       "occurrences": 2,
       "at": [
-        "modules/hall-ui/public/goals.html:140",
+        "modules/hall-ui/public/goals.html:148",
         "modules/hall-ui/public/work.html:118"
       ]
     },
@@ -364,7 +364,7 @@
       "text": "Bulk claim",
       "occurrences": 2,
       "at": [
-        "modules/hall-ui/public/goals.html:143",
+        "modules/hall-ui/public/goals.html:151",
         "modules/hall-ui/public/work.html:121"
       ]
     },
@@ -834,8 +834,8 @@
       "text": "Open tasks whose declared files overlap — sequence these before two builders collide.",
       "occurrences": 2,
       "at": [
-        "modules/hall-ui/public/goals.html:90",
-        "modules/hall-ui/public/goals.html:155"
+        "modules/hall-ui/public/goals.html:98",
+        "modules/hall-ui/public/goals.html:163"
       ]
     },
     {
@@ -2243,7 +2243,7 @@
       "surface": "builders-hall",
       "text": "Bulk actions",
       "file": "modules/hall-ui/public/goals.html",
-      "line": 140,
+      "line": 148,
       "origin": "html-attr:aria-label",
       "confidence": "certain"
     },
@@ -2261,7 +2261,7 @@
       "surface": "builders-hall",
       "text": "Bulk claim",
       "file": "modules/hall-ui/public/goals.html",
-      "line": 143,
+      "line": 151,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -2517,6 +2517,15 @@
       "origin": "js-markup",
       "confidence": "certain"
     },
+    {
+      "id": "5b3b75ee0596",
+      "surface": "builders-hall",
+      "text": "Claude's notes on this goal",
+      "file": "modules/hall-ui/public/goals.html",
+      "line": 67,
+      "origin": "html-text",
+      "confidence": "certain"
+    },
     {
       "id": "593dda3178eb",
       "surface": "builders-hall",
@@ -4016,7 +4025,7 @@
       "surface": "builders-hall",
       "text": "Every done-when criterion for this goal, with its full detail.",
       "file": "modules/hall-ui/public/goals.html",
-      "line": 178,
+      "line": 186,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -4358,7 +4367,7 @@
       "surface": "builders-hall",
       "text": "Goal tasks",
       "file": "modules/hall-ui/public/goals.html",
-      "line": 104,
+      "line": 112,
       "origin": "html-attr:aria-label",
       "confidence": "certain"
     },
@@ -6131,7 +6140,7 @@
       "surface": "builders-hall",
       "text": "Open tasks whose declared files overlap — sequence these before two builders collide.",
       "file": "modules/hall-ui/public/goals.html",
-      "line": 90,
+      "line": 98,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -6140,7 +6149,7 @@
       "surface": "builders-hall",
       "text": "Open tasks whose declared files overlap — sequence these before two builders collide.",
       "file": "modules/hall-ui/public/goals.html",
-      "line": 155,
+      "line": 163,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -8750,7 +8759,7 @@
       "surface": "builders-hall",
       "text": "Who’s on this goal, and what they’ve contributed to it.",
       "file": "modules/hall-ui/public/goals.html",
-      "line": 166,
+      "line": 174,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -11027,7 +11036,7 @@
       "surface": "builders-hall",
       "text": "← Back to goal",
       "file": "modules/hall-ui/public/goals.html",
-      "line": 152,
+      "line": 160,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -11036,7 +11045,7 @@
       "surface": "builders-hall",
       "text": "← Back to goal",
       "file": "modules/hall-ui/public/goals.html",
-      "line": 163,
+      "line": 171,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -11045,7 +11054,7 @@
       "surface": "builders-hall",
       "text": "← Back to goal",
       "file": "modules/hall-ui/public/goals.html",
-      "line": 175,
+      "line": 183,
       "origin": "html-text",
       "confidence": "certain"
     },

package/migrations/core_224_goal_subtitle.sql

@@ -0,0 +1,35 @@
+-- core_224_goal_subtitle.sql — a goal's explanation stops living inside its
+-- name (task 1003224, goal 1000072, criterion legibility-goal-names-and-claude-text).
+--
+-- THE PROBLEM THIS FIXES IS IN THE DATA. 29 of 38 open goals carry their own
+-- explanation appended to `title` after a dash — "Findability — the grep path",
+-- "Idea routing — work that has a home lands in it". The goal board is the
+-- primary work surface, so every list, card, pill and breadcrumb renders the
+-- explanation at the same weight as the name, and nothing can show one without
+-- the other. A name and a description are two fields; they were one.
+--
+-- ADDITIVE ONLY, and nullable with no default on purpose. Every existing row
+-- keeps its title byte-for-byte: this migration creates somewhere for the
+-- explanation to GO, and the separate renaming pass (task 1003225) moves it,
+-- goal by goal, with judgment — three goals all head "Agents module" and would
+-- become indistinguishable under a mechanical split. A DEFAULT '' would make
+-- "never set" and "deliberately blank" the same state, and the UI needs to tell
+-- them apart to know whether to render the affordance at all.
+--
+-- WHY THIS IS A CORE MIGRATION AND NOT A MODULE ONE. `goals` was created by
+-- repo-root migrations/160_goal_tier.sql and is a CORE table — the lifecycle
+-- module owns the queries against it, not the table itself (modules/lifecycle/
+-- CLAUDE.md: "None owned"). Fitness check 11 enforces exactly that: a module
+-- migration may only touch its own `<key>_`-prefixed tables, so an
+-- `ALTER TABLE goals` from modules/lifecycle/migrations/ is refused however
+-- lifecycle-ish it feels (ADR 0083 §Decision #5). The first draft of this file
+-- lived there and the check caught it.
+
+ALTER TABLE goals
+  ADD COLUMN IF NOT EXISTS subtitle text;
+
+COMMENT ON COLUMN goals.subtitle IS
+  'The goal''s explanation, held apart from its name. Rendered subordinate to title and on demand (task 1003224). NULL = never set, which is distinct from an empty string meaning deliberately blank.';
+
+INSERT INTO schema_migrations (version) VALUES ('core_224_goal_subtitle')
+ON CONFLICT (version) DO NOTHING;

package/modules/hall-ui/public/goals-lib.js

@@ -90,10 +90,36 @@
   const msOf = (iso) => { const t = iso ? Date.parse(iso) : NaN; return Number.isNaN(t) ? 0 : t; };
   const taskActivityMs = (t) => Math.max(msOf(t.updated_at), msOf(t.shipped_at));
 
+  // task 1003224 — on a LIST, a goal's subtitle is a TOOLTIP, never a second
+  // visible line. The board's job there is to let someone find a goal by name;
+  // rendering every explanation inline is what led 29 of 38 titles to carry
+  // their own after a dash. Escaped because it lands inside an attribute.
+  function subtitleAttr(g) {
+    const t = (g && g.subtitle ? String(g.subtitle) : '').trim();
+    return t ? ` title="${window.OTB.escapeHtml(t)}"` : '';
+  }
+
+  // task 1003224 — the three-layer prose rules for a goal's detail head, kept
+  // here rather than in goals-page.js because that file is 500 lines over the
+  // excess-lines ratchet and this one exists for exactly that reason. It WRITES
+  // to elements the caller already looked up; it never reads the DOM or touches
+  // page state, so the file's purity contract holds.
+  //   name      -> the h2, rendered by the caller
+  //   subtitle  -> subordinate, and ABSENT when unset (NULL differs from blank)
+  //   Claude's prose -> collapsed, and absent when there is nothing to reveal
+  function applyGoalProse(els, g) {
+    const subtitle = (g.subtitle || '').trim();
+    const description = g.description || '';
+    if (els.sub) { els.sub.textContent = subtitle; els.sub.hidden = !subtitle; }
+    if (els.desc) els.desc.textContent = description;
+    if (els.claude) { els.claude.hidden = !description.trim(); els.claude.open = false; }
+  }
+
   window.OTBGoalsLib = {
     DONE_STATUSES, CUT_STATUSES, isTodoStatus,
     ring, memberName,
     readinessOf, EMPTY_ROLLUP, goalRollLine,
     matchesTodoSearch, msOf, taskActivityMs,
+    subtitleAttr, applyGoalProse,
   };
 })();

package/modules/hall-ui/public/goals-page.js

@@ -40,7 +40,7 @@
   // so every call site below reads exactly as before.
   const { DONE_STATUSES, CUT_STATUSES, isTodoStatus, ring, memberName,
     readinessOf, EMPTY_ROLLUP, goalRollLine, matchesTodoSearch,
-    msOf, taskActivityMs } = window.OTBGoalsLib;
+    msOf, taskActivityMs, subtitleAttr, applyGoalProse } = window.OTBGoalsLib;
   const API = '/api/bongos';
   // Generated typed client (task 1996 / R09), served as window.BongosClient by
   // /lib/bongos-client.js. baseUrl:'' passes the full same-origin paths this file
@@ -359,7 +359,7 @@
     return `<div class="gr-row" data-goal-id="${escapeHtml(String(g.id))}">
       <div class="gr-ringcol">${ringHtml}</div>
       <div class="gr-main">
-        <a class="gr-title" href="${goalHref}">${escapeHtml(g.title || 'Untitled goal')}</a>
+        <a class="gr-title" href="${goalHref}"${subtitleAttr(g)}>${escapeHtml(g.title || 'Untitled goal')}</a>
         <div class="gr-meta">${metaHtml}</div>
         ${achieved ? '' : goalRollLine(r)}
       </div>
@@ -377,7 +377,7 @@
     return `<div class="goal-card" data-goal-id="${escapeHtml(String(g.id))}">
       <div class="goal-card__head">
         <div class="gr-ringcol">${ringHtml}</div>
-        <a class="goal-card__title gr-title" href="${goalHref}">${escapeHtml(g.title || 'Untitled goal')}</a>
+        <a class="goal-card__title gr-title" href="${goalHref}"${subtitleAttr(g)}>${escapeHtml(g.title || 'Untitled goal')}</a>
       </div>
       <div class="goal-card__meta">${metaHtml}</div>
       ${achieved ? '' : goalRollLine(r)}
@@ -1211,8 +1211,8 @@
     // task 2163 — surface the numeric goal id alongside the title (escaped title,
     // so switching from textContent to innerHTML stays injection-safe).
     if (h) h.innerHTML = `<span class="goal-detail__id" title="Goal id">#${escapeHtml(String(g.id))}</span> ${escapeHtml(g.title || 'Untitled goal')}`;
-    const desc = $('#goal-detail-desc');
-    if (desc) desc.textContent = g.description || '';
+    // task 1003224 — name / subtitle / Claude's prose at three weights; the rules live in goals-lib.
+    applyGoalProse({ sub: $('#goal-detail-subtitle'), desc: $('#goal-detail-desc'), claude: $('#goal-detail-claude') }, g);
     const meta = $('#goal-detail-meta');
     if (meta) {
       const scope = Array.isArray(g.scope_modules) && g.scope_modules.length

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

@@ -259,3 +259,71 @@ a.goal-card:hover { text-decoration: none; border-color: var(--border-strong); b
   .gr-next { align-items: flex-start; }
   .gr-next__t { max-width: 100%; }
 }
+
+/* ── task 1003224: name, subtitle, and Claude's text at three weights ────────
+   The goal board is the primary work surface, so what it shows by DEFAULT is
+   the decision this makes. The name reads first; the explanation is visibly
+   subordinate; the bulk prose Claude writes is behind a disclosure and absent
+   from the default reading path until someone asks for it.
+   Tokens only from style.css's real set (--ink-soft, --ink-faint, --rule,
+   --surface, --radius-sm) — inventing one renders as nothing, which is how
+   copy-desk.css ended up with invisible fallbacks (idea 1000821). */
+.goal-detail__subtitle {
+  color: var(--ink-soft);
+  font-size: 13px;
+  font-weight: 400;
+  line-height: 1.45;
+  max-width: 72ch;
+  margin: 3px 0 0;
+}
+
+/* The disclosure is deliberately quiet: a row someone can ignore. */
+.claude-text {
+  margin: 10px 0 0;
+  max-width: 72ch;
+  border-top: 1px solid var(--rule);
+  padding-top: 8px;
+}
+
+.claude-text__toggle {
+  color: var(--ink-faint);
+  font-size: 12px;
+  cursor: pointer;
+  list-style: none;
+  user-select: none;
+  display: inline-flex;
+  align-items: center;
+  gap: 6px;
+}
+
+/* Own marker, so the affordance reads the same in every browser. */
+.claude-text__toggle::-webkit-details-marker { display: none; }
+.claude-text__toggle::before {
+  content: '▸';
+  font-size: 10px;
+  transition: transform 120ms ease;
+}
+.claude-text[open] > .claude-text__toggle::before { transform: rotate(90deg); }
+.claude-text__toggle:hover { color: var(--ink-soft); }
+.claude-text__toggle:focus-visible {
+  outline: 2px solid var(--accent);
+  outline-offset: 2px;
+  border-radius: var(--radius-sm);
+}
+
+/* Set apart once opened, so it reads as a machine's notes rather than as the
+   goal's own definition. */
+.claude-text__body {
+  margin: 8px 0 0;
+  padding: 8px 10px;
+  background: var(--surface);
+  border-radius: var(--radius-sm);
+  color: var(--ink-soft);
+  font-size: 13px;
+  line-height: 1.5;
+  white-space: pre-wrap;
+}
+
+@media (prefers-reduced-motion: reduce) {
+  .claude-text__toggle::before { transition: none; }
+}

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

@@ -58,7 +58,15 @@
             <div class="goal-detail__head">
               <div style="flex:1;min-width:260px">
                 <h2 class="goal-detail__title" id="goal-detail-h">…</h2>
-                <p class="goal-detail__desc" id="goal-detail-desc"></p>
+                <!-- task 1003224: the goal's explanation, subordinate to its name. -->
+                <p class="goal-detail__subtitle" id="goal-detail-subtitle" hidden></p>
+                <!-- The bulk prose Claude writes. Collapsed by DEFAULT and marked as
+                     Claude's text: the owner reads the name and the punched-in points,
+                     and opens this only when they want the long form. -->
+                <details class="claude-text" id="goal-detail-claude" hidden>
+                  <summary class="claude-text__toggle">Claude's notes on this goal</summary>
+                  <p class="claude-text__body goal-detail__desc" id="goal-detail-desc"></p>
+                </details>
                 <p class="goal-card__meta" id="goal-detail-meta" style="margin-top:10px"></p>
               </div>
               <div class="goal-detail__actions" id="goal-detail-actions"></div>

package/modules/lifecycle/db-goals.js

@@ -32,7 +32,7 @@ const { deriveRequiredRank, highestRank } = require('./db-rank-authz.js');
 // Shared column list so every goal read/return shapes a row identically.
 
 const GOAL_COLS =
-  'id, version_id, title, description, scope_modules, status, visibility, created_by, succeeded_by_goal_id, sort_order, created_at, updated_at';
+  'id, version_id, title, subtitle, description, scope_modules, status, visibility, created_by, succeeded_by_goal_id, sort_order, created_at, updated_at';
 
 // Shared column list for a goal_membership_requests row (task 1951 / ADR 0112) —
 // the invite / join-request queue behind private goals. Every helper below shapes
@@ -48,7 +48,7 @@ const MEMBERSHIP_REQUEST_COLS =
 // goals) inserts no member — those goals have no owner; Archons act as one.
 
 async function createGoal(
-  { versionId, title, description = null, scopeModules = [], status = 'open', createdBy = null },
+  { versionId, title, subtitle = null, description = null, scopeModules = [], status = 'open', createdBy = null },
   deps = {}
 ) {
   const activePool = (deps.pool && process.env.NODE_ENV === 'test') ? deps.pool : pool;
@@ -56,15 +56,15 @@ async function createGoal(
     `WITH next_order AS (
        SELECT COALESCE(MAX(sort_order) + 1, 0) AS n FROM goals WHERE version_id = $1
      ), g AS (
-       INSERT INTO goals (version_id, title, description, scope_modules, status, created_by, sort_order)
-            SELECT $1, $2, $3, $4, $5, $6, next_order.n FROM next_order
+       INSERT INTO goals (version_id, title, subtitle, description, scope_modules, status, created_by, sort_order)
+            SELECT $1, $2, $7, $3, $4, $5, $6, next_order.n FROM next_order
        RETURNING ${GOAL_COLS}
      ), first_lead AS (
        INSERT INTO goal_members (goal_id, builder_id, membership_kind)
        SELECT g.id, $6::bigint, 'lead' FROM g WHERE $6::bigint IS NOT NULL
      )
      SELECT * FROM g`,
-    [versionId, title, description, scopeModules, status, createdBy]
+    [versionId, title, description, scopeModules, status, createdBy, subtitle]
   );
   return rows[0];
 }

package/modules/lifecycle/goal-edits.js

@@ -91,24 +91,26 @@ function proseDelta(field, from, to) {
 async function updateGoalProse({ goalId, patch = {} }, deps = {}) {
   const activePool = (deps.pool && process.env.NODE_ENV === 'test') ? deps.pool : pool;
   const { rows: before } = await activePool.query(
-    `SELECT id, title, description FROM goals WHERE id = $1`,
+    `SELECT id, title, subtitle, description FROM goals WHERE id = $1`,
     [goalId]
   );
   if (before.length === 0) return null;
   const prev = before[0];
   const nextTitle = Object.prototype.hasOwnProperty.call(patch, 'title') ? patch.title : prev.title;
+  const nextSubtitle = Object.prototype.hasOwnProperty.call(patch, 'subtitle') ? patch.subtitle : prev.subtitle;
   const nextDesc = Object.prototype.hasOwnProperty.call(patch, 'description') ? patch.description : prev.description;
   const deltas = [
     proseDelta('title', prev.title, nextTitle),
+    proseDelta('subtitle', prev.subtitle, nextSubtitle),
     proseDelta('description', prev.description, nextDesc),
   ].filter(Boolean);
   if (deltas.length === 0) return { goal: prev, deltas: [], unchanged: true };
   const { rows } = await activePool.query(
     `UPDATE goals
-        SET title = $2, description = $3, updated_at = now()
+        SET title = $2, description = $3, subtitle = $4, updated_at = now()
       WHERE id = $1
-     RETURNING id, version_id, title, description, status, visibility, created_by, updated_at`,
-    [goalId, nextTitle, nextDesc]
+     RETURNING id, version_id, title, subtitle, description, status, visibility, created_by, updated_at`,
+    [goalId, nextTitle, nextDesc, nextSubtitle]
   );
   return { goal: rows[0], deltas };
 }
@@ -219,13 +221,17 @@ function goalProseHandler({ db, authorizeGoalManagement, validateOrRespond, pars
     if (id === null) return;
     if (validateOrRespond(req, res, {
       title: { type: 'string', minLength: 1, maxLength: LIMITS.TITLE },
+      // No minLength: '' is a legal subtitle, meaning deliberately blank. The
+      // column's NULL (never set) is what the UI tells apart from that.
+      subtitle: { type: 'string', maxLength: LIMITS.TITLE },
       description: { type: 'string', maxLength: LIMITS.DESCRIPTION },
     })) return;
     const patch = {};
     if (Object.prototype.hasOwnProperty.call(req.body || {}, 'title')) patch.title = req.body.title;
+    if (Object.prototype.hasOwnProperty.call(req.body || {}, 'subtitle')) patch.subtitle = req.body.subtitle;
     if (Object.prototype.hasOwnProperty.call(req.body || {}, 'description')) patch.description = req.body.description;
     if (Object.keys(patch).length === 0) {
-      return res.fail('nothing_to_update', 400, { editable: ['title', 'description'] });
+      return res.fail('nothing_to_update', 400, { editable: ['title', 'subtitle', 'description'] });
     }
     try {
       const goal = await db.getGoal(id);

package/modules/lifecycle/routes/goals.js

@@ -438,7 +438,7 @@ module.exports = function buildGoalsRouter() {
     if (validateOrRespond(req, res, {
       version_id: { required: true, type: 'string', minLength: 1, maxLength: 64 },
       title: { required: true, type: 'string', minLength: 1, maxLength: LIMITS.TITLE },
-      description: { type: 'string', maxLength: LIMITS.DESCRIPTION },
+      subtitle: { type: 'string', maxLength: LIMITS.TITLE }, description: { type: 'string', maxLength: LIMITS.DESCRIPTION },
       scope_modules: { type: 'array', maxItems: 64, itemsType: 'string' },
       succeeds_goal_id: { type: 'integer', min: 1 },
     })) return;
@@ -474,7 +474,7 @@ module.exports = function buildGoalsRouter() {
       const goal = await db.createGoal({
         versionId: req.body.version_id,
         title: req.body.title,
-        description: (req.body.description || '').trim() || null,
+        subtitle: (req.body.subtitle || '').trim() || null, description: (req.body.description || '').trim() || null,
         scopeModules,
         createdBy: req.builder.id,
       });
@@ -640,7 +640,7 @@ module.exports = function buildGoalsRouter() {
     }
   });
 
-  // PATCH /goals/:id — correct a goal's own title/description (task 1003102).
+  // PATCH /goals/:id — correct a goal's own title/subtitle/description (tasks 1003102, 1003224).
   // Handler + rationale + rank note: ../goal-edits.js. Pinned in route-rank-check.
   router.patch('/goals/:id', auth.requireBuilder, goalEdits.goalProseHandler({ db, authorizeGoalManagement, validateOrRespond, parseId, LIMITS }));
   // PATCH /goals/:id/visibility — make a goal public|private (task 1952 / ADR 0112).

package/package-lock.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.322",
+  "version": "1.19.323",
   "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/src/module-api.js

@@ -1071,8 +1071,10 @@ const { validateOrRespond, LIMITS, parseId, asyncHandler, corsPublicGet, parsePa
 //         landed since 1.19.320 with no explicit bump. run 32679324013. (task 1002620)
 //   1.19.322 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
 //         landed since 1.19.321 with no explicit bump. run 32679604360. (task 1002620)
+//   1.19.323 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+//         landed since 1.19.322 with no explicit bump. run 32680138776. (task 1002620)
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.322'; // CI auto-patch carrier (ADR 0161); see changelog above
+const CORE_VERSION = '1.19.323'; // CI auto-patch carrier (ADR 0161); see changelog above
 
 // A namespaced logger so a module's log lines are attributable + consistent.
 // Usage: const log = api.logger('dev-box'); log.info('mounted');

package/tests/goal_subtitle.mjs

@@ -0,0 +1,237 @@
+// tests/goal_subtitle.mjs — a goal's explanation stops living inside its name
+// (task 1003224, goal 1000072, criterion legibility-goal-names-and-claude-text).
+//
+// Owner directive, three parts: the TITLE is a name; the SUBTITLE is a separate,
+// subordinate, on-demand field; and the bulk prose Claude writes into the
+// description is presented as Claude's text, absent from the default reading
+// path until someone asks for it.
+//
+// These tests pin the parts a future edit could quietly undo: that subtitle is
+// carried by the same prose-edit path as title (so it inherits the audit trail
+// rather than getting a private one), that the disclosure is CLOSED by default,
+// and that task creation did NOT gain a subtitle — the identical validator pair
+// appears in POST /goals and POST /goals/:id/tasks, and only the first should
+// have changed.
+import { strict as assert } from 'node:assert';
+import { test } from 'node:test';
+import { createRequire } from 'node:module';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const require = createRequire(import.meta.url);
+const ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
+const goalEdits = require(path.join(ROOT, 'modules', 'lifecycle', 'goal-edits.js'));
+const read = (...p) => fs.readFileSync(path.join(ROOT, ...p), 'utf8');
+
+const fakeReqRes = ({ body }) => {
+  const res = { code: null, statusCode: null, payload: null,
+    fail(code, status, extra) { this.code = code; this.statusCode = typeof status === 'number' ? status : status?.status ?? 400; this.payload = { error: code, ...(extra || {}) }; return this; },
+    status(c) { this.statusCode = c; return this; },
+    json(p) { this.payload = p; return this; } };
+  return { req: { params: { id: '7' }, body, builder: { id: 1, rank: 'archon' } }, res };
+};
+const deps = {
+  db: { getGoal: async () => ({ id: 7, status: 'open', created_by: 1 }), getGoalMember: async () => null },
+  authorizeGoalManagement: () => ({ ok: true }),
+  validateOrRespond: () => false,
+  parseId: () => 7,
+  LIMITS: { TITLE: 200, DESCRIPTION: 50000 },
+};
+
+// ---- the field travels the existing prose path -----------------------------
+test('a subtitle-only PATCH is accepted and recorded', () => {
+  // Not a private write path: subtitle rides title's handler so the prose-edit
+  // audit trail (deltas + record) covers it identically.
+  let patched = null; const recorded = [];
+  const handler = goalEdits.goalProseHandler({
+    ...deps,
+    update: async ({ patch }) => { patched = patch; return { goal: { id: 7, ...patch }, deltas: [goalEdits.proseDelta('subtitle', null, patch.subtitle)] }; },
+    record: async (...a) => { recorded.push(a); },
+  });
+  const { req, res } = fakeReqRes({ body: { subtitle: 'the grep path' } });
+  return handler(req, res).then(() => {
+    assert.equal(res.code, null, `must not fail — got ${res.code}`);
+    assert.deepEqual(patched, { subtitle: 'the grep path' }, 'only the named field is patched');
+    assert.equal(res.payload.deltas[0].field, 'subtitle');
+    assert.equal(recorded.length, 1, 'the edit is audit-recorded like any other prose edit');
+  });
+});
+
+test('an EMPTY subtitle is a legal value, not a no-op', () => {
+  // '' means deliberately blank; NULL means never set. The column has no
+  // default precisely so the UI can tell them apart, so '' must reach the patch.
+  let patched = null;
+  const handler = goalEdits.goalProseHandler({
+    ...deps,
+    update: async ({ patch }) => { patched = patch; return { goal: { id: 7 }, deltas: [] }; },
+    record: async () => {},
+  });
+  const { req, res } = fakeReqRes({ body: { subtitle: '' } });
+  return handler(req, res).then(() => {
+    assert.equal(res.code, null);
+    assert.ok(Object.prototype.hasOwnProperty.call(patched, 'subtitle'), 'empty string must still be a patch');
+    assert.equal(patched.subtitle, '');
+  });
+});
+
+test('nothing_to_update names subtitle as editable', () => {
+  const handler = goalEdits.goalProseHandler(deps);
+  const { req, res } = fakeReqRes({ body: { nope: 1 } });
+  return handler(req, res).then(() => {
+    assert.equal(res.code, 'nothing_to_update');
+    assert.deepEqual(res.payload.editable, ['title', 'subtitle', 'description']);
+  });
+});
+
+test('title and description still work — subtitle is additive', () => {
+  let patched = null;
+  const handler = goalEdits.goalProseHandler({
+    ...deps,
+    update: async ({ patch }) => { patched = patch; return { goal: { id: 7, ...patch }, deltas: [] }; },
+    record: async () => {},
+  });
+  const { req, res } = fakeReqRes({ body: { title: 'Findability', description: 'long prose' } });
+  return handler(req, res).then(() => {
+    assert.deepEqual(patched, { title: 'Findability', description: 'long prose' });
+    assert.ok(!Object.prototype.hasOwnProperty.call(patched, 'subtitle'), 'an unsent field is not patched to null');
+  });
+});
+
+// ---- the data layer --------------------------------------------------------
+test('the migration is additive, core-numbered, and sets no default', () => {
+  const raw = read('migrations', 'core_224_goal_subtitle.sql');
+  assert.match(raw, /ADD COLUMN IF NOT EXISTS subtitle text/);
+  // Assert over the STATEMENTS, not the prose. The first version of this test
+  // matched /DEFAULT ''/ anywhere in the file and failed on the migration's own
+  // comment explaining why it does NOT set a default — the same over-broad-source
+  // -assertion mistake this session has made repeatedly (learning 1000196).
+  const ddl = raw.split('\n').filter((l) => !/^\s*--/.test(l)).join('\n');
+  assert.ok(!/\bDROP\b/i.test(ddl), 'no destructive DDL');
+  assert.ok(!/SET NOT NULL/i.test(ddl), 'stays nullable');
+  assert.ok(!/ADD COLUMN[^;]*DEFAULT/i.test(ddl), 'the added column takes no default');
+  assert.match(ddl, /schema_migrations/, 'records itself');
+});
+
+test('subtitle is in GOAL_COLS, so every read returns it', () => {
+  // One column list drives getGoal, the list query and createGoal's RETURNING —
+  // adding it anywhere else would be a second source that can drift.
+  const src = read('modules', 'lifecycle', 'db-goals.js');
+  const cols = /const GOAL_COLS\s*=\s*\n?\s*'([^']+)'/.exec(src);
+  assert.ok(cols, 'GOAL_COLS is still a single string literal');
+  assert.ok(cols[1].split(',').map((c) => c.trim()).includes('subtitle'), 'subtitle is selected');
+});
+
+test('POST /goals gained subtitle and POST /goals/:id/tasks did NOT', () => {
+  // The identical `title` + `description` validator pair appears in both routes.
+  // A blind replace-all would have given task creation a subtitle; this pins the
+  // boundary. (The first attempt at this edit tripped on exactly that ambiguity.)
+  const src = read('modules', 'lifecycle', 'routes', 'goals.js');
+  const goalRoute = src.slice(src.indexOf("router.post('/goals',"), src.indexOf("router.get('/goals',"));
+  assert.match(goalRoute, /subtitle: \{ type: 'string'/, 'the goal route takes a subtitle');
+  const taskRoute = src.slice(src.indexOf("router.post('/goals/:id/tasks',"));
+  const taskValidator = taskRoute.slice(0, taskRoute.indexOf('})) return;'));
+  assert.ok(!/subtitle/.test(taskValidator), 'task creation must NOT accept a subtitle');
+});
+
+// ---- the default reading path ---------------------------------------------
+test("Claude's prose is a CLOSED disclosure, not visible body text", () => {
+  const html = read('modules', 'hall-ui', 'public', 'goals.html');
+  assert.match(html, /<details class="claude-text"[^>]*hidden/, 'a <details>, hidden until it has content');
+  assert.ok(!/<p class="goal-detail__desc" id="goal-detail-desc"><\/p>/.test(html),
+    'the description is no longer a bare always-visible paragraph');
+  // the description element must now live INSIDE the disclosure
+  const det = html.slice(html.indexOf('<details class="claude-text"'));
+  const inner = det.slice(0, det.indexOf('</details>'));
+  assert.match(inner, /id="goal-detail-desc"/, 'the description renders inside the disclosure');
+});
+
+// applyGoalProse is pure enough to RUN, so these two assert behaviour rather
+// than source text — a stronger pin, and the reason the rules were moved out of
+// goals-page.js (which is 500 lines over the excess-lines ratchet) in the first
+// place. Fake elements stand in for the DOM.
+const el = () => ({ textContent: null, hidden: null, open: null });
+function applyProse(goal) {
+  const lib = read('modules', 'hall-ui', 'public', 'goals-lib.js');
+  const els = { sub: el(), desc: el(), claude: el() };
+  const fn = new Function('window', `${lib}; return window.OTBGoalsLib.applyGoalProse;`)({ OTB: { escapeHtml: (x) => x } });
+  fn(els, goal);
+  return els;
+}
+
+test('the disclosure is CLOSED on render, never opened for the user', () => {
+  const els = applyProse({ description: 'Claude wrote a lot here', subtitle: 'sub' });
+  assert.equal(els.claude.open, false, 'closed');
+  assert.equal(els.claude.hidden, false, 'present, because there is prose to reveal');
+  assert.equal(els.desc.textContent, 'Claude wrote a lot here');
+});
+
+test('a goal with no description shows no empty disclosure', () => {
+  const els = applyProse({ description: '   ', subtitle: 'sub' });
+  assert.equal(els.claude.hidden, true, 'an empty disclosure invites a dead click');
+});
+
+test('an unset subtitle is hidden; a set one is shown trimmed', () => {
+  assert.equal(applyProse({}).sub.hidden, true, 'NULL subtitle renders nothing');
+  const els = applyProse({ subtitle: '  the grep path  ' });
+  assert.equal(els.sub.hidden, false);
+  assert.equal(els.sub.textContent, 'the grep path');
+});
+
+test('on a LIST the subtitle is a tooltip, never a second visible line', () => {
+  const js = read('modules', 'hall-ui', 'public', 'goals-page.js');
+  // The helper lives in goals-lib.js — the file that exists because
+  // goals-page.js is over the excess-lines ratchet (task 1003135).
+  const lib = read('modules', 'hall-ui', 'public', 'goals-lib.js');
+  assert.match(lib, /function subtitleAttr/);
+  assert.match(lib, /escapeHtml\(t\)/, 'escaped, since it lands in an attribute');
+  for (const anchor of ['class="gr-title"', 'class="goal-card__title gr-title"']) {
+    const i = js.indexOf(anchor);
+    assert.ok(i > 0, `${anchor} still rendered`);
+    assert.match(js.slice(i, i + 160), /subtitleAttr\(g\)/, `${anchor} carries the tooltip`);
+  }
+});
+
+test('every CSS token the new styles use actually exists', () => {
+  // copy-desk.css shipped with --line and --panel, which style.css never
+  // defines, so its fallbacks rendered as nothing (idea 1000821).
+  const css = read('modules', 'hall-ui', 'public', 'goals.css');
+  const style = read('modules', 'hall-ui', 'public', 'style.css');
+  const block = css.slice(css.indexOf('task 1003224'));
+  const used = new Set([...block.matchAll(/var\((--[a-z-]+)\)/g)].map((m) => m[1]));
+  assert.ok(used.size >= 4, 'the new block does use tokens');
+  for (const t of used) {
+    assert.ok(new RegExp(`^\\s*${t}:`, 'm').test(style), `${t} must be defined in style.css`);
+  }
+});
+
+test('the subtitle style is subordinate to the title style', () => {
+  const css = read('modules', 'hall-ui', 'public', 'goals.css');
+  const size = (sel) => {
+    const i = css.indexOf(sel);
+    assert.ok(i > 0, `${sel} exists`);
+    const m = /font-size:\s*([\d.]+)px/.exec(css.slice(i, i + 300));
+    return m ? Number(m[1]) : null;
+  };
+  const title = size('.goal-detail__title');
+  const sub = size('.goal-detail__subtitle');
+  assert.ok(title && sub, 'both declare a size');
+  assert.ok(sub < title, `subtitle (${sub}px) must read smaller than the name (${title}px)`);
+});
+
+test("the PATCH route's own comment names every field it accepts", () => {
+  // The API reference is GENERATED from this comment, so the generator
+  // faithfully republishes a stale one: after subtitle was added the docs still
+  // advertised 'title/description', which the grader caught as a blocker. This
+  // pins comment and validator together so the next added field cannot drift.
+  const routes = read('modules', 'lifecycle', 'routes', 'goals.js');
+  const i = routes.indexOf("router.patch('/goals/:id',");
+  assert.ok(i > 0, 'the PATCH route still exists');
+  const comment = routes.slice(routes.lastIndexOf('// PATCH /goals/:id', i), i);
+  const edits = read('modules', 'lifecycle', 'goal-edits.js');
+  const validator = edits.slice(edits.indexOf('function goalProseHandler'));
+  for (const field of ['title', 'subtitle', 'description']) {
+    assert.match(validator, new RegExp(`${field}: \\{ `), `${field} is validated`);
+    assert.ok(comment.includes(field), `the route comment must name ${field} — the API reference is generated from it`);
+  }
+});

package/tests/goal_tier.mjs

@@ -65,8 +65,10 @@ await test('createGoal: INSERT binds all six fields in order and returns the row
   );
   assert.deepEqual(out, GOAL_ROW, 'returns the inserted row');
   const ins = find(pool.queries, /INSERT INTO goals/);
-  assert.deepEqual(ins.params, ['BONGOS-V1', 'Goal hierarchy', 'desc', ['gds-core'], 'open', 42],
-    'version_id, title, description, scope_modules, status, created_by — in that order');
+  // task 1003224 appended `subtitle` as $7 rather than inserting it next to
+  // title, so $1–$6 keep the positions this test has always pinned.
+  assert.deepEqual(ins.params, ['BONGOS-V1', 'Goal hierarchy', 'desc', ['gds-core'], 'open', 42, null],
+    'version_id, title, description, scope_modules, status, created_by, then subtitle — in that order');
   assert.match(ins.sql, /RETURNING/, 'returns the row, not just inserts');
 });
 
@@ -87,8 +89,8 @@ await test('createGoal: defaults — null description/createdBy, empty scope, op
   const pool = fakePool((s) => (/INSERT INTO goals/.test(s) ? { rows: [GOAL_ROW] } : null));
   await createGoal({ versionId: 'V1', title: 'general' }, { pool });
   const ins = find(pool.queries, /INSERT INTO goals/);
-  assert.deepEqual(ins.params, ['V1', 'general', null, [], 'open', null],
-    'unspecified fields fall back to null / [] / open');
+  assert.deepEqual(ins.params, ['V1', 'general', null, [], 'open', null, null],
+    'unspecified fields fall back to null / [] / open — subtitle ($7) defaults to null too');
 });
 
 // =========================================================================