agentplane 0.1.8 → 0.2.0

package/dist/commands/task/plan-reject.command.d.ts

@@ -0,0 +1,10 @@
+import type { CommandCtx, CommandSpec } from "../../cli/spec/spec.js";
+import type { CommandContext } from "../shared/task-backend.js";
+export type TaskPlanRejectParsed = {
+    taskId: string;
+    by: string;
+    note: string;
+};
+export declare const taskPlanRejectSpec: CommandSpec<TaskPlanRejectParsed>;
+export declare function makeRunTaskPlanRejectHandler(getCtx: (cmd: string) => Promise<CommandContext>): (ctx: CommandCtx, p: TaskPlanRejectParsed) => Promise<number>;
+//# sourceMappingURL=plan-reject.command.d.ts.map

package/dist/commands/task/plan-reject.command.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"plan-reject.command.d.ts","sourceRoot":"","sources":["../../../src/commands/task/plan-reject.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,oBAAoB,GAAG;IACjC,MAAM,EAAE,MAAM,CAAC;IACf,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;CACd,CAAC;AAEF,eAAO,MAAM,kBAAkB,EAAE,WAAW,CAAC,oBAAoB,CA4ChE,CAAC;AAEF,wBAAgB,4BAA4B,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IAC7E,KAAK,UAAU,EAAE,GAAG,oBAAoB,KAAG,OAAO,CAAC,MAAM,CAAC,CAUzE"}

package/dist/commands/task/plan-reject.command.js

@@ -0,0 +1,59 @@
+import { usageError } from "../../cli/spec/errors.js";
+import { cmdTaskPlanReject } from "./plan.js";
+export const taskPlanRejectSpec = {
+    id: ["task", "plan", "reject"],
+    group: "Task",
+    summary: "Reject the current task plan (requires a note).",
+    args: [{ name: "task-id", required: true, valueHint: "<task-id>" }],
+    options: [
+        {
+            kind: "string",
+            name: "by",
+            valueHint: "<id>",
+            required: true,
+            description: "Reviewer id.",
+        },
+        {
+            kind: "string",
+            name: "note",
+            valueHint: "<text>",
+            required: true,
+            description: "Required note explaining what needs to change.",
+        },
+    ],
+    examples: [
+        {
+            cmd: 'agentplane task plan reject 202602030608-F1Q8AB --by USER --note "Need clarification"',
+            why: "Reject a plan with a note.",
+        },
+    ],
+    validateRaw: (raw) => {
+        const by = raw.opts.by;
+        if (typeof by === "string" && by.trim().length === 0) {
+            throw usageError({ spec: taskPlanRejectSpec, message: "Invalid value for --by: empty." });
+        }
+        const note = raw.opts.note;
+        if (typeof note === "string" && note.trim().length === 0) {
+            throw usageError({ spec: taskPlanRejectSpec, message: "Invalid value for --note: empty." });
+        }
+    },
+    parse: (raw) => {
+        return {
+            taskId: String(raw.args["task-id"]),
+            by: String(raw.opts.by),
+            note: String(raw.opts.note),
+        };
+    },
+};
+export function makeRunTaskPlanRejectHandler(getCtx) {
+    return async (ctx, p) => {
+        return await cmdTaskPlanReject({
+            ctx: await getCtx("task plan reject"),
+            cwd: ctx.cwd,
+            rootOverride: ctx.rootOverride,
+            taskId: p.taskId,
+            by: p.by,
+            note: p.note,
+        });
+    };
+}

package/dist/commands/task/plan-set.command.d.ts

@@ -0,0 +1,11 @@
+import type { CommandCtx, CommandSpec } from "../../cli/spec/spec.js";
+import type { CommandContext } from "../shared/task-backend.js";
+export type TaskPlanSetParsed = {
+    taskId: string;
+    text?: string;
+    file?: string;
+    updatedBy?: string;
+};
+export declare const taskPlanSetSpec: CommandSpec<TaskPlanSetParsed>;
+export declare function makeRunTaskPlanSetHandler(getCtx: (cmd: string) => Promise<CommandContext>): (ctx: CommandCtx, p: TaskPlanSetParsed) => Promise<number>;
+//# sourceMappingURL=plan-set.command.d.ts.map

package/dist/commands/task/plan-set.command.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"plan-set.command.d.ts","sourceRoot":"","sources":["../../../src/commands/task/plan-set.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,iBAAiB,GAAG;IAC9B,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,WAAW,CAAC,iBAAiB,CA4D1D,CAAC;AAEF,wBAAgB,yBAAyB,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IAC1E,KAAK,UAAU,EAAE,GAAG,iBAAiB,KAAG,OAAO,CAAC,MAAM,CAAC,CAWtE"}

package/dist/commands/task/plan-set.command.js

@@ -0,0 +1,76 @@
+import { usageError } from "../../cli/spec/errors.js";
+import { cmdTaskPlanSet } from "./plan.js";
+export const taskPlanSetSpec = {
+    id: ["task", "plan", "set"],
+    group: "Task",
+    summary: "Set a task plan (writes the Plan section and resets plan approval to pending).",
+    args: [{ name: "task-id", required: true, valueHint: "<task-id>" }],
+    options: [
+        {
+            kind: "string",
+            name: "text",
+            valueHint: "<text>",
+            description: "Plan text to write into the task README Plan section.",
+        },
+        {
+            kind: "string",
+            name: "file",
+            valueHint: "<path>",
+            description: "Read plan text from a file path (relative to cwd).",
+        },
+        {
+            kind: "string",
+            name: "updated-by",
+            valueHint: "<id>",
+            description: "Optional. Sets doc_updated_by when writing the plan.",
+        },
+    ],
+    examples: [
+        {
+            cmd: String.raw `agentplane task plan set 202602030608-F1Q8AB --text "1) Do X\n2) Verify Y" --updated-by ORCHESTRATOR`,
+            why: "Write plan from text and set doc_updated_by.",
+        },
+        {
+            cmd: "agentplane task plan set 202602030608-F1Q8AB --file plan.md",
+            why: "Write plan from a file.",
+        },
+    ],
+    validateRaw: (raw) => {
+        const hasText = typeof raw.opts.text === "string";
+        const hasFile = typeof raw.opts.file === "string";
+        if (hasText === hasFile) {
+            throw usageError({
+                spec: taskPlanSetSpec,
+                message: "Provide exactly one of --text or --file.",
+            });
+        }
+        const updatedBy = raw.opts["updated-by"];
+        if (typeof updatedBy === "string" && updatedBy.trim().length === 0) {
+            throw usageError({
+                spec: taskPlanSetSpec,
+                message: "Invalid value for --updated-by: empty.",
+            });
+        }
+    },
+    parse: (raw) => {
+        return {
+            taskId: String(raw.args["task-id"]),
+            text: typeof raw.opts.text === "string" ? raw.opts.text : undefined,
+            file: typeof raw.opts.file === "string" ? raw.opts.file : undefined,
+            updatedBy: typeof raw.opts["updated-by"] === "string" ? raw.opts["updated-by"] : undefined,
+        };
+    },
+};
+export function makeRunTaskPlanSetHandler(getCtx) {
+    return async (ctx, p) => {
+        return await cmdTaskPlanSet({
+            ctx: await getCtx("task plan set"),
+            cwd: ctx.cwd,
+            rootOverride: ctx.rootOverride,
+            taskId: p.taskId,
+            text: p.text,
+            file: p.file,
+            updatedBy: p.updatedBy,
+        });
+    };
+}

package/dist/commands/task/plan.d.ts

@@ -1,14 +1,27 @@
-export declare const TASK_PLAN_USAGE = "Usage: agentplane task plan <set|approve|reject> <task-id> [flags]";
-export declare const TASK_PLAN_USAGE_EXAMPLE = "agentplane task plan set 202602030608-F1Q8AB --text \"...\"";
-export declare const TASK_PLAN_SET_USAGE = "Usage: agentplane task plan set <task-id> (--text <text> | --file <path>) [--updated-by <id>]";
-export declare const TASK_PLAN_SET_USAGE_EXAMPLE: string;
-export declare const TASK_PLAN_APPROVE_USAGE = "Usage: agentplane task plan approve <task-id> --by <id> [--note <text>]";
-export declare const TASK_PLAN_APPROVE_USAGE_EXAMPLE = "agentplane task plan approve 202602030608-F1Q8AB --by USER --note \"OK\"";
-export declare const TASK_PLAN_REJECT_USAGE = "Usage: agentplane task plan reject <task-id> --by <id> --note <text>";
-export declare const TASK_PLAN_REJECT_USAGE_EXAMPLE = "agentplane task plan reject 202602030608-F1Q8AB --by USER --note \"Need clarification\"";
-export declare function cmdTaskPlan(opts: {
+import { type CommandContext } from "../shared/task-backend.js";
+export declare function cmdTaskPlanSet(opts: {
+    ctx?: CommandContext;
     cwd: string;
     rootOverride?: string;
-    args: string[];
+    taskId: string;
+    text?: string;
+    file?: string;
+    updatedBy?: string;
+}): Promise<number>;
+export declare function cmdTaskPlanApprove(opts: {
+    ctx?: CommandContext;
+    cwd: string;
+    rootOverride?: string;
+    taskId: string;
+    by: string;
+    note?: string;
+}): Promise<number>;
+export declare function cmdTaskPlanReject(opts: {
+    ctx?: CommandContext;
+    cwd: string;
+    rootOverride?: string;
+    taskId: string;
+    by: string;
+    note: string;
 }): Promise<number>;
 //# sourceMappingURL=plan.d.ts.map

package/dist/commands/task/plan.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"plan.d.ts","sourceRoot":"","sources":["../../../src/commands/task/plan.ts"],"names":[],"mappings":"AAYA,eAAO,MAAM,eAAe,uEAAuE,CAAC;AACpG,eAAO,MAAM,uBAAuB,gEAA8D,CAAC;AAEnG,eAAO,MAAM,mBAAmB,kGACiE,CAAC;AAClG,eAAO,MAAM,2BAA2B,QAA6G,CAAC;AAEtJ,eAAO,MAAM,uBAAuB,4EACuC,CAAC;AAC5E,eAAO,MAAM,+BAA+B,6EAC8B,CAAC;AAE3E,eAAO,MAAM,sBAAsB,yEACqC,CAAC;AACzE,eAAO,MAAM,8BAA8B,4FAC8C,CAAC;AAoG1F,wBAAsB,WAAW,CAAC,IAAI,EAAE;IACtC,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,IAAI,EAAE,MAAM,EAAE,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAwIlB"}
+{"version":3,"file":"plan.d.ts","sourceRoot":"","sources":["../../../src/commands/task/plan.ts"],"names":[],"mappings":"AASA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AA+CnC,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,GAAG,OAAO,CAAC,MAAM,CAAC,CAmElB;AAED,wBAAsB,kBAAkB,CAAC,IAAI,EAAE;IAC7C,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,CAAC,EAAE,MAAM,CAAC;CACf,GAAG,OAAO,CAAC,MAAM,CAAC,CAkFlB;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,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;CACd,GAAG,OAAO,CAAC,MAAM,CAAC,CAuDlB"}

package/dist/commands/task/plan.js

@@ -2,216 +2,221 @@ 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, missingValueMessage, usageMessage } from "../../cli/output.js";
+import { backendNotSupportedMessage } from "../../cli/output.js";
 import { CliError } from "../../shared/errors.js";
-import { loadBackendTask } from "../shared/task-backend.js";
-import { nowIso } from "./shared.js";
-export const TASK_PLAN_USAGE = "Usage: agentplane task plan <set|approve|reject> <task-id> [flags]";
-export const TASK_PLAN_USAGE_EXAMPLE = 'agentplane task plan set 202602030608-F1Q8AB --text "..."';
-export const TASK_PLAN_SET_USAGE = "Usage: agentplane task plan set <task-id> (--text <text> | --file <path>) [--updated-by <id>]";
-export const TASK_PLAN_SET_USAGE_EXAMPLE = String.raw `agentplane task plan set 202602030608-F1Q8AB --text "1) ...\n2) ..." --updated-by ORCHESTRATOR`;
-export const TASK_PLAN_APPROVE_USAGE = "Usage: agentplane task plan approve <task-id> --by <id> [--note <text>]";
-export const TASK_PLAN_APPROVE_USAGE_EXAMPLE = 'agentplane task plan approve 202602030608-F1Q8AB --by USER --note "OK"';
-export const TASK_PLAN_REJECT_USAGE = "Usage: agentplane task plan reject <task-id> --by <id> --note <text>";
-export const TASK_PLAN_REJECT_USAGE_EXAMPLE = 'agentplane task plan reject 202602030608-F1Q8AB --by USER --note "Need clarification"';
-function extractDocSection(doc, sectionName) {
-    const lines = doc.replaceAll("\r\n", "\n").split("\n");
-    let capturing = false;
-    const out = [];
-    for (const line of lines) {
-        const match = /^##\s+(.*)$/.exec(line.trim());
-        if (match) {
-            if (capturing)
-                break;
-            capturing = (match[1] ?? "").trim() === sectionName;
-            continue;
-        }
-        if (capturing)
-            out.push(line);
-    }
-    if (!capturing)
-        return null;
-    return out.join("\n").trimEnd();
-}
-function parsePlanSetFlags(args) {
-    const out = {};
-    for (let i = 0; i < args.length; i++) {
-        const arg = args[i];
-        if (!arg)
-            continue;
-        if (!arg.startsWith("--")) {
-            throw new CliError({ exitCode: 2, code: "E_USAGE", message: `Unexpected argument: ${arg}` });
-        }
-        const next = args[i + 1];
-        if (!next) {
-            throw new CliError({ exitCode: 2, code: "E_USAGE", message: missingValueMessage(arg) });
-        }
-        switch (arg) {
-            case "--text": {
-                out.text = next;
-                break;
-            }
-            case "--file": {
-                out.file = next;
-                break;
-            }
-            case "--updated-by": {
-                out.updatedBy = next;
-                break;
-            }
-            default: {
-                throw new CliError({ exitCode: 2, code: "E_USAGE", message: `Unknown flag: ${arg}` });
-            }
-        }
-        i++;
-    }
-    return out;
-}
-function parsePlanDecisionFlags(args) {
-    const out = {};
-    for (let i = 0; i < args.length; i++) {
-        const arg = args[i];
-        if (!arg)
-            continue;
-        if (!arg.startsWith("--")) {
-            throw new CliError({ exitCode: 2, code: "E_USAGE", message: `Unexpected argument: ${arg}` });
-        }
-        const next = args[i + 1];
-        if (!next) {
-            throw new CliError({ exitCode: 2, code: "E_USAGE", message: missingValueMessage(arg) });
-        }
-        switch (arg) {
-            case "--by": {
-                out.by = next;
-                break;
-            }
-            case "--note": {
-                out.note = next;
-                break;
-            }
-            default: {
-                throw new CliError({ exitCode: 2, code: "E_USAGE", message: `Unknown flag: ${arg}` });
-            }
-        }
-        i++;
-    }
-    return out;
-}
-export async function cmdTaskPlan(opts) {
-    const [subcommand, taskId, ...restArgs] = opts.args;
-    if (!subcommand || !taskId) {
+import { loadCommandContext, loadTaskFromContext, } from "../shared/task-backend.js";
+import { backendIsLocalFileBackend, getTaskStore } from "../shared/task-store.js";
+import { extractDocSection, isVerifyStepsFilled, nowIso, requiresVerify, toStringArray, } from "./shared.js";
+async function loadPlanTask(opts) {
+    const ctx = opts.ctx ??
+        (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
+    const backend = ctx.taskBackend;
+    const useStore = backendIsLocalFileBackend(ctx);
+    const store = useStore ? getTaskStore(ctx) : null;
+    const task = useStore
+        ? await store.get(opts.taskId)
+        : await loadTaskFromContext({ ctx, taskId: opts.taskId });
+    if (!backend.getTaskDoc || !backend.writeTask) {
         throw new CliError({
             exitCode: 2,
             code: "E_USAGE",
-            message: usageMessage(TASK_PLAN_USAGE, TASK_PLAN_USAGE_EXAMPLE),
+            message: backendNotSupportedMessage("task docs"),
         });
     }
+    return { ctx, backend: backend, task, useStore, store };
+}
+export async function cmdTaskPlanSet(opts) {
     try {
-        const { backend, config, resolved, task } = await loadBackendTask({
+        const { ctx, backend, task, useStore, store } = await loadPlanTask({
+            ctx: opts.ctx,
             cwd: opts.cwd,
             rootOverride: opts.rootOverride,
-            taskId,
+            taskId: opts.taskId,
         });
-        if (!backend.getTaskDoc || !backend.setTaskDoc || !backend.writeTask) {
+        const config = ctx.config;
+        const resolved = ctx.resolvedProject;
+        const hasText = typeof opts.text === "string";
+        const hasFile = typeof opts.file === "string";
+        if (hasText === hasFile) {
             throw new CliError({
                 exitCode: 2,
                 code: "E_USAGE",
-                message: backendNotSupportedMessage("task docs"),
+                message: "Provide exactly one of --text or --file.",
             });
         }
-        if (subcommand === "set") {
-            const flags = parsePlanSetFlags(restArgs);
-            const hasText = flags.text !== undefined;
-            const hasFile = flags.file !== undefined;
-            if (hasText === hasFile) {
+        let updatedBy;
+        if (typeof opts.updatedBy === "string") {
+            const trimmed = opts.updatedBy.trim();
+            if (!trimmed) {
                 throw new CliError({
                     exitCode: 2,
                     code: "E_USAGE",
-                    message: usageMessage(TASK_PLAN_SET_USAGE, TASK_PLAN_SET_USAGE_EXAMPLE),
+                    message: "Invalid value for --updated-by: empty.",
                 });
             }
-            let updatedBy;
-            if (flags.updatedBy !== undefined) {
-                const trimmed = flags.updatedBy.trim();
-                if (!trimmed) {
+            updatedBy = trimmed;
+        }
+        let text = opts.text ?? "";
+        if (hasFile) {
+            try {
+                text = await readFile(path.resolve(opts.cwd, opts.file ?? ""), "utf8");
+            }
+            catch (err) {
+                throw mapCoreError(err, { command: "task plan set", filePath: opts.file ?? "" });
+            }
+        }
+        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 nextDoc = ensureDocSections(setMarkdownSection(baseDoc, "Plan", text), config.tasks.doc.required_sections);
+        const nextTask = {
+            ...task,
+            doc: nextDoc,
+            plan_approval: { state: "pending", updated_at: null, updated_by: null, note: null },
+            ...(updatedBy ? { doc_updated_by: updatedBy } : {}),
+        };
+        await (useStore ? store.update(opts.taskId, () => nextTask) : backend.writeTask(nextTask));
+        const readmePath = path.join(resolved.gitRoot, config.paths.workflow_dir, task.id, "README.md");
+        process.stdout.write(`${readmePath}\n`);
+        return 0;
+    }
+    catch (err) {
+        if (err instanceof CliError)
+            throw err;
+        throw mapBackendError(err, { command: "task plan set", root: opts.rootOverride ?? null });
+    }
+}
+export async function cmdTaskPlanApprove(opts) {
+    try {
+        const { ctx, backend, task, useStore, store } = await loadPlanTask({
+            ctx: opts.ctx,
+            cwd: opts.cwd,
+            rootOverride: opts.rootOverride,
+            taskId: opts.taskId,
+        });
+        const config = ctx.config;
+        const by = String(opts.by ?? "").trim();
+        if (!by) {
+            throw new CliError({
+                exitCode: 2,
+                code: "E_USAGE",
+                message: "Invalid value for --by: empty.",
+            });
+        }
+        const note = typeof opts.note === "string" ? opts.note.trim() : "";
+        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 plan = extractDocSection(baseDoc, "Plan");
+        if (!plan || plan.trim().length === 0) {
+            throw new CliError({
+                exitCode: 3,
+                code: "E_VALIDATION",
+                message: `${task.id}: cannot approve plan: ## Plan section is missing or empty`,
+            });
+        }
+        const enforceVerifySteps = config.tasks.verify.enforce_on_plan_approve !== false;
+        if (enforceVerifySteps) {
+            const tags = toStringArray(task.tags);
+            const requireStepsTags = config.tasks.verify.require_steps_for_tags ?? config.tasks.verify.required_tags;
+            const spikeTag = (config.tasks.verify.spike_tag ?? "spike").trim().toLowerCase();
+            const verifyRequired = requiresVerify(tags, requireStepsTags);
+            const isSpike = tags.some((tag) => tag.trim().toLowerCase() === spikeTag);
+            if (verifyRequired || isSpike) {
+                const verifySteps = extractDocSection(baseDoc, "Verify Steps");
+                if (!isVerifyStepsFilled(verifySteps)) {
                     throw new CliError({
-                        exitCode: 2,
-                        code: "E_USAGE",
-                        message: "--updated-by must be non-empty",
+                        exitCode: 3,
+                        code: "E_VALIDATION",
+                        message: `${task.id}: cannot approve plan: ## Verify Steps section is missing/empty/unfilled ` +
+                            "(fill it before approving plan)",
                     });
                 }
-                updatedBy = trimmed;
             }
-            let text = flags.text ?? "";
-            if (hasFile) {
-                try {
-                    text = await readFile(path.resolve(opts.cwd, flags.file ?? ""), "utf8");
-                }
-                catch (err) {
-                    throw mapCoreError(err, { command: "task plan set", filePath: flags.file ?? "" });
+            if (isSpike) {
+                const notes = extractDocSection(baseDoc, "Notes");
+                if (!notes || notes.trim().length === 0) {
+                    throw new CliError({
+                        exitCode: 3,
+                        code: "E_VALIDATION",
+                        message: `${task.id}: cannot approve plan for spike: ## Notes section is missing or empty ` +
+                            "(include Findings/Decision/Next Steps)",
+                    });
                 }
             }
-            const existingDoc = (typeof task.doc === "string" ? task.doc : "") || (await backend.getTaskDoc(task.id));
-            const baseDoc = ensureDocSections(existingDoc ?? "", config.tasks.doc.required_sections);
-            const nextDoc = ensureDocSections(setMarkdownSection(baseDoc, "Plan", text), config.tasks.doc.required_sections);
-            await backend.writeTask({
-                ...task,
-                doc: nextDoc,
-                plan_approval: { state: "pending", updated_at: null, updated_by: null, note: null },
-                ...(updatedBy ? { doc_updated_by: updatedBy } : {}),
+        }
+        const nextTask = {
+            ...task,
+            plan_approval: {
+                state: "approved",
+                updated_at: nowIso(),
+                updated_by: by,
+                note: note || null,
+            },
+        };
+        await (useStore ? store.update(opts.taskId, () => nextTask) : backend.writeTask(nextTask));
+        return 0;
+    }
+    catch (err) {
+        if (err instanceof CliError)
+            throw err;
+        throw mapBackendError(err, { command: "task plan approve", root: opts.rootOverride ?? null });
+    }
+}
+export async function cmdTaskPlanReject(opts) {
+    try {
+        const { ctx, backend, task, useStore, store } = await loadPlanTask({
+            ctx: opts.ctx,
+            cwd: opts.cwd,
+            rootOverride: opts.rootOverride,
+            taskId: opts.taskId,
+        });
+        const config = ctx.config;
+        const by = String(opts.by ?? "").trim();
+        if (!by) {
+            throw new CliError({
+                exitCode: 2,
+                code: "E_USAGE",
+                message: "Invalid value for --by: empty.",
             });
-            const readmePath = path.join(resolved.gitRoot, config.paths.workflow_dir, task.id, "README.md");
-            process.stdout.write(`${readmePath}\n`);
-            return 0;
         }
-        if (subcommand === "approve" || subcommand === "reject") {
-            const flags = parsePlanDecisionFlags(restArgs);
-            const by = (flags.by ?? "").trim();
-            if (!by) {
-                const usage = subcommand === "approve" ? TASK_PLAN_APPROVE_USAGE : TASK_PLAN_REJECT_USAGE;
-                const example = subcommand === "approve"
-                    ? TASK_PLAN_APPROVE_USAGE_EXAMPLE
-                    : TASK_PLAN_REJECT_USAGE_EXAMPLE;
-                throw new CliError({ exitCode: 2, code: "E_USAGE", message: usageMessage(usage, example) });
-            }
-            const note = typeof flags.note === "string" ? flags.note.trim() : "";
-            if (subcommand === "reject" && note.length === 0) {
-                throw new CliError({
-                    exitCode: 2,
-                    code: "E_USAGE",
-                    message: usageMessage(TASK_PLAN_REJECT_USAGE, TASK_PLAN_REJECT_USAGE_EXAMPLE),
-                });
-            }
-            const existingDoc = (typeof task.doc === "string" ? task.doc : "") || (await backend.getTaskDoc(task.id));
-            const baseDoc = ensureDocSections(existingDoc ?? "", config.tasks.doc.required_sections);
-            const plan = extractDocSection(baseDoc, "Plan");
-            if (!plan || plan.trim().length === 0) {
-                throw new CliError({
-                    exitCode: 3,
-                    code: "E_VALIDATION",
-                    message: `${task.id}: cannot ${subcommand} plan: ## Plan section is missing or empty`,
-                });
-            }
-            await backend.writeTask({
-                ...task,
-                plan_approval: {
-                    state: subcommand === "approve" ? "approved" : "rejected",
-                    updated_at: nowIso(),
-                    updated_by: by,
-                    note: note || null,
-                },
+        const note = String(opts.note ?? "").trim();
+        if (!note) {
+            throw new CliError({
+                exitCode: 2,
+                code: "E_USAGE",
+                message: "Invalid value for --note: empty.",
             });
-            return 0;
         }
-        throw new CliError({
-            exitCode: 2,
-            code: "E_USAGE",
-            message: usageMessage(TASK_PLAN_USAGE, TASK_PLAN_USAGE_EXAMPLE),
-        });
+        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 plan = extractDocSection(baseDoc, "Plan");
+        if (!plan || plan.trim().length === 0) {
+            throw new CliError({
+                exitCode: 3,
+                code: "E_VALIDATION",
+                message: `${task.id}: cannot reject plan: ## Plan section is missing or empty`,
+            });
+        }
+        const nextTask = {
+            ...task,
+            plan_approval: {
+                state: "rejected",
+                updated_at: nowIso(),
+                updated_by: by,
+                note: note || null,
+            },
+        };
+        await (useStore ? store.update(opts.taskId, () => nextTask) : backend.writeTask(nextTask));
+        return 0;
     }
     catch (err) {
         if (err instanceof CliError)
             throw err;
-        throw mapBackendError(err, { command: "task plan", root: opts.rootOverride ?? null });
+        throw mapBackendError(err, { command: "task plan reject", root: opts.rootOverride ?? null });
     }
 }

package/dist/commands/task/ready.d.ts

@@ -1,4 +1,6 @@
+import { type CommandContext } from "../shared/task-backend.js";
 export declare function cmdReady(opts: {
+    ctx?: CommandContext;
     cwd: string;
     rootOverride?: string;
     taskId: string;

package/dist/commands/task/ready.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ready.d.ts","sourceRoot":"","sources":["../../../src/commands/task/ready.ts"],"names":[],"mappings":"AAMA,wBAAsB,QAAQ,CAAC,IAAI,EAAE;IACnC,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAqDlB"}
+{"version":3,"file":"ready.d.ts","sourceRoot":"","sources":["../../../src/commands/task/ready.ts"],"names":[],"mappings":"AAEA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAIpF,wBAAsB,QAAQ,CAAC,IAAI,EAAE;IACnC,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,CAoDlB"}

package/dist/commands/task/ready.js

@@ -1,14 +1,12 @@
-import { loadTaskBackend } from "../../backends/task-backend.js";
 import { mapBackendError } from "../../cli/error-map.js";
 import { successMessage, unknownEntityMessage, warnMessage } from "../../cli/output.js";
+import { loadCommandContext } from "../shared/task-backend.js";
 import { buildDependencyState } from "./shared.js";
 export async function cmdReady(opts) {
     try {
-        const { backend } = await loadTaskBackend({
-            cwd: opts.cwd,
-            rootOverride: opts.rootOverride ?? null,
-        });
-        const tasks = await backend.listTasks();
+        const ctx = opts.ctx ??
+            (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
+        const tasks = await ctx.taskBackend.listTasks();
         const depState = buildDependencyState(tasks);
         const task = tasks.find((item) => item.id === opts.taskId);
         const warnings = [];