@bongos/core 1.24.79 → 1.24.81

package/modules/lifecycle/task-classifier.js

@@ -117,18 +117,18 @@ function classifyTaskKind(title, description) {
 // -------------------------------------------------------------------------
 //
 // Orthogonal to kind. Classifies the builder-interest axis (migration 086):
-//   engineer      — the prompter is reasoning about algorithms, infra, data
+//   artisan       — the prompter is reasoning about algorithms, infra, data
 //                   plumbing, dev tooling, scanners, schemas, integrations.
 //                   The person guiding the prompt brings software-engineering
 //                   experience.
-//   artist        — producing or shipping visual artifacts: sprite/tile
-//                   generation, palettes, walk animations, aesthetic passes.
-//   ideator       — thinking/writing/scoping: quest narrative, world/level
-//                   design, lore, briefs, research, naming.
+//   artist        — producing or shipping visual artifacts (sprite/tile
+//                   generation, palettes, walk animations, aesthetic passes)
+//                   and, since task 1005141, the old ideator's lane too:
+//                   thinking/writing/scoping, narrative, lore, briefs, naming.
 //   unclassified  — signals weak or balanced; hand-triage.
 //
 // The scoring engine first decides engineer-vs-creative; a creative verdict
-// is then split into artist/ideator by splitCreativeDiscipline().
+// is then split into artist/ui by splitCreativeDiscipline().
 //
 // Calibrated against Lars's hand-classifications of 25 tasks (mig 014
 // backfill, 2026-05-12). Key insights from those decisions:
@@ -148,11 +148,11 @@ function classifyTaskKind(title, description) {
 // summed and capped. The 0.4 description weight is what stops incidental
 // creative phrasing in long descriptions from flipping engineering tasks.
 
-// Migration 086 split the old "creative" lane into 'artist' (visual asset
-// production) and 'ideator' (thinking/writing/scoping) and renamed
-// 'engineering' → 'engineer'. The binary scoring engine below still decides
-// engineer-vs-creative; a creative verdict is then disambiguated into
-// artist/ideator/ui by splitCreativeDiscipline(). When that split is unclear,
+// Migration 086 split the old "creative" lane into artist (visual asset
+// production) and ideator (thinking/writing/scoping); task 1005141 folded the
+// ideator back into the artist craft and renamed engineer → artisan. The binary
+// scoring engine below still decides engineer-vs-creative; a creative verdict
+// is then disambiguated into artist/ui by splitCreativeDiscipline(). When that split is unclear,
 // it returns 'unclassified' (hand-triage) — matching the #615 backfill rule.
 //
 // Migration 176 (task 1376) added 'ui' — UI/interface DESIGN work (Figma,
@@ -160,7 +160,7 @@ function classifyTaskKind(title, description) {
 // pixel-art `artist` lane (ADR 0079): producing interface designs is a
 // distinct discipline from producing pixel-art assets, and routes to its own
 // /design work-mode (discipline-modes.json) instead of /paint.
-const VALID_DISCIPLINES = new Set(['engineer', 'artist', 'ideator', 'ui', 'unclassified']);
+const VALID_DISCIPLINES = new Set(['artisan', 'artist', 'ui', 'unclassified']);
 
 // The disciplines a WRITE may set (task 1002988). 'unclassified' is deliberately
 // absent: it is a triage state, not a craft, and a create that lands one produces
@@ -169,12 +169,18 @@ const VALID_DISCIPLINES = new Set(['engineer', 'artist', 'ideator', 'ui', 'uncla
 // ?discipline=unclassified filter keep resolving while the backfill runs.
 // 'ui' stays: ADR 0272 keeps it as a module-contributed discipline, and the owner
 // reconfirmed that on 2026-09-17 when abandoning task 1002987.
-const WRITABLE_DISCIPLINES = new Set(['engineer', 'artist', 'ideator', 'ui']);
+const WRITABLE_DISCIPLINES = new Set(['artisan', 'artist', 'ui']);
 
 // Mirrors the `kind` branch in the create routes: act on the classifier only when
 // it is at least half sure, otherwise treat it as no answer at all.
 const DISCIPLINE_CONFIDENCE_FLOOR = 0.5;
 
+// The one place this module reads an old craft spelling as today's (task 1005141):
+// engineer → artisan, ideator → artist. 'unclassified' and 'ui' pass through.
+function normalizeDisciplineValue(d) {
+  return typeof d === 'string' ? require('../../src/module-api').crafts.normalizeCraft(d) : d;
+}
+
 // Resolve the discipline for a task WRITE. Honours an explicit valid value,
 // otherwise INFERS from title + description + touches, and reports failure so the
 // caller can 400 — per the owner decision of 2026-09-17, a bare 400 on a missing
@@ -186,6 +192,8 @@ const DISCIPLINE_CONFIDENCE_FLOOR = 0.5;
 // have left the other quietly minting the very rows this task exists to remove.
 // Returns { discipline, inferred?, confidence? } or { error, unclassifiable? }.
 function resolveWriteDiscipline({ discipline, title, description, touches } = {}) {
+  // An old spelling ('engineer', 'ideator') is today's craft, not a bad one (task 1005141).
+  discipline = normalizeDisciplineValue(discipline);
   if (discipline != null && discipline !== 'unclassified') {
     return WRITABLE_DISCIPLINES.has(discipline) ? { discipline } : { error: 'bad_discipline' };
   }
@@ -228,10 +236,10 @@ const TASK_FIELD_BOUNDS = {
 const VOTABLE_TASK_STATUSES = new Set(['confirmed', 'shipped']);
 
 // Disambiguation signals for the creative lane (migration 086). Visual
-// art-production cues → artist; narrative / design-thinking / writing cues →
-// ideator. Scored by distinct-rule hit count over title+description, plus
-// touches[] path prefixes; the stronger side wins, ties / no-signal fall to
-// 'unclassified' for hand-triage.
+// art-production cues and narrative / design-thinking / writing cues are both
+// the artist craft since task 1005141 (the ideator folded into it); the two
+// lists stay separate so each reads as what it detects. Scored by distinct-rule
+// hit count over title+description, plus touches[] path prefixes.
 const ARTIST_SIGNALS = [
   /\bpixel[-\s]?art\b/i,
   /\btile-?sheets?\b/i,
@@ -278,23 +286,24 @@ const UI_PATH_PREFIXES = [
   'docs/design/',
 ];
 
-// Split a 'creative' verdict into 'artist' | 'ideator' | 'ui' | 'unclassified'.
-// Three sub-lanes are scored by distinct-rule hit count + touches[] hits; the
-// strongest single lane wins. A tie at the top (including 0-0-0) → hand-triage.
+// Split a 'creative' verdict into 'artist' | 'ui' | 'unclassified'.
+// The art and ideas signals are summed into ONE artist score (one craft since
+// task 1005141 — scoring them as two lanes would call an even art/ideas task a
+// tie and send it to hand-triage); it competes with the ui score. The stronger
+// lane wins; a tie (including 0-0) → hand-triage.
 function splitCreativeDiscipline(title, description, touches) {
   const text = `${title || ''} ${description || ''}`;
   const artist = ARTIST_SIGNALS.filter((re) => re.test(text)).length
-               + countTouchesHits(touches, ARTIST_PATH_PREFIXES);
-  const ideator = IDEATOR_SIGNALS.filter((re) => re.test(text)).length
-                + countTouchesHits(touches, IDEATOR_PATH_PREFIXES);
+               + countTouchesHits(touches, ARTIST_PATH_PREFIXES)
+               + IDEATOR_SIGNALS.filter((re) => re.test(text)).length
+               + countTouchesHits(touches, IDEATOR_PATH_PREFIXES);
   const ui = UI_SIGNALS.filter((re) => re.test(text)).length
            + countTouchesHits(touches, UI_PATH_PREFIXES);
   const ranked = [
     { d: 'artist', n: artist },
-    { d: 'ideator', n: ideator },
     { d: 'ui', n: ui },
   ].sort((a, b) => b.n - a.n);
-  // No signal anywhere, or the top two are tied → hand-triage.
+  // No signal anywhere, or the two lanes are tied → hand-triage.
   if (ranked[0].n === 0 || ranked[0].n === ranked[1].n) return 'unclassified';
   return ranked[0].d;
 }
@@ -326,7 +335,7 @@ const CREATIVE_TEXT_RULES = [
   // UI/interface DESIGN cues (mig 176 / task 1376) — route the creative verdict
   // to the `ui` sub-lane via splitCreativeDiscipline. Kept here (not engineering)
   // so "UI redesign of the hall" / "wireframe the checkout" land creative-first;
-  // splitCreativeDiscipline then picks `ui` over `artist`/`ideator`.
+  // splitCreativeDiscipline then picks `ui` over `artist`.
   { re: /\b(UI|UX)\s+(design|redesign|pass|polish|refresh|overhaul)\b/i,                          weight: 0.8 },
   { re: /\b(user\s+(interface|experience)|interface\s+design|wireframes?|mock-?ups?)\b/i,         weight: 0.8 },
   { re: /\b(figma|claude[-\s]?design)\b.*\b(redesign|design|layout|screen|mock-?up|wireframe)\b/i, weight: 0.8 },
@@ -503,7 +512,7 @@ function classifyTaskDiscipline(title, description, touches) {
         const d = splitCreativeDiscipline(titleText, descText, touches);
         return { discipline: d, confidence: d === 'unclassified' ? 0 : cT.max };
       }
-      return { discipline: 'engineer', confidence: eT.max };
+      return { discipline: 'artisan', confidence: eT.max };
     }
     // (b) Same max-weight (e.g. both 0.85). Combine title-rule count and
     // touches[] hit count into a single asymmetry score — a side that
@@ -517,7 +526,7 @@ function classifyTaskDiscipline(title, description, touches) {
         const d = splitCreativeDiscipline(titleText, descText, touches);
         return { discipline: d, confidence: d === 'unclassified' ? 0 : 0.6 + Math.min(0.3, cAsym * 0.05) };
       }
-      return { discipline: 'engineer', confidence: 0.6 + Math.min(0.3, -cAsym * 0.05) };
+      return { discipline: 'artisan', confidence: 0.6 + Math.min(0.3, -cAsym * 0.05) };
     }
     return { discipline: 'unclassified', confidence: 0 };
   }
@@ -541,7 +550,7 @@ function classifyTaskDiscipline(title, description, touches) {
     const d = splitCreativeDiscipline(titleText, descText, touches);
     return { discipline: d, confidence: d === 'unclassified' ? 0 : creative };
   }
-  return { discipline: 'engineer', confidence: engineering };
+  return { discipline: 'artisan', confidence: engineering };
 }
 
 module.exports = {
@@ -550,6 +559,7 @@ module.exports = {
   buildBlockerVendorRe,
   VALID_KINDS,
   VALID_DISCIPLINES,
+  normalizeDisciplineValue,
   WRITABLE_DISCIPLINES,
   resolveWriteDiscipline,
   VOTABLE_TASK_STATUSES,

package/modules/lifecycle/tests/cascade.mjs

@@ -41,7 +41,7 @@ const shipped = (over = {}) => ({
   version_id: 'BONGOS-V1',
   goal_id: '1000074',
   title: 'R09 · Reword the hall empty states',
-  discipline: 'engineer',
+  discipline: 'artisan',
   kind: 'feature',
   status: 'shipped',
   ...over,
@@ -51,7 +51,7 @@ const shipped = (over = {}) => ({
 const REVIEW_RULE = {
   key: 'test-artist-review',
   discipline: 'artist',
-  when: (t) => t.discipline === 'engineer',
+  when: (t) => t.discipline === 'artisan',
   title: (t) => `Review the copy that shipped in task ${t.id}`,
   body: (t) => `Look at what ${t.title} put in front of a person.`,
 };
@@ -195,7 +195,7 @@ await test('the production table is valid, and a duplicate key is refused', () =
 
 const shippedCopy = (over = {}) => ({
   id: '7001', version_id: 'BONGOS-V1', goal_id: '1000074',
-  title: 'Reword the hall empty states', discipline: 'engineer',
+  title: 'Reword the hall empty states', discipline: 'artisan',
   kind: 'feature', status: 'shipped', module_key: 'hall-ui', source: 'manual',
   ...over,
 });
@@ -262,8 +262,8 @@ await test('"copy or visuals" is a NARROW predicate — three ways in, and a def
   assert.equal(H(shippedCopy({ module_key: null, source: 'copy-proposal' })), true, 'a landed copy proposal');
   assert.equal(H(shippedCopy({ module_key: null, source: 'manual', discipline: 'artist' })), true, 'work an artist did');
   // The default is NO — a backend/infra ship is not copy/visual work.
-  assert.equal(H(shippedCopy({ module_key: 'lifecycle', discipline: 'engineer', source: 'manual' })), false);
-  assert.equal(H(shippedCopy({ module_key: null, discipline: 'engineer', source: 'manual' })), false);
+  assert.equal(H(shippedCopy({ module_key: 'lifecycle', discipline: 'artisan', source: 'manual' })), false);
+  assert.equal(H(shippedCopy({ module_key: null, discipline: 'artisan', source: 'manual' })), false);
   assert.equal(H(null), false);
 });
 
@@ -303,7 +303,7 @@ await test('the runner still hands the artist gate to the rules, and a reader th
 
 await test('a page file changed under modules/*/public/ is visual work whatever the label says', () => {
   const H = cascade.hasCopyOrVisualWork;
-  const unlabelled = shippedCopy({ module_key: null, discipline: 'engineer' });
+  const unlabelled = shippedCopy({ module_key: null, discipline: 'artisan' });
   assert.equal(H(unlabelled), false, 'the label alone says no — the gap task 1003480 fell through');
   for (const p of [
     'modules/hall-ui/public/goal-forest-lib.js',
@@ -316,7 +316,7 @@ await test('a page file changed under modules/*/public/ is visual work whatever
 
 await test('a backend file in a UI module is NOT visual work — page files only', () => {
   const H = cascade.hasCopyOrVisualWork;
-  const unlabelled = shippedCopy({ module_key: null, discipline: 'engineer' });
+  const unlabelled = shippedCopy({ module_key: null, discipline: 'artisan' });
   assert.equal(H(unlabelled, [
     'modules/hall-ui/module.json',
     'modules/public-landing/routes/hub.js',

package/modules/onboarding/founding-mode.js

@@ -55,6 +55,9 @@ const KEYS = Object.freeze({
   // When the planet came alive (founding switched off). With STARTED_AT it bounds the
   // window a builder must have joined in to carry the founding mark (task 1004424).
   ENDED_AT: 'founding.ended_at',
+  // When the owner skipped genesis instead of finishing it (genesis-home.js skipGenesis). Write-once, so the
+  // record says the planet came alive by skip, not by the five stages.
+  SKIPPED_AT: 'founding.skipped_at',
   // Genesis is OFF unless a project turns it on (owner, 2026-11-01: not polished enough, so every
   // project opens as the full hall). Read as ordinary 'true'; absent or anything else is OFF, which is
   // also what pauses a project that was already mid-founding. A demo, or a project born with genesis,

package/modules/onboarding/genesis-home.js

@@ -426,6 +426,26 @@ async function comeAlive(builderId, deps = {}) {
   return { ok: true };
 }
 
+// skipGenesis — the owner's way out of genesis from ANY stage (incl. decide), with no Board
+// Room sitting and no finished-tasks gate. Founding mode switches off exactly as comeAlive
+// does (ended_at stamped, the full hall opens), and the skip is recorded once in
+// founding.skipped_at with the stage it left, so the record shows it was skipped, not
+// finished. Genesis still happens once (spec D8): a project that is already alive refuses.
+// Filed genesis tasks stay on the board; nothing is deleted.
+async function skipGenesis(builderId, deps = {}) {
+  const s = seams(deps);
+  const ended = await s.get(fm.KEYS.ENDED_AT);
+  if (ended && ended.value) return { ok: false, reason: 'already_alive' };
+  const mode = await fm.foundingModeFor(modeDeps(s));
+  if (!mode) return { ok: false, reason: 'not_founding' };
+  const prior = await s.get(fm.KEYS.SKIPPED_AT);
+  if (!prior || !prior.value) {
+    await s.set(fm.KEYS.SKIPPED_AT, JSON.stringify({ at: (s.now || new Date()).toISOString(), stage: mode.stage }), builderId);
+  }
+  await fm.setFoundingMode({ active: false }, builderId, modeDeps(s));
+  return { ok: true, skipped: true, from_stage: mode.stage };
+}
+
 // onStagePassed — the `board.genesis_stage.passed` listener. Best-effort: the next read
 // reconciles anyway.
 async function onStagePassed(deps = {}) {
@@ -434,7 +454,7 @@ async function onStagePassed(deps = {}) {
 
 module.exports = {
   PLAN_KEY, STAGE_INFO, PROPER, DEMO_PEOPLE, DEMO_HOURS, intIn, parsePlan, shapeGenesis, demoMinutes, demoPlan,
-  readGenesis, startFounding, savePlan, applyGovernmentChoice, closeStage, comeAlive, onStagePassed,
+  readGenesis, startFounding, savePlan, applyGovernmentChoice, closeStage, comeAlive, skipGenesis, onStagePassed,
   // the founding-mode deps this file reads the mode with, for founding-birth.js (task 1004506)
   modeDepsFor: (deps) => modeDeps(seams(deps)),
 };

package/modules/onboarding/routes/onboarding.js

@@ -186,4 +186,6 @@ function mountGenesisRoutes(router) {
     }
   });
   router.post('/founding/alive', api.requireBuilder, owns, handle((req) => genesis.comeAlive(req.builder.id)));
+  // Skip genesis: the owner opens the full hall from any stage, no sitting, no gate.
+  router.post('/founding/skip', api.requireBuilder, owns, handle((req) => genesis.skipGenesis(req.builder.id)));
 }

package/modules/page-notes/notes.js

@@ -91,10 +91,11 @@ function referencedImageIds(bodyMd) {
 // the ideas are triaged like any other, and nothing here grants anything.
 const FENCE_LOOKALIKE_RE = /~{4,}[A-Za-z0-9-]*~{4,}/g;
 // Two writers share the split: the artist's feedback doc (the palette) and the
-// ideator's notes (the lightbulb, task 1004498). Only the framing words differ.
+// artist's notes (the lightbulb, task 1004498; the ideator folded into the artist in
+// task 1005141, so the second voice is named for what it writes). Only the framing words differ.
 const VOICES = {
   artist: { who: 'artist', what: 'feedback about one page of a web app', ask: 'separate change the artist asks for or problem they point at', label: 'ARTIST-FEEDBACK', doc: 'doc', split: 'feedback' },
-  ideator: { who: 'ideator', what: 'notes, written while looking at one page of a web app', ask: 'separate idea, change or problem the ideator raises', label: 'IDEATOR-NOTES', doc: 'notes', split: 'notes' },
+  notes: { who: 'artist', what: 'notes, written while looking at one page of a web app', ask: 'separate idea, change or problem the artist raises', label: 'IDEATOR-NOTES', doc: 'notes', split: 'notes' },
 };
 function buildSplitPrompt({ pageTitle, pageId, bodyMd, voice = 'artist', nonce = crypto.randomBytes(8).toString('hex') }) {
   const v = VOICES[voice] || VOICES.artist;

package/modules/page-notes/split.js

@@ -76,7 +76,7 @@ async function fileNote(note, { grade, capture, db, log } = {}) {
 // an empty list means nothing filed and the caller says so.
 async function fileIdeatorNotes(submit, { grade, capture, log, authorId, authorName } = {}) {
   const page = { page_id: submit.page_id, page_path: submit.page_path, page_title: submit.page_title, body_md: submit.body_md };
-  let ideas = await aiIdeas({ note: page, grade, voice: 'ideator', timeoutMs: IDEATOR_TIMEOUT_MS });
+  let ideas = await aiIdeas({ note: page, grade, voice: 'notes', timeoutMs: IDEATOR_TIMEOUT_MS });
   let splitBy = 'ai';
   if (!ideas) { ideas = [N.wholeDocIdea({ pageTitle: page.page_title, pageId: page.page_id, bodyMd: page.body_md })]; splitBy = 'whole'; }
   const filed = [];

package/modules/platform-identity/craft-rollup.js

@@ -5,15 +5,16 @@
 // A reporting instance sends, per craft the builder has EARNED credits in, that
 // craft's headline counts (src/bongos/auth-admission.js readCraftRollup):
 //
-//   engineer  { shipped, credits }
-//   artist    { pages_approved, credits }
-//   ideator   { filed, full_ideas, ratified, credits }
+//   artisan   { shipped, credits }
+//   artist    { pages_approved, filed, full_ideas, ratified, credits }
+//
+// (the old engineer / artist / ideator shape is folded in at ingest, foldLegacy)
 //
 // Two halves live here, next to each other so they cannot disagree on the shape:
 //
 //   sanitizeCrafts — what upsertActivity BINDS. The report is third-party
 //     self-report (ADR 0171 D7), so the shape and the caps are the hub's, not
-//     the sender's: only the three crafts, only the named counts, each a
+//     the sender's: only the two crafts, only the named counts, each a
 //     non-negative integer under a ceiling or null. Anything else is dropped.
 //     A count is never text, so nothing here can carry a title or wording.
 //
@@ -30,10 +31,36 @@
 'use strict';
 
 const CRAFT_FIELDS = Object.freeze({
-  engineer: Object.freeze(['shipped', 'credits']),
-  artist: Object.freeze(['pages_approved', 'credits']),
-  ideator: Object.freeze(['filed', 'full_ideas', 'ratified', 'credits']),
+  artisan: Object.freeze(['shipped', 'credits']),
+  artist: Object.freeze(['pages_approved', 'filed', 'full_ideas', 'ratified', 'credits']),
 });
+
+// A reporting instance on a core older than task 1005141 still sends the old
+// crafts: `engineer`, and `artist` + `ideator` as two blocks. They are folded
+// into today's shape BEFORE the hub's caps run, so an old report and a new one
+// land identically: engineer becomes artisan, and the ideator's counts join the
+// artist block with the two credit figures summed. A craft already reported under
+// its new name wins over an old-name block for the same craft.
+function foldLegacy(raw) {
+  const out = { ...raw };
+  if (out.engineer && !out.artisan) out.artisan = out.engineer;
+  delete out.engineer;
+  const ideator = out.ideator;
+  delete out.ideator;
+  if (ideator && typeof ideator === 'object' && !Array.isArray(ideator)) {
+    const art = out.artist && typeof out.artist === 'object' && !Array.isArray(out.artist) ? out.artist : {};
+    // Each half is capped BEFORE the sum: a hostile -3 on one side must not
+    // quietly shave the other side's honest count.
+    const a = cleanCount(art.credits);
+    const i = cleanCount(ideator.credits);
+    out.artist = {
+      ...art,
+      filed: ideator.filed, full_ideas: ideator.full_ideas, ratified: ideator.ratified,
+      credits: a == null && i == null ? null : (a || 0) + (i || 0),
+    };
+  }
+  return out;
+}
 // Far above any honest count, low enough that a sum over every project stays a
 // safe integer and a hostile report cannot dominate a profile by one number.
 const MAX_CRAFT_COUNT = 10_000_000;
@@ -45,6 +72,7 @@ function cleanCount(v) {
 function sanitizeCrafts(raw) {
   const out = {};
   if (!raw || typeof raw !== 'object' || Array.isArray(raw)) return out;
+  raw = foldLegacy(raw);
   for (const [craft, fields] of Object.entries(CRAFT_FIELDS)) {
     const block = raw[craft];
     if (!block || typeof block !== 'object' || Array.isArray(block)) continue;

package/modules/platform-identity/migrations/platform_identity_036_crafts_artisan_artist.sql

@@ -0,0 +1,45 @@
+-- platform_identity_036 — the hub's per-project activity rollup follows core_296's craft
+-- rename (task 1005141, BONGOS-V3 C1): engineer → artisan, ideator → artist.
+--
+-- `disciplines` is mapped and deduped. `crafts` (platform_identity_029) is keyed by
+-- craft: the engineer block becomes the artisan block, and the ideator block's counts
+-- (filed, full_ideas, ratified) merge into the artist block, with credits summed —
+-- the shape modules/platform-identity/craft-rollup.js now sanitizes. A report from an
+-- instance still on an older core is folded the same way at ingest, so these rows and
+-- fresh reports agree.
+
+BEGIN;
+
+UPDATE platform_identity_builder_project_activity a
+   SET disciplines = ARRAY(
+         SELECT m.craft FROM (
+           SELECT CASE d WHEN 'engineer' THEN 'artisan' WHEN 'ideator' THEN 'artist' ELSE d END AS craft,
+                  min(ord) AS first_seen
+             FROM unnest(a.disciplines) WITH ORDINALITY AS u(d, ord)
+            GROUP BY 1
+         ) m ORDER BY m.first_seen)
+ WHERE a.disciplines && ARRAY['engineer', 'ideator']::text[];
+
+UPDATE platform_identity_builder_project_activity
+   SET crafts = (crafts - 'engineer') || jsonb_build_object('artisan', crafts -> 'engineer')
+ WHERE crafts ? 'engineer' AND NOT crafts ? 'artisan';
+
+UPDATE platform_identity_builder_project_activity
+   SET crafts = (crafts - 'ideator') || jsonb_build_object('artist',
+         coalesce(crafts -> 'artist', '{}'::jsonb)
+         || jsonb_strip_nulls(jsonb_build_object(
+              'filed',      crafts -> 'ideator' -> 'filed',
+              'full_ideas', crafts -> 'ideator' -> 'full_ideas',
+              'ratified',   crafts -> 'ideator' -> 'ratified'))
+         || jsonb_build_object('credits',
+              CASE WHEN jsonb_typeof(crafts -> 'artist' -> 'credits') = 'number'
+                     OR jsonb_typeof(crafts -> 'ideator' -> 'credits') = 'number'
+                   THEN to_jsonb(coalesce((crafts -> 'artist' ->> 'credits')::bigint, 0)
+                               + coalesce((crafts -> 'ideator' ->> 'credits')::bigint, 0))
+                   ELSE 'null'::jsonb END))
+ WHERE crafts ? 'ideator';
+
+INSERT INTO schema_migrations (version) VALUES ('platform_identity_036_crafts_artisan_artist')
+ON CONFLICT (version) DO NOTHING;
+
+COMMIT;

package/modules/platform-identity/platform-identity.js

@@ -671,8 +671,8 @@ async function upsertActivity({ clientId, githubId, credits = 0, tasksShipped =
   // api.LIVE_RANK_LADDER is a published doorway export (src/module-api.js, a
   // lazy getter over db-kernel's canonical four-rank ladder).
   const safeRank = api.LIVE_RANK_LADDER.includes(rank) ? rank : null;
-  const disc = (Array.isArray(disciplines) ? disciplines : [])
-    .filter((d) => typeof d === 'string' && d.length > 0 && d.length <= MAX_DISCIPLINE_LENGTH)
+  const disc = api.crafts.normalizeCrafts((Array.isArray(disciplines) ? disciplines : []) // an older core's engineer/ideator → today's crafts, as foldLegacy (task 1005193)
+    .filter((d) => typeof d === 'string' && d.length > 0 && d.length <= MAX_DISCIPLINE_LENGTH))
     .slice(0, MAX_ROLLUP_DISCIPLINES);
   const ships = sanitizeRecentShips(recentShips, clientOrigin);
   // Each craft's own counts (task 1004440), shaped and capped here — see craft-rollup.js.

package/modules/platform-identity/routes/community-leaderboard.js

@@ -55,7 +55,8 @@ module.exports = function communityLeaderboardRoutes() {
     if (!leaderboard.isWindow(window)) {
       return res.fail('bad_window', { status: 400, message: `window must be one of: ${leaderboard.WINDOWS.join(', ')}` });
     }
-    const discipline = (typeof req.query.discipline === 'string' ? req.query.discipline : '').trim();
+    // An old link's engineer/ideator reads as today's craft (task 1005141).
+    const discipline = require('../../../src/module-api').crafts.normalizeCraft((typeof req.query.discipline === 'string' ? req.query.discipline : '').trim());
     if (discipline.length > leaderboard.MAX_DISCIPLINE_CHARS) {
       return res.fail('discipline_too_long', { status: 400, message: `discipline must be at most ${leaderboard.MAX_DISCIPLINE_CHARS} characters` });
     }

package/modules/platform-identity/routes/community-search.js

@@ -63,7 +63,8 @@ module.exports = function communitySearchRoutes() {
     if (q.length > builderSearch.MAX_QUERY_CHARS) {
       return res.fail('query_too_long', { status: 400, message: `q must be at most ${builderSearch.MAX_QUERY_CHARS} characters` });
     }
-    const discipline = (typeof req.query.discipline === 'string' ? req.query.discipline : '').trim();
+    // An old link's engineer/ideator reads as today's craft (task 1005141).
+    const discipline = require('../../../src/module-api').crafts.normalizeCraft((typeof req.query.discipline === 'string' ? req.query.discipline : '').trim());
     if (discipline.length > builderSearch.MAX_DISCIPLINE_CHARS) {
       return res.fail('discipline_too_long', { status: 400, message: `discipline must be at most ${builderSearch.MAX_DISCIPLINE_CHARS} characters` });
     }

package/modules/platform-identity/routes/sso.js

@@ -691,7 +691,8 @@ module.exports = function ssoRoutes() {
         tasksShipped: b.tasks_shipped,
         karma: b.karma,
         rank: b.rank || null,
-        disciplines: Array.isArray(b.disciplines) ? b.disciplines : [],
+        // An instance on an older core still reports engineer/ideator (task 1005141).
+        disciplines: Array.isArray(b.disciplines) ? api.crafts.normalizeCrafts(b.disciplines) : [],
         recentShips: b.recent_ships,
         crafts: b.crafts,
         // The AUTHENTICATED client's registered origin — the bound every pushed

package/modules/platform-identity/tests/platform-identity.mjs

@@ -606,14 +606,14 @@ const upsertCall = (pool) => pool.calls.find((c) => /INSERT INTO platform_identi
 
 await ta('upsertActivity upserts per (client_id, github_id) with the talent snapshot', async () => {
   const pool = activityPool();
-  await P.upsertActivity({ clientId: 'c1', githubId: 42, credits: 100, tasksShipped: 5, karma: 3, rank: 'metic', disciplines: ['engineer'] }, { pool });
+  await P.upsertActivity({ clientId: 'c1', githubId: 42, credits: 100, tasksShipped: 5, karma: 3, rank: 'metic', disciplines: ['artisan'] }, { pool });
   const c = upsertCall(pool);
   assert.match(c.text, /ON CONFLICT \(client_id, github_id\) DO UPDATE/);
   // recent_ships (task 1002803) rides the same row as a jsonb STRING param —
   // an empty list when the sender reported none, never NULL. Its own cap,
   // shape and link bound are covered by tests/hub_recent_ships.mjs. The last param
   // is the report's as-of stamp (task 1004268): null when the sender gave none.
-  assert.deepEqual(c.params, ['c1', 42, 100, 5, 3, 'metic', ['engineer'], '[]', null, '{}']);
+  assert.deepEqual(c.params, ['c1', 42, 100, 5, 3, 'metic', ['artisan'], '[]', null, '{}']);
 });
 
 await ta('upsertActivity sanitizes third-party self-report (ADR 0171 D7): off-ladder rank → null, disciplines validated + capped', async () => {
@@ -622,7 +622,7 @@ await ta('upsertActivity sanitizes third-party self-report (ADR 0171 D7): off-la
     clientId: 'c1', githubId: 42,
     rank: 'Supreme Overlord', // not one of the four core rank keys → stored null
     disciplines: [
-      'engineer', 42, null, { evil: true }, // non-strings dropped
+      'artisan', 42, null, { evil: true }, // non-strings dropped
       'x'.repeat(33),                        // over-length dropped
       'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', // valid tail — capped to 8 total
     ],
@@ -633,7 +633,7 @@ await ta('upsertActivity sanitizes third-party self-report (ADR 0171 D7): off-la
   assert.ok(disc.length <= 8, `disciplines capped at 8 (got ${disc.length})`);
   assert.ok(disc.every((d) => typeof d === 'string' && d.length > 0 && d.length <= 32),
     'only bounded string elements survive');
-  assert.equal(disc[0], 'engineer', 'valid elements pass through in order');
+  assert.equal(disc[0], 'artisan', 'valid elements pass through in order');
 });
 
 await ta('upsertActivity keeps a legitimate core rank key', async () => {
@@ -733,7 +733,7 @@ await ta('getRollupForBuilder guards non-numeric github_id BEFORE any query (no
 
 await ta('getRollupForBuilder aggregates un-fanned + names only publicly-catalogued projects', async () => {
   const pool = mockPool([
-    { rows: [{ projects: 2, credits: 150, works_shipped: 7, karma: 4, last_active: 'T', disciplines: ['engineer', 'ui'] }] },
+    { rows: [{ projects: 2, credits: 150, works_shipped: 7, karma: 4, last_active: 'T', disciplines: ['artisan', 'ui'] }] },
     { rows: [
       { name: 'Public Proj', origin: 'https://pub.example', rank_held: 'metic', is_public: true },
       { name: 'Secret Proj', origin: 'https://sec.example', rank_held: 'archon', is_public: false },
@@ -741,7 +741,7 @@ await ta('getRollupForBuilder aggregates un-fanned + names only publicly-catalog
   ]);
   const out = await P.getRollupForBuilder(42, { pool });
   assert.deepEqual(out.totals, { projects: 2, credits: 150, works_shipped: 7, karma: 4 });
-  assert.deepEqual(out.disciplines, ['engineer', 'ui']);
+  assert.deepEqual(out.disciplines, ['artisan', 'ui']);
   assert.equal(out.projects.length, 1, 'only the public catalog row is named');
   // recent_ships rides the project row it belongs to (task 1002803), so a work
   // title can only surface where the project name already may.

package/modules/public-landing/public/community.html

@@ -262,9 +262,8 @@
       <div class="commFilters">
         <div class="commTabs" id="craftTabs" role="group" aria-label="Role">
           <button class="commTab" type="button" data-craft="" aria-pressed="true">Everyone</button>
-          <button class="commTab" type="button" data-craft="engineer" aria-pressed="false">Engineers</button>
+          <button class="commTab" type="button" data-craft="artisan" aria-pressed="false">Artisans</button>
           <button class="commTab" type="button" data-craft="artist" aria-pressed="false">Artists</button>
-          <button class="commTab" type="button" data-craft="ideator" aria-pressed="false">Ideators</button>
           <button class="commTab" type="button" data-craft="ui" aria-pressed="false">Designers</button>
         </div>
         <div class="commWhen">
@@ -314,9 +313,8 @@
         <label class="vh" for="findCraft">Role</label>
         <select id="findCraft">
           <option value="">Any role</option>
-          <option value="engineer">Engineers</option>
+          <option value="artisan">Artisans</option>
           <option value="artist">Artists</option>
-          <option value="ideator">Ideators</option>
           <option value="ui">Designers</option>
         </select>
         <button id="findBtn" type="submit" class="btn small">Search</button>
@@ -365,7 +363,7 @@
   var WINDOWS = ['7d', '30d', '365d', 'all'];
   var QUERY_MIN = 2;
   var QUERY_MAX = 100;
-  var CRAFT_WORD = { engineer: 'engineer', artist: 'artist', ideator: 'ideator', ui: 'designer' };
+  var CRAFT_WORD = { artisan: 'artisan', artist: 'artist', ui: 'designer' };
   // The five sub-pages this file is served as (platform-server.js's /community routes);
   // the path picks one, and its menu items are marked current. The home adds no title.
   var VIEWS = {

package/modules/public-landing/public/index.html

@@ -774,7 +774,7 @@ composition A; landing composition v3, picked by the owner 2026-08-27. No roll,
     <div class="layer">
       <h1 data-brand-tagline><span class="l1">Building Digitally,</span><span class="l2">as simple as Bongos.</span></h1>
 
-      <p class="lede">Bringing AI to the builder, ideator, and artist in everyone.</p>
+      <p class="lede">Bringing AI to the artisan and the artist in everyone.</p>
 
       <div class="doors">
         <div class="doorbar">

package/modules/public-landing/public/projects.html

@@ -1389,7 +1389,7 @@ summary{min-height:24px;padding:3px 0;}
       <div class="card mSec" id="mInvite">
         <h2>Invite a builder</h2>
         <div id="mInv">
-          <p class="finePrint">By GitHub username. The invite is sent from your project’s own hall — that is the invite that lets them in — and they see it on their projects page here too.</p>
+          <p class="finePrint">By GitHub username. The invite is sent from your project’s own hall — that is the invite that lets them in — and they see it on their projects page here too. The hall’s invite box can also invite them to the project’s GitHub repository, which is a separate invite they accept on GitHub.</p>
           <div class="attach">
             <label class="vh" for="mInviteWho">GitHub username</label>
             <input type="text" id="mInviteWho" placeholder="GitHub username" autocomplete="off" autocapitalize="off" spellcheck="false" maxlength="200">

package/modules/specialities/migrations/specialities_004_crafts_artisan_artist.sql

@@ -0,0 +1,30 @@
+-- specialities_004 — specialities follow core_296's craft rename (task 1005141,
+-- BONGOS-V3 C1): engineer → artisan, ideator → artist.
+--
+-- A speciality's discipline is immutable to its OWNER (the update path refuses a
+-- change); this is a vocabulary rename, not a move between crafts, so it rewrites the
+-- catalog and the adoption copy together.
+--
+-- The fold has one collision: specialities_adoptions allows ONE active adoption per
+-- builder per craft. A builder with an active artist speciality AND an active ideator
+-- one would hold two artist adoptions after the rename, so the ideator adoption is
+-- deactivated first (deactivating is how a builder switches; nothing is deleted, and
+-- they can re-adopt it from their speciality page).
+
+BEGIN;
+
+UPDATE specialities_adoptions i
+   SET active = FALSE
+ WHERE i.active AND i.discipline = 'ideator'
+   AND EXISTS (SELECT 1 FROM specialities_adoptions a
+                WHERE a.builder_id = i.builder_id AND a.active AND a.discipline = 'artist');
+
+UPDATE specialities_catalog   SET discipline = 'artisan' WHERE discipline = 'engineer';
+UPDATE specialities_catalog   SET discipline = 'artist'  WHERE discipline = 'ideator';
+UPDATE specialities_adoptions SET discipline = 'artisan' WHERE discipline = 'engineer';
+UPDATE specialities_adoptions SET discipline = 'artist'  WHERE discipline = 'ideator';
+
+INSERT INTO schema_migrations (version) VALUES ('specialities_004_crafts_artisan_artist')
+ON CONFLICT (version) DO NOTHING;
+
+COMMIT;