@bongos/core 1.19.879 → 1.19.881

package/docs/module-api-changelog.md

@@ -2232,5 +2232,9 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.19.877 with no explicit bump. run 35439004814. (task 1002620)
   1.19.879 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         landed since 1.19.878 with no explicit bump. run 35439529685. (task 1002620)
+  1.19.880 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.19.879 with no explicit bump. run 35453118533. (task 1002620)
+  1.19.881 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.19.880 with no explicit bump. run 35455207976. (task 1002620)
 ---------------------------------------------------------------------------
 ```

package/docs/project-context.template.md

@@ -21,8 +21,8 @@ Keep it durable framing, not live state (live state belongs in the build platfor
 do not apply to this project. Delete this comment block when done.
 -->
 
-<One paragraph: what this product is, who it's for, and the single idea that makes it distinct. The full
-"two tracks / how work is sorted" framing lives in [`CLAUDE.md`](../CLAUDE.md) §1.>
+<One paragraph: what this product is, who it's for, and the single idea that makes it distinct. How work is
+sorted — versions, and the role each task carries — is framed in [`CLAUDE.md`](../CLAUDE.md) §8.>
 
 ---
 

package/modules/hall-ui/public/dom-utils.js

@@ -146,7 +146,10 @@
     for (const k of list) {
       const opt = document.createElement('option');
       opt.value = k;
-      opt.textContent = label(k);
+      // The unset bucket is named for a reader on EVERY surface, filters included
+      // — `label` is the caller's choice for the real kinds, never for this one
+      // (task 1004038). roleOptions() names the role twin the same way.
+      opt.textContent = kindIsUnset(k) ? kindLabel(k) : label(k);
       sel.appendChild(opt);
     }
     // A refill must not drop the reader's pick; `selected` is the filing default.
@@ -154,10 +157,23 @@
     if (want) sel.value = want;
   }
 
+  // A task with no kind carries the DB default 'unclassified' — the twin of
+  // ROLE_UNSET one axis over (task 1004038, filed by task 1003936 which fixed the
+  // role axis and left this one). A word a column chose is not a word a person
+  // should read (docs/voice.md V2), so a picker shows "No kind yet" and a row tag
+  // renders NOTHING. kindIsUnset() is how a caller asks. The STORED value and the
+  // filter VALUE stay 'unclassified'; only what is read changes.
+  const KIND_UNSET = 'unclassified';
+  function kindIsUnset(k) { return !k || String(k) === KIND_UNSET; }
+
+  // NOT ESCAPE-SAFE: like roleLabel() below, this returns a transformed copy of
+  // its input, so a caller rendering it into markup must escapeHtml() the result.
+  // Every caller today assigns it to textContent, which escapes for them.
   // How a kind is named in a picker — 'blocker-resolution' → 'Blocker resolution'.
   // The same shape roleLabel takes below, and it reproduces every label the
   // hand-written <option> list in ideas.html carried, so no copy changes.
   function kindLabel(k) {
+    if (kindIsUnset(k)) return 'No kind yet';
     const s = String(k || '').replace(/-/g, ' ');
     return s ? s.charAt(0).toUpperCase() + s.slice(1) : '';
   }
@@ -407,7 +423,7 @@
     escapeHtml, $, fmtNum, num, fmtInt, fmtCompact, fmtDate, rankLabel, toast, moduleOn, disciplines,
     // task 1003936 — ONE word for the role axis, all six surfaces
     roleAxisLabel, roleLabel, roleOptions, roleIsUnset, ROLE_UNSET,
-    fillKindFilter, kindLabel, // task 1003453 — one Kind filler, all three surfaces
+    fillKindFilter, kindLabel, kindIsUnset, KIND_UNSET, // task 1003453 — one Kind filler, all three surfaces
     errCode, errMessage, errDetails, // task 1002462 — ADR 0116 envelope readers
     contributeSettingsPanel, settingsPanels,
     contributeHallWidget, hallWidgets, hallWidgetIds,

package/modules/hall-ui/public/goals-render.js

@@ -321,6 +321,11 @@
     const disciplineTag = window.OTB.roleIsUnset(t.discipline)
       ? ''
       : `<span class="task__tag">${escapeHtml(window.OTB.roleLabel(t.discipline))}</span>`;
+    // A kind nobody set is no tag at all, not the column's default word — the
+    // role tag above is the precedent this follows (task 1004038).
+    const kindTag = window.OTB.kindIsUnset(t.kind)
+      ? ''
+      : `<span class="task__tag">${escapeHtml(t.kind)}</span>`;
     const newcomerTag = t.newcomer_friendly ? '<span class="task__tag task__tag--newcomer">newcomer</span>' : '';
     const rankTag = rankTagHtml(t.requires_rank);
     const touches = Array.isArray(t.touches) ? t.touches : [];
@@ -361,7 +366,7 @@
           <p class="task__title"><a class="task__link" href="${escapeHtml(taskHref(id))}">${escapeHtml(t.title || '')}</a></p>
           <div class="task__meta">
             <span class="task__tag">${escapeHtml(t.version_id || '?')}</span>
-            <span class="task__tag">${escapeHtml(t.kind || 'unclassified')}</span>
+            ${kindTag}
             ${disciplineTag}
             ${newcomerTag}
             ${rankTag}
@@ -403,7 +408,7 @@
       <tr class="ledger__row--task" data-task-id="${escapeHtml(String(t.id))}" style="cursor:pointer" title="Open task #${escapeHtml(String(t.id))}">
         <td class="ledger__num">#${escapeHtml(String(t.id))}</td>
         <td>${escapeHtml(t.title || '')}</td>
-        <td>${escapeHtml(t.kind || '—')}</td>
+        <td>${window.OTB.kindIsUnset(t.kind) ? '—' : escapeHtml(t.kind)}</td>
         <td class="ledger__drachmae">${fmtNum(creditVal(t))}${taskSettled(t) ? '' : ' <span class="ledger__est" title="Estimate — credits settle when the task is confirmed">est.</span>'}</td>
         <td class="ledger__date">${fmtDate(t.shipped_at)}</td>
       </tr>`).join('');
@@ -502,7 +507,7 @@
     const sortBy = $('#f-goal-sort')?.value || 'cpm';
     let list = state.work.todo.filter((t) => {
       if (!matchesTodoSearch(t, sQuery)) return false;
-      if (kFilter && (t.kind || 'unclassified') !== kFilter) return false;
+      if (kFilter && (t.kind || window.OTB.KIND_UNSET) !== kFilter) return false;
       if (dFilter && (t.discipline || window.OTB.ROLE_UNSET) !== dFilter) return false;
       if (nFilter === 'yes' && !t.newcomer_friendly) return false;
       return true;

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

@@ -42,7 +42,6 @@
   // doesn't pick up the wrong picture. The image paths resolve at
   // /builders/diagrams/* (mounted in server.js).
   const DIAGRAM_AT_HEADING = [
-    { match: 'two tracks',                  src: '/builders/diagrams/05-two-track-versioning.png', alt: 'The two tracks: product (the game) and internal (the dev system)' },
     { match: 'claim → ship lifecycle',      src: '/builders/diagrams/01-task-lifecycle.png',       alt: 'The full task lifecycle from backlog through shipped' },
     { match: 'rank ladder',                 src: '/builders/diagrams/02-rank-ladder.png',          alt: 'The rank ladder — the ranks a builder can hold and how to rise' },
     { match: 'drachmae and karma',          src: '/builders/diagrams/03-drachmae-karma.png',       alt: 'How the reward currency and karma differ — pay for shipped work vs. reputation signals' },

package/modules/hall-ui/public/roadmap.css

@@ -38,17 +38,6 @@ body[data-page="roadmap"] {
 /* ---- shared atoms ----
    The chips are the deck's label voice: they name a KIND, so they take the
    label tokens rather than a hand-written uppercase of their own. */
-.rm-chip { display:inline-flex; align-items:center; gap:5px;
-  font-family:var(--label-face); font-size:var(--label-size); font-weight:400;
-  letter-spacing:var(--label-space); text-transform:var(--label-case);
-  padding:4px 9px; border-radius:var(--radius-pill); white-space:nowrap; }
-.rm-chip .rm-sq { width:7px; height:7px; border-radius:2px; }
-.rm-chip--product  { background:var(--accent-tint); color:var(--accent-ink); }
-.rm-chip--product  .rm-sq { background:var(--accent); }
-.rm-chip--internal { background:var(--info-weak); color:var(--info); }
-.rm-chip--internal .rm-sq { background:var(--sea); }
-.rm-chip--dogfood  { background:var(--rm-terra-weak); color:color-mix(in srgb, var(--terracotta) 82%, var(--ink)); }
-.rm-chip--dogfood  .rm-sq { background:var(--terracotta); }
 
 .rm-phase { display:inline-flex; align-items:center; gap:6px; font-size:11.5px; font-weight:500;
   padding:4px 10px; border-radius:var(--radius-pill); white-space:nowrap; border:var(--line) solid transparent; }

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

@@ -13,7 +13,7 @@
 // links to.
 //
 // Reads (all GET): /public/progress (lifecycle rollup + counts + %, public),
-// /public/versions (track + started/shipped dates, public), and — for every
+// /public/versions (started/shipped dates, public), and — for every
 // non-shipped version only — /goals?version=X + /goals/:id (per-goal criteria to
 // split in-progress from not-started). Shipped versions render as a pill strip
 // with no goal fetch. No writes. Server page gate: SIGNEDIN_BUILDERS_PAGE_RE.
@@ -37,11 +37,6 @@
   }
 
   const MON = ['Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec'];
-  const TRACK = {
-    product:  { label: 'Product',  cls: 'product' },
-    internal: { label: 'Platform', cls: 'internal' },
-    default:  { label: 'Dogfood',  cls: 'dogfood' },
-  };
   const PHASE = {
     building: { label: 'In flight', cls: 'building' },
     planning: { label: 'Staged',    cls: 'planning' },
@@ -60,9 +55,7 @@
 
   const pctOf = (n, d) => (d > 0 ? (n / d) * 100 : 0);
   const ym = (s) => { if (!s) return ''; const d = new Date(s); return Number.isNaN(d.getTime()) ? '' : `${MON[d.getMonth()]} ${d.getFullYear()}`; };
-  const trackOf = (v) => TRACK[v.track] || TRACK.internal;
 
-  const chip = (v) => { const t = trackOf(v); return `<span class="rm-chip rm-chip--${t.cls}"><span class="rm-sq"></span>${t.label}</span>`; };
   const phase = (v) => { const p = PHASE[v.status]; return p ? `<span class="rm-phase rm-phase--${p.cls}"><span class="rm-pd"></span>${p.label}</span>` : ''; };
 
   function ring(v) {
@@ -162,7 +155,7 @@
         <div class="rm-featgrid__l">
           <div class="rm-feat-top">
             <div class="rt">
-              <div class="rm-row">${chip(v)}${phase(v)}<span class="rm-id">${escapeHtml(v.id)}</span></div>
+              <div class="rm-row">${phase(v)}<span class="rm-id">${escapeHtml(v.id)}</span></div>
               <h3 class="rm-name">${escapeHtml(v.name)}</h3>
               <p class="rm-desc rm-desc--clamp">${escapeHtml(v.desc)}</p>
             </div>
@@ -183,7 +176,7 @@
     return `<article class="rm-card">
       <div class="rm-feat-top">
         <div class="rt">
-          <div class="rm-row">${chip(v)}${phase(v)}<span class="rm-id">${escapeHtml(v.id)}</span></div>
+          <div class="rm-row">${phase(v)}<span class="rm-id">${escapeHtml(v.id)}</span></div>
           <h3 class="rm-name">${escapeHtml(v.name)}</h3>
           <p class="rm-desc rm-desc--clamp">${escapeHtml(v.desc)}</p>
         </div>
@@ -225,7 +218,7 @@
     return `<article class="rm-card rm-card--staged" style="grid-column:1 / -1">
       <div class="rm-featgrid">
         <div class="rm-featgrid__l">
-          <div class="rm-row">${chip(v)}${phase(v)}<span class="rm-id">${escapeHtml(v.id)}</span></div>
+          <div class="rm-row">${phase(v)}<span class="rm-id">${escapeHtml(v.id)}</span></div>
           <h3 class="rm-name">${escapeHtml(v.name)}</h3>
           <p class="rm-desc rm-desc--clamp">${escapeHtml(v.desc)}</p>
           <div class="rm-staged-state">
@@ -243,7 +236,7 @@
     return `<article class="rm-card rm-card--parked" style="grid-column:1 / -1">
       <div class="rm-feat-top">
         <div class="rt">
-          <div class="rm-row">${chip(v)}${phase(v)}<span class="rm-id">${escapeHtml(v.id)}</span></div>
+          <div class="rm-row">${phase(v)}<span class="rm-id">${escapeHtml(v.id)}</span></div>
           <h3 class="rm-name">${escapeHtml(v.name)}</h3>
           <p class="rm-desc rm-desc--clamp">${escapeHtml(v.desc)}</p>
         </div>
@@ -353,8 +346,8 @@
     }
   }
 
-  // Merge /public/progress (counts, %, lifecycle) with /public/versions (track,
-  // dates) into one version model the renderers consume.
+  // Merge /public/progress (counts, %, lifecycle) with /public/versions (dates)
+  // into one version model the renderers consume.
   function mergeVersions(progress, versions) {
     const meta = new Map();
     (versions || []).forEach((v) => meta.set(v.id, v));
@@ -363,7 +356,6 @@
       return {
         id: p.version_id,
         name: p.name || m.name || p.version_id,
-        track: m.track || 'internal',
         status: p.status || m.status || 'building',
         desc: (p.done_when || m.done_when || '').trim(),
         shipped: Number(p.shipped_count) || 0,

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

@@ -536,8 +536,12 @@
     // word is the thing the round's stat-tile reference exists to fix.
     const meta = [];
     if (t.version_id) meta.push(`<b>${escapeHtml(t.version_id)}</b>`);
-    if (t.kind) meta.push(escapeHtml(t.kind));
-    if (t.discipline && t.discipline !== 'unclassified') meta.push(escapeHtml(t.discipline));
+    // Neither axis says the word its column defaults to (task 1004038). `if
+    // (t.kind)` was never a guard for it: 'unclassified' is a truthy string, so
+    // the run read it out loud. The role line below asked the same question with
+    // a literal — it is right, but a copy of roleIsUnset() is free to drift.
+    if (!window.OTB.kindIsUnset(t.kind)) meta.push(escapeHtml(t.kind));
+    if (!window.OTB.roleIsUnset(t.discipline)) meta.push(escapeHtml(t.discipline));
     // rankText() answers "any rank" for an open task, which read as
     // "min rank any rank" once the label was prefixed. An open floor is not a
     // minimum, so it is said as the absence of one.

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

@@ -429,6 +429,8 @@
       const disciplineTag = window.OTB.roleIsUnset(t.discipline)
         ? ''
         : `<span class="task__tag">${escapeHtml(window.OTB.roleLabel(t.discipline))}</span>`;
+      // A kind nobody set is no tag at all, like the role above (task 1004038).
+      const kindTag = window.OTB.kindIsUnset(t.kind) ? '' : `<span class="task__tag">${escapeHtml(t.kind)}</span>`;
       const newcomerTag = t.newcomer_friendly
         ? '<span class="task__tag task__tag--newcomer">newcomer</span>'
         : '';
@@ -459,7 +461,7 @@
             <p class="task__title"><a class="task__link" href="${escapeHtml(taskHref(t.id))}">${escapeHtml(t.title)}</a></p>
             <div class="task__meta">
               <span class="task__tag">${escapeHtml(t.version_id || '?')}</span>
-              <span class="task__tag">${escapeHtml(t.kind || 'unclassified')}</span>
+              ${kindTag}
               ${disciplineTag}
               ${newcomerTag}
               ${rankTag}
@@ -1142,7 +1144,7 @@
         <div class="history__row" data-task-id="${escapeHtml(String(t.id))}" title="Open task #${escapeHtml(String(t.id))}">
           <span class="history__id">#${escapeHtml(String(t.id))}</span>
           <span class="history__title">${escapeHtml(t.title || '(untitled)')} ${ver} ${disciplineTag} ${rankTagHtml(t.requires_rank)}<br>${goalTag}</span>
-          <span class="history__when">${escapeHtml(t.kind || '')}</span>
+          <span class="history__when">${window.OTB.kindIsUnset(t.kind) ? '' : escapeHtml(t.kind)}</span>
         </div>`;
     }).join('');
   }

package/modules/lifecycle/db-versions.js

@@ -23,7 +23,7 @@ const { createGoal } = require('./db-goals.js');
 
 async function listVersions() {
   const { rows } = await pool.query(
-    `SELECT id, name, track, status, scope_doc_path, started_at, shipped_at, done_when
+    `SELECT id, name, status, scope_doc_path, started_at, shipped_at, done_when
        FROM versions
        ORDER BY started_at`
   );
@@ -46,7 +46,7 @@ async function getVersion(id, deps = {}) {
     throw new Error('getVersion({ forUpdate }) requires deps.client — a row lock outside a transaction holds nothing');
   }
   const { rows } = await executorFor(deps, pool).query(
-    `SELECT id, name, track, status, scope_doc_path, started_at, shipped_at, done_when
+    `SELECT id, name, status, scope_doc_path, started_at, shipped_at, done_when
        FROM versions WHERE id = $1${deps.forUpdate ? ' FOR UPDATE' : ''}`,
     [id]
   );
@@ -167,10 +167,14 @@ async function versionsWithStatus(status, deps = {}) {
 // goal-less criteria, and ADR 0183 criteria satisfy themselves off LINKED tasks.
 // Criteria nobody can link a task to can never close. Stamping goal_id here is
 // what gives them something to hang from.
-async function createVersion({ id, name, status = 'planning', track = 'product', scopeDocPath = null, doneWhen = null, criteria = [], firstGoal = null, createdBy = null }, deps = {}) {
-  // versions.track is NOT NULL with no DB default (migration 003) — omitting it
-  // here made every create 500. An instance's first version is its product
-  // track, so that's the default; 'internal' is the platform's own track.
+async function createVersion({ id, name, status = 'planning', scopeDocPath = null, doneWhen = null, criteria = [], firstGoal = null, createdBy = null }, deps = {}) {
+  // No `track`: the two-track (product/internal) model was retired by task
+  // 1002006 and its code unwound by task 1003665. versions.track survives as a
+  // column, NOT NULL but carrying DEFAULT 'product' since core_205, so an INSERT
+  // that omits it is legal — which is what lets this path stop naming it. The
+  // comment that used to sit here said the column had NO default; that was true
+  // when it was written and stale by core_205, and believing it is why five
+  // insert paths kept supplying a value nobody read.
   //
   // deps.pool is the module's existing test seam (listVersions, versionProgress):
   // honoured ONLY under NODE_ENV=test, so production can never be pointed at an
@@ -179,10 +183,10 @@ async function createVersion({ id, name, status = 'planning', track = 'product',
   const txPool = (deps.pool && process.env.NODE_ENV === 'test') ? deps.pool : pool;
   return withTx(async (client) => {
     const { rows } = await client.query(
-      `INSERT INTO versions (id, name, status, track, scope_doc_path, done_when)
-            VALUES ($1, $2, $3, $4, $5, $6)
-       RETURNING id, name, status, track, scope_doc_path, started_at, shipped_at, done_when`,
-      [id, name, status, track, scopeDocPath, doneWhen]
+      `INSERT INTO versions (id, name, status, scope_doc_path, done_when)
+            VALUES ($1, $2, $3, $4, $5)
+       RETURNING id, name, status, scope_doc_path, started_at, shipped_at, done_when`,
+      [id, name, status, scopeDocPath, doneWhen]
     );
     const version = rows[0];
     // The first goal. Titled for the version it opens unless the caller names one —
@@ -340,7 +344,7 @@ async function promotePlanningVersion(exec) {
         SET status = 'building', started_at = COALESCE(started_at, now())
       WHERE id = (SELECT id FROM versions WHERE status = 'planning' ORDER BY id LIMIT 1)
         AND NOT EXISTS (SELECT 1 FROM versions WHERE status = 'building')
-      RETURNING id, name, track, status, started_at`
+      RETURNING id, name, status, started_at`
   );
   return rows[0] ?? null;
 }
@@ -352,7 +356,7 @@ async function closeVersion({ versionId, plan = [], reason, planningVersionId =
     await client.query('BEGIN');
 
     const { rows: vRows } = await client.query(
-      `SELECT id, name, track, status, scope_doc_path, started_at, shipped_at, done_when
+      `SELECT id, name, status, scope_doc_path, started_at, shipped_at, done_when
          FROM versions WHERE id = $1 FOR UPDATE`,
       [versionId]
     );
@@ -576,7 +580,7 @@ async function closeVersion({ versionId, plan = [], reason, planningVersionId =
     const { rows: closed } = await client.query(
       `UPDATE versions SET status = 'shipped', shipped_at = now()
         WHERE id = $1
-        RETURNING id, name, track, status, scope_doc_path, started_at, shipped_at, done_when`,
+        RETURNING id, name, status, scope_doc_path, started_at, shipped_at, done_when`,
       [versionId]
     );
 

package/modules/lifecycle/routes/versions.js

@@ -30,10 +30,6 @@ const { asyncHandler, validateOrRespond } = api;
 // 'building'; 'shipped'/'frozen' are end states reached via the close flow.
 const VALID_VERSION_STATUSES = new Set(['planning', 'building', 'shipped', 'frozen']);
 
-// versions.track CHECK (migration 003) — NOT NULL with no DB default, so the
-// create path must always supply one. 'product' is the caller-facing default:
-// an instance seeding its first version is seeding its product track.
-const VALID_VERSION_TRACKS = new Set(['product', 'internal']);
 
 module.exports = function buildVersionsRouter() {
   const router = express.Router();
@@ -99,7 +95,6 @@ module.exports = function buildVersionsRouter() {
       id: { required: true, type: 'string', maxLength: 64, minLength: 1 },
       name: { required: true, type: 'string', maxLength: 200, minLength: 1 },
       status: { type: 'string', maxLength: 32 },
-      track: { type: 'string', maxLength: 16 },
       done_when: { type: 'string', maxLength: 8000 },
       scope_doc_path: { type: 'string', maxLength: 256 },
       criteria: { type: 'array', maxItems: 100 },
@@ -116,11 +111,6 @@ module.exports = function buildVersionsRouter() {
       return res.fail('bad_status', 400, { valid: Array.from(VALID_VERSION_STATUSES) });
     }
 
-    const track = body.track || 'product';
-    if (!VALID_VERSION_TRACKS.has(track)) {
-      return res.fail('bad_track', 400, { valid: Array.from(VALID_VERSION_TRACKS) });
-    }
-
     if (await db.getVersion(id)) {
       return res.fail('version_exists', { status: 409, message: `Version '${id}' already exists.` });
     }
@@ -187,7 +177,7 @@ module.exports = function buildVersionsRouter() {
     let version, goal, created;
     try {
       ({ version, goal, criteria: created } = await db.createVersion({
-        id, name, status, track,
+        id, name, status,
         doneWhen: body.done_when ?? null,
         scopeDocPath: body.scope_doc_path ?? null,
         criteria,

package/modules/lifecycle/version-autoclose.js

@@ -65,7 +65,7 @@ async function autoCloseVersionIfComplete(exec, versionId) {
              AND g.status = 'open'
              AND ${NOT_MAINTENANCE}
         )
-      RETURNING v.id, v.name, v.track, v.status, v.shipped_at`,
+      RETURNING v.id, v.name, v.status, v.shipped_at`,
     [versionId]
   );
   return rows[0] ?? null;

package/package-lock.json

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

package/package.json

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

package/scripts/gds/init.js

@@ -391,9 +391,6 @@ function versionPayload(spec) {
     id: v.id,
     name: v.name,
     status: v.status || 'building',
-    // versions.track is NOT NULL (no DB default) — always send it explicitly.
-    // An instance's first version is its product track unless the spec says otherwise.
-    track: v.track === 'internal' ? 'internal' : 'product',
     done_when: v.done_when,
     criteria: [{ criterion_id: v.criterion_id ? slugify(v.criterion_id) : 'first-goal', criterion_md: v.done_when }],
   };

package/scripts/gds/provision-repo.js

@@ -55,8 +55,9 @@ function migrateCmd(inst, { privileged = false } = {}) {
 // ONLY IF the versions table is empty: idempotent, safe to re-run, and it never
 // disturbs a project that already has versions. Mirrors seedViaApi's shape — V1
 // (status 'building') + one done-when criterion so /status has a finish line from day
-// one. `track` is set EXPLICITLY: versions.track is NOT NULL with no default (mig 112
-// dropped the CHECK), and 'product' matches the canonical V1 seed in 003_pms.sql.
+// one. No `track` column: the two-track model was retired (task 1002006, unwound
+// by task 1003665). The seed runs on a freshly-MIGRATED database, so core_205's
+// DEFAULT 'product' is in place and an INSERT that omits the column is legal.
 //
 // The id/name/done_when come from instanceInitSpec so the starter version has ONE
 // source of truth (the same spec the greenfield `bongos init` seed uses). SQL string
@@ -68,8 +69,8 @@ function seedFirstVersionCmd(inst, { privileged = false } = {}) {
   const v = instanceInitSpec(inst).version;
   const q = (s) => `$v$${s}$v$`;                 // dollar-quoted literal (no value contains a single quote or $)
   const sql =
-    `INSERT INTO versions (id, name, track, status, done_when) ` +
-    `SELECT ${q(v.id)}, ${q(v.name)}, $v$product$v$, $v$building$v$, ${q(v.done_when)} ` +
+    `INSERT INTO versions (id, name, status, done_when) ` +
+    `SELECT ${q(v.id)}, ${q(v.name)}, $v$building$v$, ${q(v.done_when)} ` +
     `WHERE NOT EXISTS (SELECT 1 FROM versions); ` +
     `INSERT INTO done_when_criteria (version_id, criterion_id, criterion_md, sort_order) ` +
     `SELECT ${q(v.id)}, $v$first-goal$v$, ${q(v.done_when)}, 10 ` +

package/src/module-api.js

@@ -71,7 +71,7 @@ const { responsibilityFor, ROLE_RESPONSIBILITIES } = require('./role-responsibil
 // there. scripts/gds/bump-version.js still rewrites the literal below; it appends
 // the entry to that file. Look for a version's history there, not here.
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.879'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.19.881'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
 
 // A namespaced logger so a module's log lines are attributable + consistent.
 // Usage: const log = api.logger('dev-box'); log.info('mounted');

package/tests/hall_record_world.mjs

@@ -354,6 +354,11 @@ function renderTask(task, { lastClaim = null, viewer = null, goal = null, goalsL
       roleOptions: () => [{ value: '', label: '— any —' }, { value: 'engineer', label: 'Engineer' }, { value: 'artist', label: 'Artist' }],
       roleIsUnset: (d) => !d || d === 'unclassified',
       ROLE_UNSET: 'unclassified',
+      // task 1004038 — the kind axis asks the same question. Stubbed for the
+      // reason the comment above gives: an undefined helper throws mid-render,
+      // and the page then renders its "feed unavailable" card instead.
+      kindIsUnset: (k) => !k || k === 'unclassified',
+      KIND_UNSET: 'unclassified',
       escapeHtml,
       $: (sel) => documentObj.querySelector(sel),
       fmtDate: (iso) => String(iso).slice(0, 10),

package/tests/init.mjs

@@ -204,16 +204,18 @@ test('versionPayload: explicit criterion_id is slugified', () => {
   assert.equal(init.versionPayload(spec).criteria[0].criterion_id, 'prove-the-loop');
 });
 
-// versions.track is NOT NULL with no DB default — leaving it off the payload is
-// exactly the 1.18.18 bug (every POST /versions 500ed, blocking init outright).
-test('versionPayload: always sends an explicit, valid track', () => {
-  assert.equal(init.versionPayload(fullSpec()).track, 'product'); // instance's first version = product
-  const internal = fullSpec();
-  internal.version.track = 'internal';
-  assert.equal(init.versionPayload(internal).track, 'internal');  // explicit internal respected
-  const junk = fullSpec();
-  junk.version.track = 'bogus';
-  assert.equal(init.versionPayload(junk).track, 'product');       // junk coerced, never forwarded
+// The inverse of the case this replaces. That case pinned "always send an
+// explicit track", guarding the 1.18.18 bug where omitting it 500ed every
+// POST /versions against a NOT NULL column with no default. core_205 gave the
+// column DEFAULT 'product' and task 1003665 retired the concept, so the payload
+// must now NOT carry it — and a spec that still names one must not smuggle it
+// through to a route that no longer accepts the field.
+test('versionPayload: sends no track, and drops one a spec still names', () => {
+  assert.equal('track' in init.versionPayload(fullSpec()), false);
+  const legacy = fullSpec();
+  legacy.version.track = 'internal';
+  assert.equal('track' in init.versionPayload(legacy), false,
+    'an old spec file cannot reintroduce the field');
 });
 
 // ---- provisioning arm (ADR 0111 §3, task 1946) ----
@@ -661,12 +663,13 @@ async function runSeedTests() {
     assert.ok(findTaskPost(api, /Production topology/), 'filed the manual fallback task on error');
   });
 
-  await at('seedViaApi: version POST carries the explicit track (the NOT NULL column)', async () => {
+  await at('seedViaApi: the version POST carries NO track (task 1003665)', async () => {
     const api = fakeApi();
     await init.seedViaApi(provSpec({}), { log: () => {}, api });
     const vpost = api.calls.find((c) => c.path.endsWith('/versions'));
     assert.ok(vpost, 'POSTed the version');
-    assert.equal(vpost.body.track, 'product');
+    assert.equal('track' in vpost.body, false,
+      'POST /versions no longer accepts the field; the DB default supplies the column');
   });
 
   await at('seedViaApi: the ideator kickoff task POSTs kind=decision + discipline=ideator', async () => {