@bongos/core 1.21.24 → 1.21.26

package/.bongos-core.json

@@ -2,11 +2,11 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/gds/package-core.js",
-  "core_version": "1.21.24",
-  "core_contract": "1.21.24",
-  "source_commit": "9ed14f470c9244d95d36c136ddb7ab5353ce8578",
+  "core_version": "1.21.26",
+  "core_contract": "1.21.26",
+  "source_commit": "a0fe60e1a8c588d4e9913bcdffda9edae3267a4a",
   "source_ref": "HEAD",
-  "built_at": "2026-10-03T00:06:14.383Z",
+  "built_at": "2026-10-03T02:56:29.128Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 580,
@@ -17,7 +17,7 @@
     "gate": "passed"
   },
   "file_count": 3506,
-  "tree_sha256": "7a3a980adf2820c753d32bc3dcba9eb7b3b6471b6bf0a46db4a45b583fe2b888",
+  "tree_sha256": "bf858332458c31c371fdb0605eb745807f1f5ae5b4c42f85c6b015f7a7a36a4d",
   "files": [
     {
       "path": ".claude/skills/ask-for-help/SKILL.md",
@@ -192,7 +192,7 @@
     {
       "path": ".claude/skills/idea-triage/SKILL.md",
       "mode": "0000644",
-      "sha256": "d475e4bf8cb1b3a21f7778e5b036277a665cf497f604630160a2ec3eb5924b65"
+      "sha256": "628a9880309ba370151becbebd7497972006716ec955d06bb51a79d335dbd54f"
     },
     {
       "path": ".claude/skills/merge-mode/SKILL.md",
@@ -2832,7 +2832,7 @@
     {
       "path": "docs/module-api-changelog.md",
       "mode": "0000644",
-      "sha256": "92dde1422bdcca1dce6e857c712af4a8474a9140424681225445428783035bf8"
+      "sha256": "8b98f13ef9c22f6d73ab2065a1563410301cb24edcc3d83b81a34f453a814c6c"
     },
     {
       "path": "docs/modules-contract.md",
@@ -9787,12 +9787,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "0d93e33c8e66223f3566f9f89fc3b2f22d81942b74bdae8e8ab5db7c057c8f66"
+      "sha256": "b89ac1d6344bc2814145aee2e1a10552fb7bc4b24da346f4dcca305f47a355b7"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "ee3839fcdf18e5fd982b6a2d1f3bc55459125e13f27bfe19aef411556ef7006b"
+      "sha256": "13606c58a4cce9dbc00d018a3f3067e43d0b9d1ee553fb6069ed9b7493166659"
     },
     {
       "path": "public-docs/index.html",
@@ -9812,7 +9812,7 @@
     {
       "path": "release-notes.json",
       "mode": "0000644",
-      "sha256": "83458aeed8dc40040956f48a6670b30cea746cfbfcc62f6baf3db3201f537bf8"
+      "sha256": "270e78b9cc46481cf3aa2fb599bcbdc4bca8e4c0f5876371861e68610353f537"
     },
     {
       "path": "scripts/bongos-mcp.js",
@@ -11462,7 +11462,7 @@
     {
       "path": "scripts/gds/triage.js",
       "mode": "0000644",
-      "sha256": "a4af12591b829f9b89a5768858e8a2ba02a44d7e7ebd848e945e77f95fc55da7"
+      "sha256": "0e03060897a7d708ad54936d9942e130be4c2f22be01821ab98e7cf30ee4ef11"
     },
     {
       "path": "scripts/gds/turn-end-sound.js",
@@ -12037,7 +12037,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "278f3e5cee7b2d4b784aa79cdef9fe7570cc5bede2208dbe8c54f16095dba7f9"
+      "sha256": "91b122a9d3facf064489524a4ea1fa610466b29b855c98841de21c0e4c05e042"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -14717,7 +14717,7 @@
     {
       "path": "tests/idea_triage.mjs",
       "mode": "0000644",
-      "sha256": "836ee90dd24736c84fede53394d0d724915f73540c9b99c41762da1d37c447b2"
+      "sha256": "6960c9628f6b41ef78db6d09bcbb00ca4afb6b93c4bf57a6c39e86f6e09c4c66"
     },
     {
       "path": "tests/idea_triage_grade.mjs",

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

@@ -58,7 +58,7 @@ Bug reports from Discord have their own channel, `#bugs`, which does route.
 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.
+> **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.5b) — 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
 
@@ -119,7 +119,7 @@ show the same idea the same way.
 
 **Step 1.5a — batch near-duplicate ideas BEFORE developing anything (task 1001465).**
 
-Run `clusterIdeas` (`scripts/gds/triage.js` — pure, no I/O): it groups open quick/spark ideas that share a `kind` and a high title/body word overlap. For each cluster it returns, offer the user ONE decision covering the whole group instead of walking each member — this is the actual fix for "triage costs a ton of credits" (idea 377): a backlog of near-identical asks used to cost one full development + verdict PER idea, and now costs one:
+Run `clusterIdeas` (`scripts/gds/triage.js` — pure, no I/O): it groups open quick ideas (not Full Ideas, not sparks) that share a `kind` and a high title/body word overlap. For each cluster it returns, offer the user ONE decision covering the whole group instead of walking each member — this is the actual fix for "triage costs a ton of credits" (idea 377): a backlog of near-identical asks used to cost one full development + verdict PER idea, and now costs one:
 
 > Cluster of 3 likely duplicates: #101 "Add dark mode toggle", #145 "Dark mode support", #203 "Support a dark theme". **Keep #101 and merge the rest, discard all, walk them individually, or skip?**
 
@@ -165,7 +165,7 @@ Present each idea with:
 - title
 - body_md (full)
 - captured_at + captured_by
-- **its developed block** (from Step 1.5): summary, integration surfaces, open questions, challenges, risks, rough effort, and the subagent's recommendation
+- **its developed block** (from Step 1.5b): summary, integration surfaces, open questions, challenges, risks, rough effort, and the subagent's recommendation
 
 Then ask the user (concise prompt — Lars is an EE, not a software engineer; frame in product/outcome terms). Lead with the developed open questions, since those are what the Archon actually needs to decide:
 

package/docs/module-api-changelog.md

@@ -2875,5 +2875,9 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.21.22 with no explicit bump. run 37078725783. (task 1002620)
   1.21.24 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         landed since 1.21.23 with no explicit bump. run 37080562697. (task 1002620)
+  1.21.25 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.21.24 with no explicit bump. run 37089254828. (task 1002620)
+  1.21.26 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.21.25 with no explicit bump. run 37091481670. (task 1002620)
 ---------------------------------------------------------------------------
 ```

package/package-lock.json

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

package/package.json

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

package/release-notes.json

@@ -8900,5 +8900,17 @@
       "id": "1002830",
       "text": "Sessions now has a Read button on each row that opens the full conversation for that session. Only the session's owner or an administrator can read it, and administrator reads are logged."
     }
+  ],
+  "1.21.25": [
+    {
+      "id": "1004533",
+      "text": "Twelve goal 1000045 tasks that were counted toward other goals' criteria now count toward their own, so that goal's progress reads true."
+    }
+  ],
+  "1.21.26": [
+    {
+      "id": "1003623",
+      "text": "Fixed idea-triage cluster handling: 'keep #N' and 'merge #N' now parse; skipped clusters count toward the summary; spark ideas excluded from batch clustering; stale Step 1.5 refs now point at 1.5b. Tests added in tests/idea_tr"
+    }
   ]
 }

package/scripts/gds/triage.js

@@ -400,10 +400,12 @@ function clusterRecords(records, { textOf, kindOf, timeOf, threshold = 0.34, min
 
 // The idea-shaped call: Full Ideas are never eligible (they get their own
 // shape recommendation, SKILL.md Step 2.5, and batching one away would skip
-// that judgment), clustered on title+body_md / kind / captured_at. Same
-// signature and behavior as before the clusterRecords extraction.
+// that judgment), nor are sparks (not triage's to convert, SKILL.md); the rest
+// cluster on title+body_md / kind / captured_at. Same signature as before the
+// clusterRecords extraction.
+const NON_CLUSTER_GRADES = new Set(['full', 'spark']);
 function clusterIdeas(ideas, { threshold = 0.34, minShared = 2 } = {}) {
-  const eligible = ideas.filter((i) => (i.grade || 'quick') !== 'full');
+  const eligible = ideas.filter((i) => !NON_CLUSTER_GRADES.has(i.grade || 'quick'));
   return clusterRecords(eligible, {
     textOf: (i) => `${i.title || ''} ${i.body_md || ''}`,
     kindOf: (i) => i.kind || null,
@@ -431,7 +433,7 @@ async function handleCluster(rl, cluster, clusteredIds, state) {
   while (!handled) {
     const ans = (await prompt(rl, `  [k]eep #${cluster[0].id} & merge the rest / [d]iscard all / [i]ndividually / [s]kip cluster > `)).trim().toLowerCase();
 
-    if (ans === 's' || ans === 'skip' || ans === '') { handled = true; break; }
+    if (ans === 's' || ans === 'skip' || ans === '') { state.summary.skipped += cluster.length; handled = true; break; }
 
     if (ans === 'i' || ans === 'individually') {
       for (const m of cluster) clusteredIds.delete(m.id);
@@ -458,7 +460,7 @@ async function handleCluster(rl, cluster, clusteredIds, state) {
       handled = true; break;
     }
 
-    if (ans === 'k' || ans === 'keep' || ans.startsWith('k ') || ans.startsWith('k#')) {
+    if (/^k(eep)?(\s|#|$)/.test(ans)) {
       let canonicalId = cluster[0].id;
       const inline = ans.replace(/^k(eep)?(\s|#)*/, '').trim();
       if (inline) {
@@ -535,6 +537,9 @@ async function main() {
       await handleCluster(rl, cluster, clusteredIds, state);
     }
   }
+  // Cluster verdicts bump the summary too, but those ideas are already out of
+  // `ideas`; remember the tally so "unseen" only counts the individual walk.
+  const clusterHandled = summary.promoted + summary.discarded + summary.merged + summary.skipped;
   ideas = ideas.filter((i) => !clusteredIds.has(i.id));
   if (ideas.length === 0) {
     console.log('\nnothing left to walk individually — every open idea was resolved in a batch cluster verdict.');
@@ -593,10 +598,10 @@ async function main() {
           break;
         }
 
-        if (ans === 'm' || ans === 'merge' || ans.startsWith('m ') || ans.startsWith('m#')) {
+        if (/^m(erge)?(\s|#|$)/.test(ans)) {
           // Accept "m 12" or "m #12" inline, otherwise prompt.
           let target = null;
-          const inline = ans.replace(/^m(\s|#)*/, '').trim();
+          const inline = ans.replace(/^m(erge)?(\s|#)*/, '').trim();
           if (inline) target = Number(inline.replace(/^#/, ''));
           if (!Number.isFinite(target) || target <= 0) {
             const a = (await prompt(rl, '  merge into idea #: ')).trim().replace(/^#/, '');
@@ -700,7 +705,7 @@ async function main() {
     rl.close();
   }
 
-  const walked = summary.promoted + summary.discarded + summary.merged + summary.skipped;
+  const walked = summary.promoted + summary.discarded + summary.merged + summary.skipped - clusterHandled;
   const remaining = ideas.length - walked;
 
   console.log('\n' + '═'.repeat(72));
@@ -730,6 +735,7 @@ module.exports = {
   loadDevelopment,
   formatDevelopment,
   clusterIdeas,
+  handleCluster,
   clusterRecords,
   significantWords,
   jaccard,

package/src/module-api.js

@@ -75,7 +75,7 @@ const { responsibilityFor, ROLE_RESPONSIBILITIES } = require('./role-responsibil
 // MAJOR (see allowBoxScope below): passes the request through untouched.
 function deprecatedNoopMiddleware(_req, _res, next) { next(); }
 
-const CORE_VERSION = '1.21.24'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.21.26'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
 
 // A namespaced logger so a module's log lines are attributable + consistent.
 // Usage: const log = api.logger('discord'); log.info('mounted');

package/tests/idea_triage.mjs

@@ -18,6 +18,7 @@
 // Run:  node tests/idea_triage.mjs
 
 import { strict as assert } from 'node:assert';
+import fs from 'node:fs';
 import { createRequire } from 'node:module';
 
 const require = createRequire(import.meta.url);
@@ -162,5 +163,44 @@ await test('listOpenIdeas selects the triage-provenance columns', async () => {
   assert.match(select.sql, /triage_note/);
 });
 
+// ── task 1003623: batch-cluster follow-ups ─────────────────────────────────
+const triage = require('../scripts/gds/triage.js');
+const fakeRl = (answers) => {
+  const asked = [];
+  return { asked, question: (q, cb) => { asked.push(q); cb(answers.shift() ?? 's'); } };
+};
+const pair = [
+  { id: 1, title: 'a', kind: 'bug' },
+  { id: 2, title: 'b', kind: 'bug' },
+];
+const freshState = () => ({ summary: { promoted: 0, discarded: 0, merged: 0, skipped: 0 }, discardedIds: [], mergedIds: [] });
+
+await test('cluster verdict accepts the natural "keep #N" phrasing', async () => {
+  // A one-member cluster keeps the verdict free of API writes: nothing to merge.
+  const rl = fakeRl(['keep #1', 's']);
+  const state = freshState();
+  await triage.handleCluster(rl, [pair[0]], new Set([1]), state);
+  assert.equal(rl.asked.length, 1, 'keep #1 was understood, not re-prompted as unknown');
+  assert.equal(state.summary.skipped, 0);
+});
+
+await test('skipping a cluster counts every member as skipped', async () => {
+  const rl = fakeRl(['s']);
+  const state = freshState();
+  await triage.handleCluster(rl, pair, new Set([1, 2]), state);
+  assert.equal(state.summary.skipped, 2);
+});
+
+await test('spark-grade ideas are not eligible for batch clustering', async () => {
+  const mk = (id, grade) => ({ id, grade, kind: 'idea', title: 'dashboard filter export widget', body_md: 'dashboard filter export widget', captured_at: '2026-01-01' });
+  assert.equal(triage.clusterIdeas([mk(1, 'spark'), mk(2, 'spark')]).clusters.length, 0);
+  assert.equal(triage.clusterIdeas([mk(1, 'quick'), mk(2, 'quick')]).clusters.length, 1);
+});
+
+await test('SKILL.md has no stale bare "Step 1.5" cross-references', async () => {
+  const md = fs.readFileSync(new URL('../.claude/skills/idea-triage/SKILL.md', import.meta.url), 'utf8');
+  assert.ok(!/Step 1\.5(?![ab])/.test(md));
+});
+
 console.log(`\nIdea triage tests: ${passed} passed, ${failed} failed`);
 process.exit(failed === 0 ? 0 : 1);