@wirecat/cli-core 0.354.0 → 0.355.0

package/CHANGELOG.md

@@ -7,6 +7,21 @@ shape of this file.
 
 ## Unreleased
 
+## 0.355.0 — 19.10.2026
+
+### Fixed
+
+- **A failed scheduled run no longer keeps a chat title or a name in `last.json`** (privacy). Its `reason` was the
+  last line the run printed, which is the whole error object, message included: "no chat matches …" and the list of
+  chats or people an ambiguous name matches reached the file and the "scheduled … failing" notice. `last.json` now
+  keeps the error's `code` as the new `errorCode` and a `reason` of the code and its reason id, such as
+  `validation_error (input.invalid)`; a last line that is not an error object, such as an agent harness's, is kept
+  only as `exit <n>`. The full message stays on stderr and in the run's log, whose path `schedule list` now prints
+  after a failing schedule's reason. `last.json` is written atomically with mode 0600 even over an older file.
+  `reasonOf(stderr, exitCode?)` follows and always answers a string, and `recordRun` takes the optional `errorCode`.
+  A background `store` sync job's file likewise keeps only the error's code and names `store jobs show <id>`, which
+  prints the path of the job's log.
+
 ## 0.354.0 — 19.10.2026
 
 ### Fixed

package/dist/background/schedules.d.ts

@@ -25,15 +25,26 @@ export interface LastRun {
     output: string;
     /** Shared by every command the run started (`TRACEPARENT`), so `runs list` names them. */
     traceId?: string;
-    /** Why a failed run failed, in one line: the last thing it said, not its whole output. */
+    /** Why a failed run failed, in one line: the error's code and reason id, never its message. */
     reason?: string;
+    /** The `code` of the error the run printed, as run records keep it. */
+    errorCode?: string;
     /** The first failure of the failures in a row, so "failing since" survives every retry. */
     failingSince?: string;
     /** Failed runs in a row; absent once a run succeeds. */
     failures?: number;
 }
-/** The last line a failed run printed, which is where a CLI says why it stopped. */
-export declare const reasonOf: (output: string) => string | undefined;
+/**
+ * Why a failed run failed, from the error object the CLI printed last: its code and reason id. Never the
+ * message, which can name a chat or a person; the run's log keeps that. Any other line is another program's
+ * text and is not kept either.
+ */
+export declare const failureOf: (stderr: string, exitCode: number) => {
+    reason: string;
+    errorCode?: string;
+};
+/** The one-line reason `failureOf` keeps, for a timer that reads a run's stderr itself. */
+export declare const reasonOf: (stderr: string, exitCode?: number) => string;
 /**
  * Writes one run's result to `path`, carrying the failure streak over from the run before. Any CLI's own
  * timer can call this, so `failingNotice` reads every tool's background runs the same way.
@@ -44,6 +55,7 @@ export declare const recordRun: (path: string, run: {
     output: string;
     traceId?: string;
     reason?: string;
+    errorCode?: string;
 }) => LastRun;
 /**
  * One stderr line per failing background job, or nothing: `tg: scheduled digest failing since … (3 runs):

package/dist/background/schedules.js

@@ -1,20 +1,30 @@
 import { spawn } from "node:child_process";
 import { copyFileSync, existsSync, mkdirSync, readdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
-import { CliError, writeSecurely } from "@wirecat/cli-base";
+import { CliError, errorCodes, writeSecurely } from "@wirecat/cli-base";
 import { readOnlyLockOf } from "../ports/app.js";
 import { recordLogFor } from "../runs/record-log.js";
 import { childTrace } from "../runs/scope.js";
-const REASON_CHARS = 200;
-/** The last line a failed run printed, which is where a CLI says why it stopped. */
-export const reasonOf = (output) => {
-    const line = output
-        .split("\n")
-        .map((text) => text.trim())
-        .filter(Boolean)
-        .at(-1);
-    return line === undefined ? undefined : line.length > REASON_CHARS ? `${line.slice(0, REASON_CHARS - 1)}…` : line;
+const REASON_ID = /^[a-z0-9][a-z0-9.-]{0,79}$/;
+/**
+ * Why a failed run failed, from the error object the CLI printed last: its code and reason id. Never the
+ * message, which can name a chat or a person; the run's log keeps that. Any other line is another program's
+ * text and is not kept either.
+ */
+export const failureOf = (stderr, exitCode) => {
+    for (const line of stderr.split("\n").reverse()) {
+        try {
+            const { code, reason } = JSON.parse(line).error ?? {};
+            if (errorCodes.includes(code) && typeof reason === "string" && REASON_ID.test(reason)) {
+                return { reason: `${code} (${reason})`, errorCode: code };
+            }
+        }
+        catch { }
+    }
+    return { reason: `exit ${exitCode}` };
 };
+/** The one-line reason `failureOf` keeps, for a timer that reads a run's stderr itself. */
+export const reasonOf = (stderr, exitCode = 1) => failureOf(stderr, exitCode).reason;
 const readRun = (path) => {
     try {
         return existsSync(path) ? JSON.parse(readFileSync(path, "utf8")) : undefined;
@@ -29,17 +39,17 @@ const readRun = (path) => {
  */
 export const recordRun = (path, run) => {
     const before = readRun(path);
-    const { reason, ...rest } = run;
+    const { reason, errorCode, ...rest } = run;
     const last = run.exitCode === 0
         ? rest
         : {
             ...rest,
             ...(reason === undefined ? {} : { reason }),
+            ...(errorCode === undefined ? {} : { errorCode }),
             failingSince: before?.exitCode ? (before.failingSince ?? before.at) : run.at,
             failures: before?.exitCode ? (before.failures ?? 1) + 1 : 1,
         };
-    mkdirSync(join(path, ".."), { recursive: true, mode: 0o700 });
-    writeFileSync(path, `${JSON.stringify(last, null, 2)}\n`, { mode: 0o600 });
+    writeSecurely(path, `${JSON.stringify(last, null, 2)}\n`, 0o600);
     return last;
 };
 const shortTime = (iso) => {
@@ -371,13 +381,12 @@ export const runSchedule = async (app, system, env, file, { now = () => new Date
         exitCode,
         durationMs: ended.getTime() - began,
     });
-    const said = exitCode === 0 ? undefined : reasonOf(readFileSync(errors, "utf8"));
     const last = recordRun(join(folder, "last.json"), {
         at: ended.toISOString(),
         exitCode,
         output,
         traceId: trace.traceId,
-        ...(said === undefined ? {} : { reason: said }),
+        ...(exitCode === 0 ? {} : failureOf(readFileSync(errors, "utf8"), exitCode)),
     });
     if (agent && exitCode === 0)
         copyFileSync(output, join(folder, "latest.md"));

package/dist/cli/messenger/backfill-command.js

@@ -175,7 +175,8 @@ export const syncCommand = (messenger) => {
         catch (error) {
             if (jobId) {
                 const code = isCliFailure(error) ? error.code : "generic_failure";
-                const message = error instanceof Error ? error.message : String(error);
+                // The job file outlives the run; the message can name a chat, so only the job's log keeps it.
+                const message = `${code} — \`store jobs show ${jobId}\` names the log with the whole error`;
                 finishJob(messenger.app, jobs, jobId, { finishedAt: new Date().toISOString(), error: { code, message } });
             }
             throw error;

package/dist/cli/messenger/schedule-command.js

@@ -185,7 +185,7 @@ export const scheduleCommand = (messenger) => {
                     ? `last ${last.at.slice(0, 16).replace("T", " ")} exit ${last.exitCode}` +
                         (last.failures
                             ? ` — failing since ${(last.failingSince ?? last.at).slice(0, 16).replace("T", " ")}, ` +
-                                `${last.failures} in a row${last.reason ? `: ${last.reason}` : ""}`
+                                `${last.failures} in a row${last.reason ? `: ${last.reason}` : ""} — ${last.output}`
                             : "")
                     : "not run yet",
             ].join("  "))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wirecat/cli-core",
-  "version": "0.354.0",
+  "version": "0.355.0",
   "description": "The messenger-neutral half of a messaging CLI: domain model, message locators, rendering, name resolution, the SQLite seam, the command skeleton and shared read commands, the send guard, run records and the message store",
   "license": "Apache-2.0",
   "author": "Viacheslav Ptsarev",