@bongos/core 1.24.79 → 1.24.80

package/modules/ideas/inbox.js

@@ -396,9 +396,9 @@ async function routeIdeaToGoal({
     //    default — createTask auto-assigns the capped suggestion) and
     //    `discipline` — NO LONGER 'unclassified' (task 1002988). createTask refuses a
     //    craft it cannot infer; refusing a filer is the wrong trade for an INBOX, so we
-    //    ask the same question and fall back to 'engineer' (free to re-triage; a lost capture is not).
+    //    ask the same question and fall back to 'artisan' (free to re-triage; a lost capture is not).
     const task = await lifecycle.createTask({
-      discipline: (lifecycle.resolveWriteDiscipline?.({ title, description: bodyMd, touches: declaredTouches })?.discipline) || 'engineer',
+      discipline: (lifecycle.resolveWriteDiscipline?.({ title, description: bodyMd, touches: declaredTouches })?.discipline) || 'artisan',
       versionId: goal.version_id,
       title,
       description: bodyMd,

package/modules/lifecycle/activity.js

@@ -45,7 +45,7 @@ const { creditsEarnedBySql } = require('./db-tasks.js');
 // (migrations 086 + 176). Anything else is treated as "no filter" rather than
 // 400ing the read: a caller naming a craft that does not exist gets the honest
 // full list, not an error card in the middle of a profile.
-const CRAFTS = ['engineer', 'artist', 'ideator', 'ui', 'unclassified'];
+const CRAFTS = ['artisan', 'artist', 'ui', 'unclassified'];
 
 // One page of an activity list. Clamped here as well as at the route, because
 // these are the functions a CLI or a later surface will call directly.

package/modules/lifecycle/cascade.js

@@ -71,8 +71,8 @@
 //   key         string, stable + unique. It is half the source_ref, so RENAMING
 //               a key re-files every task the rule ever generated. Treat it as
 //               permanent.
-//   discipline  the role the generated work is FOR ('artist', 'ideator',
-//               'engineer'). This is the whole point of the mechanism: the
+//   discipline  the role the generated work is FOR ('artist' or
+//               'artisan'). This is the whole point of the mechanism: the
 //               discipline of the generated task is not the discipline of the
 //               ship that caused it.
 //   when(task, ctx)  a predicate over the shipped task row and the run CONTEXT.
@@ -216,7 +216,7 @@ const CASCADES = Object.freeze([
     key: REMOVAL_RULE_KEY,
     // ENGINEER, not artist: confirming asks whether deleted code is genuinely
     // gone or was moved, which is a question about the diff.
-    discipline: 'engineer',
+    discipline: 'artisan',
     kind: 'unclassified',
     // The anti-recursion guard FIRST: a cascade-generated task must never
     // cascade, so it is rejected before any other test runs.
@@ -250,7 +250,7 @@ const TRIGGERS = Object.freeze({
 const IDEA_CASCADES = Object.freeze([
   Object.freeze({
     key: 'engineer-scoping-on-full-idea',
-    discipline: 'engineer',
+    discipline: 'artisan',
     kind: 'unclassified',
     // A Full Idea is the ideator's hand-off point: the five questions are
     // answered and the thing is described well enough to cost. What it still

package/modules/lifecycle/goal-spec-task.js

@@ -50,7 +50,7 @@ function specTaskFields(goal, { createdBy = null } = {}) {
     ].join('\n'),
     status: 'ready',
     kind: 'spike',
-    discipline: 'ideator',
+    discipline: 'artist',
     source: SPEC_SOURCE,
     sourceRef: specSourceRef(id),
     createdBy,

package/modules/lifecycle/help-requests.js

@@ -14,19 +14,22 @@
 
 'use strict';
 
-const { pool } = require('../../src/module-api');
+const { pool, crafts: craftVocab } = require('../../src/module-api');
 
-// ADR 0174's crafts. Note the vocabulary seam: `builders.preferred_disciplines`
-// spells this same craft 'engineer', not 'builder'. THIS FUNCTION IS THE ONE PLACE
-// that knows the mapping — the matching surface (C02) must consume it rather than
-// re-deriving, or the two spellings drift and a builder silently matches nothing.
-const HELP_CRAFTS = Object.freeze(['artist', 'builder', 'ideator', 'ui']);
-const CRAFT_TO_DISCIPLINE = Object.freeze({ artist: 'artist', ideator: 'ideator', builder: 'engineer', ui: 'ui' });
+// The crafts a request can be addressed to: the two working characters plus the
+// module-contributed `ui`. Since task 1005141 (BONGOS-V3) the help-request spelling
+// and the discipline spelling are THE SAME WORD — the old seam where this craft was
+// 'builder' here and 'engineer' on a builder is closed. The maps stay, because they
+// are also where an OLD spelling is read as today's: a rolled-back release can still
+// write 'engineer'/'ideator' into preferred_disciplines, and craftMapSql() below runs
+// against that column.
+const HELP_CRAFTS = Object.freeze(['artisan', 'artist', 'ui']);
+const CRAFT_TO_DISCIPLINE = Object.freeze({ artisan: 'artisan', artist: 'artist', ui: 'ui' });
 
-const DISCIPLINE_TO_CRAFT = Object.freeze({ artist: 'artist', ideator: 'ideator', engineer: 'builder', ui: 'ui' });
+const DISCIPLINE_TO_CRAFT = Object.freeze({ artisan: 'artisan', artist: 'artist', ui: 'ui', engineer: 'artisan', ideator: 'artist' });
 
 function helpCrafts() { return [...HELP_CRAFTS]; }
-function disciplineForCraft(craft) { return CRAFT_TO_DISCIPLINE[craft] || null; }
+function disciplineForCraft(craft) { return CRAFT_TO_DISCIPLINE[craftVocab.normalizeCraft(craft)] || null; }
 function craftForDiscipline(d) { return DISCIPLINE_TO_CRAFT[d] || null; }
 
 // The caller's OWN crafts, read from builders.preferred_disciplines. This exists
@@ -90,7 +93,8 @@ function capRows(n) {
 function validateHelpRequest(input) {
   const errors = [];
   const b = input || {};
-  const craft = b.needs_craft == null ? null : String(b.needs_craft);
+  // An old spelling ('builder', 'engineer', 'ideator') is today's craft, not an unknown one.
+  const craft = b.needs_craft == null ? null : craftVocab.normalizeCraft(String(b.needs_craft));
   const person = b.needs_builder_id == null ? null : Number(b.needs_builder_id);
 
   if (craft === null && person === null) {
@@ -133,7 +137,7 @@ async function createHelpRequest(input, deps = {}) {
        (requested_by, needs_craft, needs_builder_id, what_is_stuck, task_id, goal_id)
      VALUES ($1, $2, $3, $4, $5, $6)
      RETURNING ${COLS}`,
-    [input.requestedBy, input.needsCraft ?? null, input.needsBuilderId ?? null,
+    [input.requestedBy, input.needsCraft == null ? null : craftVocab.normalizeCraft(String(input.needsCraft)), input.needsBuilderId ?? null,
       String(input.whatIsStuck).trim(), input.taskId ?? null, input.goalId ?? null]
   );
   return rows[0];
@@ -150,7 +154,7 @@ async function createHelpRequest(input, deps = {}) {
 // EXCEPT its author, since answering your own request is not help.
 async function listOpenForBuilder({ builderId, crafts = [], limit = MAX_ROWS }, deps = {}) {
   const activePool = deps.pool || pool;
-  const safeCrafts = (Array.isArray(crafts) ? crafts : []).filter((c) => HELP_CRAFTS.includes(c));
+  const safeCrafts = craftVocab.normalizeCrafts(Array.isArray(crafts) ? crafts : []).filter((c) => HELP_CRAFTS.includes(c));
   const { rows } = await activePool.query(
     `SELECT ${LIST_COLS} ${LIST_FROM}
       WHERE h.status = 'open'
@@ -193,7 +197,7 @@ async function countOpenForBuilder(builderId, deps = {}) {
 }
 
 // The DISCIPLINE_TO_CRAFT map as a SQL CASE, generated from the same object the
-// JS path uses — so the two can never disagree about what an 'engineer' is.
+// JS path uses — so the two can never disagree about what an 'artisan' is.
 function craftMapSql(col) {
   const whens = Object.entries(DISCIPLINE_TO_CRAFT)
     .map(([d, c]) => `WHEN '${d}' THEN '${c}'`).join(' ');
@@ -293,7 +297,7 @@ async function settleFromLedger(deps = {}) {
 // The author exclusion stays too: your own settled asks are the OTHER list.
 async function listArchivedForBuilder({ builderId, crafts = [], limit = MAX_ROWS }, deps = {}) {
   const activePool = deps.pool || pool;
-  const safeCrafts = (Array.isArray(crafts) ? crafts : []).filter((c) => HELP_CRAFTS.includes(c));
+  const safeCrafts = craftVocab.normalizeCrafts(Array.isArray(crafts) ? crafts : []).filter((c) => HELP_CRAFTS.includes(c));
   const { rows } = await activePool.query(
     `SELECT ${LIST_COLS} ${LIST_FROM}
       WHERE h.status <> 'open'
@@ -350,7 +354,7 @@ async function getHelpRequest(id, deps = {}) {
 // closing an ask is not accepting work, it is ending a question.
 async function setHelpRequestStatus({ id, builderId, status, crafts = [] }, deps = {}) {
   const activePool = deps.pool || pool;
-  const safeCrafts = (Array.isArray(crafts) ? crafts : []).filter((c) => HELP_CRAFTS.includes(c));
+  const safeCrafts = craftVocab.normalizeCrafts(Array.isArray(crafts) ? crafts : []).filter((c) => HELP_CRAFTS.includes(c));
   const askerOnly = status === 'withdrawn';
   const { rows } = await activePool.query(
     `UPDATE lifecycle_help_requests

package/modules/lifecycle/kickoff-checklist.js

@@ -59,11 +59,11 @@ const MERGE_GATE = {
 
 const INSTANCE_LOOK = {
   title: 'Fill the instance look with your own LLM',
-  // 'ideator' is a DISCIPLINE, not a kind — tasks_kind_check rejects it as a
+  // 'artist' is a DISCIPLINE, not a kind — tasks_kind_check rejects it as a
   // kind and the task silently vanished from the checklist (the one non-infra
   // kickoff step). Deciding the instance look is a decision by an ideator.
   kind: 'decision',
-  discipline: 'ideator',
+  discipline: 'artist',
   description: 'init wrote a lean config/branding.json and left the LOOK to you (a locked project rule: brand with your ' +
     'own LLM, not a generator). Hand docs/branding-fill-prompt.md to your AI to design a palette/fonts/wordmark, or keep the vanilla look.',
 };
@@ -178,7 +178,7 @@ function genesisChecklist({ existing = false } = {}) {
     // The ref names the step, never its position, so the brownfield step slotting in
     // ahead of scope's own cannot re-key it into a duplicate.
     out[stage] = list.map((t) => ({
-      ...t, kind: 'decision', discipline: 'ideator', ref: t === GENESIS_EXISTING_FIRST ? 'genesis:scope:existing' : `genesis:${stage}`,
+      ...t, kind: 'decision', discipline: 'artist', ref: t === GENESIS_EXISTING_FIRST ? 'genesis:scope:existing' : `genesis:${stage}`,
     }));
   }
   return out;

package/modules/lifecycle/kickoff-seed.js

@@ -57,7 +57,7 @@ async function seedKickoffIfEmpty({ builderId = null } = {}, deps = {}) {
       description: t.description || '',
       status: 'ready',
       kind: t.kind || 'infra',
-      discipline: t.discipline || 'engineer',
+      discipline: t.discipline || 'artisan',
       source: 'first-sign-in',
       createdBy: builderId,
       // BV1.R11 (task 1003598): this runs on an instance with zero tasks and, in

package/modules/lifecycle/migrations/lifecycle_021_crafts_artisan_artist.sql

@@ -0,0 +1,29 @@
+-- lifecycle_021_crafts_artisan_artist.sql — task 1005141 (BONGOS-V3 C1): the lifecycle
+-- module's craft columns follow core_296's rename (engineer → artisan, ideator → artist).
+--
+-- lifecycle_help_requests.needs_craft spelled the engineering craft 'builder' (ADR 0174);
+-- that seam closes here — it is 'artisan' like everywhere else. The CHECK is a superset
+-- of old and new words so a rolled-back release can still write (see core_296).
+--
+-- lifecycle_mingle_pairs records the two crafts AS MATCHED, for audit, under a
+-- cross-craft CHECK, in the help-request spelling. builder → artisan is renamed. An artist × ideator pair is left
+-- exactly as recorded: the fold would make it artist × artist, which is both a false
+-- record of why it was matched and a CHECK violation. The code normalizes on read.
+
+BEGIN;
+
+ALTER TABLE lifecycle_help_requests DROP CONSTRAINT IF EXISTS lifecycle_help_requests_craft_chk;
+UPDATE lifecycle_help_requests SET needs_craft = 'artisan' WHERE needs_craft IN ('builder', 'engineer');
+UPDATE lifecycle_help_requests SET needs_craft = 'artist'  WHERE needs_craft = 'ideator';
+ALTER TABLE lifecycle_help_requests ADD CONSTRAINT lifecycle_help_requests_craft_chk
+  CHECK (needs_craft IS NULL OR needs_craft IN ('artisan', 'artist', 'patron', 'ui', 'builder', 'ideator'));
+
+UPDATE lifecycle_mingle_pairs SET craft_lo = 'artisan'
+ WHERE craft_lo IN ('builder', 'engineer') AND craft_hi <> 'artisan';
+UPDATE lifecycle_mingle_pairs SET craft_hi = 'artisan'
+ WHERE craft_hi IN ('builder', 'engineer') AND craft_lo <> 'artisan';
+
+INSERT INTO schema_migrations (version) VALUES ('lifecycle_021_crafts_artisan_artist')
+ON CONFLICT (version) DO NOTHING;
+
+COMMIT;

package/modules/lifecycle/mingle.js

@@ -38,7 +38,7 @@
 //
 // THE CRAFT VOCABULARY IS NOT REDEFINED HERE. help-requests.js declares itself the
 // one place that knows how a craft maps to `builders.preferred_disciplines`
-// ('builder' the craft is 'engineer' the discipline), so this file consumes
+// (and reads an old 'engineer'/'ideator' discipline as today's), so this file consumes
 // `craftMapSql` / `craftForDiscipline` from it rather than re-deriving. Two
 // spellings of the same craft would silently match nobody.
 //
@@ -67,14 +67,14 @@ const MAX_ROWS = 100;
 const MATCHING = Object.freeze({
   rule: 'cross_craft',
   description:
-    'A pair is two builders whose crafts do not overlap at all — an engineer with '
-    + 'an artist or an ideator, never two of the same. The premise is that the '
+    'A pair is two builders whose crafts do not overlap at all — an artisan with '
+    + 'an artist or a UI designer, never two of the same. The premise is that the '
     + 'person you need is often not another one of you.',
   // Stated because it is the road not taken, and an unexplained absence reads as
   // an oversight to the next person holding the matcher.
   rejected: 'same_craft — it duplicates the craft-matched Collab request path (criterion collab-page-matches-my-craft).',
   // DISJOINT, not merely "different". A builder who is both an artist and an
-  // engineer shares a craft with every engineer, so pairing them would put two of
+  // artisan shares a craft with every artisan, so pairing them would put two of
   // the same in a room and call it cross-craft.
   overlap: 'none',
   consent: 'Both switches required: the project enables mingle AND the builder opts in. Either off means no pair.',

package/modules/lifecycle/page-tweak-write.js

@@ -161,7 +161,7 @@ async function submitPageTweak({ taskId, builderId, versionId = null, freeze },
         // page does not wait for it.
         status: 'backlog',
         kind: 'feature',
-        discipline: 'engineer',
+        discipline: 'artisan',
         priority: 4,
         touches: u.touches || [],
         source: u.source,

package/modules/lifecycle/routes/task-write-routes.js

@@ -23,7 +23,7 @@ const hierarchy = require('../hierarchy');
 const { suggestCriteria } = require('../criterion-suggest');
 const { buildGoalAdvisory } = require('../goal-advisory');
 const { buildDependencyAdvisory } = require('../dependency-advisory');
-const { classifyTaskKind, VALID_KINDS, WRITABLE_DISCIPLINES, resolveWriteDiscipline, VOTABLE_TASK_STATUSES, TASK_FIELD_BOUNDS } = require('../task-classifier');
+const { classifyTaskKind, VALID_KINDS, WRITABLE_DISCIPLINES, resolveWriteDiscipline, VOTABLE_TASK_STATUSES, TASK_FIELD_BOUNDS, normalizeDisciplineValue } = require('../task-classifier');
 const { isModuleKey, moduleKeys } = api.moduleScopeMap;
 const { validateOrRespond, LIMITS, parseId } = api;
 const { badKind, badNeeds, detectsMigration } = require('./task-route-helpers.js');
@@ -438,6 +438,8 @@ module.exports = function registerTaskWriteRoutes(router) {
     const hasParent = Object.prototype.hasOwnProperty.call(body, 'parent_task_id');
     const hasKind = Object.prototype.hasOwnProperty.call(body, 'kind');
     const hasDiscipline = Object.prototype.hasOwnProperty.call(body, 'discipline');
+    // An old craft spelling is today's craft (task 1005141): written as the new word.
+    if (hasDiscipline) body.discipline = normalizeDisciplineValue(body.discipline);
     // V3.R15 #239: newcomer_friendly is the canonical name; xenos_claimable is
     // the pre-rename alias and still accepted. Either presence triggers the
     // same update path.

package/modules/lifecycle/routes/tasks.js

@@ -63,7 +63,7 @@ const NO_PR_STATUS = Object.freeze({
   pr_state: 'none', merged: false, merge_sha: null, deploy_status: 'none', pr_url: null, pr_number: null, pr_title: null, pr_body: null,
   branch_hint: 'No pull request titled for this task is among the 100 most recent. Pass ?branch=<name> to ask about a specific branch.' });
 const seams = api;
-const { classifyTaskKind, VALID_KINDS, VALID_DISCIPLINES, VOTABLE_TASK_STATUSES, TASK_FIELD_BOUNDS } = require('../task-classifier');
+const { classifyTaskKind, VALID_KINDS, VALID_DISCIPLINES, VOTABLE_TASK_STATUSES, TASK_FIELD_BOUNDS, normalizeDisciplineValue } = require('../task-classifier');
 const { isModuleKey, moduleKeys } = api.moduleScopeMap;
 const { validateOrRespond, LIMITS, parseId } = api;
 // The onboarding state machine + the evergreen newcomer-chore restock carved into
@@ -146,7 +146,7 @@ module.exports = function buildTasksRouter() {
   const CREATED_BY_MSG = "created_by must be a positive builder id, or 'none' for the unattributed.";
 
   router.get('/tasks', auth.requireBuilder, async (req, res) => {
-    const { version, status, kind, discipline, goal_id } = req.query;
+    const { version, status, kind, goal_id } = req.query; const discipline = normalizeDisciplineValue(req.query.discipline); // old craft spellings read as today's (task 1005141)
     if (kind && !VALID_KINDS.has(kind)) {
       return badKind(res);
     }
@@ -203,7 +203,7 @@ module.exports = function buildTasksRouter() {
   router.get('/tasks/claimable',
     auth.requireBuilder,
     async (req, res) => {
-    const { version, discipline, fit } = req.query;
+    const { version, fit } = req.query; const discipline = normalizeDisciplineValue(req.query.discipline);
     if (discipline && !VALID_DISCIPLINES.has(discipline)) {
       return res.fail('bad_discipline', 400, { valid: Array.from(VALID_DISCIPLINES) });
     }

package/modules/lifecycle/ship-card.js

@@ -223,9 +223,8 @@ function fmtMins(n) {
 // "Open the work" button firing a slash command that no longer resolves. A
 // module-contributed discipline still routes to its skill ('ui' -> /design).
 const DISCIPLINE_PACK = {
-  engineer: 'docs/packs/engineer.md',
+  artisan: 'docs/packs/artisan.md',
   artist: 'docs/packs/artist.md',
-  ideator: 'docs/packs/ideator.md',
 };
 const DISCIPLINE_SKILL = { ui: 'design' };
 
@@ -241,7 +240,7 @@ function nextActionFor(stage, { taskId, discipline, sandboxUrl } = {}) {
     if (skill) return { kind: 'prompt', label: 'Open the work', cmd: `/${skill}` };
     // Default to the Engineer pack for an unknown or unset discipline, the same way
     // this used to default to /dev: engineering is the project's default loop.
-    const pack = DISCIPLINE_PACK[discipline] || DISCIPLINE_PACK.engineer;
+    const pack = DISCIPLINE_PACK[discipline] || DISCIPLINE_PACK.artisan;
     return { kind: 'prompt', label: 'Open the work', cmd: `Read ${pack} and start the work` };
   }
   return { kind: 'prompt', label: 'Continue', cmd: '/builder-start' };

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;