@bongos/core 1.19.816 → 1.19.817

package/docs/copy-registry.json

@@ -3,9 +3,9 @@
   "generator": "scripts/gds/copy-inventory.js",
   "goal": 1000074,
   "counts": {
-    "strings": 1860,
-    "unique_ids": 1614,
-    "certain": 1860,
+    "strings": 1863,
+    "unique_ids": 1617,
+    "certain": 1863,
     "heuristic": 0,
     "duplicate_ids": 170
   },
@@ -13,7 +13,7 @@
     "builders-hall": {
       "at": "modules/hall-ui/public",
       "files": 86,
-      "strings": 1370
+      "strings": 1373
     },
     "status-dashboard": {
       "at": "modules/status-ui/public",
@@ -911,7 +911,7 @@
       "occurrences": 2,
       "at": [
         "modules/hall-ui/public/settings.html:260",
-        "modules/hall-ui/public/settings.html:365"
+        "modules/hall-ui/public/settings.html:366"
       ]
     },
     {
@@ -2010,7 +2010,7 @@
       "surface": "builders-hall",
       "text": ", claiming, and shipping show — spend on explaining themselves. Richer cards say more and cost more tokens; leaner ones carry the same facts for less. By default this follows how Claude talks to you, so you can set it once in",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 361,
+      "line": 362,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -2298,7 +2298,7 @@
       "surface": "builders-hall",
       "text": "A sound plays at each turn of your work — a gentle chime when Claude finishes, a flourish when a task ships or your rank rises. Turn off any you would rather not hear.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 397,
+      "line": 398,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -2595,7 +2595,7 @@
       "surface": "builders-hall",
       "text": "Anyone in the owning project",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 317,
+      "line": 318,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -2604,7 +2604,7 @@
       "surface": "builders-hall",
       "text": "Anyone on this instance",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 318,
+      "line": 319,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -2712,7 +2712,7 @@
       "surface": "builders-hall",
       "text": "Argue from mechanism, not vibes. Always name the assumption a number rests on.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 311,
+      "line": 312,
       "origin": "html-attr:placeholder",
       "confidence": "certain"
     },
@@ -3153,7 +3153,7 @@
       "surface": "builders-hall",
       "text": "Changes save at once and take effect the next time you start a session on this machine.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 408,
+      "line": 409,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -3162,7 +3162,7 @@
       "surface": "builders-hall",
       "text": "Changes save at once. Your new choice plays the next time that event fires.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 421,
+      "line": 422,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -3171,7 +3171,7 @@
       "surface": "builders-hall",
       "text": "Changes save the moment you pick. “Default” clears your override and falls back to the system's choice.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 391,
+      "line": 392,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -3252,7 +3252,7 @@
       "surface": "builders-hall",
       "text": "Choose which sound plays for each build event — one when Claude waits for your input, another when a task ships, a coin clink when {{currency}} arrive. Each event offers a small set of choices; more will be added as new audio ships.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 414,
+      "line": 415,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -4440,7 +4440,7 @@
       "surface": "builders-hall",
       "text": "Craft it sits under",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 296,
+      "line": 297,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -4458,7 +4458,7 @@
       "surface": "builders-hall",
       "text": "Create one",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 290,
+      "line": 291,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -4476,7 +4476,7 @@
       "surface": "builders-hall",
       "text": "Create speciality",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 329,
+      "line": 330,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -4863,7 +4863,7 @@
       "surface": "builders-hall",
       "text": "Each slash-command can run on a different model. The system has a default for every skill; choose another to override it, or leave it on Default.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 377,
+      "line": 378,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -4944,7 +4944,7 @@
       "surface": "builders-hall",
       "text": "Event sounds",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 413,
+      "line": 414,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -5592,7 +5592,7 @@
       "surface": "builders-hall",
       "text": "How Claude should engage you in it",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 310,
+      "line": 311,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -5610,7 +5610,7 @@
       "surface": "builders-hall",
       "text": "How Claude talks to you",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 365,
+      "line": 366,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -5655,7 +5655,7 @@
       "surface": "builders-hall",
       "text": "How much your session cards — what",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 360,
+      "line": 361,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -5673,7 +5673,7 @@
       "surface": "builders-hall",
       "text": "How readily Claude opens a",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 341,
+      "line": 342,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -6591,7 +6591,7 @@
       "surface": "builders-hall",
       "text": "Markets, incentives, and how to price them.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 307,
+      "line": 308,
       "origin": "html-attr:placeholder",
       "confidence": "certain"
     },
@@ -6600,7 +6600,7 @@
       "surface": "builders-hall",
       "text": "May be offered for sale later",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 323,
+      "line": 324,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -7113,7 +7113,7 @@
       "surface": "builders-hall",
       "text": "No specialities yet. Create one below — it takes a name and a craft, and you can train it afterwards.",
       "file": "modules/hall-ui/public/settings-specialities.js",
-      "line": 131,
+      "line": 169,
       "origin": "js-markup",
       "confidence": "certain"
     },
@@ -7473,7 +7473,7 @@
       "surface": "builders-hall",
       "text": "Only me",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 316,
+      "line": 317,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -7860,7 +7860,7 @@
       "surface": "builders-hall",
       "text": "Pick one…",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 298,
+      "line": 299,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -8382,7 +8382,7 @@
       "surface": "builders-hall",
       "text": "Records your decision only — there is no marketplace yet, and nothing is listed anywhere.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 325,
+      "line": 326,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -8823,7 +8823,7 @@
       "surface": "builders-hall",
       "text": "Saves the moment you choose; applies from your next session.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 369,
+      "line": 370,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -9363,7 +9363,7 @@
       "surface": "builders-hall",
       "text": "Skill voices",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 376,
+      "line": 377,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -9448,12 +9448,21 @@
       "origin": "js-notice",
       "confidence": "certain"
     },
+    {
+      "id": "d03aa6752970",
+      "surface": "builders-hall",
+      "text": "Specialities this project recommends. Nothing here is switched on — read what each one adds to your sessions, then choose.",
+      "file": "modules/hall-ui/public/settings-specialities.js",
+      "line": 137,
+      "origin": "js-markup",
+      "confidence": "certain"
+    },
     {
       "id": "01f1d2b03eb4",
       "surface": "builders-hall",
       "text": "Speciality created.",
       "file": "modules/hall-ui/public/settings-specialities.js",
-      "line": 213,
+      "line": 267,
       "origin": "js-notice",
       "confidence": "certain"
     },
@@ -9534,7 +9543,7 @@
       "surface": "builders-hall",
       "text": "Stopped using it.",
       "file": "modules/hall-ui/public/settings-specialities.js",
-      "line": 157,
+      "line": 200,
       "origin": "js-notice",
       "confidence": "certain"
     },
@@ -9583,6 +9592,15 @@
       "origin": "html-text",
       "confidence": "certain"
     },
+    {
+      "id": "a1482eff4ead",
+      "surface": "builders-hall",
+      "text": "Suggested for you",
+      "file": "modules/hall-ui/public/settings-specialities.js",
+      "line": 137,
+      "origin": "js-markup",
+      "confidence": "certain"
+    },
     {
       "id": "1e40f33483eb",
       "surface": "builders-hall",
@@ -10749,7 +10767,7 @@
       "surface": "builders-hall",
       "text": "UI design",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 302,
+      "line": 303,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -10780,12 +10798,21 @@
       "origin": "js-markup",
       "confidence": "certain"
     },
+    {
+      "id": "e7e2a99a2bea",
+      "surface": "builders-hall",
+      "text": "Use this",
+      "file": "modules/hall-ui/public/settings-specialities.js",
+      "line": 148,
+      "origin": "js-markup",
+      "confidence": "certain"
+    },
     {
       "id": "e1ecd543a2ca",
       "surface": "builders-hall",
       "text": "Using this speciality.",
       "file": "modules/hall-ui/public/settings-specialities.js",
-      "line": 156,
+      "line": 199,
       "origin": "js-notice",
       "confidence": "certain"
     },
@@ -11181,7 +11208,7 @@
       "surface": "builders-hall",
       "text": "Who can use it",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 314,
+      "line": 315,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -11505,7 +11532,7 @@
       "surface": "builders-hall",
       "text": "Your choice",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 385,
+      "line": 386,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -11883,7 +11910,7 @@
       "surface": "builders-hall",
       "text": "and leave this alone.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 365,
+      "line": 366,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -12549,7 +12576,7 @@
       "surface": "builders-hall",
       "text": "new thread",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 342,
+      "line": 343,
       "origin": "html-text",
       "confidence": "certain"
     },
@@ -12693,7 +12720,7 @@
       "surface": "builders-hall",
       "text": "of work — a tangent, an investigation, an unprompted suggestion — instead of staying on your claimed task. It never limits the work on the task itself. Your rank sets the starting point; newcomers stay locked low.",
       "file": "modules/hall-ui/public/settings.html",
-      "line": 342,
+      "line": 343,
       "origin": "html-text",
       "confidence": "certain"
     },

package/docs/module-api-changelog.md

@@ -2106,5 +2106,7 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.19.814 with no explicit bump. run 35197139985. (task 1002620)
   1.19.816 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         landed since 1.19.815 with no explicit bump. run 35198566737. (task 1002620)
+  1.19.817 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.19.816 with no explicit bump. run 35200117750. (task 1002620)
 ---------------------------------------------------------------------------
 ```

package/modules/dev-box/app/src/vendor/bongos-client.cjs

@@ -856,6 +856,12 @@ function createClient(opts = {}) {
       postSpecialitiesIdDocuments: (args) => request("POST", "/specialities/{id}/documents", { hasBody: true }, args),
       // DELETE /specialities/{id}/documents/{docId} — rank: any-builder — DELETE /specialities/:id/documents/:docId
       deleteSpecialitiesIdDocumentsDocId: (args) => request("DELETE", "/specialities/{id}/documents/{docId}", { hasBody: false }, args),
+      // POST /specialities/{id}/offer — rank: metic+archon — POST /specialities/:id/offer
+      postSpecialitiesIdOffer: (args) => request("POST", "/specialities/{id}/offer", { hasBody: true }, args),
+      // POST /specialities/{id}/unoffer — rank: metic+archon — POST /specialities/:id/unoffer
+      postSpecialitiesIdUnoffer: (args) => request("POST", "/specialities/{id}/unoffer", { hasBody: true }, args),
+      // GET /specialities/offered — rank: any-builder — GET /specialities/offered
+      getSpecialitiesOffered: (args) => request("GET", "/specialities/offered", { hasBody: false }, args),
     },
     "sso": {
       // POST /sso/activity/rollup — rank: public — POST /sso/activity/rollup

package/modules/government/catalog.js

@@ -238,7 +238,7 @@ const PERMISSIONS = [
   // anyone in a room with a stranger on its own.
   { key: 'mingle.manage', system: false, floor: 'metic', guards: 'PUT /mingle/project (the project half of mingle consent), POST /mingle/round (run a pairing round) — task 1003177, goal 1000075' },
   { key: 'project.settings.manage', system: false, floor: 'metic', guards: "PATCH /project-settings/* — the project's own live knobs (the rot timer today) and the work-category set's shape (PATCH /work-categories/:id). Metic, matching goal.create's reasoning that extending the project's vocabulary is a planning act rather than an administrative one — task 1003280, goal 1000072" },
-  { key: 'project.curate', system: false, floor: 'metic', guards: 'platform-identity: GET/POST /projects, /projects/invite, PATCH /projects/:id/featured, PATCH /projects/:id/status (the takedown lever — ADR 0252 §5.1), GET /scouting (ADR 0157, was archon)' },
+  { key: 'project.curate', system: false, floor: 'metic', guards: 'platform-identity: GET/POST /projects, /projects/invite, PATCH /projects/:id/featured, PATCH /projects/:id/status (the takedown lever — ADR 0252 §5.1), GET /scouting (ADR 0157, was archon); specialities: POST /specialities/:id/{offer,unoffer} — putting the project name behind a speciality, which reaches the sessions of everyone who accepts it (task 1003972)' },
   { key: 'override_request.decide', system: false, floor: 'metic', guards: 'GET /override-requests, POST .../:id/decide (ADR 0157, was archon)' },
   { key: 'claim.release.any', system: false, floor: 'metic', scope: 'any', resource: 'claim', verb: 'release', guards: 'POST /claims/:id/release-on-behalf (ADR 0157, was archon)' },
   { key: 'idea.ratify', system: false, floor: 'metic', guards: 'POST /inbox/:id/ratify (ADR 0157, was archon)' },

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

@@ -24,7 +24,7 @@
   const { escapeHtml, $ } = window.OTB;
   const toast = (msg) => window.OTB.toast(msg);
 
-  const state = { rows: [], openId: null, busy: false };
+  const state = { rows: [], suggestions: [], openId: null, busy: false };
 
   const el = (id) => document.getElementById(id);
   const statusLine = (msg) => { const s = el('specialities-status'); if (s) s.textContent = msg || ''; };
@@ -124,7 +124,45 @@
       </form>`;
   }
 
+  // WHAT THE PROJECT SUGGESTS. Rendered as an offer and never as a fait
+  // accompli: nothing here is adopted until the builder presses the button, and
+  // the contract is shown in full first, exactly as for any other speciality
+  // somebody else wrote. A suggestion is the project's judgement, so it says
+  // whose — a builder taking on a standing instruction should see who stood
+  // behind it.
+  function renderSuggestions() {
+    const host = el('speciality-suggestions');
+    if (!host) return;
+    if (!state.suggestions.length) { host.innerHTML = ''; return; }
+    host.innerHTML = `
+      <h3 class="speciality-create__h">Suggested for you</h3>
+      <p class="prow__hint">
+        Specialities this project recommends. Nothing here is switched on — read what
+        each one adds to your sessions, then choose.
+      </p>
+      <ul class="prows">${state.suggestions.map(suggestionHtml).join('')}</ul>`;
+  }
+
+  function suggestionHtml(s) {
+    const who = s.offered_by ? ` · suggested by builder ${escapeHtml(String(s.offered_by))}` : '';
+    return `
+      <li class="prow speciality-card" data-id="${escapeHtml(String(s.id))}">
+        <div class="prow__main">
+          <div class="prow__label">
+            <strong>${escapeHtml(s.name)}</strong>
+            <span class="speciality-craft">${escapeHtml(s.discipline)}</span>
+          </div>
+          <p class="prow__hint">${escapeHtml(s.summary || 'No summary yet.')}${who}</p>
+          ${contractHtml(s)}
+        </div>
+        <div class="prow__actions">
+          <button type="button" class="btn" data-act="adopt" data-id="${escapeHtml(String(s.id))}">Use this</button>
+        </div>
+      </li>`;
+  }
+
   function render() {
+    renderSuggestions();
     const host = el('specialities-rows');
     if (!host) return;
     if (!state.rows.length) {
@@ -138,8 +176,13 @@
 
   async function load() {
     try {
-      const [cat, mine] = await Promise.all([call('GET', '/specialities'), call('GET', '/me/specialities')]);
+      const [cat, mine, offered] = await Promise.all([
+        call('GET', '/specialities'),
+        call('GET', '/me/specialities'),
+        call('GET', '/specialities/offered'),
+      ]);
       if (!cat) return;
+      state.suggestions = (offered && offered.suggestions) || [];
       const activeIds = new Set(((mine && mine.adoptions) || []).filter((a) => a.active).map((a) => String(a.id)));
       state.rows = ((cat && cat.specialities) || []).map((s) => ({ ...s, active: activeIds.has(String(s.id)) }));
       render();
@@ -164,6 +207,17 @@
   }
 
   function wire() {
+    // The suggestions block reuses the same delegated handlers — an "adopt" from a
+    // suggestion and from the catalogue are the same act and must not drift.
+    const suggestHost = el('speciality-suggestions');
+    if (suggestHost) {
+      suggestHost.addEventListener('click', (e) => {
+        const btn = e.target.closest('button[data-act]');
+        if (!btn) return;
+        act(btn.dataset.act, btn.dataset.id);
+      });
+    }
+
     const host = el('specialities-rows');
     if (host) {
       host.addEventListener('click', (e) => {

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

@@ -284,6 +284,7 @@
                 Train it by adding things you have written; it also picks things up on its own
                 as you work.
               </p>
+              <div id="speciality-suggestions"></div>
               <ul class="prows" id="specialities-rows"></ul>
 
               <form id="speciality-create" class="speciality-create">

package/modules/lifecycle/routes/task-route-helpers.js

@@ -45,8 +45,50 @@ const VALID_FITS = new Set(['autonomous', 'parallel']);
 // whole-token, case-insensitive. The former scripts/gds/capture.js mirror was
 // deleted with the same task: captured ideas have no touches[] and the server
 // re-detects on create, so the CLI copy earned its keep no longer.
+// task 1003910: "schema" is QUALIFIED, not bare.
+//
+// WHAT ACTUALLY WENT WRONG. The bug was filed as "the non-DB compound strip does
+// not cover --json-schema". It does (the `json` arm below has always been there),
+// and the live false positive it cites — task 1003902, flagged though it alters no
+// table — was not triggered by the flag at all. It was triggered by a LATER,
+// unqualified back-reference in the same paragraph: "`result` is EMPTY when a
+// schema is in play". A bare noun, with no compound around it for any strip to
+// catch.
+//
+// WHY THE SHAPE CHANGED RATHER THAN THE LIST. A blocklist of non-database
+// compounds cannot be finished: every new way of saying "schema" about something
+// that is not a table is another false positive, and each one costs a reserved
+// migration number (task 1002614 burned 213), floors the task at Metic (ADR 0270)
+// and halts an unattended chain. Enumerating `json-schema`, `schema.org` and
+// `--output-schema` would have bought three phrasings and left the one in the bug
+// report still flagging. So "schema" now has to be ATTACHED to a database noun or
+// to a change verb to count — an allowlist, which fails the safe way: an
+// unrecognised phrasing no longer flags, instead of flagging on sight.
+//
+// The other three keywords are untouched: `migration(s)`, `alter table` and
+// `create table` are already unambiguous, and they are what catches genuine
+// schema work whose prose never says the word "schema".
+//
+// A missed detection stays correctable by PATCH, exactly as a false one was.
+const DB_SCHEMA_RE = [
+  // a database noun in front: "DB schema", "table-schema", "column schema"
+  /\b(?:db|database|table|column|postgres|pg|sql|index)[-\s]+schemas?\b/.source,
+  // a change noun behind: "schema change", "schema migration", "schema bump"
+  /\bschemas?[-\s]+(?:migrations?|changes?|bumps?|updates?|diffs?|drift)\b/.source,
+  // a change verb in front: "change the schema", "alter the table schema"
+  // The noun slot before "schema" excludes the API/transport nouns the old
+  // blocklist covered: a change verb is not evidence on its own, and "update the
+  // response schema" is the task 1002648 false-positive class wearing a verb.
+  // Anything else ("the task schema", "the tasks schema") still counts.
+  // The noun slot before "schema" excludes the API/transport nouns the old
+  // blocklist covered: a change verb is not evidence on its own, and "update the
+  // response schema" is the task 1002648 false-positive class wearing a verb.
+  // Anything else ("the task schema", "the tasks schema") still counts.
+  /\b(?:change|changing|alter|altering|update|updating|modify|modifying|extend|extending)\s+(?:the\s+)?(?:(?!(?:request|response|body|validation|patch|post|put|delete|json|openapi|api|error|output|input|result|payload|event)\b)\w+\s+)?schemas?\b/.source,
+].join('|');
+
 const MIGRATION_KEYWORD_RE =
-  /\bmigrations?\b|\bschema\b|\balter\s+table\b|\bcreate\s+table\b/i;
+  new RegExp([/\bmigrations?\b/.source, DB_SCHEMA_RE, /\balter\s+table\b/.source, /\bcreate\s+table\b/.source].join('|'), 'i');
 
 // task 1002648 (idea 1000642): compounds where the keyword is qualified by a
 // NON-database noun are refactor/API language, not schema work, and are blanked
@@ -59,8 +101,13 @@ const NON_DB_COMPOUND_RE = new RegExp(
   [
     /\b(?:gate|route|code)s?[-\s]+migrations?\b/.source,
     /\bmigrations?\s+of\s+(?:gate|route|code)s?\b/.source,
-    /\b(?:request|response|body|validation|patch|post|put|delete|json|openapi|api|error)[-\s]+schemas?\b/.source,
-    /:\d+[-\s]+schemas?\b/.source,
+    // The API-schema arms this list used to carry (`request schema`, `:751
+    // schema`, `json schema`) are GONE, not lost: they existed only to subtract
+    // from a bare-"schema" rule that no longer exists. "the PATCH schema declares
+    // 11 keys" has no database noun and no change verb, so it never reaches a
+    // strip — and neither does the phrasing they missed. Keeping them would leave
+    // two mechanisms answering the same question, which is how the list grew a
+    // gap in the first place.
   ].join('|'),
   'gi'
 );

package/modules/specialities/db.js

@@ -17,7 +17,8 @@ const exec = (executor) => executor || ((text, params) => pool.query(text, param
 
 const COLUMNS = `id, slug, name, discipline, summary, contract_md,
                  owner_kind, owner_builder_id, owner_project_id,
-                 visibility, sellable, created_by, created_at, updated_at`;
+                 visibility, sellable, offered, offered_by, offered_at,
+                 created_by, created_at, updated_at`;
 
 async function createSpeciality(row, executor) {
   const q = exec(executor);
@@ -225,8 +226,46 @@ async function taskDiscipline(taskId, executor) {
   return (r.rows[0] && r.rows[0].discipline) || null;
 }
 
+// Flip the recommendation. `offered_by` and `offered_at` are written together
+// with it so a suggestion always carries whose judgement it is — a builder
+// deciding whether to accept a standing instruction can see who stood behind it.
+// Clearing an offer clears the witness too rather than leaving a stale name on a
+// recommendation nobody is making any more.
+async function setOffered(id, { offered, builderId }, executor) {
+  const q = exec(executor);
+  const r = await q(
+    `UPDATE specialities_catalog
+        SET offered = $2,
+            offered_by = CASE WHEN $2 THEN $3::bigint ELSE NULL END,
+            offered_at = CASE WHEN $2 THEN now() ELSE NULL END,
+            updated_at = now()
+      WHERE id = $1
+      RETURNING ${COLUMNS}, offered, offered_by, offered_at`,
+    [id, !!offered, builderId || null],
+  );
+  return r.rows[0] || null;
+}
+
+// Everything on offer that this viewer could see at all. The RULE that decides
+// what is actually suggested is specialities.offeredTo — this is the cheap
+// prefilter, exactly as listVisibleTo is for the catalogue.
+async function listOffered({ builderId, projectIds = [], discipline = null }, executor) {
+  const q = exec(executor);
+  const params = [builderId, projectIds.length ? projectIds : [null]];
+  let sql = `SELECT ${COLUMNS}, offered, offered_by, offered_at
+               FROM specialities_catalog
+              WHERE offered
+                AND (visibility = 'public'
+                     OR (owner_kind = 'builder' AND owner_builder_id = $1)
+                     OR (owner_kind = 'project' AND visibility = 'project' AND owner_project_id = ANY($2)))`;
+  if (discipline) { params.push(discipline); sql += ` AND discipline = $${params.length}`; }
+  sql += ' ORDER BY discipline ASC, name ASC';
+  const r = await q(sql, params);
+  return r.rows;
+}
+
 module.exports = {
-  UPDATABLE, taskDiscipline,
+  UPDATABLE, taskDiscipline, setOffered, listOffered,
   createSpeciality, getSpeciality, listVisibleTo, updateSpeciality,
   addDocument, listDocuments, getDocument, deleteDocument,
   linkLearning, countTraining,