@bongos/core 1.19.322 → 1.19.324

package/migrations/core_224_goal_subtitle.sql

@@ -0,0 +1,35 @@
+-- core_224_goal_subtitle.sql — a goal's explanation stops living inside its
+-- name (task 1003224, goal 1000072, criterion legibility-goal-names-and-claude-text).
+--
+-- THE PROBLEM THIS FIXES IS IN THE DATA. 29 of 38 open goals carry their own
+-- explanation appended to `title` after a dash — "Findability — the grep path",
+-- "Idea routing — work that has a home lands in it". The goal board is the
+-- primary work surface, so every list, card, pill and breadcrumb renders the
+-- explanation at the same weight as the name, and nothing can show one without
+-- the other. A name and a description are two fields; they were one.
+--
+-- ADDITIVE ONLY, and nullable with no default on purpose. Every existing row
+-- keeps its title byte-for-byte: this migration creates somewhere for the
+-- explanation to GO, and the separate renaming pass (task 1003225) moves it,
+-- goal by goal, with judgment — three goals all head "Agents module" and would
+-- become indistinguishable under a mechanical split. A DEFAULT '' would make
+-- "never set" and "deliberately blank" the same state, and the UI needs to tell
+-- them apart to know whether to render the affordance at all.
+--
+-- WHY THIS IS A CORE MIGRATION AND NOT A MODULE ONE. `goals` was created by
+-- repo-root migrations/160_goal_tier.sql and is a CORE table — the lifecycle
+-- module owns the queries against it, not the table itself (modules/lifecycle/
+-- CLAUDE.md: "None owned"). Fitness check 11 enforces exactly that: a module
+-- migration may only touch its own `<key>_`-prefixed tables, so an
+-- `ALTER TABLE goals` from modules/lifecycle/migrations/ is refused however
+-- lifecycle-ish it feels (ADR 0083 §Decision #5). The first draft of this file
+-- lived there and the check caught it.
+
+ALTER TABLE goals
+  ADD COLUMN IF NOT EXISTS subtitle text;
+
+COMMENT ON COLUMN goals.subtitle IS
+  'The goal''s explanation, held apart from its name. Rendered subordinate to title and on demand (task 1003224). NULL = never set, which is distinct from an empty string meaning deliberately blank.';
+
+INSERT INTO schema_migrations (version) VALUES ('core_224_goal_subtitle')
+ON CONFLICT (version) DO NOTHING;

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

@@ -90,10 +90,36 @@
   const msOf = (iso) => { const t = iso ? Date.parse(iso) : NaN; return Number.isNaN(t) ? 0 : t; };
   const taskActivityMs = (t) => Math.max(msOf(t.updated_at), msOf(t.shipped_at));
 
+  // task 1003224 — on a LIST, a goal's subtitle is a TOOLTIP, never a second
+  // visible line. The board's job there is to let someone find a goal by name;
+  // rendering every explanation inline is what led 29 of 38 titles to carry
+  // their own after a dash. Escaped because it lands inside an attribute.
+  function subtitleAttr(g) {
+    const t = (g && g.subtitle ? String(g.subtitle) : '').trim();
+    return t ? ` title="${window.OTB.escapeHtml(t)}"` : '';
+  }
+
+  // task 1003224 — the three-layer prose rules for a goal's detail head, kept
+  // here rather than in goals-page.js because that file is 500 lines over the
+  // excess-lines ratchet and this one exists for exactly that reason. It WRITES
+  // to elements the caller already looked up; it never reads the DOM or touches
+  // page state, so the file's purity contract holds.
+  //   name      -> the h2, rendered by the caller
+  //   subtitle  -> subordinate, and ABSENT when unset (NULL differs from blank)
+  //   Claude's prose -> collapsed, and absent when there is nothing to reveal
+  function applyGoalProse(els, g) {
+    const subtitle = (g.subtitle || '').trim();
+    const description = g.description || '';
+    if (els.sub) { els.sub.textContent = subtitle; els.sub.hidden = !subtitle; }
+    if (els.desc) els.desc.textContent = description;
+    if (els.claude) { els.claude.hidden = !description.trim(); els.claude.open = false; }
+  }
+
   window.OTBGoalsLib = {
     DONE_STATUSES, CUT_STATUSES, isTodoStatus,
     ring, memberName,
     readinessOf, EMPTY_ROLLUP, goalRollLine,
     matchesTodoSearch, msOf, taskActivityMs,
+    subtitleAttr, applyGoalProse,
   };
 })();

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

@@ -40,7 +40,7 @@
   // so every call site below reads exactly as before.
   const { DONE_STATUSES, CUT_STATUSES, isTodoStatus, ring, memberName,
     readinessOf, EMPTY_ROLLUP, goalRollLine, matchesTodoSearch,
-    msOf, taskActivityMs } = window.OTBGoalsLib;
+    msOf, taskActivityMs, subtitleAttr, applyGoalProse } = window.OTBGoalsLib;
   const API = '/api/bongos';
   // Generated typed client (task 1996 / R09), served as window.BongosClient by
   // /lib/bongos-client.js. baseUrl:'' passes the full same-origin paths this file
@@ -359,7 +359,7 @@
     return `<div class="gr-row" data-goal-id="${escapeHtml(String(g.id))}">
       <div class="gr-ringcol">${ringHtml}</div>
       <div class="gr-main">
-        <a class="gr-title" href="${goalHref}">${escapeHtml(g.title || 'Untitled goal')}</a>
+        <a class="gr-title" href="${goalHref}"${subtitleAttr(g)}>${escapeHtml(g.title || 'Untitled goal')}</a>
         <div class="gr-meta">${metaHtml}</div>
         ${achieved ? '' : goalRollLine(r)}
       </div>
@@ -377,7 +377,7 @@
     return `<div class="goal-card" data-goal-id="${escapeHtml(String(g.id))}">
       <div class="goal-card__head">
         <div class="gr-ringcol">${ringHtml}</div>
-        <a class="goal-card__title gr-title" href="${goalHref}">${escapeHtml(g.title || 'Untitled goal')}</a>
+        <a class="goal-card__title gr-title" href="${goalHref}"${subtitleAttr(g)}>${escapeHtml(g.title || 'Untitled goal')}</a>
       </div>
       <div class="goal-card__meta">${metaHtml}</div>
       ${achieved ? '' : goalRollLine(r)}
@@ -1211,8 +1211,8 @@
     // task 2163 — surface the numeric goal id alongside the title (escaped title,
     // so switching from textContent to innerHTML stays injection-safe).
     if (h) h.innerHTML = `<span class="goal-detail__id" title="Goal id">#${escapeHtml(String(g.id))}</span> ${escapeHtml(g.title || 'Untitled goal')}`;
-    const desc = $('#goal-detail-desc');
-    if (desc) desc.textContent = g.description || '';
+    // task 1003224 — name / subtitle / Claude's prose at three weights; the rules live in goals-lib.
+    applyGoalProse({ sub: $('#goal-detail-subtitle'), desc: $('#goal-detail-desc'), claude: $('#goal-detail-claude') }, g);
     const meta = $('#goal-detail-meta');
     if (meta) {
       const scope = Array.isArray(g.scope_modules) && g.scope_modules.length

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

@@ -259,3 +259,71 @@ a.goal-card:hover { text-decoration: none; border-color: var(--border-strong); b
   .gr-next { align-items: flex-start; }
   .gr-next__t { max-width: 100%; }
 }
+
+/* ── task 1003224: name, subtitle, and Claude's text at three weights ────────
+   The goal board is the primary work surface, so what it shows by DEFAULT is
+   the decision this makes. The name reads first; the explanation is visibly
+   subordinate; the bulk prose Claude writes is behind a disclosure and absent
+   from the default reading path until someone asks for it.
+   Tokens only from style.css's real set (--ink-soft, --ink-faint, --rule,
+   --surface, --radius-sm) — inventing one renders as nothing, which is how
+   copy-desk.css ended up with invisible fallbacks (idea 1000821). */
+.goal-detail__subtitle {
+  color: var(--ink-soft);
+  font-size: 13px;
+  font-weight: 400;
+  line-height: 1.45;
+  max-width: 72ch;
+  margin: 3px 0 0;
+}
+
+/* The disclosure is deliberately quiet: a row someone can ignore. */
+.claude-text {
+  margin: 10px 0 0;
+  max-width: 72ch;
+  border-top: 1px solid var(--rule);
+  padding-top: 8px;
+}
+
+.claude-text__toggle {
+  color: var(--ink-faint);
+  font-size: 12px;
+  cursor: pointer;
+  list-style: none;
+  user-select: none;
+  display: inline-flex;
+  align-items: center;
+  gap: 6px;
+}
+
+/* Own marker, so the affordance reads the same in every browser. */
+.claude-text__toggle::-webkit-details-marker { display: none; }
+.claude-text__toggle::before {
+  content: '▸';
+  font-size: 10px;
+  transition: transform 120ms ease;
+}
+.claude-text[open] > .claude-text__toggle::before { transform: rotate(90deg); }
+.claude-text__toggle:hover { color: var(--ink-soft); }
+.claude-text__toggle:focus-visible {
+  outline: 2px solid var(--accent);
+  outline-offset: 2px;
+  border-radius: var(--radius-sm);
+}
+
+/* Set apart once opened, so it reads as a machine's notes rather than as the
+   goal's own definition. */
+.claude-text__body {
+  margin: 8px 0 0;
+  padding: 8px 10px;
+  background: var(--surface);
+  border-radius: var(--radius-sm);
+  color: var(--ink-soft);
+  font-size: 13px;
+  line-height: 1.5;
+  white-space: pre-wrap;
+}
+
+@media (prefers-reduced-motion: reduce) {
+  .claude-text__toggle::before { transition: none; }
+}

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

@@ -58,7 +58,15 @@
             <div class="goal-detail__head">
               <div style="flex:1;min-width:260px">
                 <h2 class="goal-detail__title" id="goal-detail-h">…</h2>
-                <p class="goal-detail__desc" id="goal-detail-desc"></p>
+                <!-- task 1003224: the goal's explanation, subordinate to its name. -->
+                <p class="goal-detail__subtitle" id="goal-detail-subtitle" hidden></p>
+                <!-- The bulk prose Claude writes. Collapsed by DEFAULT and marked as
+                     Claude's text: the owner reads the name and the punched-in points,
+                     and opens this only when they want the long form. -->
+                <details class="claude-text" id="goal-detail-claude" hidden>
+                  <summary class="claude-text__toggle">Claude's notes on this goal</summary>
+                  <p class="claude-text__body goal-detail__desc" id="goal-detail-desc"></p>
+                </details>
                 <p class="goal-card__meta" id="goal-detail-meta" style="margin-top:10px"></p>
               </div>
               <div class="goal-detail__actions" id="goal-detail-actions"></div>

package/modules/lifecycle/db-goals.js

@@ -32,7 +32,7 @@ const { deriveRequiredRank, highestRank } = require('./db-rank-authz.js');
 // Shared column list so every goal read/return shapes a row identically.
 
 const GOAL_COLS =
-  'id, version_id, title, description, scope_modules, status, visibility, created_by, succeeded_by_goal_id, sort_order, created_at, updated_at';
+  'id, version_id, title, subtitle, description, scope_modules, status, visibility, created_by, succeeded_by_goal_id, sort_order, created_at, updated_at';
 
 // Shared column list for a goal_membership_requests row (task 1951 / ADR 0112) —
 // the invite / join-request queue behind private goals. Every helper below shapes
@@ -48,7 +48,7 @@ const MEMBERSHIP_REQUEST_COLS =
 // goals) inserts no member — those goals have no owner; Archons act as one.
 
 async function createGoal(
-  { versionId, title, description = null, scopeModules = [], status = 'open', createdBy = null },
+  { versionId, title, subtitle = null, description = null, scopeModules = [], status = 'open', createdBy = null },
   deps = {}
 ) {
   const activePool = (deps.pool && process.env.NODE_ENV === 'test') ? deps.pool : pool;
@@ -56,15 +56,15 @@ async function createGoal(
     `WITH next_order AS (
        SELECT COALESCE(MAX(sort_order) + 1, 0) AS n FROM goals WHERE version_id = $1
      ), g AS (
-       INSERT INTO goals (version_id, title, description, scope_modules, status, created_by, sort_order)
-            SELECT $1, $2, $3, $4, $5, $6, next_order.n FROM next_order
+       INSERT INTO goals (version_id, title, subtitle, description, scope_modules, status, created_by, sort_order)
+            SELECT $1, $2, $7, $3, $4, $5, $6, next_order.n FROM next_order
        RETURNING ${GOAL_COLS}
      ), first_lead AS (
        INSERT INTO goal_members (goal_id, builder_id, membership_kind)
        SELECT g.id, $6::bigint, 'lead' FROM g WHERE $6::bigint IS NOT NULL
      )
      SELECT * FROM g`,
-    [versionId, title, description, scopeModules, status, createdBy]
+    [versionId, title, description, scopeModules, status, createdBy, subtitle]
   );
   return rows[0];
 }

package/modules/lifecycle/goal-edits.js

@@ -91,24 +91,26 @@ function proseDelta(field, from, to) {
 async function updateGoalProse({ goalId, patch = {} }, deps = {}) {
   const activePool = (deps.pool && process.env.NODE_ENV === 'test') ? deps.pool : pool;
   const { rows: before } = await activePool.query(
-    `SELECT id, title, description FROM goals WHERE id = $1`,
+    `SELECT id, title, subtitle, description FROM goals WHERE id = $1`,
     [goalId]
   );
   if (before.length === 0) return null;
   const prev = before[0];
   const nextTitle = Object.prototype.hasOwnProperty.call(patch, 'title') ? patch.title : prev.title;
+  const nextSubtitle = Object.prototype.hasOwnProperty.call(patch, 'subtitle') ? patch.subtitle : prev.subtitle;
   const nextDesc = Object.prototype.hasOwnProperty.call(patch, 'description') ? patch.description : prev.description;
   const deltas = [
     proseDelta('title', prev.title, nextTitle),
+    proseDelta('subtitle', prev.subtitle, nextSubtitle),
     proseDelta('description', prev.description, nextDesc),
   ].filter(Boolean);
   if (deltas.length === 0) return { goal: prev, deltas: [], unchanged: true };
   const { rows } = await activePool.query(
     `UPDATE goals
-        SET title = $2, description = $3, updated_at = now()
+        SET title = $2, description = $3, subtitle = $4, updated_at = now()
       WHERE id = $1
-     RETURNING id, version_id, title, description, status, visibility, created_by, updated_at`,
-    [goalId, nextTitle, nextDesc]
+     RETURNING id, version_id, title, subtitle, description, status, visibility, created_by, updated_at`,
+    [goalId, nextTitle, nextDesc, nextSubtitle]
   );
   return { goal: rows[0], deltas };
 }
@@ -219,13 +221,17 @@ function goalProseHandler({ db, authorizeGoalManagement, validateOrRespond, pars
     if (id === null) return;
     if (validateOrRespond(req, res, {
       title: { type: 'string', minLength: 1, maxLength: LIMITS.TITLE },
+      // No minLength: '' is a legal subtitle, meaning deliberately blank. The
+      // column's NULL (never set) is what the UI tells apart from that.
+      subtitle: { type: 'string', maxLength: LIMITS.TITLE },
       description: { type: 'string', maxLength: LIMITS.DESCRIPTION },
     })) return;
     const patch = {};
     if (Object.prototype.hasOwnProperty.call(req.body || {}, 'title')) patch.title = req.body.title;
+    if (Object.prototype.hasOwnProperty.call(req.body || {}, 'subtitle')) patch.subtitle = req.body.subtitle;
     if (Object.prototype.hasOwnProperty.call(req.body || {}, 'description')) patch.description = req.body.description;
     if (Object.keys(patch).length === 0) {
-      return res.fail('nothing_to_update', 400, { editable: ['title', 'description'] });
+      return res.fail('nothing_to_update', 400, { editable: ['title', 'subtitle', 'description'] });
     }
     try {
       const goal = await db.getGoal(id);

package/modules/lifecycle/routes/goals.js

@@ -438,7 +438,7 @@ module.exports = function buildGoalsRouter() {
     if (validateOrRespond(req, res, {
       version_id: { required: true, type: 'string', minLength: 1, maxLength: 64 },
       title: { required: true, type: 'string', minLength: 1, maxLength: LIMITS.TITLE },
-      description: { type: 'string', maxLength: LIMITS.DESCRIPTION },
+      subtitle: { type: 'string', maxLength: LIMITS.TITLE }, description: { type: 'string', maxLength: LIMITS.DESCRIPTION },
       scope_modules: { type: 'array', maxItems: 64, itemsType: 'string' },
       succeeds_goal_id: { type: 'integer', min: 1 },
     })) return;
@@ -474,7 +474,7 @@ module.exports = function buildGoalsRouter() {
       const goal = await db.createGoal({
         versionId: req.body.version_id,
         title: req.body.title,
-        description: (req.body.description || '').trim() || null,
+        subtitle: (req.body.subtitle || '').trim() || null, description: (req.body.description || '').trim() || null,
         scopeModules,
         createdBy: req.builder.id,
       });
@@ -640,7 +640,7 @@ module.exports = function buildGoalsRouter() {
     }
   });
 
-  // PATCH /goals/:id — correct a goal's own title/description (task 1003102).
+  // PATCH /goals/:id — correct a goal's own title/subtitle/description (tasks 1003102, 1003224).
   // Handler + rationale + rank note: ../goal-edits.js. Pinned in route-rank-check.
   router.patch('/goals/:id', auth.requireBuilder, goalEdits.goalProseHandler({ db, authorizeGoalManagement, validateOrRespond, parseId, LIMITS }));
   // PATCH /goals/:id/visibility — make a goal public|private (task 1952 / ADR 0112).

package/package-lock.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.322",
+  "version": "1.19.324",
   "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/architect-audit.js

@@ -32,6 +32,7 @@ const fs = require('node:fs');
 const path = require('node:path');
 const { execFileSync } = require('node:child_process');
 const { apiCall, cliClient, arg } = require('./cli-lib');
+const { resolveDefaultVersion } = require('./version-select.js');
 const grader = require('../../modules/grading/grader');
 const llmCache = require('../../src/bongos/llm-cache'); // System 5 (task 1360): cache identical audits over an unchanged tree
 const { branding } = require('../../src/branding');
@@ -183,7 +184,12 @@ async function main() {
   const args = process.argv.slice(2);
   const scope = arg('--scope', args);
   const paths = arg('--paths', args);
-  const version = arg('--version', args) || 'GDS-V3';
+  // task 1003166 (G02): was 'GDS-V3', long since shipped.
+  const version = arg('--version', args) || await resolveDefaultVersion();
+  if (!version) {
+    console.error('architect-audit: no version is building; pass --version <id>.');
+    process.exit(1);
+  }
   const dryRun = args.includes('--dry-run');
 
   if (!scope || !paths) {

package/scripts/gds/audit-deps.js

@@ -21,6 +21,7 @@
 
 const { touchesOverlap } = require('../../src/bongos/path-match.js');
 const { cliClient } = require('./cli-lib');
+const { resolveDefaultVersion } = require('./version-select.js');
 // task 1003153 — the DEAD-edge detector core, shared with the claim refusal so
 // "abandoned dependency" means one thing across the CLI, the route and this audit.
 const { findStrandedTasks } = require('../../modules/lifecycle/dead-deps.js');
@@ -128,7 +129,16 @@ async function main() {
     process.exitCode = await reportStranded({ asJson: argv.includes('--json') });
     return;
   }
-  const ver = argv.includes('--version') ? argv[argv.indexOf('--version') + 1] : 'GDS-V3';
+  // task 1003166 (G02): the default was 'GDS-V3', which has shipped — so a bare
+  // `audit-deps` audited a closed version and reported nothing useful.
+  const ver = argv.includes('--version')
+    ? argv[argv.indexOf('--version') + 1]
+    : await resolveDefaultVersion();
+  if (!ver) {
+    console.error('audit-deps: no version is building; pass --version <id>.');
+    process.exitCode = 1;
+    return;
+  }
   const minRank = argv.includes('--min') ? argv[argv.indexOf('--min') + 1] : 'low';
   const asJson = argv.includes('--json');
   const order = { high: 3, medium: 2, low: 1 };

package/scripts/gds/capture.js

@@ -54,22 +54,10 @@ const readline = require('node:readline');
 const { cliClient, requireSession, arg, hasFlag } = require('./cli-lib');
 
 
-async function resolveDefaultVersion() {
-  const api = await cliClient();
-  const r = await api.versions.getVersions();
-  if (!r.ok) return null;
-  const versions = r.data?.versions ?? [];
-  // Most recent building version. Fall back to any building version if
-  // started_at is null on all of them.
-  const building = versions.filter((v) => v.status === 'building');
-  if (building.length === 0) return null;
-  building.sort((a, b) => {
-    const ta = a.started_at ? new Date(a.started_at).getTime() : 0;
-    const tb = b.started_at ? new Date(b.started_at).getTime() : 0;
-    return tb - ta;
-  });
-  return building[0].id;
-}
+// task 1003166 (G02): the selection rule moved to version-select.js so the nine
+// scripts that used to hardcode a version id share ONE implementation. Kept as a
+// local name so every call site below reads unchanged.
+const { resolveDefaultVersion } = require('./version-select.js');
 
 // Full JSON, not console.error's depth-2 inspect — a validation 400's nested
 // details otherwise print as "[ [Object] ]" and hide which field failed.

package/scripts/gds/fitness.js

@@ -1910,7 +1910,7 @@ const CHECKS = [
   checkWriteRoutesValidated, // Check 18 — BV1 R12 / task 1999: every body-reading write route validates (ADR 0118)
   require('./fitness-ratchets.js').checkQualityRatchets, // Check 20 — task 1003129: quality budgets only tighten (mechanics + knip CI step live in fitness-ratchets.js)
   require('./doc-cli-guard.js').checkDocNamesRealCli, // Check 22 — task 1003165 / G01: a doc that names a CLI invocation must match the real CLI
-  require('./skill-preflight.js').checkOpsSkillsDeclareMachine, // Check 23 — task 1003167 / G03: an ops skill declares the machine it needs
+  require('./skill-preflight.js').checkOpsSkillsDeclareMachine, require('./version-literal-guard.js').checkNoVersionLiterals, // Checks 23 (task 1003167/G03: a skill declares its machine) + 24 (task 1003166/G02: no script hardcodes a version id) — one line because fitness.js is 500 over the excess-lines ratchet and a new check must not cost a line (idea 1000810)
   require('./control-manifest.js').checkControlsExist, // Check 21 — task 1003131: goal-1000076 controls assert their own existence (the ADR 0022 assurance-of-assurance)
 ];
 

package/scripts/gds/session-eval.js

@@ -19,6 +19,7 @@
 // Pure helper (no I/O) — resolves the per-rank wandering default so the
 // off-task signal below only fires for the CONSTRAINING ranks (task 1268).
 const wanderingPrefs = require('../../modules/builder-settings/wandering-prefs'); // carved → modules/builder-settings/ (BV1.R80 / ADR 0093 §1)
+const { resolveDefaultVersion } = require('./version-select.js');
 
 // ---- classification vocabularies -----------------------------------------
 
@@ -547,6 +548,9 @@ if (require.main === module) {
         : []);
       let filed = 0;
       let skipped = 0;
+      // task 1003166 (G02): resolved ONCE — it is invariant for the run, and this
+      // is a network round trip. The first version of this called it per finding.
+      const suggestedVersion = await resolveDefaultVersion();
       for (const s of systemic) {
         const tmpl = FINDING_TEMPLATES[s.type] || { title: s.type, body: '' };
         const title = `${PREFIX} ${tmpl.title}`.slice(0, 200);
@@ -561,7 +565,8 @@ if (require.main === module) {
           body: {
             title,
             body_md: bodyMd,
-            suggested_version: 'GDS-V4',
+            // task 1003166 (G02): was 'GDS-V4', which has shipped.
+            suggested_version: suggestedVersion,
           },
         }).catch(() => null);
         if (r && r.ok) filed++;

package/scripts/gds/ship-flow.js

@@ -20,6 +20,10 @@ const { autoLinkifyChangedDocs, mergeMainIntoBranch, preflight } = require('./sh
 const { applyBaseOverride, computeChangedFiles, computeGraderBaseline, runSmokeAsync } = require('./ship-grade.js');
 const { buildPriorRoundContext, computeGradeResult, gradeAndRecord, printCreditsAndAchievements, printGradeOutcome } = require('./ship-grade-steps.js');
 const { apiErrorLine, normalizeApiError } = require('./ship-io.js');
+// task 1003205's carve left pushVia() used at regradeMain but never imported, so
+// EVERY resume of a 'completed' task died with ReferenceError before resolving.
+const { pushVia } = require('./ship-deploy-target.js');
+
 const { diffstatFor, finishShip } = require('./ship-finish.js');
 const { evaluateRegradeEligibility, evaluateResumeAction, resumeMerge } = require('./ship-resume.js');
 const { noteServerReward } = require('./ship-state.js');

package/scripts/gds/smoke-batch-claim.sh

@@ -24,6 +24,14 @@ source "$(dirname "$0")/lib.sh"
 API="$(gds_api_base)"
 TOKEN="$(gds_resolve_token)" || exit $?
 
+# task 1003166 (G02): resolve the version at RUNTIME rather than naming one. A
+# literal works until that version ships, then `version_closed` refuses every
+# create and the smoke fails as if the code broke (idea 1000823).
+VERSION_ID=$(node "$(dirname "$0")/version-select.js") || {
+  echo "FAIL: could not resolve a building version — see version-select.js" >&2
+  exit 1
+}
+
 ts=$(date +%Y%m%d-%H%M%S)
 
 # Wrapper: METHOD PATH [BODY] → echoes "HTTP_CODE\nPAYLOAD". Never exits.
@@ -49,7 +57,7 @@ mk_task() {
 import json,sys
 t = '${touches_csv}'.split(',')
 print(json.dumps({
-  'version_id': 'BONGOS-V1',
+  'version_id': '${VERSION_ID}',
   'title': '__smoke__ batch-claim ${ts}-${suffix}',
   'description': 'smoke',
   'status': 'ready',

package/scripts/gds/smoke-batch-release.sh

@@ -31,6 +31,14 @@ source "$(dirname "$0")/lib.sh"
 API="$(gds_api_base)"
 TOKEN="$(gds_resolve_token)" || exit $?
 
+# task 1003166 (G02): resolve the version at RUNTIME rather than naming one. A
+# literal works until that version ships, then `version_closed` refuses every
+# create and the smoke fails as if the code broke (idea 1000823).
+VERSION_ID=$(node "$(dirname "$0")/version-select.js") || {
+  echo "FAIL: could not resolve a building version — see version-select.js" >&2
+  exit 1
+}
+
 ts=$(date +%Y%m%d-%H%M%S)
 
 # Wrapper: METHOD PATH [BODY] → echoes "PAYLOAD\nHTTP_CODE". Never exits.
@@ -55,7 +63,7 @@ mk_task() {
 import json,sys
 t = '${touches_csv}'.split(',')
 print(json.dumps({
-  'version_id': 'BONGOS-V1',
+  'version_id': '${VERSION_ID}',
   'title': '__smoke__ batch-release ${ts}-${suffix}',
   'description': 'smoke',
   'status': 'ready',

package/scripts/gds/smoke-claim-consistency.sh

@@ -86,7 +86,7 @@ create_task() {
   body=$(python3 -c "
 import json
 print(json.dumps({
-  'version_id': 'PMS-V2',
+  'version_id': '${VERSION_ID}',
   'title': '${title}',
   'description': 'Synthetic smoke task. Auto-cleaned by smoke-claim-consistency.sh.',
   'status': 'ready',
@@ -124,6 +124,15 @@ echo "API: ${API}"
 echo "Titles: ${SMOKE_TITLE_A} | ${SMOKE_TITLE_B}"
 echo
 
+# task 1003166 (G02): resolve the version at RUNTIME. This file used to hardcode
+# one; that version shipped, `version_closed` refused every create, and the smoke
+# could not pass while /merge-mode still advertised it green (idea 1000823).
+VERSION_ID=$(node "$(dirname "$0")/version-select.js") || {
+  echo "FAIL: could not resolve a building version — see version-select.js" >&2
+  exit 1
+}
+echo "Version: ${VERSION_ID}"
+
 echo "--- 1. create 2 smoke tasks ---"
 TASK_A=$(create_task "${SMOKE_TITLE_A}" "A")
 TASK_B=$(create_task "${SMOKE_TITLE_B}" "B")

package/scripts/gds/smoke-write.sh

@@ -58,6 +58,15 @@ echo "API: ${API}"
 echo "Title: ${SMOKE_TITLE}"
 echo
 
+# task 1003166 (G02): resolve the version at RUNTIME. This file used to hardcode
+# one; that version shipped, `version_closed` refused every create, and the smoke
+# could not pass while /merge-mode still advertised it green (idea 1000823).
+VERSION_ID=$(node "$(dirname "$0")/version-select.js") || {
+  echo "FAIL: could not resolve a building version — see version-select.js" >&2
+  exit 1
+}
+echo "Version: ${VERSION_ID}"
+
 echo "--- 0. snapshot progress before ---"
 PROGRESS_BEFORE=$(snapshot_progress)
 echo "${PROGRESS_BEFORE}"
@@ -67,7 +76,7 @@ echo "--- 1. create task ---"
 CREATE_BODY=$(python3 -c "
 import json
 print(json.dumps({
-  'version_id': 'PMS-V1',
+  'version_id': '${VERSION_ID}',
   'title': '${SMOKE_TITLE}',
   'description': 'Synthetic smoke task. Auto-shipped by smoke-write.sh.',
   'status': 'ready',

package/scripts/gds/triage.js

@@ -38,6 +38,7 @@
 const readline = require('node:readline');
 const fs = require('node:fs');
 const { cliClient, requireSession } = require('./cli-lib');
+const { resolveDefaultVersion } = require('./version-select.js');
 
 function prompt(rl, question) {
   return new Promise((resolve) => rl.question(question, (a) => resolve(a)));
@@ -384,7 +385,9 @@ async function main() {
 
         if (ans === 'p' || ans === 'promote') {
           // Promote: version, priority, title, goal — each defaulted from the idea.
-          const defaultVersion = idea.suggested_version || 'PMS-V2';
+          // task 1003166 (G02): the fallback was 'PMS-V2', which has shipped —
+          // promoting into it would have been refused as version_closed.
+          const defaultVersion = idea.suggested_version || await resolveDefaultVersion();
           const versionAns = (await prompt(rl, `  version [${defaultVersion}]: `)).trim();
           const versionId = versionAns || defaultVersion;