@basou/cli 0.47.0 → 0.48.0

package/dist/index.js

@@ -1260,14 +1260,16 @@ Input format (a JSON array; one object per decision):
     }
   ]
 
-"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.
+"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.
 
-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.
+"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' /
@@ -1311,14 +1313,6 @@ async function warnLinkedFilesOutsideRoots(input) {
   } catch {
   }
 }
-function warnMissingKind(decisions, indices) {
-  for (const index of indices) {
-    const title = (decisions[index]?.title ?? "").trim();
-    console.error(
-      `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.`
-    );
-  }
-}
 async function doRunDecisionRecord(options, ctx) {
   const cwd = ctx.cwd ?? process.cwd();
   const repositoryRoot = await resolveRepositoryRootForDecision(cwd);
@@ -1416,8 +1410,7 @@ async function doRunDecisionCapture(options, ctx) {
   const paths = basouPaths4(repositoryRoot);
   await assertWorkspaceInitialized2(paths.root);
   const raw = await readCaptureInput(options, ctx);
-  const { decisions, missingKind } = parseCaptureInput(raw);
-  warnMissingKind(decisions, missingKind);
+  const { decisions } = parseCaptureInput(raw);
   await warnLinkedFilesOutsideRoots({
     linkedFiles: decisions.flatMap((d) => d.linked_files ?? []),
     cwd,
@@ -1680,12 +1673,30 @@ function parseCaptureInput(raw) {
   }
   const decisions = [];
   const missingKind = [];
-  parsed.forEach((item, index) => {
-    const input = validateCaptureItem(item, index);
-    decisions.push(input);
-    if (input.kind === void 0) missingKind.push(index);
-  });
-  return { decisions, 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)) {
@@ -1702,11 +1713,11 @@ 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}'.`);
-    }
     out.kind = obj.kind;
   }
   if (obj.rationale !== void 0) {
@@ -1798,12 +1809,10 @@ function captureItemToPayload(item) {
   return payload;
 }
 function previewKindMarker(kind) {
-  if (kind === "track") return " [TRACK]";
-  return kind === "decision" ? " [DECISION]" : " [NO KIND]";
+  return kind === "track" ? " [TRACK]" : " [DECISION]";
 }
 function recordedKindMarker(kind) {
-  if (kind === "track") return " [TRACK]";
-  return kind === "decision" ? "" : " [NO KIND]";
+  return kind === "track" ? " [TRACK]" : "";
 }
 function printCapturePreview(options, decisions) {
   if (options.json === true) {
@@ -1942,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}`
     );
   }
 }
@@ -13805,7 +13815,7 @@ async function assertWorkspaceInitialized15(basouRoot) {
 function readBuildStamp() {
   if (false) return void 0;
   try {
-    return JSON.parse('{"version":"0.47.0","commit":"831e0cb","committedAt":"2026-09-20T22:28:13+09:00"}');
+    return JSON.parse('{"version":"0.48.0","commit":"2e8fee7","committedAt":"2026-09-21T11:35:12+09:00"}');
   } catch {
     return void 0;
   }