@bongos/core 1.21.98 → 1.21.100

package/scripts/gds/publish-manifest.js

@@ -90,6 +90,7 @@ const PUBLISH_ALLOWLIST = [
   'README.md',
   'LICENSE',
   'GOVERNANCE.md',
+  'GOVERNMENT.md',                   // the plain-language guide to a project's own government (task 1004659)
   'CONTRIBUTING.md',
   'CODE_OF_CONDUCT.md',
   'docs/adr/',                       // sanitized — ADR 0065 §4 "docs/ADRs (sanitized)"

package/src/bongos/module-scope-map.js

@@ -596,6 +596,7 @@ const UNOWNED_ROOTS = Object.freeze({
   'CODE_OF_CONDUCT.md': 'project governance text',
   'CONTRIBUTING.md': 'project governance text',
   'GOVERNANCE.md': 'project governance text',
+  'GOVERNMENT.md': 'project governance text',
   // Top-level product/design statements, read by humans before any module matters.
   'README.md': 'repo front door',
   'DESIGN.md': 'top-level design statement',

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.98'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.21.100'; // 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/government_walk.mjs

@@ -0,0 +1,56 @@
+// tests/government_walk.mjs — the government walk stays true (task 1004658, goal
+// 1000111, criterion gov-proven-and-kept-proven).
+//
+// scripts/gds/government-walk.js walks a throwaway project's whole government —
+// genesis, every form, every building's act-alone gate, every matter action — and
+// writes the pass/fail table to docs/governance/government-walk.md. This runs the
+// walk again in its own process (it points the instance root at a temp folder and
+// swaps the government db layer for its in-memory store, so it must not share a
+// module graph with other tests) and fails if ANY row differs from the committed
+// table: a change that breaks a decision flips a row from PASS to FAIL here, and a
+// fix to a filed bug flips one back, which asks for the table to be regenerated.
+//
+// Also pins the walk's own coverage, so it cannot quietly walk less: every bound
+// route of every building has a request in the walk, and every router a bound route
+// lives on is mounted.
+//
+// Run:  node --test tests/government_walk.mjs
+
+import { strict as assert } from 'node:assert';
+import { test } from 'node:test';
+import { spawnSync } from 'node:child_process';
+import path from 'node:path';
+import { createRequire } from 'node:module';
+import { fileURLToPath } from 'node:url';
+
+const require = createRequire(import.meta.url);
+const ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
+const SCRIPT = path.join(ROOT, 'scripts', 'gds', 'government-walk.js');
+const walk = require(SCRIPT);
+const dr = require(path.join(ROOT, 'modules', 'government', 'decision-rules.js'));
+const decisionBoard = require(path.join(ROOT, 'modules', 'government', 'decision-board.js'));
+
+test('the walk covers every bound route of every building', () => {
+  const bound = dr.AREAS.flatMap((a) => a.acts).sort();
+  assert.deepEqual(Object.keys(walk.REQUESTS).sort(), bound, 'a route added to decision-rules AREAS[].acts needs a request in the walk');
+});
+
+test('the walk passes a matter for every action a passed matter can carry out', () => {
+  const planned = walk.MATTER_PLAN.map(([area, , action]) => `${area}.${action.type}`).sort();
+  const actions = Object.entries(decisionBoard.MATTER_ACTIONS)
+    .flatMap(([area, acts]) => Object.keys(acts).map((type) => `${area}.${type}`)).sort();
+  assert.ok(actions.length > 0, 'the matter-action table is not empty');
+  assert.deepEqual(planned, actions, 'a new matter action needs a matter in the walk');
+  // and every planned action is one the server would accept as written
+  for (const [area, , action] of walk.MATTER_PLAN) {
+    assert.equal(decisionBoard.validateMatterAction(area, action).ok, true, `${area} ${action.type} validates`);
+  }
+});
+
+test('the walk re-runs and matches the committed pass/fail table', () => {
+  const env = { ...process.env };
+  delete env.DATABASE_URL;
+  const r = spawnSync(process.execPath, [SCRIPT, '--check'], { cwd: ROOT, env, encoding: 'utf8', timeout: 240000 });
+  assert.equal(r.status, 0, `government-walk --check failed (exit ${r.status}):\n${r.stdout}\n${r.stderr}`);
+  assert.match(r.stdout, /matches the committed table/);
+});