@bongos/core 1.19.226 → 1.19.228

package/.bongos-core.json

@@ -2,22 +2,22 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/gds/package-core.js",
-  "core_version": "1.19.226",
-  "core_contract": "1.19.226",
-  "source_commit": "b99c0b52cf19ffd33ad98d0fc3dd763011559401",
+  "core_version": "1.19.228",
+  "core_contract": "1.19.228",
+  "source_commit": "f14babca30bfba954ddd62b510ebc1ed4749d9fa",
   "source_ref": "HEAD",
-  "built_at": "2026-08-18T04:50:04.434Z",
+  "built_at": "2026-08-18T05:08:08.027Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 311,
     "agent_docs_stubbed": 21,
-    "functional_verbatim": 1308,
+    "functional_verbatim": 1309,
     "rules": 3,
     "gate_literals": 3,
     "gate": "passed"
   },
-  "file_count": 1640,
-  "tree_sha256": "024eb9d71d1dd27389cb4db58a347eadfad7412b37e48d140a9c113cc8ee9234",
+  "file_count": 1641,
+  "tree_sha256": "2a30117336498006b3241dc75e5157ce5fdc462327dbdf3618e469ad9defb158",
   "files": [
     {
       "path": ".claude/skills/blocker-review/SKILL.md",
@@ -2902,7 +2902,7 @@
     {
       "path": "modules/government/board.js",
       "mode": "0000644",
-      "sha256": "0711f087cc61ed19f91fb207705a9e4831422c9acf29ae8402f87926a31d2a51"
+      "sha256": "6a2b7ebfcbba1442f45efe6df40131b03455fa0fff9ecce51e87696a72c5b117"
     },
     {
       "path": "modules/government/catalog.js",
@@ -2917,7 +2917,7 @@
     {
       "path": "modules/government/db.js",
       "mode": "0000644",
-      "sha256": "1f50cef895c7937aebbb365aa7a9ce1b1681bd097034c0045d94458cd369f50a"
+      "sha256": "e20e5a4aa8a79441e5860ceda2f19e41cbdb8fd85c0eda5d00f572eee5701f20"
     },
     {
       "path": "modules/government/migrations/governance_001_tables.sql",
@@ -3007,7 +3007,7 @@
     {
       "path": "modules/government/routes/government.js",
       "mode": "0000644",
-      "sha256": "41ac023e96901b9b3cf8a5059a5c26ff5f86b564d7bb37dc8d5a55a206db85fb"
+      "sha256": "63ab50af4bc8d36db246eacfc9041c18cf0af89279f8ffe8e89f67978773401b"
     },
     {
       "path": "modules/government/routes/rank-sync.js",
@@ -4302,12 +4302,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "93e2a0f8b239aa724d99178fa5cfcd7794d7a09c9e820a9ba3f8da94a81ec2b0"
+      "sha256": "41bf0bb43e6a556ddf9598e5208bc478dd9bfc5d92f46ff64b6af6ef3ecac0eb"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "378978670cd1efdcc3929f44178118e321f8e381bb923aff175716af2b24134d"
+      "sha256": "1b62282a15730fccfbe7ae8b3a2cb2321c4b1f9947bba37928150613db0e6d7d"
     },
     {
       "path": "public-docs/index.html",
@@ -4932,7 +4932,7 @@
     {
       "path": "scripts/gds/fitness.js",
       "mode": "0000644",
-      "sha256": "a255afac6eb4a2b8269e896032e8eabf10f104aed5a0f1e8d0a7348695625424"
+      "sha256": "00eacd557419ebdba21347e215ef0f197de8cd839f6af778bf90ff2901823a0e"
     },
     {
       "path": "scripts/gds/gate-review.js",
@@ -5922,7 +5922,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "d7a3216e0393ff4a4cb0fe3c7823b9c8a294ff45549a4a698b95c78ea93e4ad8"
+      "sha256": "985014c8a83781918e242757e48bf93cbc1df612b1eb4560177aeba85b30b0f7"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -6844,6 +6844,11 @@
       "mode": "0000644",
       "sha256": "8f5354cbacf15c9b49500f96b265fe52f5c0d4500a4ec7b41b4819a149782893"
     },
+    {
+      "path": "tests/government_board_close.mjs",
+      "mode": "0000644",
+      "sha256": "4408ec434f36b70a589b0592814733d3f88d48aa9aff38c08c9a2eb6c6c9f7b3"
+    },
     {
       "path": "tests/government_board_membership.mjs",
       "mode": "0000644",

package/modules/government/board.js

@@ -40,9 +40,10 @@
 // take effect on the very next window without a process restart, the same
 // instant-effect posture the resolver takes (ADR 0016).
 
+const api = require('../../src/module-api');
 const db = require('./db');
 const { loadGovernmentConfig } = require('./config');
-const { isBoardMember } = require('./board-membership');
+const { isBoardMember, resolveBoardMembers } = require('./board-membership');
 
 // ADR 0175 §5: the completeness bar a Full Idea must clear to reach the board.
 // Held equal to economy's IDEA_FULL_PASS_BAR by test — see the header.
@@ -198,6 +199,158 @@ async function castVote({ itemId, voterId, direction, reasonMd = null, sectionKe
   return { ok: true, vote: inserted, fresh: true, karma_delta: VOTE_KARMA_DELTA };
 }
 
+// ── close + tally (BV1.R10 of goal 1000069, ADR 0175 §3) ────────────────────
+//
+// EVERY RULE DECLARED, NONE EMERGENT. The tally is one line: a single reasoned
+// objection returns the idea; anything else — including ZERO votes — passes.
+// Absent members are NULLED by construction rather than by branch: the tally
+// only ever reads the votes that were cast, so a member who never voted simply
+// does not exist in it, and absence can never block (§2 — the sitting model).
+// A zero-vote pass is safe because the idea already cleared the grade gate,
+// the author could not vote for themselves, and the ballot is open.
+//
+// TWO TRIGGERS, ONE CLOSE PATH:
+//   • after a vote — first_ratifier closes on the FIRST vote (monarchy: the
+//     Archon ratifies or returns, done); consent closes early only on FULL
+//     TURNOUT (every current member has voted — recorded positions and the
+//     vote karma are worth the wait, and no later vote can change the outcome
+//     anyway since there is no vote-changing in v1);
+//   • on expiry — R11's poller calls the same closeItem, so the clock and the
+//     turnout can never disagree about what a close means.
+// An unknown pass_rule closes NOTHING after a vote (fail-closed — only the
+// clock can end it), the same strict-end posture as R05's sanitizer.
+//
+// THE CLOSE IS STRUCTURALLY SINGLE-WINNER: closeBoardItem's UPDATE carries
+// WHERE closed_at IS NULL, so two concurrent closers (a racing vote and the
+// expiry sweep, say) write exactly one outcome and the loser reads it back.
+//
+// RATIFICATION PAYS THE AUTHOR HERE — this is what fires the +3
+// awardAuthorRatifyKarma R08 declared (idempotent by reason key, so a re-driven
+// close never double-pays). THE PASS EVENT IS DELIBERATELY NOT EMITTED HERE:
+// that is R12's contract (docs/specs/board-room-pass-event-contract.md), and
+// its once-per-idea semantics deserve their own slice — the sweep-friendly
+// `WHERE outcome = 'passed'` shape survives precisely because closing and
+// emitting are separate acts.
+
+// The whole counting rule. Absence is nulled by what this function DOESN'T
+// read: only cast votes exist here.
+function tallyOutcome(votes) {
+  return (votes || []).some((v) => v.direction === 'no') ? 'returned' : 'passed';
+}
+
+// Should a just-cast vote close the item now, under the SNAPSHOTTED rule?
+function shouldCloseAfterVote({ constitution, votes, members } = {}) {
+  const rule = (constitution || {}).pass_rule;
+  if (rule === 'first_ratifier') return (votes || []).length >= 1;
+  if (rule === 'consent') {
+    if (!(constitution || {}).close_early_on_full_turnout) return false;
+    if (!members || !members.length) return false; // an empty board has no turnout to complete
+    const voters = new Set((votes || []).map((v) => String(v.voter_id)));
+    return members.every((m) => voters.has(String(m.builder_id)));
+  }
+  return false; // unknown rule: only the clock closes it
+}
+
+// The pass-event payload (R12, docs/specs/board-room-pass-event-contract.md
+// Decision 2) — IDENTICAL shape for the in-transaction port call and the
+// post-commit event, so there is one thing to reason about. Deliberately
+// carries no tally, no voter list, no constitution: the economy module has no
+// business reading how the vote went, only that it passed.
+function passPayload(closed, { authorId, completenessScore, passedAt }) {
+  const isIdea = closed.subject_type === 'full_idea';
+  return {
+    item_id: closed.id,
+    subject_type: closed.subject_type,
+    subject_id: closed.subject_id,
+    idea_id: isIdea ? closed.subject_id : null,
+    author_id: authorId,
+    completeness_score: completenessScore,
+    passed_at: passedAt,
+  };
+}
+
+// Close an item: tally what was cast, write the outcome, pay on a pass.
+// Idempotent end to end — the single-winner UPDATE plus every award's
+// reason-key guard. `cause` is for the caller's telemetry ('vote' | 'expiry').
+//
+// R12 (the pass-event contract): MONEY RIDES THE CLOSE TRANSACTION. Stream 3's
+// credit is booked through the `reward` PORT inside the same transaction that
+// writes the outcome — if the credit write fails, the close rolls back and the
+// item stays open, so the vote is retried rather than silently unpaid.
+// `resolveOptional` returning null (no economy module), or a reward port that
+// does not yet carry awardIdeaGovernancePassCredit (IC5 unbuilt), is a CLEAN
+// NO-OP: the item still closes, nothing is booked, and the durable
+// `outcome = 'passed'` row lets IC5's re-drive sweep pay later. The port is
+// resolved and the payload's reads happen BEFORE the transaction opens.
+//
+// `board.item.passed` is then emitted AFTER commit, emitAsync, for non-money
+// reactions only (R22's Discord mirror, future notifications) — emitAsync
+// swallows listener errors, which is right for a broadcast and disqualifying
+// for a payment, which is exactly why the money went through the port above.
+async function closeItem(item, { cause = 'vote' } = {}) {
+  if (!item || item.closed_at) return { closed: false, reason: 'already_closed' };
+  const votes = await db.getVotesForItem(item.id);
+  const outcome = tallyOutcome(votes);
+
+  // The payload's reads, ahead of the transaction (reads never ride a tx they
+  // don't need). Only a pass needs them, and only a full_idea can pay.
+  let authorId = null;
+  let completenessScore = null;
+  let credit = null;
+  if (outcome === 'passed') {
+    authorId = await db.getBoardItemAuthor(item);
+    if (item.subject_type === 'full_idea') {
+      completenessScore = await db.getIdeaCompleteness(item.subject_id);
+    }
+  }
+
+  const reward = outcome === 'passed' && item.subject_type === 'full_idea'
+    ? (api.resolveOptional && api.resolveOptional('reward'))
+    : null;
+  const payMoney = reward && typeof reward.awardIdeaGovernancePassCredit === 'function';
+
+  const closed = await db.closeBoardItem({
+    itemId: item.id,
+    outcome,
+    afterClose: payMoney
+      ? async (client) => {
+          credit = await reward.awardIdeaGovernancePassCredit(client, {
+            ideaId: item.subject_id,
+            authorId,
+            completenessScore,
+          });
+        }
+      : null,
+  });
+  if (!closed) return { closed: false, reason: 'already_closed' };
+
+  const authorKarma = outcome === 'passed' ? await awardAuthorRatifyKarma(closed) : null;
+
+  if (outcome === 'passed') {
+    const payload = passPayload(closed, { authorId, completenessScore, passedAt: closed.closed_at });
+    try {
+      if (api.emitAsync) await api.emitAsync('board.item.passed', payload);
+    } catch (_) { /* emitAsync isolates listener errors; nothing that moves credits subscribes */ }
+  }
+
+  return { closed: true, item: closed, outcome, cause, author_karma: authorKarma, ...(credit ? { credit } : {}) };
+}
+
+// The post-vote hook R08's route fires after a FRESH cast. Reads everything it
+// judges — the item, the votes, the LIVE members under the snapshot's rule —
+// per request, uncached, then hands closeItem the decision.
+async function maybeCloseAfterVote(itemId) {
+  const item = await db.getBoardItemForVote(Number(itemId));
+  if (!item || item.closed_at) return { closed: false, reason: 'not_open' };
+  const constitution = item.constitution || {};
+  const votes = await db.getVotesForItem(item.id);
+  const members = await resolveBoardMembers(constitution);
+  if (!shouldCloseAfterVote({ constitution, votes, members })) {
+    return { closed: false, reason: 'window_still_open' };
+  }
+  return closeItem(item, { cause: 'vote' });
+}
+
 module.exports = {
   FULL_IDEA_PASS_BAR,
   ideaPassesBar,
@@ -208,4 +361,9 @@ module.exports = {
   awardAuthorRatifyKarma,
   validateVote,
   castVote,
+  tallyOutcome,
+  shouldCloseAfterVote,
+  passPayload,
+  closeItem,
+  maybeCloseAfterVote,
 };

package/modules/government/db.js

@@ -372,6 +372,71 @@ async function getBoardVote(itemId, voterId) {
 // so the idempotent re-cast can never double-pay. granted_by NULL: earned from
 // the system by the act, not granted by a person (the governance_002 backfill
 // convention). karma_log is a shared core table; same access rule as above.
+// Every vote cast on an item, in cast order — the tally's whole input. Absent
+// members are nulled by construction: they have no row here to read.
+async function getVotesForItem(itemId) {
+  const { rows } = await pool.query(
+    `SELECT * FROM government_board_votes WHERE item_id = $1 ORDER BY cast_at, id`,
+    [itemId],
+  );
+  return rows;
+}
+
+// The close write — STRUCTURALLY SINGLE-WINNER: WHERE closed_at IS NULL means
+// two concurrent closers (a racing vote and the expiry sweep) write exactly one
+// outcome; the loser gets null and reads the winner's row back if it cares.
+// closed_at comes from the database clock, like every timestamp on this table.
+//
+// R12 (the pass-event contract, Decision 1) made this a TRANSACTION: money
+// rides the close, so `afterClose(client, closedRow)` runs INSIDE it — if the
+// credit write fails, the close rolls back and the item stays open (the vote is
+// retried, never silently unpaid; "the ledger and the outcome can never
+// disagree"). No afterClose (or a null one) degrades to the plain single-
+// statement close R10 shipped.
+async function closeBoardItem({ itemId, outcome, afterClose = null }) {
+  if (typeof afterClose !== 'function') {
+    const { rows } = await pool.query(
+      `UPDATE government_board_items
+          SET closed_at = now(), outcome = $2
+        WHERE id = $1 AND closed_at IS NULL
+        RETURNING *`,
+      [itemId, outcome],
+    );
+    return rows[0] || null;
+  }
+  const client = await pool.connect();
+  try {
+    await client.query('BEGIN');
+    const { rows } = await client.query(
+      `UPDATE government_board_items
+          SET closed_at = now(), outcome = $2
+        WHERE id = $1 AND closed_at IS NULL
+        RETURNING *`,
+      [itemId, outcome],
+    );
+    const closed = rows[0] || null;
+    if (closed) await afterClose(client, closed);
+    await client.query('COMMIT');
+    return closed;
+  } catch (err) {
+    try { await client.query('ROLLBACK'); } catch (_) { /* connection-level failure */ }
+    throw err;
+  } finally {
+    client.release();
+  }
+}
+
+// The subject's stored completeness score, for the pass-event payload (R12).
+// ideas_grades is another module's table; READING it is the sanctioned pattern
+// (the boundary is code imports, not table access) — null when unscored.
+async function getIdeaCompleteness(ideaId) {
+  const { rows } = await pool.query(
+    `SELECT completeness_score FROM ideas_grades WHERE idea_id = $1`,
+    [ideaId],
+  );
+  return rows[0] ? rows[0].completeness_score : null;
+}
+
 // Author-ratification karma (+3, board.js AUTHOR_RATIFY_KARMA_DELTA) — fired by
 // the R10 close on a passed item. IDEMPOTENT: the reason key is unique per item
 // and the INSERT is one atomic WHERE-NOT-EXISTS statement (the economy
@@ -423,4 +488,7 @@ module.exports = {
   getBoardVote,
   insertVoteKarma,
   insertAuthorRatifyKarma,
+  getVotesForItem,
+  closeBoardItem,
+  getIdeaCompleteness,
 };

package/modules/government/routes/government.js

@@ -445,6 +445,19 @@ module.exports = function buildGovernmentRouter() {
           400; // bad_direction / yes_carries_nothing / objection_needs_reason / objection_needs_section
         return res.fail(out.reason, status, { item_id: itemId });
       }
+      // R10: a fresh cast may CLOSE the sitting — first_ratifier closes on the
+      // first vote (monarchy: the Archon ratifies or returns, done); consent
+      // closes early on full turnout. The close is single-winner in the schema,
+      // so a concurrent racer is safe, and a close hiccup never loses the vote
+      // (already recorded) — fail-open, the capture-path posture.
+      let closure = null;
+      if (out.fresh) {
+        try {
+          closure = await board.maybeCloseAfterVote(itemId);
+        } catch (closeErr) {
+          console.error('[gds] post-vote close failed (vote recorded, close left to the expiry sweep):', closeErr);
+        }
+      }
       // Idempotent re-cast reads back the standing vote — 200, not 201, and no
       // karma line: nothing new was recorded.
       res.status(out.fresh ? 201 : 200).json({
@@ -452,6 +465,7 @@ module.exports = function buildGovernmentRouter() {
         vote: out.vote,
         already_voted: !out.fresh,
         ...(out.fresh ? { karma_delta: out.karma_delta } : {}),
+        ...(closure && closure.closed ? { item_closed: true, outcome: closure.outcome } : {}),
       });
     } catch (err) {
       console.error('[gds] POST /government/board/items/:itemId/votes', err);

package/package-lock.json

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

package/package.json

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

@@ -1688,6 +1688,16 @@ const VOCAB_ALLOW = [
        + 'repo, so the old spelling is still read. Dropping it would silently stop '
        + 'honouring a label override someone already configured.',
   },
+  {
+    file: 'modules/government/board.js',
+    match: 'governancepasscredit',
+    why: 'the R02 pass-event contract (docs/specs/board-room-pass-event-contract.md, agreed '
+       + 'cross-goal 2026-08-17) names the economy-side award awardIdeaGovernancePassCredit, '
+       + 'matching the credit_log reason family idea.credit.governance_pass:<id> that streams '
+       + '1-3 share. The board calls the CONSUMER\'s agreed method name — renaming it here '
+       + 'would break the contract, and the reason strings are durable ledger rows. Scoped to '
+       + 'the method name, so a stray "governance" elsewhere in board.js still reds the build.',
+  },
   {
     file: 'modules/government/module.json',
     match: 'role',

package/src/module-api.js

@@ -879,8 +879,12 @@ const { validateOrRespond, LIMITS, parseId, asyncHandler, corsPublicGet, parsePa
 //         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)
+//   1.19.227 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+//         landed since 1.19.226 with no explicit bump. run 32101041951. (task 1002620)
+//   1.19.228 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+//         landed since 1.19.227 with no explicit bump. run 32101659669. (task 1002620)
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.226'; // CI auto-patch carrier (ADR 0161); see changelog above
+const CORE_VERSION = '1.19.228'; // 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/government_board_close.mjs

@@ -0,0 +1,256 @@
+// tests/government_board_close.mjs — close + tally: consent, nulled absences,
+// zero-vote pass (BV1.R10 of goal 1000069, ADR 0175 §3).
+//
+// DB-free (db monkeypatched, the sibling board tests' idiom). The declared
+// rules, each pinned:
+//
+//   1. THE TALLY IS ONE LINE — one reasoned objection returns the idea;
+//      anything else, INCLUDING ZERO VOTES, passes. Absence is nulled by
+//      construction: the tally reads only cast votes, so an absent member has
+//      nothing to read and can never block.
+//   2. TWO TRIGGERS, ONE PATH — first_ratifier closes on the first vote;
+//      consent closes early only on FULL TURNOUT of the LIVE members under the
+//      snapshot's rule; an unknown rule closes nothing after a vote.
+//   3. THE CLOSE IS SINGLE-WINNER — WHERE closed_at IS NULL in the SQL; a
+//      losing racer gets already_closed and no karma.
+//   4. RATIFICATION PAYS THE AUTHOR HERE — +3 on 'passed', nothing on
+//      'returned' — and the PASS EVENT is deliberately NOT emitted (R12's).
+import assert from 'node:assert/strict';
+import { test } from 'node:test';
+import { createRequire } from 'node:module';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const require = createRequire(import.meta.url);
+const ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
+const GOV = path.join(ROOT, 'modules', 'government');
+const board = require(path.join(GOV, 'board.js'));
+const db = require(path.join(GOV, 'db.js'));
+
+const { tallyOutcome, shouldCloseAfterVote, closeItem, maybeCloseAfterVote, AUTHOR_RATIFY_KARMA_DELTA } = board;
+
+const yes = (voter) => ({ voter_id: voter, direction: 'yes' });
+const no = (voter, section = 'fit') => ({ voter_id: voter, direction: 'no', reason_md: 'r', section_key: section });
+const member = (id) => ({ builder_id: String(id), github_login: `m${id}`, display_name: `M${id}` });
+
+async function withDb(stubs, fn) {
+  const originals = {};
+  for (const [k, v] of Object.entries(stubs)) { originals[k] = db[k]; db[k] = v; }
+  try { return await fn(); }
+  finally { for (const [k, v] of Object.entries(originals)) db[k] = v; }
+}
+
+// ── rule 1: the tally ────────────────────────────────────────────────────────
+
+test('one reasoned objection returns; everything else — zero votes included — passes', () => {
+  assert.equal(tallyOutcome([]), 'passed', 'the zero-vote pass: silence is consent, absence never blocks');
+  assert.equal(tallyOutcome([yes('1')]), 'passed');
+  assert.equal(tallyOutcome([yes('1'), yes('2'), yes('3')]), 'passed');
+  assert.equal(tallyOutcome([yes('1'), no('2'), yes('3')]), 'returned', '4-yes/1-no returns — real minority power, the spec\'s recorded trade-off');
+  assert.equal(tallyOutcome([no('1')]), 'returned');
+});
+
+test('absence is nulled BY CONSTRUCTION — the tally has no denominator to block', () => {
+  // A 5-member board where only 2 voted: the tally sees 2 votes, not 3 absences.
+  // There is no argument to pass the member list through — that is the design.
+  assert.equal(tallyOutcome.length, 1, 'tallyOutcome takes votes and nothing else');
+  assert.equal(tallyOutcome([yes('1'), yes('2')]), 'passed');
+});
+
+// ── rule 2: the two triggers ─────────────────────────────────────────────────
+
+const MONARCHY = { membership: 'rank:archon', pass_rule: 'first_ratifier', window_minutes: null, close_early_on_full_turnout: true };
+const CONSENT = { membership: 'rank:archon', pass_rule: 'consent', window_minutes: 90, close_early_on_full_turnout: true };
+
+test('first_ratifier closes on the FIRST vote — monarchy: the Archon acts, done', () => {
+  assert.equal(shouldCloseAfterVote({ constitution: MONARCHY, votes: [yes('1')], members: [member(1)] }), true);
+  assert.equal(shouldCloseAfterVote({ constitution: MONARCHY, votes: [no('1')], members: [member(1)] }), true, 'a first NO closes too — as returned');
+  assert.equal(shouldCloseAfterVote({ constitution: MONARCHY, votes: [], members: [member(1)] }), false);
+});
+
+test('consent closes early only on FULL TURNOUT of the live members', () => {
+  const members = [member(1), member(2), member(3)];
+  assert.equal(shouldCloseAfterVote({ constitution: CONSENT, votes: [yes('1')], members }), false, 'partial turnout waits for the clock');
+  assert.equal(shouldCloseAfterVote({ constitution: CONSENT, votes: [yes('1'), no('2')], members }), false,
+    'even a decided outcome waits — recorded positions and vote karma are worth the sitting');
+  assert.equal(shouldCloseAfterVote({ constitution: CONSENT, votes: [yes('1'), no('2'), yes('3')], members }), true);
+  // number/string ids compare equal across the turnout check
+  assert.equal(shouldCloseAfterVote({ constitution: CONSENT, votes: [{ voter_id: 1, direction: 'yes' }], members: [member(1)] }), true);
+});
+
+test('close-early OFF, an empty board, and an unknown rule all refuse to close after a vote', () => {
+  assert.equal(shouldCloseAfterVote({ constitution: { ...CONSENT, close_early_on_full_turnout: false }, votes: [yes('1')], members: [member(1)] }), false);
+  assert.equal(shouldCloseAfterVote({ constitution: CONSENT, votes: [yes('1')], members: [] }), false, 'an empty board has no turnout to complete');
+  assert.equal(shouldCloseAfterVote({ constitution: { ...CONSENT, pass_rule: 'unanimous' }, votes: [yes('1')], members: [member(1)] }), false,
+    'unknown rule: fail-closed — only the clock closes it');
+});
+
+// ── rules 3+4: closeItem ─────────────────────────────────────────────────────
+
+const openItem = (over = {}) => ({
+  id: '11', subject_type: 'full_idea', subject_id: '7', opened_by: null,
+  closed_at: null, constitution: MONARCHY, ...over,
+});
+
+test('a pass pays the author (+3, idempotent downstream); a return pays nobody', async () => {
+  const karma = [];
+  await withDb({
+    getVotesForItem: async () => [yes('42')],
+    closeBoardItem: async ({ itemId, outcome }) => ({ ...openItem(), id: String(itemId), closed_at: 't', outcome }),
+    getBoardItemAuthor: async () => '7',
+    getIdeaCompleteness: async () => 82,
+    insertAuthorRatifyKarma: async (args) => { karma.push(args); return true; },
+  }, async () => {
+    const out = await closeItem(openItem(), { cause: 'vote' });
+    assert.equal(out.closed, true);
+    assert.equal(out.outcome, 'passed');
+    assert.deepEqual(out.author_karma, { paid: true, karma_delta: AUTHOR_RATIFY_KARMA_DELTA });
+  });
+  assert.equal(karma.length, 1);
+  assert.equal(karma[0].delta, AUTHOR_RATIFY_KARMA_DELTA);
+
+  await withDb({
+    getVotesForItem: async () => [no('42')],
+    closeBoardItem: async ({ itemId, outcome }) => ({ ...openItem(), id: String(itemId), closed_at: 't', outcome }),
+    getBoardItemAuthor: async () => { throw new Error('must not be reached — a return pays nobody'); },
+    insertAuthorRatifyKarma: async () => { throw new Error('must not be reached'); },
+  }, async () => {
+    const out = await closeItem(openItem());
+    assert.equal(out.outcome, 'returned');
+    assert.equal(out.author_karma, null);
+  });
+});
+
+test('the losing racer gets already_closed and pays nothing', async () => {
+  await withDb({
+    getVotesForItem: async () => [yes('42')],
+    getBoardItemAuthor: async () => '7',
+    getIdeaCompleteness: async () => 82,
+    closeBoardItem: async () => null, // the WHERE closed_at IS NULL conflict path
+    insertAuthorRatifyKarma: async () => { throw new Error('must not be reached'); },
+  }, async () => {
+    assert.deepEqual(await closeItem(openItem()), { closed: false, reason: 'already_closed' });
+  });
+  assert.deepEqual(await closeItem(openItem({ closed_at: 't' })), { closed: false, reason: 'already_closed' },
+    'an already-closed item is refused before any read');
+});
+
+// ── maybeCloseAfterVote: the whole post-vote path ────────────────────────────
+
+test('monarchy end to end: the first yes closes the item as passed, author paid', async () => {
+  const karma = [];
+  await withDb({
+    getBoardItemForVote: async () => openItem(),
+    getVotesForItem: async () => [yes('42')],
+    getActiveBuildersWithRank: async () => [{ builder_id: '42', github_login: 'a', display_name: 'A' }],
+    closeBoardItem: async ({ outcome }) => ({ ...openItem(), closed_at: 't', outcome }),
+    getBoardItemAuthor: async () => '7',
+    getIdeaCompleteness: async () => 82,
+    insertAuthorRatifyKarma: async (args) => { karma.push(args); return true; },
+  }, async () => {
+    const out = await maybeCloseAfterVote(11);
+    assert.equal(out.closed, true);
+    assert.equal(out.outcome, 'passed');
+    assert.equal(out.cause, 'vote');
+  });
+  assert.equal(karma.length, 1);
+});
+
+test('consent with turnout outstanding leaves the window open', async () => {
+  await withDb({
+    getBoardItemForVote: async () => openItem({ constitution: CONSENT }),
+    getVotesForItem: async () => [yes('1')],
+    getActiveBuildersWithRank: async () => [member(1), member(2)], // 1 of 2 voted
+    closeBoardItem: async () => { throw new Error('must not be reached'); },
+  }, async () => {
+    assert.deepEqual(await maybeCloseAfterVote(11), { closed: false, reason: 'window_still_open' });
+  });
+});
+
+// ── the structural pins ──────────────────────────────────────────────────────
+
+test('the close UPDATE is single-winner in the SQL — in BOTH the plain and the transactional form', () => {
+  const src = fs.readFileSync(path.join(GOV, 'db.js'), 'utf8');
+  const block = src.slice(src.indexOf('async function closeBoardItem'), src.indexOf('async function getIdeaCompleteness'));
+  assert.equal((block.match(/WHERE id = \$1 AND closed_at IS NULL/g) || []).length, 2,
+    'two racers write exactly one outcome, with or without an in-tx credit hook');
+  assert.match(block, /ROLLBACK/, 'a failed in-tx credit rolls the close back — the vote is retried, never silently unpaid');
+});
+
+// ── R12: money rides the port inside the tx; the event is post-commit courtesy ──
+
+test('a pass books the stream-3 credit INSIDE the close transaction, through the reward port', async () => {
+  const api = require(path.join(ROOT, 'src', 'module-api.js'));
+  const hadReward = api.hasProvider('reward');
+  if (!hadReward) {
+    api.registerProvider('reward', {
+      awardIdeaGovernancePassCredit: async (client, args) => ({ credited: 82, reason: 'paid', _client: client, _args: args }),
+    });
+  }
+  const seen = [];
+  try {
+    await withDb({
+      getVotesForItem: async () => [yes('42')],
+      getBoardItemAuthor: async () => '7',
+      getIdeaCompleteness: async () => 82,
+      closeBoardItem: async ({ outcome, afterClose }) => {
+        const closed = { ...openItem(), closed_at: 't', outcome };
+        // the db layer hands afterClose ITS transaction client — simulate that
+        if (afterClose) await afterClose({ __tx: true }, closed);
+        seen.push({ outcome, hadHook: typeof afterClose === 'function' });
+        return closed;
+      },
+      insertAuthorRatifyKarma: async () => true,
+    }, async () => {
+      const out = await closeItem(openItem(), { cause: 'vote' });
+      assert.equal(out.closed, true);
+      if (out.credit) { // present only when a reward provider carries IC5's method
+        assert.equal(out.credit.reason, 'paid');
+        assert.deepEqual(out.credit._client, { __tx: true }, 'the award ran on the CLOSE transaction\'s client');
+        assert.equal(out.credit._args.ideaId, '7');
+        assert.equal(out.credit._args.completenessScore, 82);
+      }
+      assert.equal(seen[0].outcome, 'passed');
+    });
+  } finally { /* provider registry is process-global; leave whatever was there */ }
+});
+
+test('a RETURNED item books nothing and emits nothing — only a pass pays or announces', async () => {
+  await withDb({
+    getVotesForItem: async () => [no('42')],
+    closeBoardItem: async ({ outcome, afterClose }) => {
+      assert.equal(afterClose, null, 'no in-tx hook on a return — nothing to book');
+      return { ...openItem(), closed_at: 't', outcome };
+    },
+    getBoardItemAuthor: async () => { throw new Error('must not be reached'); },
+    getIdeaCompleteness: async () => { throw new Error('must not be reached'); },
+  }, async () => {
+    const out = await closeItem(openItem());
+    assert.equal(out.outcome, 'returned');
+    assert.equal(out.credit, undefined);
+  });
+});
+
+test('the payload matches the contract shape, and an amendment carries idea_id null', () => {
+  const closed = { id: '11', subject_type: 'full_idea', subject_id: '7', closed_at: 'T' };
+  const p = board.passPayload(closed, { authorId: '42', completenessScore: 82, passedAt: 'T' });
+  assert.deepEqual(Object.keys(p).sort(),
+    ['author_id', 'completeness_score', 'idea_id', 'item_id', 'passed_at', 'subject_id', 'subject_type'],
+    'exactly the contract\'s seven fields — no tally, no voter list, no constitution');
+  assert.equal(p.idea_id, '7', 'idea_id === subject_id for a full_idea');
+  const amendment = board.passPayload({ ...closed, subject_type: 'constitutional_amendment' },
+    { authorId: '3', completenessScore: null, passedAt: 'T' });
+  assert.equal(amendment.idea_id, null, 'an amendment traverses the same path and must book nothing — the consumer filters on subject_type');
+});
+
+test('the event is emitAsync AFTER the close, and money never rides it (source pins)', () => {
+  const boardSrc = fs.readFileSync(path.join(GOV, 'board.js'), 'utf8');
+  const close = boardSrc.slice(boardSrc.indexOf('async function closeItem'), boardSrc.indexOf('async function maybeCloseAfterVote'));
+  assert.match(close, /emitAsync\('board\.item\.passed'/, 'the courtesy event exists for R22');
+  assert.ok(close.indexOf('closeBoardItem') < close.indexOf("emitAsync('board.item.passed'"),
+    'emitted after the close write — a payment must not depend on a listener having been alive');
+  assert.match(close, /awardIdeaGovernancePassCredit/, 'the money goes through the PORT');
+  const emitBlock = close.slice(close.indexOf("emitAsync('board.item.passed'"));
+  assert.ok(!emitBlock.includes('award'), 'nothing that moves credits touches the event path');
+});