@bongos/core 1.21.38 → 1.21.39

package/modules/hall-ui/public/street-inside.js

@@ -34,8 +34,6 @@
 //    the seat is drawn empty and says so, and the rule is the building's own
 //    matters' rule, else the design's default, named as the default
 //    (holderOf()).
-//  * On the table here: the docket's matters at this building. Each row opens
-//    the short card (GovStreet.cardFor(); task 1004528 makes it act).
 //  * The record: the building's history (docket.history: rulings, amendments,
 //    sign-offs, releases, payouts) and its matters, filed by month, one drawer
 //    a month, newest first.
@@ -176,7 +174,7 @@
   //   key  one of the nine civic keys
   //   b    the building, in civicInfo()'s shape: { group, name, holds, seats,
   //        who, rule, how, cap, growth, floors: [{ name, line, count, status,
-  //        lines }], record: [{ label, items }], onFile, table: [rows], annex }
+  //        lines }], record: [{ label, items }], onFile, annex }
   //   st   { floor, drawer, picks, vacant, reviewHeld, evening }
   // Everything the prototype's template binds, as data, plus `selFloor` and
   // `selDrawer`. The geometry is the prototype's to the pixel.
@@ -261,7 +259,6 @@
         : { key: s, init: s, bg: T.seatFor.bg, fg: T.seatFor.fg, border: 'none' };
     const g = City.glyph(b.rule, 9, 9, 16);
     const pct = Math.round(occ / cap * 100);
-    const table = Array.isArray(b.table) ? b.table : [];
     const f = b.floors[selFloor];
     const quietWord = f && QUIET[f.status] !== undefined ? ' · ' + (f.status === 'proposed' ? 'proposed' : QUIET[f.status]) : '';
     return {
@@ -269,7 +266,6 @@
       seats: b.seats.map(seatOf), whoText: b.who, gf: g.fill, gl: g.line, howText: b.how,
       fill: pct, fillColor: pct >= 80 ? '#fb9569' : T.line,
       growth: (vacant ? 'Not built yet. ' : occ + ' of ' + cap + ' floors in use. ') + b.growth,
-      hasTable: table.length > 0, table: table,
       openKicker: 'floor ' + (selFloor + 1) + (f && f.status === 'pending' ? ' · being decided' : quietWord),
       sym: f ? Object.assign(symbolPath(floorSymbol(f.name)), { c: '#fb9569' }) : { d: '', f: '', c: 'none' },
       flipA: ((st.picks || 0) % 2) === 0, flipB: ((st.picks || 0) % 2) === 1,
@@ -528,11 +524,6 @@
       if (!vacant && spec[2] && spec[2].held) fl.name = spec[2].held;
       return fl;
     });
-    const order = { needs: 0, moving: 1, settled: 2 };
-    const table = matters.slice().sort((a, b) => order[stateOf(a)] - order[stateOf(b)]).map((m) => {
-      const s = stateOf(m), when = S.whenWord(m.deadline, c.now);
-      return { key: String(m.id), matter: String(m.id), label: String(m.title || m.short || ''), status: s === 'needs' ? 'needs you' + (when ? ' · ' + when : '') : s === 'moving' ? 'moving' + (when ? ' · ' + when : '') : (when || 'settled'), state: s, color: s === 'needs' ? 'accent' : 'ink' };
-    });
     const history = docket && Array.isArray(docket.history) ? docket.history.filter((x) => x && x.building === key) : [];
     const record = recordOf(matters, c.now, history);
     return {
@@ -541,7 +532,7 @@
       growth: vacant ? plan.vacantGrowth : plan.growth,
       floors: floors, record: record,
       onFile: !docket ? 'the docket could not be read just now' : matters.length ? matters.length + (matters.length === 1 ? ' matter' : ' matters') + ' on the docket' + (history.length ? ' · ' + history.length + ' on the record' : '') : history.length ? history.length + ' on the record' : 'nothing on file yet',
-      table: table, annex: !vacant && floors.length >= plan.cap ? plan.annex : null, vacant: vacant,
+      annex: !vacant && floors.length >= plan.cap ? plan.annex : null, vacant: vacant,
     };
   }
 
@@ -558,9 +549,8 @@
     const docket = o.docket && Array.isArray(o.docket.matters) ? o.docket : null;
     const evening = !!docket && !(docket.counts && Number(docket.counts.needs) > 0);
     const bv = buildingView(T, o.key, b, { floor: o.floor, drawer: o.drawer, picks: o.picks, vacant: b.vacant, reviewHeld: !b.vacant, evening: evening, cleared: o.cleared });
-    bv.table = bv.table.map((r) => Object.assign({}, r, { color: r.color === 'accent' ? T.accentText : T.ink }));
     bv.key = o.key; bv.mode = mode; bv.vacant = b.vacant;
-    bv.label = b.name + ' in section: ' + b.floors.length + ' of ' + bv.cap + ' floors in use' + (bv.hasTable ? ', ' + bv.table.length + ' on the table' : '') + '.';
+    bv.label = b.name + ' in section: ' + b.floors.length + ' of ' + bv.cap + ' floors in use.';
     return bv;
   }
 

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

@@ -195,7 +195,7 @@ html[data-fullscreen="on"] .street-top { padding-right: 184px; }
    on the hall's tokens. Under 900 px the artboard gives way to a column: the
    drawing in its own sideways strip, the panel below it. */
 .inside { position: absolute; inset: 0; display: flex; overflow-x: auto; overflow-y: auto; background: var(--bg); }
-.inside[hidden], .inside-annex[hidden], .inside-dhead[hidden], .inside-legend[hidden], .inside-table[hidden], .inside-open[hidden], .inside-note[hidden], .inside-panel[hidden], .inside-more[hidden] { display: none; }
+.inside[hidden], .inside-annex[hidden], .inside-dhead[hidden], .inside-legend[hidden], .inside-open[hidden], .inside-note[hidden], .inside-panel[hidden], .inside-more[hidden] { display: none; }
 .inside-sizer { position: relative; margin: auto; flex: none; overflow: hidden; }
 .inside-board { position: absolute; left: 0; top: 0; width: 1440px; height: 708px; transform-origin: 0 0; transform: scale(var(--inside-scale, 1)); color: var(--ink); }
 .inside-head { position: absolute; left: 32px; top: 16px; display: flex; align-items: center; gap: 16px; z-index: 2; }
@@ -265,10 +265,6 @@ html[data-fullscreen="on"] .street-top { padding-right: 184px; }
 .inside-meter__fill { height: 6px; width: 0; background: color-mix(in srgb, var(--ink) 30%, var(--bg)); }
 .inside-meter__fill--full { background: var(--accent); }
 .inside-meter__line { font-size: 13px; color: var(--ink-soft); line-height: 1.45; }
-.inside-table { display: flex; flex-direction: column; gap: 8px; }
-.inside-row { display: flex; justify-content: space-between; align-items: center; gap: 12px; width: 100%; background: none; border: 0; border-top: 1px solid var(--street-rule); padding: 10px 0; cursor: pointer; font-family: var(--font-body); font-size: 15px; text-align: left; color: var(--ink); }
-.inside-row--needs { color: var(--street-accent-text); }
-.inside-row__status { font-size: 12.5px; color: var(--ink-faint); white-space: nowrap; }
 
 /* the chosen floor, its sign large; the block re-enters on every pick */
 .inside-open { display: flex; flex-direction: column; gap: 10px; }
@@ -362,7 +358,7 @@ html[data-fullscreen="on"] .street-top { padding-right: 184px; }
 .inside-ditem { display: flex; justify-content: space-between; gap: 14px; padding: 8px 0; border-top: 1px solid var(--street-rule); font-size: 14px; line-height: 1.4; }
 .inside-ditem__meta { color: var(--ink-faint); white-space: nowrap; }
 
-.inside-back:focus-visible, .inside-textbtn:focus-visible, .inside-row:focus-visible, .inside-drawer:focus-visible, .inside-item:focus-visible, .inside-more:focus-visible { outline: 2px solid var(--accent); outline-offset: 3px; }
+.inside-back:focus-visible, .inside-textbtn:focus-visible, .inside-drawer:focus-visible, .inside-item:focus-visible, .inside-more:focus-visible { outline: 2px solid var(--accent); outline-offset: 3px; }
 .inside-floor:focus-visible { outline: 2px solid var(--accent); outline-offset: -3px; }
 .inside-name:focus { outline: none; }
 .inside-name:focus-visible { outline: 2px solid var(--accent); outline-offset: 4px; }

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

@@ -166,10 +166,6 @@
           <span class="inside-meter__line" id="insideGrowth"></span>
         </div>
       </div>
-      <div class="inside-table" id="insideTableWrap" hidden>
-        <span class="street-label">on the table here</span>
-        <div id="insideTable"></div>
-      </div>
       <div class="inside-open" id="insideOpenA"></div>
       <div class="inside-open" id="insideOpenB" hidden></div>
       <div class="inside-record">

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

@@ -493,19 +493,6 @@
     var fill = $('insideFill');
     if (fill) { fill.style.width = bv.fill + '%'; fill.className = 'inside-meter__fill' + (bv.fill >= 80 ? ' inside-meter__fill--full' : ''); }
     setText('insideGrowth', bv.growth);
-    show('insideTableWrap', bv.hasTable);
-    var table = $('insideTable'); clear(table);
-    bv.table.forEach(function (r) {
-      var b = node('button', 'inside-row' + (r.state === 'needs' ? ' inside-row--needs' : ''));
-      b.setAttribute('type', 'button');
-      b.setAttribute('data-matter', r.matter);
-      b.setAttribute('aria-controls', 'streetCard');
-      b.setAttribute('aria-expanded', view.open === r.matter ? 'true' : 'false');
-      b.appendChild(node('span', 'inside-row__label', r.label));
-      b.appendChild(node('span', 'inside-row__status', r.status));
-      b.addEventListener('click', function () { openCard(r.matter, b); });
-      if (table) table.appendChild(b);
-    });
     // the chosen floor: two blocks that take turns, so the block re-enters on every pick
     var on = $(bv.flipA ? 'insideOpenA' : 'insideOpenB'), off = $(bv.flipA ? 'insideOpenB' : 'insideOpenA');
     if (off) { off.hidden = true; clear(off); }
@@ -1104,12 +1091,10 @@
     // The pins were redrawn, so focus goes back to the new button for the same matter.
     if (restore !== false && was && was.getAttribute) {
       var key = was.getAttribute('data-matter');
-      // the pins and the table rows were redrawn: focus the new control for the same matter
+      // the pins were redrawn: focus the new pin for the same matter
       // .children is an HTMLCollection, which concat does not spread: copy it first
       var list = function (id) { var el = $(id); return el && el.children ? Array.prototype.slice.call(el.children) : []; };
-      var pins = list('streetPins');
-      var rows = list('insideTable');
-      var pool = view.inside ? [].concat(rows, pins) : [].concat(pins, rows);
+      var pool = list('streetPins');
       for (var i = 0; i < pool.length; i++) {
         if (pool[i].getAttribute('data-matter') === key) { if (pool[i].focus) pool[i].focus(); break; }
       }

package/modules/hall-ui/records/governor-street.md

@@ -21,12 +21,12 @@ Built 1:1 on `buildingView()` and `districtView()` of the same prototype. The in
 Files: `street-inside.js` (`window.GovInside`, node-require-safe, pure): the prototype's `crown()`, `workItem()`, `floorSymbol()`, `symbolPath()` (all forty symbols) and the section drawing itself, verbatim, plus the real-data adapters. `street.js` builds the nodes; `street.css` carries the interior's styles under the `inside-` prefix.
 
 - **The building in section.** Every storey a real floor (slab, cut walls with glazing, three lamps, a partition with a doorway, the framed sign with the floor's symbol, its name and state lettered on the wall, its count, its level number), dark empty floors up to the building's capacity, the crown per building, the lift in its shaft, the orange frame and orange sign on the chosen floor, and the annex lot once the building is full. An unheld Review is the dashed ghost (`GovStreet.reviewHeldOf()`, so the street and the interior agree).
-- **The right panel.** Name and what it holds, who holds it (seat chips), how it decides (the rule glyph), **change how this works**, the fill meter with its growth line, on the table here, the chosen floor with its sign large, and the record as monthly filing drawers.
+- **The right panel.** Name and what it holds, who holds it (seat chips), how it decides (the rule glyph), **change how this works**, the fill meter with its growth line, the chosen floor with its sign large, and the record as monthly filing drawers.
 - **Floors are the prototype's structure; a number appears only when the docket holds it.** `GovInside.PLAN` keeps each building's floors, order and capacity. A floor bound to a docket source shows that source's open matters at this building (count, line, needs you, the lines); one bound to a reading shows its figure; Planning's Goals floor counts the open goals. Every other floor says "not recorded yet" on the wall, lettered faint, with no number. A source that is off, coming, not yours or unreadable says which, never a figure. Filling every floor with real data is task 1004527, through `floorFor()` and `PLAN`.
 - **Who holds it.** The docket names no holders yet (task 1004529), so the seat is an empty dashed chip and says "who holds it is not set on this project yet". The rule is the one the building's open matters are decided by, else the design's default, which the words call "by default". `holderOf()` reads a docket building's `holders` / `rule` / `who` first, so 1004529 lands without touching the drawing.
 - **The lift and the flip.** The lift is one node kept across draws, so picking a floor slides it by a CSS transform transition; the chosen floor's block is two blocks that take turns (`flipA` / `flipB`), so it re-enters on every pick. Both are dead under reduced motion.
 - **Change how this works** is present and honest: pressed, it says drafting an amendment from here is not built yet (task 1004530 wires it).
-- **On the table here** rows open the same short card a pin opens, through `openCard()`; Escape gives focus back to the row.
+- **No "on the table here" list.** The prototype's list of the building's open matters was removed on the owner's direction (task 1004565): the floor desks carry every matter as an actionable row, so the list was duplication. Inside a building a matter is its desk row; the short card opens from the street's pins.
 - **Districts.** The goal's tasks (`GET /tasks?goal_id=N`, read once per goal): shape from the discipline (tower engineer, studio artist, glass front UI, pavilion ideator; none is a tower), stage from the status (backlog and blocked a staked plot, ready a slab, active a scaffold, completed and confirmed built with a flag, shipped lit), label the kind; height from the estimate. Up to ten stand on the road, work in flight first; the rest are counted with a link to the goal. Each one opens its task.
 - **Narrow.** Under 900 px the board becomes a column: the drawing in its own sideways strip (never scaled under 0.6, so a floor stays a 24 px target), then the panel.
 

package/package-lock.json

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

package/package.json

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

@@ -8992,5 +8992,11 @@
       "id": "1004564",
       "text": "If you put something up for a vote and nobody has voted yet, you can now withdraw it with one tap on your desk. Once anyone votes, it stays with the vote. A small database change makes \"withdrawn\" a valid way for a sitting to"
     }
+  ],
+  "1.21.39": [
+    {
+      "id": "1004565",
+      "text": "Inside each building on the governor street, the separate \"on the table here\" list is gone. The floor desks already show every matter as a row you can act on, so the list only repeated them. The rest of the building panel look"
+    }
   ]
 }

package/src/module-api.js

@@ -75,7 +75,7 @@ const { responsibilityFor, ROLE_RESPONSIBILITIES } = require('./role-responsibil
 // MAJOR (see allowBoxScope below): passes the request through untouched.
 function deprecatedNoopMiddleware(_req, _res, next) { next(); }
 
-const CORE_VERSION = '1.21.38'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.21.39'; // 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/hall_street_inside.mjs

@@ -218,7 +218,7 @@ test('a source that is off, coming, not yours or unreadable never shows a number
   const none = I.civicInfo('release', { docket: null, now: NOW });
   assert.ok(none.floors.every((f) => f.count === '' && f.status !== 'ok' && f.status !== 'pending'), 'no docket: no numbers anywhere');
   assert.equal(none.onFile, 'the docket could not be read just now');
-  assert.deepEqual(none.table, []);
+  assert.deepEqual(none.record, [], 'no docket: nothing filed');
 });
 
 test('a floor bound to a reading shows its figure and line when live; Planning counts the open goals', () => {
@@ -237,12 +237,12 @@ test('a floor bound to a reading shows its figure and line when live; Planning c
   assert.ok(m.texts.some((t) => t.v === '5' && t.s === 20), 'only open goals are counted on the wall');
 });
 
-test('on the table: the building\'s matters, needs first; each row names the matter it opens', () => {
-  const ms = [matter({ id: 'board:1', source: 'board', building: 'charter', state: 'moving', title: 'B', deadline: '2026-10-04T09:00:00.000Z' }), matter({ id: 'board:2', source: 'board', building: 'charter', state: 'needs', title: 'A', deadline: '2026-10-02T20:00:00.000Z' }), matter({ id: 'blockers:3' })];
+test('no separate list of matters: the floor desks carry every matter, so the model keeps no table (task 1004565)', () => {
+  const ms = [matter({ id: 'board:1', source: 'board', building: 'charter', state: 'moving', title: 'B' }), matter({ id: 'board:2', source: 'board', building: 'charter', state: 'needs', title: 'A' })];
   const m = I.buildingModel({ key: 'charter', docket: docket(ms), mode: 'light', now: NOW });
-  assert.deepEqual(m.table.map((r) => [r.matter, r.label, r.status, r.state]), [['board:2', 'A', 'needs you · today', 'needs'], ['board:1', 'B', 'moving · sun', 'moving']]);
-  assert.equal(m.table[0].color, S.theme('light').accentText);
-  assert.ok(m.hasTable);
+  assert.ok(!('table' in m) && !('hasTable' in m), 'no table fields on the model');
+  assert.doesNotMatch(m.label, /on the table/, 'the building\'s label no longer counts a table');
+  assert.match(m.label, /floors in use\.$/);
 });
 
 test('the record files the building\'s matters by the month each was opened, newest drawer first', () => {
@@ -524,7 +524,8 @@ const FEEDS = (ms, extra = {}) => ({
 
 test('the markup carries the interior: one persistent lift, two flip blocks, the panel and the back link', () => {
   const html = read('street.html').replace(/<!--[\s\S]*?-->/g, '');
-  for (const id of ['streetInside', 'insideBack', 'insideSvg', 'insideFloors', 'insideLift', 'insideAnnex', 'insidePanel', 'insideName', 'insideAmend', 'insideAmendNote', 'insideFill', 'insideTable', 'insideOpenA', 'insideOpenB', 'insideDrawers', 'insideDrawerItems', 'insideDName', 'insideLegend', 'insideItems']) assert.match(html, new RegExp(`id="${id}"`), id);
+  for (const id of ['streetInside', 'insideBack', 'insideSvg', 'insideFloors', 'insideLift', 'insideAnnex', 'insidePanel', 'insideName', 'insideAmend', 'insideAmendNote', 'insideFill', 'insideOpenA', 'insideOpenB', 'insideDrawers', 'insideDrawerItems', 'insideDName', 'insideLegend', 'insideItems']) assert.match(html, new RegExp(`id="${id}"`), id);
+  assert.doesNotMatch(html, /insideTable|on the table here/i, 'the "on the table here" list is gone (task 1004565)');
   assert.equal((html.match(/<h1\b/g) || []).length, 1, 'the street keeps its one h1; the interior names are h2');
   assert.match(html, /<section class="inside" id="streetInside"[^>]*hidden>/, 'hidden until an address names a building or a district');
   assert.match(html, /<button type="button" class="inside-textbtn" id="insideAmend"/);
@@ -539,8 +540,7 @@ test('a building address draws the interior in place of the street, from the doc
   assert.equal(p.byId.get('insideFloors').children.length, 7, 'a button per floor in use');
   assert.equal(p.byId.get('insideFloors').children[4].attrs['aria-pressed'], 'true', 'the floor being decided is chosen');
   assert.equal(p.byId.get('insideLift').style.transform, 'translateY(-208px)');
-  assert.equal(p.byId.get('insideTable').children.length, 1);
-  assert.equal(p.byId.get('insideTable').children[0].className, 'inside-row inside-row--needs');
+  assert.equal(p.byId.get('insideTable'), undefined, 'no separate list of matters');
   assert.equal(p.byId.get('insideSeats').children[0].className, 'inside-seat inside-seat--open');
   assert.equal(p.focused.el, null, 'a page opened at the address keeps the browser\'s own start');
   p.location.hash = '#building=rules';
@@ -575,15 +575,17 @@ test('picking a floor moves the SAME lift node and re-enters the chosen block; d
   assert.match(p.byId.get('insideAmendNote').textContent, /could not be read/);
 });
 
-test('a table row opens the short card; Escape gives focus back to the row; back to the street returns to the door', async () => {
+test('a matter is a desk row: its name opens it in place; back to the street returns to the door', async () => {
   const ms = [matter({ id: 'board:1', source: 'board', building: 'charter', state: 'needs', title: '<b>x</b>' })];
   const p = await bootPage({ hash: '#building=charter', feeds: FEEDS(ms) });
-  p.byId.get('insideTable').children[0].fire('click');
-  assert.equal(p.byId.get('streetCard').hidden, false);
-  assert.equal(p.byId.get('streetCardTitle').textContent, '<b>x</b>', 'docket text lands as text');
-  p.fireWin('keydown', { key: 'Escape', preventDefault() {} });
-  assert.equal(p.byId.get('streetCard').hidden, true);
-  assert.equal(p.focused.el.attrs['data-matter'], 'board:1', 'focus goes back to the row');
+  const desk = () => (p.byId.get('insideOpenA').hidden ? p.byId.get('insideOpenB') : p.byId.get('insideOpenA'));
+  const find = (el, pred) => { if (pred(el)) return el; for (const c of el.children) { const hit = find(c, pred); if (hit) return hit; } return null; };
+  const nameOf = () => find(desk(), (el) => el.attrs && el.attrs['data-matter'] === 'board:1');
+  assert.ok(nameOf(), 'the desk carries the matter as a row');
+  assert.equal(nameOf().attrs['aria-expanded'], 'false');
+  nameOf().fire('click');
+  assert.equal(nameOf().attrs['aria-expanded'], 'true', 'the row opens in place');
+  assert.equal(p.byId.get('streetCard').hidden, true, 'no card is needed inside: the desk acts');
   p.byId.get('insideBack').fire('click');
   assert.deepEqual(p.history, ['/builders/street']);
   assert.equal(p.byId.get('streetInside').hidden, true);