@trazum/cli 1.46.0 → 1.47.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trazum/cli",
-  "version": "1.46.0",
+  "version": "1.47.0",
   "description": "Trazum CLI: find where your LLM bill goes, price every finding per month, and enforce token budgets in CI.",
   "license": "MIT",
   "author": "David Mu\u00f1oz Rey",
@@ -37,7 +37,7 @@
     "prepublishOnly": "npm run build && npm test"
   },
   "dependencies": {
-    "@trazum/core": "1.46.0"
+    "@trazum/core": "1.47.0"
   },
   "devDependencies": {
     "@types/node": "^26.2.0",

package/src/i18n/en.ts

@@ -1814,8 +1814,18 @@ ${bold('EXAMPLES')}
       'Point this at a saved plan and a newer log: trazum verify plan.json --against usage.jsonl. It says, per action, whether the change arrived, did not arrive, or cannot be told — and never fewer than those three.',
     needsAgainst: () =>
       '--against <newer.jsonl|dir> is required. A plan can only be verified against a log that came after it; without one there is nothing to hold the prediction to.',
-    badPlan: (path) =>
-      `${path} is not a plan document this tool can verify — expected the JSON that "trazum plan -o" writes (schemaVersion 1, with an actions array).`,
+    badPlan: (path, why) =>
+      `${path} is not a plan document this tool can verify: ${why}. Expected the JSON that "trazum plan -o" writes.`,
+    planRefusal: (why) =>
+      why.kind === 'not-json'
+        ? 'it is not valid JSON'
+        : why.kind === 'not-an-object'
+          ? 'the top level is not a JSON object'
+          : why.kind === 'wrong-schema-version'
+            ? `schemaVersion is ${JSON.stringify(why.found)} rather than 1`
+            : why.kind === 'actions-not-a-list'
+              ? 'there is no actions array'
+              : `action ${why.index + 1} is malformed (${why.because})`,
     heading: (actions, planDate) =>
       planDate === null
         ? `Did it work? ${actions} actions from an undated plan, against this log`

package/src/i18n/es.ts

@@ -1844,8 +1844,18 @@ ${bold('EJEMPLOS')}
       'Apunta esto a un plan guardado y a un registro posterior: trazum verify plan.json --against usage.jsonl. Dice, por acción, si el cambio llegó, no llegó o no se puede saber — y nunca menos de esos tres.',
     needsAgainst: () =>
       '--against <nuevo.jsonl|dir> es obligatorio. Un plan solo puede verificarse contra un registro posterior; sin uno no hay nada a lo que someter la predicción.',
-    badPlan: (path) =>
-      `${path} no es un documento de plan que esta herramienta pueda verificar — se esperaba el JSON que escribe "trazum plan -o" (schemaVersion 1, con un array actions).`,
+    badPlan: (path, why) =>
+      `${path} no es un documento de plan que esta herramienta pueda verificar: ${why}. Se esperaba el JSON que escribe "trazum plan -o".`,
+    planRefusal: (why) =>
+      why.kind === 'not-json'
+        ? 'no es JSON válido'
+        : why.kind === 'not-an-object'
+          ? 'el nivel superior no es un objeto JSON'
+          : why.kind === 'wrong-schema-version'
+            ? `schemaVersion es ${JSON.stringify(why.found)} en vez de 1`
+            : why.kind === 'actions-not-a-list'
+              ? 'no hay un array actions'
+              : `la acción ${why.index + 1} está mal formada (${why.because})`,
     heading: (actions, planDate) =>
       planDate === null
         ? `¿Funcionó? ${actions} acciones de un plan sin fecha, contra este registro`

package/src/i18n/types.ts

@@ -1269,7 +1269,22 @@ export interface CliMessages {
     noTarget(): string;
     needsAgainst(): string;
     /** Not a plan document: wrong shape, wrong version, or not JSON at all. */
-    badPlan(path: string): string;
+    /**
+     * `why` is the typed reason from `parsePlanDocument`, rendered so the
+     * refusal names what is wrong rather than only that something is. A file
+     * that is valid JSON and not a plan and a plan with one bad action are
+     * different problems with different fixes.
+     */
+    badPlan(path: string, why: string): string;
+    /** The typed refusal from `parsePlanDocument`, in one clause. */
+    planRefusal(
+      why:
+        | { kind: 'not-json' }
+        | { kind: 'not-an-object' }
+        | { kind: 'wrong-schema-version'; found: unknown }
+        | { kind: 'actions-not-a-list' }
+        | { kind: 'action-malformed'; index: number; because: string },
+    ): string;
     heading(actions: string, planDate: string | null): string;
     counts(arrived: string, notArrived: string, cannotTell: string): string;
     /** Two price lists are two measurements; every dollar line inherits this. */

package/src/index.ts

@@ -37,6 +37,7 @@ import {
   DEFAULT_USAGE,
   detectFromSource,
   matchLocale,
+  parsePlanDocument,
   proposeInit,
   MIN_RATE_DAYS,
   parseConfig,
@@ -3546,17 +3547,21 @@ async function commandVerify(
   const againstPath = stringFlag(args, 'against');
   if (againstPath === undefined) throw new Error(t.verify.needsAgainst());
 
-  let plan: PlanDocument & { createdAt?: string };
-  try {
-    const parsed = JSON.parse(await readFile(planPath, 'utf8'));
-    if (parsed?.schemaVersion !== 1 || !Array.isArray(parsed.actions)) {
-      throw new Error(t.verify.badPlan(planPath));
-    }
-    plan = parsed;
-  } catch (error) {
-    if (error instanceof SyntaxError) throw new Error(t.verify.badPlan(planPath));
-    throw error;
+  /**
+   * One validator, shared with the browser since 1.47.
+   *
+   * The check here used to be `schemaVersion === 1 && Array.isArray(actions)`
+   * and nothing more, which accepts a file whose actions are arbitrary
+   * objects — `verifyPlan` would then read `label` off `undefined`, match it
+   * against no slice, and report `cannot-tell: workload-vanished` for every
+   * one. A verification of a document that was never a plan, rendered exactly
+   * like a real one.
+   */
+  const parsed = parsePlanDocument(await readFile(planPath, 'utf8'));
+  if (!parsed.ok) {
+    throw new Error(t.verify.badPlan(planPath, t.verify.planRefusal(parsed.why)));
   }
+  const plan = parsed.plan;
 
   const GZ = LOG_EXTENSIONS.map((ext) => `${ext}.gz`);
   const READABLE = [...LOG_EXTENSIONS, ...GZ];