agentplane 0.3.9 → 0.3.11

package/dist/commands/pr/open.js

@@ -1,19 +1,18 @@
-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 { createCliEmitter, workflowModeMessage } from "../../cli/output.js";
+import { createCliEmitter } from "../../cli/output.js";
 import { CliError } from "../../shared/errors.js";
-import { writeJsonStableIfChanged, writeTextIfChanged } from "../../shared/write-if-changed.js";
-import { gitCurrentBranch } from "../shared/git-ops.js";
-import { buildOpenedPrMeta, parsePrMeta } from "../shared/pr-meta.js";
-import { loadBackendTask, loadCommandContext, } from "../shared/task-backend.js";
-import { resolvePrPaths } from "./internal/pr-paths.js";
-import { renderPrReviewTemplate } from "./internal/review-template.js";
-function nowIso() {
-    return new Date().toISOString();
+import { syncPrArtifacts } from "./internal/sync.js";
+function prOpenOutcomeDetails(meta, openOutcome) {
+    if (openOutcome)
+        return openOutcome.message;
+    if (typeof meta.pr_number === "number" && meta.pr_number > 0) {
+        return meta.pr_url?.trim()
+            ? `linked to GitHub PR #${meta.pr_number}: ${meta.pr_url.trim()}`
+            : `linked to GitHub PR #${meta.pr_number}`;
+    }
+    return "local PR artifacts synced; remote PR creation staged";
 }
 export async function cmdPrOpen(opts) {
     try {
@@ -26,52 +25,17 @@ export async function cmdPrOpen(opts) {
                 message: "Invalid value for --author.",
             });
         }
-        const ctx = opts.ctx ??
-            (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
-        const { task } = await loadBackendTask({
-            ctx,
+        const { meta, prDir, resolved, openOutcome } = await syncPrArtifacts({
+            ctx: opts.ctx,
             cwd: opts.cwd,
             rootOverride: opts.rootOverride,
             taskId: opts.taskId,
+            mode: "open",
+            author,
+            branch: opts.branch,
+            remoteMode: opts.syncOnly ? "sync-only" : "auto",
         });
-        const { resolved, config, prDir, metaPath, diffstatPath, verifyLogPath, reviewPath } = await resolvePrPaths({ ...opts, ctx });
-        if (config.workflow_mode !== "branch_pr") {
-            throw new CliError({
-                exitCode: exitCodeForError("E_USAGE"),
-                code: "E_USAGE",
-                message: workflowModeMessage(config.workflow_mode, "branch_pr"),
-            });
-        }
-        const branch = (opts.branch ?? (await gitCurrentBranch(resolved.gitRoot))).trim();
-        if (!branch) {
-            throw new CliError({
-                exitCode: exitCodeForError("E_USAGE"),
-                code: "E_USAGE",
-                message: "Branch could not be resolved (use --branch).",
-            });
-        }
-        await mkdir(prDir, { recursive: true });
-        const now = nowIso();
-        let meta = null;
-        if (await fileExists(metaPath)) {
-            const raw = await readFile(metaPath, "utf8");
-            meta = parsePrMeta(raw, task.id);
-        }
-        const createdAt = meta?.created_at ?? now;
-        const nextMeta = buildOpenedPrMeta({
-            taskId: task.id,
-            branch,
-            at: now,
-            previousMeta: meta,
-        });
-        await writeJsonStableIfChanged(metaPath, nextMeta);
-        await writeTextIfChanged(diffstatPath, "");
-        await writeTextIfChanged(verifyLogPath, "");
-        if (!(await fileExists(reviewPath))) {
-            const review = renderPrReviewTemplate({ author, createdAt, branch });
-            await atomicWriteFile(reviewPath, review, "utf8");
-        }
-        output.success("pr open", path.relative(resolved.gitRoot, prDir));
+        output.success("pr open", path.relative(resolved.gitRoot, prDir), prOpenOutcomeDetails(meta, openOutcome ?? null));
         return 0;
     }
     catch (err) {

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

@@ -7,6 +7,7 @@ export type PrOpenParsed = {
     taskId: string;
     author: string;
     branch: string | null;
+    syncOnly: boolean;
 };
 export declare const prOpenSpec: CommandSpec<PrOpenParsed>;
 export type PrUpdateParsed = {
@@ -22,11 +23,25 @@ export type PrNoteParsed = {
     author: string;
     body: string;
 };
+export type PrCloseParsed = {
+    prNumber: number;
+    repo?: string;
+    comment?: string;
+    deleteRemoteBranch: boolean;
+};
+export type PrCloseSupersededParsed = {
+    taskId: string;
+    deleteRemoteBranch: boolean;
+};
 export declare const prNoteSpec: CommandSpec<PrNoteParsed>;
+export declare const prCloseSpec: CommandSpec<PrCloseParsed>;
+export declare const prCloseSupersededSpec: CommandSpec<PrCloseSupersededParsed>;
 export declare function makeRunPrHandler(_getCtx: (cmd: string) => Promise<CommandContext>): CommandHandler<GroupCommandParsed>;
 export declare function makeRunPrOpenHandler(getCtx: (cmd: string) => Promise<CommandContext>): (ctx: CommandCtx, p: PrOpenParsed) => Promise<number>;
 export declare function makeRunPrUpdateHandler(getCtx: (cmd: string) => Promise<CommandContext>): (ctx: CommandCtx, p: PrUpdateParsed) => Promise<number>;
 export declare function makeRunPrCheckHandler(getCtx: (cmd: string) => Promise<CommandContext>): (ctx: CommandCtx, p: PrCheckParsed) => Promise<number>;
 export declare function makeRunPrNoteHandler(getCtx: (cmd: string) => Promise<CommandContext>): (ctx: CommandCtx, p: PrNoteParsed) => Promise<number>;
+export declare function makeRunPrCloseHandler(getCtx: (cmd: string) => Promise<CommandContext>): (ctx: CommandCtx, p: PrCloseParsed) => Promise<number>;
+export declare function makeRunPrCloseSupersededHandler(getCtx: (cmd: string) => Promise<CommandContext>): (ctx: CommandCtx, p: PrCloseSupersededParsed) => Promise<number>;
 export {};
 //# sourceMappingURL=pr.command.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"pr.command.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/pr.command.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AACtF,OAAO,EAIL,KAAK,kBAAkB,EACxB,MAAM,4BAA4B,CAAC;AAEpC,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAIhE,KAAK,aAAa,GAAG,kBAAkB,CAAC;AAExC,eAAO,MAAM,MAAM,EAAE,WAAW,CAAC,aAAa,CAgB7C,CAAC;AAEF,MAAM,MAAM,YAAY,GAAG;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAAA;CAAE,CAAC;AAErF,eAAO,MAAM,UAAU,EAAE,WAAW,CAAC,YAAY,CAgChD,CAAC;AAEF,MAAM,MAAM,cAAc,GAAG;IAAE,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC;AAEhD,eAAO,MAAM,YAAY,EAAE,WAAW,CAAC,cAAc,CAOpD,CAAC;AAEF,MAAM,MAAM,aAAa,GAAG;IAAE,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC;AAE/C,eAAO,MAAM,WAAW,EAAE,WAAW,CAAC,aAAa,CAOlD,CAAC;AAEF,MAAM,MAAM,YAAY,GAAG;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAA;CAAE,CAAC;AAE5E,eAAO,MAAM,UAAU,EAAE,WAAW,CAAC,YAAY,CA0ChD,CAAC;AAWF,wBAAgB,gBAAgB,CAC9B,OAAO,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,GAChD,cAAc,CAAC,kBAAkB,CAAC,CAEpC;AAED,wBAAgB,oBAAoB,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IACrE,KAAK,UAAU,EAAE,GAAG,YAAY,KAAG,OAAO,CAAC,MAAM,CAAC,CAUjE;AAED,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IACvE,KAAK,UAAU,EAAE,GAAG,cAAc,KAAG,OAAO,CAAC,MAAM,CAAC,CAQnE;AAED,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IACtE,KAAK,UAAU,EAAE,GAAG,aAAa,KAAG,OAAO,CAAC,MAAM,CAAC,CAQlE;AAED,wBAAgB,oBAAoB,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IACrE,KAAK,UAAU,EAAE,GAAG,YAAY,KAAG,OAAO,CAAC,MAAM,CAAC,CAUjE"}
+{"version":3,"file":"pr.command.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/pr.command.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AACtF,OAAO,EAIL,KAAK,kBAAkB,EACxB,MAAM,4BAA4B,CAAC;AAEpC,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAWhE,KAAK,aAAa,GAAG,kBAAkB,CAAC;AAExC,eAAO,MAAM,MAAM,EAAE,WAAW,CAAC,aAAa,CA8B7C,CAAC;AAEF,MAAM,MAAM,YAAY,GAAG;IACzB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,QAAQ,EAAE,OAAO,CAAC;CACnB,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,WAAW,CAAC,YAAY,CAuChD,CAAC;AAEF,MAAM,MAAM,cAAc,GAAG;IAAE,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC;AAEhD,eAAO,MAAM,YAAY,EAAE,WAAW,CAAC,cAAc,CAOpD,CAAC;AAEF,MAAM,MAAM,aAAa,GAAG;IAAE,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC;AAE/C,eAAO,MAAM,WAAW,EAAE,WAAW,CAAC,aAAa,CAOlD,CAAC;AAEF,MAAM,MAAM,YAAY,GAAG;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAA;CAAE,CAAC;AAC5E,MAAM,MAAM,aAAa,GAAG;IAC1B,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,kBAAkB,EAAE,OAAO,CAAC;CAC7B,CAAC;AAEF,MAAM,MAAM,uBAAuB,GAAG;IACpC,MAAM,EAAE,MAAM,CAAC;IACf,kBAAkB,EAAE,OAAO,CAAC;CAC7B,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,WAAW,CAAC,YAAY,CA0ChD,CAAC;AAEF,eAAO,MAAM,WAAW,EAAE,WAAW,CAAC,aAAa,CAkDlD,CAAC;AAEF,eAAO,MAAM,qBAAqB,EAAE,WAAW,CAAC,uBAAuB,CAwBtE,CAAC;AAWF,wBAAgB,gBAAgB,CAC9B,OAAO,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,GAChD,cAAc,CAAC,kBAAkB,CAAC,CAEpC;AAED,wBAAgB,oBAAoB,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IACrE,KAAK,UAAU,EAAE,GAAG,YAAY,KAAG,OAAO,CAAC,MAAM,CAAC,CAWjE;AAED,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IACvE,KAAK,UAAU,EAAE,GAAG,cAAc,KAAG,OAAO,CAAC,MAAM,CAAC,CAQnE;AAED,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IACtE,KAAK,UAAU,EAAE,GAAG,aAAa,KAAG,OAAO,CAAC,MAAM,CAAC,CAQlE;AAED,wBAAgB,oBAAoB,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IACrE,KAAK,UAAU,EAAE,GAAG,YAAY,KAAG,OAAO,CAAC,MAAM,CAAC,CAUjE;AAED,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IACtE,KAAK,UAAU,EAAE,GAAG,aAAa,KAAG,OAAO,CAAC,MAAM,CAAC,CAWlE;AAED,wBAAgB,+BAA+B,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IAChF,KAAK,UAAU,EAAE,GAAG,uBAAuB,KAAG,OAAO,CAAC,MAAM,CAAC,CAS5E"}

package/dist/commands/pr/pr.command.js

@@ -1,20 +1,33 @@
 import { loadDirectSubcommandNames, parseGroupCommand, throwGroupCommandUsage, } from "../../cli/group-command.js";
 import { usageError } from "../../cli/spec/errors.js";
-import { cmdPrCheck, cmdPrNote, cmdPrOpen, cmdPrUpdate } from "./index.js";
+import { cmdPrCheck, cmdPrClose, cmdPrCloseSuperseded, cmdPrNote, cmdPrOpen, cmdPrUpdate, } from "./index.js";
 export const prSpec = {
     id: ["pr"],
     group: "PR",
-    summary: "Manage local PR artifacts for a task (branch_pr workflow).",
-    synopsis: ["agentplane pr <open|update|check|note> <task-id> [options]"],
+    summary: "Manage local PR review and GitHub publication artifacts for a task (branch_pr workflow).",
+    synopsis: [
+        "agentplane pr <open|update|check|note|close|close-superseded> <task-id|pr-number> [options]",
+    ],
     args: [{ name: "cmd", required: false, variadic: true, valueHint: "<cmd>" }],
     examples: [
         { cmd: "agentplane pr open 202602030608-F1Q8AB --author CODER", why: "Create PR artifacts." },
-        { cmd: "agentplane pr update 202602030608-F1Q8AB", why: "Refresh diffstat and summary." },
+        {
+            cmd: "agentplane pr update 202602030608-F1Q8AB",
+            why: "Refresh review.md plus github-title/body projections.",
+        },
         { cmd: "agentplane pr check 202602030608-F1Q8AB", why: "Validate PR artifacts." },
         {
             cmd: 'agentplane pr note 202602030608-F1Q8AB --author REVIEWER --body "Looks good"',
             why: "Append a handoff note.",
         },
+        {
+            cmd: 'agentplane pr close 123 --comment "Superseded by #456" --delete-remote-branch',
+            why: "Close a stale GitHub PR through REST and optionally delete its remote head branch.",
+        },
+        {
+            cmd: "agentplane pr close-superseded 202604072308-A1XE27 --delete-remote-branch",
+            why: "Close a superseded task PR from task artifacts after protected-main closure.",
+        },
     ],
     parse: (raw) => parseGroupCommand(raw),
 };
@@ -37,6 +50,12 @@ export const prOpenSpec = {
             valueHint: "<name>",
             description: "Branch name (default: current branch).",
         },
+        {
+            kind: "boolean",
+            name: "sync-only",
+            default: false,
+            description: "Only write local PR artifacts; do not create a remote GitHub PR.",
+        },
     ],
     examples: [{ cmd: "agentplane pr open 202602030608-F1Q8AB --author CODER", why: "Open." }],
     validateRaw: (raw) => {
@@ -49,12 +68,13 @@ export const prOpenSpec = {
         taskId: String(raw.args["task-id"]),
         author: String(raw.opts.author),
         branch: typeof raw.opts.branch === "string" ? raw.opts.branch : null,
+        syncOnly: Boolean(raw.opts["sync-only"]),
     }),
 };
 export const prUpdateSpec = {
     id: ["pr", "update"],
     group: "PR",
-    summary: "Update PR artifacts (diffstat and auto summary).",
+    summary: "Update PR artifacts (review packet, diffstat, and GitHub projections).",
     args: [{ name: "task-id", required: true, valueHint: "<task-id>" }],
     examples: [{ cmd: "agentplane pr update 202602030608-F1Q8AB", why: "Update artifacts." }],
     parse: (raw) => ({ taskId: String(raw.args["task-id"]) }),
@@ -110,6 +130,80 @@ export const prNoteSpec = {
         body: String(raw.opts.body),
     }),
 };
+export const prCloseSpec = {
+    id: ["pr", "close"],
+    group: "PR",
+    summary: "Close a GitHub PR through REST with optional remote head-branch deletion.",
+    args: [{ name: "pr-number", required: true, valueHint: "<pr-number>" }],
+    options: [
+        {
+            kind: "string",
+            name: "repo",
+            valueHint: "<owner/name>",
+            description: "Optional. GitHub owner/repo override (defaults to origin remote).",
+        },
+        {
+            kind: "string",
+            name: "comment",
+            valueHint: "<text>",
+            description: "Optional. Add a close comment before closing the PR.",
+        },
+        {
+            kind: "boolean",
+            name: "delete-remote-branch",
+            default: false,
+            description: "Delete the remote head branch after a successful close when it belongs to the target repo.",
+        },
+    ],
+    examples: [
+        {
+            cmd: 'agentplane pr close 123 --comment "Superseded by #456" --delete-remote-branch',
+            why: "Close a stale PR and remove its remote head branch when it belongs to the same repo.",
+        },
+    ],
+    validateRaw: (raw) => {
+        const prNumber = Number.parseInt(String(raw.args["pr-number"] ?? ""), 10);
+        if (!Number.isInteger(prNumber) || prNumber <= 0) {
+            throw usageError({ spec: prCloseSpec, message: "Invalid value for <pr-number>." });
+        }
+        if (typeof raw.opts.repo === "string" && raw.opts.repo.trim() === "") {
+            throw usageError({ spec: prCloseSpec, message: "Invalid value for --repo: empty." });
+        }
+        if (typeof raw.opts.comment === "string" && raw.opts.comment.trim() === "") {
+            throw usageError({ spec: prCloseSpec, message: "Invalid value for --comment: empty." });
+        }
+    },
+    parse: (raw) => ({
+        prNumber: Number.parseInt(String(raw.args["pr-number"]), 10),
+        repo: typeof raw.opts.repo === "string" ? raw.opts.repo : undefined,
+        comment: typeof raw.opts.comment === "string" ? raw.opts.comment : undefined,
+        deleteRemoteBranch: raw.opts["delete-remote-branch"] === true,
+    }),
+};
+export const prCloseSupersededSpec = {
+    id: ["pr", "close-superseded"],
+    group: "PR",
+    summary: "Close a superseded task PR using task artifacts.",
+    args: [{ name: "task-id", required: true, valueHint: "<task-id>" }],
+    options: [
+        {
+            kind: "boolean",
+            name: "delete-remote-branch",
+            default: false,
+            description: "Delete the remote head branch after a successful close when it belongs to the task repo.",
+        },
+    ],
+    examples: [
+        {
+            cmd: "agentplane pr close-superseded 202604072308-A1XE27 --delete-remote-branch",
+            why: "Repair a stale task PR after the task has already been merged to main.",
+        },
+    ],
+    parse: (raw) => ({
+        taskId: String(raw.args["task-id"]),
+        deleteRemoteBranch: raw.opts["delete-remote-branch"] === true,
+    }),
+};
 async function runPrRootGroup(_ctx, p) {
     throwGroupCommandUsage({
         spec: prSpec,
@@ -130,6 +224,7 @@ export function makeRunPrOpenHandler(getCtx) {
             taskId: p.taskId,
             author: p.author,
             branch: p.branch ?? undefined,
+            syncOnly: p.syncOnly,
         });
     };
 }
@@ -165,3 +260,27 @@ export function makeRunPrNoteHandler(getCtx) {
         });
     };
 }
+export function makeRunPrCloseHandler(getCtx) {
+    return async (ctx, p) => {
+        return await cmdPrClose({
+            ctx: await getCtx("pr close"),
+            cwd: ctx.cwd,
+            rootOverride: ctx.rootOverride,
+            prNumber: p.prNumber,
+            repo: p.repo,
+            comment: p.comment,
+            deleteRemoteBranch: p.deleteRemoteBranch,
+        });
+    };
+}
+export function makeRunPrCloseSupersededHandler(getCtx) {
+    return async (ctx, p) => {
+        return await cmdPrCloseSuperseded({
+            ctx: await getCtx("pr close-superseded"),
+            cwd: ctx.cwd,
+            rootOverride: ctx.rootOverride,
+            taskId: p.taskId,
+            deleteRemoteBranch: p.deleteRemoteBranch,
+        });
+    };
+}

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

@@ -1 +1 @@
-{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/update.ts"],"names":[],"mappings":"AAaA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AASnC,wBAAsB,WAAW,CAAC,IAAI,EAAE;IACtC,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,CAwFlB"}
+{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/update.ts"],"names":[],"mappings":"AAOA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AAmEnC,wBAAsB,WAAW,CAAC,IAAI,EAAE;IACtC,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,CA4BlB"}

package/dist/commands/pr/update.js

@@ -1,92 +1,76 @@
 import { readFile } from "node:fs/promises";
 import path from "node:path";
-import { resolveBaseBranch } from "@agentplaneorg/core";
 import { mapBackendError } from "../../cli/error-map.js";
-import { fileExists } from "../../cli/fs-utils.js";
-import { createCliEmitter, workflowModeMessage } from "../../cli/output.js";
+import { createCliEmitter } from "../../cli/output.js";
 import { CliError } from "../../shared/errors.js";
-import { writeJsonStableIfChanged, writeTextIfChanged } from "../../shared/write-if-changed.js";
-import { execFileAsync, gitEnv } from "../shared/git.js";
-import { gitCurrentBranch } from "../shared/git-ops.js";
-import { buildUpdatedPrMeta, parsePrMeta } from "../shared/pr-meta.js";
+import { gitRevParse } from "../shared/git-ops.js";
 import { loadBackendTask, loadCommandContext, } from "../shared/task-backend.js";
-import { resolvePrPaths } from "./internal/pr-paths.js";
-import { updateAutoSummaryBlock } from "./internal/review-template.js";
-function nowIso() {
-    return new Date().toISOString();
+import { assessPrArtifactFreshness } from "./internal/freshness.js";
+import { syncPrArtifacts } from "./internal/sync.js";
+async function readTextIfExists(filePath) {
+    try {
+        return await readFile(filePath, "utf8");
+    }
+    catch (err) {
+        const code = err?.code;
+        if (code === "ENOENT")
+            return null;
+        throw err;
+    }
+}
+async function warnOnStaleVerifyAfterUpdate(opts) {
+    const branch = opts.meta.branch?.trim() ?? "";
+    if (!branch)
+        return;
+    const ctx = opts.ctx ??
+        (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
+    const { config, task } = await loadBackendTask({
+        ctx,
+        cwd: opts.cwd,
+        rootOverride: opts.rootOverride,
+        taskId: opts.taskId,
+    });
+    const requiresVerify = Boolean(task.verify && task.verify.length > 0);
+    if (!requiresVerify)
+        return;
+    const branchHeadSha = await gitRevParse(opts.resolved.gitRoot, [`${branch}^{commit}`]);
+    const verifyLogText = await readTextIfExists(path.join(opts.prDir, "verify.log"));
+    const freshness = await assessPrArtifactFreshness({
+        gitRoot: opts.resolved.gitRoot,
+        workflowDir: path.join(opts.resolved.gitRoot, config.paths.workflow_dir),
+        taskId: opts.taskId,
+        branchHeadSha,
+        metaHeadSha: opts.meta.head_sha ?? null,
+        metaLastVerifiedSha: opts.meta.last_verified_sha ?? null,
+        metaVerifyStatus: opts.meta.verify?.status ?? null,
+        taskVerificationState: task.verification?.state ?? null,
+        verifyLogText,
+        requiresVerify,
+    });
+    if (opts.meta.last_verified_sha && !freshness.verifyFresh) {
+        opts.output.warn(`Verify state stale: last_verified_sha=${opts.meta.last_verified_sha} current_head=${branchHeadSha}; run \`agentplane verify ${opts.taskId} --ok --by <ROLE> --note "Verified: ..."\` before integrating`);
+    }
 }
 export async function cmdPrUpdate(opts) {
     try {
         const output = createCliEmitter();
-        const ctx = opts.ctx ??
-            (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
-        await loadBackendTask({
-            ctx,
+        const { meta, prDir, resolved } = await syncPrArtifacts({
+            ctx: opts.ctx,
             cwd: opts.cwd,
             rootOverride: opts.rootOverride,
             taskId: opts.taskId,
+            mode: "update",
         });
-        const { resolved, config, prDir, metaPath, diffstatPath, reviewPath } = await resolvePrPaths({
-            ...opts,
-            ctx,
-        });
-        if (config.workflow_mode !== "branch_pr") {
-            throw new CliError({
-                exitCode: 2,
-                code: "E_USAGE",
-                message: workflowModeMessage(config.workflow_mode, "branch_pr"),
-            });
-        }
-        if (!(await fileExists(metaPath)) || !(await fileExists(reviewPath))) {
-            const missing = [];
-            if (!(await fileExists(metaPath)))
-                missing.push(path.relative(resolved.gitRoot, metaPath));
-            if (!(await fileExists(reviewPath)))
-                missing.push(path.relative(resolved.gitRoot, reviewPath));
-            throw new CliError({
-                exitCode: 3,
-                code: "E_VALIDATION",
-                message: `PR artifacts missing: ${missing.join(", ")} (run \`agentplane pr open\`)`,
-            });
-        }
-        const baseBranch = await resolveBaseBranch({
+        await warnOnStaleVerifyAfterUpdate({
+            output,
+            ctx: opts.ctx,
             cwd: opts.cwd,
-            rootOverride: opts.rootOverride ?? null,
-            cliBaseOpt: null,
-            mode: config.workflow_mode,
-        });
-        if (!baseBranch) {
-            throw new CliError({
-                exitCode: 2,
-                code: "E_USAGE",
-                message: "Base branch could not be resolved (use `agentplane branch base set`).",
-            });
-        }
-        const branch = await gitCurrentBranch(resolved.gitRoot);
-        const { stdout: diffStatOut } = await execFileAsync("git", ["diff", "--stat", `${baseBranch}...HEAD`], { cwd: resolved.gitRoot, env: gitEnv() });
-        const diffstat = diffStatOut.trimEnd();
-        await writeTextIfChanged(diffstatPath, diffstat ? `${diffstat}\n` : "");
-        const { stdout: headOut } = await execFileAsync("git", ["rev-parse", "HEAD"], {
-            cwd: resolved.gitRoot,
-            env: gitEnv(),
+            rootOverride: opts.rootOverride,
+            taskId: opts.taskId,
+            prDir,
+            resolved,
+            meta,
         });
-        const headSha = headOut.trim();
-        const summaryLines = [
-            `- Updated: ${nowIso()}`,
-            `- Branch: ${branch}`,
-            `- Head: ${headSha.slice(0, 12)}`,
-            "- Diffstat:",
-            "```",
-            diffstat || "No changes detected.",
-            "```",
-        ];
-        const reviewText = await readFile(reviewPath, "utf8");
-        const nextReview = updateAutoSummaryBlock(reviewText, summaryLines.join("\n"));
-        await writeTextIfChanged(reviewPath, nextReview);
-        const rawMeta = await readFile(metaPath, "utf8");
-        const meta = parsePrMeta(rawMeta, opts.taskId);
-        const nextMeta = buildUpdatedPrMeta({ meta, branch, at: nowIso() });
-        await writeJsonStableIfChanged(metaPath, nextMeta);
         output.success("pr update", path.relative(resolved.gitRoot, prDir));
         return 0;
     }

package/dist/commands/recipes/impl/commands/cache-prune.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cache-prune.d.ts","sourceRoot":"","sources":["../../../../../src/commands/recipes/impl/commands/cache-prune.ts"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,aAAa,CAAC;AA6BzD,wBAAsB,yBAAyB,CAAC,IAAI,EAAE;IACpD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,qBAAqB,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,CAAC,CAkFlB"}
+{"version":3,"file":"cache-prune.d.ts","sourceRoot":"","sources":["../../../../../src/commands/recipes/impl/commands/cache-prune.ts"],"names":[],"mappings":"AAaA,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,aAAa,CAAC;AA6BzD,wBAAsB,yBAAyB,CAAC,IAAI,EAAE;IACpD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,qBAAqB,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,CAAC,CA8FlB"}

package/dist/commands/recipes/impl/commands/cache-prune.js

@@ -1,9 +1,11 @@
 import { readdir, rm } from "node:fs/promises";
 import path from "node:path";
+import { defaultConfig } from "@agentplaneorg/core";
 import { mapCoreError } from "../../../../cli/error-map.js";
 import { fileExists } from "../../../../cli/fs-utils.js";
 import { infoMessage, successMessage } from "../../../../cli/output.js";
 import { CliError } from "../../../../shared/errors.js";
+import { ensureActionApproved } from "../../../shared/approval-requirements.js";
 import { readInstalledRecipesFile, writeInstalledRecipesFile } from "../installed-recipes.js";
 import { resolveGlobalRecipesDir, resolveInstalledRecipesPath } from "../paths.js";
 async function listRecipeCacheEntries(cacheDir) {
@@ -49,6 +51,12 @@ export async function cmdRecipeCachePruneParsed(opts) {
                 process.stdout.write(`${infoMessage(`dry-run: would remove ${cacheEntries.length} cached recipes`)}\n`);
                 return 0;
             }
+            await ensureActionApproved({
+                action: "dangerous_fs",
+                config: defaultConfig(),
+                yes: false,
+                reason: `recipes cache prune --all (${cacheEntries.length} cached recipes)`,
+            });
             await rm(cacheDir, { recursive: true, force: true });
             await writeInstalledRecipesFile(resolveInstalledRecipesPath(), {
                 schema_version: 1,
@@ -72,6 +80,12 @@ export async function cmdRecipeCachePruneParsed(opts) {
             process.stdout.write(`${infoMessage(`dry-run: would remove ${prune.length} cached recipes`)}\n`);
             return 0;
         }
+        await ensureActionApproved({
+            action: "dangerous_fs",
+            config: defaultConfig(),
+            yes: false,
+            reason: `recipes cache prune (${prune.length} cached recipes)`,
+        });
         const recipeDirs = new Set();
         for (const entry of prune) {
             recipeDirs.add(path.dirname(entry.path));

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

@@ -37,7 +37,7 @@ export async function cmdRecipeInstall(opts) {
         const ensureApproved = async (reason) => {
             if (networkApproved)
                 return;
-            await ensureNetworkApproved({ config, yes: opts.yes, reason });
+            await ensureNetworkApproved({ action: "recipe_install", config, yes: opts.yes, reason });
             networkApproved = true;
         };
         const tempRoot = await mkdtemp(path.join(os.tmpdir(), "agentplane-recipe-"));

package/dist/commands/recipes/impl/commands/list-remote.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"list-remote.d.ts","sourceRoot":"","sources":["../../../../../src/commands/recipes/impl/commands/list-remote.ts"],"names":[],"mappings":"AAWA,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,aAAa,CAAC;AAEzD,wBAAsB,yBAAyB,CAAC,IAAI,EAAE;IACpD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,qBAAqB,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,CAAC,CAwClB"}
+{"version":3,"file":"list-remote.d.ts","sourceRoot":"","sources":["../../../../../src/commands/recipes/impl/commands/list-remote.ts"],"names":[],"mappings":"AAWA,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,aAAa,CAAC;AAEzD,wBAAsB,yBAAyB,CAAC,IAAI,EAAE;IACpD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,qBAAqB,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,CAAC,CAyClB"}

package/dist/commands/recipes/impl/commands/list-remote.js

@@ -25,6 +25,7 @@ export async function cmdRecipeListRemoteParsed(opts) {
         });
         if (willFetchRemote) {
             await ensureNetworkApproved({
+                action: "recipe_list_remote",
                 config,
                 yes: flags.yes,
                 reason: "recipes list-remote fetches the remote recipes index",

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

@@ -1 +1 @@
-{"version":3,"file":"remove.d.ts","sourceRoot":"","sources":["../../../../../src/commands/recipes/impl/commands/remove.ts"],"names":[],"mappings":"AAYA,wBAAsB,qBAAqB,CAAC,IAAI,EAAE;IAChD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,EAAE,EAAE,MAAM,CAAC;CACZ,GAAG,OAAO,CAAC,MAAM,CAAC,CAwBlB"}
+{"version":3,"file":"remove.d.ts","sourceRoot":"","sources":["../../../../../src/commands/recipes/impl/commands/remove.ts"],"names":[],"mappings":"AAaA,wBAAsB,qBAAqB,CAAC,IAAI,EAAE;IAChD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,EAAE,EAAE,MAAM,CAAC;CACZ,GAAG,OAAO,CAAC,MAAM,CAAC,CA+BlB"}

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

@@ -1,9 +1,10 @@
-import { resolveProject } from "@agentplaneorg/core";
+import { loadConfig, resolveProject } from "@agentplaneorg/core";
 import { rm } from "node:fs/promises";
 import { mapCoreError } from "../../../../cli/error-map.js";
 import { exitCodeForError } from "../../../../cli/exit-codes.js";
 import { successMessage } from "../../../../cli/output.js";
 import { CliError } from "../../../../shared/errors.js";
+import { ensureActionApproved } from "../../../shared/approval-requirements.js";
 import { readProjectInstalledRecipes } from "../project-installed-recipes.js";
 import { resolveProjectInstalledRecipeDir } from "../paths.js";
 export async function cmdRecipeRemoveParsed(opts) {
@@ -12,6 +13,7 @@ export async function cmdRecipeRemoveParsed(opts) {
             cwd: opts.cwd,
             rootOverride: opts.rootOverride ?? null,
         });
+        const loaded = await loadConfig(resolved.agentplaneDir);
         const installed = await readProjectInstalledRecipes(resolved);
         const entry = installed.recipes.find((recipe) => recipe.id === opts.id);
         if (!entry) {
@@ -22,6 +24,12 @@ export async function cmdRecipeRemoveParsed(opts) {
             });
         }
         const recipeDir = resolveProjectInstalledRecipeDir(resolved, entry.id);
+        await ensureActionApproved({
+            action: "dangerous_fs",
+            config: loaded.config,
+            yes: false,
+            reason: `recipes remove ${entry.id}@${entry.version}`,
+        });
         await rm(recipeDir, { recursive: true, force: true });
         process.stdout.write(`${successMessage("removed recipe", `${entry.id}@${entry.version}`)}\n`);
         return 0;

package/dist/commands/release/apply.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"apply.command.d.ts","sourceRoot":"","sources":["../../../src/commands/release/apply.command.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AA6B1E,OAAO,KAAK,EAAE,kBAAkB,EAA0C,MAAM,kBAAkB,CAAC;AA0NnG,eAAO,MAAM,gBAAgB,EAAE,WAAW,CAAC,kBAAkB,CAiF5D,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,cAAc,CAAC,kBAAkB,CAmF9D,CAAC;AAEF,OAAO,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC"}
+{"version":3,"file":"apply.command.d.ts","sourceRoot":"","sources":["../../../src/commands/release/apply.command.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AA8B1E,OAAO,KAAK,EAAE,kBAAkB,EAA0C,MAAM,kBAAkB,CAAC;AAkOnG,eAAO,MAAM,gBAAgB,EAAE,WAAW,CAAC,kBAAkB,CA0E5D,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,cAAc,CAAC,kBAAkB,CA2E9D,CAAC;AAEF,OAAO,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC"}