agentplane 0.2.2 → 0.2.4

package/dist/commands/branch/status.js

@@ -1,5 +1,6 @@
 import { loadConfig, resolveBaseBranch, resolveProject } from "@agentplaneorg/core";
 import { mapCoreError } from "../../cli/error-map.js";
+import { exitCodeForError } from "../../cli/exit-codes.js";
 import { unknownEntityMessage } from "../../cli/output.js";
 import { CliError } from "../../shared/errors.js";
 import { gitAheadBehind } from "../shared/git-diff.js";
@@ -20,25 +21,29 @@ export async function cmdBranchStatus(opts) {
             mode: loaded.config.workflow_mode,
         });
         if (!branch) {
-            throw new CliError({ exitCode: 2, code: "E_USAGE", message: "Invalid value for --branch." });
+            throw new CliError({
+                exitCode: exitCodeForError("E_USAGE"),
+                code: "E_USAGE",
+                message: "Invalid value for --branch.",
+            });
         }
         if (!base) {
             throw new CliError({
-                exitCode: 2,
+                exitCode: exitCodeForError("E_USAGE"),
                 code: "E_USAGE",
                 message: "Base branch could not be resolved (use `agentplane branch base set` or --base).",
             });
         }
         if (!(await gitBranchExists(resolved.gitRoot, branch))) {
             throw new CliError({
-                exitCode: 2,
+                exitCode: exitCodeForError("E_USAGE"),
                 code: "E_USAGE",
                 message: unknownEntityMessage("branch", branch),
             });
         }
         if (!(await gitBranchExists(resolved.gitRoot, base))) {
             throw new CliError({
-                exitCode: 2,
+                exitCode: exitCodeForError("E_USAGE"),
                 code: "E_USAGE",
                 message: unknownEntityMessage("base branch", base),
             });

package/dist/commands/doctor.command.d.ts

@@ -0,0 +1,8 @@
+import type { CommandHandler, CommandSpec } from "../cli/spec/spec.js";
+type DoctorParsed = {
+    fix: boolean;
+};
+export declare const doctorSpec: CommandSpec<DoctorParsed>;
+export declare const runDoctor: CommandHandler<DoctorParsed>;
+export {};
+//# sourceMappingURL=doctor.command.d.ts.map

package/dist/commands/doctor.command.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"doctor.command.d.ts","sourceRoot":"","sources":["../../src/commands/doctor.command.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AAIvE,KAAK,YAAY,GAAG;IAClB,GAAG,EAAE,OAAO,CAAC;CACd,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,WAAW,CAAC,YAAY,CAWhD,CAAC;AAkHF,eAAO,MAAM,SAAS,EAAE,cAAc,CAAC,YAAY,CAoBlD,CAAC"}

package/dist/commands/doctor.command.js

@@ -0,0 +1,137 @@
+import fs from "node:fs/promises";
+import path from "node:path";
+import { resolveProject } from "@agentplaneorg/core";
+import { warnMessage, successMessage } from "../cli/output.js";
+import { loadCommandContext } from "./shared/task-backend.js";
+export const doctorSpec = {
+    id: ["doctor"],
+    group: "Quality",
+    summary: "Check structural invariants of an agentplane workspace (and optionally apply safe fixes).",
+    options: [{ kind: "boolean", name: "fix", default: false, description: "Apply safe fixes." }],
+    examples: [
+        { cmd: "agentplane doctor", why: "Check layering and workspace invariants." },
+        { cmd: "agentplane doctor --fix", why: "Apply safe-only fixes (idempotent)." },
+    ],
+    parse: (raw) => ({ fix: raw.opts.fix === true }),
+};
+async function listTsFiles(rootDir) {
+    const out = [];
+    async function walk(absDir) {
+        const entries = await fs.readdir(absDir, { withFileTypes: true });
+        for (const ent of entries) {
+            if (ent.name.startsWith("."))
+                continue;
+            if (ent.name === "__snapshots__")
+                continue;
+            if (ent.name === "node_modules")
+                continue;
+            const abs = path.join(absDir, ent.name);
+            if (ent.isDirectory()) {
+                await walk(abs);
+                continue;
+            }
+            if (ent.isFile() && ent.name.endsWith(".ts")) {
+                out.push({ absPath: abs, relPath: path.relative(rootDir, abs) });
+            }
+        }
+    }
+    await walk(rootDir);
+    return out;
+}
+function extractImports(source) {
+    const imports = [];
+    const re = /^\s*import\s+(?:type\s+)?(?:[^"']*?\s+from\s+)?["']([^"']+)["']\s*;?/gm;
+    for (const match of source.matchAll(re)) {
+        imports.push(match[1] ?? "");
+    }
+    return imports.filter(Boolean);
+}
+async function checkLayering(repoRoot) {
+    const problems = [];
+    const agentplaneSrcRoot = path.join(repoRoot, "packages", "agentplane", "src");
+    const cliRoot = path.join(agentplaneSrcRoot, "cli");
+    const cliFiles = await listTsFiles(cliRoot);
+    for (const f of cliFiles) {
+        const src = await fs.readFile(f.absPath, "utf8");
+        const imports = extractImports(src);
+        const hits = imports.filter((s) => s.includes("/adapters/") ||
+            s.includes("../adapters") ||
+            s.includes("../../adapters") ||
+            s.includes("../../../adapters"));
+        if (hits.length > 0) {
+            problems.push(`${f.relPath} imports adapters directly: ${hits.join(", ")}`);
+        }
+    }
+    const roots = [path.join(agentplaneSrcRoot, "usecases"), path.join(agentplaneSrcRoot, "ports")];
+    const banned = [
+        "node:fs",
+        "node:fs/promises",
+        "fs",
+        "fs/promises",
+        "node:path",
+        "path",
+        "simple-git",
+        "isomorphic-git",
+    ];
+    for (const root of roots) {
+        const files = await listTsFiles(root);
+        for (const f of files) {
+            const src = await fs.readFile(f.absPath, "utf8");
+            const imports = extractImports(src);
+            const hits = imports.filter((s) => banned.some((b) => s === b || s.startsWith(`${b}/`)));
+            if (hits.length > 0) {
+                problems.push(`${f.relPath} imports banned modules: ${hits.join(", ")}`);
+            }
+        }
+    }
+    return problems;
+}
+async function safeFixGitignore(repoRoot) {
+    const gitignorePath = path.join(repoRoot, ".gitignore");
+    let existing = "";
+    try {
+        existing = await fs.readFile(gitignorePath, "utf8");
+    }
+    catch {
+        // If .gitignore doesn't exist, do not create it implicitly (keep doctor safe).
+        return { changed: false, note: "Skip: .gitignore not found." };
+    }
+    const lines = existing.split(/\r?\n/);
+    const entry = ".agentplane/.upgrade/";
+    if (lines.some((l) => l.trim() === entry)) {
+        return { changed: false, note: "OK: .gitignore already ignores .agentplane/.upgrade/." };
+    }
+    const next = `${existing.trimEnd()}\n${entry}\n`;
+    await fs.writeFile(gitignorePath, next, "utf8");
+    return { changed: true, note: "Fixed: added .agentplane/.upgrade/ to .gitignore." };
+}
+async function safeFixTaskIndex(repoRoot) {
+    try {
+        // Best-effort: rebuilding the index is a side-effect of listing tasks for the local backend.
+        const ctx = await loadCommandContext({ cwd: repoRoot, rootOverride: null });
+        await ctx.taskBackend.listTasks();
+        return { changed: true, note: "OK: rebuilt tasks index cache (best-effort)." };
+    }
+    catch {
+        return { changed: false, note: "Skip: could not rebuild tasks index cache." };
+    }
+}
+export const runDoctor = async (ctx, p) => {
+    const resolved = await resolveProject({ cwd: ctx.cwd, rootOverride: ctx.rootOverride ?? null });
+    const repoRoot = resolved.gitRoot;
+    const problems = await checkLayering(repoRoot);
+    if (problems.length > 0) {
+        console.error(warnMessage(`doctor found ${problems.length} problem(s):`));
+        for (const prob of problems)
+            console.error(`- ${prob}`);
+        return 1;
+    }
+    if (p.fix) {
+        const fix = await safeFixGitignore(repoRoot);
+        console.log(successMessage("doctor fix", undefined, fix.note));
+        const idx = await safeFixTaskIndex(repoRoot);
+        console.log(successMessage("doctor fix", undefined, idx.note));
+    }
+    console.log(successMessage("doctor", undefined, "OK"));
+    return 0;
+};

package/dist/commands/guard/impl/allow.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"allow.d.ts","sourceRoot":"","sources":["../../../../src/commands/guard/impl/allow.ts"],"names":[],"mappings":"AAKA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAEvF,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,MAAM,EAAE,CAS9D;AAED,wBAAsB,qBAAqB,CAAC,IAAI,EAAE;IAChD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAOpB;AAED,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB,GAAG,OAAO,CAAC,IAAI,CAAC,CAiBhB;AAED,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,GAAG,EAAE,cAAc,CAAC;IACpB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,UAAU,EAAE,OAAO,CAAC;IACpB,SAAS,EAAE,MAAM,CAAC;CACnB,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAoCpB"}
+{"version":3,"file":"allow.d.ts","sourceRoot":"","sources":["../../../../src/commands/guard/impl/allow.ts"],"names":[],"mappings":"AAMA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAEvF,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,MAAM,EAAE,CAS9D;AAED,wBAAsB,qBAAqB,CAAC,IAAI,EAAE;IAChD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAOpB;AAED,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB,GAAG,OAAO,CAAC,IAAI,CAAC,CAqBhB;AAED,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,GAAG,EAAE,cAAc,CAAC;IACpB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,UAAU,EAAE,OAAO,CAAC;IACpB,SAAS,EAAE,MAAM,CAAC;CACnB,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAoCpB"}

package/dist/commands/guard/impl/allow.js

@@ -1,4 +1,5 @@
 import { resolveProject } from "@agentplaneorg/core";
+import { exitCodeForError } from "../../../cli/exit-codes.js";
 import { gitPathIsUnderPrefix, normalizeGitPathPrefix } from "../../../shared/git-path.js";
 import { CliError } from "../../../shared/errors.js";
 import { GitContext } from "../../shared/git-context.js";
@@ -29,13 +30,17 @@ export async function ensureGitClean(opts) {
         (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
     const staged = await ctx.git.statusStagedPaths();
     if (staged.length > 0) {
-        throw new CliError({ exitCode: 5, code: "E_GIT", message: "Working tree has staged changes" });
+        throw new CliError({
+            exitCode: exitCodeForError("E_GIT"),
+            code: "E_GIT",
+            message: "Working tree has staged changes",
+        });
     }
     // Policy-defined "clean" ignores untracked files.
     const unstaged = await ctx.git.statusUnstagedTrackedPaths();
     if (unstaged.length > 0) {
         throw new CliError({
-            exitCode: 5,
+            exitCode: exitCodeForError("E_GIT"),
             code: "E_GIT",
             message: "Working tree has unstaged changes",
         });

package/dist/commands/guard/impl/close-message.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"close-message.d.ts","sourceRoot":"","sources":["../../../../src/commands/guard/impl/close-message.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AAqHlE,MAAM,MAAM,kBAAkB,GAAG;IAC/B,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,CAAC;CACd,CAAC;AAEF,wBAAsB,uBAAuB,CAAC,IAAI,EAAE;IAClD,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,QAAQ,CAAC;IACf,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB,GAAG,OAAO,CAAC,kBAAkB,CAAC,CAkD9B;AAED,wBAAgB,qBAAqB,CAAC,IAAI,EAAE;IAC1C,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,MAAM,CAET"}
+{"version":3,"file":"close-message.d.ts","sourceRoot":"","sources":["../../../../src/commands/guard/impl/close-message.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AAsHlE,MAAM,MAAM,kBAAkB,GAAG;IAC/B,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,CAAC;CACd,CAAC;AAEF,wBAAsB,uBAAuB,CAAC,IAAI,EAAE;IAClD,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,QAAQ,CAAC;IACf,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB,GAAG,OAAO,CAAC,kBAAkB,CAAC,CAsD9B;AAED,wBAAgB,qBAAqB,CAAC,IAAI,EAAE;IAC1C,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,MAAM,CAET"}

package/dist/commands/guard/impl/close-message.js

@@ -1,5 +1,6 @@
 import path from "node:path";
 import { extractTaskSuffix } from "@agentplaneorg/core";
+import { exitCodeForError } from "../../../cli/exit-codes.js";
 import { CliError } from "../../../shared/errors.js";
 import { execFileAsync, gitEnv } from "../../shared/git.js";
 function uniqSorted(values) {
@@ -114,12 +115,16 @@ export async function buildCloseCommitMessage(opts) {
         : [];
     const isSpike = tags.includes("spike");
     if (String(task.status || "").toUpperCase() !== "DONE") {
-        throw new CliError({ exitCode: 2, code: "E_USAGE", message: `Task is not DONE: ${task.id}` });
+        throw new CliError({
+            exitCode: exitCodeForError("E_USAGE"),
+            code: "E_USAGE",
+            message: `Task is not DONE: ${task.id}`,
+        });
     }
     const implCommit = task.commit?.hash?.trim() ?? "";
     if (!implCommit) {
         throw new CliError({
-            exitCode: 2,
+            exitCode: exitCodeForError("E_USAGE"),
             code: "E_USAGE",
             message: `Task is missing recorded commit metadata: ${task.id} (finish with --commit or set commit on the task).`,
         });

package/dist/commands/pr/check.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"check.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/check.ts"],"names":[],"mappings":"AAQA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AAKnC,wBAAsB,UAAU,CAAC,IAAI,EAAE;IACrC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAoElB"}
+{"version":3,"file":"check.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/check.ts"],"names":[],"mappings":"AASA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AAKnC,wBAAsB,UAAU,CAAC,IAAI,EAAE;IACrC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAwElB"}

package/dist/commands/pr/check.js

@@ -1,6 +1,7 @@
 import { readFile } from "node:fs/promises";
 import path from "node:path";
 import { mapBackendError } from "../../cli/error-map.js";
+import { exitCodeForError } from "../../cli/exit-codes.js";
 import { fileExists } from "../../cli/fs-utils.js";
 import { successMessage, workflowModeMessage } from "../../cli/output.js";
 import { CliError } from "../../shared/errors.js";
@@ -21,7 +22,7 @@ export async function cmdPrCheck(opts) {
         const { resolved, config, prDir, metaPath, diffstatPath, verifyLogPath, reviewPath } = await resolvePrPaths({ ...opts, ctx });
         if (config.workflow_mode !== "branch_pr") {
             throw new CliError({
-                exitCode: 2,
+                exitCode: exitCodeForError("E_USAGE"),
                 code: "E_USAGE",
                 message: workflowModeMessage(config.workflow_mode, "branch_pr"),
             });
@@ -65,7 +66,11 @@ export async function cmdPrCheck(opts) {
             }
         }
         if (errors.length > 0) {
-            throw new CliError({ exitCode: 3, code: "E_VALIDATION", message: errors.join("\n") });
+            throw new CliError({
+                exitCode: exitCodeForError("E_VALIDATION"),
+                code: "E_VALIDATION",
+                message: errors.join("\n"),
+            });
         }
         process.stdout.write(`${successMessage("pr check", path.relative(resolved.gitRoot, prDir))}\n`);
         return 0;

package/dist/commands/pr/integrate/artifacts.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"artifacts.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/integrate/artifacts.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAWnE,wBAAsB,0BAA0B,CAAC,IAAI,EAAE;IACrD,GAAG,EAAE,cAAc,CAAC;IACpB,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IAC9B,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC;IACV,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,CAAC,CA0CD"}
+{"version":3,"file":"artifacts.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/integrate/artifacts.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAWnE,wBAAsB,0BAA0B,CAAC,IAAI,EAAE;IACrD,GAAG,EAAE,cAAc,CAAC;IACpB,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IAC9B,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC;IACV,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,CAAC,CA8CD"}

package/dist/commands/pr/integrate/artifacts.js

@@ -1,4 +1,5 @@
 import path from "node:path";
+import { exitCodeForError } from "../../../cli/exit-codes.js";
 import { CliError } from "../../../shared/errors.js";
 import { readPrArtifact } from "../internal/pr-paths.js";
 import { validateReviewContents } from "../internal/review-template.js";
@@ -39,7 +40,11 @@ export async function readAndValidatePrArtifacts(opts) {
     if (reviewText)
         validateReviewContents(reviewText, errors);
     if (errors.length > 0) {
-        throw new CliError({ exitCode: 3, code: "E_VALIDATION", message: errors.join("\n") });
+        throw new CliError({
+            exitCode: exitCodeForError("E_VALIDATION"),
+            code: "E_VALIDATION",
+            message: errors.join("\n"),
+        });
     }
     return { verifyLogText };
 }

package/dist/commands/pr/integrate/internal/merge.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"merge.d.ts","sourceRoot":"","sources":["../../../../../src/commands/pr/integrate/internal/merge.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,4BAA4B,CAAC;AAIzD,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,eAAe,EAAE,MAAM,CAAC;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,MAAM,EAAE,CAAC;CACzB,GAAG,OAAO,CAAC,MAAM,CAAC,CAuElB;AAED,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAoBlB;AAED,wBAAsB,oBAAoB,CAAC,IAAI,EAAE;IAC/C,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,CAAC;IACrB,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,eAAe,EAAE,MAAM,CAAC;IACxB,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,OAAO,CAAC;IACnB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,kBAAkB,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,eAAe,EAAE,OAAO,CAAC;IACzB,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC;IACV,SAAS,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;IACtB,aAAa,EAAE;QAAE,MAAM,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IACrD,kBAAkB,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,eAAe,EAAE,OAAO,CAAC;CAC1B,CAAC,CAsDD"}
+{"version":3,"file":"merge.d.ts","sourceRoot":"","sources":["../../../../../src/commands/pr/integrate/internal/merge.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,4BAA4B,CAAC;AAIzD,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,eAAe,EAAE,MAAM,CAAC;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,MAAM,EAAE,CAAC;CACzB,GAAG,OAAO,CAAC,MAAM,CAAC,CAuElB;AAED,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAoBlB;AAED,wBAAsB,oBAAoB,CAAC,IAAI,EAAE;IAC/C,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,CAAC;IACrB,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,eAAe,EAAE,MAAM,CAAC;IACxB,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,OAAO,CAAC;IACnB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,kBAAkB,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,eAAe,EAAE,OAAO,CAAC;IACzB,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC;IACV,SAAS,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;IACtB,aAAa,EAAE;QAAE,MAAM,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IACrD,kBAAkB,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,eAAe,EAAE,OAAO,CAAC;CAC1B,CAAC,CAsDD"}

package/dist/commands/pr/integrate/internal/merge.js

@@ -1,4 +1,5 @@
 import { extractTaskSuffix, validateCommitSubject } from "@agentplaneorg/core";
+import { exitCodeForError } from "../../../../cli/exit-codes.js";
 import { CliError } from "../../../../shared/errors.js";
 import { execFileAsync, gitEnv } from "../../../shared/git.js";
 import { gitRevParse } from "../../../shared/git-ops.js";
@@ -16,7 +17,7 @@ export async function runSquashMerge(opts) {
             env: gitEnv(),
         });
         const message = err instanceof Error ? err.message : "git merge --squash failed";
-        throw new CliError({ exitCode: 2, code: "E_GIT", message });
+        throw new CliError({ exitCode: exitCodeForError("E_GIT"), code: "E_GIT", message });
     }
     const { stdout: staged } = await execFileAsync("git", ["diff", "--cached", "--name-only"], {
         cwd: opts.gitRoot,
@@ -28,7 +29,7 @@ export async function runSquashMerge(opts) {
             env: gitEnv(),
         });
         throw new CliError({
-            exitCode: 2,
+            exitCode: exitCodeForError("E_USAGE"),
             code: "E_USAGE",
             message: `Nothing to integrate: ${opts.branch} is already merged into ${opts.base}`,
         });
@@ -65,7 +66,7 @@ export async function runSquashMerge(opts) {
             env: gitEnv(),
         });
         const message = err instanceof Error ? err.message : "git commit failed";
-        throw new CliError({ exitCode: 2, code: "E_GIT", message });
+        throw new CliError({ exitCode: exitCodeForError("E_GIT"), code: "E_GIT", message });
     }
     return await gitRevParse(opts.gitRoot, ["HEAD"]);
 }
@@ -83,7 +84,7 @@ export async function runMergeCommit(opts) {
     catch (err) {
         await execFileAsync("git", ["merge", "--abort"], { cwd: opts.gitRoot, env: gitEnv() });
         const message = err instanceof Error ? err.message : "git merge failed";
-        throw new CliError({ exitCode: 2, code: "E_GIT", message });
+        throw new CliError({ exitCode: exitCodeForError("E_GIT"), code: "E_GIT", message });
     }
     return await gitRevParse(opts.gitRoot, ["HEAD"]);
 }
@@ -94,7 +95,7 @@ export async function runRebaseFastForward(opts) {
     catch (err) {
         await execFileAsync("git", ["rebase", "--abort"], { cwd: opts.worktreePath, env: gitEnv() });
         const message = err instanceof Error ? err.message : "git rebase failed";
-        throw new CliError({ exitCode: 2, code: "E_GIT", message });
+        throw new CliError({ exitCode: exitCodeForError("E_GIT"), code: "E_GIT", message });
     }
     let branchHeadSha = await gitRevParse(opts.gitRoot, [opts.branch]);
     let alreadyVerifiedSha = opts.alreadyVerifiedSha;
@@ -130,7 +131,7 @@ export async function runRebaseFastForward(opts) {
             env: gitEnv(),
         }).catch(() => null);
         const message = err instanceof Error ? err.message : "git merge --ff-only failed";
-        throw new CliError({ exitCode: 2, code: "E_GIT", message });
+        throw new CliError({ exitCode: exitCodeForError("E_GIT"), code: "E_GIT", message });
     }
     const mergeHash = await gitRevParse(opts.gitRoot, ["HEAD"]);
     branchHeadSha = await gitRevParse(opts.gitRoot, [opts.branch]);

package/dist/commands/pr/integrate/internal/prepare.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"prepare.d.ts","sourceRoot":"","sources":["../../../../../src/commands/pr/integrate/internal/prepare.ts"],"names":[],"mappings":"AAWA,OAAO,EACL,eAAe,EAEf,KAAK,cAAc,EACpB,MAAM,iCAAiC,CAAC;AAUzC,OAAO,EAAe,KAAK,MAAM,EAAE,MAAM,4BAA4B,CAAC;AAEtE,MAAM,MAAM,iBAAiB,GAAG;IAC9B,GAAG,EAAE,cAAc,CAAC;IACpB,QAAQ,EAAE,cAAc,CAAC,iBAAiB,CAAC,CAAC;IAC5C,YAAY,EAAE,cAAc,CAAC,QAAQ,CAAC,CAAC;IACvC,IAAI,EAAE,OAAO,CAAC,UAAU,CAAC,OAAO,eAAe,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IAE1D,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,EAAE,MAAM,CAAC;IAEtB,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,MAAM,CAAC;IACrB,aAAa,EAAE,MAAM,CAAC;IAEtB,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IAEb,aAAa,EAAE,MAAM,CAAC;IAEtB,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,kBAAkB,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,eAAe,EAAE,OAAO,CAAC;CAC1B,CAAC;AAEF,wBAAsB,gBAAgB,CAAC,IAAI,EAAE;IAC3C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,SAAS,EAAE,OAAO,CAAC;CACpB,GAAG,OAAO,CAAC,iBAAiB,CAAC,CAoJ7B"}
+{"version":3,"file":"prepare.d.ts","sourceRoot":"","sources":["../../../../../src/commands/pr/integrate/internal/prepare.ts"],"names":[],"mappings":"AAYA,OAAO,EACL,eAAe,EAEf,KAAK,cAAc,EACpB,MAAM,iCAAiC,CAAC;AAUzC,OAAO,EAAe,KAAK,MAAM,EAAE,MAAM,4BAA4B,CAAC;AAEtE,MAAM,MAAM,iBAAiB,GAAG;IAC9B,GAAG,EAAE,cAAc,CAAC;IACpB,QAAQ,EAAE,cAAc,CAAC,iBAAiB,CAAC,CAAC;IAC5C,YAAY,EAAE,cAAc,CAAC,QAAQ,CAAC,CAAC;IACvC,IAAI,EAAE,OAAO,CAAC,UAAU,CAAC,OAAO,eAAe,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IAE1D,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,EAAE,MAAM,CAAC;IAEtB,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,MAAM,CAAC;IACrB,aAAa,EAAE,MAAM,CAAC;IAEtB,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IAEb,aAAa,EAAE,MAAM,CAAC;IAEtB,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,kBAAkB,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,eAAe,EAAE,OAAO,CAAC;CAC1B,CAAC;AAEF,wBAAsB,gBAAgB,CAAC,IAAI,EAAE;IAC3C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,SAAS,EAAE,OAAO,CAAC;CACpB,GAAG,OAAO,CAAC,iBAAiB,CAAC,CAwJ7B"}

package/dist/commands/pr/integrate/internal/prepare.js

@@ -2,6 +2,7 @@ import path from "node:path";
 import { readFile } from "node:fs/promises";
 import { resolveBaseBranch } from "@agentplaneorg/core";
 import { fileExists } from "../../../../cli/fs-utils.js";
+import { exitCodeForError } from "../../../../cli/exit-codes.js";
 import { unknownEntityMessage, workflowModeMessage } from "../../../../cli/output.js";
 import { CliError } from "../../../../shared/errors.js";
 import { ensureGitClean } from "../../../guard/index.js";
@@ -26,7 +27,7 @@ export async function prepareIntegrate(opts) {
     const loadedConfig = ctx.config;
     if (loadedConfig.workflow_mode !== "branch_pr") {
         throw new CliError({
-            exitCode: 2,
+            exitCode: exitCodeForError("E_USAGE"),
             code: "E_USAGE",
             message: workflowModeMessage(loadedConfig.workflow_mode, "branch_pr"),
         });
@@ -35,7 +36,11 @@ export async function prepareIntegrate(opts) {
     ensureVerificationSatisfiedIfRequired(task, loadedConfig);
     await ensureGitClean({ cwd: opts.cwd, rootOverride: opts.rootOverride });
     if (opts.base?.trim().length === 0) {
-        throw new CliError({ exitCode: 2, code: "E_USAGE", message: "Invalid value for --base." });
+        throw new CliError({
+            exitCode: exitCodeForError("E_USAGE"),
+            code: "E_USAGE",
+            message: "Invalid value for --base.",
+        });
     }
     const baseBranch = await resolveBaseBranch({
         cwd: opts.cwd,
@@ -45,7 +50,7 @@ export async function prepareIntegrate(opts) {
     });
     if (!baseBranch) {
         throw new CliError({
-            exitCode: 2,
+            exitCode: exitCodeForError("E_USAGE"),
             code: "E_USAGE",
             message: "Base branch could not be resolved (use `agentplane branch base set` or --base).",
         });
@@ -53,7 +58,7 @@ export async function prepareIntegrate(opts) {
     const currentBranch = await gitCurrentBranch(resolved.gitRoot);
     if (currentBranch !== baseBranch) {
         throw new CliError({
-            exitCode: 5,
+            exitCode: exitCodeForError("E_GIT"),
             code: "E_GIT",
             message: `integrate must run on base branch ${baseBranch} (current: ${currentBranch})`,
         });

package/dist/commands/pr/integrate/verify.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"verify.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/integrate/verify.ts"],"names":[],"mappings":"AAaA,wBAAgB,kBAAkB,CAAC,IAAI,EAAE;IACvC,SAAS,EAAE,OAAO,CAAC;IACnB,mBAAmB,EAAE,OAAO,CAAC;IAC7B,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,OAAO,CAAC;CACpB,GAAG;IACF,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,kBAAkB,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,eAAe,EAAE,OAAO,CAAC;CAC1B,CAgBA;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,YAAY,EAAE,MAAM,CAAC;IACrB,aAAa,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,EAAE,CAAC,CA+BjD"}
+{"version":3,"file":"verify.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/integrate/verify.ts"],"names":[],"mappings":"AAcA,wBAAgB,kBAAkB,CAAC,IAAI,EAAE;IACvC,SAAS,EAAE,OAAO,CAAC;IACnB,mBAAmB,EAAE,OAAO,CAAC;IAC7B,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,OAAO,CAAC;CACpB,GAAG;IACF,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,kBAAkB,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,eAAe,EAAE,OAAO,CAAC;CAC1B,CAgBA;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,YAAY,EAAE,MAAM,CAAC;IACrB,aAAa,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,EAAE,CAAC,CAgCjD"}

package/dist/commands/pr/integrate/verify.js

@@ -1,5 +1,6 @@
 import { extractLastVerifiedSha, runShellCommand } from "../../shared/pr-meta.js";
 import { successMessage } from "../../../cli/output.js";
+import { exitCodeForError } from "../../../cli/exit-codes.js";
 import { CliError } from "../../../shared/errors.js";
 function normalizeVerifyCommands(rawVerify) {
     return Array.isArray(rawVerify)
@@ -41,9 +42,10 @@ export async function runVerifyCommands(opts) {
         });
         if (result.code !== 0) {
             throw new CliError({
-                exitCode: result.code || 1,
+                exitCode: exitCodeForError("E_IO"),
                 code: "E_IO",
                 message: `Verify command failed: ${command}`,
+                context: { commandExitCode: result.code || 1 },
             });
         }
     }

package/dist/commands/pr/note.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"note.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/note.ts"],"names":[],"mappings":"AASA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAKpF,wBAAsB,SAAS,CAAC,IAAI,EAAE;IACpC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;CACd,GAAG,OAAO,CAAC,MAAM,CAAC,CA0ClB"}
+{"version":3,"file":"note.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/note.ts"],"names":[],"mappings":"AAUA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAKpF,wBAAsB,SAAS,CAAC,IAAI,EAAE;IACpC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;CACd,GAAG,OAAO,CAAC,MAAM,CAAC,CAkDlB"}

package/dist/commands/pr/note.js

@@ -2,6 +2,7 @@ import { readFile } from "node:fs/promises";
 import path from "node:path";
 import { atomicWriteFile } from "@agentplaneorg/core";
 import { mapCoreError } from "../../cli/error-map.js";
+import { exitCodeForError } from "../../cli/exit-codes.js";
 import { fileExists } from "../../cli/fs-utils.js";
 import { successMessage, workflowModeMessage } from "../../cli/output.js";
 import { CliError } from "../../shared/errors.js";
@@ -13,17 +14,25 @@ export async function cmdPrNote(opts) {
         const author = opts.author.trim();
         const body = opts.body.trim();
         if (!author) {
-            throw new CliError({ exitCode: 2, code: "E_USAGE", message: "Invalid value for --author." });
+            throw new CliError({
+                exitCode: exitCodeForError("E_USAGE"),
+                code: "E_USAGE",
+                message: "Invalid value for --author.",
+            });
         }
         if (!body) {
-            throw new CliError({ exitCode: 2, code: "E_USAGE", message: "Invalid value for --body." });
+            throw new CliError({
+                exitCode: exitCodeForError("E_USAGE"),
+                code: "E_USAGE",
+                message: "Invalid value for --body.",
+            });
         }
         const ctx = opts.ctx ??
             (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
         const { config, reviewPath, resolved } = await resolvePrPaths({ ...opts, ctx });
         if (config.workflow_mode !== "branch_pr") {
             throw new CliError({
-                exitCode: 2,
+                exitCode: exitCodeForError("E_USAGE"),
                 code: "E_USAGE",
                 message: workflowModeMessage(config.workflow_mode, "branch_pr"),
             });
@@ -31,7 +40,7 @@ export async function cmdPrNote(opts) {
         if (!(await fileExists(reviewPath))) {
             const relReviewPath = path.relative(resolved.gitRoot, reviewPath);
             throw new CliError({
-                exitCode: 3,
+                exitCode: exitCodeForError("E_VALIDATION"),
                 code: "E_VALIDATION",
                 message: `Missing ${relReviewPath} (run \`agentplane pr open\`)`,
             });

package/dist/commands/pr/open.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"open.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/open.ts"],"names":[],"mappings":"AAYA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AASnC,wBAAsB,SAAS,CAAC,IAAI,EAAE;IACpC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,GAAG,OAAO,CAAC,MAAM,CAAC,CAsElB"}
+{"version":3,"file":"open.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/open.ts"],"names":[],"mappings":"AAaA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AASnC,wBAAsB,SAAS,CAAC,IAAI,EAAE;IACpC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,GAAG,OAAO,CAAC,MAAM,CAAC,CA0ElB"}

package/dist/commands/pr/open.js

@@ -2,6 +2,7 @@ import { mkdir, readFile } from "node:fs/promises";
 import path from "node:path";
 import { atomicWriteFile } from "@agentplaneorg/core";
 import { mapBackendError } from "../../cli/error-map.js";
+import { exitCodeForError } from "../../cli/exit-codes.js";
 import { fileExists } from "../../cli/fs-utils.js";
 import { successMessage, workflowModeMessage } from "../../cli/output.js";
 import { CliError } from "../../shared/errors.js";
@@ -18,7 +19,11 @@ export async function cmdPrOpen(opts) {
     try {
         const author = opts.author.trim();
         if (!author) {
-            throw new CliError({ exitCode: 2, code: "E_USAGE", message: "Invalid value for --author." });
+            throw new CliError({
+                exitCode: exitCodeForError("E_USAGE"),
+                code: "E_USAGE",
+                message: "Invalid value for --author.",
+            });
         }
         const ctx = opts.ctx ??
             (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
@@ -31,7 +36,7 @@ export async function cmdPrOpen(opts) {
         const { resolved, config, prDir, metaPath, diffstatPath, verifyLogPath, reviewPath } = await resolvePrPaths({ ...opts, ctx });
         if (config.workflow_mode !== "branch_pr") {
             throw new CliError({
-                exitCode: 2,
+                exitCode: exitCodeForError("E_USAGE"),
                 code: "E_USAGE",
                 message: workflowModeMessage(config.workflow_mode, "branch_pr"),
             });
@@ -39,7 +44,7 @@ export async function cmdPrOpen(opts) {
         const branch = (opts.branch ?? (await gitCurrentBranch(resolved.gitRoot))).trim();
         if (!branch) {
             throw new CliError({
-                exitCode: 2,
+                exitCode: exitCodeForError("E_USAGE"),
                 code: "E_USAGE",
                 message: "Branch could not be resolved (use --branch).",
             });

package/dist/commands/recipes/impl/apply.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"apply.d.ts","sourceRoot":"","sources":["../../../../src/commands/recipes/impl/apply.ts"],"names":[],"mappings":"AAYA,OAAO,KAAK,EAAE,kBAAkB,EAAE,cAAc,EAAE,MAAM,YAAY,CAAC;AAErE,wBAAsB,aAAa,CAAC,IAAI,EAAE;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,EAAE,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAarF;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,QAAQ,EAAE,cAAc,CAAC;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;IACtB,UAAU,EAAE,kBAAkB,CAAC;CAChC,GAAG,OAAO,CAAC,IAAI,CAAC,CAgDhB;AAED,wBAAsB,oBAAoB,CAAC,IAAI,EAAE;IAC/C,QAAQ,EAAE,cAAc,CAAC;IACzB,SAAS,EAAE,MAAM,CAAC;CACnB,GAAG,OAAO,CAAC,IAAI,CAAC,CA0BhB"}
+{"version":3,"file":"apply.d.ts","sourceRoot":"","sources":["../../../../src/commands/recipes/impl/apply.ts"],"names":[],"mappings":"AAaA,OAAO,KAAK,EAAE,kBAAkB,EAAE,cAAc,EAAE,MAAM,YAAY,CAAC;AAErE,wBAAsB,aAAa,CAAC,IAAI,EAAE;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,EAAE,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAarF;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,QAAQ,EAAE,cAAc,CAAC;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;IACtB,UAAU,EAAE,kBAAkB,CAAC;CAChC,GAAG,OAAO,CAAC,IAAI,CAAC,CAgDhB;AAED,wBAAsB,oBAAoB,CAAC,IAAI,EAAE;IAC/C,QAAQ,EAAE,cAAc,CAAC;IACzB,SAAS,EAAE,MAAM,CAAC;CACnB,GAAG,OAAO,CAAC,IAAI,CAAC,CA0BhB"}

package/dist/commands/recipes/impl/apply.js

@@ -1,5 +1,6 @@
 import { cp, mkdir, readFile, readdir, rename, rm } from "node:fs/promises";
 import path from "node:path";
+import { exitCodeForError } from "../../../cli/exit-codes.js";
 import { fileExists, getPathKind } from "../../../cli/fs-utils.js";
 import { invalidFieldMessage, missingFileMessage } from "../../../cli/output.js";
 import { CliError } from "../../../shared/errors.js";
@@ -50,7 +51,7 @@ export async function applyRecipeAgents(opts) {
         if (await getPathKind(targetPath)) {
             if (opts.onConflict === "fail") {
                 throw new CliError({
-                    exitCode: 5,
+                    exitCode: exitCodeForError("E_IO"),
                     code: "E_IO",
                     message: `Agent already exists: ${targetId}`,
                 });

package/dist/commands/recipes/impl/commands/explain.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"explain.d.ts","sourceRoot":"","sources":["../../../../../src/commands/recipes/impl/commands/explain.ts"],"names":[],"mappings":"AAQA,wBAAsB,sBAAsB,CAAC,IAAI,EAAE;IACjD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,EAAE,EAAE,MAAM,CAAC;CACZ,GAAG,OAAO,CAAC,MAAM,CAAC,CAmFlB"}
+{"version":3,"file":"explain.d.ts","sourceRoot":"","sources":["../../../../../src/commands/recipes/impl/commands/explain.ts"],"names":[],"mappings":"AASA,wBAAsB,sBAAsB,CAAC,IAAI,EAAE;IACjD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,EAAE,EAAE,MAAM,CAAC;CACZ,GAAG,OAAO,CAAC,MAAM,CAAC,CAmFlB"}

package/dist/commands/recipes/impl/commands/explain.js

@@ -1,4 +1,5 @@
 import { mapCoreError } from "../../../../cli/error-map.js";
+import { exitCodeForError } from "../../../../cli/exit-codes.js";
 import { CliError } from "../../../../shared/errors.js";
 import { collectRecipeScenarioDetails } from "../scenario.js";
 import { formatJsonBlock } from "../format.js";
@@ -10,7 +11,7 @@ export async function cmdRecipeExplainParsed(opts) {
         const entry = installed.recipes.find((recipe) => recipe.id === opts.id);
         if (!entry) {
             throw new CliError({
-                exitCode: 5,
+                exitCode: exitCodeForError("E_IO"),
                 code: "E_IO",
                 message: `Recipe not installed: ${opts.id}`,
             });

package/dist/commands/recipes/impl/commands/info.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"info.d.ts","sourceRoot":"","sources":["../../../../../src/commands/recipes/impl/commands/info.ts"],"names":[],"mappings":"AAMA,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,EAAE,EAAE,MAAM,CAAC;CACZ,GAAG,OAAO,CAAC,MAAM,CAAC,CAsDlB"}
+{"version":3,"file":"info.d.ts","sourceRoot":"","sources":["../../../../../src/commands/recipes/impl/commands/info.ts"],"names":[],"mappings":"AAOA,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,EAAE,EAAE,MAAM,CAAC;CACZ,GAAG,OAAO,CAAC,MAAM,CAAC,CAsDlB"}

package/dist/commands/recipes/impl/commands/info.js

@@ -1,4 +1,5 @@
 import { mapCoreError } from "../../../../cli/error-map.js";
+import { exitCodeForError } from "../../../../cli/exit-codes.js";
 import { CliError } from "../../../../shared/errors.js";
 import { readInstalledRecipesFile } from "../installed-recipes.js";
 import { resolveInstalledRecipesPath } from "../paths.js";
@@ -8,7 +9,7 @@ export async function cmdRecipeInfoParsed(opts) {
         const entry = installed.recipes.find((recipe) => recipe.id === opts.id);
         if (!entry) {
             throw new CliError({
-                exitCode: 5,
+                exitCode: exitCodeForError("E_IO"),
                 code: "E_IO",
                 message: `Recipe not installed: ${opts.id}`,
             });