@bongos/core 1.24.27 → 1.24.28

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

@@ -18,6 +18,8 @@
        has — the rail layout, the group head, the token ladder, the voices grid. -->
   <link rel="stylesheet" href="/builders/panel.css">
   <link rel="stylesheet" href="/builders/settings.css">
+  <!-- the speciality tools sheet (task 1005052), shared with My agents -->
+  <link rel="stylesheet" href="/builders/speciality-tools.css">
 </head>
 <body data-page="settings" data-page-title="Settings">
 
@@ -305,53 +307,9 @@
               <div id="speciality-suggestions"></div>
               <ul class="prows" id="specialities-rows"></ul>
 
-              <form id="speciality-create" class="speciality-create">
-                <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>
-              </form>
+              <!-- The create form is rendered by speciality-tools.js (task 1005052), the
+                   one copy Settings and My agents share; this is only its host. -->
+              <form id="speciality-create" class="speciality-create"></form>
 
               <p class="settings-status" id="specialities-status" role="status" aria-live="polite"></p>
             </section>
@@ -534,7 +492,9 @@
      the same precedent and for the same reason — settings.js is against the
      1500-line ratchet, and this panel shares nothing with the rest of the page. -->
 <script src="/builders/settings-interaction.js?v=2026-10-17-laws"></script>
-<script src="/builders/settings-specialities.js?v=2026-09-17-specialities"></script>
+<!-- task 1005052: the speciality tools are ONE component Settings and My agents
+     both mount (it was settings-specialities.js). Task 1005044 removes this mount. -->
+<script src="/builders/speciality-tools.js?v=2026-11-03-shared"></script>
 <!-- task 1002518: the card-richness ladder. Own file on the same precedent —
      the panel grew from one switch to a rung picker with a derived default, and
      settings.js is against the 1500-line ratchet. -->

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,16 @@
 # 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 it shares with Settings (`speciality-tools.{js,css}`, task 1005052), `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. Task 1005044 removes the Settings mount; the code stays. Tests: `tests/speciality_offer_ui.mjs` and `tests/speciality_walkthrough_ui.mjs` run every tool on BOTH pages' own markup (`tests/helpers.mjs` `bootSpecialityTools`). 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.
+- **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,6 @@ 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).

package/package-lock.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.24.27",
+  "version": "1.24.28",
   "description": "Cloud Bongos — the AI-first build platform core (GDS + platform surfaces + module system), installed as a versioned dependency (ADR 0108).",
   "license": "AGPL-3.0-or-later",
   "main": "src/platform-server.js",

package/release-notes.json

@@ -11095,5 +11095,11 @@
       "id": "1005053",
       "text": "The account, projects, community and status pages now show the Bongos logo turning while they wait for data, instead of a plain \"Reading…\" line."
     }
+  ],
+  "1.24.28": [
+    {
+      "id": "1005052",
+      "text": "My agents now has every speciality tool Settings has (create one, train it, choose what it offers, take one on after reading its full instructions, pick its skills, stop using it), and session cards and each skill's model can"
+    }
   ]
 }

package/scripts/hall-preview/server.js

@@ -195,6 +195,10 @@ const CANNED_WRITES = [
   // the Market Watch weekly ask's answer (task 1004940), so its "sent." state renders.
   ['POST', /^market-watch\/ask\/answer$/, 'market-watch__ask__answer.post'],
   ['PATCH', /^specialities\/\d+$/, 'specialities__12.patch'],
+  // the rest of the speciality tools, now on My agents too (task 1005052): stop
+  // using one, and train your own, so their toasts and redraws render here.
+  ['POST', /^specialities\/\d+\/abandon$/, 'specialities__12__abandon.post'],
+  ['POST', /^specialities\/\d+\/documents$/, 'specialities__12__documents.post'],
   // a vote cast from the street's matter card (task 1004528), so the card's
   // answer (the toast, the building lighting) renders here too.
   ['POST', /^government\/board\/items\/\d+\/votes$/, 'government__board__items__votes.post'],

package/src/module-api.js

@@ -81,7 +81,7 @@ const { sleep } = require('./sleep');
 // MAJOR (see allowBoxScope below): passes the request through untouched.
 function deprecatedNoopMiddleware(_req, _res, next) { next(); }
 
-const CORE_VERSION = '1.24.27'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.24.28'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
 
 // A namespaced logger so a module's log lines are attributable + consistent.
 // Usage: const log = api.logger('discord'); log.info('mounted');

package/tests/hall_loading.mjs

@@ -136,7 +136,9 @@ test('the placeholders the page scripts write wear it too, and a failure stops i
     'criterion-uat-ui.js': '<p class="hall-loading hall-loading--inline" role="status">Loading…</p>',
     'watch.js': '<p class="hall-loading" role="status">Reading…</p>',
     'studio.js': '<p class="hall-loading" role="status">Reading the reviews…</p>',
-    'settings-specialities.js': '<p class="hall-loading hall-loading--inline" role="status">Loading this project’s skills…</p>',
+    // the speciality tools, Settings' and My agents' one copy (task 1005052): the offer
+    // editor's and the create form's skills picker both write it
+    'speciality-tools.js': '<p class="hall-loading hall-loading--inline" role="status">Loading this project’s skills…</p>',
     'settings-software-update.js': '<p class="hall-loading hall-loading--inline" role="status">Checking for a newer version…</p>',
     'goals-page.js': '<div class="hall-loading" role="status">Loading…</div>',
   };

package/tests/hall_my_agents.mjs

@@ -440,10 +440,11 @@ test('PAGE: in the hall family, with no prototype key', () => {
   const html = read('my-agents.html');
   assert.match(html, /<body data-page="my-agents"/);
   const sheets = [...html.matchAll(/href="\/builders\/([a-z-]+\.css)/g)].map((m) => m[1]);
-  assert.deepEqual(sheets, ['style.css', 'oversight.css', 'my-agents.css'], 'rides the oversight archetype, linked before its own sheet');
+  assert.deepEqual(sheets, ['style.css', 'oversight.css', 'speciality-tools.css', 'my-agents.css'],
+    'rides the oversight archetype; the shared speciality sheet (task 1005052) before its own, so the page skin wins');
   const scripts = [...html.matchAll(/src="\/builders\/([a-z-]+)\.js/g)].map((m) => m[1]);
   assert.deepEqual(scripts.slice(0, 4), ['dom-utils', 'shell-trail', 'craft-mode', 'shell']); // dom-utils first: the shell escapes through it (task 1004876)
-  for (const s of ['gov-living', 'gov-positions', 'my-agents']) assert.ok(scripts.includes(s), `loads ${s}.js`);
+  for (const s of ['gov-living', 'gov-positions', 'my-agents', 'speciality-tools']) assert.ok(scripts.includes(s), `loads ${s}.js`);
   assert.doesNotMatch(html, /ma-key|data-proto|Prototype data/, 'nothing on the page is prototype data, so there is no key');
   assert.doesNotMatch(read('my-agents.js'), /governor-overlay/, 'and it reads no overlay');
   assert.match(html, /id="hall-main"/);
@@ -480,3 +481,104 @@ test('PAGE: no animation loop or timer-driven motion in the page script', () =>
   // the one timer takes the saved tick's attribute back off; CSS plays the motion
   assert.equal((code.match(/setTimeout\(/g) || []).length, 1);
 });
+
+// ---------------------------------------------------------------------------
+// 6. everything Settings can do here too (task 1005052)
+// ---------------------------------------------------------------------------
+
+const { createPrefRecorder: recorder } = require(path.join(ROOT, 'scripts', 'hall-preview', 'pref-writes.js'));
+
+test('DEFAULTS: session cards can go back to following your interaction profile', () => {
+  const me = fx('me');
+  assert.equal(me.render.override_level, null, 'the fixture viewer follows the profile');
+  const r = row(viewOf(), 'render');
+  assert.deepEqual(r.options[0], { value: '', label: 'follow my profile', pressed: true }, 'the way back comes first, and is the choice in force');
+  assert.deepEqual(r.options.slice(1).map((o) => o.value), me.render.levels.map((l) => l.value), 'then the four levels');
+  assert.ok(r.options.slice(1).every((o) => !o.pressed), 'a level is pressed only when it is your own pick');
+  assert.equal(r.fact, MyAgents.SETTING.render.words[me.render.derived_level], 'the fact is the level the profile gives');
+  assert.equal(r.note, 'follows your profile');
+  const pinned = fx('me');
+  pinned.render = { ...pinned.render, override_level: 'minimal', level: 'minimal' };
+  const p = row(viewOf({ me: pinned }), 'render');
+  assert.equal(p.options[0].pressed, false, 'a pinned level is not "follow my profile"');
+  assert.deepEqual(p.options.filter((o) => o.pressed).map((o) => o.value), ['minimal']);
+  assert.equal(p.fact, 'one line');
+  assert.equal(p.note, null);
+  assert.deepEqual(MyAgents.writeFor('render', '', { me }), { method: 'PATCH', path: '/me/render', body: { widgets: null }, slice: 'render' }, 'the way back CLEARS the pick (null), it does not pin today\'s level');
+  assert.deepEqual(MyAgents.writeFor('render', 'minimal', { me }).body, { widgets: 'minimal' });
+});
+
+test('DEFAULTS: the real shapers take the way back for session cards, and the row reads it as chosen', () => {
+  const rec = recorder({ log() {} });
+  const me = fx('me');
+  assert.equal(rec.write('me/render', MyAgents.writeFor('render', 'minimal', { me }).body, { me }).body.override_level, 'minimal');
+  const back = rec.write('me/render', MyAgents.writeFor('render', '', { me }).body, { me });
+  assert.equal(back.status, 200, 'the real route accepts it');
+  assert.equal(back.body.override_level, null);
+  assert.equal(back.body.level, back.body.derived_level, 'the cards follow the profile again');
+  const after = { ...me, render: back.body };
+  assert.equal(row(viewOf({ me: after }), 'render').options[0].pressed, true);
+});
+
+test('DEFAULTS: each skill voice can go back to its default model', () => {
+  const sk = fx('me__skill-prefs');
+  const subs = row(viewOf(), 'skills').skills;
+  assert.equal(subs.length, sk.effective.length);
+  for (const s of subs) {
+    assert.deepEqual(s.options[0], { value: '', label: 'default', pressed: true }, `${s.skill}: the default comes first, and is in force`);
+    assert.deepEqual(s.options.slice(1).map((o) => o.value), sk.allowed_models);
+    assert.ok(s.options.slice(1).every((o) => !o.pressed), `${s.skill}: no model is pressed you did not pick`);
+    assert.equal(s.note, null);
+  }
+  const own = JSON.parse(JSON.stringify(sk));
+  own.effective[0] = { ...own.effective[0], override_model: 'opus', effective_model: 'opus' };
+  const s0 = row(viewOf({ skills: own }), 'skills').skills[0];
+  assert.equal(s0.options[0].pressed, false);
+  assert.deepEqual(s0.options.filter((o) => o.pressed).map((o) => o.value), ['opus']);
+  assert.equal(s0.value, 'opus');
+  assert.equal(s0.note, 'default: ' + sk.effective[0].default_model, 'with your own pick, the default it replaces is said');
+  assert.match(row(viewOf({ skills: own }), 'skills').fact, /1 yours$/);
+  assert.deepEqual(MyAgents.writeFor('skill.' + sk.effective[0].skill, '', {}).body, { [sk.effective[0].skill]: null }, 'the way back clears the pick');
+});
+
+test('DEFAULTS: the real shapers take the way back for a skill voice', () => {
+  const rec = recorder({ log() {} });
+  const me = fx('me');
+  const sk = fx('me__skill-prefs');
+  const skill = sk.effective[0].skill;
+  rec.write('me/skill-prefs', { [skill]: 'opus' }, { me, skillFixture: sk });
+  const back = rec.write('me/skill-prefs', MyAgents.writeFor('skill.' + skill, '', {}).body, { me, skillFixture: sk });
+  assert.equal(back.status, 200);
+  assert.equal(skill in back.body.prefs, false, 'the override is gone');
+  const e = back.body.effective.find((x) => x.skill === skill);
+  assert.equal(e.override_model, null);
+  assert.equal(e.effective_model, e.default_model);
+  assert.equal(row(viewOf({ skills: back.body }), 'skills').skills[0].options[0].pressed, true);
+});
+
+test('PAGE: My agents mounts the SAME speciality component Settings does, in a section of its own', () => {
+  const html = read('my-agents.html');
+  assert.ok(!fs.existsSync(path.join(HALL, 'settings-specialities.js')), 'one copy: the Settings file was carved into the shared one');
+  for (const page of ['my-agents.html', 'settings.html']) assert.match(read(page), /src="\/builders\/speciality-tools\.js/, `${page} loads it`);
+  const at = html.indexOf('id="specialities-scroll"');
+  assert.ok(at > 0, 'the section is on the page');
+  assert.match(html.slice(html.lastIndexOf('<section', at), at), /class="scroll ma-know"/, 'a panel of its own');
+  const desk = html.slice(html.indexOf('id="ma-desk"'), html.indexOf('</section>', html.indexOf('id="ma-desk"')));
+  assert.doesNotMatch(desk, /specialities-rows/, 'not inside the desk, which my-agents.js redraws on every change and would wipe a form half-filled');
+  // tests/speciality_offer_ui.mjs and tests/speciality_walkthrough_ui.mjs drive every tool on this page's markup
+});
+
+test('PAGE: the desk adopts nothing in one tap — taking a speciality on goes through its instructions, read in full', () => {
+  const v = viewOf();
+  assert.equal(row(v, 'specialities'), undefined, 'no desk row adopts');
+  assert.equal(v.groups.some((g) => g.key === 'know'), false, 'the specialities are their own section');
+  assert.doesNotMatch(codeOf(read('my-agents.js')), /\/adopt'|data-adopt/, 'my-agents.js posts no adopt of its own');
+});
+
+test('PAGE: when a speciality changes, what your agent reads first is read again', () => {
+  // speciality-tools.js dispatches it after an adopt, a stop or a skills save
+  // (tests/speciality_walkthrough_ui.mjs); the page listens and re-reads /me.
+  const code = codeOf(read('my-agents.js'));
+  assert.match(code, /document\.addEventListener\('otb:specialities-changed', /);
+  assert.match(codeOf(read('speciality-tools.js')), /new CustomEvent\('otb:specialities-changed'\)/);
+});