@bongos/core 1.24.46 → 1.24.48

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

@@ -60,6 +60,9 @@
    shadows, motion, stacking — every surface loads. Imported FIRST so this
    sheet's own rules follow it; the hall's scale names below are aliases of it. */
 @import url("/modules/ui-design/public/tokens.css");
+/* The hall's one button and one field (task 1005084): .btn and its modifiers,
+   the text field / select / textarea. Straight after the scales it spends. */
+@import url("controls.css");
 @import url("page-notes.css");
 @import url("craft-review.css");
 
@@ -430,20 +433,7 @@ code, kbd, pre {
    the composition around it. */
 .num, .tabular { font-variant-numeric: tabular-nums; }
 
-select, input[type="text"], input[type="search"], textarea {
-  font-family: var(--font-body);
-  font-size: 14px;
-  color: var(--ink);
-  background: none;
-  border: 1px solid var(--rule);
-  border-radius: var(--radius-pill);
-  min-height: 30px;
-  padding: 5px 12px;
-}
-select:hover, input[type="text"]:hover, input[type="search"]:hover { border-color: var(--ink-faint); }
-textarea { border-radius: var(--radius-sm); }
-select:disabled { opacity: 0.55; cursor: not-allowed; }
-select option { background: var(--bg-card); color: var(--ink); }
+/* The field (text input, select, textarea) is controls.css's (task 1005084). */
 
 button { font-family: var(--font-body); }
 
@@ -1422,8 +1412,7 @@ html.setup-gate-open { overflow: hidden; }
 }
 .setup-gate code { font-family: var(--font-mono); font-size: 14px; padding: 1px 6px; border-radius: 6px; background: var(--glass); }
 .setup-gate__crafts { display: flex; flex-wrap: wrap; gap: 8px; margin-top: 12px; }
-.setup-gate__craft,
-.setup-gate__save {
+.setup-gate__craft {
   min-height: 36px;
   padding: 0 16px;
   border-radius: var(--radius-pill);
@@ -1435,8 +1424,8 @@ html.setup-gate-open { overflow: hidden; }
   cursor: pointer;
 }
 .setup-gate__craft[aria-pressed="true"] { border-color: var(--accent); background: var(--accent-tint); }
-.setup-gate__save { margin-top: 12px; background: var(--accent); border-color: var(--accent); color: var(--accent-on); }
-.setup-gate__save:disabled { opacity: .5; cursor: default; }
+/* The save is the shared primary door (controls.css, task 1005084). */
+.setup-gate__save { margin-top: 12px; }
 .setup-gate__err { margin: 8px 0 0; font-size: 14px; color: var(--danger); }
 .setup-gate__foot { margin: 24px 0 0; font-size: 14px; }
 .setup-gate__foot a { color: var(--ink-soft); }
@@ -2634,23 +2623,16 @@ main [id] { scroll-margin-top: calc(var(--topbar-h) + 16px); }
      wherever there is room and drops it where there is not. */
   min-width: min(150px, 100%);
   max-width: 100%;
-  min-height: 32px;
-  font-family: var(--font-body);
-  padding: 4px 8px;
-  font-size: 13.5px;
   text-transform: none;
   letter-spacing: normal;
   font-weight: 400;
 }
-/* task 1829: the Work Board free-text search box, sized to match the filter
-   selects it sits alongside. */
+/* task 1829: the Work Board free-text search box. Both it and the selects are
+   the shared field (controls.css, task 1005084); the filter only sets widths
+   and undoes the label voice they would inherit. */
 .filter input[type="search"] {
   min-width: min(200px, 100%);
   max-width: 100%;
-  min-height: 32px;
-  font-family: var(--font-body);
-  padding: 4px 8px;
-  font-size: 13.5px;
   text-transform: none;
   letter-spacing: normal;
   font-weight: 400;
@@ -2673,94 +2655,8 @@ main [id] { scroll-margin-top: calc(var(--topbar-h) + 16px); }
 }
 .pager__range { font-family: var(--font-mono); font-size: 12px; color: var(--ink-faint); font-variant-numeric: tabular-nums; }
 
-/* Pill or nothing (DESIGN.md, Shapes). A ghost button is a hairline pill; the
-   one filled shape on a page is the accent door, and only where the action is
-   the live thing. 28px of box plus the row’s own air clears the 24px floor. */
-.btn-ghost {
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  gap: 6px;
-  min-height: 30px;
-  min-width: 56px;
-  padding: 0 14px;
-  font-size: 13px;
-  text-transform: lowercase;
-  color: var(--ink-soft);
-  background: none;
-  border: 1px solid var(--rule);
-  border-radius: var(--radius-pill);
-  cursor: pointer;
-}
-.btn-ghost:hover:not(:disabled) { border-color: var(--ink-faint); color: var(--ink); text-decoration: none; }
-.btn-ghost:disabled { color: var(--ink-faint); border-color: var(--rule-soft); cursor: default; }
-
-/* The accent door: the page’s one primary action. --accent-on is the dark ink
-   that goes on the salmon; white on it fails AA and is banned. */
-.btn-accent {
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  gap: 6px;
-  min-height: 34px;
-  min-width: 74px;
-  padding: 0 16px;
-  font-size: 13.5px;
-  font-weight: 500;
-  text-transform: lowercase;
-  color: var(--accent-on);
-  background: var(--accent);
-  border: 0;
-  border-radius: var(--radius-pill);
-  cursor: pointer;
-}
-.btn-accent:hover:not(:disabled) { background: var(--accent-lift); color: var(--accent-on); text-decoration: none; }
-.btn-accent:disabled { background: var(--glass); color: var(--ink-faint); cursor: default; }
-
-/* `.btn` / `.btn--ghost` — the SECOND button vocabulary in the hall, used by
-   ideas, idea, copy-desk, my-projects and collab. Task 1003305 found it had no
-   rule anywhere in the surface: every one of those buttons was rendering as a
-   browser default control, on a page whose every other shape is a pill. They
-   are not renamed to .btn-accent/.btn-ghost because the class is spelled into
-   markup and into tests/idea_spark_hall.mjs; instead the two vocabularies are
-   made to say the same thing. `.btn` is the accent door and `.btn--ghost` its
-   quiet sibling, which is exactly how the calling pages already use the pair
-   (submit vs. discard, mark vs. withdraw). --accent-on is the dark ink for the
-   salmon; white on it fails AA and DESIGN.md bans it. */
-.btn {
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  gap: 6px;
-  min-height: 34px;
-  min-width: 74px;
-  padding: 0 16px;
-  font-family: var(--font-body);
-  font-size: 13.5px;
-  font-weight: 500;
-  text-transform: lowercase;
-  color: var(--accent-on);
-  background: var(--accent);
-  /* transparent rather than absent: collab.css's `.btn--quiet` sets only a
-     border-COLOUR, and a modifier cannot paint an edge that was never reserved. */
-  border: var(--line) solid transparent;
-  border-radius: var(--radius-pill);
-  cursor: pointer;
-}
-.btn:hover:not(:disabled) { background: var(--accent-lift); color: var(--accent-on); text-decoration: none; }
-.btn:disabled { background: var(--glass); color: var(--ink-faint); cursor: default; }
-.btn--ghost {
-  min-height: 30px;
-  min-width: 56px;
-  padding: 0 14px;
-  font-size: 13px;
-  font-weight: 400;
-  color: var(--ink-soft);
-  background: none;
-  border-color: var(--rule);
-}
-.btn--ghost:hover:not(:disabled) { background: none; border-color: var(--ink-faint); color: var(--ink); }
-.btn--ghost:disabled { background: none; color: var(--ink-faint); border-color: var(--rule-soft); }
+/* The buttons — .btn and its modifiers, and the older names pages still write
+   (.btn-accent, .btn-ghost, .btn--ghost …) — are controls.css's (task 1005084). */
 
 /* ---------- profile / standing ---------- */
 
@@ -3040,11 +2936,8 @@ main [id] { scroll-margin-top: calc(var(--topbar-h) + 16px); }
    gap keeps the label and the select together when the lede wraps. */
 .curve__filter { display: inline-flex; align-items: center; gap: 6px; margin-left: 10px; }
 .curve__filter-label { color: var(--ink-soft); }
-.curve__filter-select {
-  font: inherit; color: inherit; background: var(--surface-2);
-  border: 1px solid var(--border-strong); border-radius: var(--radius-sm);
-  padding: 2px 6px; max-width: 16ch;
-}
+/* the shared field (controls.css); only its width is the filter's */
+.curve__filter-select { max-width: 16ch; }
 
 .curve__headline {
   display: grid;
@@ -3297,22 +3190,6 @@ main [id] { scroll-margin-top: calc(var(--topbar-h) + 16px); }
 }
 .setup__divider::before, .setup__divider::after { content: ""; flex: 1; border-top: 1px solid var(--rule-soft); }
 
-.setup__btn {
-  display: inline-flex;
-  align-items: center;
-  gap: 8px;
-  height: 34px;
-  padding: 0 16px;
-  border-radius: var(--radius-sm);
-  border: 1px solid var(--accent);
-  background: var(--accent);
-  color: #fff;
-  font-size: 13.5px;
-  font-weight: 600;
-  cursor: pointer;
-}
-.setup__btn:hover:not(:disabled) { background: var(--accent-deep); border-color: var(--accent-deep); color: #fff; text-decoration: none; }
-.setup__btn:disabled, .setup__btn--open[aria-disabled="true"] { opacity: 0.5; cursor: not-allowed; }
 .setup__getcode { margin: 10px 0; }
 /* A <summary> is a control, so it owes the 24px floor like any other; it
    measured 20px tall (task 1003304). The height is bought with padding rather
@@ -3387,31 +3264,9 @@ html[data-door="out"] #hall-main { width: 100%; }
   line-height: 18px;
 }
 .landing__cta { margin: 14px 0; }
-.landing__btn {
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  gap: 10px;
-  min-height: 46px;
-  padding: 0 22px;
-  border-radius: var(--radius-pill);
-  border: 0;
-  background: var(--accent);
-  color: var(--accent-on);
-  font-size: 14px;
-  font-weight: 500;
-  text-transform: lowercase;
-  cursor: pointer;
-}
-.landing__btn:hover { background: var(--accent-lift); text-decoration: none; color: var(--accent-on); }
+/* .landing__btn is the shared primary door, --quiet its ghost (controls.css). */
 .landing__btn--block { display: flex; width: 100%; }
 .landing__gh { flex-shrink: 0; }
-.landing__btn--quiet {
-  background: transparent;
-  color: var(--ink);
-  box-shadow: inset 0 0 0 1px var(--border-strong);
-}
-.landing__btn--quiet:hover { background: var(--surface-2); color: var(--ink); }
 .landing__note { font-size: 12.5px; color: var(--ink-faint); }
 .landing__error { color: var(--danger); }
 .landing__error-h { color: var(--danger); margin: 0 0 4px; font-size: 14px; }
@@ -3642,29 +3497,8 @@ html[data-door="out"] #hall-main { width: 100%; }
    the touches chip and the whole-card selection target (promoted from work.css,
    task 1003659, so their cascade position is the same on every page). */
 
-/* The page's one primary action, and the only accent FILL on the board.
-   --accent-on is the dark ink that goes on the salmon: white on it fails AA and
-   DESIGN.md bans it outright, which is what the old `color: #fff` was. */
-.task__claim {
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  min-height: 34px;
-  min-width: 74px;
-  padding: 0 16px;
-  background: var(--accent);
-  border: 0;
-  color: var(--accent-on);
-  border-radius: var(--radius-pill);
-  font-size: 13.5px;
-  font-weight: 500;
-  text-transform: lowercase;
-  cursor: pointer;
-  white-space: nowrap;
-  transition: background 150ms, opacity 150ms;
-}
-.task__claim:hover:not(:disabled) { background: var(--accent-lift); color: var(--accent-on); }
-.task__claim:disabled { background: var(--glass); color: var(--ink-faint); cursor: default; }
+/* The page's one primary action, and the only accent FILL on the board: the
+   shared primary door at the row size (controls.css, task 1005084). */
 
 /* Files a claimable task is responsible for — a compact count chip in the meta
    row; the full path list rides the tooltip (task 2170 density pass). The mono
@@ -3778,27 +3612,13 @@ html[data-door="out"] #hall-main { width: 100%; }
   color: var(--ink-faint);
 }
 
-.wip__release {
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  min-height: 30px;
-  padding: 0 14px;
-  background: none;
-  border: var(--line) solid var(--rule);
-  color: var(--ink-soft);
-  border-radius: var(--radius-pill);
-  font-size: 13px;
-  text-transform: lowercase;
-  cursor: pointer;
-  transition: background 150ms, color 150ms, border-color 150ms;
-}
+/* .wip__release is the shared ghost at the row size (controls.css); a release
+   is the undo of a claim, so its hover warns. */
 .wip__release:hover:not(:disabled) {
   background: var(--danger-weak);
   color: var(--danger);
   border-color: color-mix(in srgb, var(--danger) 40%, transparent);
 }
-.wip__release:disabled { opacity: 0.55; cursor: default; }
 
 .wip__select {
   appearance: none;
@@ -4064,38 +3884,8 @@ html[data-door="out"] #hall-main { width: 100%; }
 .td-dep:hover { text-decoration: underline; }
 .td-dep-status { color: var(--ink-faint); font-size: 12px; margin-left: 6px; }
 .td-actions { display: flex; flex-wrap: wrap; gap: 10px; align-items: center; margin: 18px 0 0; }
-/* Pill or nothing (DESIGN.md, Shapes): the ghost pill, with --btn--primary the
-   one accent door. It carried a literal `#fff` on the salmon until task 1003305
-   — the exact pairing DESIGN.md bans, because it fails AA. */
-.td-btn {
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  font: inherit;
-  font-size: 13px;
-  min-height: 30px;
-  min-width: 56px;
-  padding: 0 14px;
-  border: var(--line) solid var(--rule);
-  border-radius: var(--radius-pill);
-  background: none;
-  color: var(--ink-soft);
-  text-transform: lowercase;
-  cursor: pointer;
-}
-.td-btn:hover:not(:disabled) { border-color: var(--ink-faint); color: var(--ink); text-decoration: none; }
-.td-btn--primary {
-  min-height: 34px;
-  min-width: 74px;
-  padding: 0 16px;
-  font-weight: 500;
-  background: var(--accent);
-  color: var(--accent-on);
-  border-color: var(--accent);
-}
-.td-btn--primary:hover:not(:disabled) { background: var(--accent-lift); border-color: var(--accent-lift); color: var(--accent-on); }
-.td-btn:disabled { color: var(--ink-faint); border-color: var(--rule-soft); cursor: default; }
-.td-btn--primary:disabled { background: var(--glass); }
+/* .td-btn is the shared ghost at the row size and .td-btn--primary the accent
+   door (controls.css, task 1005084). */
 .td-action-note { flex-basis: 100%; font-size: 12.5px; color: var(--ink-soft); margin: 4px 0 0; }
 
 /* Idea triage inside the record overlay (task 1002447). A two-column grid so
@@ -4103,17 +3893,9 @@ html[data-door="out"] #hall-main { width: 100%; }
 .td-form { display: grid; grid-template-columns: 1fr 1fr; gap: 8px 10px; margin: 14px 0 0; }
 .td-field { display: flex; flex-direction: column; gap: 3px; min-width: 0; }
 .td-field--wide { grid-column: 1 / -1; }
-/* The base `select, input` rule already carries the world's field; this only
-   re-states the sizing the two-column grid needs. */
-.td-field input, .td-field select {
-  font-size: 13px;
-  min-width: 0;
-  max-width: 100%;
-}
-/* The 24px floor (task 1004301 found it failing): at 13px the base field's padding left a
-   number input 21px tall beside a select that clears it — the task page's "Wait for task"
-   box. Sized to sit level with its select. */
-.td-field input { min-height: 26px; box-sizing: border-box; }
+/* The field is the shared one (controls.css); the two-column grid only needs
+   it to shrink with its column. */
+.td-field input, .td-field select { min-width: 0; max-width: 100%; }
 .td-form .td-actions, .td-form-hint, .td-form .td-action-note { grid-column: 1 / -1; }
 .td-form .td-actions { margin: 4px 0 0; }
 .td-form-hint { font-size: 12.5px; color: var(--ink-soft); margin: 0; }
@@ -4316,7 +4098,6 @@ html[data-door="out"] #hall-main { width: 100%; }
 .filter-views__tag { font-size: 11px; color: var(--ink-faint); margin-left: 6px; }
 .filter-views__empty, .filter-views__note { font-size: 12.5px; color: var(--ink-soft); margin: 0 0 10px; }
 .filter-views__note:empty { display: none; }
-.filter-views__row .btn-ghost { min-height: 24px; padding: 0 8px; font-size: 12px; }
 .filter-views__save { display: flex; flex-wrap: wrap; align-items: flex-end; gap: 8px; }
 .filter-views__save .filter { flex: 1 1 180px; }
 /* On a phone the button can sit anywhere in a wrapped row, so the menu spans the
@@ -4378,9 +4159,6 @@ html[data-door="out"] #hall-main { width: 100%; }
    than edit the markup, the name joins the lede it meant. */
 .scroll__note { margin: 0 0 14px; font-size: 13.5px; color: var(--ink-soft); max-width: var(--measure); }
 
-/* collab.js:73-74 — the settle buttons asked for a small variant of .btn that
-   does not exist, so "Someone helped" rendered full-size on every row. */
-.btn--sm { min-height: 26px; padding: 0 11px; font-size: 12.5px; }
 
 /* The notice line: roster.js and blockers.js notice() set body.className='note'
    for an empty / error message, and neither page's sheet defines it. */

package/modules/ui-design/scripts/gen-design-tokens.js

@@ -0,0 +1,102 @@
+#!/usr/bin/env node
+'use strict';
+// modules/ui-design/scripts/gen-design-tokens.js — build config/design-tokens.neutral.json
+// from the two files that already own its values (task 1005082, goal 1000138).
+//
+// WHY. The design-tool token file (read by the Claude Design and Figma adapters)
+// was hand-written, and its spacing (4/8/16/24/40/64) and motion (200/250ms) had
+// drifted from what every page actually uses (the 6/10/14/22/34/52 rhythm and the
+// 150/200ms durations in the shared token sheet). A designer starting from it got
+// a spacing scale no page has. Now nothing in that file is typed by hand:
+//
+//   colours + font stacks + font links  ← config/branding.neutral.json theme.ui
+//   spacing, corners, motion             ← modules/ui-design/public/tokens.css (--ds-*)
+//
+// The file keeps its existing key names (config/design-tokens.schema.json closes
+// them, and an instance's config/design-tokens.json overrides by those names); each
+// key is mapped to the --ds-* step it means, below.
+//
+//   node modules/ui-design/scripts/gen-design-tokens.js           write the file
+//   node modules/ui-design/scripts/gen-design-tokens.js --check   exit 1 if it is stale
+//
+// Guarded by tests/design_tokens_generated.mjs.
+
+const fs = require('node:fs');
+const path = require('node:path');
+
+const ROOT = path.resolve(__dirname, '..', '..', '..');
+const SHEET = path.join(ROOT, 'modules', 'ui-design', 'public', 'tokens.css');
+const PACK = path.join(ROOT, 'config', 'branding.neutral.json');
+const OUT = path.join(ROOT, 'config', 'design-tokens.neutral.json');
+
+// The schema's key → the --ds-* custom property it is generated from.
+const MAP = Object.freeze({
+  scale: { xs: '--ds-space-hair', sm: '--ds-space-xs', md: '--ds-space-sm', lg: '--ds-space-md', xl: '--ds-space-lg', '2xl': '--ds-space-xl' },
+  radius: { sm: '--ds-radius-tile', md: '--ds-radius-panel', lg: '--ds-radius-panel', pill: '--ds-radius-pill' },
+  motion: { fast: '--ds-dur-fast', normal: '--ds-dur-base', slow: '--ds-dur-arrive', easing: '--ds-ease' },
+});
+
+// The pack carries no mono stack (it is not one of the fifteen), so the design
+// side states the label/mono face DESIGN.md names (typography.label).
+const MONO = '"JetBrains Mono", ui-monospace, "Cascadia Mono", Consolas, monospace';
+
+const NOTE = 'GENERATED by node modules/ui-design/scripts/gen-design-tokens.js; do not edit by hand. '
+  + 'Colours, font stacks and font links are config/branding.neutral.json theme.ui; scale, radius and motion are '
+  + 'the --ds-* steps of modules/ui-design/public/tokens.css (the mapping is in the generator). To override, copy '
+  + 'values into config/design-tokens.json (the instance pack). See DESIGN.md "The hall" and '
+  + 'modules/ui-design/docs/design-contract.md.';
+
+// The first :root { … } block of the sheet: the light/base values. The dark and
+// reduced-motion blocks restate a few of them and are deliberately not read.
+function readSheetVars(css) {
+  const noComments = css.replace(/\/\*[\s\S]*?\*\//g, '');
+  const m = noComments.match(/:root\s*\{([^}]*)\}/);
+  if (!m) throw new Error('tokens.css: no :root block');
+  const vars = {};
+  for (const decl of m[1].split(';')) {
+    const d = decl.match(/^\s*(--[a-z0-9-]+)\s*:\s*([\s\S]+?)\s*$/i);
+    if (d) vars[d[1]] = d[2].replace(/\s+/g, ' ');
+  }
+  return vars;
+}
+
+function build({ sheet = fs.readFileSync(SHEET, 'utf8'), pack = JSON.parse(fs.readFileSync(PACK, 'utf8')) } = {}) {
+  const vars = readSheetVars(sheet);
+  const ui = (pack.theme && pack.theme.ui) || {};
+  const pick = (group) => {
+    const out = {};
+    for (const [key, prop] of Object.entries(MAP[group])) {
+      if (!(prop in vars)) throw new Error(`tokens.css: ${prop} (for ${group}.${key}) is not declared`);
+      out[key] = vars[prop];
+    }
+    return out;
+  };
+  const fonts = { display: ui.fonts.display, body: ui.fonts.body, mono: MONO };
+  if (Array.isArray(ui.fonts.links)) fonts.links = ui.fonts.links.slice();
+  const tokens = {
+    _note: NOTE,
+    colors: Object.assign({}, ui.colors),
+    fonts,
+    scale: pick('scale'),
+    radius: pick('radius'),
+    motion: pick('motion'),
+  };
+  return JSON.stringify(tokens, null, 2) + '\n';
+}
+
+module.exports = { build, readSheetVars, MAP, OUT };
+
+if (require.main === module) {
+  const next = build();
+  const current = fs.existsSync(OUT) ? fs.readFileSync(OUT, 'utf8') : '';
+  if (process.argv.includes('--check')) {
+    if (current === next) { process.stdout.write('design-tokens.neutral.json: current\n'); process.exitCode = 0; }
+    else {
+      process.stderr.write('design-tokens.neutral.json is STALE: run node modules/ui-design/scripts/gen-design-tokens.js\n');
+      process.exitCode = 1;
+    }
+  } else {
+    if (current !== next) fs.writeFileSync(OUT, next);
+    process.stdout.write((current === next ? 'design-tokens.neutral.json: unchanged' : 'design-tokens.neutral.json: written') + '\n');
+  }
+}

package/package-lock.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.24.46",
+  "version": "1.24.48",
   "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/release-notes.json

@@ -11247,5 +11247,17 @@
       "id": "1005080",
       "text": "The screenshot checker builders use before shipping a design change shows the real hall pages again, instead of a pop-up covering every page. Two small accessibility fixes came with it: a few tiny links are now big enough to t"
     }
+  ],
+  "1.24.47": [
+    {
+      "id": "1005082",
+      "text": "The design rulebook now describes the builders' hall as it really is, and spells out the name: 'cloud bongos' in the logo, 'Cloud Bongos' in sentences. The design-tool token file is now built automatically from the real styles"
+    }
+  ],
+  "1.24.48": [
+    {
+      "id": "1005084",
+      "text": "The builders' hall now has one button and one text box style everywhere instead of eight button sizes and five box styles; main buttons are the full 46px pill with dark text on the orange, and the setup screen's save button is"
+    }
   ]
 }

package/src/module-api.js

@@ -81,7 +81,7 @@ const { sleep } = require('./sleep');
 // MAJOR (see allowBoxScope below): passes the request through untouched.
 function deprecatedNoopMiddleware(_req, _res, next) { next(); }
 
-const CORE_VERSION = '1.24.46'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.24.48'; // 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/bulk_toolbar.mjs

@@ -196,7 +196,10 @@ test('goals.html no longer borrows work.css, and work.css no longer holds what i
     assert.ok(!new RegExp(`^${sel.replace('.', '\\.')}(\\s|:|,|\\{)`, 'm').test(work), `work.css must not restate ${sel}`);
   }
   const style = read('style.css');
-  for (const sel of ['.claim-count-line', '.wip__release', '.wip__select']) {
+  for (const sel of ['.claim-count-line', '.wip__select']) {
     assert.ok(new RegExp(`^${sel.replace('.', '\\.')} \\{`, 'm').test(style), `style.css carries ${sel}`);
   }
+  // task 1005084: .wip__release is the shared ghost (controls.css); style.css keeps its warning hover
+  assert.match(read('controls.css'), /\.wip__release/, 'the shared sheet draws .wip__release');
+  assert.match(style, /^\.wip__release:hover:not\(:disabled\) \{/m, 'style.css keeps the release\'s warning hover');
 });

package/tests/design_tokens_generated.mjs

@@ -0,0 +1,56 @@
+// tests/design_tokens_generated.mjs — config/design-tokens.neutral.json is GENERATED
+// from the branding pack and the shared token sheet, and regenerates byte-identical
+// (task 1005082, goal 1000138).
+//
+// WHY. The hand-written file's spacing and motion had drifted from every page's
+// (4/8/16px against the 6/10/14px rhythm). The generator removes the hand; this
+// test is what notices a hand coming back, or a token-sheet edit nobody regenerated.
+//
+// Run:  node --test tests/design_tokens_generated.mjs
+
+import { strict as assert } from 'node:assert';
+import { test } from 'node:test';
+import fs from 'node:fs';
+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 gen = require(path.join(ROOT, 'modules', 'ui-design', 'scripts', 'gen-design-tokens.js'));
+
+test('the committed token file is exactly what the generator writes', () => {
+  const committed = fs.readFileSync(path.join(ROOT, 'config', 'design-tokens.neutral.json'), 'utf8');
+  assert.equal(committed, gen.build(),
+    'config/design-tokens.neutral.json is stale or hand-edited: run node modules/ui-design/scripts/gen-design-tokens.js');
+});
+
+test('every scale, radius and motion value is the shared sheet\'s --ds-* value', () => {
+  const vars = gen.readSheetVars(fs.readFileSync(path.join(ROOT, 'modules', 'ui-design', 'public', 'tokens.css'), 'utf8'));
+  const tokens = JSON.parse(gen.build());
+  for (const group of ['scale', 'radius', 'motion']) {
+    for (const [key, prop] of Object.entries(gen.MAP[group])) {
+      assert.equal(tokens[group][key], vars[prop], `${group}.${key} must be ${prop}`);
+    }
+  }
+  assert.equal(tokens.scale.xs, '6px', 'the rhythm starts at the 6px hair step');
+});
+
+test('a token-sheet change reaches the generated file (the generator reads the sheet, not a copy)', () => {
+  const sheet = ':root { --ds-space-hair: 7px; --ds-space-xs: 10px; --ds-space-sm: 14px; --ds-space-md: 22px;'
+    + ' --ds-space-lg: 34px; --ds-space-xl: 52px; --ds-radius-tile: 10px; --ds-radius-panel: 16px;'
+    + ' --ds-radius-pill: 999px; --ds-dur-fast: 150ms; --ds-dur-base: 200ms; --ds-dur-arrive: 700ms;'
+    + ' --ds-ease: cubic-bezier(.2, .7, .3, 1); }';
+  const pack = JSON.parse(fs.readFileSync(path.join(ROOT, 'config', 'branding.neutral.json'), 'utf8'));
+  assert.equal(JSON.parse(gen.build({ sheet, pack })).scale.xs, '7px');
+});
+
+test('a --ds-* step the mapping needs but the sheet lost is an error, not a silent gap', () => {
+  const pack = JSON.parse(fs.readFileSync(path.join(ROOT, 'config', 'branding.neutral.json'), 'utf8'));
+  assert.throws(() => gen.build({ sheet: ':root { --ds-space-hair: 6px; }', pack }), /is not declared/);
+});
+
+test('only the base :root block is read: the dark and reduced-motion restatements do not leak in', () => {
+  const vars = gen.readSheetVars(':root { --ds-dur-fast: 150ms; } @media (prefers-reduced-motion: reduce) { :root { --ds-dur-fast: 0ms; } }');
+  assert.equal(vars['--ds-dur-fast'], '150ms');
+});

package/tests/hall_board_world.mjs

@@ -42,6 +42,7 @@ const HALL = path.join(ROOT, 'modules', 'hall-ui', 'public');
 const read = (f) => fs.readFileSync(path.join(HALL, f), 'utf8');
 
 const styleCss = read('style.css');
+const controlsCss = read('controls.css');
 const workCss = read('work.css');
 const workHtml = read('work.html');
 
@@ -104,7 +105,8 @@ test('a prose link buys BOTH axes — padding for width, negative margin to give
 // ---- 3. the world's own bans ------------------------------------------------
 
 test('nothing on the board paints text white on the accent', () => {
-  const claim = rule(styleCss, '.task__claim');
+  // task 1005084: the claim is the shared primary door, drawn in controls.css
+  const claim = controlsCss.replace(/\/\*[\s\S]*?\*\//g, '').match(/([^{}]*\.task__claim\b[^{}]*)\{([^{}]*background:[^{}]*)\}/)?.[2] ?? null;
   assert.ok(claim, '.task__claim has a rule');
   assert.match(claim, /background:\s*var\(--accent\)/, 'the claim button is the page\'s one accent fill');
   assert.match(claim, /color:\s*var\(--accent-on\)/,
@@ -116,7 +118,9 @@ test('nothing on the board paints text white on the accent', () => {
 });
 
 test('the shared task + bulk-toolbar rules live in style.css (the shared base), not the page-local work.css', () => {
-  for (const sel of ['.task__age', '.task__age--stale', '.task__tag--newcomer', '.task__tag--mine', '.task__tag--rank', '.task__tag--rank-open', '.task__claim', '.task__touches', '.task', '.bulk-toolbar', '.bulk-toolbar__count', '.bulk-toolbar__actions', '.bulk-toolbar__btn', '.bulk-toolbar__clear']) {
+  // .task__claim left for the shared controls.css (task 1005084); work.css still must not restate it
+  assert.equal(rule(workCss, '.task__claim'), null, 'work.css must not restate .task__claim');
+  for (const sel of ['.task__age', '.task__age--stale', '.task__tag--newcomer', '.task__tag--mine', '.task__tag--rank', '.task__tag--rank-open', '.task__touches', '.task', '.bulk-toolbar', '.bulk-toolbar__count', '.bulk-toolbar__actions', '.bulk-toolbar__btn', '.bulk-toolbar__clear']) {
     assert.ok(rule(styleCss, sel), `style.css carries ${sel}, beside .task__tag`);
     assert.equal(rule(workCss, sel), null, `work.css must not restate ${sel}`);
   }

package/tests/hall_builders_page_pass.mjs

@@ -147,10 +147,12 @@ test('A: an older branding pack with no joinability falls back to the apply door
 });
 
 test('A: the quiet fill is a defined variant, not an inline colour', () => {
-  const css = read('style.css');
-  const i = css.indexOf('\n.landing__btn--quiet {');
-  assert.ok(i > 0, 'style.css defines .landing__btn--quiet');
-  const block = css.slice(i, css.indexOf('}', i));
+  // task 1005084: .landing__btn--quiet is the shared ghost in controls.css
+  const css = read('controls.css').replace(/\/\*[\s\S]*?\*\//g, '');
+  const m = css.match(/([^{}]*\.landing__btn--quiet[\s,][^{}]*)\{([^{}]*)\}/);
+  assert.ok(m, 'controls.css defines .landing__btn--quiet');
+  const block = m[2];
+  assert.match(block, /background:\s*transparent/, 'the quiet variant is the ghost');
   assert.deepEqual(block.match(/#[0-9a-f]{3,8}\b/gi) || [], [], 'the variant carries no literal colour');
   // Same geometry as the request button means it inherits .landing__btn's
   // min-height and padding rather than restating them.