@bongos/core 1.19.225 → 1.19.226

package/.bongos-core.json

@@ -2,11 +2,11 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/gds/package-core.js",
-  "core_version": "1.19.225",
-  "core_contract": "1.19.225",
-  "source_commit": "1f495d4acc6813ee9a5cb936244452e840b7fde3",
+  "core_version": "1.19.226",
+  "core_contract": "1.19.226",
+  "source_commit": "b99c0b52cf19ffd33ad98d0fc3dd763011559401",
   "source_ref": "HEAD",
-  "built_at": "2026-08-18T04:37:38.138Z",
+  "built_at": "2026-08-18T04:50:04.434Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 311,
@@ -17,7 +17,7 @@
     "gate": "passed"
   },
   "file_count": 1640,
-  "tree_sha256": "2b8fa57da152a118479d80755587e79a98aff9c540845309b0548b2b6573fadd",
+  "tree_sha256": "024eb9d71d1dd27389cb4db58a347eadfad7412b37e48d140a9c113cc8ee9234",
   "files": [
     {
       "path": ".claude/skills/blocker-review/SKILL.md",
@@ -3517,7 +3517,7 @@
     {
       "path": "modules/ideas/completeness.js",
       "mode": "0000644",
-      "sha256": "84d977372c79a46f8481176b7cdc740cbaca9480af3464b7b66183aad6d6e180"
+      "sha256": "07e5666362c91f92d34388b4f6f4507cc5387866d317970478e6cae417c19975"
     },
     {
       "path": "modules/ideas/inbox.js",
@@ -3547,7 +3547,7 @@
     {
       "path": "modules/ideas/routes/inbox.js",
       "mode": "0000644",
-      "sha256": "e6dd2d295186a081592009ae0b2341a477bdcc71c8b0cf10747f22a811f4683c"
+      "sha256": "9e9288725b66b696bdc565e693c82f046bd78665ae070e3f571ff987f82a7b7b"
     },
     {
       "path": "modules/ideas/templates.js",
@@ -4302,12 +4302,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "4dbb837e40ad8c1e5d1ee8085874c70f618899982e417ac8645234e072d3e04b"
+      "sha256": "93e2a0f8b239aa724d99178fa5cfcd7794d7a09c9e820a9ba3f8da94a81ec2b0"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "02009e59987400a1715dd4a03a0ec172006a633fd106ab0b188fec705e5fc522"
+      "sha256": "378978670cd1efdcc3929f44178118e321f8e381bb923aff175716af2b24134d"
     },
     {
       "path": "public-docs/index.html",
@@ -5922,7 +5922,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "605f8e482d11f3e2c764a76109b3f63d4a4781630d03c44b7ebd3dd2077f389a"
+      "sha256": "d7a3216e0393ff4a4cb0fe3c7823b9c8a294ff45549a4a698b95c78ea93e4ad8"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -7077,7 +7077,7 @@
     {
       "path": "tests/idea_completeness.mjs",
       "mode": "0000644",
-      "sha256": "e7b6af463185ea77f40d62408026c55c7fb077f18999b972564b3b5bc7656ca5"
+      "sha256": "7e2024b3f1fa79a6bdec55b134be6337d4867df0a77fedb205f32b783a3d6193"
     },
     {
       "path": "tests/idea_full_capture.mjs",

package/modules/ideas/completeness.js

@@ -69,4 +69,47 @@ function completenessScore(fieldValues, opts) {
   return scoreFieldValues(fieldValues, opts).score;
 }
 
-module.exports = { completenessScore, scoreFieldValues, substanceOf };
+// ── the sub-bar rejection payload (BV1.R09 of goal 1000069, ADR 0175 §4) ────
+//
+// Criterion C2's other half: an idea that never reaches the board (sub-bar,
+// ADR 0175 §5) still returns to its author WITH A REASON THEY CAN ACT ON. The
+// reason is computed, not written: the per-field breakdown above already knows
+// which section carried the least substance, so the weakest section is flagged
+// automatically — same shape as a board objection (reason_md + section_key,
+// government_003's vote payload), so R13's return flow renders one thing.
+//
+// DETERMINISTIC AND DERIVED, NEVER STORED: the scorer is deterministic, so the
+// flag is recomputable from field_values at any time — storing it would be a
+// second copy that drifts. Ties go to TEMPLATE ORDER (the fields array is
+// already in it), so the flag is stable for identical inputs.
+//
+// BAR-AGNOSTIC BY DESIGN: whether a score is sub-bar is decided where the bar
+// lives (economy's IDEA_FULL_PASS_BAR / the board's gate) — this constructs the
+// payload once a caller has decided. Passing an unscorable input returns null,
+// the same "cannot score" posture as everything above.
+
+function weakestField(fieldValues, opts) {
+  const { fields } = scoreFieldValues(fieldValues, opts);
+  if (!fields.length) return null;
+  return fields.reduce((worst, f) => (f.ratio < worst.ratio ? f : worst));
+}
+
+// The rejection payload for a sub-bar Full Idea: { section_key, reason_md } in
+// the board-objection shape. The reason names the section by its TEMPLATE LABEL
+// (the words the author saw when writing) and states the shortfall in the
+// scorer's own honest terms — substance against the floor, nothing smarter.
+function weakestSectionPayload(fieldValues, { templateKey = DEFAULT_TEMPLATE_KEY, registry = BUILTIN_TEMPLATES } = {}) {
+  const worst = weakestField(fieldValues, { templateKey, registry });
+  if (!worst) return null;
+  const template = templateFor(templateKey, registry);
+  const field = template.fields.find((f) => f.key === worst.key);
+  const label = field ? field.label : worst.key;
+  return {
+    section_key: worst.key,
+    reason_md: `Completeness is below the bar, and "${label}" is the thinnest section — `
+      + `${worst.substance} character${worst.substance === 1 ? '' : 's'} of substance against a floor of ${worst.floor}. `
+      + 'Flesh that section out and resubmit; the score is deterministic, so the same words always score the same.',
+  };
+}
+
+module.exports = { completenessScore, scoreFieldValues, substanceOf, weakestField, weakestSectionPayload };

package/modules/ideas/routes/inbox.js

@@ -7,7 +7,7 @@ const inbox = require('../inbox');
 // Full Idea capture (task 1002919 / BV1.R143): the template validator (R141)
 // and the deterministic scorer (R142) — both pure, both walk the declaration.
 const { validateFieldValues } = require('../templates');
-const { completenessScore } = require('../completeness');
+const { completenessScore, weakestSectionPayload } = require('../completeness');
 const { validateOrRespond, LIMITS, parseId } = api;
 
 const INBOX_STATUS_VALUES = ['open', 'promoted', 'discarded', 'merged'];
@@ -166,6 +166,17 @@ module.exports = function buildInboxRouter() {
               });
               if (w && w.opened && w.item) {
                 idea.board_window = { item_id: w.item.id, closes_at: w.item.closes_at };
+              } else if (w && w.reason === 'below_bar') {
+                // R09 (goal 1000069, criterion C2): a sub-bar idea returns to
+                // its author WITH A REASON THEY CAN ACT ON — the weakest section
+                // auto-flagged from the per-field scores, in the same shape as a
+                // board objection. Riding the BOARD'S OWN below_bar answer keeps
+                // the bar in exactly one place (board.js); this module never
+                // decides what "sub-bar" means. Derived, not stored — the scorer
+                // is deterministic, so R13's return flow recomputes the same
+                // flag from field_values whenever it needs it.
+                const flag = weakestSectionPayload(body.field_values, templateKey ? { templateKey } : undefined);
+                if (flag) idea.rejection = flag;
               }
             }
           } catch (boardErr) {

package/package-lock.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.225",
+  "version": "1.19.226",
   "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

@@ -877,8 +877,10 @@ const { validateOrRespond, LIMITS, parseId, asyncHandler, corsPublicGet, parsePa
 //         landed since 1.19.223 with no explicit bump. run 32098083321. (task 1002620)
 //   1.19.225 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
 //         landed since 1.19.224 with no explicit bump. run 32099793591. (task 1002620)
+//   1.19.226 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+//         landed since 1.19.225 with no explicit bump. run 32100543772. (task 1002620)
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.225'; // CI auto-patch carrier (ADR 0161); see changelog above
+const CORE_VERSION = '1.19.226'; // 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/idea_completeness.mjs

@@ -94,5 +94,35 @@ test('the scorer walks the declaration — a different template scores by ITS fi
   assert.equal(C.completenessScore(COMPLETE, opts), 0);
 });
 
+// ── the sub-bar rejection payload (BV1.R09 of goal 1000069, criterion C2) ───
+
+test('the weakest section is the LOWEST ratio, with ties going to template order', () => {
+  const values = { ...COMPLETE, big_picture: 'thin', why_matters: 'thin' }; // two equally thin
+  const worst = C.weakestField(values);
+  assert.equal(worst.key, 'big_picture', 'the earlier of two equal-thinnest fields, by template order');
+  // strictly-thinnest wins regardless of position
+  assert.equal(C.weakestField({ ...COMPLETE, fit: 'x' }).key, 'fit');
+});
+
+test('the rejection payload is board-objection-shaped, names the template LABEL, and states the shortfall', () => {
+  const flag = C.weakestSectionPayload({ ...COMPLETE, fit: 'too thin' });
+  assert.equal(flag.section_key, 'fit', 'exactly one of the five section keys — the board-vote shape');
+  assert.ok(flag.reason_md.includes('How does the Idea Fit'), 'the reason speaks the words the author saw');
+  assert.ok(/\d+ characters? of substance against a floor of \d+/.test(flag.reason_md),
+    'the shortfall is stated in the scorer\'s own honest terms');
+  const field = BASE_TEMPLATE.fields.find((f) => f.key === 'fit');
+  assert.ok(flag.reason_md.includes(String(field.substanceMin)), 'the floor named is the template\'s real one');
+});
+
+test('the payload is DERIVED, deterministic, and null on unscorable input — never stored, never throws', () => {
+  const values = { ...COMPLETE, purpose: 'meh' };
+  assert.deepEqual(C.weakestSectionPayload(values), C.weakestSectionPayload(values),
+    'same words, same flag — recomputable whenever R13 needs it, so storing it would only drift');
+  for (const bad of [null, undefined, 42, [], 'text']) {
+    assert.equal(C.weakestSectionPayload(bad), null, `${String(bad)} cannot be flagged`);
+  }
+  assert.equal(C.weakestSectionPayload({}, { templateKey: 'no-such-template' }), null);
+});
+
 console.log(`\nidea_completeness: ${passed} passed, ${failed} failed`);
 if (failed > 0) process.exit(1);