@bongos/core 1.18.3 → 1.18.18

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

@@ -109,31 +109,9 @@
     return `${prefix}--${v > 0 ? 'up' : v < 0 ? 'down' : 'flat'}`;
   }
 
-  // renderPager — the works-wrought / roster twin pager. Clamps state.page into
-  // range (mutating it), hides the pager when there is ≤1 page, writes the
-  // "start–end of total" range label, and toggles prev/next. Returns { start } so
-  // the caller can slice. NOTE: the leaderboard pager deliberately does NOT use
-  // this — it never clamps and keeps its range/buttons always visible.
-  function renderPager(state, total, pageSize, { pagerEl, rangeEl, prevBtn, nextBtn }) {
-    const pageCount = Math.max(1, Math.ceil(total / pageSize));
-    if (state.page >= pageCount) state.page = pageCount - 1;
-    if (state.page < 0) state.page = 0;
-    const start = state.page * pageSize;
-    if (pagerEl) {
-      if (pageCount > 1) {
-        pagerEl.hidden = false;
-        if (rangeEl) {
-          const end = Math.min(start + pageSize, total);
-          rangeEl.textContent = `${start + 1}–${end} of ${total}`;
-        }
-        if (prevBtn) prevBtn.disabled = state.page === 0;
-        if (nextBtn) nextBtn.disabled = state.page >= pageCount - 1;
-      } else {
-        pagerEl.hidden = true;
-      }
-    }
-    return { start };
-  }
+  // (task 2174: the hand-rolled renderPager the ships + roster tables shared is
+  // gone — both now use the OTBKit pager, which clamps, persists per-page and
+  // paints the bar from one implementation.)
 
   // ---- repo info (the clone URL the welcome walkthrough hands a newcomer) ---
   //
@@ -248,6 +226,11 @@
   // Standing-card "Action needed" banner survives re-renders that don't carry
   // needs (same pattern as lastCostState).
   let lastNeedsState = null;
+  // ADR 0120 part 5: cache the most recent /me `needs_rebase_tasks` so the
+  // Standing-card "Rebase owed" banner survives re-renders that don't carry it
+  // (same pattern as lastNeedsState). These are un-landed confirmed tasks that
+  // GATE new claims and whose reward is pending until they land on ship.
+  let lastRebaseState = null;
 
   // Cached /me payload from the most recent loadAll() call. Used by the
   // welcome-chip reopen path (#408) so we can re-render the welcome panel
@@ -300,30 +283,59 @@
   // source of truth is now the hall-widget registry (window.OTB, dom-utils.js):
   // hall-widgets.js registers the core widgets above (loads before this script),
   // a module's client script registers its own (gated by loader.js + moduleOn).
-  // SECTIONS() derives the nav rows from the registry, preserving order + the
-  // per-widget gates (welcomeChip / archonOnly) carried on the registry entry.
-  const SECTIONS = () => window.OTB.hallWidgets().map((wgt) => ({
-    id: wgt.id,
-    label: wgt.label,
-    view: wgt.view || 'home',
-    welcomeChip: !!wgt.welcomeChip,
-    archonOnly: !!wgt.archonOnly,
-  }));
+  // The rail (renderNav) reads window.OTB.hallWidgets() straight, applying the
+  // per-widget gates (archonOnly) carried on the registry entry.
 
   // task 1723: which container a widget mounts into. A widget may declare
-  // `zone` (a fine-grained Home slot) or `view` (home/standing/achievements/
-  // leaderboard); both default to the Home main zone so pre-redesign module
-  // widgets still land somewhere visible. Every container lives INSIDE <main>,
-  // so the `main .scroll` sweeps (newcomer gate, landing hide-all) keep working.
+  // `zone` (a fine-grained Home slot), `group` (a Standing rail section, task
+  // 2174) or `view` (home/standing/leaderboard); all default to the Home main
+  // zone so pre-redesign module widgets still land somewhere visible. Every
+  // container lives INSIDE <main>, so the `main .scroll` sweeps (newcomer gate,
+  // landing hide-all) keep working.
   const VIEW_CONTAINERS = {
     home: 'zone-home-main',
     standing: 'zone-standing',
-    achievements: 'zone-achievements',
     leaderboard: 'zone-leaderboard',
   };
+
+  // task 2174: Standing is an identity band + four rail sections grouped by the
+  // question they answer. This array is the rail's order and its labels; each
+  // entry gets a #standing-<key> anchor container inside #zone-standing that the
+  // group's widgets mount into. The Profile card declares no group, so it stays
+  // above them as the band.
+  const STANDING_GROUPS = [
+    { key: 'work', label: 'Work' },
+    { key: 'karma', label: 'Karma' },
+    { key: 'progress', label: 'Progress' },
+    { key: 'achievements', label: 'Achievements' },
+  ];
+  const standingGroupId = (key) => `standing-${key}`;
+
+  // Create ALL four group containers the first time any group is asked for, so
+  // the rail's order is STANDING_GROUPS' — not whichever group's first widget
+  // happens to register earliest. Creating them on demand (rather than up front)
+  // is what keeps the Profile band on top: an ungrouped Standing widget appends
+  // straight to #zone-standing, so anything registered before the first grouped
+  // widget lands above the groups. Idempotent.
+  function ensureStandingGroups() {
+    const zone = document.getElementById('zone-standing');
+    if (!zone || document.getElementById(standingGroupId(STANDING_GROUPS[0].key))) return;
+    for (const g of STANDING_GROUPS) {
+      const el = document.createElement('div');
+      el.id = standingGroupId(g.key);
+      el.className = 'standing-group';
+      zone.appendChild(el);
+    }
+  }
+
   function widgetHost(wgt) {
     const zone = wgt.zone ? document.getElementById(`zone-${wgt.zone}`) : null;
     if (zone) return zone;
+    if (wgt.group) {
+      ensureStandingGroups();
+      const group = document.getElementById(standingGroupId(wgt.group));
+      if (group) return group;
+    }
     const viewId = VIEW_CONTAINERS[wgt.view || 'home'] || VIEW_CONTAINERS.home;
     return document.getElementById(viewId) || document.querySelector('main');
   }
@@ -409,10 +421,11 @@
     window.OTBBranding?.applyCurrencyLabels?.(main);
   }
 
-  // The IntersectionObserver instance survives across re-renders (it's
-  // disconnected and re-created when nav contents change). Stored on
-  // module scope so nav re-renders can dispose of the previous one.
-  let navObserver = null;
+  // task 2174: the Standing rail is an OTBKit sectionRail in scrollspy mode —
+  // the kit owns the IntersectionObserver that used to live here as
+  // setupNavObserver(). Built lazily on the first renderNav and refreshed after,
+  // so the same instance survives re-renders.
+  let standingRail = null;
 
   // task 1723: the old sticky chip strip is gone. renderNav now settles the
   // three nav surfaces the shell owns or hosts:
@@ -420,8 +433,10 @@
   //      the server enforces every page + endpoint regardless, ADR 0016);
   //   2. the Home re-open affordance for a dismissed welcome banner (the old
   //      welcome chip's only job);
-  //   3. the Standing view's in-page section rail (#standing-rail), rebuilt from
-  //      the registry's standing widgets that are currently shown.
+  //   3. the Standing view's in-page section rail (#standing-rail) — task 2174:
+  //      one entry per GROUP that currently has something to show, not one per
+  //      widget. A group with every member hidden (a newcomer, an empty karma
+  //      feed) hides itself and drops off the rail.
   // Keeps its (builder, onboarding) signature — all call sites pass them.
   function renderNav(builder, onboarding) {
     const isArchon = (builder?.rank || '').toLowerCase() === 'archon';
@@ -446,25 +461,30 @@
       reopen.hidden = !(welcomeEl && welcomeEl.hidden && !graduated);
     }
 
-    // Standing section rail: one link per visible standing panel.
+    // Standing section rail: one link per group with a visible member.
     const rail = document.getElementById('standing-rail');
     if (rail) {
-      const items = SECTIONS().filter((s) => {
-        if (s.view !== 'standing') return false;
-        if (s.archonOnly && !isArchon) return false;
-        const el = document.getElementById(s.id);
-        return !!el && !el.hidden;
-      });
-      rail.innerHTML = items.map((s) =>
-        `<a class="rail-link" href="#${escapeHtml(s.id)}" data-section="${escapeHtml(s.id)}">${escapeHtml(s.label)}</a>`
-      ).join('');
-      rail.hidden = items.length < 2;
+      const shown = new Set(
+        window.OTB.hallWidgets()
+          .filter((wgt) => {
+            if (!wgt.group) return false;
+            if (wgt.archonOnly && !isArchon) return false;
+            const el = document.getElementById(wgt.id);
+            return !!el && !el.hidden;
+          })
+          .map((wgt) => wgt.group),
+      );
+      // An empty group would otherwise leave a gap in the panel column.
+      for (const g of STANDING_GROUPS) {
+        const el = document.getElementById(standingGroupId(g.key));
+        if (el) el.hidden = !shown.has(g.key);
+      }
+      const items = STANDING_GROUPS
+        .filter((g) => shown.has(g.key))
+        .map((g) => ({ id: standingGroupId(g.key), label: g.label }));
+      if (standingRail) standingRail.refresh(items);
+      else standingRail = window.OTBKit.makeSectionRail({ mount: rail, items, mode: 'scrollspy' });
     }
-
-    // Rebuild the observer to track whichever sections are currently in
-    // the rail. Doing this on each renderNav call covers both initial load
-    // and post-dismiss/re-open re-renders.
-    setupNavObserver();
   }
 
   // #748: newcomer gate. A Xenos sees a deliberately spare hall — their
@@ -502,56 +522,6 @@
     // gates. This function keeps only the widget-section sweep.
   }
 
-  function setupNavObserver() {
-    if (navObserver) {
-      navObserver.disconnect();
-      navObserver = null;
-    }
-    const list = document.getElementById('standing-rail');
-    if (!list) return;
-
-    const chipsBySection = new Map();
-    list.querySelectorAll('a[data-section]').forEach((a) => {
-      chipsBySection.set(a.dataset.section, a);
-    });
-    if (chipsBySection.size === 0) return;
-
-    const sections = Array.from(chipsBySection.keys())
-      .map((id) => document.getElementById(id))
-      .filter((el) => el && !el.hidden);
-
-    if (sections.length === 0) return;
-
-    // rootMargin trims the active band to roughly the upper third of the
-    // viewport (top 40% ignored, bottom 55% ignored — a ~5% horizontal
-    // strip about a third of the way down the page). The active flip
-    // happens when a section headline crosses that band, not when the
-    // section first peeks in. Matches the rationale in the task spec.
-    navObserver = new IntersectionObserver((entries) => {
-      for (const entry of entries) {
-        if (entry.isIntersecting) {
-          const id = entry.target.id;
-          chipsBySection.forEach((chip, sectionId) => {
-            chip.classList.toggle('rail-link--active', sectionId === id);
-          });
-          // Auto-scroll the active chip into view on mobile so the user
-          // can see what section they're in without having to scroll the
-          // nav strip themselves.
-          const active = chipsBySection.get(id);
-          if (active && window.matchMedia('(max-width: 640px)').matches) {
-            active.scrollIntoView({ behavior: 'smooth', inline: 'center', block: 'nearest' });
-          }
-          break;
-        }
-      }
-    }, {
-      rootMargin: '-40% 0px -55% 0px',
-      threshold: 0,
-    });
-
-    sections.forEach((sec) => navObserver.observe(sec));
-  }
-
   function reopenWelcomeFromNav(e) {
     e.preventDefault();
     const id = window.__buildersSelfId;
@@ -712,6 +682,7 @@
     const karma = lastKarmaState;
     const cost = lastCostState;
     const needs = lastNeedsState;
+    const rebase = lastRebaseState;
     const avatar = builder.avatar_url
       ? `<img class="profile__avatar" src="${escapeHtml(builder.avatar_url)}" alt="">`
       : `<div class="profile__avatar" aria-hidden="true"></div>`;
@@ -770,7 +741,20 @@
          </div>`
       : '';
 
+    // ADR 0120 part 5: the "Rebase owed" banner. `needs_rebase_tasks` are un-landed
+    // confirmed tasks the publish-reconciler flagged — they GATE new claims (part 3)
+    // and their per-task reward is PENDING until they land on ship (part 1). Rendered
+    // above the other banners since it blocks the queue; reads the SAME /me field the
+    // CLI (/builder-start) does. `task N` form, never a bare #N (the ref rule).
+    const rebaseTasks = Array.isArray(rebase) ? rebase.filter((t) => t && t.id != null) : [];
+    const rebaseBanner = rebaseTasks.length
+      ? `<div class="profile__rebase" role="status">
+           <strong>Rebase owed — new claims paused</strong> — ${rebaseTasks.length} confirmed ${rebaseTasks.length === 1 ? 'task' : 'tasks'} can't land yet, so ${rebaseTasks.length === 1 ? 'its' : 'their'} reward is pending (it lands on ship). Rebase + re-ship or release ${rebaseTasks.length === 1 ? 'it' : 'them'}: ${rebaseTasks.map((t) => 'task ' + escapeHtml(String(t.id))).join(', ')}.
+         </div>`
+      : '';
+
     $('#profile-body').innerHTML = `
+      ${rebaseBanner}
       ${needsBanner}
       ${budgetBanner}
       ${avatar}
@@ -824,6 +808,7 @@
     const homeStanding = document.getElementById('home-standing-body');
     if (homeStanding) {
       homeStanding.innerHTML = `
+        ${rebaseBanner}
         ${needsBanner}
         ${budgetBanner}
         <div class="profile profile--compact">
@@ -1329,11 +1314,10 @@
   // filtered, sorted, and paged entirely client-side. Mirrors the status site's
   // Lately Inscribed pattern (modules/status-ui/public/status.js renderShippedPage), with
   // an additional version filter since a builder's history spans versions.
-  const SHIPS_PAGE_SIZE = 10;
-  const shipState = {
-    all: [],
-    page: 0,
-  };
+  const shipState = { all: [] };
+  // task 2174: paging moved to the OTBKit pager (per-page select + persistence);
+  // 10 stays the default page size this table has always used.
+  const shipsPager = window.OTBKit.makePager('ships', 'otb.standing.shipsPerPage', { defaultPerPage: 10 });
 
   function filterShips(ships) {
     const win = $('#f-ships-window')?.value || 'all';
@@ -1380,9 +1364,6 @@
     const tbody = $('#recent-ships tbody');
     const countEl = $('#ships-count');
     const pagerEl = $('#ships-pager');
-    const rangeEl = $('#ships-range');
-    const prevBtn = $('#ships-prev');
-    const nextBtn = $('#ships-next');
 
     const filtered = filterShips(shipState.all);
     const mode = $('#f-ships-sort')?.value || 'recent';
@@ -1426,8 +1407,7 @@
       return;
     }
 
-    const { start } = renderPager(shipState, sorted.length, SHIPS_PAGE_SIZE, { pagerEl, rangeEl, prevBtn, nextBtn });
-    const slice = sorted.slice(start, start + SHIPS_PAGE_SIZE);
+    const slice = window.OTBKit.paginate(sorted, shipsPager);
 
     tbody.innerHTML = slice.map((s) => `
       <tr>
@@ -1442,11 +1422,11 @@
     `).join('');
   }
 
-  // Re-render when filters change. Reset to page 0 so the new filter starts at
+  // Re-render when filters change. Reset to page 1 so the new filter starts at
   // the top — otherwise switching to a smaller window can leave you on page 9
   // of a 2-page result.
   function onShipsFilterChange() {
-    shipState.page = 0;
+    shipsPager.page = 1;
     renderRecentShips();
   }
 
@@ -1723,12 +1703,17 @@
   // isn't a peer's work.
 
   let acclaimCanVote = false;
+  // task 1002446: this was the hall's last unpaged list. The endpoint returns the
+  // whole feed in one call, so the rows are held here and the kit's client-side
+  // pager slices them — a page change re-renders without a refetch.
+  const acclaimState = { tasks: [] };
+  const acclaimPager = window.OTBKit.makePager('acclaim', 'otb.standing.acclaimPerPage', { defaultPerPage: 10 });
 
   function renderAcclaimFeed(data) {
     const scroll = $('#acclaim-scroll');
-    const tbody = $('#acclaim-feed tbody');
-    if (!scroll || !tbody) return;
+    if (!scroll) return;
     const tasks = (data && Array.isArray(data.tasks)) ? data.tasks : [];
+    acclaimState.tasks = tasks;
     acclaimCanVote = !!(data && data.can_vote);
     if (tasks.length === 0) {
       // Nothing to acclaim yet (e.g. the only confirmed work so far is your
@@ -1738,7 +1723,18 @@
     }
     // #748: a gated newcomer never shows the acclaim feed.
     scroll.hidden = newcomerGated;
-    tbody.innerHTML = tasks.map((t) => {
+    // task 2174: this feed lands after loadAll built the rail, and it can be the
+    // only visible member of the Karma group (an empty karma feed hides itself).
+    // Re-run renderNav so the group — and its rail entry — appear.
+    if (lastMeState?.builder) renderNav(lastMeState.builder, lastMeState.onboarding || null);
+    renderAcclaimRows();
+  }
+
+  function renderAcclaimRows() {
+    const tbody = $('#acclaim-feed tbody');
+    if (!tbody) return;
+    const slice = window.OTBKit.paginate(acclaimState.tasks, acclaimPager);
+    tbody.innerHTML = slice.map((t) => {
       const by = escapeHtml(t.display_name || (t.github_login ? `@${t.github_login}` : `builder #${t.shipped_by}`));
       const net = Number(t.net_votes) || 0;
       const netCls = signClasses(net, 'acclaim__net');
@@ -1797,6 +1793,12 @@
     buttons.forEach((b) => { b.disabled = true; });
     try {
       const res = await postJSON(`${API}/tasks/${encodeURIComponent(taskId)}/vote`, { vote });
+      // task 1002446: the row is now paged, so the vote has to land in the held
+      // rows too — otherwise flipping to page 2 and back re-renders the stale
+      // server tally over the one just cast.
+      acclaimState.tasks = acclaimState.tasks.map((t) => (Number(t.id) === taskIdNum
+        ? { ...t, net_votes: res.net_votes, my_vote: res.my_vote }
+        : t));
       // Reflect the authoritative net + which button is active.
       const netEl = $(`[data-acclaim-net="${taskId}"]`);
       if (netEl) {
@@ -1823,32 +1825,10 @@
   // the hall's old Claim section + its renderTasks/copy-prompt machinery were
   // removed. work.js carries the equivalent rendering for that page.
 
-  // ---- red-team rewards (bounty table, V3.R65 #284) ------------------------
-
-  // Severity tiers ship low → critical; the server already returns them in that
-  // order (src/bongos/security.js SEVERITIES), but we cap the row classes here so
-  // the colouring is deterministic regardless of payload ordering.
-  function renderBountyTable(rows) {
-    const tbody = $('#bounty-table tbody');
-    if (!tbody) return;
-    if (!Array.isArray(rows) || !rows.length) {
-      tbody.innerHTML = emptyRow(3, 'The reward schedule could not be loaded.');
-      return;
-    }
-    tbody.innerHTML = rows.map((r) => {
-      const sev = escapeHtml(r.severity || '');
-      const overridden = r.overridden
-        ? ` <span style="color:var(--ink-faint);font-size:0.82rem">(tuned)</span>`
-        : '';
-      return `
-        <tr>
-          <td style="text-transform:capitalize;font-weight:600">${sev}</td>
-          <td class="ledger__drachmae">${fmtNum(r.drachmae)}${overridden}</td>
-          <td style="color:var(--ink-soft)">${escapeHtml(r.blurb || '')}</td>
-        </tr>
-      `;
-    }).join('');
-  }
+  // (task 2174: the red-team bounty rate card — V3.R65 #284 — left Standing for
+  // /drachmae. It is a published reward SCHEDULE, i.e. reference material that
+  // reads the same for every builder, not part of anyone's personal standing.
+  // drachmae.js now fetches /public/bounty-table and renders it there.)
 
   // ---- idea-inbox triage cue (task 1471) -----------------------------------
 
@@ -1872,13 +1852,18 @@
         const kind = i.kind
           ? `<span class="triage-inbox__kind">${escapeHtml(i.kind)}</span>`
           : '';
+        // task 1002447: an explicit verb, matching the Accept/Decline affordance
+        // the goal slice puts on its own rows. It opens the record overlay, where
+        // promote/merge/discard live — the forms need more room than a summary
+        // row has, and the placement doctrine keeps this card a summary.
         return `<li class="triage-inbox__item">`
           + `<a class="triage-inbox__link" href="#/idea/${encodeURIComponent(i.id)}">${title}</a>`
           + kind
+          + `<button class="btn-ghost" type="button" data-triage-idea="${escapeHtml(String(i.id))}">Triage</button>`
           + `</li>`;
       }).join('');
       const more = openCount > 5
-        ? `<p class="ginbox__pending">…${fmtNum(openCount - 5)} more open ideas — open any row to triage in place.</p>`
+        ? `<p class="ginbox__pending">…${fmtNum(openCount - 5)} more open ideas — triage clears them one at a time.</p>`
         : '';
       box.innerHTML = `<h3 class="ginbox__sec-h">Ideas awaiting triage <span class="count-pill">${fmtNum(openCount)}</span></h3>`
         + `<ul class="triage-inbox">${rows}</ul>${more}`;
@@ -1886,6 +1871,30 @@
     window.OTBAttention?.refresh();
   }
 
+  // task 1002447: refetch the slice so a verdict advances the queue in place —
+  // the next open ideas move up into the five visible rows instead of the card
+  // sitting stale until a reload. Metic+ only, exactly as the initial load is:
+  // loadAll sets the flag from the /me it already has, so both paths read one
+  // gate and a non-triager never issues an /inbox request at all.
+  let canTriage = false;
+  function loadTriageInbox() {
+    if (!canTriage) return;
+    getJSON(`${API}/inbox`)
+      .then((data) => renderTriageInbox(data))
+      .catch(() => { /* nice-to-have; never break the hall */ });
+  }
+
+  // task-detail.js announces a landed verdict from the record overlay; the row
+  // it acted on is one of ours.
+  window.addEventListener('otb:idea-triaged', loadTriageInbox);
+
+  // The row's explicit verb: open the record overlay, where the three triage
+  // forms live. Delegated so it survives every re-render of the slice.
+  document.addEventListener('click', (e) => {
+    const btn = e.target.closest('[data-triage-idea]');
+    if (btn) window.location.hash = `#/idea/${encodeURIComponent(btn.dataset.triageIdea)}`;
+  });
+
   // ---- Home "Next up" slice (task 2184) -------------------------------------
 
   // Fill the top of the "Your work" card with the first two rows of the
@@ -1950,12 +1959,12 @@
 
   const lbState = {
     rows: [],
-    page: 0,
-    pageSize: 25,   // task 1750: top-100 Roll — 25/page keeps it to a few pages
-
     sortBy: 'drachmae',     // 'rank' | 'drachmae' | 'karma' | 'tokens'
     sortDir: 'desc',        // 'desc' | 'asc'
   };
+  // task 2174: OTBKit pager. 25/page is what task 1750 set for the top-100 Roll;
+  // the per-page select now lets a builder widen it and the choice persists.
+  const lbPager = window.OTBKit.makePager('lb', 'otb.standing.leaderboardPerPage', { defaultPerPage: 25 });
 
   // Stable sort. Server returns rows in drachmae-DESC order, which is also the
   // default view; for other modes (rank, karma, drachmae-ASC), re-sort here.
@@ -2051,9 +2060,9 @@
 
   function renderLeaderboard() {
     const rows = sortedLeaderboardRows();
-    const start = lbState.page * lbState.pageSize;
-    const end = start + lbState.pageSize;
-    const slice = rows.slice(start, end);
+    // paginate() clamps the page, slices, and paints the whole #lb-* bar.
+    const slice = window.OTBKit.paginate(rows, lbPager);
+    const start = (lbPager.page - 1) * lbPager.perPage;
     const tbody = $('#leaderboard tbody');
 
     if (!slice.length) {
@@ -2094,12 +2103,6 @@
         : 'none';
       head.setAttribute('aria-sort', ariaSort);
     }
-
-    $('#lb-range').textContent = rows.length
-      ? `${start + 1}–${Math.min(end, rows.length)} of ${rows.length}`
-      : '0';
-    $('#lb-prev').disabled = lbState.page === 0;
-    $('#lb-next').disabled = end >= rows.length;
   }
 
   // Cycle: pick column → desc → asc → back to default (drachmae desc).
@@ -2120,7 +2123,7 @@
       lbState.sortBy = key;
       lbState.sortDir = 'desc';
     }
-    lbState.page = 0;
+    lbPager.page = 1;
     renderLeaderboard();
   }
 
@@ -2188,6 +2191,9 @@
     lastKarmaState = me.karma || lastKarmaState;
     lastCostState = me.cost || lastCostState;
     lastNeedsState = me.needs || lastNeedsState;
+    // ADR 0120 part 5: owed-rebase list is authoritative on every /me (an empty
+    // array means "nothing owed now"), so take it verbatim when present.
+    if (Array.isArray(me.needs_rebase_tasks)) lastRebaseState = me.needs_rebase_tasks;
     renderOnboarding(me.onboarding || null, me.discord);
     // If they graduated while watching, hide the welcome panel + rebuild the nav
     // (drops the Welcome/Path chips) — the same transition loadAll would render.
@@ -2218,6 +2224,7 @@
     lastKarmaState = me.karma || null;
     lastCostState = me.cost || null;
     lastNeedsState = me.needs || null; // task 1013 / ADR 0073: action-needed banner
+    lastRebaseState = Array.isArray(me.needs_rebase_tasks) ? me.needs_rebase_tasks : null; // ADR 0120 part 5: owed-rebase banner
     // #408: stash the whole /me payload so reopenWelcomeFromNav can re-render
     // the welcome panel without a fresh round-trip.
     lastMeState = me;
@@ -2275,22 +2282,18 @@
     // task 1471: idea-inbox triage cue — Metic+ only (triage is a Metic+ action).
     // Fetched fire-and-forget so a slow/large inbox can't hold up the hall; the
     // widget is hiddenByDefault, so a non-triager never loads or sees idea data.
-    if (['metic', 'archon'].includes(normalizeRank(me.builder?.rank))) {
-      getJSON(`${API}/inbox`)
-        .then((data) => renderTriageInbox(data))
-        .catch(() => { /* nice-to-have; never break the hall */ });
-    }
+    canTriage = ['metic', 'archon'].includes(normalizeRank(me.builder?.rank));
+    loadTriageInbox();
 
     // Then the rest, in parallel. Any individual section failure shouldn't take
     // down the page — log and show an inline placeholder.
     // #726: the per-skill model picker (Voices) moved to /builders/settings,
     // so the hall no longer fetches /me/skill-prefs here.
-    const [achAll, ships, versionsResp, lb, bounty] = await Promise.allSettled([
+    const [achAll, ships, versionsResp, lb] = await Promise.allSettled([
       getJSON(`${API}/achievements`),
       getJSON(`${API}/me/recent-ships?limit=all`),
       getJSON(`${API}/versions`),
       getJSON(`${API}/leaderboard`),
-      getJSON(`${API}/public/bounty-table`),
     ]);
 
     if (achAll.status === 'fulfilled') {
@@ -2323,7 +2326,7 @@
 
     if (lb.status === 'fulfilled') {
       lbState.rows = lb.value.leaderboard || [];
-      lbState.page = 0;
+      lbPager.page = 1;
       renderLeaderboard();
       // #757: the Standing card painted before the Roll loaded, so re-render it
       // now that lbState.rows exists — this is when its "#N on the Roll" stat
@@ -2332,13 +2335,6 @@
     } else {
       $('#leaderboard tbody').innerHTML = emptyRow(5, 'The leaderboard could not be loaded.');
     }
-
-    if (bounty.status === 'fulfilled') {
-      renderBountyTable(bounty.value.table || []);
-    } else {
-      $('#bounty-table tbody').innerHTML = emptyRow(3, 'The reward schedule could not be loaded.');
-    }
-
   }
 
   function wireEvents() {
@@ -2408,22 +2404,11 @@
     $('#f-ships-window')?.addEventListener('change', onShipsFilterChange);
     $('#f-ships-version')?.addEventListener('change', onShipsFilterChange);
     $('#f-ships-sort')?.addEventListener('change', renderRecentShips);
-    $('#ships-prev')?.addEventListener('click', () => {
-      if (shipState.page > 0) { shipState.page--; renderRecentShips(); }
-    });
-    $('#ships-next')?.addEventListener('click', () => {
-      shipState.page++;
-      renderRecentShips();
-    });
-
-
-    $('#lb-prev').addEventListener('click', () => {
-      if (lbState.page > 0) { lbState.page--; renderLeaderboard(); }
-    });
-    $('#lb-next').addEventListener('click', () => {
-      const max = Math.ceil(lbState.rows.length / lbState.pageSize) - 1;
-      if (lbState.page < max) { lbState.page++; renderLeaderboard(); }
-    });
+    // task 2174: the kit wires prev/next/per-page for both tables against the
+    // #ships-*/#lb-* ids its own pagerBarHtml() stamped in hall-widgets.js.
+    window.OTBKit.wirePager(shipsPager, renderRecentShips);
+    window.OTBKit.wirePager(lbPager, renderLeaderboard);
+    window.OTBKit.wirePager(acclaimPager, renderAcclaimRows);
 
     // Sortable leaderboard headers — clickable + keyboard-operable. Each
     // sortable column registers under LB_SORT_HEADERS; the click handler
@@ -2442,28 +2427,11 @@
       });
     }
 
-    // Logout link: the /api/bongos/auth/logout endpoint is POST + requires the
-    // session cookie. Convert the anchor click into a fetch then bounce home.
-    $('#logout-link').addEventListener('click', async (e) => {
-      e.preventDefault();
-      try {
-        await api.request('POST', `${API}/auth/logout`);
-      } catch (_) { /* even if it fails we drop the cookie below */ }
-      // Best-effort cookie clear on the client side too. Mirror the SERVER
-      // logout (src/bongos/routes/auth.js): expire BOTH the current `gds_session`
-      // and the legacy `pms_session`, each in BOTH the host-only form and the
-      // apex-scoped `Domain=.amazonprimea.com` form (#726 — the session cookie
-      // is set apex-scoped on production hosts so it crosses subdomains; an
-      // expire only matches the cookie whose Domain attribute it names, so the
-      // host-only Max-Age=0 alone leaves the apex cookie live). Clearing only
-      // `pms_session` host-only — the prior behaviour — left a real
-      // `gds_session` cookie behind on this offline fallback path.
-      for (const name of ['gds_session', 'pms_session']) {
-        document.cookie = `${name}=; Max-Age=0; path=/`;
-        document.cookie = `${name}=; Max-Age=0; path=/; Domain=.${location.hostname.split('.').slice(-2).join('.')}`;
-      }
-      window.location.assign('/');
-    });
+    // Sign-out is wired centrally in shell.js (doLogout + a delegated click
+    // listener on the sign-out anchors), so it runs the real POST -> cookie-clear
+    // -> bounce-home on EVERY hall page, not only this index. It used to live
+    // here alone, which is exactly why sign-out silently no-op'd everywhere else
+    // (a bare GET to the POST-only endpoint; task 1002245). Nothing to wire here.
   }
 
   // ---- Live updates (#569 / ADR 0030 → R55 / ADR 0069) -------------------

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

@@ -107,6 +107,8 @@
 <!-- The app shell (sidebar + top bar + footer). Dependency-free; runs at parse
      time so the chrome exists before the deferred page scripts execute. -->
 <script src="/builders/shell.js?v=2026-07-02-shell1"></script>
+<!-- jump palette (Ctrl/Cmd-K). Optional by design: without it the shell's slot stays hidden. -->
+<script src="/builders/palette.js" defer></script>
 <script src="/builders/diagram-viewer.js?v=2026-06-14-diagviewer-1" defer></script>
 <script src="/builders/copy-on-click.js?v=2026-06-01-copy-1" defer></script>
 </body>