agentplane 0.3.5 → 0.3.7

package/dist/commands/task/verify-ok.command.js

@@ -1,38 +1,11 @@
-import { usageError } from "../../cli/spec/errors.js";
 import { cmdTaskVerifyOk } from "./verify-record.js";
+import { parseVerifyCommonOptions, validateVerifyDetailsFileExclusive, validateVerifyNonEmptyInput, verifyCommonOptions, } from "./verify-command-shared.js";
 export const taskVerifyOkSpec = {
     id: ["task", "verify", "ok"],
     group: "Task",
     summary: "Record verification as OK (updates Verification section and verification frontmatter).",
     args: [{ name: "task-id", required: true, valueHint: "<task-id>" }],
-    options: [
-        { kind: "string", name: "by", valueHint: "<id>", required: true, description: "Verifier id." },
-        {
-            kind: "string",
-            name: "note",
-            valueHint: "<text>",
-            required: true,
-            description: "Short verification note.",
-        },
-        {
-            kind: "string",
-            name: "details",
-            valueHint: "<text>",
-            description: "Optional details text (mutually exclusive with --file).",
-        },
-        {
-            kind: "string",
-            name: "file",
-            valueHint: "<path>",
-            description: "Optional details file path (mutually exclusive with --details).",
-        },
-        {
-            kind: "boolean",
-            name: "quiet",
-            default: false,
-            description: "Suppress normal output (still prints errors).",
-        },
-    ],
+    options: verifyCommonOptions,
     examples: [
         {
             cmd: 'agentplane task verify ok 202602030608-F1Q8AB --by REVIEWER --note "Looks good"',
@@ -40,30 +13,15 @@ export const taskVerifyOkSpec = {
         },
     ],
     validateRaw: (raw) => {
-        const details = raw.opts.details;
-        const file = raw.opts.file;
-        if (typeof details === "string" && typeof file === "string") {
-            throw usageError({
-                spec: taskVerifyOkSpec,
-                message: "Provide at most one of --details or --file.",
-            });
-        }
-        const by = raw.opts.by;
-        if (typeof by === "string" && by.trim().length === 0) {
-            throw usageError({ spec: taskVerifyOkSpec, message: "Invalid value for --by: empty." });
-        }
-        const note = raw.opts.note;
-        if (typeof note === "string" && note.trim().length === 0) {
-            throw usageError({ spec: taskVerifyOkSpec, message: "Invalid value for --note: empty." });
-        }
+        validateVerifyDetailsFileExclusive(raw, taskVerifyOkSpec, {
+            message: "Provide at most one of --details or --file.",
+        });
+        validateVerifyNonEmptyInput(raw, taskVerifyOkSpec, "by");
+        validateVerifyNonEmptyInput(raw, taskVerifyOkSpec, "note");
     },
     parse: (raw) => ({
         taskId: String(raw.args["task-id"]),
-        by: String(raw.opts.by),
-        note: String(raw.opts.note),
-        details: typeof raw.opts.details === "string" ? raw.opts.details : undefined,
-        file: typeof raw.opts.file === "string" ? raw.opts.file : undefined,
-        quiet: raw.opts.quiet === true,
+        ...parseVerifyCommonOptions(raw),
     }),
 };
 export function makeRunTaskVerifyOkHandler(getCtx) {

package/dist/commands/task/verify-record.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"verify-record.d.ts","sourceRoot":"","sources":["../../../src/commands/task/verify-record.ts"],"names":[],"mappings":"AAUA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AAanC,KAAK,WAAW,GAAG,IAAI,GAAG,cAAc,CAAC;AAgJzC,wBAAsB,eAAe,CAAC,IAAI,EAAE;IAC1C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CA4ClB;AAED,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CA4ClB;AAED,wBAAsB,eAAe,CAAC,IAAI,EAAE;IAC1C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,WAAW,CAAC;IACnB,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CA4ClB"}
+{"version":3,"file":"verify-record.d.ts","sourceRoot":"","sources":["../../../src/commands/task/verify-record.ts"],"names":[],"mappings":"AAUA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AAqBnC,KAAK,WAAW,GAAG,IAAI,GAAG,cAAc,CAAC;AA0RzC,wBAAsB,eAAe,CAAC,IAAI,EAAE;IAC1C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAElB;AAED,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAMlB;AAED,wBAAsB,eAAe,CAAC,IAAI,EAAE;IAC1C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,WAAW,CAAC;IACnB,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAElB"}

package/dist/commands/task/verify-record.js

@@ -3,12 +3,12 @@ import { readFile } from "node:fs/promises";
 import path from "node:path";
 import { ensureDocSections, setMarkdownSection } from "@agentplaneorg/core";
 import { mapBackendError, mapCoreError } from "../../cli/error-map.js";
-import { backendNotSupportedMessage } from "../../cli/output.js";
+import { backendNotSupportedMessage, successMessage } from "../../cli/output.js";
 import { CliError } from "../../shared/errors.js";
 import { ensureReconciledBeforeMutation } from "../shared/reconcile-check.js";
 import { loadCommandContext, loadTaskFromContext, } from "../shared/task-backend.js";
-import { backendIsLocalFileBackend, getTaskStore } from "../shared/task-store.js";
-import { appendTaskEvent, extractDocSection, normalizeTaskDocVersion, normalizeVerificationSectionLayout, nowIso, VERIFICATION_RESULTS_BEGIN, VERIFICATION_RESULTS_END, } from "./shared.js";
+import { appendTaskEventIntent, backendIsLocalFileBackend, getTaskStore, setTaskFieldsIntent, setTaskSectionIntent, touchTaskDocMetaIntent, } from "../shared/task-store.js";
+import { appendTaskEvent, decodeEscapedTaskTextNewlines, extractDocSection, normalizeTaskDocVersion, normalizeVerificationSectionLayout, nowIso, VERIFICATION_RESULTS_BEGIN, VERIFICATION_RESULTS_END, } from "./shared.js";
 function appendBetweenMarkers(sectionText, entryText, version) {
     const ensured = normalizeVerificationSectionLayout(sectionText, version);
     const beginIdx = ensured.indexOf(VERIFICATION_RESULTS_BEGIN);
@@ -65,108 +65,113 @@ async function recordVerificationResult(opts) {
     }
     const useStore = backendIsLocalFileBackend(ctx);
     const store = useStore ? getTaskStore(ctx) : null;
-    const task = useStore
-        ? await store.get(opts.taskId)
-        : await loadTaskFromContext({ ctx, taskId: opts.taskId });
-    const existingDoc = useStore
-        ? String(task.doc ?? "")
-        : (typeof task.doc === "string" ? task.doc : "") || (await backend.getTaskDoc(task.id));
-    const baseDoc = ensureDocSections(existingDoc ?? "", config.tasks.doc.required_sections);
-    const verificationSection = extractDocSection(baseDoc, "Verification") ?? "";
-    const verifySteps = extractDocSection(baseDoc, "Verify Steps");
-    const verifyStepsHash = verifySteps
-        ? sha256Hex(verifySteps.replaceAll("\r\n", "\n").trim())
-        : null;
-    const docVersion = normalizeTaskDocVersion(task.doc_version);
-    const verifyStepsRef = [
-        `doc_version=${String(docVersion)}`,
-        `doc_updated_at=${String(task.doc_updated_at ?? "missing")}`,
-        `excerpt_hash=sha256:${verifyStepsHash ?? "missing"}`,
-    ].join(", ");
+    const task = useStore ? null : await loadTaskFromContext({ ctx, taskId: opts.taskId });
     const at = nowIso();
-    const entry = renderVerificationEntry({
-        at,
-        state: opts.state,
-        by: opts.by,
-        note: opts.note,
-        details: opts.details ?? null,
-        verifyStepsRef,
-    });
-    const nextVerification = appendBetweenMarkers(verificationSection, entry, docVersion);
-    const nextDoc = ensureDocSections(setMarkdownSection(baseDoc, "Verification", nextVerification), config.tasks.doc.required_sections);
-    const nextTask = {
-        ...task,
-        status: opts.state === "needs_rework" ? "DOING" : task.status,
-        commit: opts.state === "needs_rework" ? null : (task.commit ?? null),
-        doc: nextDoc,
-        doc_updated_by: opts.by,
-        events: appendTaskEvent(task, {
-            type: "verify",
+    if (useStore) {
+        await store.mutate(opts.taskId, (current) => {
+            const existingDoc = String(current.doc ?? "");
+            const baseDoc = ensureDocSections(existingDoc, config.tasks.doc.required_sections);
+            const verificationSection = extractDocSection(baseDoc, "Verification") ?? "";
+            const verifySteps = extractDocSection(baseDoc, "Verify Steps");
+            const verifyStepsHash = verifySteps
+                ? sha256Hex(verifySteps.replaceAll("\r\n", "\n").trim())
+                : null;
+            const docVersion = normalizeTaskDocVersion(current.doc_version);
+            const verifyStepsRef = [
+                `doc_version=${String(docVersion)}`,
+                `doc_updated_at=${String(current.doc_updated_at ?? "missing")}`,
+                `excerpt_hash=sha256:${verifyStepsHash ?? "missing"}`,
+            ].join(", ");
+            const entry = renderVerificationEntry({
+                at,
+                state: opts.state,
+                by: opts.by,
+                note: opts.note,
+                details: opts.details ?? null,
+                verifyStepsRef,
+            });
+            const nextVerification = appendBetweenMarkers(verificationSection, entry, docVersion);
+            return [
+                setTaskFieldsIntent({
+                    status: opts.state === "needs_rework" ? "DOING" : current.status,
+                    commit: opts.state === "needs_rework" ? null : (current.commit ?? null),
+                    verification: {
+                        state: opts.state,
+                        updated_at: at,
+                        updated_by: opts.by,
+                        note: opts.note,
+                    },
+                }),
+                setTaskSectionIntent({
+                    section: "Verification",
+                    text: nextVerification,
+                    requiredSections: config.tasks.doc.required_sections,
+                }),
+                appendTaskEventIntent({
+                    type: "verify",
+                    at,
+                    author: opts.by,
+                    state: opts.state,
+                    note: opts.note,
+                }),
+                touchTaskDocMetaIntent({ updatedBy: opts.by }),
+            ];
+        });
+    }
+    else {
+        const remoteTask = task;
+        const existingDoc = (typeof remoteTask.doc === "string" ? remoteTask.doc : "") ||
+            (await backend.getTaskDoc(remoteTask.id));
+        const baseDoc = ensureDocSections(existingDoc ?? "", config.tasks.doc.required_sections);
+        const verificationSection = extractDocSection(baseDoc, "Verification") ?? "";
+        const verifySteps = extractDocSection(baseDoc, "Verify Steps");
+        const verifyStepsHash = verifySteps
+            ? sha256Hex(verifySteps.replaceAll("\r\n", "\n").trim())
+            : null;
+        const docVersion = normalizeTaskDocVersion(remoteTask.doc_version);
+        const verifyStepsRef = [
+            `doc_version=${String(docVersion)}`,
+            `doc_updated_at=${String(remoteTask.doc_updated_at ?? "missing")}`,
+            `excerpt_hash=sha256:${verifyStepsHash ?? "missing"}`,
+        ].join(", ");
+        const entry = renderVerificationEntry({
             at,
-            author: opts.by,
             state: opts.state,
+            by: opts.by,
             note: opts.note,
-        }),
-        verification: {
-            state: opts.state,
-            updated_at: at,
-            updated_by: opts.by,
-            note: opts.note,
-        },
-    };
-    await (useStore ? store.update(opts.taskId, () => nextTask) : backend.writeTask(nextTask));
-    if (!opts.quiet) {
-        const readmePath = path.join(resolved.gitRoot, config.paths.workflow_dir, task.id, "README.md");
-        process.stdout.write(`${readmePath}\n`);
-    }
-}
-export async function cmdTaskVerifyOk(opts) {
-    const by = String(opts.by ?? "").trim();
-    const note = String(opts.note ?? "").trim();
-    if (!by || !note) {
-        throw new CliError({
-            exitCode: 2,
-            code: "E_USAGE",
-            message: "Missing required inputs: --by and --note.",
+            details: opts.details ?? null,
+            verifyStepsRef,
         });
-    }
-    if (typeof opts.details === "string" && typeof opts.file === "string") {
-        throw new CliError({
-            exitCode: 2,
-            code: "E_USAGE",
-            message: "Options --details and --file are mutually exclusive.",
+        const nextVerification = appendBetweenMarkers(verificationSection, entry, docVersion);
+        const nextDoc = ensureDocSections(setMarkdownSection(baseDoc, "Verification", nextVerification), config.tasks.doc.required_sections);
+        await backend.writeTask({
+            ...remoteTask,
+            status: opts.state === "needs_rework" ? "DOING" : remoteTask.status,
+            commit: opts.state === "needs_rework" ? null : (remoteTask.commit ?? null),
+            doc: nextDoc,
+            doc_updated_by: opts.by,
+            events: appendTaskEvent(remoteTask, {
+                type: "verify",
+                at,
+                author: opts.by,
+                state: opts.state,
+                note: opts.note,
+            }),
+            verification: {
+                state: opts.state,
+                updated_at: at,
+                updated_by: opts.by,
+                note: opts.note,
+            },
         });
     }
-    let details = typeof opts.details === "string" ? opts.details : null;
-    if (typeof opts.file === "string") {
-        try {
-            details = await readFile(path.resolve(opts.cwd, opts.file), "utf8");
-        }
-        catch (err) {
-            throw mapCoreError(err, { command: "task verify ok", filePath: opts.file });
-        }
-    }
-    try {
-        await recordVerificationResult({
-            ctx: opts.ctx,
-            cwd: opts.cwd,
-            rootOverride: opts.rootOverride,
-            taskId: opts.taskId,
-            state: "ok",
-            by,
-            note,
-            details,
-            quiet: opts.quiet,
-        });
-        return 0;
-    }
-    catch (err) {
-        if (err instanceof CliError)
-            throw err;
-        throw mapBackendError(err, { command: "task verify ok", root: opts.rootOverride ?? null });
+    if (!opts.quiet) {
+        const readmePath = path.join(resolved.gitRoot, config.paths.workflow_dir, opts.taskId, "README.md");
+        const relReadmePath = path.relative(resolved.gitRoot, readmePath);
+        process.stdout.write(`${successMessage("verified", opts.taskId, `state=${opts.state} readme=${relReadmePath}`)}\n`);
     }
 }
-export async function cmdTaskVerifyRework(opts) {
+async function resolveVerifyRecordInput(opts) {
     const by = String(opts.by ?? "").trim();
     const note = String(opts.note ?? "").trim();
     if (!by || !note) {
@@ -189,55 +194,16 @@ export async function cmdTaskVerifyRework(opts) {
             details = await readFile(path.resolve(opts.cwd, opts.file), "utf8");
         }
         catch (err) {
-            throw mapCoreError(err, { command: "task verify rework", filePath: opts.file });
+            throw mapCoreError(err, { command: opts.command, filePath: opts.file });
         }
     }
-    try {
-        await recordVerificationResult({
-            ctx: opts.ctx,
-            cwd: opts.cwd,
-            rootOverride: opts.rootOverride,
-            taskId: opts.taskId,
-            state: "needs_rework",
-            by,
-            note,
-            details,
-            quiet: opts.quiet,
-        });
-        return 0;
-    }
-    catch (err) {
-        if (err instanceof CliError)
-            throw err;
-        throw mapBackendError(err, { command: "task verify rework", root: opts.rootOverride ?? null });
+    if (typeof details === "string") {
+        details = decodeEscapedTaskTextNewlines(details);
     }
+    return { by, note, details };
 }
-export async function cmdVerifyParsed(opts) {
-    const by = String(opts.by ?? "").trim();
-    const note = String(opts.note ?? "").trim();
-    if (!by || !note) {
-        throw new CliError({
-            exitCode: 2,
-            code: "E_USAGE",
-            message: "Missing required inputs: --by and --note.",
-        });
-    }
-    if (typeof opts.details === "string" && typeof opts.file === "string") {
-        throw new CliError({
-            exitCode: 2,
-            code: "E_USAGE",
-            message: "Options --details and --file are mutually exclusive.",
-        });
-    }
-    let details = typeof opts.details === "string" ? opts.details : null;
-    if (typeof opts.file === "string") {
-        try {
-            details = await readFile(path.resolve(opts.cwd, opts.file), "utf8");
-        }
-        catch (err) {
-            throw mapCoreError(err, { command: "verify", filePath: opts.file });
-        }
-    }
+async function executeVerifyRecordCommand(opts) {
+    const input = await resolveVerifyRecordInput(opts);
     try {
         await recordVerificationResult({
             ctx: opts.ctx,
@@ -245,9 +211,9 @@ export async function cmdVerifyParsed(opts) {
             rootOverride: opts.rootOverride,
             taskId: opts.taskId,
             state: opts.state,
-            by,
-            note,
-            details,
+            by: input.by,
+            note: input.note,
+            details: input.details,
             quiet: opts.quiet,
         });
         return 0;
@@ -255,6 +221,19 @@ export async function cmdVerifyParsed(opts) {
     catch (err) {
         if (err instanceof CliError)
             throw err;
-        throw mapBackendError(err, { command: "verify", root: opts.rootOverride ?? null });
+        throw mapBackendError(err, { command: opts.command, root: opts.rootOverride ?? null });
     }
 }
+export async function cmdTaskVerifyOk(opts) {
+    return await executeVerifyRecordCommand({ ...opts, state: "ok", command: "task verify ok" });
+}
+export async function cmdTaskVerifyRework(opts) {
+    return await executeVerifyRecordCommand({
+        ...opts,
+        state: "needs_rework",
+        command: "task verify rework",
+    });
+}
+export async function cmdVerifyParsed(opts) {
+    return await executeVerifyRecordCommand({ ...opts, command: "verify" });
+}

package/dist/commands/task/verify-rework.command.d.ts

@@ -1,12 +1,8 @@
 import type { CommandCtx, CommandSpec } from "../../cli/spec/spec.js";
 import type { CommandContext } from "../shared/task-backend.js";
-export type TaskVerifyReworkParsed = {
+import { type VerifyCommonParsed } from "./verify-command-shared.js";
+export type TaskVerifyReworkParsed = VerifyCommonParsed & {
     taskId: string;
-    by: string;
-    note: string;
-    details?: string;
-    file?: string;
-    quiet: boolean;
 };
 export declare const taskVerifyReworkSpec: CommandSpec<TaskVerifyReworkParsed>;
 export declare function makeRunTaskVerifyReworkHandler(getCtx: (cmd: string) => Promise<CommandContext>): (ctx: CommandCtx, p: TaskVerifyReworkParsed) => Promise<number>;

package/dist/commands/task/verify-rework.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"verify-rework.command.d.ts","sourceRoot":"","sources":["../../../src/commands/task/verify-rework.command.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAEtE,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAIhE,MAAM,MAAM,sBAAsB,GAAG;IACnC,MAAM,EAAE,MAAM,CAAC;IACf,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,OAAO,CAAC;CAChB,CAAC;AAEF,eAAO,MAAM,oBAAoB,EAAE,WAAW,CAAC,sBAAsB,CAkEpE,CAAC;AAEF,wBAAgB,8BAA8B,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IAC/E,KAAK,UAAU,EAAE,GAAG,sBAAsB,KAAG,OAAO,CAAC,MAAM,CAAC,CAa3E"}
+{"version":3,"file":"verify-rework.command.d.ts","sourceRoot":"","sources":["../../../src/commands/task/verify-rework.command.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AACtE,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAGhE,OAAO,EAKL,KAAK,kBAAkB,EACxB,MAAM,4BAA4B,CAAC;AAEpC,MAAM,MAAM,sBAAsB,GAAG,kBAAkB,GAAG;IACxD,MAAM,EAAE,MAAM,CAAC;CAChB,CAAC;AAEF,eAAO,MAAM,oBAAoB,EAAE,WAAW,CAAC,sBAAsB,CAwBpE,CAAC;AAEF,wBAAgB,8BAA8B,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IAC/E,KAAK,UAAU,EAAE,GAAG,sBAAsB,KAAG,OAAO,CAAC,MAAM,CAAC,CAa3E"}

package/dist/commands/task/verify-rework.command.js

@@ -1,38 +1,11 @@
-import { usageError } from "../../cli/spec/errors.js";
 import { cmdTaskVerifyRework } from "./verify-record.js";
+import { parseVerifyCommonOptions, validateVerifyDetailsFileExclusive, validateVerifyNonEmptyInput, verifyCommonOptions, } from "./verify-command-shared.js";
 export const taskVerifyReworkSpec = {
     id: ["task", "verify", "rework"],
     group: "Task",
     summary: "Record verification as needs rework (resets commit, sets status to DOING, updates Verification).",
     args: [{ name: "task-id", required: true, valueHint: "<task-id>" }],
-    options: [
-        { kind: "string", name: "by", valueHint: "<id>", required: true, description: "Verifier id." },
-        {
-            kind: "string",
-            name: "note",
-            valueHint: "<text>",
-            required: true,
-            description: "Short verification note.",
-        },
-        {
-            kind: "string",
-            name: "details",
-            valueHint: "<text>",
-            description: "Optional details text (mutually exclusive with --file).",
-        },
-        {
-            kind: "string",
-            name: "file",
-            valueHint: "<path>",
-            description: "Optional details file path (mutually exclusive with --details).",
-        },
-        {
-            kind: "boolean",
-            name: "quiet",
-            default: false,
-            description: "Suppress normal output (still prints errors).",
-        },
-    ],
+    options: verifyCommonOptions,
     examples: [
         {
             cmd: 'agentplane task verify rework 202602030608-F1Q8AB --by REVIEWER --note "Needs changes"',
@@ -40,30 +13,15 @@ export const taskVerifyReworkSpec = {
         },
     ],
     validateRaw: (raw) => {
-        const details = raw.opts.details;
-        const file = raw.opts.file;
-        if (typeof details === "string" && typeof file === "string") {
-            throw usageError({
-                spec: taskVerifyReworkSpec,
-                message: "Provide at most one of --details or --file.",
-            });
-        }
-        const by = raw.opts.by;
-        if (typeof by === "string" && by.trim().length === 0) {
-            throw usageError({ spec: taskVerifyReworkSpec, message: "Invalid value for --by: empty." });
-        }
-        const note = raw.opts.note;
-        if (typeof note === "string" && note.trim().length === 0) {
-            throw usageError({ spec: taskVerifyReworkSpec, message: "Invalid value for --note: empty." });
-        }
+        validateVerifyDetailsFileExclusive(raw, taskVerifyReworkSpec, {
+            message: "Provide at most one of --details or --file.",
+        });
+        validateVerifyNonEmptyInput(raw, taskVerifyReworkSpec, "by");
+        validateVerifyNonEmptyInput(raw, taskVerifyReworkSpec, "note");
     },
     parse: (raw) => ({
         taskId: String(raw.args["task-id"]),
-        by: String(raw.opts.by),
-        note: String(raw.opts.note),
-        details: typeof raw.opts.details === "string" ? raw.opts.details : undefined,
-        file: typeof raw.opts.file === "string" ? raw.opts.file : undefined,
-        quiet: raw.opts.quiet === true,
+        ...parseVerifyCommonOptions(raw),
     }),
 };
 export function makeRunTaskVerifyReworkHandler(getCtx) {

package/dist/commands/upgrade/apply.d.ts

@@ -7,6 +7,8 @@ export declare function ensureCleanTrackedTreeForUpgrade(gitRoot: string): Promi
 export declare function createUpgradeCommit(opts: {
     gitRoot: string;
     paths: string[];
+    tasksPath: string;
+    workflowDir: string;
     versionLabel: string;
     source: "local_assets" | "upgrade_bundle" | "repo_tarball";
     additions: number;

package/dist/commands/upgrade/apply.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"apply.d.ts","sourceRoot":"","sources":["../../../src/commands/upgrade/apply.ts"],"names":[],"mappings":"AAWA,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,YAAY,CAAC;AAUtD,wBAAsB,2BAA2B,CAAC,IAAI,EAAE;IACtD,eAAe,EAAE,MAAM,CAAC;IACxB,cAAc,EAAE,MAAM,EAAE,CAAC;CAC1B,GAAG,OAAO,CAAC,IAAI,CAAC,CAKhB;AAED,wBAAsB,gCAAgC,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CA+BrF;AAED,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,YAAY,EAAE,MAAM,CAAC;IACrB,MAAM,EAAE,cAAc,GAAG,gBAAgB,GAAG,cAAc,CAAC;IAC3D,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,sBAAsB,EAAE,MAAM,CAAC;CAChC,GAAG,OAAO,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CAAC,CAoEpD;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,MAAM,EAAE,OAAO,CAAC;IAChB,YAAY,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAClC,WAAW,EAAE,MAAM,CAAC;IACpB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,aAAa,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,MAAM,GAAG,IAAI,CAAC;CAC/C,GAAG,OAAO,CAAC,IAAI,CAAC,CA2ChB;AAED,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACnC,yBAAyB,EAAE,MAAM,GAAG,IAAI,CAAC;IACzC,2BAA2B,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3C,mBAAmB,EAAE,OAAO,CAAC;IAC7B,SAAS,EAAE,MAAM,CAAC;IAClB,eAAe,EAAE,MAAM,CAAC;IACxB,MAAM,EAAE,cAAc,GAAG,gBAAgB,GAAG,cAAc,CAAC;IAC3D,aAAa,EAAE,mBAAmB,EAAE,CAAC;IACrC,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;CACjB,GAAG,OAAO,CAAC,OAAO,CAAC,CA4CnB"}
+{"version":3,"file":"apply.d.ts","sourceRoot":"","sources":["../../../src/commands/upgrade/apply.ts"],"names":[],"mappings":"AAYA,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,YAAY,CAAC;AAUtD,wBAAsB,2BAA2B,CAAC,IAAI,EAAE;IACtD,eAAe,EAAE,MAAM,CAAC;IACxB,cAAc,EAAE,MAAM,EAAE,CAAC;CAC1B,GAAG,OAAO,CAAC,IAAI,CAAC,CAKhB;AAED,wBAAsB,gCAAgC,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CA+BrF;AAED,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,EAAE,MAAM,CAAC;IACrB,MAAM,EAAE,cAAc,GAAG,gBAAgB,GAAG,cAAc,CAAC;IAC3D,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,sBAAsB,EAAE,MAAM,CAAC;CAChC,GAAG,OAAO,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CAAC,CA8FpD;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,MAAM,EAAE,OAAO,CAAC;IAChB,YAAY,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAClC,WAAW,EAAE,MAAM,CAAC;IACpB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,aAAa,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,MAAM,GAAG,IAAI,CAAC;CAC/C,GAAG,OAAO,CAAC,IAAI,CAAC,CA2ChB;AAED,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACnC,yBAAyB,EAAE,MAAM,GAAG,IAAI,CAAC;IACzC,2BAA2B,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3C,mBAAmB,EAAE,OAAO,CAAC;IAC7B,SAAS,EAAE,MAAM,CAAC;IAClB,eAAe,EAAE,MAAM,CAAC;IACxB,MAAM,EAAE,cAAc,GAAG,gBAAgB,GAAG,cAAc,CAAC;IAC3D,aAAa,EAAE,mBAAmB,EAAE,CAAC;IACrC,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;CACjB,GAAG,OAAO,CAAC,OAAO,CAAC,CA4CnB"}

package/dist/commands/upgrade/apply.js

@@ -5,6 +5,7 @@ import { backupPath, fileExists } from "../../cli/fs-utils.js";
 import { exitCodeForError } from "../../cli/exit-codes.js";
 import { withDiagnosticContext } from "../../shared/diagnostics.js";
 import { CliError } from "../../shared/errors.js";
+import { protectedPathKindForFile } from "../../shared/protected-paths.js";
 import { execFileAsync, gitEnv } from "../shared/git.js";
 async function safeRemovePath(targetPath) {
     try {
@@ -74,10 +75,41 @@ export async function createUpgradeCommit(opts) {
         `Source: ${opts.source}\n` +
         `Managed-Changes: add=${opts.additions}, update=${opts.updates}, unchanged=${opts.unchanged}\n` +
         `Incidents-Appended: ${opts.incidentsAppendedCount}\n`;
+    const allow = {
+        allowTasks: false,
+        allowPolicy: false,
+        allowConfig: false,
+        allowHooks: false,
+        allowCI: false,
+    };
+    for (const filePath of uniquePaths) {
+        const kind = protectedPathKindForFile({
+            filePath,
+            tasksPath: opts.tasksPath,
+            workflowDir: opts.workflowDir,
+        });
+        if (kind === "tasks")
+            allow.allowTasks = true;
+        if (kind === "policy")
+            allow.allowPolicy = true;
+        if (kind === "config")
+            allow.allowConfig = true;
+        if (kind === "hooks")
+            allow.allowHooks = true;
+        if (kind === "ci")
+            allow.allowCI = true;
+    }
     try {
         await execFileAsync("git", ["commit", "-m", subject, "-m", body], {
             cwd: opts.gitRoot,
-            env: gitEnv(),
+            env: {
+                ...gitEnv(),
+                AGENTPLANE_ALLOW_TASKS: allow.allowTasks ? "1" : "0",
+                AGENTPLANE_ALLOW_POLICY: allow.allowPolicy ? "1" : "0",
+                AGENTPLANE_ALLOW_CONFIG: allow.allowConfig ? "1" : "0",
+                AGENTPLANE_ALLOW_HOOKS: allow.allowHooks ? "1" : "0",
+                AGENTPLANE_ALLOW_CI: allow.allowCI ? "1" : "0",
+            },
             maxBuffer: 10 * 1024 * 1024,
         });
     }

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

@@ -1 +1 @@
-{"version":3,"file":"upgrade.command.d.ts","sourceRoot":"","sources":["../../src/commands/upgrade.command.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AAEvE,OAAO,EAAoB,KAAK,YAAY,EAAE,MAAM,cAAc,CAAC;AAEnE,MAAM,MAAM,aAAa,GAAG,YAAY,CAAC;AAEzC,eAAO,MAAM,WAAW,EAAE,WAAW,CAAC,aAAa,CA8JlD,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,cAAc,CAAC,aAAa,CACsB,CAAC"}
+{"version":3,"file":"upgrade.command.d.ts","sourceRoot":"","sources":["../../src/commands/upgrade.command.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AAEvE,OAAO,EAAoB,KAAK,YAAY,EAAE,MAAM,cAAc,CAAC;AAEnE,MAAM,MAAM,aAAa,GAAG,YAAY,CAAC;AAEzC,eAAO,MAAM,WAAW,EAAE,WAAW,CAAC,aAAa,CAwLlD,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,cAAc,CAAC,aAAa,CACsB,CAAC"}

package/dist/commands/upgrade.command.js

@@ -72,6 +72,12 @@ export const upgradeSpec = {
             default: false,
             description: "Report changes without modifying files.",
         },
+        {
+            kind: "boolean",
+            name: "migrate-task-docs",
+            default: false,
+            description: "After applying the framework upgrade, migrate legacy task README docs in the same run.",
+        },
         {
             kind: "boolean",
             name: "no-backup",
@@ -115,6 +121,10 @@ export const upgradeSpec = {
             cmd: "agentplane upgrade --bundle ./agentplane-upgrade.tar.gz --checksum ./agentplane-upgrade.tar.gz.sha256",
             why: "Upgrade from local bundle files (no network).",
         },
+        {
+            cmd: "agentplane upgrade --yes --migrate-task-docs",
+            why: "Apply the framework upgrade and migrate legacy task README docs in one operator run.",
+        },
     ],
     parse: (raw) => {
         const noBackup = raw.opts["no-backup"] === true;
@@ -130,6 +140,7 @@ export const upgradeSpec = {
             checksumAsset: raw.opts["checksum-asset"],
             dryRun: raw.opts["dry-run"] === true,
             backup: !noBackup,
+            migrateTaskDocs: raw.opts["migrate-task-docs"] === true,
             yes: raw.opts.yes === true,
         };
     },
@@ -151,6 +162,20 @@ export const upgradeSpec = {
                 message: "Remote upgrade options (--tag/--source/--asset/--checksum-asset) require --remote.",
             });
         }
+        if (p.migrateTaskDocs && p.mode === "agent") {
+            throw usageError({
+                spec: upgradeSpec,
+                command: "upgrade",
+                message: "Option --migrate-task-docs cannot be used with --agent.",
+            });
+        }
+        if (p.migrateTaskDocs && p.dryRun) {
+            throw usageError({
+                spec: upgradeSpec,
+                command: "upgrade",
+                message: "Option --migrate-task-docs cannot be used with --dry-run.",
+            });
+        }
     },
 };
 export const runUpgrade = (ctx, flags) => cmdUpgradeParsed({ cwd: ctx.cwd, rootOverride: ctx.rootOverride, flags });

package/dist/commands/upgrade.d.ts

@@ -10,6 +10,7 @@ export type UpgradeFlags = {
     allowTarball: boolean;
     dryRun: boolean;
     backup: boolean;
+    migrateTaskDocs: boolean;
     yes: boolean;
 };
 export declare function cmdUpgradeParsed(opts: {

package/dist/commands/upgrade.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"upgrade.d.ts","sourceRoot":"","sources":["../../src/commands/upgrade.ts"],"names":[],"mappings":"AA4CA,MAAM,MAAM,YAAY,GAAG;IACzB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,IAAI,EAAE,OAAO,GAAG,MAAM,CAAC;IACvB,MAAM,EAAE,OAAO,CAAC;IAChB,YAAY,EAAE,OAAO,CAAC;IACtB,MAAM,EAAE,OAAO,CAAC;IAChB,MAAM,EAAE,OAAO,CAAC;IAChB,GAAG,EAAE,OAAO,CAAC;CACd,CAAC;AAuMF,wBAAsB,gBAAgB,CAAC,IAAI,EAAE;IAC3C,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,YAAY,CAAC;CACrB,GAAG,OAAO,CAAC,MAAM,CAAC,CAsiBlB;AAED,OAAO,EACL,kCAAkC,EAClC,qBAAqB,EACrB,iCAAiC,GAClC,MAAM,qBAAqB,CAAC"}
+{"version":3,"file":"upgrade.d.ts","sourceRoot":"","sources":["../../src/commands/upgrade.ts"],"names":[],"mappings":"AA+CA,MAAM,MAAM,YAAY,GAAG;IACzB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,IAAI,EAAE,OAAO,GAAG,MAAM,CAAC;IACvB,MAAM,EAAE,OAAO,CAAC;IAChB,YAAY,EAAE,OAAO,CAAC;IACtB,MAAM,EAAE,OAAO,CAAC;IAChB,MAAM,EAAE,OAAO,CAAC;IAChB,eAAe,EAAE,OAAO,CAAC;IACzB,GAAG,EAAE,OAAO,CAAC;CACd,CAAC;AAuMF,wBAAsB,gBAAgB,CAAC,IAAI,EAAE;IAC3C,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,YAAY,CAAC;CACrB,GAAG,OAAO,CAAC,MAAM,CAAC,CA2kBlB;AAED,OAAO,EACL,kCAAkC,EAClC,qBAAqB,EACrB,iCAAiC,GAClC,MAAM,qBAAqB,CAAC"}