@bongos/core 1.19.929 → 1.19.930

package/.bongos-core.json

@@ -2,11 +2,11 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/gds/package-core.js",
-  "core_version": "1.19.929",
-  "core_contract": "1.19.929",
-  "source_commit": "300ec325e8b1e8735b28ce4907a49713af43b7bb",
+  "core_version": "1.19.930",
+  "core_contract": "1.19.930",
+  "source_commit": "fb20df23e5e36d2bfd74b336f3ef6e3f5057ba3f",
   "source_ref": "HEAD",
-  "built_at": "2026-09-21T00:33:55.485Z",
+  "built_at": "2026-09-21T00:58:50.507Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 511,
@@ -17,7 +17,7 @@
     "gate": "passed"
   },
   "file_count": 2857,
-  "tree_sha256": "b7d75fba10c492d6a30b192c1370f1227048a7b0edb5d208cc0512c600edc816",
+  "tree_sha256": "01ea0d274ae8ff7138d09158c7abdcb7129fbad964171405bc9f52ef5ad6879b",
   "files": [
     {
       "path": ".claude/skills/ask-for-help/SKILL.md",
@@ -2677,7 +2677,7 @@
     {
       "path": "docs/module-api-changelog.md",
       "mode": "0000644",
-      "sha256": "a61f890e064e49e509f09f4fa8c9cedfc7530c72f801fbcac09d68171425bd2b"
+      "sha256": "2a4873f628ec91df4ed57e32c38354f55cd72386ea5befab9261a6c315c82fd9"
     },
     {
       "path": "docs/modules-contract.md",
@@ -5297,7 +5297,7 @@
     {
       "path": "modules/hall-ui/public/style.css",
       "mode": "0000644",
-      "sha256": "a7824a889c87fe40725350fd9543929e69e1370fb17023af0447f8254887d72e"
+      "sha256": "8c653bd371b5263c0c1c731a81dd07775e122196bea8b49722b71537bdc60f19"
     },
     {
       "path": "modules/hall-ui/public/task-detail.js",
@@ -5332,7 +5332,7 @@
     {
       "path": "modules/hall-ui/public/thinking.css",
       "mode": "0000644",
-      "sha256": "941cbce0f3cc3984b8477b70834e7f822f40bd37a1cdd3fd4564e99edcde4c6a"
+      "sha256": "b3ee054dbb547d72103e7b2c26f945429c8691213ef3a2b2607677019d99dc0c"
     },
     {
       "path": "modules/hall-ui/public/thinking.html",
@@ -7947,12 +7947,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "d90ee9a8611192f6d7f3a6ab95ba43160ab06b6ea9ef3992e8fb37bf025518a1"
+      "sha256": "d6258275d7d14df92610d44fdf2bd09f96538ca3e5046473654fdf935dd7de31"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "dd796959bb2a29b312aeb003b80daa596bb2ed4d41d9ab45b488f65b533d536f"
+      "sha256": "4d1a541b89dd3a34524fa90b236adb7f1cc4d83c060742399ef018016f22048f"
     },
     {
       "path": "public-docs/index.html",
@@ -9952,7 +9952,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "08276641afda214baeb8e6bc410c6d622a218ae3ed759f2a769f762ad39b50a1"
+      "sha256": "990ccb4288082fdebd7ee0810fc9c04d09d8bd00b78cfd01dc06dae61e626770"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -11772,7 +11772,7 @@
     {
       "path": "tests/hall_board_world.mjs",
       "mode": "0000644",
-      "sha256": "4e1ace5465f811cd22e1080dfd607271e51807bcdc1f53198a9898fc63af7760"
+      "sha256": "264373b87ff2d022b54479ca06a4b01f7a052a8670237b79cbee5c6c907bb69e"
     },
     {
       "path": "tests/hall_builders_page_pass.mjs",

package/docs/module-api-changelog.md

@@ -2332,5 +2332,7 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.19.927 with no explicit bump. run 35547446984. (task 1002620)
   1.19.929 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         landed since 1.19.928 with no explicit bump. run 35548065983. (task 1002620)
+  1.19.930 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.19.929 with no explicit bump. run 35549362371. (task 1002620)
 ---------------------------------------------------------------------------
 ```

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

@@ -3163,10 +3163,20 @@ main [id] { scroll-margin-top: calc(var(--topbar-h) + 16px); }
 /* tab strip — the .setup__tab idiom in the .setup__toggle frame, promoted */
 /* A tab strip is hairlines, not a segmented box: one rule under the row and a
    2px accent underline on the open tab, the lit-step grammar again. */
+/* Both axes are named on purpose, and the pair is load-bearing — do not
+   "simplify" this back to the single `overflow-x`. Per the overflow spec a
+   non-`visible` value on one axis computes the OTHER axis to `auto` too, so
+   `overflow-x: auto` alone made the strip a scroll container vertically as
+   well; `.setup__tab`'s `margin-bottom: -1px` (the underline sitting on the
+   strip's hairline, below) then overflowed it by exactly 1px and every hall
+   page with tabs grew a vertical scrollbar it could never scroll. `hidden` is
+   the fix and `visible` is not: `visible` beside an `auto` is the value the
+   spec rewrites, which would put the bug straight back (task 1004130). */
 .kit-tabs {
   display: flex;
   max-width: 100%;
   overflow-x: auto;
+  overflow-y: hidden;
   border: 0;
   border-bottom: 1px solid var(--rule);
   border-radius: 0;

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

@@ -61,15 +61,6 @@
    the kit's own margin assumes a panel directly beneath and here a table
    follows. */
 .tr-filters { margin: 0 0 var(--sp-sm); }
-/* `.kit-tabs` sets `overflow-x: auto`, which per spec makes the OTHER axis
-   `auto` too — and the strip's own `.setup__tab { margin-bottom: -1px }` leaves
-   it exactly 1px taller than its box, so a vertical scrollbar appears beside a
-   horizontal strip that cannot scroll vertically. It is the shared component's
-   defect rather than this page's, and it is not fixed in style.css from here:
-   that rule is on every page carrying a tab strip, and quietly changing the
-   hall's chrome inside a page's task is how a sheet stops being reviewable.
-   Scoped to this page, with the shared fix filed separately. */
-.tr-filters .kit-tabs { overflow-y: hidden; }
 
 /* ---- the rows ------------------------------------------------------------ */
 /* The four short columns are pinned and the two prose ones share what is left,

package/package-lock.json

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

package/package.json

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

@@ -71,7 +71,7 @@ const { responsibilityFor, ROLE_RESPONSIBILITIES } = require('./role-responsibil
 // there. scripts/gds/bump-version.js still rewrites the literal below; it appends
 // the entry to that file. Look for a version's history there, not here.
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.929'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.19.930'; // 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('dev-box'); log.info('mounted');

package/tests/hall_board_world.mjs

@@ -470,6 +470,38 @@ test('the tab strip is left alone — it scrolls in its own row and never moved
   const strip = /\.setup__tabs\s*\{([^}]*)\}/.exec(styleCss) || /\.kit-tabs\s*\{([^}]*)\}/.exec(styleCss);
   assert.ok(strip, 'the tab strip rule is present');
   assert.match(strip[1], /overflow-x:\s*auto/, 'it scrolls itself, which is why it is not the overflow cause');
+
+  // …and the OTHER axis is named too (task 1004130). Per the overflow spec a
+  // non-`visible` value on one axis computes the other to `auto`, so the line
+  // above alone made the strip a vertical scroll container as well — and
+  // `.setup__tab`'s `margin-bottom: -1px` overflowed it by exactly 1px, so
+  // every hall page with tabs grew a vertical scrollbar that could not scroll.
+  // The regression is invisible on an overlay-scrollbar platform, which is why
+  // it is pinned here rather than left to be spotted in a screenshot.
+  assert.match(strip[1], /overflow-y:\s*hidden/,
+    'the strip pins overflow-y so the spec cannot promote it to a second scroller');
+
+  // `visible` would NOT do: beside an `auto` it is the value the spec rewrites,
+  // which is the bug itself. Only a value that suppresses the axis is legal.
+  assert.doesNotMatch(strip[1], /overflow-y:\s*(visible|auto|scroll)\b/,
+    'overflow-y: visible/auto/scroll would put the stray scrollbar straight back');
+
+  // The 1px that overflows is the underline sitting ON the strip's hairline —
+  // the intended look, and the reason the axis is suppressed instead of the
+  // margin being removed.
+  const tab = rule(styleCss, '.setup__tab');
+  assert.ok(tab, '.setup__tab rule present in style.css');
+  assert.match(tab, /margin-bottom:\s*-1px/,
+    'the tab underline still overlaps the strip hairline — the overflow-y pin is what makes that free');
+});
+
+// The page-local workaround the room carried while the shared rule was broken
+// is gone with it (task 1004130, done-when 3). A stale duplicate would be a
+// comment in the tree asserting a defect that no longer exists.
+test('the room no longer carries its own copy of the tab-strip fix', () => {
+  const thinkingCss = read('thinking.css');
+  assert.doesNotMatch(thinkingCss, /\.kit-tabs\s*\{[^}]*overflow-y/,
+    'thinking.css must not restate overflow-y on .kit-tabs — style.css owns it now');
 });
 
 // ---- the board's filters drive BOTH queue tabs (task 1003828) ---------------