@bongos/core 1.19.289 → 1.19.290

package/.bongos-core.json

@@ -2,22 +2,22 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/gds/package-core.js",
-  "core_version": "1.19.289",
-  "core_contract": "1.19.289",
-  "source_commit": "7fcb844542a0043f5c5beff9a14978d08dba62a9",
+  "core_version": "1.19.290",
+  "core_contract": "1.19.290",
+  "source_commit": "a983fae1fa7615fda22b12d59ff393255fd81a19",
   "source_ref": "HEAD",
-  "built_at": "2026-08-22T03:47:31.708Z",
+  "built_at": "2026-08-22T04:46:05.299Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 320,
     "agent_docs_stubbed": 22,
-    "functional_verbatim": 1388,
+    "functional_verbatim": 1389,
     "rules": 3,
     "gate_literals": 3,
     "gate": "passed"
   },
-  "file_count": 1730,
-  "tree_sha256": "08c303ce54af46b2a131d468064b1ec877f3120d3d6df123a8a5e4c88be18af6",
+  "file_count": 1731,
+  "tree_sha256": "f3100f6df33ff1d5f9e4da0d536a0cc792b0c7ad9e3c945c87324554b34105db",
   "files": [
     {
       "path": ".claude/skills/blocker-review/SKILL.md",
@@ -172,7 +172,7 @@
     {
       "path": ".claude/skills/idea-triage/SKILL.md",
       "mode": "0000644",
-      "sha256": "fa61787e4ab5aafe6703fa94dc1dedb65c0554d1cc0362f6404801dd2c33fdb2"
+      "sha256": "c74c93659c4576b32d829ebb2774feb8771125b31b677e070841f4cc25ed0232"
     },
     {
       "path": ".claude/skills/ideate/SKILL.md",
@@ -1437,12 +1437,12 @@
     {
       "path": "docs/copy-inventory.md",
       "mode": "0000644",
-      "sha256": "0738a5765a16d7e63dd103d75195f972f2a6a863ae22271f5fdb74e7ac171467"
+      "sha256": "8cb5bb088aae803e6889571712cc4439fc870c88e104b1bb81dbd5fd1f9b9bd3"
     },
     {
       "path": "docs/copy-registry.json",
       "mode": "0000644",
-      "sha256": "5271f093cccfded7a3461ae0a83781f6fc41b5d7f78b0084165e463ed2b09b7d"
+      "sha256": "18599cab2b3de43e33513c20cfaa241d3fe0545b861a27942d7f63d11f646750"
     },
     {
       "path": "docs/design-contract.md",
@@ -3632,7 +3632,7 @@
     {
       "path": "modules/hall-ui/public/task-detail.js",
       "mode": "0000644",
-      "sha256": "fc9224e79a1003434f7cb35925b3264b26bb4aea1d32c1b24723de4918530bef"
+      "sha256": "e2c0702ec2eaa2c2d63de4d181f6118295a28e743f0df52722e5e717c3e96b72"
     },
     {
       "path": "modules/hall-ui/public/task-lib.js",
@@ -4522,12 +4522,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "4a910e78786de976213abedacfe08553775d9651b46579e1388941d416be82db"
+      "sha256": "e96dcf6ef882c61be154cfcb5d902942a1cad182173d11b63c824fdfb87f9879"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "9919d125bfcfaccf90bc06727eeec3f410882a6e1c83481e63f79f7a5d2d7a0b"
+      "sha256": "b1b86eeccda0d20d4c9b739016bd727f4867fa59de6a9dc5526b24ba5a6c92dd"
     },
     {
       "path": "public-docs/index.html",
@@ -5782,7 +5782,7 @@
     {
       "path": "scripts/gds/triage.js",
       "mode": "0000644",
-      "sha256": "8cca4ccf9b5cb86bb9c00ea925ef4bd8790bdb6bb39c4d3afaba85cf072e9760"
+      "sha256": "1c43db4e69ff4966addd91314638b3b2a0c3f7c91f832f09e4a162e5fd0789aa"
     },
     {
       "path": "scripts/gds/update-channel.js",
@@ -6172,7 +6172,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "fbb190fb4b0b50ef8374412a009f1a4799ecbdb15b37caabac3c8a138ca29622"
+      "sha256": "6d275b0dea8e96343483360f9429816f5b6b3a754d71e9ae0a3700481a22e0aa"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -7964,6 +7964,11 @@
       "mode": "0000644",
       "sha256": "211aa868b86a2b663f5525c3d2ddfa406498d608363d28c0786219d3c0b6e249"
     },
+    {
+      "path": "tests/promote_goal_id.mjs",
+      "mode": "0000644",
+      "sha256": "24471b652054a82e985b86fdbbe2dc281114cf9829aed6c9b91348217e7f2460"
+    },
     {
       "path": "tests/prose_edits.mjs",
       "mode": "0000644",

package/.claude/skills/idea-triage/SKILL.md

@@ -1,10 +1,40 @@
 ---
 name: idea-triage
-description: Daily walk through the Bongos idea_inbox — for each open idea, prompt to promote (→ task), discard, or merge with another idea. Triggers when the user says "/idea-triage", "triage ideas", "review the inbox", "walk the idea inbox", or runs as a scheduled daily task. Metic+ rank only (Metic and Archon; not Xenos).
+description: Daily walk through the Bongos idea_inbox, which since task 1003071 holds only HOMELESS work — bugs and goal-bound captures now route to a task at filing time and never reach the inbox. For each open idea, prompt to promote (→ task, naming a goal), discard, or merge with another idea. Triggers when the user says "/idea-triage", "triage ideas", "review the inbox", "walk the idea inbox", or runs as a scheduled daily task. Metic+ rank only (Metic and Archon; not Xenos).
 ---
 
 You are running the daily idea-triage session for Example. Open ideas in the Bongos `idea_inbox` table get walked one-at-a-time and verdicted, so the inbox doesn't accumulate stale entries.
 
+## The inbox is HOMELESS-ONLY now (task 1003071 / R13, goal 1000071)
+
+**What changed.** Since R01 (task 1003065) a bug or goal-bound capture ROUTES at
+filing time — it becomes a task in its goal, in one transaction, and never appears
+here. The inbox is therefore no longer "everything anyone filed"; it holds only
+**work with no home yet**. Expect it to be smaller, and read what IS in it as
+work that genuinely needs a decision rather than a queue to clear.
+
+**Two consequences for how you run this skill:**
+
+1. **Every promote names a goal.** The promote step now asks for one, defaulted
+   for you, and `-` opts out explicitly. Take the default unless you disagree with
+   it. Opting out is legal but rarely right: a promote with no goal is **not**
+   goal-less — the server files it under the version's catch-all `<version> —
+   general` goal — and that bucket carries no done-when criteria, so since
+   [ADR 0183](../../../docs/adr/<redacted>.md) made criteria
+   close themselves off LINKED tasks, work parked there can never advance
+   anything. The general bucket used to be the silent default; it is now a choice
+   you type.
+2. **The default is inherited, not guessed, when a hint exists.** Precedence is
+   the stored `suggested_goal_id` (written at capture precisely so triage inherits
+   the guess) FIRST, then the R07 suggester (`GET /goals/suggestions`). When the
+   suggester supplies it you are shown the score and the matched terms — check
+   them: token overlap can match an incidental word, so a low score with one
+   matched term is weak evidence, not a recommendation.
+
+**If an idea should have routed at filing and didn't**, that is worth noticing
+rather than just promoting past: it usually means the filer named no goal, and the
+capture-time suggester (R08/R09/R11 surfaces) is where the fix belongs.
+
 > **This skill is no longer the only way to triage.** Since task 1002447 a Metic+ builder can promote, discard and merge from the **hall** — the Home "Needs your attention" card's idea rows each carry a *Triage* verb that opens the record overlay, where all three forms live. No Claude session involved. What this skill adds over the hall is the **parallel development pass** (Step 1.5) — an idea fleshed out into open questions/risks before the verdict — so reach for it when the queue needs judgment, and use the hall when it just needs clearing.
 
 ## Rank gate — Metic+ only
@@ -70,7 +100,7 @@ node scripts/gds/triage.js --development "${TMPDIR:-/tmp}/otb-idea-development.j
 
 `triage.js` prints each idea's developed block above its verdict prompt. It never *generates* the analysis (that's this skill's subagent fan-out) — it only displays what the cache holds, and runs exactly as before when no cache is present.
 
-> **Not in this cut (follow-up leg).** The next step is to let the Archon push the developed `open_questions` *back to the submitter* — a round-trip in the Discord `#ideas` thread / DM before promotion — so the submitter says how they'd want it integrated. That needs the Discord bot interaction + idea↔thread linking (idea #216 / [ADR 0033](../../../docs/adr/<redacted>.md)) and is tracked separately. For now the open questions are answered by the Archon in-session.
+> **Not in this cut (follow-up leg).** The next step is to let the Archon push the developed `open_questions` *back to the submitter* — a round-trip in the Discord `#ideas` thread / DM before promotion — so the submitter says how they'd want it integrated. That needs the Discord bot interaction + idea↔thread linking (idea `#216` / [ADR 0033](../../../docs/adr/<redacted>.md)) and is tracked separately. For now the open questions are answered by the Archon in-session.
 
 **Step 2 — walk the list, idea by idea.**
 

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).
 
-**1337 strings** across 3 surfaces · 1186 distinct pieces of copy · 1337 certain, 0 heuristic · 121 shown in more than one place.
+**1339 strings** across 3 surfaces · 1188 distinct pieces of copy · 1339 certain, 0 heuristic · 121 shown in more than one place.
 
 ## Surfaces
 
 | surface | lives at | files | strings |
 |---|---|---|---|
-| `builders-hall` | `modules/hall-ui/public` | 61 | 1053 |
+| `builders-hall` | `modules/hall-ui/public` | 61 | 1055 |
 | `status-dashboard` | `modules/status-ui/public` | 4 | 98 |
 | `landing` | `public-landing` | 2 | 186 |
 
@@ -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 |
 | … | | _81 more in the JSON registry_ | |
 
-## `builders-hall` — 1053 strings
+## `builders-hall` — 1055 strings
 
 | id | conf | text | where |
 |---|---|---|---|
@@ -326,7 +326,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `7f600651bbf1` | certain | Couldn’t load your projects right now. | `modules/hall-ui/public/my-projects.js:115` |
 | `38827a954f3b` | certain | Create a custom rank first. | `modules/hall-ui/public/government.js:397` |
 | `dd8ead7adc9a` | certain | Create rank | `modules/hall-ui/public/government.html:62` |
-| `d7beb841395e` | certain | Create task | `modules/hall-ui/public/task-detail.js:300` |
+| `d7beb841395e` | certain | Create task | `modules/hall-ui/public/task-detail.js:354` |
 | `533f24219487` | certain | Create your | `modules/hall-ui/public/builders.js:984` |
 | `e91a4d2e3e1c` | certain | Created rank {…}. | `modules/hall-ui/public/government.js:380` |
 | `5968d13a9dbe` | certain | Criteria met — awaiting review | `modules/hall-ui/public/goals-page.js:341` |
@@ -364,7 +364,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `854a21d2eb39` | certain | Eight achievements recognize sustained contribution. Earned ones are highlighted; the rest show what’s left to | `modules/hall-ui/public/hall-widgets.js:197` |
 | `58d381535285` | certain | Engagement reports | `modules/hall-ui/public/watch.html:120` |
 | `35d1ae09a6b3` | certain | Enter a project URL. | `modules/hall-ui/public/my-projects.js:122` |
-| `bd7a40639a1c` | certain | Estimate (min) | `modules/hall-ui/public/task-detail.js:295` |
+| `bd7a40639a1c` | certain | Estimate (min) | `modules/hall-ui/public/task-detail.js:349` |
 | `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` |
@@ -387,7 +387,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `b9267033b1f1` | certain | Flag this copy | `modules/hall-ui/public/copy-desk.html:87` |
 | `448d00771dc8` | certain | Flagged. It is in the queue. | `modules/hall-ui/public/copy-desk.js:281` |
 | `617f13fa3fdb` | certain | Flip the switch. | `modules/hall-ui/public/builders.js:984` |
-| `42cee5cb3584` | certain | Fold this idea into # | `modules/hall-ui/public/task-detail.js:369` |
+| `42cee5cb3584` | certain | Fold this idea into # | `modules/hall-ui/public/task-detail.js:444` |
 | `647bcfcfdb03` | certain | Fresh CLI token issued. Paste it in your terminal: | `modules/hall-ui/public/settings.js:885` |
 | `55c71d76a94c` | certain | From creation to production — and the exact step where {{currency}} are awarded (at | `modules/hall-ui/public/diagrams.html:61` |
 | `a025080147a1` | certain | Full Idea | `modules/hall-ui/public/government.js:669` |
@@ -490,7 +490,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `0f9f86c556a2` | certain | Loading the primer… | `modules/hall-ui/public/primer.html:55` |
 | `ceec2218efe3` | certain | Loading the roadmap… | `modules/hall-ui/public/roadmap.html:24` |
 | `ceec2218efe3` | certain | Loading the roadmap… | `modules/hall-ui/public/roadmap.html:29` |
-| `d28d321e8a1e` | certain | Loading versions… | `modules/hall-ui/public/task-detail.js:258` |
+| `d28d321e8a1e` | certain | Loading versions… | `modules/hall-ui/public/task-detail.js:307` |
 | `3c43f157035e` | certain | Loading your goals… | `modules/hall-ui/public/goals.js:95` |
 | `7a239a17afc8` | certain | Loading your preferences… | `modules/hall-ui/public/settings.html:26` |
 | `04df1bd30129` | certain | Loading your rank… | `modules/hall-ui/public/ranks.html:23` |
@@ -564,7 +564,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `2c19f41704b9` | certain | Not yet available — {{worldName}} | `modules/hall-ui/public/not-ready.html:6` |
 | `2c19f41704b9` | certain | Not yet available — {{worldName}} | `modules/hall-ui/public/not-ready.html:6` |
 | `571b9b22dbda` | certain | Not yet ready — blocked by dependencies or awaiting promotion. These become claimable when their blockers ship | `modules/hall-ui/public/work.html:87` |
-| `0c59d118db9b` | certain | Note (optional) | `modules/hall-ui/public/task-detail.js:372` |
+| `0c59d118db9b` | certain | Note (optional) | `modules/hall-ui/public/task-detail.js:447` |
 | `95f8f8a4b288` | certain | Nothing flagged here. That is either good news or an unread queue — the duplicates below are the cheapest plac | `modules/hall-ui/public/copy-desk.js:172` |
 | `fec5d78497e8` | certain | Nothing in the backlog right now. | `modules/hall-ui/public/work.js:953` |
 | `18ce0723be0d` | certain | Nothing is waiting — no held changes need your approval. | `modules/hall-ui/public/gate.js:188` |
@@ -642,7 +642,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `26e6c9879035` | certain | Reading the diagrams | `modules/hall-ui/public/diagrams.html:48` |
 | `81cd2cdf954d` | certain | Reading the request… | `modules/hall-ui/public/pair.html:23` |
 | `5b581cca590f` | certain | Ready — tasks to do | `modules/hall-ui/public/work.html:53` |
-| `9ea556e7b78b` | certain | Reason (optional) | `modules/hall-ui/public/task-detail.js:423` |
+| `9ea556e7b78b` | certain | Reason (optional) | `modules/hall-ui/public/task-detail.js:498` |
 | `7df3fd4ebf41` | certain | Reason (shown to the builder) | `modules/hall-ui/public/harbor.js:146` |
 | `06cbba1440b1` | certain | Rebase owed — new claims paused | `modules/hall-ui/public/builders.js:752` |
 | `58fb6fe4b47e` | certain | Recent history could not be loaded right now. | `modules/hall-ui/public/work.js:1052` |
@@ -974,7 +974,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `06aab18fc3b5` | certain | holds the repo. Follow the numbered path to see how a credential-less box gets its work to prod. This is the l | `modules/hall-ui/public/diagrams.html:83` |
 | `17ab8db79e1f` | certain | holds the worker, the code, and your local memory; the | `modules/hall-ui/public/diagrams.html:83` |
 | `7d5cbe3d81e4` | certain | how long a vote stays open | `modules/hall-ui/public/government.html:40` |
-| `dacb901c4f10` | certain | idea number | `modules/hall-ui/public/task-detail.js:370` |
+| `dacb901c4f10` | certain | idea number | `modules/hall-ui/public/task-detail.js:445` |
 | `b2b399fdd6a4` | certain | in progress | `modules/hall-ui/public/goals-page.js:423` |
 | `b2b399fdd6a4` | certain | in progress | `modules/hall-ui/public/roadmap.js:108` |
 | `b2b399fdd6a4` | certain | in progress | `modules/hall-ui/public/roadmap.js:194` |
@@ -987,6 +987,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `c767b3e07826` | certain | is only the screen; the | `modules/hall-ui/public/diagrams.html:83` |
 | `aa530bd15088` | certain | itself). The registry stays the single source of truth for what the hall shows; a fitness check (scripts/gds/f | `modules/hall-ui/public/index.html:44` |
 | `d46875348b9c` | certain | ladder · read-only | `modules/hall-ui/public/government.js:116` |
+| `2ce7738b1eb0` | certain | loading goals… | `modules/hall-ui/public/task-detail.js:371` |
 | `8c22f0ee5edb` | certain | looked at | `modules/hall-ui/public/copy-desk.js:114` |
 | `754bdec368ed` | certain | lowest credits | `modules/hall-ui/public/hall-widgets.js:136` |
 | `cffba8409070` | certain | manage its visibility in Settings → Cross-project record | `modules/hall-ui/public/profile.js:200` |
@@ -1053,8 +1054,8 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `1651cb1876f5` | certain | who sits on the board | `modules/hall-ui/public/government.html:40` |
 | `4109de2e0bb9` | certain | whole surface | `modules/hall-ui/public/copy-desk.js:88` |
 | `4109de2e0bb9` | certain | whole surface | `modules/hall-ui/public/copy-desk.js:251` |
-| `143f77e2b782` | certain | why these are the same | `modules/hall-ui/public/task-detail.js:373` |
-| `ec7c228cb4ce` | certain | why this isn’t worth doing | `modules/hall-ui/public/task-detail.js:424` |
+| `143f77e2b782` | certain | why these are the same | `modules/hall-ui/public/task-detail.js:448` |
+| `ec7c228cb4ce` | certain | why this isn’t worth doing | `modules/hall-ui/public/task-detail.js:499` |
 | `66d979d0db89` | certain | your vote | `modules/hall-ui/public/government.js:691` |
 | `7e2f2aac0645` | certain | {…} You always see your own numbers here. | `modules/hall-ui/public/settings.js:580` |
 | `43b99dd1b755` | certain | {…} You can also ask in Discord #support. | `modules/hall-ui/public/box-panel.js:358` |
@@ -1104,6 +1105,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `cbd220dd9184` | certain | — a later push must be re-approved. | `modules/hall-ui/public/gate.js:87` |
 | `a6ae9cbdee30` | certain | — all builders — | `modules/hall-ui/public/builders.js:1692` |
 | `729d4ce22b99` | certain | — any — | `modules/hall-ui/public/hall-widgets.js:132` |
+| `9ac518dfa95e` | certain | — none (the version's general bucket) — | `modules/hall-ui/public/task-detail.js:302` |
 | `da9962dc1729` | certain | — or open any diagram fullscreen. | `modules/hall-ui/public/diagrams.html:42` |
 | `d4ff85363f89` | certain | — so if Claude Desktop then says | `modules/hall-ui/public/box-panel.js:160` |
 | `e5c5e94152b3` | certain | — that's all the tooling you need up front. | `modules/hall-ui/public/builders.js:1051` |
@@ -1122,7 +1124,7 @@ Identical copy within one surface. Sometimes right (a repeated button), sometime
 | `7926e30955d2` | certain | ← Back to goal | `modules/hall-ui/public/goals.html:163` |
 | `7926e30955d2` | certain | ← Back to goal | `modules/hall-ui/public/goals.html:175` |
 | `1e7cf7b1e64e` | certain | ← Newer | `modules/hall-ui/public/sessions.html:56` |
-| `2180e0ecf138` | certain | → no idea # | `modules/hall-ui/public/task-detail.js:391` |
+| `2180e0ecf138` | certain | → no idea # | `modules/hall-ui/public/task-detail.js:466` |
 | `d5d76f1c0dfe` | certain | ○ reserved | `modules/hall-ui/public/diagrams.html:53` |
 | `97b0773e72a5` | certain | ● LIVE | `modules/hall-ui/public/diagrams.html:51` |
 | `1375334c77f6` | certain | ⚠ This box is running | `modules/hall-ui/public/box-panel.js:202` |

package/docs/copy-registry.json

@@ -3,9 +3,9 @@
   "generator": "scripts/gds/copy-inventory.js",
   "goal": 1000074,
   "counts": {
-    "strings": 1337,
-    "unique_ids": 1186,
-    "certain": 1337,
+    "strings": 1339,
+    "unique_ids": 1188,
+    "certain": 1339,
     "heuristic": 0,
     "duplicate_ids": 121
   },
@@ -13,7 +13,7 @@
     "builders-hall": {
       "at": "modules/hall-ui/public",
       "files": 61,
-      "strings": 1053
+      "strings": 1055
     },
     "status-dashboard": {
       "at": "modules/status-ui/public",
@@ -3481,7 +3481,7 @@
       "surface": "builders-hall",
       "text": "Create task",
       "file": "modules/hall-ui/public/task-detail.js",
-      "line": 300,
+      "line": 354,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -3823,7 +3823,7 @@
       "surface": "builders-hall",
       "text": "Estimate (min)",
       "file": "modules/hall-ui/public/task-detail.js",
-      "line": 295,
+      "line": 349,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -4030,7 +4030,7 @@
       "surface": "builders-hall",
       "text": "Fold this idea into #",
       "file": "modules/hall-ui/public/task-detail.js",
-      "line": 369,
+      "line": 444,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -4957,7 +4957,7 @@
       "surface": "builders-hall",
       "text": "Loading versions…",
       "file": "modules/hall-ui/public/task-detail.js",
-      "line": 258,
+      "line": 307,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -5623,7 +5623,7 @@
       "surface": "builders-hall",
       "text": "Note (optional)",
       "file": "modules/hall-ui/public/task-detail.js",
-      "line": 372,
+      "line": 447,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -6325,7 +6325,7 @@
       "surface": "builders-hall",
       "text": "Reason (optional)",
       "file": "modules/hall-ui/public/task-detail.js",
-      "line": 423,
+      "line": 498,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -9313,7 +9313,7 @@
       "surface": "builders-hall",
       "text": "idea number",
       "file": "modules/hall-ui/public/task-detail.js",
-      "line": 370,
+      "line": 445,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -9425,6 +9425,15 @@
       "origin": "js-markup",
       "confidence": "certain"
     },
+    {
+      "id": "2ce7738b1eb0",
+      "surface": "builders-hall",
+      "text": "loading goals…",
+      "file": "modules/hall-ui/public/task-detail.js",
+      "line": 371,
+      "origin": "js-markup",
+      "confidence": "certain"
+    },
     {
       "id": "8c22f0ee5edb",
       "surface": "builders-hall",
@@ -10024,7 +10033,7 @@
       "surface": "builders-hall",
       "text": "why these are the same",
       "file": "modules/hall-ui/public/task-detail.js",
-      "line": 373,
+      "line": 448,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -10033,7 +10042,7 @@
       "surface": "builders-hall",
       "text": "why this isn’t worth doing",
       "file": "modules/hall-ui/public/task-detail.js",
-      "line": 424,
+      "line": 499,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -10478,6 +10487,15 @@
       "origin": "js-markup",
       "confidence": "certain"
     },
+    {
+      "id": "9ac518dfa95e",
+      "surface": "builders-hall",
+      "text": "— none (the version's general bucket) —",
+      "file": "modules/hall-ui/public/task-detail.js",
+      "line": 302,
+      "origin": "js-markup",
+      "confidence": "certain"
+    },
     {
       "id": "da9962dc1729",
       "surface": "builders-hall",
@@ -10645,7 +10663,7 @@
       "surface": "builders-hall",
       "text": "→ no idea #",
       "file": "modules/hall-ui/public/task-detail.js",
-      "line": 391,
+      "line": 466,
       "origin": "js-text-assign",
       "confidence": "certain"
     },

package/modules/hall-ui/public/task-detail.js

@@ -254,6 +254,55 @@
         + '</div>';
     }
 
+    // task 1003071 (R13) — the goal picker's options, same precedence as the CLI
+    // triage path: the STORED hint (idea.suggested_goal_id, written by
+    // inbox.captureIdea precisely so triage inherits the guess) wins; the R07
+    // suggester only fills in when there is no hint. Returns option HTML.
+    async function goalOptionsFor(versionId) {
+      // Number() so a junk stored value cannot reach the DOM as value="NaN" and
+      // then be POSTed as a goal id.
+      var raw = Number(idea.suggested_goal_id);
+      var hint = Number.isFinite(raw) && raw > 0 ? raw : null;
+      var suggestions = [];
+      // The suggester is a convenience, so an outage degrades to "no suggestions"
+      // and must not stop a triage verdict — the hint and the opt-out below are
+      // both still reachable without it.
+      try {
+        var q = '?version=' + encodeURIComponent(versionId)
+          + '&title=' + encodeURIComponent(idea.title || '')
+          + '&body=' + encodeURIComponent(String(idea.body_md || '').slice(0, 2000))
+          + '&limit=5';
+        // softAuth: a cosmetic suggestion list must not navigate the builder away
+        // from a half-filled promote form. The submit re-auths if the session died.
+        var res = await getJSON(API + '/goals/suggestions' + q, { softAuth: true });
+        suggestions = (res && res.suggestions) || [];
+      } catch (_) { suggestions = []; }
+      var opts = [];
+      var seen = {};
+      if (hint != null) {
+        seen[hint] = true;
+        opts.push('<option value="' + hint + '" selected>goal ' + hint + ' — carried from the filing</option>');
+      }
+      suggestions.forEach(function (g, i) {
+        if (seen[g.id]) return;
+        seen[g.id] = true;
+        var why = (g.matched_terms || []).join(', ');
+        // The score + matched terms are shown on purpose: token overlap can match
+        // an incidental word, so the operator judges the suggestion rather than
+        // trusting it (task 1003062's stated limit).
+        opts.push('<option value="' + esc(String(g.id)) + '"' + (hint == null && i === 0 ? ' selected' : '') + '>'
+          + esc(String(g.id)) + ' — ' + esc(g.title || '') + (why ? ' (matched: ' + esc(why) + ')' : '') + '</option>');
+      });
+      // Always offer the opt-out, and make it the fallback when nothing was found.
+      // Sending no goal_id does not create a goal-less task — db.createTask files
+      // it under the version's catch-all "general" goal — so the label names where
+      // it actually lands. That bucket carries no done-when criteria, which is why
+      // it should be a choice rather than the silent default it used to be.
+      opts.push('<option value=""' + (opts.length === 0 ? ' selected' : '')
+        + '>— none (the version\'s general bucket) —</option>');
+      return opts.join('');
+    }
+
     async function renderPromote() {
       box.innerHTML = '<p class="deeplink-card__note">Loading versions…</p>';
       // Both dead ends offer the way back through the same delegated [data-go]
@@ -283,6 +332,10 @@
       // CHECKed 1-5, so clamp rather than let an out-of-range value match no
       // option and leave the browser silently defaulting to P1.
       var pri = Math.min(5, Math.max(1, Number(idea.suggested_priority) || 3));
+      // Fetched for the PREFERRED version; the change listener below re-fetches,
+      // because goals belong to a version and showing another version's goals
+      // would offer a home the server would reject.
+      var goalOptions = await goalOptionsFor(preferred);
       var priOptions = [1, 2, 3, 4, 5].map(function (p) {
         return '<option value="' + p + '"' + (p === pri ? ' selected' : '') + '>P' + p + '</option>';
       }).join('');
@@ -291,6 +344,7 @@
         + '<label class="td-field td-field--wide"><span>Title</span>'
         + '<input type="text" name="title" maxlength="' + TITLE_MAX + '" required value="' + esc(idea.title || '') + '"></label>'
         + '<label class="td-field td-field--wide"><span>Version</span><select name="version">' + options + '</select></label>'
+        + '<label class="td-field td-field--wide"><span>Goal</span><select name="goal" data-goal-select>' + goalOptions + '</select></label>'
         + '<label class="td-field"><span>Priority</span><select name="priority">' + priOptions + '</select></label>'
         + '<label class="td-field"><span>Estimate (min)</span>'
         + '<input type="number" name="est" min="0" max="100000" step="5" placeholder="—"></label>'
@@ -304,6 +358,23 @@
         ev.preventDefault();
         submitPromote(ev.target);
       });
+      // Re-fetch the goal list when the version changes (task 1003071). Two quick
+      // changes race — the fetches can resolve out of order — so only the newest
+      // request is allowed to write, and only it re-enables the control.
+      var vsel = box.querySelector('select[name="version"]');
+      var gsel = box.querySelector('[data-goal-select]');
+      if (vsel && gsel) {
+        var goalFetch = 0;
+        vsel.addEventListener('change', async function () {
+          var mine = ++goalFetch;
+          gsel.disabled = true;
+          gsel.innerHTML = '<option value="">loading goals…</option>';
+          var html = await goalOptionsFor(vsel.value);
+          if (mine !== goalFetch) return;
+          gsel.innerHTML = html;
+          gsel.disabled = false;
+        });
+      }
     }
 
     async function submitPromote(form) {
@@ -325,6 +396,10 @@
         source: 'idea_inbox',
         source_ref: 'idea:' + idea.id,
       };
+      // task 1003071 (R13): the whole point — a triaged promotion NAMES its goal.
+      // Empty means the operator chose the version's general bucket, which the
+      // server applies itself when goal_id is absent.
+      if (form.goal && form.goal.value !== '') payload.goal_id = Number(form.goal.value);
       if (form.est.value !== '') payload.est_minutes = Number(form.est.value);
       if (form.credits.value !== '') payload.credits_reward = Number(form.credits.value);
       // 'criterion-proposal' is an idea-only kind; everything else is shared with

package/package-lock.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.289",
+  "version": "1.19.290",
   "description": "Cloud Bongos — the AI-first build platform core (GDS + platform surfaces + module system), installed as a versioned dependency (ADR 0108).",
   "license": "AGPL-3.0-or-later",
   "main": "src/platform-server.js",

package/scripts/gds/triage.js

@@ -151,11 +151,80 @@ async function patchIdea(id, patch) {
   return r.data?.idea;
 }
 
-async function createTask({ versionId, title, description, priority, sourceRef }) {
-  // NOTE: POST /api/gds/tasks doesn't currently accept 'kind' — tasks default
-  // to 'unclassified' and the server-side classifier (B1) runs separately.
-  // The idea's auto-classified kind is therefore informational only at this
-  // step; the task description preserves the body for re-classification.
+// resolveGoalDefault — what goal should a triaged promotion land in? (task
+// 1003071 / R13.)
+//
+// PRECEDENCE IS DELIBERATE:
+//   1. `idea.suggested_goal_id` — the STORED hint. inbox.captureIdea writes it
+//      even when the matrix declined to route, precisely so "/idea-triage
+//      inherits the guess instead of re-deriving it" (core_222, its own comment).
+//      A human or a filing surface already named this goal; a heuristic must not
+//      overrule that.
+//   2. the R07 suggester (GET /goals/suggestions) — only when there is no hint.
+//      Best-scoring candidate, and its score/matched_terms are shown so the
+//      operator can see WHY before accepting.
+// Returns { goalId, label, source } — goalId null when neither yields anything.
+// The caller prints `source` as provenance, because "the suggester guessed this"
+// and "the filing already named this" deserve different trust. Never throws: a
+// suggester outage degrades to "no default", it does not block a triage verdict.
+//
+// `client` is injectable so the precedence can be driven end-to-end in a test
+// rather than pattern-matched in the source. (Task 1003102 shipped a handler
+// whose collaborators were not injectable; its tests read the source, passed,
+// and the route 500'd on every call. Not repeating that.)
+async function resolveGoalDefault(idea, versionId, { client = cliClient } = {}) {
+  // Guarded: a junk stored value must not become goalId NaN and then a goal_id
+  // the server rejects with 400 on every retry.
+  const hint = Number(idea.suggested_goal_id);
+  if (Number.isInteger(hint) && hint > 0) {
+    return { goalId: hint, label: `goal ${hint}`, source: 'hint' };
+  }
+  try {
+    const api = await client();
+    // The GENERATED typed method (regenerated when R07 landed), not the generic
+    // request() escape hatch — so a path or param rename reds the client build.
+    const r = await api.goals.getGoalsSuggestions({
+      query: { version: versionId, title: idea.title || '', body: (idea.body_md || '').slice(0, 2000), limit: 3 },
+    });
+    const top = r.ok && r.data && Array.isArray(r.data.suggestions) ? r.data.suggestions[0] : null;
+    if (!top) return { goalId: null, label: null, source: 'none' };
+    const why = (top.matched_terms || []).join('/');
+    return { goalId: Number(top.id), label: `${top.id} ${trim(top.title, 44)} (score ${top.score}${why ? ' · ' + why : ''})`, source: 'suggester' };
+  } catch (_) {
+    return { goalId: null, label: null, source: 'unavailable' };
+  }
+}
+
+// What the operator's answer to the goal prompt means. Blank ACCEPTS the shown
+// default — declining has to be said ('-' or 'none'), because silently dropping a
+// real default on a bare Enter is how work ends up in the general bucket by
+// accident. Declining is legal: the server then applies the catch-all itself.
+function applyGoalAnswer(defaultGoalId, rawAnswer) {
+  const answer = String(rawAnswer == null ? '' : rawAnswer).trim();
+  if (answer === '') return { goalId: defaultGoalId, warn: null };
+  if (answer === '-' || answer.toLowerCase() === 'none') return { goalId: null, warn: null };
+  const parsed = Number(answer);
+  if (!Number.isInteger(parsed) || parsed <= 0) {
+    return { goalId: null, warn: "not a goal id — falling back to the version's general bucket" };
+  }
+  return { goalId: parsed, warn: null };
+}
+
+async function createTask({ versionId, title, description, priority, sourceRef, goalId = null }) {
+  // (The old note here said POST /api/gds/tasks does not accept 'kind'. That is
+  // no longer true — routes/tasks.js takes it, and the hall promote path already
+  // sends it. Left un-sent from THIS path deliberately: changing what kind a
+  // triaged promotion lands with is a behaviour change task R13 was not asked to
+  // make, and the server classifier still runs. Corrected rather than deleted so
+  // the next reader does not re-derive it.)
+  //
+  // task 1003071 (R13): goal_id is the point of this task. Omitting it does NOT
+  // leave the task goal-less — db.createTask (modules/lifecycle/db.js) defaults a
+  // goal-less create into the version's catch-all "<version> — general" goal. That
+  // bucket is the actual defect: it carries no done-when criteria, so since ADR
+  // 0183 made criteria close themselves off LINKED tasks, work parked there can
+  // never advance anything, and it does not read as real goal work on the board.
+  // Naming the goal here is what stops a triaged promotion falling into it.
   const body = {
     version_id: versionId,
     title,
@@ -165,6 +234,7 @@ async function createTask({ versionId, title, description, priority, sourceRef }
     source: 'idea_inbox',
     source_ref: sourceRef ?? null,
   };
+  if (goalId != null) body.goal_id = Number(goalId);
   const api = await cliClient();
   const r = await api.tasks.postTasks({ body });
   if (!r.ok) {
@@ -296,7 +366,7 @@ async function main() {
         }
 
         if (ans === 'p' || ans === 'promote') {
-          // Promote: ask for version + priority (defaulted from idea).
+          // Promote: version, priority, title, goal — each defaulted from the idea.
           const defaultVersion = idea.suggested_version || 'PMS-V2';
           const versionAns = (await prompt(rl, `  version [${defaultVersion}]: `)).trim();
           const versionId = versionAns || defaultVersion;
@@ -308,6 +378,23 @@ async function main() {
           const titleAns = (await prompt(rl, `  title [${trim(idea.title, 60)}]: `)).trim();
           const title = titleAns || idea.title;
 
+          // task 1003071 (R13): a triaged promotion NAMES its goal. The default
+          // comes from the stored hint, else the R07 suggester; '-' opts out
+          // explicitly, which falls through to the version's catch-all "general"
+          // bucket — now a choice someone typed rather than the silent default.
+          // The EFFECTIVE title, not idea.title: the suggester matches on title
+          // terms, and the operator may just have retitled this.
+          const gd = await resolveGoalDefault({ ...idea, title }, versionId);
+          if (gd.source === 'suggester') console.log(`  suggested goal: ${gd.label}`);
+          if (gd.source === 'unavailable') console.log('  (goal suggester unreachable — enter a goal id, or "-" to accept the general bucket)');
+          const goalPrompt = gd.goalId != null
+            ? `  goal [${gd.goalId}] ("-" for the version's general bucket): `
+            : '  goal (blank/"-" for the version\'s general bucket): ';
+          const goalAns = (await prompt(rl, goalPrompt)).trim();
+          const picked = applyGoalAnswer(gd.goalId, goalAns);
+          if (picked.warn) console.log(`  ! ${picked.warn}`);
+          const goalId = picked.goalId;
+
           try {
             const task = await createTask({
               versionId,
@@ -315,6 +402,7 @@ async function main() {
               description: idea.body_md || '',
               priority: Number.isFinite(priority) ? priority : null,
               sourceRef: `idea:${idea.id}`,
+              goalId,
             });
             await patchIdea(idea.id, {
               status: 'promoted',
@@ -323,7 +411,7 @@ async function main() {
             });
             summary.promoted++;
             promotedIds.push({ ideaId: idea.id, taskId: task.id });
-            console.log(`  ✓ promoted #${idea.id} → task #${task.id} (${versionId}, P${priority})`);
+            console.log(`  ✓ promoted #${idea.id} → task #${task.id} (${versionId}, P${priority}${goalId != null ? `, goal ${goalId}` : ', → general bucket'})`);
           } catch (err) {
             console.error(`  ! ${err.message}`);
           }
@@ -360,6 +448,9 @@ if (require.main === module) {
 }
 
 module.exports = {
+  resolveGoalDefault,
+  applyGoalAnswer,
+  createTask,
   resolveDevelopmentPath,
   parseDevelopment,
   loadDevelopment,

package/src/module-api.js

@@ -1005,8 +1005,10 @@ const { validateOrRespond, LIMITS, parseId, asyncHandler, corsPublicGet, parsePa
 //         landed since 1.19.287 with no explicit bump. run 32546561232. (task 1002620)
 //   1.19.289 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
 //         landed since 1.19.288 with no explicit bump. run 32549912116. (task 1002620)
+//   1.19.290 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+//         landed since 1.19.289 with no explicit bump. run 32552538814. (task 1002620)
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.289'; // CI auto-patch carrier (ADR 0161); see changelog above
+const CORE_VERSION = '1.19.290'; // 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/promote_goal_id.mjs

@@ -0,0 +1,316 @@
+// tests/promote_goal_id.mjs — a triaged promotion lands IN A GOAL (task 1003071 / R13).
+//
+// Before this, both promote paths — the hall triage overlay and the /idea-triage
+// CLI — created a task with no goal_id at all. That does NOT mean the task came
+// out goal-less: db.createTask (modules/lifecycle/db.js) defaults a goal-less
+// create into the version's catch-all "<version> — general" goal. That bucket IS
+// the defect. It carries no done-when criteria, so since ADR 0183 made criteria
+// close themselves off LINKED tasks, work parked there can never advance
+// anything — and it does not read as real goal work on the board. Falling into it
+// had to stop being the silent default.
+//
+// What these pin:
+//  1. PRECEDENCE. The stored `suggested_goal_id` hint beats the R07 suggester.
+//     inbox.captureIdea writes that hint even when the matrix declined to route,
+//     specifically so triage "inherits the guess instead of re-deriving it"
+//     (core_222's own comment). A heuristic must never overrule a named goal.
+//  2. OPT-OUT IS EXPLICIT. Taking the general bucket is still allowed — it just
+//     has to be chosen, not defaulted into.
+//  3. DEGRADES, NEVER BLOCKS. A suggester outage must not stop a triage verdict.
+//  4. BOTH PATHS send goal_id. Two surfaces, one contract — the same class of
+//     miss as the two claim-insert sites (task 1003137).
+//
+// Run:  node tests/promote_goal_id.mjs
+
+import { strict as assert } from 'node:assert';
+import { createRequire } from 'node:module';
+import { readFileSync } from 'node:fs';
+import { makeRunner } from './helpers.mjs';
+
+process.env.NODE_ENV = 'test';
+const require = createRequire(import.meta.url);
+const { test, summary } = makeRunner();
+
+// triage.js captures `cliClient` by destructuring at require time, so a stub has
+// to be in place BEFORE the module loads. Dropping both from the require cache
+// and re-requiring is what makes that possible per-test.
+function freshTriage() {
+  delete require.cache[require.resolve('../scripts/gds/triage.js')];
+  return require('../scripts/gds/triage.js');
+}
+
+const TRIAGE = readFileSync(new URL('../scripts/gds/triage.js', import.meta.url), 'utf8');
+const HALL = readFileSync(new URL('../modules/hall-ui/public/task-detail.js', import.meta.url), 'utf8');
+const SKILL = readFileSync(new URL('../.claude/skills/idea-triage/SKILL.md', import.meta.url), 'utf8');
+
+// ==========================================================================
+// BEHAVIOURAL — these INVOKE the code. The static assertions further down pin
+// the wiring a unit test cannot reach (the DOM strings, the skill doc); they do
+// not stand in for actually running it.
+// ==========================================================================
+
+const triage = require('../scripts/gds/triage.js');
+
+// A stub of the generated typed client: records the call, answers from a script.
+function stubClient(answer) {
+  const calls = [];
+  const client = async () => ({
+    goals: {
+      getGoalsSuggestions: async (req) => {
+        calls.push(req);
+        if (answer instanceof Error) throw answer;
+        return answer;
+      },
+    },
+  });
+  return { client, calls };
+}
+
+const SUGGESTED = {
+  ok: true,
+  data: { suggestions: [{ id: 1000071, title: 'Idea routing', score: 7, matched_terms: ['triage', 'goal'] }] },
+};
+
+await test('BEHAVIOUR: a junk stored hint is ignored, not turned into goalId NaN', async () => {
+  for (const junk of ['not-an-id', '', '0', '-3', '4.5', {}]) {
+    const { client, calls } = stubClient(SUGGESTED);
+    const r = await triage.resolveGoalDefault({ suggested_goal_id: junk, title: 't' }, 'BV1', { client });
+    assert.notEqual(r.source, 'hint', `${JSON.stringify(junk)} is not a usable hint`);
+    assert.equal(calls.length, 1, 'and it falls through to the suggester rather than dead-ending');
+  }
+});
+
+await test('BEHAVIOUR: a stored hint wins and the suggester is never even called', async () => {
+  const { client, calls } = stubClient(SUGGESTED);
+  const r = await triage.resolveGoalDefault({ suggested_goal_id: 1000058, title: 'x' }, 'BV1', { client });
+  assert.equal(r.goalId, 1000058);
+  assert.equal(r.source, 'hint');
+  assert.equal(calls.length, 0, 'a named goal must not be re-derived — that is the whole point of the column');
+});
+
+await test('BEHAVIOUR: with no hint, the top suggestion becomes the default, labelled with its score', async () => {
+  const { client, calls } = stubClient(SUGGESTED);
+  const r = await triage.resolveGoalDefault({ title: 'goal triage routing', body_md: 'b' }, 'BV1', { client });
+  assert.equal(r.goalId, 1000071);
+  assert.equal(r.source, 'suggester');
+  assert.match(r.label, /score 7/);
+  assert.match(r.label, /triage\/goal/, 'the matched terms are what let the operator judge it');
+  assert.equal(calls.length, 1);
+  assert.equal(calls[0].query.version, 'BV1', 'goals are per-version — the wrong version offers an invalid home');
+  assert.equal(calls[0].query.limit, 3);
+});
+
+await test('BEHAVIOUR: the body is capped before it goes on a query string', async () => {
+  const { client, calls } = stubClient(SUGGESTED);
+  await triage.resolveGoalDefault({ title: 't', body_md: 'x'.repeat(9000) }, 'BV1', { client });
+  assert.equal(calls[0].query.body.length, 2000, 'an un-capped body would blow the URL length');
+});
+
+await test('BEHAVIOUR: a suggester THROW degrades to no default, it does not propagate', async () => {
+  const { client } = stubClient(new Error('ECONNREFUSED'));
+  const r = await triage.resolveGoalDefault({ title: 't' }, 'BV1', { client });
+  assert.deepEqual(r, { goalId: null, label: null, source: 'unavailable' },
+    'a triage verdict must survive the suggester being down');
+});
+
+await test('BEHAVIOUR: a non-ok or empty response is "none", distinct from "unavailable"', async () => {
+  const empty = await triage.resolveGoalDefault({ title: 't' }, 'BV1',
+    { client: stubClient({ ok: true, data: { suggestions: [] } }).client });
+  assert.equal(empty.source, 'none');
+  const notOk = await triage.resolveGoalDefault({ title: 't' }, 'BV1',
+    { client: stubClient({ ok: false, status: 500, data: null }).client });
+  assert.equal(notOk.source, 'none', 'the two are reported differently because the operator advice differs');
+});
+
+await test('BEHAVIOUR: blank ACCEPTS the default; only "-"/"none" declines', () => {
+  assert.deepEqual(triage.applyGoalAnswer(1000071, ''), { goalId: 1000071, warn: null });
+  assert.deepEqual(triage.applyGoalAnswer(1000071, '   '), { goalId: 1000071, warn: null },
+    'whitespace is a bare Enter, not a decision');
+  assert.deepEqual(triage.applyGoalAnswer(1000071, '-'), { goalId: null, warn: null });
+  assert.deepEqual(triage.applyGoalAnswer(1000071, 'none'), { goalId: null, warn: null });
+  assert.deepEqual(triage.applyGoalAnswer(1000071, 'NONE'), { goalId: null, warn: null });
+  assert.deepEqual(triage.applyGoalAnswer(1000071, '1000058'), { goalId: 1000058, warn: null },
+    'an explicit id overrides the default');
+});
+
+await test('BEHAVIOUR: junk clears the goal and SAYS SO rather than landing on a wrong one', () => {
+  for (const junk of ['abc', '12.5', '-4', '0', 'goal 7']) {
+    const r = triage.applyGoalAnswer(1000071, junk);
+    assert.equal(r.goalId, null, `"${junk}" must not resolve to a goal`);
+    assert.match(r.warn, /general bucket/, `"${junk}" must say where it will land instead`);
+  }
+  // Number('') is 0 and Number(' ') is 0 — both would silently become goal 0
+  // without the blank check above running first. Guarding the pair together.
+  assert.equal(triage.applyGoalAnswer(9, '').goalId, 9);
+});
+
+await test('BEHAVIOUR: createTask puts goal_id on the body, and omits it entirely when null', async () => {
+  const seen = [];
+  const fakeClient = {
+    tasks: { postTasks: async (req) => { seen.push(req.body); return { ok: true, data: { task: { id: 7 } } }; } },
+  };
+  const cliLib = require('../scripts/gds/cli-lib.js');
+  const realCliClient = cliLib.cliClient;
+  cliLib.cliClient = async () => fakeClient;
+  try {
+    // triage.js destructured cliClient at require time, so patching the module
+    // object cannot reach it — re-require in a fresh registry instead.
+    const fresh = freshTriage();
+    await fresh.createTask({ versionId: 'BV1', title: 't', description: 'd', priority: 3, sourceRef: 'idea:1', goalId: 1000071 });
+    await fresh.createTask({ versionId: 'BV1', title: 't', description: 'd', priority: 3, sourceRef: 'idea:2' });
+    assert.equal(seen[0].goal_id, 1000071, 'the goal must reach the server');
+    assert.ok(!('goal_id' in seen[1]),
+    'and opting out must send NO goal_id key — the server\'s catch-all only applies on absent');
+    assert.equal(seen[0].source, 'idea_inbox', 'provenance still travels');
+    assert.equal(seen[0].status, 'backlog');
+  } finally {
+    cliLib.cliClient = realCliClient;
+  }
+});
+
+await test('BEHAVIOUR: a failed create throws with the status, it does not return a phantom task', async () => {
+  const cliLib = require('../scripts/gds/cli-lib.js');
+  const realCliClient = cliLib.cliClient;
+  cliLib.cliClient = async () => ({
+    tasks: { postTasks: async () => ({ ok: false, status: 422, data: { error: 'NO_SUCH_GOAL' } }) },
+  });
+  try {
+    const fresh = freshTriage();
+    await assert.rejects(
+      () => fresh.createTask({ versionId: 'BV1', title: 't', description: 'd', goalId: 999999999 }),
+      /422/,
+    );
+  } finally {
+    cliLib.cliClient = realCliClient;
+  }
+});
+
+// ==========================================================================
+// STATIC — the surfaces a unit test cannot invoke (browser DOM strings, docs).
+// ==========================================================================
+
+// ---- 4. both paths send goal_id -------------------------------------------
+
+await test('BOTH PATHS: the CLI createTask and the hall payload each send goal_id', () => {
+  assert.match(TRIAGE, /if \(goalId != null\) body\.goal_id = Number\(goalId\);/,
+    'CLI: goal_id must reach the create body');
+  assert.match(TRIAGE, /async function createTask\(\{[^}]*goalId/,
+    'CLI: createTask must accept it rather than ignore the caller');
+  assert.match(HALL, /payload\.goal_id = Number\(form\.goal\.value\)/,
+    'hall: goal_id must reach the create payload');
+  assert.match(HALL, /data-goal-select/, 'hall: and there must be a picker to choose it from');
+});
+
+await test('BOTH PATHS: the stale "does not accept kind" note is corrected, not left to mislead', () => {
+  assert.ok(!/doesn't currently accept 'kind'/.test(TRIAGE),
+    'that claim was false — routes/tasks.js takes kind and the hall path sends it');
+  assert.match(TRIAGE, /no longer true/, 'and the correction says so explicitly');
+});
+
+// ---- 1. precedence: the stored hint wins ---------------------------------
+
+// (The CLI's precedence is proven by the BEHAVIOUR block above — it drives the
+// real function and asserts the suggester is never called on a hint. A source-shape
+// assertion here would only restate that, less reliably.)
+
+await test('PRECEDENCE: the hall picker marks the hint selected, and only auto-selects a suggestion without one', () => {
+  const fn = HALL.slice(HALL.indexOf('async function goalOptionsFor'), HALL.indexOf('async function renderPromote'));
+  assert.match(fn, /idea\.suggested_goal_id/, 'the hint is read');
+  assert.match(fn, /carried from the filing/, 'and labelled as inherited, not guessed');
+  assert.match(fn, /hint == null && i === 0 \? ' selected' : ''/,
+    'a suggestion is pre-selected ONLY when there is no hint to honour');
+});
+
+// ---- the suggester's weak-evidence signal is shown, not hidden -----------
+
+await test('the suggester default shows score + matched terms so it can be judged', () => {
+  assert.match(TRIAGE, /score \$\{top\.score\}/, 'CLI shows the score');
+  assert.match(TRIAGE, /matched_terms/, 'CLI shows why it matched');
+  assert.match(HALL, /matched: /, 'hall shows why it matched');
+  // task 1003062 documented that token overlap can match an incidental word; the
+  // visible score is what stops that being an invisible wrong default.
+  assert.match(HALL, /incidental word/, 'and the reason it is shown is written down');
+});
+
+// ---- 2. opting out is explicit ------------------------------------------
+
+await test('OPT-OUT: the prompt itself offers the way out, and the branch delegates the meaning', () => {
+  const branch = TRIAGE.slice(TRIAGE.indexOf('const gd = await resolveGoalDefault'), TRIAGE.indexOf('const task = await createTask'));
+  assert.match(branch, /general bucket/, 'the prompt has to say where opting out lands');
+  assert.match(branch, /applyGoalAnswer\(gd\.goalId, goalAns\)/,
+    'the branch must delegate to the tested helper, not re-implement the parsing inline');
+  assert.match(branch, /picked\.warn/, 'and surface its warning');
+  assert.match(branch, /resolveGoalDefault\(\{ \.\.\.idea, title \}, versionId\)/,
+    'the suggester must see the title the operator just confirmed, not the filed one');
+  // The semantics themselves are covered by the BEHAVIOUR block above — this
+  // only pins that main() is actually wired to them.
+});
+
+await test('OPT-OUT: the hall names where opting out actually lands, and only defaults to it last', () => {
+  const fn = HALL.slice(HALL.indexOf('async function goalOptionsFor'), HALL.indexOf('async function renderPromote'));
+  // "— no goal —" would be a lie: the server files a goal_id-less create under
+  // the version's catch-all. The label has to say so.
+  assert.match(fn, /general bucket/);
+  assert.ok(!/— no goal —/.test(fn), 'the honest label replaced the misleading one');
+  assert.match(fn, /opts\.length === 0 \? ' selected' : ''/, 'the opt-out is the fallback, not the default');
+  assert.match(fn, /a choice rather than the silent default/, 'and the intent is stated');
+});
+
+await test('the CLI result line names the goal, or says the task fell into the general bucket', () => {
+  assert.match(TRIAGE, /goal \$\{goalId\}` : ', → general bucket'/,
+    'where it landed must be visible in the output, both ways');
+});
+
+// ---- 3. degrades, never blocks -----------------------------------------
+
+await test('DEGRADES: a suggester outage yields no default rather than throwing', () => {
+  const fn = TRIAGE.slice(TRIAGE.indexOf('async function resolveGoalDefault'), TRIAGE.indexOf('async function createTask'));
+  assert.match(fn, /catch \(_\) \{\s*return \{ goalId: null, label: null, source: 'unavailable' \}/,
+    'the CLI must swallow it and report unavailable');
+  assert.match(TRIAGE, /goal suggester unreachable/, 'and tell the operator what happened');
+  const hallFn = HALL.slice(HALL.indexOf('async function goalOptionsFor'), HALL.indexOf('async function renderPromote'));
+  assert.match(hallFn, /catch \(_\) \{ suggestions = \[\]; \}/, 'the hall must degrade to no suggestions');
+  assert.match(hallFn, /must not stop a triage verdict/, 'the reasoning is recorded');
+});
+
+await test('the hall guards the stored hint against a junk value reaching the DOM', () => {
+  const fn = HALL.slice(HALL.indexOf('async function goalOptionsFor'), HALL.indexOf('async function renderPromote'));
+  assert.match(fn, /Number\.isFinite\(raw\) && raw > 0 \? raw : null/,
+    'value="NaN" would be POSTed as a goal id');
+});
+
+await test('the hall degrades a DEAD SESSION to no suggestions instead of navigating away', () => {
+  const fn = HALL.slice(HALL.indexOf('async function goalOptionsFor'), HALL.indexOf('async function renderPromote'));
+  // getJSON without softAuth calls location.assign on a 401 — from a background
+  // fetch that would discard whatever the builder had typed into the form.
+  assert.match(fn, /softAuth: true/);
+});
+
+await test('DEGRADES: the hall re-fetches goals when the version changes, newest-wins', () => {
+  // Two quick version changes race; an out-of-order resolve would paint another
+  // version's goals, all of which the server would reject.
+  assert.match(HALL, /var mine = \+\+goalFetch;/);
+  assert.match(HALL, /if \(mine !== goalFetch\) return;/);
+  // Goals belong to a version; keeping stale options would offer a home the
+  // server would reject.
+  assert.match(HALL, /vsel\.addEventListener\('change'/);
+  assert.match(HALL, /await goalOptionsFor\(vsel\.value\)/);
+});
+
+await test('the CLI uses the GENERATED typed client method, not the raw escape hatch', () => {
+  assert.match(TRIAGE, /api\.goals\.getGoalsSuggestions\(/,
+    'so a path or param rename reds the client build instead of failing at runtime');
+});
+
+// ---- the skill contract ------------------------------------------------
+
+await test('SKILL DOC: the inbox is declared homeless-only, with both consequences spelled out', () => {
+  assert.match(SKILL, /HOMELESS-ONLY/, 'the contract is stated');
+  assert.match(SKILL, /Every promote names a goal/, 'consequence 1');
+  assert.match(SKILL, /general` goal/, 'and it is honest about what opting out means');
+  assert.match(SKILL, /inherited, not guessed/, 'consequence 2 — precedence');
+  assert.match(SKILL, /0183/, 'and why a goal-less task now matters more than it used to');
+  assert.match(SKILL, /HOMELESS/i, 'the description line carries it too');
+});
+
+summary();