agent-trajectories 0.5.7 → 0.5.8

package/dist/cli/index.js

@@ -414,7 +414,7 @@ import {
   unlink,
   writeFile
 } from "fs/promises";
-import { join } from "path";
+import { basename, dirname, isAbsolute, join, relative } from "path";
 
 // src/export/markdown.ts
 function exportToMarkdown(trajectory) {
@@ -594,6 +594,20 @@ function getSearchPaths() {
   }
   return [join(process.cwd(), ".trajectories")];
 }
+function describeReadFailure(reason, error) {
+  if (reason === "schema_violation" && error && typeof error === "object" && "issues" in error) {
+    const issues = error.issues ?? [];
+    if (issues.length > 0) {
+      const first = issues[0];
+      const where = first.path.length > 0 ? first.path.join(".") : "root";
+      const extra = issues.length > 1 ? ` (+${issues.length - 1} more)` : "";
+      return `${where}: ${first.message}${extra}`;
+    }
+    return "schema validation failed";
+  }
+  if (error instanceof Error) return error.message;
+  return String(error);
+}
 var indexLocks = /* @__PURE__ */ new Map();
 function withIndexLock(path2, task) {
   const prev = indexLocks.get(path2) ?? Promise.resolve();
@@ -610,6 +624,7 @@ var FileStorage = class {
   activeDir;
   completedDir;
   indexPath;
+  lastReconcileSummary;
   constructor(baseDir) {
     this.baseDir = baseDir ?? process.cwd();
     const dataDir = process.env.TRAJECTORIES_DATA_DIR;
@@ -659,7 +674,8 @@ var FileStorage = class {
       alreadyIndexed: 0,
       skippedMalformedJson: 0,
       skippedSchemaViolation: 0,
-      skippedIoError: 0
+      skippedIoError: 0,
+      failures: []
     };
     await withIndexLock(this.indexPath, async () => {
       const index = await this.loadIndex();
@@ -686,6 +702,11 @@ var FileStorage = class {
           } else {
             summary.skippedIoError += 1;
           }
+          summary.failures.push({
+            path: result.path,
+            reason: result.reason,
+            message: describeReadFailure(result.reason, result.error)
+          });
           continue;
         }
         const trajectory = result.trajectory;
@@ -720,8 +741,74 @@ var FileStorage = class {
       }
       console.warn(`[trajectories] ${parts.join(", ")}`);
     }
+    this.lastReconcileSummary = summary;
     return summary;
   }
+  /**
+   * Returns the most recent reconcile summary, if any. Lets the CLI
+   * inspect the failures collected during `initialize()` without having
+   * to re-walk the directory tree (and re-emit the warn line).
+   */
+  getLastReconcileSummary() {
+    return this.lastReconcileSummary;
+  }
+  /**
+   * Move trajectory files that fail to load into `.trajectories/invalid/`
+   * so reconcile no longer scans them. Only quarantines parse and schema
+   * failures — transient io_error failures are left in place because the
+   * file may load fine on the next attempt.
+   *
+   * Returns the list of files that were moved (with their original paths
+   * and the destination directory) so the caller can report what changed.
+   */
+  async quarantineInvalid() {
+    const summary = await this.reconcileIndex();
+    const targetDir = join(this.trajectoriesDir, "invalid");
+    const candidates = summary.failures.filter((f) => f.reason !== "io_error");
+    if (candidates.length === 0) {
+      return { moved: [], targetDir };
+    }
+    await mkdir(targetDir, { recursive: true });
+    const moved = [];
+    for (const failure of candidates) {
+      const dest = await this.resolveQuarantineDest(failure.path, targetDir);
+      try {
+        await mkdir(dirname(dest), { recursive: true });
+        await rename(failure.path, dest);
+        moved.push(failure);
+      } catch (error) {
+        console.warn(
+          `[trajectories] failed to quarantine ${failure.path}: ${error instanceof Error ? error.message : String(error)}`
+        );
+      }
+    }
+    return { moved, targetDir };
+  }
+  /**
+   * Pick a destination path under `targetDir` for a quarantined file.
+   *
+   * Preserves the file's relative location under the trajectories root
+   * (e.g. `completed/2026-04/foo.json` → `invalid/completed/2026-04/foo.json`)
+   * so two invalid files that share a basename across `active/` and
+   * `completed/` don't collapse onto each other and silently overwrite.
+   *
+   * Falls back to a numeric-suffix scheme for paths that live outside
+   * the trajectories directory or that, after relative resolution, would
+   * still collide with something already quarantined.
+   */
+  async resolveQuarantineDest(sourcePath, targetDir) {
+    const rel = relative(this.trajectoriesDir, sourcePath);
+    const safeRel = rel && !rel.startsWith("..") && !isAbsolute(rel) ? rel : basename(sourcePath);
+    let dest = join(targetDir, safeRel);
+    if (!existsSync(dest)) return dest;
+    const ext = safeRel.endsWith(".json") ? ".json" : "";
+    const stem = ext ? safeRel.slice(0, -ext.length) : safeRel;
+    for (let i = 1; i < 1e3; i += 1) {
+      dest = join(targetDir, `${stem}.${i}${ext}`);
+      if (!existsSync(dest)) return dest;
+    }
+    return dest;
+  }
   /**
    * Recursively collect all .json file paths under `dir` into `out`.
    * Silently treats a missing directory as empty.
@@ -1103,7 +1190,7 @@ import {
   unlinkSync,
   writeFileSync
 } from "fs";
-import { dirname, join as join4 } from "path";
+import { dirname as dirname2, join as join4 } from "path";
 
 // src/compact/config.ts
 import { existsSync as existsSync2, readFileSync } from "fs";
@@ -2807,7 +2894,7 @@ function getDefaultOutputPath(compacted, workflowId) {
   return join4(compactedDir, `${compacted.id}_${dateStr}.json`);
 }
 function saveCompactionArtifacts(compacted, outputPath, markdownEnabled) {
-  const dir = dirname(outputPath);
+  const dir = dirname2(outputPath);
   if (!existsSync3(dir)) {
     mkdirSync(dir, { recursive: true });
   }
@@ -3393,6 +3480,51 @@ function registerDecisionCommand(program2) {
   });
 }
 
+// src/cli/commands/doctor.ts
+function registerDoctorCommand(program2) {
+  program2.command("doctor").description(
+    "List trajectory files that fail to load; optionally quarantine them"
+  ).option(
+    "--quarantine",
+    "Move invalid files to .trajectories/invalid/ so reconcile stops scanning them"
+  ).action(async (opts) => {
+    const storage = new FileStorage();
+    await storage.initialize();
+    const summary = storage.getLastReconcileSummary();
+    const failures = summary?.failures ?? [];
+    if (failures.length === 0) {
+      console.log("No invalid trajectory files found.");
+      return;
+    }
+    console.log(`Found ${failures.length} invalid trajectory file(s):`);
+    for (const failure of failures) {
+      console.log(`  ${failure.path}`);
+      console.log(`    reason:  ${failure.reason}`);
+      console.log(`    detail:  ${failure.message}`);
+    }
+    if (!opts.quarantine) {
+      console.log(
+        "\nRun `trail doctor --quarantine` to move these files into .trajectories/invalid/."
+      );
+      return;
+    }
+    const result = await storage.quarantineInvalid();
+    if (result.moved.length === 0) {
+      console.log(
+        "\nNo files were moved (io_error failures are not auto-quarantined)."
+      );
+      return;
+    }
+    console.log(
+      `
+Moved ${result.moved.length} file(s) to ${result.targetDir}:`
+    );
+    for (const failure of result.moved) {
+      console.log(`  ${failure.path}`);
+    }
+  });
+}
+
 // src/cli/commands/enable.ts
 import { execSync as execSync3 } from "child_process";
 import { existsSync as existsSync5 } from "fs";
@@ -4627,9 +4759,32 @@ function registerStartCommand(program2) {
 
 // src/cli/commands/status.ts
 function registerStatusCommand(program2) {
-  program2.command("status").description("Show active trajectory status").action(async () => {
+  program2.command("status").description("Show active trajectory status").option(
+    "-v, --verbose",
+    "Show paths and validation errors for any trajectory files that failed to load"
+  ).action(async (opts) => {
     const storage = new FileStorage();
     await storage.initialize();
+    if (opts.verbose) {
+      const summary = storage.getLastReconcileSummary();
+      const failures = summary?.failures ?? [];
+      if (failures.length === 0) {
+        console.log("All trajectory files loaded cleanly.");
+      } else {
+        console.log(
+          `Skipped ${failures.length} trajectory file(s) during reconcile:`
+        );
+        for (const failure of failures) {
+          console.log(`  ${failure.path}`);
+          console.log(`    reason:  ${failure.reason}`);
+          console.log(`    detail:  ${failure.message}`);
+        }
+        console.log(
+          "\nRun `trail doctor --quarantine` to move them aside and silence this warning."
+        );
+      }
+      console.log("");
+    }
     const active = await storage.getActive();
     if (!active) {
       console.log("No active trajectory");
@@ -4702,6 +4857,7 @@ function registerCommands(program2) {
   registerExportCommand(program2);
   registerEnableCommand(program2);
   registerCompactCommand(program2);
+  registerDoctorCommand(program2);
 }
 
 // src/cli/version.ts