@bongos/core 1.19.386 → 1.19.387

package/modules/hall-ui/public/government.css

@@ -101,3 +101,73 @@
   font-family: ui-monospace, SFMono-Regular, Menlo, monospace; font-size: 12px;
   color: var(--accent-text); background: var(--surface-2); padding: 1px 5px; border-radius: 4px;
 }
+
+/* ---- the Board Room: one sitting, one decision (task 1003270) -------------
+   The item card is NOT `.gov-section`. It used to be, which meant the title of
+   the idea being voted on rendered through `.gov-section__head` — 11.5px,
+   uppercase, letterspaced, i.e. styled as a table header. Here the title is a
+   title, the ballot sits directly under it, the idea's own words are body text,
+   and the constitutional metadata is the only thing behind a click. */
+.gov-item {
+  margin: 0 0 18px; border: 1px solid var(--border); border-radius: var(--radius);
+  background: var(--surface); overflow: hidden;
+}
+.gov-item__head {
+  display: flex; flex-wrap: wrap; align-items: baseline; justify-content: space-between;
+  gap: 6px 12px; padding: 12px 14px 10px;
+}
+.gov-item__title {
+  margin: 0; min-width: 0; font-size: 16px; font-weight: 600; line-height: 1.35;
+  color: var(--ink); overflow-wrap: anywhere;
+}
+.gov-item__title a { color: inherit; }
+.gov-item__state { display: inline-flex; align-items: center; gap: 8px; font-size: 12px; }
+.gov-item__amend { margin: 0; padding: 0 14px 10px; color: var(--ink-soft); font-size: 12.5px; overflow-wrap: anywhere; }
+
+/* The primary action, directly under the title — that adjacency is the fix. */
+.gov-item__act { padding: 2px 14px 14px; }
+.gov-item__cast { margin: 0; color: var(--ink-soft); font-size: 12.5px; }
+
+/* The idea itself: open, and set as prose rather than as permission rows. */
+.gov-idea { padding: 12px 14px 2px; border-top: 1px solid var(--border); }
+.gov-idea__part { margin: 0 0 12px; }
+.gov-idea__label { margin: 0 0 3px; font-size: 12px; font-weight: 600; color: var(--ink-soft); }
+.gov-idea__body {
+  margin: 0; color: var(--ink); font-size: 13.5px; line-height: 1.5;
+  white-space: pre-wrap; overflow-wrap: anywhere;
+}
+
+/* Secondary: the tally rides the summary (visible without a click — ADR 0175
+   §9), the rules and the open ballot sit inside it. */
+.gov-item__meta { border-top: 1px solid var(--border); }
+.gov-item__summary {
+  padding: 8px 14px; cursor: pointer; color: var(--ink-soft); font-size: 12px;
+  list-style-position: inside;
+}
+.gov-item__summary:hover { color: var(--ink); }
+.gov-item__metabody { padding: 10px 14px 2px; background: var(--surface-2); }
+.gov-item__metabody .gov-section { margin: 0 0 12px; background: var(--surface); }
+.gov-item__metabody .gov-empty { padding: 0 0 12px; }
+
+/* ---- the ballot controls ------------------------------------------------- */
+.gov-vote { display: flex; flex-direction: column; align-items: flex-start; gap: 8px; }
+.gov-vote > .gov-grid-note { margin: 0; max-width: 62ch; }
+.gov-vote__no { align-self: stretch; }
+/* The disclosure IS the secondary button, so its marker must go — otherwise the
+   triangle sits inside the pill. */
+.gov-vote__no > summary { display: inline-block; cursor: pointer; }
+.gov-vote__no > summary::marker { content: ''; }
+.gov-vote__no > summary::-webkit-details-marker { display: none; }
+.gov-vote__no > *:not(summary) { margin-top: 8px; }
+.gov-vote__no > textarea.gov-input { display: block; width: 100%; max-width: 62ch; }
+.gov-vote__sections { display: flex; flex-wrap: wrap; gap: 0 18px; }
+.gov-vote__sections .gov-perm {
+  padding: 2px 0; border-bottom: 0; cursor: pointer; font-size: 12.5px; color: var(--ink);
+}
+
+/* The ballot rows reuse `.gov-assign-row`, whose four flex children (name, pill,
+   timestamp, objection) cannot fit one 390w line — the objection is a paragraph,
+   not a cell, so it takes its own row and the rest wraps. Scoped to the metabody
+   so the Permissions and Constitution tabs keep their single-line rows. */
+.gov-item__metabody .gov-assign-row { flex-wrap: wrap; }
+.gov-item__metabody .gov-assign-row .gov-perm__guards { flex: 1 1 100%; margin-top: 4px; }

package/modules/hall-ui/public/government.html

@@ -37,14 +37,21 @@
                R18/R19; until then the section frames the model honestly. -->
           <section id="gov-panel-constitution" hidden>
             <h2 class="gov-h">Constitution</h2>
-            <p class="gov-grid-note">Three dials decide how this project is governed: <strong>who sits on the board</strong> (a rank — everyone holding it is a member), <strong>what counts as passing</strong> (a first ratifier, or the consent of the sitting), and <strong>how long a vote stays open</strong> (a timed sitting, or no clock at all). They are configuration on the same machinery at every setting — a project starts with the founder ratifying alone and opens up by changing numbers, not code.</p>
+            <!-- Task 1003270: the ~60-word explainer that used to sit here said,
+                 at length, what the three live sentences below say from the real
+                 config. A reader met the paragraph before the values. -->
+            <p class="gov-grid-note">Three dials, all settings rather than code: <strong>who sits on the board</strong>, <strong>what counts as passing</strong>, and <strong>how long a vote stays open</strong>.</p>
             <div id="gov-constitution"></div>
           </section>
 
           <!-- §2 Board Room — open items, the clock, your vote (R16/R17). -->
           <section id="gov-panel-board" hidden>
             <h2 class="gov-h">Board Room</h2>
-            <p class="gov-grid-note">Where a Full Idea that cleared the completeness grade comes to be ratified — and where the constitution itself is amended. Open items appear here with their clock and a live tally; an objection must carry a reason and name the section it faults, which is why votes are cast here and only mirrored to chat.</p>
+            <!-- Task 1003270: one line, then the ballot. Everything the old
+                 ~50-word paragraph explained is now said by the cards themselves
+                 — each one carries who votes and how it passes, in plain words,
+                 under its own summary. -->
+            <p class="gov-grid-note">Read an idea, then decide. An objection needs a reason and the section it faults.</p>
             <div id="gov-board"></div>
           </section>
 

package/modules/hall-ui/public/government.js

@@ -42,6 +42,22 @@
 // items and the vote. Until a frame is filled it states what belongs there
 // and shows an honest kit empty state — never a fake or a hardcoded value that
 // could drift from the real config.
+//
+// TASK 1003270 rebuilt the BOARD ROOM CARD around one claim: the idea is the
+// subject, everything else is apparatus. So the idea's title renders at title
+// size and its five sections render open as prose; the ballot sits directly
+// beneath the title; and the government apparatus — who votes, how it passes,
+// the clock, the open ballot — moved behind one summary that still shows the
+// tally. Two rules came out of it and should survive the next edit here:
+//   • NO RAW CONFIG KEY REACHES A READER. `first_ratifier` / `rank:archon` /
+//     `why_matters` are grammar for the server; a person gets the sentence
+//     (decidedByHtml, rankNames, SECTION_LABELS). The single exception is a
+//     membership predicate that cannot be PARSED, where the broken string is
+//     the actionable fact.
+//   • COPY THAT EXPLAINS THE UI TO ITSELF IS DELETED, not shortened.
+// What simplifying may NOT touch is ADR 0175 §8: an objection still carries a
+// reason and names one section. That requirement is why voting lives in the
+// hall and not on a chat reaction.
 
 (() => {
   'use strict';
@@ -497,7 +513,12 @@
       // and fixable, a board described as working is neither.
       return `Membership predicate <code>${escapeHtml(raw || 'unset')}</code> cannot be read, so <strong>nobody</strong> sits on the board until it is corrected.`;
     }
-    const strong = keys.map((k) => `<strong>${escapeHtml(k)}</strong>`);
+    // Task 1003270: the DISPLAY label, not the Greek enum key. The enum is the
+    // grammar an owner types into the predicate (and the charter library below
+    // still shows it verbatim, which is that library's whole job); this sentence
+    // is the one that answers "who governs", so it uses the words the rest of
+    // the hall shows a rank in.
+    const strong = keys.map((k) => `<strong>${escapeHtml(window.OTB.rankLabel(k))}</strong>`);
     if (keys.length === 1) {
       return `Every active builder holding the ${strong[0]} rank — and only that rank — sits on the board. Membership follows the rank, live.`;
     }
@@ -536,7 +557,7 @@
     const early = board.close_early_on_full_turnout
       ? ', closing early the moment every member has voted'
       : '';
-    return `A sitting runs <strong>${escapeHtml(String(board.window_minutes))} minutes</strong>${early}. An unvoted window nulls that member's vote — absence never blocks.`;
+    return `A sitting runs <strong>${fmtWindow(board.window_minutes)}</strong>${early}. An unvoted window nulls that member's vote — absence never blocks.`;
   }
 
   function historyRow(h) {
@@ -545,8 +566,10 @@
     const pill = h.outcome === 'passed'
       ? '<span class="gov-pill gov-pill--seeded">ratified</span>'
       : '<span class="gov-pill gov-pill--locked">returned</span>';
+    // Task 1003270: the rule by name, not by config key. An unknown value still
+    // shows verbatim — a rule this build does not know about is worth seeing.
     const under = h.decided_under && h.decided_under.pass_rule
-      ? ` · decided under <code>${escapeHtml(h.decided_under.pass_rule)}</code>`
+      ? ` · decided ${escapeHtml(PASS_RULE_NAME[h.decided_under.pass_rule] || `under ${h.decided_under.pass_rule}`)}`
       : '';
     return `<div class="gov-assign-row">
       <span class="gov-assign-row__who">${who} proposed <code>${escapeHtml(JSON.stringify(h.proposed))}</code>${h.rationale_md ? ` — ${escapeHtml(h.rationale_md)}` : ''}</span>
@@ -717,11 +740,14 @@
       const who = escapeHtml(v.display_name || v.github_login || `builder ${v.voter_id}`);
       const mine = String(v.voter_id) === String(viewerId) ? ' <span class="gov-held-note">your vote</span>' : '';
       const when = escapeHtml(String(v.cast_at).slice(0, 16).replace('T', ' '));
+      // Task 1003270: the faulted section by its LABEL — the words the author
+      // wrote under and the objector picked from. `why_matters` is a jsonb key,
+      // not something a reader should have to decode.
       const objection = v.direction === 'no'
-        ? `<div class="gov-perm__guards">objected to <code>${escapeHtml(v.section_key || '')}</code> — ${escapeHtml(v.reason_md || '')}</div>`
+        ? `<div class="gov-perm__guards">objected to <strong>${escapeHtml(SECTION_LABELS[v.section_key] || v.section_key || 'the idea')}</strong> — ${escapeHtml(v.reason_md || '')}</div>`
         : '';
       return `<div class="gov-assign-row"><span class="gov-assign-row__who">${who}${mine}</span>
-        <span class="gov-pill ${v.direction === 'no' ? 'gov-pill--locked' : 'gov-pill--seeded'}">${v.direction}</span>
+        <span class="gov-pill ${v.direction === 'no' ? 'gov-pill--locked' : 'gov-pill--seeded'}">${v.direction === 'no' ? 'objected' : 'yes'}</span>
         <span class="muted">${when}</span>${objection}</div>`;
     }).join('');
     return `<div class="gov-section">${rows}</div>`;
@@ -742,20 +768,102 @@
 
   // The idea's five sections, IN FRONT of the voter — the entire reason voting
   // happens in the hall rather than on a reaction. Rendered in template order.
+  //
+  // TASK 1003270 — OPEN, AND SET AS PROSE. This used to be a <details> summarised
+  // "The idea, in full · completeness 100/100": the substance the vote is about
+  // sat behind a click while the constitutional metadata above it stayed expanded,
+  // which is exactly backwards. The labels were <code> pills over
+  // `.gov-perm__guards`, so an author's answers were styled as config rows. The
+  // score moved into the collapsed metadata (decidedByHtml) — it describes the
+  // grade the idea cleared, not the idea.
   function ideaSectionsHtml(item) {
     const fv = item.subject && item.subject.field_values;
     if (!fv) return '';
-    const rows = Object.keys(SECTION_LABELS).map((key) => `
-      <div class="gov-perm">
-        <div class="gov-perm__main">
-          <code class="gov-perm__key">${escapeHtml(SECTION_LABELS[key])}</code>
-          <div class="gov-perm__guards">${escapeHtml(String(fv[key] ?? ''))}</div>
-        </div>
-      </div>`).join('');
-    const score = item.subject.completeness_score;
-    return `<details class="gov-section"><summary class="gov-section__head">The idea, in full${score != null ? ` \u00b7 completeness ${escapeHtml(String(score))}/100` : ''}</summary>${rows}</details>`;
+    const rows = Object.keys(SECTION_LABELS).map((key) => {
+      // An unanswered section is dropped rather than rendered as a bare heading:
+      // a graded Full Idea has all five, and a blank one is only noise.
+      const body = String(fv[key] ?? '').trim();
+      if (!body) return '';
+      return `<div class="gov-idea__part">
+        <h4 class="gov-idea__label">${escapeHtml(SECTION_LABELS[key])}</h4>
+        <p class="gov-idea__body">${escapeHtml(body)}</p>
+      </div>`;
+    }).join('');
+    return rows
+      ? `<div class="gov-idea">${rows}</div>`
+      : '<p class="gov-empty">This idea has no written sections.</p>';
+  }
+
+  // ---- the sitting's rules, in plain words (task 1003270) --------------------
+
+  // The card used to carry `tally: 0 yes · 0 no — sitting under <first_ratifier>,
+  // membership <rank:archon>` — raw config keys, in a reader's face, ABOVE the
+  // idea they were being asked to decide. These say the same thing in the words
+  // a person thinks in, filed under a summary where secondary information
+  // belongs.
+  //
+  // Read from the item's SNAPSHOT constitution (the rules in force when the
+  // window opened), never today's config — the same rules the server tallies
+  // under. Rank KEYS never surface: the display label comes from the branding
+  // pack via OTB.rankLabel, as on every other hall surface. The one place a raw
+  // predicate still shows is an UNREADABLE one, where the broken string is the
+  // actionable fact and hiding it would help nobody.
+  function rankNames(keys) {
+    const labels = keys.map((k) => escapeHtml(window.OTB.rankLabel(k)));
+    if (labels.length === 1) return labels[0];
+    return `${labels.slice(0, -1).join(', ')} or ${labels[labels.length - 1]}`;
+  }
+
+  // A sitting length in the unit a person would say it in. The config stores
+  // minutes; "2880 minutes" is a number nobody converts in their head.
+  function fmtWindow(minutes) {
+    const m = Number(minutes);
+    if (!Number.isFinite(m) || m <= 0) return `${escapeHtml(String(minutes))} minutes`;
+    const plural = (n, word) => `${n} ${word}${n === 1 ? '' : 's'}`;
+    if (m % 1440 === 0) return plural(m / 1440, 'day');
+    if (m % 60 === 0) return plural(m / 60, 'hour');
+    return plural(m, 'minute');
   }
 
+  // The short name of a pass rule, for a place a whole sentence will not fit.
+  const PASS_RULE_NAME = {
+    first_ratifier: 'by first ratifier',
+    consent: 'by consent',
+    majority: 'by majority',
+  };
+
+  const PASS_RULE_PLAIN = {
+    first_ratifier: 'The first vote decides it: a yes ratifies, an objection returns it to its author.',
+    consent: 'It passes unless someone objects \u2014 silence is agreement, and one reasoned objection returns it.',
+    majority: 'It passes only when more than half the board votes yes. Silence is not agreement here.',
+  };
+
+  function decidedByHtml(item) {
+    const c = item.constitution || {};
+    const past = !!item.closed_at;
+    const keys = membershipRankKeys(c.membership);
+    const who = keys
+      ? `Every active <strong>${rankNames(keys)}</strong> ${past ? 'could vote on this' : 'may vote on this'}.`
+      : `Nobody ${past ? 'could vote' : 'may vote'} \u2014 the membership setting <code>${escapeHtml(String(c.membership || 'unset'))}</code> cannot be read.`;
+    const rule = PASS_RULE_PLAIN[c.pass_rule] || 'How this sitting passes could not be read.';
+    const clock = c.window_minutes === null || c.window_minutes === undefined
+      ? `No deadline \u2014 it ${past ? 'stayed' : 'stays'} open until a vote decides it.`
+      : `The sitting ${past ? 'ran' : 'runs'} ${fmtWindow(c.window_minutes)}${c.close_early_on_full_turnout ? ', closing early once every member has voted' : ''}.`;
+    const score = item.subject ? item.subject.completeness_score : null;
+    const grade = score == null
+      ? ''
+      : `<p class="gov-grid-note">It cleared the completeness grade at ${escapeHtml(String(score))}/100.</p>`;
+    return `<p class="gov-grid-note">${who}</p>
+      <p class="gov-grid-note">${rule}</p>
+      <p class="gov-grid-note">${clock}</p>${grade}`;
+  }
+
+  // TASK 1003270: the ballot is the point of the card, so its copy earns its
+  // place or goes. What stayed is the ADR 0175 §8 requirement itself — an
+  // objection carries a REASON and names ONE SECTION — because that requirement
+  // is the whole reason voting lives in the hall instead of on a chat reaction.
+  // Simplifying may not weaken it, so the reason box and the five-way picker are
+  // untouched; only the sentences around them shrank.
   function voteFormHtml(item, viewerId) {
     if (item.closed_at) return '';
     const mine = item.votes.find((v) => String(v.voter_id) === String(viewerId));
@@ -770,16 +878,16 @@
     const isOwnIdea = item.subject_type === 'full_idea'
       && item.subject && String(item.subject.author_id) === String(viewerId);
     const sections = Object.keys(SECTION_LABELS).map((key) => `
-      <label class="gov-perm" style="cursor:pointer"><input type="radio" name="vote-section-${escapeHtml(item.id)}" value="${escapeHtml(key)}"> <span>${escapeHtml(SECTION_LABELS[key])}</span></label>`).join('');
+      <label class="gov-perm"><input type="radio" name="vote-section-${escapeHtml(item.id)}" value="${escapeHtml(key)}"> <span>${escapeHtml(SECTION_LABELS[key])}</span></label>`).join('');
     const ownNote = isOwnIdea
-      ? `<p class="gov-grid-note">This is your idea. You can vote on it \u2014 your position belongs on the record \u2014 but it can\u2019t be decided by your vote alone, and a vote on your own idea earns no karma.</p>`
+      ? `<p class="gov-grid-note">Your own idea. You may vote, but it can\u2019t be decided by your vote alone, and a vote on your own idea earns no karma.</p>`
       : '';
     return `<div class="gov-vote" data-item="${escapeHtml(item.id)}">
       ${ownNote}
       <button type="button" class="gov-btn" data-vote="yes">Vote yes</button>
       <details class="gov-vote__no">
         <summary class="gov-btn gov-btn--ghost">Object\u2026</summary>
-        <p class="gov-grid-note">An objection returns the idea to its author with your reason and the section it faults \u2014 the feedback they revise against.</p>
+        <p class="gov-grid-note">This sends the idea back to its author. Say why, and which section.</p>
         <textarea class="gov-input" data-vote-reason placeholder="Why \u2014 the reason the author will act on" maxlength="5000" rows="3"></textarea>
         <div class="gov-vote__sections">${sections}</div>
         <button type="button" class="gov-btn" data-vote="no">Cast objection</button>
@@ -807,27 +915,60 @@
     loadBoard(); // re-render either way — the room shows what stands
   }
 
+  // ONE SITTING, ONE DECISION (task 1003270). The card reads top-down in the
+  // order a person actually needs it:
+  //
+  //   1. what is being decided — the idea's TITLE, at title size. It used to
+  //      render through `.gov-section__head`, i.e. 11.5px uppercase letterspaced:
+  //      the subject of the vote styled as a table header.
+  //   2. the ACTION, immediately under it. Everything that used to sit between
+  //      the two — a tally of raw config keys, a sentence explaining that the UI
+  //      had no vote form (it does), and a collapsed idea — is gone or moved, so
+  //      "Vote yes" lands above the fold on an ordinary window instead of below
+  //      a wall of prose.
+  //   3. the idea itself, open (ideaSectionsHtml).
+  //   4. the constitutional metadata, collapsed — who votes, how it passes, the
+  //      clock, and the open ballot. Still one click away, never invisible: the
+  //      tally rides the summary, which is the part of it a reader scans for
+  //      (ADR 0175 §9 — a board whose votes are invisible constrains nobody).
+  //
+  // A CLOSED sitting has no action and no idea body: its head carries the
+  // outcome and the record lives under the same summary.
   function itemCard(item, viewerId) {
     const yes = item.votes.filter((v) => v.direction === 'yes').length;
     const no = item.votes.filter((v) => v.direction === 'no').length;
-    const state = item.closed_at
-      ? `<span class="gov-pill ${item.outcome === 'passed' ? 'gov-pill--seeded' : 'gov-pill--locked'}">${escapeHtml(item.outcome || '')}</span>`
-      : `<span class="gov-pill gov-pill--custom">open</span> <span class="muted">${escapeHtml(clockLine(item))}</span>`;
+    const isOpen = !item.closed_at;
+    const state = isOpen
+      ? `<span class="gov-pill gov-pill--custom">open</span> <span class="muted">${escapeHtml(clockLine(item))}</span>`
+      : `<span class="gov-pill ${item.outcome === 'passed' ? 'gov-pill--seeded' : 'gov-pill--locked'}">${escapeHtml(item.outcome || '')}</span>`;
     const amendmentDetail = item.subject_type === 'constitutional_amendment' && item.subject
-      ? `<div class="gov-perm__guards">proposes: <code>${escapeHtml(JSON.stringify(item.subject.proposed))}</code>${item.subject.rationale_md ? ` — ${escapeHtml(item.subject.rationale_md)}` : ''}</div>`
+      ? `<p class="gov-item__amend">Proposes <code>${escapeHtml(JSON.stringify(item.subject.proposed))}</code>${item.subject.rationale_md ? ` \u2014 ${escapeHtml(item.subject.rationale_md)}` : ''}</p>`
       : '';
     const mine = item.votes.find((v) => String(v.voter_id) === String(viewerId));
-    const myLine = item.closed_at ? '' : (mine
-      ? `<p class="gov-grid-note">You voted <strong>${escapeHtml(mine.direction)}</strong>. One vote per member — no changes.</p>`
-      : `<p class="gov-grid-note">You have not voted on this item. Casting arrives with the vote form.</p>`);
-    return `<section class="gov-section" id="board-item-${escapeHtml(item.id)}">
-      <div class="gov-section__head">${subjectLine(item)} ${state}</div>
+    // A voter who has cast sees WHAT they cast where the buttons were. The line
+    // that used to fill this slot for everyone else — announcing that the reader
+    // had not voted and that casting was still to come — is deleted outright: it
+    // explained the UI to itself, and the form it promised shipped in R17.
+    const action = !isOpen ? ''
+      : mine
+        ? `<p class="gov-item__cast">You ${mine.direction === 'no' ? 'objected' : 'voted yes'}. One vote per member.</p>`
+        : voteFormHtml(item, viewerId);
+    const tally = `${yes} yes \u00b7 ${no} objection${no === 1 ? '' : 's'}`;
+    return `<section class="gov-item" id="board-item-${escapeHtml(item.id)}">
+      <div class="gov-item__head">
+        <h3 class="gov-item__title">${subjectLine(item)}</h3>
+        <span class="gov-item__state">${state}</span>
+      </div>
       ${amendmentDetail}
-      <p class="gov-grid-note">tally: ${yes} yes · ${no} no — sitting under <code>${escapeHtml((item.constitution && item.constitution.pass_rule) || '?')}</code>, membership <code>${escapeHtml((item.constitution && item.constitution.membership) || '?')}</code></p>
-      ${myLine}
-      ${item.closed_at ? '' : ideaSectionsHtml(item)}
-      ${ballotHtml(item, viewerId)}
-      ${voteFormHtml(item, viewerId)}
+      ${action ? `<div class="gov-item__act">${action}</div>` : ''}
+      ${isOpen ? ideaSectionsHtml(item) : ''}
+      <details class="gov-item__meta">
+        <summary class="gov-item__summary">${tally} \u00b7 who votes, and how it ${isOpen ? 'passes' : 'passed'}</summary>
+        <div class="gov-item__metabody">
+          ${decidedByHtml(item)}
+          ${ballotHtml(item, viewerId)}
+        </div>
+      </details>
     </section>`;
   }
 
@@ -905,7 +1046,9 @@
     const rank = (me && (me.builder || me).rank || '').toLowerCase();
     const isArchon = rank === 'archon';
 
-    document.getElementById('gov-sub').textContent = 'How this project is governed — the constitution it runs, the Board Room, and the ranks & permissions that administer it.';
+    // Task 1003270: the subtitle used to restate the three tab labels sitting
+    // directly beneath it, in ~20 words, above everything actionable.
+    document.getElementById('gov-sub').textContent = 'How this project is governed.';
     document.getElementById('gov-body').hidden = false;
     wireTabs({ isArchon });
     loadBoard();        // the room loads for everyone the page admits

package/package-lock.json

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

package/package.json

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

@@ -1212,8 +1212,10 @@ const { validateOrRespond, LIMITS, parseId, asyncHandler, corsPublicGet, parsePa
 //         landed since 1.19.384 with no explicit bump. run 33118682829. (task 1002620)
 //   1.19.386 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
 //         landed since 1.19.385 with no explicit bump. run 33120468176. (task 1002620)
+//   1.19.387 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+//         landed since 1.19.386 with no explicit bump. run 33121066592. (task 1002620)
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.386'; // CI auto-patch carrier (ADR 0161); see changelog above
+const CORE_VERSION = '1.19.387'; // 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_room_copy.mjs

@@ -0,0 +1,131 @@
+// tests/government_board_room_copy.mjs — the Board Room card reads as one
+// decision (task 1003270, goal 1000068).
+//
+// The owner's report was that the room was "extremely poor and unclear": the
+// idea being voted on was collapsed while raw config keys — `first_ratifier`,
+// `rank:archon`, `why_matters` — were expanded above it, and the Vote button
+// sat under a wall of prose. Copy rots back, so the two rules that came out of
+// that pass are pinned here rather than left to the next reader's judgment:
+//
+//   1. NO RAW CONFIG KEY REACHES A READER. Every pass rule the server can hold
+//      has a plain sentence, checked against the REAL PASS_RULES list — a rule
+//      added server-side with no copy fails here instead of shipping a card
+//      that prints its enum. Rank keys go through OTB.rankLabel.
+//   2. THE ORDER IS title → action → idea → apparatus. That order IS the fix;
+//      an edit that puts prose back between the title and the ballot has undone
+//      the task.
+//
+// Source pins, like tests/government_board_vote_ui.mjs: the hall script is an
+// IIFE over the DOM and cannot be imported into node wholesale.
+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 CLIENT = fs.readFileSync(path.join(ROOT, 'modules', 'hall-ui', 'public', 'government.js'), 'utf8');
+const HTML = fs.readFileSync(path.join(ROOT, 'modules', 'hall-ui', 'public', 'government.html'), 'utf8');
+const { PASS_RULES } = require(path.join(ROOT, 'modules', 'government', 'config.js'));
+
+const card = () => CLIENT.slice(CLIENT.indexOf('function itemCard'), CLIENT.indexOf('async function loadBoard'));
+
+// Deleted copy is asserted against the RENDERING source only. The comments in
+// government.js deliberately quote what was removed and why — that history is
+// the point of them, and a test that forbade it would push the record out of
+// the file it explains.
+const CODE = CLIENT.split(/\r?\n/).filter((l) => !/^\s*\/\//.test(l)).join('\n');
+
+test('every pass rule the server can hold has a plain-language sentence', () => {
+  const block = CLIENT.slice(CLIENT.indexOf('const PASS_RULE_PLAIN'), CLIENT.indexOf('function decidedByHtml'));
+  assert.ok(PASS_RULES.length >= 3, `sanity: found the real rule set (${PASS_RULES})`);
+  for (const rule of PASS_RULES) {
+    assert.ok(block.includes(`${rule}:`), `PASS_RULE_PLAIN must explain '${rule}' — a rule with no copy renders its enum key`);
+  }
+  // And the sentence must not simply echo the key back at the reader.
+  for (const rule of PASS_RULES) {
+    const line = block.split('\n').find((l) => l.trim().startsWith(`${rule}:`));
+    assert.ok(!/'[^']*\b(first_ratifier|consent_rule|pass_rule)\b/.test(line), `'${rule}' copy leaks a config key: ${line}`);
+  }
+});
+
+test('the short name map covers the same rule set (the history line)', () => {
+  const block = CLIENT.slice(CLIENT.indexOf('const PASS_RULE_NAME'), CLIENT.indexOf('const PASS_RULE_PLAIN'));
+  for (const rule of PASS_RULES) {
+    assert.ok(block.includes(`${rule}:`), `PASS_RULE_NAME must name '${rule}'`);
+  }
+});
+
+test('a rank reaches the reader as a LABEL, never as the enum key', () => {
+  assert.match(CLIENT, /function rankNames\(keys\)[\s\S]*?window\.OTB\.rankLabel\(k\)/,
+    'the board card resolves rank keys through the branding pack');
+  const membership = CLIENT.slice(CLIENT.indexOf('function membershipSentence'), CLIENT.indexOf('function passRuleSentence'));
+  assert.match(membership, /window\.OTB\.rankLabel\(k\)/,
+    'so does the Constitution sentence — `archon` is an enum, "Owner" is what a person reads');
+});
+
+test('the card order is title → action → the idea → the apparatus', () => {
+  const c = card();
+  const at = (needle) => {
+    const i = c.indexOf(needle);
+    assert.notEqual(i, -1, `itemCard no longer contains ${needle}`);
+    return i;
+  };
+  assert.ok(at('gov-item__title') < at('gov-item__act'),
+    'the action follows the title directly — that adjacency is the whole fix');
+  assert.ok(at('gov-item__act') < at('ideaSectionsHtml(item)'),
+    'nothing may be inserted between the title and the ballot');
+  assert.ok(at('ideaSectionsHtml(item)') < at('decidedByHtml(item)'),
+    'the idea outranks the rules it is decided under');
+  assert.ok(at('decidedByHtml(item)') < at('ballotHtml(item, viewerId)'),
+    'who votes, then who has');
+});
+
+test('the idea renders OPEN — never behind a summary', () => {
+  const sections = CLIENT.slice(CLIENT.indexOf('function ideaSectionsHtml'), CLIENT.indexOf('function rankNames'));
+  assert.ok(!/<details/.test(sections),
+    'the substance of the vote is not a disclosure — that was the reported bug');
+  assert.match(sections, /gov-idea__body/, 'and it is set as prose, not as a config row');
+});
+
+test('the tally stays visible without a click (ADR 0175 §9)', () => {
+  const c = card();
+  assert.match(c, /gov-item__summary">\$\{tally\}/,
+    'a board whose votes are invisible constrains nobody — the count rides the summary');
+});
+
+test('the self-explaining copy is DELETED, not reworded', () => {
+  assert.ok(!/Casting arrives with the vote form/.test(CODE),
+    'the sentence that explained the UI to itself — and the form it promised has shipped');
+  assert.ok(!/tally: \$\{yes\} yes/.test(CLIENT), 'the raw-config tally line is gone');
+  assert.ok(!/The idea, in full/.test(CODE), 'so is the summary that hid the idea');
+});
+
+test('the section a voter faults is shown by NAME, in the ballot and the picker', () => {
+  const ballot = CLIENT.slice(CLIENT.indexOf('function ballotHtml'), CLIENT.indexOf('// ---- the vote form'));
+  assert.match(ballot, /SECTION_LABELS\[v\.section_key\]/,
+    '`why_matters` is a jsonb key; the objector picked a named section and the reader should see that name');
+});
+
+test('the objection still carries a reason AND one section (ADR 0175 §8)', () => {
+  // The requirement simplifying may not weaken: it is why voting lives in the
+  // hall instead of on a chat reaction.
+  const form = CLIENT.slice(CLIENT.indexOf('function voteFormHtml'), CLIENT.indexOf('async function castVoteFromRoom'));
+  assert.match(form, /data-vote-reason/, 'the reason box survives');
+  assert.match(form, /gov-vote__sections/, 'so does the five-way section picker');
+  assert.match(form, /Object\.keys\(SECTION_LABELS\)\.map/, 'with one radio per declared section');
+});
+
+test('the page frames its panels in one line each, not a paragraph', () => {
+  // The two intros used to run ~60 and ~50 words before anything actionable.
+  for (const heading of ['Constitution', 'Board Room']) {
+    const i = HTML.indexOf(`<h2 class="gov-h">${heading}</h2>`);
+    assert.notEqual(i, -1, `${heading} panel heading missing`);
+    const note = /<p class="gov-grid-note">([\s\S]*?)<\/p>/.exec(HTML.slice(i));
+    assert.ok(note, `${heading} has no framing line`);
+    const words = note[1].replace(/<[^>]+>/g, ' ').trim().split(/\s+/).length;
+    assert.ok(words <= 30, `${heading}'s intro is ${words} words — a reader meets it before anything actionable`);
+  }
+});