@bongos/core 1.24.42 → 1.24.44

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

@@ -25,12 +25,11 @@
   const { moduleOn } = window.OTB;
 
   // ---- tiny helpers --------------------------------------------------------
-  // escapeHtml / $ / fmtNum / fmtDate / rankLabel / toast now come from the
-  // shared window.OTB toolkit (dom-utils.js, loaded first). Thin wrappers keep
-  // this file's exact variants (date-only fmtDate; '—' rankLabel empty; a
-  // 2200ms clearing toast). renderRankBadge + RANK_ORDER stay local below.
+  // escapeHtml / $ / fmtNum / rankLabel / toast now come from the shared
+  // window.OTB toolkit (dom-utils.js, loaded first). Thin wrappers keep this
+  // file's exact variants ('—' rankLabel empty; a 2200ms clearing toast).
+  // renderRankBadge + RANK_ORDER stay local below.
   const { escapeHtml, $, fmtNum } = window.OTB;
-  const fmtDate = (iso) => window.OTB.fmtDate(iso);
   const rankLabel = (rank) => window.OTB.rankLabel(rank);
   const toast = (msg) => window.OTB.toast(msg, { duration: 2200, clear: true });
 
@@ -64,16 +63,6 @@
     return res.data;
   }
 
-  async function patchJSON(path, body) {
-    const res = await api.request('PATCH', path, { body });
-    if (res.status === 401) {
-      window.location.assign(`${API}/auth/web/start`);
-      throw new Error('redirecting to auth');
-    }
-    if (!res.ok) throw new Error(`${path} → ${res.status}`);
-    return res.data;
-  }
-
   async function postJSON(path, body) {
     const res = await api.request('POST', path, { body });
     if (res.status === 401) {
@@ -104,27 +93,8 @@
     return res.data;
   }
 
-  // ---- shared render helpers (C10) ----------------------------------------
-
-  // emptyRow — the placeholder/error <tr> every ledger table renders when it has
-  // nothing (or failed) to show. `message` is plain copy (escaped defensively
-  // even though current callers pass static literals).
-  function emptyRow(colspan, message) {
-    return `<tr><td colspan="${colspan}" class="ledger__empty">${escapeHtml(message)}</td></tr>`;
-  }
-
-  // signClasses — the up/down/flat BEM modifier for a signed number. `prefix` is
-  // the block, e.g. signClasses(delta, 'karma-feed__delta') → 'karma-feed__delta--up'.
-  // The +/- TEXT differs per feed (fmtNum vs raw int vs '±0'), so callers format
-  // the displayed value themselves.
-  function signClasses(n, prefix) {
-    const v = Number(n) || 0;
-    return `${prefix}--${v > 0 ? 'up' : v < 0 ? 'down' : 'flat'}`;
-  }
-
-  // (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.)
+  // (task 1005050: emptyRow and signClasses went to profile-own.js with the
+  // ledgers and the karma feed that used them; nothing on Home draws one.)
 
   // ---- repo info (the clone URL the setup screen hands a newcomer) ---
   //
@@ -222,26 +192,12 @@
   // 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), `group` (a Standing rail section, task
-  // 2174) or `view` (home/standing); all default to the Home main
-  // zone so pre-redesign module widgets still land somewhere visible. Every
+  // `zone` (a fine-grained Home slot) or `view` (only 'home' since task 1005050
+  // folded the Standing view into the Profile page); 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.
 
-  // 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.
-
-  // 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.
-
-
   // mountHallWidgets — stamp every registered widget's <section> into its view
   // container (falling back to <main>), in registry order, then render its
   // skeleton via mount(host). Idempotent: a second call (defensive) re-uses an
@@ -257,14 +213,9 @@
   // The budgets themselves become a CI fitness assertion in task 2177.
 
 
-  // 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.
-  // The rail (#standing-rail) has 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.
+  // (The Standing view's section rail lived in renderNav until task 1005050
+  // folded the view into the Profile page.) renderNav keeps its (builder,
+  // onboarding) signature — all call sites pass them.
 
   // #748: newcomer gate. A Xenos sees a deliberately spare hall — their
   // onboarding path and their Standing card — until they graduate to Thetes.
@@ -315,358 +266,9 @@
   // renderDiscordLink / unlinkDiscord (F7, ADR 0033) moved to
   // public-builders/settings.js — these controls now live on /builders/settings.
 
-  async function saveDisciplines() {
-    const selected = Array.from(document.querySelectorAll('.disc-pref'))
-      .filter((el) => el.checked)
-      .map((el) => el.value);
-    const btn = $('#disc-save');
-    if (btn) { btn.disabled = true; btn.textContent = 'Saving…'; }
-    try {
-      const resp = await patchJSON(`${API}/me/disciplines`, { preferred_disciplines: selected });
-      renderProfile(resp.builder);
-      toast(selected.length ? `Preferred work: ${selected.join(', ')}` : 'Preference cleared.');
-    } catch (err) {
-      if (err && err.message === 'redirecting to auth') return;
-      console.error('[builders] save disciplines failed', err);
-      toast('Could not save your preference. Try again.');
-      if (btn) { btn.disabled = false; btn.textContent = 'Save'; }
-    }
-  }
-
-  // ---- achievements --------------------------------------------------------
-
-  function renderAchievements(all, unlocked, lockedWithProgress) {
-    const unlockedById = new Map(unlocked.map((u) => [u.achievement_id, u]));
-    // Map achievement_id → progress_text for locked entries. Falsy/empty
-    // progress just shows the credit-bonus line as before.
-    const progressById = new Map(
-      (lockedWithProgress || []).map((l) => [l.achievement_id, l.progress_text || '']),
-    );
-    const grid = $('#ach-grid');
-    if (!all.length) {
-      grid.innerHTML = `<div class="profile__placeholder">No achievements earned yet.</div>`;
-      return;
-    }
-    grid.innerHTML = all.map((a) => {
-      const got = unlockedById.get(a.id);
-      const cls = got ? 'ach' : 'ach ach--locked';
-      const bonus = a.credit_bonus ? `<p class="ach__bonus">+${fmtNum(a.credit_bonus)} ${(window.OTBBranding?.currencyLabel?.() ?? 'credits')}</p>` : '';
-      const when = got ? `<p class="ach__bonus">Won ${fmtDate(got.unlocked_at)}</p>` : '';
-      const progress = !got && progressById.has(a.id) && progressById.get(a.id)
-        ? `<p class="ach__progress">${escapeHtml(progressById.get(a.id))}</p>`
-        : '';
-      return `
-        <div class="${cls}" title="${escapeHtml(a.id)}">
-          <div class="ach__icon" aria-hidden="true">${escapeHtml(a.icon || '✦')}</div>
-          <div class="ach__body">
-            <p class="ach__name">${escapeHtml(a.name)}</p>
-            <p class="ach__desc">${escapeHtml(a.description || '')}</p>
-            ${when || bonus}
-            ${progress}
-          </div>
-        </div>
-      `;
-    }).join('');
-  }
-
-  // ---- recent ships --------------------------------------------------------
-
-  // All shipped works for this builder, fetched once via ?limit=all then
-  // 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.
-  // task 2174: paging moved to the OTBKit pager (per-page select + persistence);
-  // 10 stays the default page size this table has always used.
-
-
-  // task 1957 — a shipped row's payout is the ACTUAL ledger figure (credits_awarded),
-  // not the creation-time estimate. taskCreditValue picks the right one by status.
-
-
-
-  // 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() {
-    shipsPager.page = 1;
-    renderRecentShips();
-  }
-
-  // ---- recent grades (task 197) -------------------------------------------
-
-  function renderRecentGrades(grades) {
-    const tbody = $('#recent-grades tbody');
-    if (!tbody) return;
-    if (!grades || !grades.length) {
-      tbody.innerHTML = emptyRow(5, 'No work reviewed yet.');
-      return;
-    }
-    tbody.innerHTML = grades.map((g) => {
-      // A grader OUTAGE row (panel_outcome 'unavailable') carries a placeholder
-      // score of 0 — label it, never render it as a 0.0 fail (task 1002652).
-      const isOutage = g.panel_outcome === 'unavailable';
-      // A json-null gating axis (task 1003318 — the Quality worker never ran)
-      // is likewise not a judgment: the score is the surviving axis, not a fail.
-      const isLensOut = !isOutage && !!g.rubric_scores && g.rubric_scores.code_quality === null;
-      const score = isOutage || isLensOut ? '—' : Number(g.score).toFixed(1);
-      // A verdict is verification state: it wears the fixed vocabulary through
-      // .ledger__verdict--* (style.css), never an inline colour. The literals
-      // that stood here painted `passed` from a --olive that is not a token of
-      // the world, so every hall ever rendered it as the fallback (task 1003495).
-      const verdict = isOutage
-        ? `<span class="ledger__verdict--none">not graded (grader outage)</span>`
-        : isLensOut
-          ? `<span class="ledger__verdict--none">not graded (quality lens unavailable)</span>`
-          : g.passed
-          ? `<span class="ledger__verdict--ok">passed</span>`
-          : `<span class="ledger__verdict--danger">fell short</span>`;
-      return `
-        <tr>
-          <td class="ledger__num">#${g.task_id}</td>
-          <td>${escapeHtml(g.title || '')}</td>
-          <td class="ledger__drachmae">${isOutage || isLensOut ? '—' : `${score} / 10`}</td>
-          <td>${verdict}</td>
-          <td class="ledger__date">${fmtDate(g.graded_at)}</td>
-        </tr>
-      `;
-    }).join('');
-  }
-
-  function renderRollingNote(rolling) {
-    const note = $('#grades-rolling-note');
-    if (!note) return;
-    if (!rolling || !rolling.n || Number(rolling.n) === 0) {
-      note.hidden = true;
-      return;
-    }
-    const avg = Number(rolling.avg_score).toFixed(1);
-    const passed = Number(rolling.n_passed || 0);
-    const n = Number(rolling.n);
-    note.textContent = `Last 14 days: ${avg} / 10 average across ${n} graded work${n === 1 ? '' : 's'} (${passed} passed).`;
-    note.hidden = false;
-  }
-
-  // ---- the dev curve (V3.R72 #291, C5) ------------------------------------
-  //
-  // Per-builder historical analytics: ship rate, cost-per-task, duration, and
-  // grade pass-rate over time, plus a top-kinds fingerprint. Charts are inline
-  // SVG sparklines — no chart library (the task spec's explicit constraint).
-  // An Archon additionally gets a "You / All builders" scope toggle backed by
-  // the ecosystem aggregate (individuals stripped), satisfying the
-  // cross-builder view requirement without a per-builder roster on this page.
-
-
-  // Build a tiny SVG line sparkline from an array of numeric y-values. Returns
-  // a self-contained <svg> string. A flat/short series degrades to a baseline.
-
-
-
-  const pct = (v) => (v == null ? '—' : `${Math.round(Number(v) * 100)}%`);
-  const usd = (v) => `$${(Number(v) || 0).toFixed(2)}`;
-  const mins = (v) => (v == null ? '—' : `${Math.round(Number(v))}m`);
-
-  // Render the curve for whichever scope is active. `data` is either the self
-  // payload (/analytics/builder/:id) or the ecosystem payload, normalized below.
-
-  // Fire-and-forget loader: fetch the builder's own curve (and, for an Archon,
-  // the ecosystem aggregate) and render. A failed/empty feed simply leaves the
-  // section hidden rather than holding up the rest of the hall.
-  async function loadAnalytics(builder) {
-    const scroll = $('#analytics-scroll');
-    if (!scroll || !builder?.id) return;
-    analyticsState.builderId = builder.id;
-    analyticsState.scope = 'self';
-    analyticsState.kind = null;
-    try {
-      const self = await getJSON(`${API}/analytics/builder/${builder.id}?days=90`);
-      analyticsState.self = self;
-      // The picker's options come from THIS first, unfiltered load and are then
-      // left alone — see kindFilterHtml. A later filtered payload's top_kinds is
-      // deliberately not allowed to overwrite them.
-      analyticsState.kinds = Array.isArray(self?.top_kinds) ? self.top_kinds : [];
-      // A gated newcomer with nothing shipped: keep the section hidden.
-      if (!self || (self.headline?.total_shipped || 0) === 0) { scroll.hidden = true; return; }
-      scroll.hidden = false;
-      renderCurveBody();
-      renderNav(builder, lastMeState?.onboarding || null); // re-show the Curve chip now that the section is visible
-      // THE BENCHMARK IS FOR EVERY BUILDER, NOT JUST THE ARCHON (task 1004005).
-      // This fetch used to sit behind `if (analyticsState.isArchon)`, so the
-      // ecosystem average a builder is meant to measure themselves against was
-      // fetched for exactly one of them. The route is `rank: public` — no auth
-      // middleware, CORS '*', per-builder identity stripped server-side — and the
-      // unauthenticated status page already reads it, so the gate protected
-      // nothing and only withheld the comparison. Still fire-and-forget: the
-      // ecosystem feed is optional context, never a reason to hold up the curve.
-      getJSON(`${API}/public/analytics/ecosystem?days=90`)
-        // ALWAYS re-render, not only in the 'eco' scope. That condition was
-        // correct while the two scopes merely SWAPPED which single series was
-        // drawn; now the ecosystem series is also the benchmark overlaid on the
-        // 'self' view, so a self-scoped reader who arrived before this resolved
-        // would sit with no overlay until they touched an unrelated control.
-        .then((eco) => { analyticsState.eco = eco; renderCurveBody(); })
-        .catch(() => { /* ecosystem optional */ });
-    } catch (err) {
-      console.error('[builders] analytics load failed', err);
-      scroll.hidden = true;
-    }
-  }
-
-  // Re-fetch BOTH series under a kind filter (task 1004005). Both, always: an
-  // overlay that narrowed one side and not the other would compare one kind of
-  // work against all work and still look like a like-for-like benchmark — the
-  // failure this task's own text calls worse than no benchmark at all.
-  //
-  // `kinds` is deliberately not refreshed here; see kindFilterHtml.
-  let curveKindSeq = 0;
-  async function reloadCurveForKind(kind) {
-    const id = analyticsState.builderId;
-    if (!id) return;
-    analyticsState.kind = kind;
-    // A dropdown can be changed faster than a round trip returns, and the two
-    // fetches can land out of order. Only the newest request may write state.
-    const seq = ++curveKindSeq;
-    const q = kind ? `&kind=${encodeURIComponent(kind)}` : '';
-    renderCurveBody(); // reflect the new selection immediately
-    try {
-      const [self, eco] = await Promise.all([
-        getJSON(`${API}/analytics/builder/${id}?days=90${q}`),
-        getJSON(`${API}/public/analytics/ecosystem?days=90${q}`).catch(() => null),
-      ]);
-      if (seq !== curveKindSeq) return;
-      analyticsState.self = self;
-      if (eco) analyticsState.eco = eco;
-      renderCurveBody();
-    } catch (err) {
-      if (seq !== curveKindSeq) return;
-      console.error('[builders] curve kind filter failed', err);
-    }
-  }
-
-  // ---- karma feed (V3.R26 #246) -------------------------------------------
-  //
-  // The recent karma_log entries for the current builder. The /me payload
-  // already includes them under `karma.recent`. The scroll stays hidden when
-  // the builder has no history yet (zero karma is a common starting state,
-  // so we don't want to push an empty table at every new citizen).
-
-  function renderKarmaFeed(entries) {
-    const scroll = $('#karma-scroll');
-    const tbody = $('#karma-feed tbody');
-    if (!scroll || !tbody) return;
-    if (!Array.isArray(entries) || entries.length === 0) {
-      // Hide the scroll entirely if the builder has no karma history.
-      // Keeps the page tidy for fresh citizens; reappears the moment a
-      // karma_log row lands.
-      scroll.hidden = true;
-      return;
-    }
-    // #748: a gated newcomer never shows the karma feed, even if karma exists.
-    scroll.hidden = newcomerGated;
-    tbody.innerHTML = entries.map((k) => {
-      const delta = Number(k.delta) || 0;
-      const deltaCls = signClasses(delta, 'karma-feed__delta');
-      const deltaText = delta > 0 ? `+${fmtNum(delta)}` : fmtNum(delta);
-      const granted = k.granted_by
-        ? `@${escapeHtml(k.granted_by_login || k.granted_by_display_name || `builder #${k.granted_by}`)}`
-        : '<span style="opacity:0.6;">— all builders —</span>';
-      return `
-        <tr>
-          <td class="ledger__date">${escapeHtml(fmtDate(k.recorded_at))}</td>
-          <td class="karma-feed__delta ${deltaCls}">${deltaText}</td>
-          <td>${escapeHtml(k.reason || '')}</td>
-          <td>${granted}</td>
-        </tr>
-      `;
-    }).join('');
-  }
-
-  // ---- peer acclaim feed (V3.R25 #245) ------------------------------------
-  //
-  // Other builders' recently confirmed/shipped work, each with a +/- vote
-  // widget. Votes fold into the shipping builder's karma on a weekly tally,
-  // lagged 7 days (server-enforced). Metic+ may vote; the endpoint returns
-  // can_vote so Xenos see the tallies read-only. The endpoint already excludes
-  // the caller's own ships (anti-self-vote), so there's nothing to vote on that
-  // 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');
-    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
-      // own). Hide rather than show an empty ledger.
-      scroll.hidden = true;
-      return;
-    }
-    // #748: a gated newcomer never shows the acclaim feed.
-    scroll.hidden = newcomerGated;
-    // 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) => {
-      // task 1004432: an approved page is its artist's; the applying session rides along as a helper.
-      const helperName = t.helper_display_name || (t.helper_login ? `@${t.helper_login}` : '');
-      const by = escapeHtml(t.display_name || (t.github_login ? `@${t.github_login}` : `builder #${t.shipped_by}`)) + (helperName ? ` <span class="acclaim__evidence">with help from ${escapeHtml(helperName)}</span>` : '');
-      // task 1004833: the votes are THE shared list control (vote-control.js),
-      // pre-painted from the feed's own net_votes/my_vote so it costs no extra
-      // read. A viewer below the vote floor sees the net alone.
-      const net = Number(t.net_votes) || 0;
-      const votes = acclaimCanVote && window.OTBVote
-        ? window.OTBVote.taskHtml(t)
-        : `<span class="vote__net" title="Metic rank and above may vote">${net > 0 ? `+${net}` : `${net}`}</span>`;
-      const title = escapeHtml(t.value_summary || t.title || `task #${t.id}`);
-      return `
-        <tr>
-          <td>${title}${acclaimEvidenceHtml(t)}</td>
-          <td>${by}</td>
-          <td>${votes}</td>
-        </tr>
-      `;
-    }).join('');
-  }
-
-  async function loadAcclaimFeed() {
-    try {
-      const data = await getJSON(`${API}/tasks/peer-votes/feed?limit=25`);
-      renderAcclaimFeed(data);
-    } catch (err) {
-      // Non-fatal: a feed failure shouldn't take down the hall.
-      console.warn('[hall] acclaim feed failed:', err.message);
-      const scroll = $('#acclaim-scroll');
-      if (scroll) scroll.hidden = true;
-    }
-  }
-
-  // task 1004833: the shared vote control casts the vote and repaints its own
-  // row; it then raises 'otb-vote' with the server's answer. The feed is paged,
-  // so the vote must land in the HELD rows too (task 1002446), or flipping to
-  // page 2 and back re-renders the stale tally over the one just cast.
-  function onAcclaimVote(ev) {
-    const d = ev && ev.detail;
-    if (!d || d.kind !== 'task') return;
-    const taskIdNum = Number(d.id);
-    acclaimState.tasks = acclaimState.tasks.map((t) => (Number(t.id) === taskIdNum
-      ? { ...t, net_votes: d.net_votes, my_vote: d.my_vote }
-      : t));
-  }
+  // (task 1005050: the achievements grid, the ship ledger, reviews, the dev curve
+  // and the karma and peer-recognition feeds were the Accomplishments view, which
+  // folded into the Profile page: profile-own.js draws the private half there.)
 
   // ---- task browser (removed #750) -----------------------------------------
   // The claimable-works list now lives solely on the /work Work Board (#748);
@@ -799,20 +401,6 @@
     // #731: the CLI-token re-issue card (V3.R91 #401) and the Discord connect/
     // link card (F7, ADR 0033) moved to /builders/settings — the hall's Standing
     // card is identity-only now.
-    renderRecentGrades(me.grades?.recent || []);
-    renderRollingNote(me.grades?.rolling || null);
-    // V3.R72 (#291, C5): the dev curve. Loaded fire-and-forget — a slow or
-    // empty analytics feed must not hold up the rest of the hall, and the
-    // section un-hides itself (and re-renders the nav) only when there's data.
-    loadAnalytics(me.builder);
-    // V3.R26 (#246): karma feed — the recent karma_log entries powering the
-    // "why did my karma move" answer. The /me payload bundles this so the
-    // page doesn't need a follow-up call.
-    renderKarmaFeed(me.karma?.recent || []);
-    // V3.R25 (#245): peer acclaim feed — a separate endpoint (not bundled in
-    // /me because it spans all builders' work), loaded fire-and-forget so a
-    // slow/failed feed can't hold up the rest of the hall.
-    loadAcclaimFeed();
     // #408: build the sticky nav now that every section has settled its
     // hidden state (karma → security/roster).
     // The renderNav call also wires the IntersectionObserver for active-
@@ -832,41 +420,15 @@
     // 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] = await Promise.allSettled([
-      getJSON(`${API}/achievements`),
-      // task 1004854: Home's newest few + a count — never the whole history,
-      // which only the Accomplishments ledger fetches (hall-render.js loadShipLedger).
+    const [ships, lb] = await Promise.allSettled([
+      // task 1004854: Home's newest few + a count — never the whole history
+      // (the Profile page's Works ledger reads that, since task 1005050).
       getJSON(`${API}/me/recent-ships?limit=${HOME_SHIPS_LIMIT}`),
-      getJSON(`${API}/versions`),
       getJSON(`${API}/public/leaderboard`), // task 1004437: the list with each builder's roles
     ]);
 
-    if (achAll.status === 'fulfilled') {
-      // Show top 10 by sort_order — already ordered server-side. (Set is locked at 8
-      // for V2, so the slice is defensive.) Unlocked + locked-with-progress come
-      // straight from the enriched /me payload.
-      renderAchievements(
-        (achAll.value.achievements || []).slice(0, 10),
-        me.unlocked || [],
-        me.locked_with_progress || [],
-      );
-    } else {
-      $('#ach-grid').innerHTML = `<div class="profile__placeholder">Achievements could not be loaded.</div>`;
-    }
-
-    if (ships.status === 'fulfilled') {
-      setRecentShips(ships.value);
-    } else {
-      $('#recent-ships tbody').innerHTML = emptyRow(4, 'Your work history could not be loaded.');
-    }
-
-    if (versionsResp.status === 'fulfilled') {
-      const versions = versionsResp.value.versions || [];
-      // Populate the Works Wrought version filter — limited to versions this
-      // builder has actually shipped to, so the dropdown can't offer empty
-      // options. (The old claimable-works version filter moved to /work, #750.)
-      populateShipVersionFilter(versions);
-    }
+    // Only the Standing chip's shipped count reads this; a failure leaves its dash.
+    if (ships.status === 'fulfilled') setRecentShips(ships.value);
 
     if (lb.status === 'fulfilled') {
       lbState.rows = lb.value.leaderboard || [];
@@ -883,51 +445,6 @@
   }
 
   function wireEvents() {
-    // #profile-body is re-rendered after load, so delegate the save click.
-    $('#profile-body').addEventListener('click', (e) => {
-      if (e.target.closest('#disc-save')) saveDisciplines();
-    });
-
-    // V3.R72 (#291): the dev-curve scope toggle. Delegated because
-    // renderCurveBody replaces the toggle's innerHTML — and from the whole
-    // Progress section, NOT #curve-body: the controls render into
-    // #curve-scope-note in the section's lede, a sibling of #curve-body, so a
-    // listener there never heard them (task 1004079).
-    $('#analytics-scroll')?.addEventListener('click', (e) => {
-      const btn = e.target.closest('.curve__scope-btn[data-scope]');
-      if (!btn) return;
-      const scope = btn.dataset.scope === 'eco' ? 'eco' : 'self';
-      if (scope === analyticsState.scope) return;
-      analyticsState.scope = scope;
-      renderCurveBody();
-    });
-
-    // The kind filter (task 1004005). Delegated like the scope toggle, and for
-    // the same reason: renderCurveBody replaces the control's innerHTML, so a
-    // listener bound to the element itself would not survive the first render.
-    // 'change', not 'input' — a keyboard user arrowing through a native select
-    // fires input per option, and each one would be a pair of round trips.
-    $('#analytics-scroll')?.addEventListener('change', (e) => {
-      const sel = e.target.closest('#curve-kind');
-      if (!sel) return;
-      const kind = sel.value ? sel.value : null;
-      if (kind === analyticsState.kind) return;
-      reloadCurveForKind(kind);
-    });
-
-    // task 1004833: the shared vote control reports each landed vote to the feed.
-    $('#acclaim-feed')?.addEventListener('otb-vote', onAcclaimVote);
-
-    $('#f-ships-window')?.addEventListener('change', onShipsFilterChange);
-    $('#f-ships-version')?.addEventListener('change', onShipsFilterChange);
-    $('#f-ships-sort')?.addEventListener('change', renderRecentShips);
-    // 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(acclaimPager, renderAcclaimRows);
-
-
     // 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
@@ -1032,8 +549,8 @@
   // reads what this page holds NOW rather than what it held at boot. Everything
   // else is stable for the life of the page. This call sits AFTER every
   // declaration it passes — moving it earlier is a temporal-dead-zone crash.
-  const { acclaimEvidenceHtml, analyticsState, lbState, mountHallWidgets, renderCurveBody, renderGreeting, renderLanding, renderSessionRefused, renderNav, renderProfile, renderRecentShips, HOME_SHIPS_LIMIT, populateShipVersionFilter, setRecentShips, shipsPager } = window.OTBHallRender({
-    API, RANK_ORDER, api, brandWorld, curLabel, emptyRow, fmtDate, getJSON, mins, normalizeRank, pct, postJSON, rankLabel, rankWeight, signClasses, toast, usd,
+  const { lbState, mountHallWidgets, renderGreeting, renderLanding, renderSessionRefused, renderNav, renderProfile, HOME_SHIPS_LIMIT, setRecentShips } = window.OTBHallRender({
+    API, RANK_ORDER, api, brandWorld, curLabel, normalizeRank, postJSON, rankLabel, rankWeight, toast,
     get lastCostState() { return lastCostState; },
     get lastKarmaState() { return lastKarmaState; },
     get lastMeState() { return lastMeState; },

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

@@ -105,10 +105,10 @@
   </div>
 </div>
 
-<script src="/builders/dom-utils.js?v=2026-10-25-escape"></script>
-<script src="/builders/shell-trail.js?v=2026-10-08-city-tab" defer></script>
-<script src="/builders/craft-mode.js?v=2026-11-05-oversight" defer></script>
-<script src="/builders/shell.js?v=2026-11-05-setup-gate" defer></script>
+<script src="/builders/dom-utils.js?v=2026-11-05-numbers"></script>
+<script src="/builders/shell-trail.js?v=2026-11-05-numbers" defer></script>
+<script src="/builders/craft-mode.js?v=2026-11-05-goals" defer></script>
+<script src="/builders/shell.js?v=2026-11-05-goals" defer></script>
 <script src="/builders/palette.js" defer></script>
 <script src="/builders/gov-living.js?v=2026-10-09-positions1" defer></script>
 <script src="/builders/gov-feeds.js?v=2026-10-09-living3" defer></script>

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

@@ -159,10 +159,10 @@
 
 <div id="toast" class="toast" role="status" aria-live="polite" data-visible="0"></div>
 
-<script src="/builders/dom-utils.js?v=2026-10-25-escape"></script>
-<script src="/builders/shell-trail.js?v=2026-10-08-city-tab"></script>
-<script src="/builders/craft-mode.js?v=2026-11-05-oversight"></script>
-<script src="/builders/shell.js?v=2026-11-05-setup-gate"></script>
+<script src="/builders/dom-utils.js?v=2026-11-05-numbers"></script>
+<script src="/builders/shell-trail.js?v=2026-11-05-numbers"></script>
+<script src="/builders/craft-mode.js?v=2026-11-05-goals"></script>
+<script src="/builders/shell.js?v=2026-11-05-goals"></script>
 <script src="/builders/palette.js" defer></script>
 <script src="/builders/collab-lib.js?v=2026-09-24-archive"></script>
 <script src="/builders/collab.js?v=2026-09-24-archive"></script>

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

@@ -51,10 +51,10 @@
 
 <div id="toast" class="toast" role="status" aria-live="polite" data-visible="0"></div>
 
-<script src="/builders/dom-utils.js?v=2026-10-25-escape"></script>
-<script src="/builders/shell-trail.js?v=2026-10-08-city-tab"></script>
-<script src="/builders/craft-mode.js?v=2026-11-05-oversight"></script>
-<script src="/builders/shell.js?v=2026-11-05-setup-gate"></script>
+<script src="/builders/dom-utils.js?v=2026-11-05-numbers"></script>
+<script src="/builders/shell-trail.js?v=2026-11-05-numbers"></script>
+<script src="/builders/craft-mode.js?v=2026-11-05-goals"></script>
+<script src="/builders/shell.js?v=2026-11-05-goals"></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/hall-kit.js"></script>