@bongos/core 1.19.934 → 1.19.935

package/.bongos-core.json

@@ -2,11 +2,11 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/gds/package-core.js",
-  "core_version": "1.19.934",
-  "core_contract": "1.19.934",
-  "source_commit": "6054b55eb2141143f4d11a42b1bda4268b8b5cd8",
+  "core_version": "1.19.935",
+  "core_contract": "1.19.935",
+  "source_commit": "b3b2e4e86881e90300f7b23d75f5e1508eb2bf81",
   "source_ref": "HEAD",
-  "built_at": "2026-09-21T03:13:03.324Z",
+  "built_at": "2026-09-21T03:42:26.193Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 511,
@@ -17,7 +17,7 @@
     "gate": "passed"
   },
   "file_count": 2864,
-  "tree_sha256": "d88bd8ff77498c91b75ca7ae50c9789681f652af911d6af41c1534ee4099b67c",
+  "tree_sha256": "e3c4fa4ce99958763ab765d6846942c8fad965add8df64e1f54a7e375806c926",
   "files": [
     {
       "path": ".claude/skills/ask-for-help/SKILL.md",
@@ -2677,7 +2677,7 @@
     {
       "path": "docs/module-api-changelog.md",
       "mode": "0000644",
-      "sha256": "58d9600e5e86ec9ff6ac1f3e585b1a8935d26372c16909d96b9129de68b91554"
+      "sha256": "4c9bccb55d99dc3c2588b73ae95d365511eb433e27265822725aeed38420992b"
     },
     {
       "path": "docs/modules-contract.md",
@@ -5557,7 +5557,7 @@
     {
       "path": "modules/ideas/templates.js",
       "mode": "0000644",
-      "sha256": "3bbec1061e010f8e35e04db89b5cf0f08d1c1512a6ef2c7a1a0ce8c4828c7ec7"
+      "sha256": "deef66d514a2ea090274acc05e85ffefdfd6c0bf0279b64f7bee16af7b2e1471"
     },
     {
       "path": "modules/ideas/trail-descent.js",
@@ -7977,12 +7977,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "c7cfc9418097ef5d95196a983c18f530596de47a261f713a224fd44a14a0c735"
+      "sha256": "cc78b4e84d90eba27a329d88ae83b507df318962b057d9281c3cc1b858063e05"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "0cb7773b2dc025a6341b3461a6abb85dbff149e246f509b725bd362ea11d61dd"
+      "sha256": "12f2fd99b6ac30b3bd8a443f576cf083c97eee82623b4392b714b107ab0c666b"
     },
     {
       "path": "public-docs/index.html",
@@ -9982,7 +9982,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "0cf2ed963899d5343c619126aac9193ce576dcb47c334f6124c85400a36eb0d1"
+      "sha256": "fe49958cdbfc0121ca262cd95b1fef21fc3028aa5da19250fb4bab2e68e42ccd"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -12017,7 +12017,7 @@
     {
       "path": "tests/idea_completeness.mjs",
       "mode": "0000644",
-      "sha256": "a7d6e79e240f7397109afc2522a4d3a255154149b3499581de7b22cfe7761f47"
+      "sha256": "cbbe8fa2dbf3c593b322641caa1550bd1805089669ef2658c4c92df3c9cdc6f6"
     },
     {
       "path": "tests/idea_credit_split.mjs",
@@ -12052,7 +12052,7 @@
     {
       "path": "tests/idea_full_capture.mjs",
       "mode": "0000644",
-      "sha256": "95b53f1acfa61f723741b4d33b5e882525903ccc549ae01f3094a788251a60ec"
+      "sha256": "a5d507f3fac77d70904ae0c9c9c49feb4c5eed26889e69f931fdf15fb9959960"
     },
     {
       "path": "tests/idea_full_pass_credit.mjs",
@@ -12147,7 +12147,7 @@
     {
       "path": "tests/idea_templates.mjs",
       "mode": "0000644",
-      "sha256": "8b21073cc2a766c749ca5687767c8c8fc6da6bf1149160f57d239d9163d0a1a6"
+      "sha256": "7e1dd5417391cacd137967c388ba2f086b75ec5aca3fbb2703b3a5eb2ec2313d"
     },
     {
       "path": "tests/idea_trail_descent.mjs",

package/docs/module-api-changelog.md

@@ -2342,5 +2342,7 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.19.932 with no explicit bump. run 35555853466. (task 1002620)
   1.19.934 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         landed since 1.19.933 with no explicit bump. run 35556739619. (task 1002620)
+  1.19.935 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.19.934 with no explicit bump. run 35558269194. (task 1002620)
 ---------------------------------------------------------------------------
 ```

package/modules/ideas/templates.js

@@ -82,20 +82,48 @@ const BASE_V1_TEMPLATE = Object.freeze({
 // the v1 label. `purpose`, `big_picture` and `why_matters` are NOT carried over —
 // they belong to v1 and stay there.
 //
-// THE FLOORS ARE FLAT 40, MATCHING v1, AND THAT IS TEMPORARY. The new questions
-// are not interchangeable ("scope it as best as you can" deserves more room than
-// "what made you have the idea"), but the score feeds credits, so choosing the
-// numbers is its own deliberate act — task 1004091 (BV2.R12) owns it. Shipping a
-// guess here would have been an unreviewed change to what a thought pays.
+// THE FLOORS, CHOSEN PER FIELD (task 1004091 / BV2.R12, spec R21). v2 shipped at
+// a placeholder flat 40 because picking the numbers is its own deliberate act:
+// `completeness_score` is the mean of per-field ratios, and that number gates
+// both the ideator credit (economy's IDEA_FULL_PASS_BAR = 60) and whether a Full
+// Idea reaches the board at all (government's FULL_IDEA_PASS_BAR). A floor is not
+// a style note; it is a price.
+//
+// THE TOTAL IS HELD AT 200, EXACTLY v1's FIVE × 40. What a thought costs to file
+// is an economic decision the owner has already made once; this task was asked to
+// distribute that cost across questions that are not interchangeable, not to
+// re-price filing. Redistributing inside a fixed total is the change that needs no
+// owner ruling. (It also keeps the template clear of ADR 0186's interim karma-
+// eligibility guard — ≥3 required fields and ≥120 total `substanceMin`.)
+//
+// WHY EACH NUMBER, IN ONE LINE EACH. The rule is how much text the question
+// HONESTLY takes, never how much we would like to receive:
+//   why_had 25       — the anti-ghost-writing field, and the shortest honest
+//                      answer is a sentence. Nobody but the filer can say what
+//                      made them have the idea, so this is precisely the floor
+//                      that must NOT push an agent to pad; the never-invent-a-
+//                      field rule (the Ideator pack) is strongest here.
+//   scope 65         — "as best as you can" is an invitation to the longest
+//                      answer in the set, and this field is the one that becomes
+//                      the work. The highest floor in the template.
+//   build_context 45 — a prior-knowledge dump: real but bounded, and legitimately
+//                      short when there genuinely is little to know.
+//   fit 30           — an argument, and a good one is compact. Deliberately low:
+//                      a long justification is usually a weak one.
+//   success 35       — a concrete picture of the finished thing; a few sentences.
+//
+// EACH RATIO IS CAPPED AT 1, SO THE TOTAL IS NOT A BUDGET. A 600-character scope
+// answer cannot buy a blank `why_had`; the floors bind field by field, which is
+// the whole reason they differ.
 const BASE_V2_TEMPLATE = Object.freeze({
   key: 'base-v2',
   label: 'Full Idea',
   fields: Object.freeze([
-    Object.freeze({ key: 'why_had',       label: 'What made you have the idea',                                required: true, substanceMin: 40 }),
-    Object.freeze({ key: 'scope',         label: 'Scope it as best as you can here',                           required: true, substanceMin: 40 }),
-    Object.freeze({ key: 'build_context', label: 'What should an engineer or artist know prior to building',    required: true, substanceMin: 40 }),
-    Object.freeze({ key: 'fit',           label: 'How is this long term justified in the project',              required: true, substanceMin: 40 }),
-    Object.freeze({ key: 'success',       label: 'What does a successful implementation of the idea look like', required: true, substanceMin: 40 }),
+    Object.freeze({ key: 'why_had',       label: 'What made you have the idea',                                required: true, substanceMin: 25 }),
+    Object.freeze({ key: 'scope',         label: 'Scope it as best as you can here',                           required: true, substanceMin: 65 }),
+    Object.freeze({ key: 'build_context', label: 'What should an engineer or artist know prior to building',    required: true, substanceMin: 45 }),
+    Object.freeze({ key: 'fit',           label: 'How is this long term justified in the project',              required: true, substanceMin: 30 }),
+    Object.freeze({ key: 'success',       label: 'What does a successful implementation of the idea look like', required: true, substanceMin: 35 }),
   ]),
 });
 

package/package-lock.json

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

package/package.json

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

@@ -71,7 +71,7 @@ const { responsibilityFor, ROLE_RESPONSIBILITIES } = require('./role-responsibil
 // there. scripts/gds/bump-version.js still rewrites the literal below; it appends
 // the entry to that file. Look for a version's history there, not here.
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.934'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.19.935'; // 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('dev-box'); log.info('mounted');

package/tests/idea_completeness.mjs

@@ -40,6 +40,11 @@ const COMPLETE = Object.fromEntries(
 // the current template's keys is quietly asserting the template never changes.
 const [FIELD_1, FIELD_2, , FIELD_4] = BASE_TEMPLATE.fields.map((f) => f.key);
 
+// Since task 1004091 (BV2.R12) the floors DIFFER per field, so a fixture may no
+// longer assume "40" — or, worse, assume that equal text in two sections is equal
+// credit. Every expectation below is derived from the declaration.
+const floorOf = (key) => BASE_TEMPLATE.fields.find((f) => f.key === key).substanceMin;
+
 test('a complete idea scores 100, an empty one 0', () => {
   assert.equal(C.completenessScore(COMPLETE), 100);
   assert.equal(C.completenessScore({}), 0);
@@ -51,13 +56,25 @@ test('same input, same score — byte-for-byte determinism', () => {
   assert.deepEqual(C.scoreFieldValues(COMPLETE), C.scoreFieldValues(COMPLETE));
 });
 
-test('a thin answer earns its proportion, not the full share', () => {
-  const oneThin = { ...COMPLETE, [FIELD_1]: 'ten chars.' }; // 10 of the 40-char floor
+test('a thin answer earns its proportion of ITS OWN floor, not a flat share', () => {
+  const TEN = 'ten chars.'; // exactly 10 characters of substance
+  const oneThin = { ...COMPLETE, [FIELD_1]: TEN };
   const s = C.completenessScore(oneThin);
-  assert.ok(s > 80 && s < 100, `four full + one quarter-credit field ≈ 85, got ${s}`);
+  assert.ok(s > 80 && s < 100, `four full fields + one part-credit field, got ${s}`);
   const breakdown = C.scoreFieldValues(oneThin).fields.find((f) => f.key === FIELD_1);
   assert.equal(breakdown.substance, 10);
-  assert.equal(breakdown.ratio, 0.25);
+  assert.equal(breakdown.floor, floorOf(FIELD_1), 'the breakdown publishes the field\'s own floor');
+  assert.equal(breakdown.ratio, 10 / floorOf(FIELD_1));
+
+  // BV2.R12's actual deliverable: the same ten characters are worth DIFFERENT
+  // credit in different sections, because the questions are not interchangeable.
+  // A test that only ever looked at one field would pass just as happily against
+  // the flat-40 placeholder this task replaced.
+  const other = C.scoreFieldValues({ ...COMPLETE, [FIELD_2]: TEN })
+    .fields.find((f) => f.key === FIELD_2);
+  assert.equal(other.ratio, 10 / floorOf(FIELD_2));
+  assert.notEqual(other.ratio, breakdown.ratio,
+    'identical text scores lower where the question asks for more');
 });
 
 test('padding earns exactly nothing: whitespace and blank lines are not substance', () => {
@@ -111,7 +128,16 @@ test('the scorer walks the declaration — a different template scores by ITS fi
 // ── 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, [FIELD_2]: 'thin', [FIELD_4]: 'thin' }; // two equally thin
+  // A GENUINE tie takes different amounts of text in each field now that the
+  // floors differ (BV2.R12): two-fifths of each field's OWN floor is the same
+  // ratio, where two identical strings would not be. The fixture asserts it is
+  // really a tie before asserting how the tie breaks — otherwise this case
+  // quietly stops testing tie-breaking at all.
+  const twoFifths = (key) => 'x'.repeat(Math.round(floorOf(key) * 0.4));
+  const values = { ...COMPLETE, [FIELD_2]: twoFifths(FIELD_2), [FIELD_4]: twoFifths(FIELD_4) };
+  const broken = C.scoreFieldValues(values).fields;
+  assert.equal(broken.find((f) => f.key === FIELD_2).ratio, broken.find((f) => f.key === FIELD_4).ratio,
+    'the fixture really is a tie');
   const worst = C.weakestField(values);
   assert.equal(worst.key, FIELD_2, 'the earlier of two equal-thinnest fields, by template order');
   // strictly-thinnest wins regardless of position

package/tests/idea_full_capture.mjs

@@ -61,8 +61,14 @@ function call(bodyIn) {
   });
 }
 
+// Since task 1004091 (BV2.R12) the floors differ per field, so one fixed
+// sentence no longer clears every one of them — this suite wants a payload that
+// scores 100, so each answer is padded to the field's OWN declared floor.
 const FULL = Object.fromEntries(
-  BASE_TEMPLATE.fields.map((f) => [f.key, `A substantive answer for ${f.key}, long enough to clear the floor.`]),
+  BASE_TEMPLATE.fields.map((f) => [
+    f.key,
+    `A substantive answer for ${f.key}, long enough to clear the floor.`.padEnd(f.substanceMin, '.'),
+  ]),
 );
 
 let passed = 0;

package/tests/idea_templates.mjs

@@ -74,6 +74,40 @@ test('base-v2 declares the owner\'s new five, in order, and IS the store default
   }
 });
 
+// ── the substance floors (task 1004091, BV2.R12, spec R21) ──────────────────
+//
+// The floors are a PRICE, not a style note: `completeness_score` is the mean of
+// per-field ratios, and that number gates the ideator credit (economy's
+// IDEA_FULL_PASS_BAR) and whether a Full Idea reaches the board at all. So the
+// two claims worth pinning are the two that would be silently reverted: that the
+// floors actually differ per question, and that redistributing them did not
+// change what filing a Full Idea costs in total.
+
+test('BV2.R12: v2\'s floors differ per question, and total exactly what v1 cost', () => {
+  const total = (t) => t.fields.reduce((sum, f) => sum + f.substanceMin, 0);
+  assert.equal(total(T.BASE_V2_TEMPLATE), total(T.BASE_V1_TEMPLATE),
+    're-pricing what a thought costs to file is the owner\'s call, not this template\'s — '
+    + 'R12 distributes v1\'s total across questions that are not interchangeable, it does not move it');
+
+  const floors = T.BASE_V2_TEMPLATE.fields.map((f) => f.substanceMin);
+  assert.ok(new Set(floors).size > 1,
+    'flat floors are the placeholder BV2.R12 replaced — the questions are not interchangeable');
+
+  // Named by key, because the REASONING is what a future edit would undo: the
+  // field that invites the longest answer carries the highest floor, and the
+  // anti-ghost-writing field — the one nobody but the filer can answer — carries
+  // the lowest, so no floor ever pushes an agent to pad it (the Ideator pack).
+  const floorOf = (key) => T.BASE_V2_TEMPLATE.fields.find((f) => f.key === key).substanceMin;
+  assert.equal(Math.max(...floors), floorOf('scope'), '"scope it as best as you can" asks for the most');
+  assert.equal(Math.min(...floors), floorOf('why_had'), '"what made you have the idea" asks for the least');
+
+  // ADR 0186's interim karma-eligibility guard, restated here so a later
+  // reshuffle cannot drop the template below its own economics.
+  assert.ok(T.BASE_V2_TEMPLATE.fields.filter((f) => f.required).length >= 3
+    && total(T.BASE_V2_TEMPLATE) >= 120,
+  'ADR 0186 T5: ≥3 required fields and ≥120 total substanceMin to stay karma-eligible');
+});
+
 test('the declaration is frozen — a consumer cannot fork it in place', () => {
   assert.throws(() => { T.BASE_TEMPLATE.fields.push({ key: 'rogue' }); });
   assert.throws(() => { T.BASE_V1_TEMPLATE.fields.push({ key: 'rogue' }); });