@bongos/core 1.24.37 → 1.24.38

package/tests/hall_nav.mjs

@@ -243,7 +243,10 @@ test('sidebar groups read by audience, in order', () => {
   // 'Archon' → 'Governance' in ADR 0157 (the group's pages opened to Metic, so a header
   // naming the top rank was wrong), then → 'Government' by ADR 0174: a header
   // reading "Archon" over items a Metic can open would misdescribe the ladder.
-  assert.deepEqual(nav.map((g) => g.label), ['', 'You', 'Community', 'Docs', 'Government']);
+  // task 1005071 (owner decision 2026-11-04, from the sitemap audit): every group is
+  // labelled, and the old unlabeled project group split into Work, Make and Govern.
+  // Docs became Help; Government became Run the project.
+  assert.deepEqual(nav.map((g) => g.label), ['Work', 'Make', 'Community', 'You', 'Govern', 'Run the project', 'Help']);
   // task 1003113 (R02 of goal 1000074): 'copy-desk' joins the unlabeled group,
   // after Tasks. It belongs with the project's own work and not in Docs — it is a
   // QUEUE somebody acts on, not reference material — and not in You or Community,
@@ -285,12 +288,15 @@ test('sidebar groups read by audience, in order', () => {
   // task 1004939 (goal 1000137): 'market-watch' joins last — the project's scouts on its
   // market, so its subject is the project. Module-gated: it is dropped wherever the
   // default-off market-watch module is not enabled (this boot reports no module state).
-  assert.deepEqual(nav[0].items.map((i) => i.id), ['home', 'roadmap', 'goals', 'goal-map', 'work', 'blockers', 'board-room', 'city', 'thinking', 'studio', 'market-watch'],
-    'the unlabeled group is the project itself (task 1002425 order)');
+  // task 1005071: the project group split three ways. Work is the project's state;
+  // Make is where things get made; Govern is the board's rooms.
+  assert.deepEqual(nav[0].items.map((i) => i.id), ['home', 'work', 'goals', 'goal-map', 'roadmap', 'blockers', 'thinking'],
+    'Work = the project itself (task 1005071 order)');
+  assert.deepEqual(nav[1].items.map((i) => i.id), ['studio', 'market-watch'], 'Make = where things get made');
   // task 1004685 (the living-city prototype, round-3 board 6): 'my-agents' joins You —
   // how your own Claude works for you, the personal half of "the city steers
   // everyone, My agents steers yours". Its subject is the viewer's own agents.
-  assert.deepEqual(nav[1].items.map((i) => i.id), ['standing', 'profile', 'my-agents'], 'You = your own numbers and your own agents');
+  assert.deepEqual(nav[3].items.map((i) => i.id), ['standing', 'profile', 'my-agents'], 'You = your own numbers and your own agents');
   // task 1003444: 'people' LEFT the group — the owner deleted the hall's
   // cross-project directory (ADR 0337 D5); its home is the hub, and /people
   // redirects to /roster. Builders is the hall's directory now.
@@ -305,19 +311,20 @@ test('sidebar groups read by audience, in order', () => {
   // 'leaderboard') and is now a real page at /roster labelled Builders. That is
   // what made this group read as one real page plus a link into the home page.
   assert.deepEqual(nav[2].items.map((i) => i.id), ['collab', 'roster'], 'Community = everyone else, in this project');
+  assert.deepEqual(nav[4].items.map((i) => i.id), ['city', 'board-room'], 'Govern = the board\'s rooms, the city first');
   assert.ok(!nav.flatMap((g) => g.items).some((i) => /^\/people\b/.test(i.href)), 'no nav item points at the deleted /people');
   // task 1004473: 'commands' — every skill and CLI verb, read live — is reference
   // material that reads the same for every builder, so it joins Docs.
   // task 1005046 (owner decision 2026-11-03, idea 1001311): ONE reading room. Ranks,
   // Economy and Diagrams explained what the Primer explains; they are its chapters
   // now, so Docs is the Primer and the four references that are not a document.
-  assert.deepEqual(nav[3].items.map((i) => i.id),
+  assert.deepEqual(nav[6].items.map((i) => i.id),
     ['primer', 'commands', 'atlas', 'modules', 'apidocs'], 'Docs = reference material');
   // 'government' (the Permissions tab, BV1.R113) is the group's one archon-gated item.
   // 'project-settings' joins the Government group at the metic floor (task 1003280,
   // goal 1000072) — it sits before 'government', which is the one archon-only item.
   // 'board-room' LED this group from the ADR 0266 carve until task 1003744 moved
-  // it to the project group — see the note on nav[0]. What is left is rank surface
+  // it to the project group — see the note on Govern (nav[4]). What is left is rank surface
   // throughout, which is what lets the group keep its metic head honestly.
   // 'deploy' (task 1003159, ADR 0293) is the SECOND archon item, beside 'government':
   // its gate is the `core.pin.move` atom, archon-floor and system:true. It sits with
@@ -328,7 +335,7 @@ test('sidebar groups read by audience, in order', () => {
   // 'market-watch-settings' (task 1004938) LEFT the group in task 1005045: the owner saw
   // "Market Watch" twice and had to guess which was which (owner decision 2026-11-03,
   // idea 1001310). The settings are an owner-only Settings tab on /market-watch now.
-  assert.deepEqual(nav[4].items.map((i) => i.id), ['watch', 'gate', 'sessions', 'project-settings', 'money', 'deploy', 'government']);
+  assert.deepEqual(nav[5].items.map((i) => i.id), ['watch', 'gate', 'sessions', 'project-settings', 'money', 'deploy', 'government']);
   // task 1005045: and no label is left twice anywhere in the sidebar, so the next pair
   // fails here instead of in front of the owner.
   const labels = [...boot().sidebarHtml.matchAll(/<a class="nav-item" [^>]*>[\s\S]*?<span class="nav-item__label">([^<]*)<\/span>/g)].map((m) => m[1]);
@@ -355,7 +362,7 @@ test('Ranks, Economy and Diagrams are Primer chapters, not pages', () => {
   for (const f of ['ranks.html', 'drachmae.html', 'diagrams.html']) {
     assert.ok(!fs.existsSync(path.join(HALL, f)), `${f} left the hall`);
   }
-  const primer = nav.find((g) => g.label === 'Docs').items.find((i) => i.id === 'primer');
+  const primer = nav.find((g) => g.label === 'Help').items.find((i) => i.id === 'primer');
   assert.equal(primer.href, '/primer', 'the Primer is the reading room the three became');
 });
 
@@ -564,11 +571,11 @@ test('collapsible groups carry a stable storage key (collapse persistence)', ()
   // hall-nav-collapsed storage key, so any builder who had collapsed the old "archon"
   // group gets the new one expanded once — cosmetic, and preferable to keeping a stale
   // slug that no longer matches the visible label.
-  for (const g of ['you', 'community', 'docs', 'government']) {
+  // task 1005071: a label with spaces hyphenates, so the key stays a valid id.
+  for (const g of ['work', 'make', 'community', 'you', 'govern', 'run-the-project', 'help']) {
     assert.match(html, new RegExp(`data-group="${g}"`), `${g} is collapsible (hall-nav-collapsed key)`);
   }
-  assert.ok(!/nav-group--collapsible[^>]*>(?!<button)/.test(html.split('<div class="nav-group')[1] || ''),
-    'the unlabeled project group stays always-expanded');
+  assert.ok(!/aria-controls="[^"]* /.test(html), 'no aria-controls id carries a space');
 });
 
 // ── a nav item can carry a COUNT badge (task 1003737) ───────────────────────
@@ -780,7 +787,7 @@ test('no glyph path data reads as an IP address to the release leak scan', () =>
 // block (task 1004713). These drive the real shell and the real gov-positions.js.
 
 const tick = () => new Promise((resolve) => setImmediate(resolve));
-const GOVERNING = ['board-room', 'city', 'watch', 'gate', 'sessions', 'project-settings', 'money', 'deploy', 'government'];
+const GOVERNING = ['city', 'board-room', 'watch', 'gate', 'sessions', 'project-settings', 'money', 'deploy', 'government'];
 const lead = (building, title) => ({ kind: 'lead', seat: 'lead:' + building, building, title });
 const block = (held) => ({ held, title: held.length ? held[0].title : null, has_position: held.length > 0 });
 
@@ -799,7 +806,7 @@ test('POSITION: exactly the governing surfaces carry the flag — the City, the
   const html = boot().sidebarHtml;
   const flagged = [...html.matchAll(/id="nav-([^"]+)"[^>]*data-position="1"/g)].map((m) => m[1]);
   assert.deepEqual(flagged, GOVERNING);
-  assert.match(html, /class="nav-group__head" data-gate="government" data-position="1" data-group="government"/,
+  assert.match(html, /class="nav-group__head" data-gate="government" data-position="1" data-group="run-the-project"/,
     'the Government head carries it too, so no label is left over an emptied group');
   const nav = parseNav(html);
   assert.ok(!nav.find((g) => g.label === 'You').items.some((i) => flagged.includes(i.id)), 'your own pages never need a seat');

package/tests/nav_permission_atoms.mjs

@@ -96,7 +96,7 @@ test('the hall-preview fixture carries every atom /me resolves', () => {
 // Market Watch settings nav item and its fuel glyph; task 1005041: the Update pill replaced the
 // core-update banner; task 1005045: the Market Watch settings item left (one sidebar item);
 // task 1005046: Ranks, Economy and Diagrams left Docs (one reading room, the Primer).
-const SHELL_SHA = '662e036a340ad33f';
+const SHELL_SHA = 'baf8a15f025eba68';
 
 test('shell.js has not changed without its cache-bust being bumped', () => {
   const actual = crypto.createHash('sha256').update(SHELL).digest('hex').slice(0, 16);