@bongos/core 1.24.27 → 1.24.29

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

@@ -1,12 +1,15 @@
 // public-builders/settings.js — the personal Settings page (/builders/settings).
 //
 // Open to ANY authenticated builder (no rank gate — a builder owns their own
-// preferences; the server enforces the same posture on /me/skill-prefs and
-// /me/disciplines). Two sections:
+// preferences; the server enforces the same posture on /me/disciplines). Among
+// its sections:
 //
-//   1. VOICES  — per-skill model picker (GET/PATCH /api/bongos/me/skill-prefs).
-//   2. CRAFT   — preferred disciplines (GET /api/bongos/me, PATCH /api/bongos/me/disciplines).
-//   5. SOUNDS  — per-sound on/off toggles (GET/PATCH /api/bongos/me/sound-prefs, #786).
+//   CRAFT   — preferred disciplines (GET /api/bongos/me, PATCH /api/bongos/me/disciplines).
+//   SOUNDS  — per-sound on/off toggles (GET/PATCH /api/bongos/me/sound-prefs, #786).
+//
+// The agent settings (how Claude talks to you, wandering, session cards, skill
+// voices, specialities) are NOT here since task 1005044: My agents is their one
+// page, and settings-moved.js sends their old deep links there.
 //
 // On a 401 it redirects into the web sign-in flow, like the other hall pages.
 // Each section loads behind its own try/catch so one failing endpoint doesn't
@@ -32,102 +35,6 @@
     window.location.assign(`${API}/auth/web/start`);
   }
 
-  // ---- model labels (match the hall's wording in builders.js) --------------
-
-  const MODEL_LABELS = {
-    opus: 'Opus',
-    sonnet: 'Sonnet',
-    haiku: 'Haiku',
-    script: 'no LLM',
-  };
-  const modelLabel = (m) => MODEL_LABELS[m] || m || '—';
-
-  // ---- section 1: VOICES (per-skill model picker) --------------------------
-
-  // Render one row per skill in `effective`. Each row: skill name, the system
-  // default (muted), and a <select> whose value is the builder's override (or
-  // "" for "fall back to default"). The allowed model set is echoed back by the
-  // endpoint so we don't mirror it from a second source.
-  function renderVoices(data) {
-    const body = $('#voices-body');
-    if (!body) return;
-    const rows = Array.isArray(data?.effective) ? data.effective : [];
-    const allowed = Array.isArray(data?.allowed_models)
-      ? data.allowed_models
-      : ['opus', 'sonnet', 'haiku', 'script'];
-    if (rows.length === 0) {
-      body.innerHTML = `<tr><td colspan="3">${window.OTBKit.emptyStateHtml('No skills registered.')}</td></tr>`;
-      return;
-    }
-    body.innerHTML = rows.map((row) => {
-      const opts = [`<option value="">Default (${escapeHtml(modelLabel(row.default_model))})</option>`]
-        .concat(allowed.map((m) => {
-          const sel = row.override_model === m ? ' selected' : '';
-          return `<option value="${escapeHtml(m)}"${sel}>${escapeHtml(modelLabel(m))}</option>`;
-        }))
-        .join('');
-      return (
-        `<tr data-skill="${escapeHtml(row.skill)}">` +
-        `<td class="voices__skill"><code>${escapeHtml(row.skill)}</code></td>` +
-        `<td><span class="voices__default" title="System default for this skill">${escapeHtml(modelLabel(row.default_model))}</span></td>` +
-        '<td>' +
-        `<select class="voices__select" data-skill="${escapeHtml(row.skill)}" aria-label="Voice for ${escapeHtml(row.skill)}">${opts}</select>` +
-        '</td>' +
-        '</tr>'
-      );
-    }).join('');
-  }
-
-  // PATCH a single skill's override. Body shape (see me.js + skill-prefs.js):
-  //   { "<skill-name>": "<model>" | null }
-  // where null clears the override (fall back to the system default).
-  async function saveVoice(skill, modelOrNull) {
-    const status = $('#voices-status');
-    if (status) status.textContent = 'Saving…';
-    try {
-      const body = { [skill]: modelOrNull };
-      const res = await api.request('PATCH', `${API}/me/skill-prefs`, { body });
-      if (res.status === 401) { toAuth(); return; }
-      if (!res.ok) throw new Error(`skill-prefs PATCH → ${res.status}`);
-      const data = res.data;
-      renderVoices(data);
-      const msg = modelOrNull
-        ? `${skill} → ${modelLabel(modelOrNull)}.`
-        : `Cleared ${skill} — falls back to default.`;
-      if (status) status.textContent = msg;
-      toast(msg);
-    } catch (err) {
-      console.error('[settings] save voice failed', err);
-      if (status) status.textContent = 'Could not save. Try again.';
-      toast('Could not save your choice. Try again.');
-    }
-  }
-
-  async function loadVoices() {
-    try {
-      const res = await api.request('GET', `${API}/me/skill-prefs`);
-      if (res.status === 401) { toAuth(); return; }
-      if (!res.ok) throw new Error(`skill-prefs GET → ${res.status}`);
-      const data = res.data;
-      renderVoices(data);
-      $('#voices-scroll').hidden = false;
-      // Delegate change events so post-render selects stay wired.
-      const table = $('#voices-table');
-      table.addEventListener('change', (e) => {
-        const sel = e.target.closest('.voices__select');
-        if (!sel) return;
-        const skill = sel.dataset.skill;
-        const next = sel.value === '' ? null : sel.value;
-        saveVoice(skill, next);
-      });
-    } catch (err) {
-      console.error('[settings] failed to load voices', err);
-      $('#voices-scroll').hidden = false;
-      $('#voices-body').innerHTML =
-        '<tr><td colspan="3" class="voices-empty">The skill list could not be loaded. Reload the page to try again.</td></tr>';
-    }
-  }
-
   // ---- section 5: SOUNDS (per-sound on/off toggles, #786) ------------------
 
   // Render one row per sound from `effective`. Each row: label + description and
@@ -392,104 +299,6 @@
     }
   }
 
-  // ---- section 2b: WANDERING (off-task thread suppression, task 1268) -------
-  //
-  // GET/PATCH /api/bongos/me/wandering. The payload (wandering-prefs.describeForApi)
-  // carries the level catalog with per-rank `allowed`, the effective level, the
-  // rank default, and the ceiling. Single-select: clicking an allowed level
-  // PATCHes { wandering: n }. Levels above the rank cap render disabled — the
-  // server enforces the same clamp, this is just the friendly UI half. Reuses
-  // the .sound-row / .sound-toggle styling.
-  function renderWandering(data) {
-    const wrap = $('#wander-rows');
-    if (!wrap) return;
-    const levels = Array.isArray(data?.levels) ? data.levels : [];
-    const eff = typeof data?.effective_level === 'number' ? data.effective_level : null;
-    const def = typeof data?.default_level === 'number' ? data.default_level : null;
-    if (levels.length === 0) {
-      wrap.innerHTML = window.OTBKit.emptyStateHtml('No levels registered.');
-      return;
-    }
-    // One pick among four: a radio row per level (panel.css `.prow--pick`), the
-    // whole row its label. A level above the rank ceiling is a disabled radio
-    // that says so in the label voice rather than a dead "Rank-locked" button.
-    // a project law may fix or cap the level too (task 1004724, settings-law-locks.js)
-    const Laws = window.OTBSettingsLaws || null;
-    const lawLock = Laws ? Laws.lockFor(data, 'wandering') : null;
-    wrap.innerHTML = levels.map((lv) => {
-      const chosen = lv.n === eff;
-      const byLaw = lv.allowed && !!lawLock && !Laws.allows(lawLock, lv.key);
-      const locked = !lv.allowed || byLaw;
-      const id = `wander-${lv.n}`;
-      const tags = (lv.n === def ? '<span class="prow__tag">default for your rank</span>' : '') +
-        (!lv.allowed ? '<span class="prow__tag">rank-locked</span>' : '') +
-        (byLaw ? `<span class="prow__tag">${escapeHtml(Laws.tagFor(lawLock))}</span>` : '');
-      return (
-        `<label class="prow prow--pick" for="${id}" data-level="${lv.n}"${locked ? ' aria-disabled="true"' : ''}>` +
-        `<input class="pradio" type="radio" name="wandering" id="${id}" value="${lv.n}"` +
-        `${chosen ? ' checked' : ''}${locked ? ' disabled' : ''}>` +
-        '<span class="prow__main">' +
-        `<span class="prow__label">${escapeHtml(lv.label)}${tags}</span>` +
-        `<span class="prow__desc">${escapeHtml(lv.blurb || '')}</span>` +
-        '</span></label>'
-      );
-    }).join('');
-    const note = $('#wandering-note');
-    if (note) {
-      note.textContent = ((data && data.effective_label && data.max_key)
-        ? `Now: ${data.effective_label}. Your rank allows up to ${data.max_key}.`
-        : '') + (lawLock ? ' ' + Laws.lawLine(lawLock) : '');
-    }
-  }
-
-  // PATCH the wandering level. Body: { wandering: <0..3> } (see me.js +
-  // wandering-prefs.js). A level above the rank ceiling is rejected server-side
-  // (422) — but those buttons render disabled, so it shouldn't be reachable.
-  async function saveWandering(level, btn) {
-    const status = $('#wandering-status');
-    if (status) status.textContent = 'Saving…';
-    if (btn) btn.disabled = true;
-    try {
-      const res = await api.request('PATCH', `${API}/me/wandering`, { body: { wandering: level } });
-      if (res.status === 401) { toAuth(); return; }
-      if (!res.ok) throw new Error(`wandering PATCH → ${res.status}`);
-      const data = res.data;
-      renderWandering(data);
-      const msg = `Wandering → ${data.effective_label || level}.`;
-      if (status) status.textContent = msg;
-      toast(msg);
-    } catch (err) {
-      console.error('[settings] save wandering failed', err);
-      if (status) status.textContent = 'Could not save. Try again.';
-      toast('Could not save your wandering level. Try again.');
-      if (btn) btn.disabled = false;
-    }
-  }
-
-  async function loadWandering() {
-    try {
-      const res = await api.request('GET', `${API}/me/wandering`);
-      if (res.status === 401) { toAuth(); return; }
-      if (!res.ok) throw new Error(`wandering GET → ${res.status}`);
-      const data = res.data;
-      renderWandering(data);
-      $('#wandering-scroll').hidden = false;
-      // Delegate the change so post-render radios stay wired (single-select).
-      const wrap = $('#wander-rows');
-      wrap.addEventListener('change', (e) => {
-        const r = e.target.closest('.pradio');
-        if (!r || r.disabled || !r.checked) return; // ignore locked; a radio only fires on becoming checked
-        const level = Number(r.value);
-        if (Number.isInteger(level)) saveWandering(level, r);
-      });
-    } catch (err) {
-      console.error('[settings] failed to load wandering', err);
-      $('#wandering-scroll').hidden = false;
-      $('#wander-rows').innerHTML =
-        '<p class="voices-empty">Your wandering level could not be loaded. Reload the page to try again.</p>';
-    }
-  }
-
   // ---- section: PROFILE AND PRIVACY — a link out to the hub (task 1004259)
   //
   // Handle, bio, links, public/private, recruiters and the hide-my-numbers switch
@@ -1100,7 +909,7 @@
   const RAIL_GROUPS = RAIL_ITEMS.map((i) => i.id).concat('extensions');
   let rail = null;
 
-  // task 2172 — stable per-panel anchors (/settings#cli, /settings#voices, …)
+  // task 2172 — stable per-panel anchors (/settings#cli, /settings#sounds, …)
   // so skills and docs can deep-link a panel without depending on internal
   // section ids. #art-key predates the map and IS its section id (the hall's
   // "Add your key" action + the CLI nudge target it) — kept verbatim.
@@ -1117,10 +926,6 @@
     'cli': 'cli-scroll',
     'autobongos': 'autobongos-scroll',
     'art-key': 'art-key',
-    'interaction': 'interaction-scroll',
-    'wandering': 'wandering-scroll',
-    'rendering': 'render-scroll',
-    'voices': 'voices-scroll',
     'sounds': 'sounds-scroll',
     'event-sounds': 'event-sounds-scroll',
     'software-update': 'software-update-scroll',
@@ -1162,6 +967,14 @@
     }
   }
 
+  // task 1005044 — a link to an agent setting that moved to My agents
+  // (settings-moved.js) goes on there; replace(), so Back does not bounce here.
+  function leaveIfMoved() {
+    const to = window.OTBSettingsMoved && window.OTBSettingsMoved.movedTarget(window.location.hash);
+    if (to) window.location.replace(to);
+    return !!to;
+  }
+
   function wireRail() {
     const mount = document.getElementById('settings-rail');
     if (!mount || !window.OTBKit) return;
@@ -1178,7 +991,7 @@
     });
     rail.activate('account');
     applyHash({ scroll: false });
-    window.addEventListener('hashchange', () => applyHash());
+    window.addEventListener('hashchange', () => { if (!leaveIfMoved()) applyHash(); });
   }
 
   // Hide the rail + groups entirely while the not-signed-in gate is showing —
@@ -1251,21 +1064,12 @@
       rail.activate($('#settings-groups')?.dataset.active || 'account');
     }
 
-    // Voices loads independently — its own endpoint + its own try/catch.
-    await loadVoices();
     // Sounds are switched off for now (task 1004554 — .claude/sounds/play.js's
     // SOUNDS_ON). The per-sound toggles (#786) and the per-event picker (task 582)
     // would only offer choices that do nothing, so neither loads; the Sound group
     // shows the one-line note instead. To restore: call loadSounds() and
     // loadEventSounds() here again and drop the note.
     $('#sounds-off-scroll').hidden = false;
-    // (The interaction profile panel self-boots from settings-interaction.js —
-    // its own script on the settings-sessions.js precedent, task 1002514.)
-    // Wandering loads independently too (off-task thread suppression, task 1268).
-    await loadWandering();
-    // (Rendering — the card-richness ladder — is settings-render.js, on the same
-    // precedent: it became a four-rung picker in task 1002518 and this file is
-    // against the 1500-line ratchet.)
     // Profile and privacy — one row linking out to the hub's account page
     // (task 1004259); hidden where the page carries no hub origin.
     loadAccountHub();
@@ -1273,10 +1077,11 @@
     // the `needs` art_key item that drives the covered/action-needed banner.
     await loadArtKey(me);
     // Every section has now loaded/revealed — land any panel-anchor deep link
-    // (/settings#cli, #voices, …) on its visible section.
+    // (/settings#cli, #art-key, …) on its visible section.
     applyHash();
   }
 
+  if (leaveIfMoved()) return;
   wireRail();
   load();
 })();

package/modules/hall-ui/public/settings.states.json

@@ -9,11 +9,9 @@
   "states": {
     "account": { "auth": true, "actions": [["wait", 900]], "expect": { "visible": ["#settings-h1", "#display-name-input", "#craft-chips"] } },
     "access": { "auth": true, "url": "/settings#access", "actions": [["wait", 900]], "expect": { "visible": ["#cli-reissue-btn", "#sessions-list", "#artkey-card"] } },
-    "preferences": { "auth": true, "url": "/settings#preferences", "actions": [["wait", 900]], "expect": { "visible": ["#wander-rows", "#render-rows", "#specialities-rows"] } },
-    "skills-walkthrough": { "auth": true, "url": "/settings#preferences", "actions": [["wait", 900], ["click", "#specialities-rows button[data-act=\"adopt\"]"], ["wait", 700]], "expect": { "visible": [".skill-walk", ".skill-walk__facts", "button[data-walk=\"on\"]", "button[data-walk=\"off\"]"] } },
-    "offered-skills": { "auth": true, "url": "/settings#preferences", "actions": [["wait", 900], ["click", "#specialities-rows button[data-act=\"offer\"]"], ["wait", 700]], "expect": { "visible": ["form.speciality-offer", ".speciality-offer__list", ".speciality-offer__filter"] } },
+    "preferences": { "auth": true, "url": "/settings#preferences", "actions": [["wait", 900]], "expect": { "visible": ["#agents-moved-preferences", "#agents-moved-preferences a[href=\"/my-agents#talk\"]", "#usage-reports-h"] } },
     "install-reports": { "_": "The Module install reports switch (task 1003793): the opt-out for counting your store-module installs in a module's score, on by default. Scrolled into view by focusing its default choice.", "auth": true, "url": "/settings#preferences", "actions": [["wait", 900], ["clickIfVisible", "[data-craft-close]"], ["wait", 300], ["focus", "#usage-reports-on"], ["wait", 300]], "expect": { "visible": ["#usage-reports-h", "#usage-reports-on", "#usage-reports-off"] } },
-    "sound": { "auth": true, "url": "/settings#sound", "actions": [["wait", 900]], "expect": { "visible": ["#voices-table", "#sound-rows", "#event-sound-rows"] } },
+    "sound": { "auth": true, "url": "/settings#sound", "actions": [["wait", 900]], "expect": { "visible": ["#agents-moved-sound", "#agents-moved-sound a[href=\"/my-agents#skills\"]", "#sounds-off-h"] } },
     "update": { "auth": true, "url": "/settings#software-update", "actions": [["wait", 1200]], "expect": { "visible": ["#software-update-h", "#su-body", "#su-update", ".su-versions"] } }
   },
   "reducedMotion": { "auth": true, "actions": [["wait", 900]] }

package/modules/hall-ui/public/speciality-tools.css

@@ -0,0 +1,140 @@
+/* speciality-tools.css — the SPECIALITY TOOLS sheet (task 1005052): the structural
+   rules of the one component Settings and My agents both mount
+   (speciality-tools.js) — the card, the train, offer and create forms, the
+   instructions block and the skills walkthrough. Moved out of settings.css
+   verbatim, so Settings draws exactly as it did; each page sheet adds its own
+   look on top (Settings: panel.css, linked before this; My agents: the `.ma-know`
+   skin in my-agents.css, linked after). */
+
+/* Specialities panel (task 1003855, ADR 0296). A speciality is a named bundle of
+   knowledge and terms under a craft; this styles the list, the train form and the
+   create form. Tokens only — no literal colours, so light/dark follow the page. */
+.speciality-craft {
+  font-size: 0.78rem;
+  text-transform: lowercase;
+  opacity: 0.72;
+  margin-left: 0.4rem;
+}
+.speciality-badge {
+  font-size: 0.7rem;
+  padding: 0.05rem 0.4rem;
+  border-radius: 0.5rem;
+  margin-left: 0.4rem;
+  border: 1px solid currentColor;
+}
+.speciality-counts { opacity: 0.82; }
+.speciality-card { flex-wrap: wrap; }
+.speciality-train,
+.speciality-create {
+  flex-basis: 100%;
+  margin-top: 0.75rem;
+  padding-top: 0.75rem;
+  border-top: 1px solid var(--rule, rgba(128, 128, 128, 0.25));
+  display: flex;
+  flex-direction: column;
+  gap: 0.6rem;
+}
+.speciality-create__h { margin: 0; font-size: 0.95rem; }
+.speciality-field { display: flex; flex-direction: column; gap: 0.25rem; }
+.speciality-field > span { font-size: 0.85rem; }
+.speciality-field small { display: block; opacity: 0.7; font-weight: normal; }
+.speciality-field input[type='text'],
+.speciality-field select,
+.speciality-field textarea {
+  width: 100%;
+  box-sizing: border-box;
+}
+/* The checkbox row reads as one sentence, so it runs inline rather than stacked. */
+.speciality-field--check { flex-direction: row; align-items: flex-start; gap: 0.5rem; }
+.speciality-field--check input { margin-top: 0.2rem; }
+.speciality-actions { display: flex; gap: 0.5rem; }
+/* What a speciality OFFERS (task 1004404): the owner's pick-list of this
+   instance's skills, each with its plain line. Sixty-odd rows, so the list
+   scrolls inside its own box rather than pushing the form's buttons off-screen;
+   the reading lines keep the measure. */
+.speciality-field input[type='search'] { width: 100%; box-sizing: border-box; }
+.speciality-offer,
+.speciality-offer-box { flex-basis: 100%; display: flex; flex-direction: column; gap: 0.6rem; }
+.speciality-offer { margin-top: 0.75rem; padding-top: 0.75rem; border-top: 1px solid var(--rule); }
+.speciality-offer > *, .speciality-offer-box > * { max-width: var(--measure); }
+.speciality-offer-box > summary { cursor: pointer; font-size: 0.85rem; min-height: 24px; padding-block: 2px; box-sizing: border-box; }
+.speciality-offer-box > summary small { opacity: 0.7; }
+.speciality-offer__choices, #speciality-create-skills { display: flex; flex-direction: column; gap: 0.6rem; }
+.speciality-offer__list {
+  list-style: none;
+  margin: 0;
+  padding: 0 0.25rem 0 0;
+  display: flex;
+  flex-direction: column;
+  gap: 0.5rem;
+  max-height: 24rem;
+  overflow-y: auto;
+  overflow-x: hidden;
+}
+.speciality-offer__list li[hidden] { display: none; }
+.speciality-offer__list strong { font-family: var(--font-mono); font-weight: 600; }
+.speciality-offer__list .speciality-field--check > span { font-size: 0.85rem; line-height: 1.45; }
+/* The adopted contract, shown in full before "Use this" — another builder's prose
+   reaching your sessions is the one thing here that must never be folded away. */
+.speciality-contract { flex-basis: 100%; margin-top: 0.5rem; font-size: 0.85rem; }
+/* 24px tall: the panel was never revealed until task 1004073, so the kit had
+   never measured this toggle; it came back 21px. */
+.speciality-contract > summary { cursor: pointer; opacity: 0.85; min-height: 24px; padding-block: 2px; box-sizing: border-box; }
+.speciality-contract__body {
+  white-space: pre-wrap;
+  margin: 0.4rem 0 0;
+  padding: 0.5rem 0.65rem;
+  border-left: 2px solid var(--rule, rgba(128, 128, 128, 0.35));
+  border-radius: 0;
+  font-family: inherit;
+  opacity: 0.92;
+}
+/* The skills walkthrough (task 1004073, ADR 0310 §3): one offered skill at a
+   time, three answers as a definition list, then a decision. It sits inside the
+   card like the train form, so it inherits the card's ground in both modes. */
+.skill-walk {
+  flex-basis: 100%;
+  margin-top: 0.75rem;
+  padding-top: 0.75rem;
+  border-top: 1px solid var(--rule);
+  display: flex;
+  flex-direction: column;
+  gap: 0.6rem;
+}
+/* The measure caps the reading lines, never the section: a capped flex-basis
+   lets the section fit beside the card's actions instead of wrapping under them. */
+.skill-walk > * { max-width: var(--measure); }
+.skill-walk__progress { margin: 0; font-size: 0.8rem; color: var(--ink-soft); }
+.skill-walk__name { margin: 0; font-family: var(--font-mono); font-size: 1rem; color: var(--ink); }
+.skill-walk__facts { margin: 0; display: grid; gap: 0.2rem 0; }
+.skill-walk__facts dt { font-size: 0.8rem; color: var(--ink-soft); margin-top: 0.4rem; }
+.skill-walk__facts dt:first-child { margin-top: 0; }
+.skill-walk__facts dd { margin: 0; color: var(--ink); line-height: 1.5; }
+.skill-walk__note { font-size: 0.8rem; }
+/* "Leave it off" and "Turn it on" are the panel family's plain pill at EQUAL
+   weight: the default is off, so the page must not lean on the reader toward on.
+   Only "Save my choices" takes the accent. A decision already made reads through
+   aria-pressed when the person steps Back to it. */
+.skill-walk__actions { flex-wrap: wrap; }
+.skill-walk__actions .pbtn[aria-pressed="true"] { border-color: var(--ink-soft); }
+.skill-walk__recap { list-style: none; margin: 0; padding: 0; display: flex; flex-direction: column; gap: 0.5rem; }
+.skill-walk__recap li { display: flex; flex-wrap: wrap; align-items: baseline; gap: 0 0.4rem; }
+.skill-walk__recap .prow__hint { flex-basis: 100%; margin: 0; }
+.skill-walk__recap strong { font-family: var(--font-mono); font-weight: 600; }
+/* "Finish later" is a way out, not a choice about a skill, so it reads as a quiet
+   link below the decision rather than a third button beside it. */
+.skill-walk__later {
+  align-self: flex-start;
+  background: none;
+  border: 0;
+  padding: 0 0.25rem;
+  margin-inline-start: -0.25rem;
+  min-height: 24px;
+  font: inherit;
+  font-size: 0.8rem;
+  color: var(--ink-soft);
+  text-decoration: underline;
+  text-underline-offset: 3px;
+  cursor: pointer;
+}
+.skill-walk__later:hover { color: var(--ink); }

package/modules/hall-ui/public/{settings-specialities.js → speciality-tools.js}

@@ -1,5 +1,6 @@
-// modules/hall-ui/public/settings-specialities.js — the "Your specialities" panel
-// on /builders/settings (task 1003855; ADR 0296).
+// modules/hall-ui/public/speciality-tools.js — THE SPECIALITY TOOLS, one component
+// mounted by two pages: Settings' "Your specialities" panel (task 1003855; ADR
+// 0296) and My agents' "What it knows" section (task 1005052).
 //
 // WHAT A SPECIALITY IS, for anyone reading this file first: a craft says HOW you
 // work (engineer / ideator / artist / ui); a speciality says what you work ON and
@@ -7,10 +8,21 @@
 // contract you write here is injected into every session where you hold a claim
 // in that craft.
 //
-// Its own file rather than more of settings.js, on the settings-interaction.js /
-// settings-sessions.js precedent and for the same stated reason: settings.js sits
-// against the 1500-line size ratchet. Self-contained — it shares nothing with the
-// rest of the page but `window.OTB` and the generated client, both loaded before it.
+// ONE COMPONENT, TWO MOUNTS (task 1005052). This was settings-specialities.js;
+// it was carved out so My agents gets every tool Settings has without a second
+// copy. A page mounts it by carrying the hosts, by id, inside one section:
+//   #specialities-scroll      the section (ships `hidden`; revealed by load())
+//   #speciality-suggestions   what the project suggests
+//   #specialities-rows        the catalogue, one card per speciality (a <ul>)
+//   #speciality-create        an EMPTY <form>: createFormHtml() below fills it,
+//                             so the create form's markup exists once, here
+//   #specialities-status      the status line
+// Nothing else is page-specific: each page's own sheet draws the same markup in
+// its own look (Settings through panel.css, My agents through my-agents.css's
+// `.ma-know` skin; the structural rules are speciality-tools.css). Task 1005044
+// removes the Settings mount (its section and this file's script tag); the code
+// stays. Self-contained — it shares nothing with either page but `window.OTB`,
+// loaded before it, and one DOM event it dispatches (changed(), below).
 //
 // TRAINING IS TWO-FOLD, and only half of it is visible here. Documents you add are
 // the half you drive; the other half accumulates on its own from learnings captured
@@ -33,6 +45,13 @@
   const el = (id) => document.getElementById(id);
   const statusLine = (msg) => { const s = el('specialities-status'); if (s) s.textContent = msg || ''; };
 
+  // What a session is sent just moved (an adopt, a stop, a skills save): say so,
+  // so a page that shows it can read it again. My agents' "What your agent reads
+  // first" listens (task 1005052); Settings does not, and nothing here waits on it.
+  function changed() {
+    if (typeof CustomEvent === 'function' && document.dispatchEvent) document.dispatchEvent(new CustomEvent('otb:specialities-changed'));
+  }
+
   async function call(method, path, body) {
     const res = await fetch(`${API}${path}`, {
       method,
@@ -203,6 +222,7 @@
         state.walk = null;
         toast(`Saved — ${enabled.length} of ${steps.length} skill${steps.length === 1 ? '' : 's'} on.`);
         await load();
+        changed();
         return;
       } catch (err) {
         w.saving = false;
@@ -458,6 +478,7 @@
         if (state.walk && state.walk.id === String(id)) state.walk = null;
       }
       await load();
+      if (action === 'adopt' || action === 'abandon') changed();
       // Adopting is the moment the ruling is about: a speciality that offers
       // skills opens straight into the walkthrough, with every one still off.
       if (action === 'adopt') {
@@ -585,8 +606,64 @@
     });
   }
 
+  // THE CREATE FORM, rendered into the page's empty `<form id="speciality-create">`
+  // (task 1005052). It lived in settings.html until a second page mounted these
+  // tools; here it is one copy both pages share, filled before wire() reads its
+  // fields. The markup is the one settings.html carried, unchanged.
+  function createFormHtml() {
+    return `
+                <h3 class="speciality-create__h">Create one</h3>
+                <label class="speciality-field">
+                  <span>Name</span>
+                  <input type="text" name="name" maxlength="80" required placeholder="Economics" />
+                </label>
+                <label class="speciality-field">
+                  <span>Role it sits under</span>
+                  <select name="discipline" required>
+                    <option value="">Pick one…</option>
+                    <option value="engineer">Engineer</option>
+                    <option value="ideator">Ideator</option>
+                    <option value="artist">Artist</option>
+                    <option value="ui">UI design</option>
+                  </select>
+                </label>
+                <label class="speciality-field">
+                  <span>Summary <small>(optional)</small></span>
+                  <input type="text" name="summary" maxlength="500" placeholder="Markets, incentives, and how to price them." />
+                </label>
+                <label class="speciality-field">
+                  <span>How Claude should engage you in it <small>(optional)</small></span>
+                  <textarea name="contract_md" rows="4" maxlength="4000" placeholder="Argue from mechanism, not vibes. Always name the assumption a number rests on."></textarea>
+                </label>
+                <label class="speciality-field">
+                  <span>Who can use it</span>
+                  <select name="visibility">
+                    <option value="private">Only me</option>
+                    <option value="project">Anyone in the owning project</option>
+                    <option value="public">Anyone on this instance</option>
+                  </select>
+                </label>
+                <label class="speciality-field speciality-field--check">
+                  <input type="checkbox" name="sellable" />
+                  <span>
+                    May be offered for sale later
+                    <small>Records your decision only — there is no marketplace yet, and nothing is listed anywhere.</small>
+                  </span>
+                </label>
+                <details id="speciality-create-offer" class="speciality-offer-box">
+                  <summary>Skills it offers <small>(optional)</small></summary>
+                  <div id="speciality-create-skills"><p class="hall-loading hall-loading--inline" role="status">Loading this project’s skills…</p></div>
+                </details>
+                <div class="speciality-actions">
+                  <button type="submit" class="btn">Create speciality</button>
+                </div>
+              `;
+  }
+
   function boot() {
     if (!el('specialities-rows')) return; // panel not on this page
+    const create = el('speciality-create');
+    if (create) create.innerHTML = createFormHtml();
     wire();
     load();
   }

package/modules/hall-ui/records/my-agents.md

@@ -1,14 +1,17 @@
 # My agents, and governing as a held seat — /my-agents and the nav (task 1004685, the living-city prototype)
 
-Covers: `my-agents.{html,js,css,states.json}`, `gov-positions.js`, the `position` flag in `shell.js` NAV, the seat in `craft-mode.js`, and the harness recorder `scripts/hall-preview/pref-writes.js`. Built on round-3 boards 6 ("the city and My agents"), 7 ("how strict") and 8 ("governance is a position") of the living-city design canvas. **Every row is real since task 1004714**: the settings, the laws' locks and caps, and Make it a law. The strictness preset strip is real too since task 1004715 (`GET /government/presets`, [ADR 0380](../../../docs/adr/0380-a-strictness-preset-is-a-bundle-of-laws-switched-by-one-laws-vote.md)).
+Covers: `my-agents.{html,js,css,states.json}`, the speciality tools (`speciality-tools.{js,css}`, task 1005052), Settings' pointer here and its old-link map (`settings-moved.js`, task 1005044), `gov-positions.js`, the `position` flag in `shell.js` NAV, the seat in `craft-mode.js`, and the harness recorder `scripts/hall-preview/pref-writes.js`. Built on round-3 boards 6 ("the city and My agents"), 7 ("how strict") and 8 ("governance is a position") of the living-city design canvas. **Every row is real since task 1004714**: the settings, the laws' locks and caps, and Make it a law. The strictness preset strip is real too since task 1004715 (`GET /government/presets`, [ADR 0380](../../../docs/adr/0380-a-strictness-preset-is-a-bundle-of-laws-switched-by-one-laws-vote.md)).
 
 ## What it is
 
-- **My agents** (in You, after Profile): every setting that shapes how your own Claude works, one line each — a name of at most three words, one fact, one-tap chips (the owner's desk line rule). Real rows come from `GET /me` (interaction profile, wandering, model allocation, session cards), `GET /me/skill-prefs`, `GET /specialities` and the sounds switch; they write to the real routes. The five settings no table held (when stuck, when it asks, how it asks, at once, going live) were dropped by task 1004714 rather than shown as settings nothing stores or obeys; how an agent asks is a project rule now (the ask shape, ADR 0372-0373).
+- **My agents** (in You, after Profile): every setting that shapes how your own Claude works, one line each — a name of at most three words, one fact, one-tap chips (the owner's desk line rule). Real rows come from `GET /me` (interaction profile, wandering, model allocation, session cards), `GET /me/skill-prefs` and the sounds switch; they write to the real routes. Session cards and each skill voice offer their way back to the default as the FIRST chip (`follow my profile`, `default`; task 1005052): it writes `null`, which clears your own pick rather than pinning today's value, and a level or model is pressed only when it is your own pick. The five settings no table held (when stuck, when it asks, how it asks, at once, going live) were dropped by task 1004714 rather than shown as settings nothing stores or obeys; how an agent asks is a project rule now (the ask shape, ADR 0372-0373).
 - **Only a law in force locks a row** (task 1004714). The rows read gov-living's model, which holds nothing but real reads (task 1004717 deleted the prototype overlay), so nothing else locks, caps or advises one. The strictness strip (real, task 1004715) names the preset the laws in force amount to ("No preset yet" on the hub) and previews what each WOULD lock of these rows — the server's locks after a switch, fixed or capped (Open workshop none, Guild off-task, Factory off-task and the three model slots) — with a tile and one pip per lockable row; it never locks a row. Previewing another preset offers "put it to a vote" to a builder who may open a board item: one real laws vote (`POST /government/board/presets`).
 - **What your agent reads first** quotes the REAL contract text `/me` carries (the Conductor relays it), says plainly when a knob is not sent (permissive wandering is `inject: false`), and puts the project's REAL laws above it: `/me`'s `laws` block, the very words every session gets first (task 1004707). With no law in force that block is not drawn.
 - **A real law's lock is real.** `/me`'s `laws.locks` says what each law in force did to your settings; the server already applied it (the slice's effective value obeys it), so the row is locked, naming the law and its number in the real book, with your own choice underneath. A law that only CAPS a setting (`{ setting, max }`) does not lock the row: the server names every value it still allows (`allowed` on each lock, `law-locks.js` allowedFor — the same list a refused PATCH carries), the chips offer those, and the line says `capped by <law> · Law N`. A cap that leaves one value, or a server that names none, reads as the whole-row lock. A lock the server could not enforce (a setting no table holds yet) locks nothing. A write the law forbids answers 409 `locked_by_law`, and the toast says the law.
 - **Make it a law** lists your own non-default choices that no law already locks or caps; proposing one files a real laws vote (`POST /government/board/laws`, task 1004706): one law, in the desk's voice, locking the setting to your value, sat under the constitution in force. It is offered only to a builder `/me` says may open a board item (`permissions['board.item.open']`, one of `NAV_PERMISSION_ATOMS`); an unreadable map offers nothing. A filed habit links to its vote in the Board Room.
+- **What it knows is the speciality tools, not desk rows** (task 1005052). The section `#specialities-scroll` under the desk is filled by `speciality-tools.js` — the SAME component Settings mounts (it was `settings-specialities.js`), finding its hosts by id: create (it renders the create form into the page's empty `<form id="speciality-create">`, so the markup exists once), train, offered skills, adopt with someone else's full instructions open in the same card, the skills walkthrough, stop using, and the project's suggestions. It is its own section because a desk row holds at most two buttons and `my-agents.js` redraws `#ma-rows` on every change, which would wipe a half-filled form. Its markup is in the panel family's words (`.prow`, `.btn`, `.pbtn`); this page has no `panel.css`, so `my-agents.css`'s `.ma-know` skin draws them here as hairline rows and ghost pills, no accent (the structure is `speciality-tools.css`, shared). The desk's old one-tap adopt chips are gone: they adopted someone else's speciality without showing its instructions. After an adopt, a stop or a skills save the component dispatches `otb:specialities-changed` and this page re-reads `/me`, so "What your agent reads first" stays true. Since task 1005044 this is the component's only host. Tests: `tests/speciality_offer_ui.mjs` and `tests/speciality_walkthrough_ui.mjs` run every tool on this page's own markup (`tests/helpers.mjs` `bootSpecialityTools`, `SPECIALITY_PAGES`). Kit states `what-it-knows`, `speciality-walk`, `speciality-forms`; the harness serves an in-use Pricing (`me__specialities.json`) and canned stop-using and train answers.
+- **This is the ONE page for the agent settings** (task 1005044, the owner's call on idea 1001309). How Claude talks to you, Wandering, Session cards, Skill voices and the specialities were edited on Settings too, on the same routes, with no hint the two were one setting. Settings now carries none of them: a line in its Preferences group ("Your agents") and one in Sound ("Skill voices") link here, and `settings-moved.js` sends every old deep link (`/settings#interaction`, `#wandering`, `#rendering`, `#render`, `#voices`, `#specialities` and the old section ids) on with `location.replace`. **The anchors this page lands** are `my-agents.js` `ANCHORS`: `#talk`, `#work`, `#models` (the desk groups, `id="ma-group-<key>"`), `#skills` (opens Per skill) and `#knows` (What it knows, watched until speciality-tools.js reveals it — no timer, the motion test allows one). The palette offers the five (`palette.js` SECTIONS, `tests/hall_palette.mjs`). Settings' old panels went with their only host: `settings-interaction.js`, `settings-render.js`, `settings-law-locks.js` are deleted; what each promised is now proven here (`tests/hall_my_agents.mjs` `MOVED:`, `tests/government_tidy_up.mjs` case 5). Not carried over on purpose: Settings drew wandering levels above your rank's ceiling as disabled rows; here they are simply not offered.
+- **The fact keeps a floor.** The row grid is `minmax(118px, 160px) minmax(132px, 1fr) auto` since task 1005052: session cards' five chips wrap rather than squeeze its fact to one word a line.
 - **Governing is a held seat** (real since task 1004713, ADR 0377). `GET /me` carries `positions`, resolved by the server (`modules/government/positions.js`: the owner seat, a Lead in office, a seat on a building's sitting, the board under the constitution in force). `gov-positions.js` only reads that block — no constitution or rank. Items flagged `position: true` (the City, the Board Room, the Government group) hide for a builder the server says holds none, and the craft mode names Governing by the seat ("Planning Lead") or drops it. A `/me` with no block is unknown and keeps today's nav.
 
 ## The rules an edit here must keep

package/modules/hall-ui/records/panel-family.md

@@ -29,4 +29,8 @@ Also fixed here, as the board task handed forward: the two sub-24px links on Hom
 
 **Settings: adopting a speciality walks through its skills (task 1004073, ADR 0310 §3)** — the owner ruled an adopted speciality's skills start OFF "and you should walk through what comes with each". So adopting one that offers skills opens `.skill-walk` inside its card: one skill per step (what it does, when to reach for it, what it costs, what it needs, who can run it), "Leave it off" / "Turn it on" at EQUAL weight (plain `.pbtn`, no accent — the default is off, so the page must not lean toward on), Back, then a recap restating each skill in a line, then ONE `PUT` of the whole set. "Finish later" saves nothing. Focus moves to each new step heading. Two things found on the way: the specialities panel shipped `hidden` and nothing ever revealed it, so adoption was unreachable from the hall; and its contract `<summary>` measured 21px once visible (now 24). **Kit:** `settings.states.json` `skills-walkthrough` + `preferences`, ALL CLEAN at 1440/390/320 × dark/light against the harness, with fixtures `specialities*.json` built from the real skills on disk and canned answers for adopt and save. Tests: `tests/speciality_walkthrough_ui.mjs` (runs the real page script).
 
+**Settings: the speciality panel became a shared component (task 1005052)** — `settings-specialities.js` is now `speciality-tools.js`, mounted by Settings and by My agents (records/my-agents.md); it renders the create form into settings.html's now-empty `<form id="speciality-create">`, and its rules moved verbatim from `settings.css` to `speciality-tools.css`, linked right after it. Settings was verified unchanged against a before-capture: the same elements, every computed style and the same text in five states (rest, train, offered skills, the create picker, the walkthrough). **Known gap, older than this:** the `skills-walkthrough` and `offered-skills` states never close the first-visit craft picker, so their clicks time out under the kit and the page reader (both notes are on main); with a `clickIfVisible [data-craft-close]` step they are ALL CLEAN.
+
 **Settings: a speciality's owner chooses what it offers (task 1004404, ADR 0310 §1)** — until this only the API could set an offer, so the walkthrough above never appeared for a speciality made in the hall. The create form gains a `<details>` "Skills it offers" picker (filled on first open, so a builder who never offers one never pays for the list), and an owned card gains "Offered skills", an inline `form.speciality-offer` like Train. Both list `GET /specialities/installed-skills` — only what this instance has — each row a checkbox with the skill's plain line and cost, the words adopters will see, under a line saying offering turns nothing on. A "Find a skill" box narrows the sixty-odd rows (Enter never submits), the list scrolls in its own box, an offered skill removed since is named and dropped on save rather than silently kept, and saving is ONE `PATCH { skills }`. **The checkboxes take the shared 24px rule in `panel.css`** (the `.chip-ctl input` / `.craft-pref` block, one selector added) rather than a fourth copy of the `::before` box; without it 128 targets measured 13px. **Kit:** `settings.states.json` `offered-skills` + `preferences`, ALL CLEAN at 1440/390/320 × dark/light, with fixtures `specialities__installed-skills.json` (the real 64 skills, explained) and an owned speciality in `specialities.json`, plus canned POST/PATCH answers. Tests: `tests/speciality_offer_ui.mjs` (runs the real page script).
+
+**Settings: the agent settings left for My agents (task 1005044)** — How Claude talks to you, the specialities, Wandering, Rendering and Skill voices are gone from Settings (owner decision, idea 1001309): they were edited here and on My agents on the same routes. Preferences keeps Module install reports plus a `#agents-moved-preferences` line linking `/my-agents#talk`; Sound keeps the sounds-off note plus `#agents-moved-sound` linking `/my-agents#skills`. Old deep links go on through `settings-moved.js` (records/my-agents.md). `settings-interaction.js`, `settings-render.js` and `settings-law-locks.js` were deleted with their only host, and `settings.css` lost the voices table. The `skills-walkthrough` and `offered-skills` states left `settings.states.json` (My agents' `speciality-walk` and `speciality-forms` cover them). `tests/hall_settings_world.mjs` pins the absence, the two lines and the redirects.