@basou/cli 0.46.0 → 0.47.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,14 @@ 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" is 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 only as the latest). Every other field is optional.
+
+DEPRECATED: omitting "kind" still writes the item, as a decision, but warns --
+and becomes an ERROR in the next release. It has no default because getting it
+wrong fails silently: a track filed as a decision simply never comes back.
 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 +1275,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,12 +1311,11 @@ 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) {
+function warnMissingKind(decisions, indices) {
+  for (const index of indices) {
     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.`
+      `basou: decision[${index}] ("${title.slice(0, 40)}") declares no "kind" \u2014 it is recorded as a point-in-time decision and will NOT resurface in orient. Set "kind": "track" for an unfinished direction, or "kind": "decision" to say it is settled. Omitting it becomes an ERROR in the next release.`
     );
   }
 }
@@ -1412,8 +1416,8 @@ 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, missingKind } = parseCaptureInput(raw);
+  warnMissingKind(decisions, missingKind);
   await warnLinkedFilesOutsideRoots({
     linkedFiles: decisions.flatMap((d) => d.linked_files ?? []),
     cwd,
@@ -1675,13 +1679,13 @@ function parseCaptureInput(raw) {
     throw new Error("Input array must contain at least one decision.");
   }
   const decisions = [];
-  const markerWithoutKind = [];
+  const missingKind = [];
   parsed.forEach((item, index) => {
-    const { input, kindWasPresent } = validateCaptureItem(item, index);
+    const input = validateCaptureItem(item, index);
     decisions.push(input);
-    if (!kindWasPresent && TRACK_MARKER_IN_TITLE.test(input.title)) markerWithoutKind.push(index);
+    if (input.kind === void 0) missingKind.push(index);
   });
-  return { decisions, markerWithoutKind };
+  return { decisions, missingKind };
 }
 function validateCaptureItem(item, index) {
   if (typeof item !== "object" || item === null || Array.isArray(item)) {
@@ -1703,7 +1707,7 @@ function validateCaptureItem(item, index) {
     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 +1746,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,8 +1797,13 @@ function captureItemToPayload(item) {
   if (item.input.kind !== void 0) payload.kind = item.input.kind;
   return payload;
 }
-function kindMarker(kind) {
-  return kind === "track" ? " [TRACK]" : " [DECISION]";
+function previewKindMarker(kind) {
+  if (kind === "track") return " [TRACK]";
+  return kind === "decision" ? " [DECISION]" : " [NO KIND]";
+}
+function recordedKindMarker(kind) {
+  if (kind === "track") return " [TRACK]";
+  return kind === "decision" ? "" : " [NO KIND]";
 }
 function printCapturePreview(options, decisions) {
   if (options.json === true) {
@@ -1805,7 +1814,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 +1835,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) {
@@ -12705,6 +12714,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 +12741,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 +12754,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 +13331,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 +13805,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.47.0","commit":"831e0cb","committedAt":"2026-09-20T22:28:13+09:00"}');
   } catch {
     return void 0;
   }