@basou/cli 0.46.0 → 0.48.0

package/dist/program.js

@@ -1250,7 +1250,8 @@ Input format (a JSON array; one object per decision):
       "rationale": "Workspace protocol and a content-addressed store fit our layout.",
       "alternatives": ["npm workspaces", "yarn"],
       "rejected_reason": "npm hoisting caused phantom-dependency bugs",
-      "linked_files": ["pnpm-workspace.yaml"]
+      "linked_files": ["pnpm-workspace.yaml"],
+      "kind": "decision"
     },
     {
       "title": "Form-based admin editing is the next track (only 6/19 sections done)",
@@ -1259,10 +1260,16 @@ Input format (a JSON array; one object per decision):
     }
   ]
 
-Only "title" is required; every other field is optional. Set "kind": "track" to
-record a strategic, UNFINISHED direction (+ why): orientation/handoff resurface
-open tracks every session until you close one with 'basou decision void <id>'.
-Absent / "decision" is a point-in-time decision (surfaced only as the latest).
+"title" and "kind" are both required. "kind" names the vessel: "track" records a
+strategic, UNFINISHED direction (+ why) and orientation/handoff resurface it
+every session until you close it with 'basou decision void <id>', while
+"decision" records a point-in-time call (surfaced as the latest). Every other
+field is optional.
+
+"kind" has no default because getting it wrong fails silently: a track filed as
+a decision never enters the open-track list, so nothing holds it there -- the
+next decision simply takes its place. Omitting it on ANY item refuses the whole
+batch and writes nothing, so a partial write can never be reported as a success.
 All decisions are written into one ad-hoc session timestamped now, so
 orientation surfaces them as the latest decisions. Run from a workspace-view
 directory and it resolves to the planning repo, like 'basou orient' /
@@ -1270,7 +1277,7 @@ directory and it resolves to the planning repo, like 'basou orient' /
 
 Example (heredoc on stdin):
   basou decision capture <<'JSON'
-  [{ "title": "Ship the capture command", "rationale": "Close the why-capture gap" }]
+  [{ "title": "Ship the capture command", "rationale": "Close the why-capture gap", "kind": "decision" }]
   JSON
 `;
 async function runDecisionRecord(options, ctx = {}) {
@@ -1306,15 +1313,6 @@ async function warnLinkedFilesOutsideRoots(input) {
   } catch {
   }
 }
-var TRACK_MARKER_IN_TITLE = /^\s*track\s*[:\uff1a]|[[(\uff08\uff3b\u3010]\s*track\s*[\])\uff09\uff3d\u3011]/i;
-function warnTrackMarkerWithoutKind(decisions, markerWithoutKind) {
-  for (const index of markerWithoutKind) {
-    const title = (decisions[index]?.title ?? "").trim();
-    console.error(
-      `basou: decision[${index}] title carries a track marker (${title.slice(0, 40)}) but "kind" is absent \u2014 it is recorded as a point-in-time decision and will NOT resurface in orient. Set "kind": "track" to open a track, or "kind": "decision" to say the marker is part of the title.`
-    );
-  }
-}
 async function doRunDecisionRecord(options, ctx) {
   const cwd = ctx.cwd ?? process.cwd();
   const repositoryRoot = await resolveRepositoryRootForDecision(cwd);
@@ -1412,8 +1410,7 @@ async function doRunDecisionCapture(options, ctx) {
   const paths = basouPaths4(repositoryRoot);
   await assertWorkspaceInitialized2(paths.root);
   const raw = await readCaptureInput(options, ctx);
-  const { decisions, markerWithoutKind } = parseCaptureInput(raw);
-  warnTrackMarkerWithoutKind(decisions, markerWithoutKind);
+  const { decisions } = parseCaptureInput(raw);
   await warnLinkedFilesOutsideRoots({
     linkedFiles: decisions.flatMap((d) => d.linked_files ?? []),
     cwd,
@@ -1675,13 +1672,31 @@ function parseCaptureInput(raw) {
     throw new Error("Input array must contain at least one decision.");
   }
   const decisions = [];
-  const markerWithoutKind = [];
-  parsed.forEach((item, index) => {
-    const { input, kindWasPresent } = validateCaptureItem(item, index);
-    decisions.push(input);
-    if (!kindWasPresent && TRACK_MARKER_IN_TITLE.test(input.title)) markerWithoutKind.push(index);
-  });
-  return { decisions, markerWithoutKind };
+  const missingKind = [];
+  try {
+    parsed.forEach((item, index) => {
+      const input = validateCaptureItem(item, index);
+      decisions.push(input);
+      if (input.kind === void 0) missingKind.push(index);
+    });
+  } catch (error) {
+    const detail = error instanceof Error ? error.message : String(error);
+    throw new Error(`${detail} ${batchDisposition(parsed.length)}`, { cause: error });
+  }
+  if (missingKind.length > 0) {
+    throw new Error(`${missingKindError(missingKind)} ${batchDisposition(parsed.length)}`);
+  }
+  return { decisions };
+}
+var MISSING_KIND_SAMPLE = 10;
+function batchDisposition(total) {
+  return `Nothing was written: all ${total} item(s) in this batch were refused.`;
+}
+function missingKindError(indices) {
+  const shown = indices.slice(0, MISSING_KIND_SAMPLE).map((i) => `decision[${i}]`);
+  const overflow = indices.length - shown.length;
+  const list = overflow > 0 ? `${shown.join(", ")} (... +${overflow} more)` : shown.join(", ");
+  return `${list}: "kind" is required. It names the vessel: "track" for an unfinished direction, which orient keeps in its open-track list until you close it with 'basou decision void', or "decision" for a settled point-in-time call, which the next decision replaces. Add "kind" to every item and re-run.`;
 }
 function validateCaptureItem(item, index) {
   if (typeof item !== "object" || item === null || Array.isArray(item)) {
@@ -1698,12 +1713,12 @@ function validateCaptureItem(item, index) {
   if (typeof obj.title !== "string" || isBlank(obj.title)) {
     throw new Error(`decision[${index}].title must be a non-empty string.`);
   }
+  if (obj.kind !== void 0 && obj.kind !== "decision" && obj.kind !== "track") {
+    throw new Error(`decision[${index}].kind must be "decision" or "track", got '${obj.kind}'.`);
+  }
   const out = { title: obj.title };
   if (obj.kind !== void 0) {
-    if (obj.kind !== "decision" && obj.kind !== "track") {
-      throw new Error(`decision[${index}].kind must be "decision" or "track", got '${obj.kind}'.`);
-    }
-    if (obj.kind === "track") out.kind = "track";
+    out.kind = obj.kind;
   }
   if (obj.rationale !== void 0) {
     out.rationale = requireNonEmptyString(obj.rationale, index, "rationale");
@@ -1742,7 +1757,7 @@ function validateCaptureItem(item, index) {
       }
     });
   }
-  return { input: out, kindWasPresent: obj.kind !== void 0 };
+  return out;
 }
 function requireNonEmptyString(value, index, field) {
   if (typeof value !== "string" || isBlank(value)) {
@@ -1793,9 +1808,12 @@ function captureItemToPayload(item) {
   if (item.input.kind !== void 0) payload.kind = item.input.kind;
   return payload;
 }
-function kindMarker(kind) {
+function previewKindMarker(kind) {
   return kind === "track" ? " [TRACK]" : " [DECISION]";
 }
+function recordedKindMarker(kind) {
+  return kind === "track" ? " [TRACK]" : "";
+}
 function printCapturePreview(options, decisions) {
   if (options.json === true) {
     console.log(JSON.stringify({ dry_run: true, count: decisions.length, decisions }));
@@ -1805,7 +1823,7 @@ function printCapturePreview(options, decisions) {
     `Would capture ${decisions.length} decision${decisions.length === 1 ? "" : "s"} (dry run; nothing written):`
   );
   for (const decision of decisions) {
-    console.log(`- ${decision.title}${kindMarker(decision.kind)}`);
+    console.log(`- ${decision.title}${previewKindMarker(decision.kind)}`);
   }
 }
 function printCaptureResult(options, result) {
@@ -1826,7 +1844,7 @@ function printCaptureResult(options, result) {
     `Captured ${result.items.length} decision${result.items.length === 1 ? "" : "s"} in ad-hoc session ${sid}:`
   );
   for (const item of result.items) {
-    console.log(`- ${item.decisionId}: ${item.input.title}${kindMarker(item.input.kind)}`);
+    console.log(`- ${item.decisionId}: ${item.input.title}${recordedKindMarker(item.input.kind)}`);
   }
 }
 function pickRichFields(options) {
@@ -1933,13 +1951,14 @@ function printDecisionResult(options, result) {
   }
   const trackPrefix = result.rich.kind === "track" ? "track " : "";
   const rationaleSuffix = result.rich.rationale !== void 0 ? ` (rationale: ${result.rich.rationale})` : "";
+  const vesselSuffix = result.rich.kind === "track" ? "" : " \u2014 a point-in-time decision, not an open track (use --track for a direction that should keep coming back until you close it)";
   if (result.mode === "ad-hoc") {
     console.log(
-      `Recorded ${trackPrefix}${result.decisionId} in ad-hoc session ${sid}${rationaleSuffix}`
+      `Recorded ${trackPrefix}${result.decisionId} in ad-hoc session ${sid}${rationaleSuffix}${vesselSuffix}`
     );
   } else {
     console.log(
-      `Recorded ${trackPrefix}${result.decisionId} in session ${sid} (${result.sessionStatus})${rationaleSuffix}`
+      `Recorded ${trackPrefix}${result.decisionId} in session ${sid} (${result.sessionStatus})${rationaleSuffix}${vesselSuffix}`
     );
   }
 }
@@ -12705,6 +12724,18 @@ var VIEW_HTML = `<!doctype html>
     return el('span', { class: 'badge ok', text: 'up to date' });
   }
 
+  // Three states hide behind a bare 'in-flight 0': nothing ever recorded,
+  // everything finished, and a task store that cannot be read -- where the
+  // count is not zero but UNKNOWN. Say which one it is, as orient does.
+  // Kept as a named top-level function with no free variables so the test suite
+  // can lift it out of this template and run it; inlined in the card it was
+  // unreachable from any test.
+  function taskFlightLabel(w) {
+    if (w.unreadableTaskCount > 0) return 'in-flight unknown (' + w.unreadableTaskCount + ' unreadable)';
+    if (w.anyTaskEverRecorded === false) return 'no tasks recorded';
+    return 'in-flight ' + w.inFlightCount;
+  }
+
   function portfolioCard(w, generatedAt) {
     if (!w.initialized) {
       return el('div', { class: 'card pcard muted' }, [
@@ -12720,6 +12751,7 @@ var VIEW_HTML = `<!doctype html>
     }
     var pend = w.pendingApprovals || [];
     var pendText = 'pending ' + pend.length + (pend.length ? ' (' + highestRisk(pend) + ')' : '');
+    var flightText = taskFlightLabel(w);
     var now = w.latestSession ? ((w.latestSession.label || '(session)') + ' [' + w.latestSession.status + ']') : '(no live sessions)';
     var dec = w.latestDecision ? w.latestDecision.title : '(no decisions yet)';
     var newest = (w.freshness && w.freshness.newestStartedAt) ? w.freshness.newestStartedAt : null;
@@ -12732,7 +12764,7 @@ var VIEW_HTML = `<!doctype html>
       ]),
       el('div', { class: 'f', text: 'now: ' + now }),
       el('div', { class: 'f', text: 'latest: ' + dec }),
-      el('div', { class: 'f', text: 'in-flight ' + w.inFlightCount + '  |  ' + pendText + '  |  suspect ' + w.suspectCount }),
+      el('div', { class: 'f', text: flightText + '  |  ' + pendText + '  |  suspect ' + w.suspectCount }),
       el('div', { class: 'f muted', text: 'sessions ' + w.sessionCount + '  |  newest ' + relAge(newest, generatedAt) })
     ]);
   }
@@ -13309,6 +13341,13 @@ async function portfolioCard(ws, nowIso) {
       sessionCount: s.sessionCount,
       suspectCount: s.suspects.length,
       inFlightCount: s.inFlightTasks.length,
+      // Three states hide behind `inFlightCount: 0`: nothing ever recorded,
+      // everything finished, and a store whose task files cannot be read (where
+      // the count is not 0 but UNKNOWN). `orient` branches all three ways; the
+      // card carries the two extra facts so it can too, instead of reporting a
+      // loader blind spot as an empty record.
+      anyTaskEverRecorded: s.anyTaskEverRecorded,
+      unreadableTaskCount: s.unreadableTaskCount,
       pendingApprovals: s.pendingApprovals.map((a) => ({
         risk: a.risk,
         kind: a.kind,
@@ -13776,7 +13815,7 @@ async function assertWorkspaceInitialized15(basouRoot) {
 function readBuildStamp() {
   if (false) return void 0;
   try {
-    return JSON.parse('{"version":"0.46.0","commit":"dbc9c57","committedAt":"2026-09-19T15:43:35+09:00"}');
+    return JSON.parse('{"version":"0.48.0","commit":"2e8fee7","committedAt":"2026-09-21T11:35:12+09:00"}');
   } catch {
     return void 0;
   }