@bongos/core 1.23.20 → 1.23.21

package/.bongos-core.json

@@ -2,22 +2,22 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/bongos/package-core.js",
-  "core_version": "1.23.20",
-  "core_contract": "1.23.20",
-  "source_commit": "217e868b0755b50e4f51bbc8146c3eb0b29fa052",
+  "core_version": "1.23.21",
+  "core_contract": "1.23.21",
+  "source_commit": "5624212239928e3e082efd5b058186e35a6cd51d",
   "source_ref": "HEAD",
-  "built_at": "2026-10-28T14:49:17.728Z",
+  "built_at": "2026-10-29T02:04:20.723Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 616,
     "agent_docs_stubbed": 29,
-    "functional_verbatim": 3604,
+    "functional_verbatim": 3606,
     "rules": 3,
     "gate_literals": 3,
     "gate": "passed"
   },
-  "file_count": 4250,
-  "tree_sha256": "5fc0e2f0f6b6c70f9e38aa2fefd3ab18e77d2df3f77bd2b0ad9ffa120676bc81",
+  "file_count": 4252,
+  "tree_sha256": "74b0778836bfca86383004de0425bcf0d9cf92a310070c7d2d704786a2110c70",
   "files": [
     {
       "path": ".claude/skills/ask-for-help/SKILL.md",
@@ -2937,7 +2937,7 @@
     {
       "path": "docs/module-api-changelog.md",
       "mode": "0000644",
-      "sha256": "04293fa9f41ebf2defd92af5512adb9a4d747ab6e5a9af384f26cc2737db695d"
+      "sha256": "7cca771f6d0fc816d449fa7478f5b01c9d5f25ec21bef1db28338c7f2f4e38e0"
     },
     {
       "path": "docs/modules-contract.md",
@@ -4489,6 +4489,11 @@
       "mode": "0000644",
       "sha256": "f9f4e8e009fab44c47fd3651c5d44da94cec1fd4cb7ad0931515fadbd6526280"
     },
+    {
+      "path": "modules/copy-desk/chrome-pages.js",
+      "mode": "0000644",
+      "sha256": "970750a57f2bd8324a908dbf9845407713d0cd99a94686dce9f75a02cc8e1538"
+    },
     {
       "path": "modules/copy-desk/docx.js",
       "mode": "0000644",
@@ -4527,7 +4532,7 @@
     {
       "path": "modules/copy-desk/page-data.js",
       "mode": "0000644",
-      "sha256": "12fb4d4d4d2b8787ce6bd571883777a51c4367898dff713f9832e87882e4f90c"
+      "sha256": "dcce3339f174c63ba5ae84f48dd0c167c1308aeb17eb31280f459aa1c549e8e1"
     },
     {
       "path": "modules/copy-desk/page-decisions.js",
@@ -4589,6 +4594,11 @@
       "mode": "0000644",
       "sha256": "662ef9fbb323904f50fd60f3636517e474e6c74a0679dd05b7a6b9135d671c3d"
     },
+    {
+      "path": "modules/copy-desk/tests/copy_chrome_pages.mjs",
+      "mode": "0000644",
+      "sha256": "8103d82f34ab9506b9e9dca5e9f50892eb75d86f5175a2aec193b09fa4a26b3c"
+    },
     {
       "path": "modules/copy-desk/tests/copy_docx.mjs",
       "mode": "0000644",
@@ -4602,7 +4612,7 @@
     {
       "path": "modules/copy-desk/tests/copy_no_cms.mjs",
       "mode": "0000644",
-      "sha256": "3c5401ec7fbf0aa8a255f0a20ce4a25e285b5c7b3ceb90b88ba1434a837f17b6"
+      "sha256": "74a8241778ef5d0055b555fad41530a92cc88fe5760c25cda033d2898d206d89"
     },
     {
       "path": "modules/copy-desk/tests/copy_page_auto_close.mjs",
@@ -11022,12 +11032,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "16cc1dbb15d7cbd210116b8acf20bbe1260aa016346b2926553144ea154ecd6f"
+      "sha256": "3e38ce5d5f5522e4d6479a3430df28858f36be6e79d2cd11808ed7e128fc4f3b"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "16007ece65168cf42065523f9686d4e88d1a39c8a6c900a4cc10e1cb543da5a9"
+      "sha256": "2b11d8a46c71afe00d28af274de396b51a4d9b02c69bdd8b94cd2a1a42245495"
     },
     {
       "path": "public-docs/index.html",
@@ -11047,7 +11057,7 @@
     {
       "path": "release-notes.json",
       "mode": "0000644",
-      "sha256": "96427d40bf22a0e940f9d758c4a0a75c0e6a1316df5235b38a4dd8adc122c757"
+      "sha256": "bac27f8f10894962e543895c14c9288b5bf5b950189c0fd57b97fec57c776cac"
     },
     {
       "path": "scripts/bongos/CLAUDE.md",
@@ -11377,7 +11387,7 @@
     {
       "path": "scripts/bongos/copy-apply.js",
       "mode": "0000644",
-      "sha256": "de6740eb1337c57f4aaff876ae30f32ecd41ead23702c577837a2d7b41684302"
+      "sha256": "9bbad1fba796c563cb103951e7f583effd9c2f9bccf3eff26ebe3e5fe53b6116"
     },
     {
       "path": "scripts/bongos/copy-inventory.js",
@@ -12652,7 +12662,7 @@
     {
       "path": "scripts/bongos/tweak-renders.js",
       "mode": "0000644",
-      "sha256": "b0925b4fc9a7074b67503d5a9c0e724be8ccaf8c12441f460b687035ba1977ee"
+      "sha256": "4cd4613fac825b740b3e1dd33b4919b51149afaf4c87eff1608e884198d3b83d"
     },
     {
       "path": "scripts/bongos/uat.js",
@@ -14922,7 +14932,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "74ae7e2fecaa02bb0c71e8afa7f6ccbb760a9f1312310148de94f63b21e15025"
+      "sha256": "292f7c0252f9ce177228d2e07178fb0d0aadb70a21bb228c33209fb0501d491e"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -15952,22 +15962,22 @@
     {
       "path": "tests/copy_desk_page_docx.mjs",
       "mode": "0000644",
-      "sha256": "626cfdb3e5dcef8ed67f45c2f2f9978ee14083e592ecda5359e3743785378f36"
+      "sha256": "af625c8c4819d7f0da0ebc676d9c41f579b9e8fe6fae8490c3d35f4a0dc48972"
     },
     {
       "path": "tests/copy_desk_page_editor.mjs",
       "mode": "0000644",
-      "sha256": "055a2da4169ba7bcc1f910bc0349abb4ccc015b0d95f3604bb64a5b7b9986221"
+      "sha256": "80fa1c728842097c1db2644f6d06e4e8f9c69a2f637a5ee8efe0b2b76c0508ed"
     },
     {
       "path": "tests/copy_desk_page_next.mjs",
       "mode": "0000644",
-      "sha256": "2649574d0cb0b3a6e261e654117cba0d692d1f4ffc0baa659cf4237559d8e352"
+      "sha256": "ac23b3e050c66a2f5a4ac796c73a22830d641706fc1fa98e65f0a545ee99d4cc"
     },
     {
       "path": "tests/copy_desk_page_reads.mjs",
       "mode": "0000644",
-      "sha256": "b8a0ae3befa8fe9336ba25da443ec32eed4930969567438a358290705de0d0ea"
+      "sha256": "54e0c9aaa64709c3804aeaf74107a46ca8aeae4a4054377a290c97ea58ce4163"
     },
     {
       "path": "tests/copy_desk_page_release.mjs",
@@ -15977,7 +15987,7 @@
     {
       "path": "tests/copy_desk_page_submit.mjs",
       "mode": "0000644",
-      "sha256": "c1014ec06b183c120bd95620f303e814c95437d710cf6ac6b4870106526a393a"
+      "sha256": "ee649108f713c33ffcca804b81efb77e5ccb33ea01a6053d18d2aade3f29a755"
     },
     {
       "path": "tests/copy_guarded_lines.mjs",
@@ -19272,7 +19282,7 @@
     {
       "path": "tests/profile_role_stats.mjs",
       "mode": "0000644",
-      "sha256": "f19d360e648c15576818313f80ea3079b933ccfc1f4ecb710198d993f2fa4054"
+      "sha256": "f800cbd9b93e57b4a7e45c79c1879263d55ad0617730f6471ff57103a1262793"
     },
     {
       "path": "tests/profile_role_tabs.mjs",

package/docs/module-api-changelog.md

@@ -3255,5 +3255,7 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.23.18 with no explicit bump. run 40613099968. (task 1002620)
   1.23.20 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         landed since 1.23.19 with no explicit bump. run 40709473534. (task 1002620)
+  1.23.21 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.23.20 with no explicit bump. run 40804017640. (task 1002620)
 ---------------------------------------------------------------------------
 ```

package/modules/copy-desk/chrome-pages.js

@@ -0,0 +1,178 @@
+// modules/copy-desk/chrome-pages.js — the shared page chrome as its own pages
+// (task 1004904, ADR 0341 D3's "shared" placement).
+//
+// The reader (scripts/bongos/page-reader.js) records every visible line of a page,
+// and the skip link, top bar, navigation and footer are on every page of a
+// surface, so each page's reading carried the same few dozen lines. An artist
+// tweaking one page was shown, and could rewrite, words that change on all of
+// them. This is the read-side split:
+//
+//   * A real page's reading loses its chrome lines. They are not shown, not
+//     editable, not counted, not in its drift.
+//   * Each surface gains one page per chrome section: "<surface>:top-bar",
+//     "<surface>:navigation", "<surface>:footer", "<surface>:skip-link". Each
+//     lists every DISTINCT line of that section once, in the order it first
+//     appears, and goes through the same claim, draft, submit and approve flow
+//     as any page.
+//
+// PURE, like pages.js: it derives from the two committed artifacts and writes
+// nothing. The generators and the committed files are untouched, so the
+// reader's --stale check and the registry are exactly as they were, and a
+// surface with no chrome gains no page.
+
+'use strict';
+
+const crypto = require('node:crypto');
+
+// section name the reader writes -> the page it becomes
+const CHROME = Object.freeze([
+  Object.freeze({ section: 'top bar', slug: 'top-bar', title: 'Top bar' }),
+  Object.freeze({ section: 'navigation', slug: 'navigation', title: 'Navigation' }),
+  Object.freeze({ section: 'footer', slug: 'footer', title: 'Footer' }),
+  Object.freeze({ section: 'skip link', slug: 'skip-link', title: 'Skip link' }),
+  Object.freeze({ section: 'sidebar', slug: 'sidebar', title: 'Sidebar' }),
+]);
+
+const CHROME_SECTIONS = new Set(CHROME.map((c) => c.section));
+
+function isChromeLine(line) {
+  return !!line && line.placement === 'shared' && CHROME_SECTIONS.has(line.section);
+}
+
+function hash(parts) {
+  return crypto.createHash('sha256').update(parts.join('\n')).digest('hex').slice(0, 16);
+}
+
+function key(n) { return `L${String(n).padStart(4, '0')}`; }
+
+function countsOf(lines) {
+  return {
+    lines: lines.length,
+    placed: lines.filter((l) => l.placement === 'placed').length,
+    shared: lines.filter((l) => l.placement === 'shared').length,
+    unplaced: lines.filter((l) => l.placement === 'unplaced').length,
+  };
+}
+
+// The chrome page ids a surface can have, so a caller can tell one from a page.
+// A reading names its surface; one that does not is read from its id.
+function surfaceOf(page) { return page.surface || String(page.id || '').split(':')[0]; }
+
+function chromePageId(surface, slug) { return `${surface}:${slug}`; }
+function isChromePageId(id) {
+  const m = /^[a-z0-9-]+:([a-z0-9-]+)$/.exec(String(id || ''));
+  return !!m && CHROME.some((c) => c.slug === m[1]);
+}
+
+// readings -> readings, every real page without its chrome and the chrome pages
+// after them. Returns { readings, chrome } where chrome maps a surface to the
+// pages it gained (the inventory is built from it).
+function splitReadings(readings) {
+  const pages = Array.isArray(readings && readings.pages) ? readings.pages : [];
+  const real = [];
+  // surface -> slug -> { lines: Map text -> line, from: page, states: Set, sources: Set of pages }
+  const gathered = new Map();
+  const order = [];
+  for (const p of pages) {
+    const lines = Array.isArray(p && p.lines) ? p.lines : [];
+    const kept = [];
+    for (const l of lines) {
+      if (!isChromeLine(l)) { kept.push(l); continue; }
+      const spec = CHROME.find((c) => c.section === l.section);
+      const surface = surfaceOf(p);
+      if (!gathered.has(surface)) { gathered.set(surface, new Map()); order.push(surface); }
+      const bySlug = gathered.get(surface);
+      if (!bySlug.has(spec.slug)) bySlug.set(spec.slug, { spec, lines: new Map(), from: p, states: new Set(), sources: new Set() });
+      const g = bySlug.get(spec.slug);
+      const text = String(l.text);
+      const seen = g.lines.get(text);
+      if (!seen) g.lines.set(text, { ...l });
+      else if (!seen.file && l.file) { seen.file = l.file; seen.line = l.line; seen.string_id = l.string_id; }
+      if (!g.sources.has(p)) {
+        g.sources.add(p);
+        for (const s of p.states_read || []) g.states.add(s);
+      }
+    }
+    if (kept.length === lines.length) { real.push(p); continue; }
+    real.push({
+      ...p,
+      ...(p.counts ? { counts: countsOf(kept) } : {}),
+      lines: kept,
+    });
+  }
+  const chrome = new Map();
+  const made = [];
+  for (const surface of order) {
+    const bySlug = gathered.get(surface);
+    const list = [];
+    for (const spec of CHROME) {
+      const g = bySlug.get(spec.slug);
+      if (!g) continue;
+      const lines = [...g.lines.values()].map((l, i) => ({ ...l, key: key(i + 1) }));
+      const id = chromePageId(surface, spec.slug);
+      made.push({
+        id,
+        surface,
+        title: spec.title,
+        states_read: [...g.states],
+        files_hash: hash([...g.sources].map((s) => String(s.files_hash || ''))),
+        reading_hash: hash(lines.map((l) => `${l.section}\t${l.text}`)),
+        counts: countsOf(lines),
+        omitted: {},
+        notes: [`The ${spec.title.toLowerCase()} shown on every ${surface} page, each line once. Read from ${g.sources.size} pages.`],
+        lines,
+      });
+      list.push({ id, spec, from: g.from, sources: g.sources, lines });
+    }
+    chrome.set(surface, list);
+  }
+  const out = { ...readings, pages: [...real, ...made] };
+  if (readings.counts) {
+    // Totals are counted from the lines, not from each page's own counts: a
+    // reading need not carry them.
+    const all = countsOf(out.pages.flatMap((p) => p.lines || []));
+    out.counts = { ...readings.counts, pages: out.pages.length, ...all };
+  }
+  return { readings: out, chrome };
+}
+
+// inventory -> inventory, with the chrome pages of each surface after that
+// surface's own pages. A chrome page opens on the first page it was read from
+// (the live page shows it), and its files are the ones its lines come from, so
+// the claim's touches and the "changed since" read name the shell, not a page.
+function addChromePages(inventory, chrome) {
+  const pages = Array.isArray(inventory && inventory.pages) ? inventory.pages : [];
+  const byId = new Map(pages.map((p) => [p.id, p]));
+  const added = [];
+  for (const [surface, list] of chrome) {
+    for (const c of list) {
+      const src = byId.get(c.from.id) || {};
+      const files = [...new Set(c.lines.map((l) => l.file).filter(Boolean))];
+      added.push({
+        id: c.id,
+        surface,
+        module: src.module || null,
+        page: c.spec.slug,
+        id_from: 'shared-chrome',
+        title: c.spec.title,
+        document_title: `${c.spec.title} — shown on every ${surface} page`,
+        path: src.path || null,
+        url: src.url || null,
+        files,
+        states_files: [],
+        states: [],
+      });
+    }
+  }
+  const all = [...pages, ...added];
+  const out = { ...inventory, pages: all };
+  if (inventory.counts) out.counts = { ...inventory.counts, pages: all.length };
+  if (Array.isArray(inventory.surfaces)) {
+    const tally = new Map();
+    for (const p of all) tally.set(p.surface, (tally.get(p.surface) || 0) + 1);
+    out.surfaces = inventory.surfaces.map((s) => ({ ...s, pages: tally.get(s.id) || 0 }));
+  }
+  return out;
+}
+
+module.exports = { isChromeLine, isChromePageId, splitReadings, addChromePages };

package/modules/copy-desk/page-data.js

@@ -24,6 +24,7 @@
 
 const fs = require('node:fs');
 const path = require('node:path');
+const chromePages = require('./chrome-pages');
 
 const ROOT = path.resolve(__dirname, '..', '..');
 
@@ -73,7 +74,38 @@ function load(which) {
   return value;
 }
 
-function loadInventory() { return load('inventory'); }
-function loadReadings() { return load('readings'); }
+// The shared chrome (skip link, top bar, navigation, footer) is read out of
+// every page and handed back as pages of its own (chrome-pages.js, task
+// 1004904). Derived per raw load, so the committed files stay what the
+// generators wrote; a WeakMap keeps one derivation per cached read.
+const _derived = new WeakMap();
+function derive() {
+  const rawReadings = load('readings');
+  if (!rawReadings.ok) return null;
+  const hit = _derived.get(rawReadings);
+  if (hit) return hit;
+  const split = chromePages.splitReadings(rawReadings.data);
+  const value = { readings: { ok: true, data: split.readings, at: rawReadings.at }, chrome: split.chrome };
+  _derived.set(rawReadings, value);
+  return value;
+}
+
+const _derivedInventory = new WeakMap();
+function loadInventory() {
+  const raw = load('inventory');
+  if (!raw.ok) return raw;
+  const d = derive();
+  if (!d) return raw;
+  const key = d.readings;
+  const hit = _derivedInventory.get(raw);
+  if (hit && hit.key === key) return hit.value;
+  const value = { ok: true, data: chromePages.addChromePages(raw.data, d.chrome), at: raw.at };
+  _derivedInventory.set(raw, { key, value });
+  return value;
+}
+function loadReadings() {
+  const d = derive();
+  return d ? d.readings : load('readings');
+}
 
 module.exports = { ARTIFACTS, loadInventory, loadReadings, _setRoot };

package/modules/copy-desk/tests/copy_chrome_pages.mjs

@@ -0,0 +1,127 @@
+// modules/copy-desk/tests/copy_chrome_pages.mjs — the shared page chrome as its
+// own pages (task 1004904), pure.
+//
+// The skip link, top bar, navigation and footer are on every page of a surface.
+// A real page's reading must not carry them, and each surface must gain one
+// page per chrome section, each line once. Run over a small hand-made reading
+// (the rules) and over the committed artifacts (the real data holds them).
+
+import { strict as assert } from 'node:assert';
+import { createRequire } from 'node:module';
+
+const require = createRequire(import.meta.url);
+const chrome = require('../chrome-pages.js');
+const pageData = require('../page-data.js');
+
+let passed = 0;
+let failed = 0;
+function test(name, fn) {
+  try { fn(); passed++; console.log(`  ok  ${name}`); }
+  catch (err) { failed++; console.log(`  FAIL ${name}`); console.log(`     ${err.stack || err.message}`); }
+}
+
+const L = (key, section, text, placement, row) => ({ key, section, text, placement, ...(row || {}) });
+const READINGS = {
+  counts: { pages: 3, lines: 9, placed: 3, shared: 5, unplaced: 1 },
+  pages: [
+    { id: 'landing:index', surface: 'landing', reading_hash: 'h-index', files_hash: 'f1', states_read: ['in'], lines: [
+      L('L0001', 'skip link', 'Skip to content', 'shared', { file: 'shell.js', line: 9, string_id: 'a1' }),
+      L('L0002', 'top bar', 'Projects', 'shared', { file: 'nav.js', line: 3, string_id: 'a2' }),
+      L('L0003', 'hero', 'Build here', 'placed', { file: 'index.html', line: 4, string_id: 'a3' }),
+    ] },
+    { id: 'landing:account', surface: 'landing', reading_hash: 'h-acct', counts: { lines: 5, placed: 1, shared: 3, unplaced: 1 }, files_hash: 'f2', states_read: ['in', 'out'], lines: [
+      L('L0001', 'skip link', 'Skip to content', 'shared'),
+      L('L0002', 'top bar', 'Projects', 'shared'),
+      L('L0003', 'top bar', 'Sign in', 'shared'),
+      L('L0004', 'hero', 'Your account', 'placed'),
+      L('L0005', 'list', 'Something unplaced', 'unplaced'),
+    ] },
+    // No surface on the reading: it is read from the id.
+    { id: 'status:explore', reading_hash: 'h-exp', files_hash: 'f3', lines: [L('L0001', 'hero', 'Explore', 'placed')] },
+  ],
+};
+const INVENTORY = {
+  counts: { pages: 3, surfaces: 2 },
+  surfaces: [{ id: 'landing', pages: 2 }, { id: 'status', pages: 1 }],
+  pages: [
+    { id: 'landing:index', surface: 'landing', module: 'public-landing', page: 'index', title: 'Home', path: '/', url: 'https://{apex}/', files: ['index.html'] },
+    { id: 'landing:account', surface: 'landing', module: 'public-landing', page: 'account', title: 'Account', path: '/account', url: 'https://{apex}/account', files: ['account.html'] },
+    { id: 'status:explore', surface: 'status', module: 'status-ui', page: 'explore', title: 'Explore', path: '/explore', url: 'https://status.{apex}/explore', files: ['explore.html'] },
+  ],
+};
+
+const split = chrome.splitReadings(READINGS);
+const byId = (rd, id) => rd.pages.find((p) => p.id === id);
+
+test('a real page no longer carries its chrome, and keeps its own lines and keys', () => {
+  assert.deepEqual(byId(split.readings, 'landing:index').lines.map((l) => [l.key, l.section]), [['L0003', 'hero']]);
+  assert.deepEqual(byId(split.readings, 'landing:account').lines.map((l) => l.key), ['L0004', 'L0005']);
+  assert.equal(byId(split.readings, 'landing:index').reading_hash, 'h-index', 'a draft on the reading stays on it');
+  assert.deepEqual(byId(split.readings, 'landing:account').counts, { lines: 2, placed: 1, shared: 0, unplaced: 1 });
+});
+
+test('each chrome section becomes one page per surface, every distinct line once, in first-seen order', () => {
+  const top = byId(split.readings, 'landing:top-bar');
+  assert.deepEqual(top.lines.map((l) => [l.key, l.text]), [['L0001', 'Projects'], ['L0002', 'Sign in']]);
+  assert.deepEqual(byId(split.readings, 'landing:skip-link').lines.map((l) => l.text), ['Skip to content']);
+  assert.equal(byId(split.readings, 'landing:footer'), undefined, 'a surface with no footer gains no footer page');
+  assert.equal(byId(split.readings, 'status:top-bar'), undefined, 'a surface with no chrome gains no chrome page');
+});
+
+test('a chrome line keeps the registry row of its first occurrence that has one', () => {
+  const top = byId(split.readings, 'landing:top-bar').lines[0];
+  assert.deepEqual([top.file, top.line, top.string_id, top.placement], ['nav.js', 3, 'a2', 'shared']);
+});
+
+test('the reading totals follow the pages', () => {
+  assert.equal(split.readings.counts.pages, 5);
+  assert.equal(split.readings.counts.lines, 1 + 2 + 1 + 2 + 1);
+  assert.equal(split.readings.counts.shared, 3);
+});
+
+test('the inventory gains the same pages, after their surface, opening on a page that shows them', () => {
+  const inv = chrome.addChromePages(INVENTORY, split.chrome);
+  assert.deepEqual(inv.pages.map((p) => p.id), ['landing:index', 'landing:account', 'status:explore', 'landing:top-bar', 'landing:skip-link']);
+  const top = inv.pages.find((p) => p.id === 'landing:top-bar');
+  assert.deepEqual([top.surface, top.page, top.title, top.url], ['landing', 'top-bar', 'Top bar', 'https://{apex}/']);
+  assert.deepEqual(top.files, ['nav.js']);
+  assert.equal(inv.counts.pages, 5);
+  assert.deepEqual(inv.surfaces.map((s) => [s.id, s.pages]), [['landing', 4], ['status', 1]]);
+});
+
+test('a chrome page id is told from a page id', () => {
+  assert.equal(chrome.isChromePageId('landing:top-bar'), true);
+  assert.equal(chrome.isChromePageId('landing:index'), false);
+});
+
+test('only a shared line in a chrome section is chrome', () => {
+  assert.equal(chrome.isChromeLine(L('L1', 'top bar', 'x', 'placed')), false);
+  assert.equal(chrome.isChromeLine(L('L1', 'hero', 'x', 'shared')), false);
+  assert.equal(chrome.isChromeLine(L('L1', 'top bar', 'x', 'shared')), true);
+});
+
+test('the committed readings: no real page lists a skip link, top bar, navigation or footer line', () => {
+  const rd = pageData.loadReadings();
+  assert.equal(rd.ok, true);
+  for (const p of rd.data.pages) {
+    if (chrome.isChromePageId(p.id)) continue;
+    const bad = p.lines.filter(chrome.isChromeLine);
+    assert.equal(bad.length, 0, `${p.id} still lists ${bad.length} chrome lines`);
+  }
+});
+
+test('the committed data: every surface that had a top bar has a Top bar page, in both reads', () => {
+  const rd = pageData.loadReadings().data;
+  const inv = pageData.loadInventory().data;
+  for (const surface of ['builders', 'landing', 'status']) {
+    const id = `${surface}:top-bar`;
+    const r = rd.pages.find((p) => p.id === id);
+    assert.ok(r && r.lines.length > 0, `${id} in the readings`);
+    assert.ok(inv.pages.find((p) => p.id === id), `${id} in the inventory`);
+    assert.equal(new Set(r.lines.map((l) => l.text)).size, r.lines.length, `${id} lists each line once`);
+  }
+  assert.equal(rd.pages.length, inv.pages.length, 'the two reads name the same pages');
+});
+
+console.log(`\n${passed} passed, ${failed} failed`);
+process.exit(failed ? 1 : 0);

package/modules/copy-desk/tests/copy_no_cms.mjs

@@ -70,7 +70,7 @@ const stripComments = (src) => src
 // Task 1004435 added routes/copy-desk-artist.js (one artist's approved pages, a
 // read) and artist-stats.js's readArtistPages under it.
 const ROUTE_FILES = fs.readdirSync(path.join(MODULE_DIR, 'routes')).filter((f) => f.endsWith('.js')).sort().map((f) => `routes/${f}`);
-const MODULE_FILES = ['registry.js', 'flags.js', 'proposals.js', 'pages.js', 'page-status.js', 'page-data.js', 'docx.js', 'page-decisions.js', 'artist-stats.js', 'auto-close.js', ...ROUTE_FILES];
+const MODULE_FILES = ['registry.js', 'flags.js', 'proposals.js', 'pages.js', 'page-status.js', 'page-data.js', 'chrome-pages.js', 'docx.js', 'page-decisions.js', 'artist-stats.js', 'auto-close.js', ...ROUTE_FILES];
 
 console.log('copy-desk: NO LIVE CMS');
 

package/package-lock.json

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

package/package.json

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

@@ -10509,5 +10509,11 @@
       "id": "1002983",
       "text": "The Community home now shows a rising builder and the projects on the move this month, under the builders of the week, month and year."
     }
+  ],
+  "1.23.21": [
+    {
+      "id": "1004904",
+      "text": "In Tweak Mode, the shared top bar, navigation, footer and skip link no longer clutter every page; each is now its own page to review once."
+    }
   ]
 }

package/scripts/bongos/copy-apply.js

@@ -62,6 +62,7 @@ const ci = require('./copy-inventory.js');
 const proposals = require('../../modules/copy-desk/proposals.js');
 // The page tweak format's one reader and writer (ADR 0341 D4), for --batch.
 const pages = require('../../modules/copy-desk/pages.js');
+const chromePages = require('../../modules/copy-desk/chrome-pages.js');
 const { apiCall, arg, hasFlag, cliExit } = require('./cli-lib.js');
 
 const REPO_ROOT = path.resolve(__dirname, '..', '..');
@@ -559,7 +560,8 @@ function rereadStalePages({ spawn = spawnSync } = {}) {
 function readingFor(pageId, { readFile = (p) => fs.readFileSync(p, 'utf8') } = {}) {
   try {
     const doc = JSON.parse(readFile(path.join(REPO_ROOT, 'docs', 'page-readings.json')));
-    return (doc.pages || []).find((p) => p.id === pageId) || null;
+    // A shared-chrome page (task 1004904) exists only in the derived reading.
+    return (chromePages.splitReadings(doc).readings.pages || []).find((p) => p.id === pageId) || null;
   } catch { return null; }
 }
 

package/scripts/bongos/tweak-renders.js

@@ -87,10 +87,17 @@ function slotFilesIn(dir, { exists = (p) => fs.existsSync(p) } = {}) {
 
 // ---- the browser half --------------------------------------------------------
 
+// The inventory as Tweak Mode sees it, shared-chrome pages included (task 1004904):
+// the committed file does not list them, so planning from it alone is unknown_page.
+function derivedInventory() {
+  const r = require('../../modules/copy-desk/page-data.js').loadInventory();
+  return r.ok ? r.data : { pages: [] };
+}
+
 async function shoot({ pageId, phase, out }) {
   const reader = require('./page-reader.js');
   const kit = require('../../modules/ui-design/kit/lib');
-  const inventory = JSON.parse(fs.readFileSync(path.join(REPO_ROOT, reader.INVENTORY_REL), 'utf8'));
+  const inventory = derivedInventory();
   const planned = planForPage(pageId, { inventory, planRenders: reader.planRenders });
   if (!planned.ok) { console.error(`tweak-renders: ${planned.code} (${pageId})`); return 2; }
   const { page, plan } = planned;
@@ -172,7 +179,7 @@ async function main(args) {
     if (!pageId) { console.error('usage: node scripts/bongos/tweak-renders.js shoot --page <id> --phase before|after --out <dir>'); return 2; }
     if (cmd === 'plan') {
       const reader = require('./page-reader.js');
-      const inventory = JSON.parse(fs.readFileSync(path.join(REPO_ROOT, reader.INVENTORY_REL), 'utf8'));
+      const inventory = derivedInventory();
       const planned = planForPage(pageId, { inventory, planRenders: reader.planRenders });
       if (!planned.ok) { console.error(`tweak-renders: ${planned.code} (${pageId})`); return 2; }
       for (const phase of taskVisuals.TWEAK_RENDER_PHASES) for (const s of shotsFor(phase, planned.plan)) console.log(`${s.slot}\t${s.width}px\t${s.renderMode}\t${planned.plan.url}`);

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.23.20'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.23.21'; // 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/copy_desk_page_docx.mjs

@@ -75,12 +75,12 @@ const errDetail = (b) => (b && b.error && b.error.details) || {};
 
 const L = (key, section, text, placement, row) => ({ key, section, text, placement, ...(row || {}) });
 const STUDIO_LINES = [
-  L('L0001', 'top bar', 'Studio', 'shared', { file: 'modules/hall-ui/public/shell.js', line: 40, string_id: 'sh0001' }),
-  L('L0002', 'top bar', 'Ctrl K', 'shared'),
+  L('L0001', 'shell bar', 'Studio', 'shared', { file: 'modules/hall-ui/public/shell.js', line: 40, string_id: 'sh0001' }),
+  L('L0002', 'shell bar', 'Ctrl K', 'shared'),
   L('L0003', 'greeting', 'Welcome back', 'placed', { file: 'modules/hall-ui/public/studio.html', line: 12, string_id: 'st0003' }),
   L('L0004', 'greeting', '{…} pages tweaked', 'placed', { file: 'modules/hall-ui/public/studio.js', line: 88, string_id: 'st0004' }),
   L('L0005', 'greeting', 'Your desk is quiet today', 'unplaced'),
-  L('L0006', 'footer', 'Resume where you left off', 'placed', { file: 'modules/hall-ui/public/studio.html', line: 90, string_id: 'st0006' }),
+  L('L0006', 'shell foot', 'Resume where you left off', 'placed', { file: 'modules/hall-ui/public/studio.html', line: 90, string_id: 'st0006' }),
 ];
 const INDEX_LINES = [
   L('L0001', 'hero', 'Build here', 'placed', { file: 'modules/public-landing/public/index.html', line: 3, string_id: 'ix0001' }),

package/tests/copy_desk_page_editor.mjs

@@ -64,11 +64,11 @@ const errCode = (b) => (b && b.error && b.error.code) || null;
 
 const L = (key, section, text, placement, row) => ({ key, section, text, placement, ...(row || {}) });
 const STUDIO_LINES = [
-  L('L0001', 'top bar', 'Studio', 'shared', { file: 'modules/hall-ui/public/shell.js', line: 40, string_id: 'sh0001' }),
-  L('L0002', 'top bar', 'Ctrl K', 'shared'),
+  L('L0001', 'shell bar', 'Studio', 'shared', { file: 'modules/hall-ui/public/shell.js', line: 40, string_id: 'sh0001' }),
+  L('L0002', 'shell bar', 'Ctrl K', 'shared'),
   L('L0003', 'greeting', 'Welcome back', 'placed', { file: 'modules/hall-ui/public/studio.html', line: 12, string_id: 'st0003' }),
   L('L0004', 'greeting', 'Your desk is quiet today', 'unplaced'),
-  L('L0005', 'footer', 'Resume where you left off', 'placed', { file: 'modules/hall-ui/public/studio.html', line: 90, string_id: 'st0005' }),
+  L('L0005', 'shell foot', 'Resume where you left off', 'placed', { file: 'modules/hall-ui/public/studio.html', line: 90, string_id: 'st0005' }),
 ];
 const readingsWith = (studioHash, lines = STUDIO_LINES) => ({
   schema: 1,
@@ -96,13 +96,13 @@ try {
     assert.equal(r.body.page.surface_word, 'builders hall');
     assert.equal(r.body.reading.reading_hash, 'studio-h1');
     assert.deepEqual(r.body.reading.lines.map((l) => [l.key, l.section, l.text, l.note]), [
-      ['L0001', 'top bar', 'Studio', 'shared'],
+      ['L0001', 'shell bar', 'Studio', 'shared'],
       // Shared shell text with no registry row cannot be applied mechanically,
       // so it goes to an engineer at submit, and the note says so.
-      ['L0002', 'top bar', 'Ctrl K', 'engineer'],
+      ['L0002', 'shell bar', 'Ctrl K', 'engineer'],
       ['L0003', 'greeting', 'Welcome back', null],
       ['L0004', 'greeting', 'Your desk is quiet today', 'engineer'],
-      ['L0005', 'footer', 'Resume where you left off', null],
+      ['L0005', 'shell foot', 'Resume where you left off', null],
     ]);
     assert.deepEqual(r.body.notes, { shared: 'changes this on every page', engineer: 'goes to an engineer', guarded: 'must still say' });
     assert.equal(r.body.round, null);

package/tests/copy_desk_page_next.mjs

@@ -101,7 +101,8 @@ try {
 
     const { status, body } = await getNext(ALICE);
     assert.equal(status, 200, JSON.stringify(body));
-    assert.deepEqual(order(body), ['landing:projects', 'status:explore', 'builders:studio', 'landing:index']);
+    assert.deepEqual(order(body), ['landing:projects', 'status:explore', 'builders:studio', 'landing:index', 'landing:skip-link']);
+    // landing:skip-link is the surface's shared chrome, a page of its own, untweaked
     assert.equal(body.recommended.why, '2 people asked for it');
     assert.equal(body.recommended.why_line, 'landing · 2 people asked for it · untweaked');
     assert.equal(body.alternatives[1].why, '1 line changed since your last pass', 'the round artist is the web claim, alice');
@@ -145,7 +146,7 @@ try {
     assert.equal(take.body.page_id, 'status:explore');
     const after = (await getNext(CAROL)).body;
     assert.deepEqual(after.skipped.map((s) => s.page_id).sort(), ['landing:projects', 'status:explore']);
-    assert.deepEqual(order(after), ['builders:studio', 'landing:index']);
+    assert.deepEqual(order(after), ['builders:studio', 'landing:index', 'landing:skip-link', 'builders:top-bar']);
   });
 
   await test('a submitted round is in flight: skipped, and the claim agrees', async () => {

package/tests/copy_desk_page_reads.mjs

@@ -115,7 +115,8 @@ try {
     assert.equal(p('builders:studio').status_label, 'Text tweaked 1x');
     assert.equal(p('builders:studio').drift.changed_count, 1);
     assert.equal(p('builders:studio').open_round.state, 'writing');
-    assert.deepEqual(body.surfaces.map((s) => s.line), ['1 of 2 tweaked', '1 of 1 tweaked', '0 of 1 tweaked']);
+    // Each surface's shared chrome is a page of its own now: landing gains a skip link page, builders a top bar page.
+    assert.deepEqual(body.surfaces.map((s) => s.line), ['1 of 3 tweaked', '1 of 2 tweaked', '0 of 1 tweaked']);
     assert.deepEqual(body.readings, { ok: true, at: 'docs/page-readings.json' });
     assert.deepEqual(body.inventory, { at: 'docs/page-inventory.json' });
   });

package/tests/copy_desk_page_submit.mjs

@@ -65,8 +65,8 @@ const errDetail = (b) => (b && b.error && b.error.details) || {};
 // and an unplaced line.
 const L = (key, section, text, placement, row) => ({ key, section, text, placement, ...(row || {}) });
 const STUDIO_LINES = [
-  L('L0001', 'top bar', 'Studio', 'shared', { file: 'modules/hall-ui/public/shell.js', line: 40, string_id: 'sh0001' }),
-  L('L0002', 'top bar', 'Ctrl K', 'shared'),
+  L('L0001', 'shell bar', 'Studio', 'shared', { file: 'modules/hall-ui/public/shell.js', line: 40, string_id: 'sh0001' }),
+  L('L0002', 'shell bar', 'Ctrl K', 'shared'),
   L('L0003', 'greeting', 'Welcome back', 'placed', { file: 'modules/hall-ui/public/studio.html', line: 12, string_id: 'st0003' }),
   L('L0004', 'greeting', '{…} pages tweaked', 'placed', { file: 'modules/hall-ui/public/studio.js', line: 88, string_id: 'st0004' }),
   L('L0005', 'greeting', 'Your desk is quiet today', 'unplaced'),
@@ -360,6 +360,34 @@ try {
       guarded._setList(null);
     }
   });
+
+  // Task 1004904: a shared-chrome page is a page like any other, whole path.
+  await test('a chrome page: claimed, drafted and submitted through the real routes, its lines placed from the first occurrence', async () => {
+    const TOP = '/copy-desk/pages/builders:top-bar';
+    const chromeLine = (key, text, row) => L(key, 'top bar', text, 'shared', row);
+    const r0 = readingsWith(
+      { hash: 'studio-h1', lines: [chromeLine('L0001', 'Projects', { file: 'modules/hall-ui/public/shell.js', line: 41, string_id: 'sh0041' }), chromeLine('L0002', 'Sign out', { file: 'modules/hall-ui/public/shell.js', line: 42, string_id: 'sh0042' }), ...STUDIO_LINES.slice(2)] },
+      { hash: 'index-h2', lines: INDEX_LINES },
+    );
+    writeReadings(r0);
+    const studio = await call('GET', '/copy-desk/pages/builders:studio', ALICE);
+    assert.equal(studio.status, 200);
+    const page = await call('GET', TOP, ALICE);
+    assert.equal(page.status, 200, JSON.stringify(page.body));
+    assert.equal((await call('POST', `${TOP}/claim`, ALICE)).status, 201);
+    const read = await call('GET', `${TOP}/draft`, ALICE);
+    assert.equal(read.status, 200, JSON.stringify(read.body));
+    const hash = read.body.reading.reading_hash || read.body.reading_hash;
+    assert.ok(hash, 'the chrome page has a reading hash of its own');
+    assert.deepEqual(read.body.reading.lines.map((l) => l.text), ['Projects', 'Sign out']);
+    const saved = await call('PUT', `${TOP}/draft`, ALICE, { reading_hash: hash, lines: [{ key: 'L0001', after: 'Your projects' }] });
+    assert.equal(saved.status, 200, JSON.stringify(saved.body));
+    const sub = await call('POST', `${TOP}/submit`, ALICE);
+    assert.equal(sub.status, 200, JSON.stringify(sub.body));
+    const round = db.tasks.find((t) => t.source_ref === 'page-tweak/builders:top-bar/r1');
+    const batch = pages.parseBatchBlock(round.description).batch;
+    assert.deepEqual(batch.lines.map((l) => [l.key, l.file, l.line, l.before, l.after]), [['L0001', 'modules/hall-ui/public/shell.js', 41, 'Projects', 'Your projects']]);
+  });
 } finally {
   server.close();
   pageData._setRoot(null);

package/tests/profile_role_stats.mjs

@@ -69,7 +69,8 @@ const LEDGER_ROWS = [
 // ---- the page-tweak ledger (Ines and one other artist) ----------------------
 
 const pages = require('../modules/copy-desk/pages.js');
-const inventory = require('../docs/page-inventory.json');
+// The studio's own inventory: the real pages plus each surface's chrome pages.
+const inventory = require('../modules/copy-desk/page-data.js').loadInventory().data;
 const onSurface = (s) => inventory.pages.filter((p) => p.surface === s).map((p) => p.id);
 const LANDING = onSurface('landing');
 const BUILDERS = onSurface('builders');