@bongos/core 1.24.47 → 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/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@bongos/core",
-  "version": "1.24.47",
+  "version": "1.24.48",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bongos/core",
-      "version": "1.24.47",
+      "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.47",
+  "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

@@ -11253,5 +11253,11 @@
       "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.47'; // 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/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.