agentplane 0.3.5 → 0.3.7

package/dist/commands/task/plan.js

@@ -6,7 +6,7 @@ import { backendNotSupportedMessage } from "../../cli/output.js";
 import { CliError } from "../../shared/errors.js";
 import { loadCommandContext, loadTaskFromContext, } from "../shared/task-backend.js";
 import { backendIsLocalFileBackend, getTaskStore } from "../shared/task-store.js";
-import { ensureAgentFilledRequiredDocSections, extractDocSection, extractTaskObservationSection, isVerifyStepsFilled, nowIso, normalizeTaskDocVersion, taskObservationSectionName, requiresVerifyStepsByPrimary, toStringArray, } from "./shared.js";
+import { decodeEscapedTaskTextNewlines, ensureAgentFilledRequiredDocSections, extractDocSection, extractTaskObservationSection, isVerifyStepsFilled, nowIso, normalizeTaskDocVersion, taskObservationSectionName, requiresVerifyStepsByPrimary, toStringArray, } from "./shared.js";
 async function loadPlanTask(opts) {
     const ctx = opts.ctx ??
         (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
@@ -28,6 +28,68 @@ async function loadPlanTask(opts) {
 function normalizeForComparison(text) {
     return text.replaceAll("\r\n", "\n").trim();
 }
+function buildPlanDocUpdate(opts) {
+    const baseDoc = ensureDocSections(opts.currentDocRaw ?? "", opts.requiredSections);
+    const currentPlan = extractDocSection(baseDoc, "Plan") ?? "";
+    const planChanged = normalizeForComparison(currentPlan) !== normalizeForComparison(opts.text);
+    const nextDoc = ensureDocSections(setMarkdownSection(baseDoc, "Plan", opts.text), opts.requiredSections);
+    return {
+        currentPlan,
+        nextPlan: extractDocSection(nextDoc, "Plan") ?? "",
+        planChanged,
+        docChanged: nextDoc !== baseDoc,
+        nextDoc,
+    };
+}
+function assertPlanSectionPresent(taskId, doc, action) {
+    const plan = extractDocSection(doc, "Plan");
+    if (!plan || plan.trim().length === 0) {
+        throw new CliError({
+            exitCode: 3,
+            code: "E_VALIDATION",
+            message: `${taskId}: cannot ${action} plan: ## Plan section is missing or empty`,
+        });
+    }
+}
+function assertPlanCanBeApproved(opts) {
+    assertPlanSectionPresent(opts.task.id, opts.doc, "approve");
+    ensureAgentFilledRequiredDocSections({
+        task: opts.task,
+        config: opts.config,
+        doc: opts.doc,
+        action: "approve plan",
+    });
+    const enforceVerifySteps = opts.config.tasks.verify.enforce_on_plan_approve !== false;
+    if (!enforceVerifySteps)
+        return;
+    const tags = toStringArray(opts.task.tags);
+    const spikeTag = (opts.config.tasks.verify.spike_tag ?? "spike").trim().toLowerCase();
+    const verifyRequired = requiresVerifyStepsByPrimary(tags, opts.config);
+    const isSpike = tags.some((tag) => tag.trim().toLowerCase() === spikeTag);
+    if (verifyRequired || isSpike) {
+        const verifySteps = extractDocSection(opts.doc, "Verify Steps");
+        if (!isVerifyStepsFilled(verifySteps)) {
+            throw new CliError({
+                exitCode: 3,
+                code: "E_VALIDATION",
+                message: `${opts.task.id}: cannot approve plan: ## Verify Steps section is missing/empty/unfilled ` +
+                    "(fill it before approving plan)",
+            });
+        }
+    }
+    if (!isSpike)
+        return;
+    const observationSection = taskObservationSectionName(normalizeTaskDocVersion(opts.task.doc_version));
+    const observation = extractTaskObservationSection(opts.doc, normalizeTaskDocVersion(opts.task.doc_version));
+    if (!observation || observation.trim().length === 0) {
+        throw new CliError({
+            exitCode: 3,
+            code: "E_VALIDATION",
+            message: `${opts.task.id}: cannot approve plan for spike: ## ${observationSection} section is missing or empty ` +
+                "(include Findings/Decision/Next Steps)",
+        });
+    }
+}
 export async function cmdTaskPlanSet(opts) {
     try {
         const { ctx, backend, task, useStore, store } = await loadPlanTask({
@@ -60,6 +122,9 @@ export async function cmdTaskPlanSet(opts) {
             updatedBy = trimmed;
         }
         let text = opts.text ?? "";
+        if (hasText) {
+            text = decodeEscapedTaskTextNewlines(text);
+        }
         if (hasFile) {
             try {
                 text = await readFile(path.resolve(opts.cwd, opts.file ?? ""), "utf8");
@@ -68,28 +133,80 @@ export async function cmdTaskPlanSet(opts) {
                 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 currentPlan = extractDocSection(baseDoc, "Plan") ?? "";
-        const planChanged = normalizeForComparison(currentPlan) !== normalizeForComparison(text);
-        const nextDoc = ensureDocSections(setMarkdownSection(baseDoc, "Plan", text), config.tasks.doc.required_sections);
-        const docChanged = nextDoc !== baseDoc;
         const readmePath = path.join(resolved.gitRoot, config.paths.workflow_dir, task.id, "README.md");
-        if (!planChanged && !docChanged && !updatedBy) {
-            process.stdout.write(`${readmePath}\n`);
-            return 0;
+        if (useStore) {
+            let expectedCurrentPlan;
+            await store.patch(opts.taskId, (current) => {
+                const { currentPlan, nextPlan, planChanged, docChanged } = buildPlanDocUpdate({
+                    currentDocRaw: String(current.doc ?? ""),
+                    text,
+                    requiredSections: config.tasks.doc.required_sections,
+                });
+                if (!planChanged && !docChanged && !updatedBy)
+                    return null;
+                if (!docChanged) {
+                    return {
+                        ...(planChanged
+                            ? {
+                                task: {
+                                    plan_approval: {
+                                        state: "pending",
+                                        updated_at: null,
+                                        updated_by: null,
+                                        note: null,
+                                    },
+                                },
+                            }
+                            : {}),
+                        ...(updatedBy ? { docMeta: { touch: true, updatedBy } } : {}),
+                    };
+                }
+                expectedCurrentPlan ??= currentPlan;
+                return {
+                    doc: {
+                        kind: "set-section",
+                        section: "Plan",
+                        text: nextPlan,
+                        requiredSections: config.tasks.doc.required_sections,
+                        expectedCurrentText: expectedCurrentPlan,
+                    },
+                    ...(planChanged
+                        ? {
+                            task: {
+                                plan_approval: {
+                                    state: "pending",
+                                    updated_at: null,
+                                    updated_by: null,
+                                    note: null,
+                                },
+                            },
+                        }
+                        : {}),
+                    ...(updatedBy ? { docMeta: { updatedBy } } : {}),
+                };
+            });
+        }
+        else {
+            const existingDoc = (typeof task.doc === "string" ? task.doc : "") || (await backend.getTaskDoc(task.id));
+            const { planChanged, docChanged, nextDoc } = buildPlanDocUpdate({
+                currentDocRaw: existingDoc,
+                text,
+                requiredSections: config.tasks.doc.required_sections,
+            });
+            if (!planChanged && !docChanged && !updatedBy) {
+                process.stdout.write(`${readmePath}\n`);
+                return 0;
+            }
+            const nextTask = {
+                ...task,
+                doc: nextDoc,
+                ...(planChanged
+                    ? { plan_approval: { state: "pending", updated_at: null, updated_by: null, note: null } }
+                    : {}),
+                ...(updatedBy ? { doc_updated_by: updatedBy } : {}),
+            };
+            await backend.writeTask(nextTask);
         }
-        const nextTask = {
-            ...task,
-            doc: nextDoc,
-            ...(planChanged
-                ? { 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));
         process.stdout.write(`${readmePath}\n`);
         return 0;
     }
@@ -117,64 +234,36 @@ export async function cmdTaskPlanApprove(opts) {
             });
         }
         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`,
-            });
-        }
-        ensureAgentFilledRequiredDocSections({
-            task,
-            config,
-            doc: baseDoc,
-            action: "approve plan",
-        });
-        const enforceVerifySteps = config.tasks.verify.enforce_on_plan_approve !== false;
-        if (enforceVerifySteps) {
-            const tags = toStringArray(task.tags);
-            const spikeTag = (config.tasks.verify.spike_tag ?? "spike").trim().toLowerCase();
-            const verifyRequired = requiresVerifyStepsByPrimary(tags, config);
-            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: 3,
-                        code: "E_VALIDATION",
-                        message: `${task.id}: cannot approve plan: ## Verify Steps section is missing/empty/unfilled ` +
-                            "(fill it before approving plan)",
-                    });
-                }
-            }
-            if (isSpike) {
-                const observationSection = taskObservationSectionName(normalizeTaskDocVersion(task.doc_version));
-                const observation = extractTaskObservationSection(baseDoc, normalizeTaskDocVersion(task.doc_version));
-                if (!observation || observation.trim().length === 0) {
-                    throw new CliError({
-                        exitCode: 3,
-                        code: "E_VALIDATION",
-                        message: `${task.id}: cannot approve plan for spike: ## ${observationSection} section is missing or empty ` +
-                            "(include Findings/Decision/Next Steps)",
-                    });
-                }
-            }
-        }
-        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));
+        const approvedAt = nowIso();
+        await (useStore
+            ? store.patch(opts.taskId, (current) => {
+                const currentDoc = ensureDocSections(String(current.doc ?? ""), config.tasks.doc.required_sections);
+                assertPlanCanBeApproved({ task: current, config, doc: currentDoc });
+                return {
+                    task: {
+                        plan_approval: {
+                            state: "approved",
+                            updated_at: approvedAt,
+                            updated_by: by,
+                            note: note || null,
+                        },
+                    },
+                };
+            })
+            : (async () => {
+                const existingDoc = (typeof task.doc === "string" ? task.doc : "") || (await backend.getTaskDoc(task.id));
+                const baseDoc = ensureDocSections(existingDoc ?? "", config.tasks.doc.required_sections);
+                assertPlanCanBeApproved({ task, config, doc: baseDoc });
+                await backend.writeTask({
+                    ...task,
+                    plan_approval: {
+                        state: "approved",
+                        updated_at: approvedAt,
+                        updated_by: by,
+                        note: note || null,
+                    },
+                });
+            })());
         return 0;
     }
     catch (err) {
@@ -208,28 +297,36 @@ export async function cmdTaskPlanReject(opts) {
                 message: "Invalid value for --note: empty.",
             });
         }
-        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));
+        const rejectedAt = nowIso();
+        await (useStore
+            ? store.patch(opts.taskId, (current) => {
+                const currentDoc = ensureDocSections(String(current.doc ?? ""), config.tasks.doc.required_sections);
+                assertPlanSectionPresent(current.id, currentDoc, "reject");
+                return {
+                    task: {
+                        plan_approval: {
+                            state: "rejected",
+                            updated_at: rejectedAt,
+                            updated_by: by,
+                            note: note || null,
+                        },
+                    },
+                };
+            })
+            : (async () => {
+                const existingDoc = (typeof task.doc === "string" ? task.doc : "") || (await backend.getTaskDoc(task.id));
+                const baseDoc = ensureDocSections(existingDoc ?? "", config.tasks.doc.required_sections);
+                assertPlanSectionPresent(task.id, baseDoc, "reject");
+                await backend.writeTask({
+                    ...task,
+                    plan_approval: {
+                        state: "rejected",
+                        updated_at: rejectedAt,
+                        updated_by: by,
+                        note: note || null,
+                    },
+                });
+            })());
         return 0;
     }
     catch (err) {

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

@@ -1 +1 @@
-{"version":3,"file":"set-status.command.d.ts","sourceRoot":"","sources":["../../../src/commands/task/set-status.command.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAOtE,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAIhE,MAAM,MAAM,mBAAmB,GAAG;IAChC,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,OAAO,CAAC;IACf,GAAG,EAAE,OAAO,CAAC;IACb,iBAAiB,EAAE,OAAO,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,EAAE,MAAM,EAAE,CAAC;IACtB,eAAe,EAAE,OAAO,CAAC;IACzB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,kBAAkB,EAAE,OAAO,CAAC;IAC5B,mBAAmB,EAAE,OAAO,CAAC;IAC7B,KAAK,EAAE,OAAO,CAAC;CAChB,CAAC;AAEF,eAAO,MAAM,iBAAiB,EAAE,WAAW,CAAC,mBAAmB,CAyK9D,CAAC;AAEF,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IAC5E,KAAK,UAAU,EAAE,GAAG,mBAAmB,KAAG,OAAO,CAAC,MAAM,CAAC,CAsBxE"}
+{"version":3,"file":"set-status.command.d.ts","sourceRoot":"","sources":["../../../src/commands/task/set-status.command.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAOtE,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAIhE,MAAM,MAAM,mBAAmB,GAAG;IAChC,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,OAAO,CAAC;IACf,GAAG,EAAE,OAAO,CAAC;IACb,iBAAiB,EAAE,OAAO,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,EAAE,MAAM,EAAE,CAAC;IACtB,eAAe,EAAE,OAAO,CAAC;IACzB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,kBAAkB,EAAE,OAAO,CAAC;IAC5B,mBAAmB,EAAE,OAAO,CAAC;IAC7B,KAAK,EAAE,OAAO,CAAC;CAChB,CAAC;AAEF,eAAO,MAAM,iBAAiB,EAAE,WAAW,CAAC,mBAAmB,CA0K9D,CAAC;AAEF,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,IAC5E,KAAK,UAAU,EAAE,GAAG,mBAAmB,KAAG,OAAO,CAAC,MAAM,CAAC,CAsBxE"}

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

@@ -71,7 +71,7 @@ export const taskSetStatusSpec = {
             kind: "boolean",
             name: "commit-allow-tasks",
             default: true,
-            description: "Allow committing under .agentplane/tasks when commit-from-comment is used.",
+            description: "Allow the tasks export snapshot plus artifacts under the active task subtree when commit-from-comment is used.",
         },
         {
             kind: "boolean",

package/dist/commands/task/set-status.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"set-status.d.ts","sourceRoot":"","sources":["../../../src/commands/task/set-status.ts"],"names":[],"mappings":"AAQA,OAAO,EAIL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AAiBnC,wBAAsB,gBAAgB,CAAC,IAAI,EAAE;IAC3C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,OAAO,CAAC;IACf,GAAG,EAAE,OAAO,CAAC;IACb,iBAAiB,EAAE,OAAO,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,EAAE,MAAM,EAAE,CAAC;IACtB,eAAe,EAAE,OAAO,CAAC;IACzB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,kBAAkB,EAAE,OAAO,CAAC;IAC5B,mBAAmB,EAAE,OAAO,CAAC;IAC7B,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAwJlB"}
+{"version":3,"file":"set-status.d.ts","sourceRoot":"","sources":["../../../src/commands/task/set-status.ts"],"names":[],"mappings":"AAQA,OAAO,EAIL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AA2BnC,wBAAsB,gBAAgB,CAAC,IAAI,EAAE;IAC3C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,OAAO,CAAC;IACf,GAAG,EAAE,OAAO,CAAC;IACb,iBAAiB,EAAE,OAAO,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,EAAE,MAAM,EAAE,CAAC;IACtB,eAAe,EAAE,OAAO,CAAC;IACzB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,kBAAkB,EAAE,OAAO,CAAC;IAC5B,mBAAmB,EAAE,OAAO,CAAC;IAC7B,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CA2OlB"}

package/dist/commands/task/set-status.js

@@ -5,8 +5,8 @@ import { CliError } from "../../shared/errors.js";
 import { commitFromComment } from "../guard/index.js";
 import { ensureActionApproved } from "../shared/approval-requirements.js";
 import { loadCommandContext, loadTaskFromContext, resolveDocUpdatedBy, } from "../shared/task-backend.js";
-import { backendIsLocalFileBackend, getTaskStore } from "../shared/task-store.js";
-import { appendTaskEvent, defaultCommitEmojiForStatus, ensureCommentCommitAllowed, ensureStatusTransitionAllowed, normalizeTaskDocVersion, normalizeTaskStatus, nowIso, readCommitInfo, resolveTaskDependencyState, resolvePrimaryTag, toStringArray, } from "./shared.js";
+import { appendTaskCommentIntent, appendTaskEventIntent, backendIsLocalFileBackend, getTaskStore, mutateTaskStore, setTaskFieldsIntent, touchTaskDocMetaIntent, } from "../shared/task-store.js";
+import { appendTaskEvent, dependencyWarningMessages, defaultCommitEmojiForStatus, ensureCommentCommitAllowed, resolveCommentCommitWarning, ensureStatusTransitionAllowed, normalizeTaskDocVersion, normalizeTaskStatus, nowIso, readCommitInfo, resolveTaskDependencyState, resolvePrimaryTag, toStringArray, } from "./shared.js";
 export async function cmdTaskSetStatus(opts) {
     const nextStatus = normalizeTaskStatus(opts.status);
     if (nextStatus === "DONE" && !opts.force) {
@@ -42,38 +42,40 @@ export async function cmdTaskSetStatus(opts) {
             ? await store.get(opts.taskId)
             : await loadTaskFromContext({ ctx, taskId: opts.taskId });
         const currentStatus = String(task.status || "TODO").toUpperCase();
-        ensureStatusTransitionAllowed({
-            currentStatus,
-            nextStatus,
-            force: opts.force,
-        });
-        if (!opts.force && (nextStatus === "DOING" || nextStatus === "DONE")) {
-            const dep = await resolveTaskDependencyState(task, ctx.taskBackend);
-            if (dep.missing.length > 0 || dep.incomplete.length > 0) {
-                if (!opts.quiet) {
-                    if (dep.missing.length > 0) {
-                        process.stderr.write(`${warnMessage(`missing deps: ${dep.missing.join(", ")}`)}\n`);
-                    }
-                    if (dep.incomplete.length > 0) {
-                        process.stderr.write(`${warnMessage(`incomplete deps: ${dep.incomplete.join(", ")}`)}\n`);
+        if (!useStore) {
+            ensureStatusTransitionAllowed({
+                currentStatus,
+                nextStatus,
+                force: opts.force,
+            });
+            if (!opts.force && (nextStatus === "DOING" || nextStatus === "DONE")) {
+                const dep = await resolveTaskDependencyState(task, ctx.taskBackend);
+                if (dep.missing.length > 0 || dep.incomplete.length > 0) {
+                    if (!opts.quiet) {
+                        if (dep.missing.length > 0) {
+                            process.stderr.write(`${warnMessage(`missing deps: ${dep.missing.join(", ")}`)}\n`);
+                        }
+                        if (dep.incomplete.length > 0) {
+                            process.stderr.write(`${warnMessage(`incomplete deps: ${dep.incomplete.join(", ")}`)}\n`);
+                        }
                     }
+                    throw new CliError({
+                        exitCode: 2,
+                        code: "E_USAGE",
+                        message: `Task is not ready: ${task.id} (use --force to override)`,
+                    });
                 }
-                throw new CliError({
-                    exitCode: 2,
-                    code: "E_USAGE",
-                    message: `Task is not ready: ${task.id} (use --force to override)`,
-                });
             }
+            ensureCommentCommitAllowed({
+                enabled: opts.commitFromComment,
+                config,
+                action: "task set-status",
+                confirmed: opts.confirmStatusCommit,
+                quiet: opts.quiet,
+                statusFrom: currentStatus,
+                statusTo: nextStatus,
+            });
         }
-        ensureCommentCommitAllowed({
-            enabled: opts.commitFromComment,
-            config,
-            action: "task set-status",
-            confirmed: opts.confirmStatusCommit,
-            quiet: opts.quiet,
-            statusFrom: currentStatus,
-            statusTo: nextStatus,
-        });
         const existingComments = Array.isArray(task.comments)
             ? task.comments.filter((item) => !!item && typeof item.author === "string" && typeof item.body === "string")
             : [];
@@ -87,6 +89,13 @@ export async function cmdTaskSetStatus(opts) {
         }
         const at = nowIso();
         const eventAuthor = resolveDocUpdatedBy(task, opts.author);
+        const commitInfo = opts.commit ? await readCommitInfo(resolved.gitRoot, opts.commit) : null;
+        const nextCommit = opts.commit
+            ? { hash: commitInfo.hash, message: commitInfo.message }
+            : undefined;
+        let currentStatusForCommit = currentStatus;
+        let primaryTagForCommit = resolvePrimaryTag(toStringArray(task.tags), ctx).primary;
+        let deferredWarnings = [];
         const next = {
             ...task,
             status: nextStatus,
@@ -104,10 +113,81 @@ export async function cmdTaskSetStatus(opts) {
             doc_updated_by: eventAuthor,
         };
         if (opts.commit) {
-            const commitInfo = await readCommitInfo(resolved.gitRoot, opts.commit);
-            next.commit = { hash: commitInfo.hash, message: commitInfo.message };
+            next.commit = nextCommit;
+        }
+        try {
+            await (useStore
+                ? mutateTaskStore(store, opts.taskId, async (current) => {
+                    deferredWarnings = [];
+                    const currentStatus = String(current.status || "TODO").toUpperCase();
+                    currentStatusForCommit = currentStatus;
+                    primaryTagForCommit = resolvePrimaryTag(toStringArray(current.tags), ctx).primary;
+                    ensureStatusTransitionAllowed({
+                        currentStatus,
+                        nextStatus,
+                        force: opts.force,
+                    });
+                    if (!opts.force && (nextStatus === "DOING" || nextStatus === "DONE")) {
+                        const dep = await resolveTaskDependencyState(current, ctx.taskBackend);
+                        deferredWarnings = [...deferredWarnings, ...dependencyWarningMessages(dep)];
+                        if (dep.missing.length > 0 || dep.incomplete.length > 0) {
+                            throw new CliError({
+                                exitCode: 2,
+                                code: "E_USAGE",
+                                message: `Task is not ready: ${current.id} (use --force to override)`,
+                            });
+                        }
+                    }
+                    const commitWarning = resolveCommentCommitWarning({
+                        enabled: opts.commitFromComment,
+                        config,
+                        action: "task set-status",
+                        confirmed: opts.confirmStatusCommit,
+                        quiet: opts.quiet,
+                        statusFrom: currentStatus,
+                        statusTo: nextStatus,
+                    });
+                    if (commitWarning)
+                        deferredWarnings.push(commitWarning);
+                    const currentEventAuthor = resolveDocUpdatedBy(current, opts.author);
+                    const intents = [
+                        setTaskFieldsIntent({
+                            status: nextStatus,
+                            ...(nextCommit ? { commit: nextCommit } : {}),
+                        }),
+                        appendTaskEventIntent({
+                            type: "status",
+                            at,
+                            author: currentEventAuthor,
+                            from: currentStatus,
+                            to: nextStatus,
+                            note: commentBody,
+                        }),
+                        touchTaskDocMetaIntent({
+                            updatedBy: currentEventAuthor,
+                            version: normalizeTaskDocVersion(current.doc_version),
+                        }),
+                    ];
+                    if (commentBody) {
+                        intents.splice(1, 0, appendTaskCommentIntent({ author: opts.author, body: commentBody }));
+                    }
+                    return intents;
+                })
+                : ctx.taskBackend.writeTask(next));
+        }
+        catch (err) {
+            if (err instanceof CliError && !opts.quiet) {
+                for (const warning of new Set(deferredWarnings)) {
+                    process.stderr.write(`${warnMessage(warning)}\n`);
+                }
+            }
+            throw err;
+        }
+        if (!opts.quiet) {
+            for (const warning of new Set(deferredWarnings)) {
+                process.stderr.write(`${warnMessage(warning)}\n`);
+            }
         }
-        await (useStore ? store.update(opts.taskId, () => next) : ctx.taskBackend.writeTask(next));
         // tasks.json is export-only; generated via `agentplane task export`.
         if (opts.commitFromComment) {
             if (!opts.body) {
@@ -122,9 +202,9 @@ export async function cmdTaskSetStatus(opts) {
                 cwd: opts.cwd,
                 rootOverride: opts.rootOverride,
                 taskId: opts.taskId,
-                primaryTag: resolvePrimaryTag(toStringArray(task.tags), ctx).primary,
+                primaryTag: primaryTagForCommit,
                 author: opts.author,
-                statusFrom: currentStatus,
+                statusFrom: currentStatusForCommit,
                 statusTo: nextStatus,
                 commentBody: opts.body,
                 formattedComment: formatCommentBodyForCommit(opts.body, config),

package/dist/commands/task/shared/dependencies.d.ts

@@ -10,6 +10,7 @@ export declare function ensureTaskDependsOnGraphIsAcyclic(opts: {
     dependsOn: string[];
 }): Promise<void>;
 export declare function resolveTaskDependencyState(task: TaskData, backend: Pick<TaskBackend, "getTask" | "getTasks">): Promise<DependencyState>;
+export declare function dependencyWarningMessages(dep: DependencyState): string[];
 export declare function buildDependencyState(tasks: TaskData[]): Map<string, DependencyState>;
 export declare function formatTaskLine(task: TaskData, depState?: DependencyState): string;
 //# sourceMappingURL=dependencies.d.ts.map

package/dist/commands/task/shared/dependencies.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"dependencies.d.ts","sourceRoot":"","sources":["../../../../src/commands/task/shared/dependencies.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AAK/E,MAAM,MAAM,eAAe,GAAG;IAC5B,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,UAAU,EAAE,MAAM,EAAE,CAAC;CACtB,CAAC;AAkCF,wBAAsB,iCAAiC,CAAC,IAAI,EAAE;IAC5D,OAAO,EAAE,IAAI,CAAC,WAAW,EAAE,WAAW,CAAC,CAAC;IACxC,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,EAAE,CAAC;CACrB,GAAG,OAAO,CAAC,IAAI,CAAC,CA2BhB;AAED,wBAAsB,0BAA0B,CAC9C,IAAI,EAAE,QAAQ,EACd,OAAO,EAAE,IAAI,CAAC,WAAW,EAAE,SAAS,GAAG,UAAU,CAAC,GACjD,OAAO,CAAC,eAAe,CAAC,CAuB1B;AAED,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,QAAQ,EAAE,GAAG,GAAG,CAAC,MAAM,EAAE,eAAe,CAAC,CAqBpF;AAgBD,wBAAgB,cAAc,CAAC,IAAI,EAAE,QAAQ,EAAE,QAAQ,CAAC,EAAE,eAAe,GAAG,MAAM,CAejF"}
+{"version":3,"file":"dependencies.d.ts","sourceRoot":"","sources":["../../../../src/commands/task/shared/dependencies.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AAK/E,MAAM,MAAM,eAAe,GAAG;IAC5B,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,UAAU,EAAE,MAAM,EAAE,CAAC;CACtB,CAAC;AAkCF,wBAAsB,iCAAiC,CAAC,IAAI,EAAE;IAC5D,OAAO,EAAE,IAAI,CAAC,WAAW,EAAE,WAAW,CAAC,CAAC;IACxC,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,EAAE,CAAC;CACrB,GAAG,OAAO,CAAC,IAAI,CAAC,CA2BhB;AAED,wBAAsB,0BAA0B,CAC9C,IAAI,EAAE,QAAQ,EACd,OAAO,EAAE,IAAI,CAAC,WAAW,EAAE,SAAS,GAAG,UAAU,CAAC,GACjD,OAAO,CAAC,eAAe,CAAC,CAuB1B;AAED,wBAAgB,yBAAyB,CAAC,GAAG,EAAE,eAAe,GAAG,MAAM,EAAE,CASxE;AAED,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,QAAQ,EAAE,GAAG,GAAG,CAAC,MAAM,EAAE,eAAe,CAAC,CAqBpF;AAgBD,wBAAgB,cAAc,CAAC,IAAI,EAAE,QAAQ,EAAE,QAAQ,CAAC,EAAE,eAAe,GAAG,MAAM,CAejF"}

package/dist/commands/task/shared/dependencies.js

@@ -83,6 +83,16 @@ export async function resolveTaskDependencyState(task, backend) {
     }
     return { dependsOn, missing, incomplete };
 }
+export function dependencyWarningMessages(dep) {
+    const warnings = [];
+    if (dep.missing.length > 0) {
+        warnings.push(`missing deps: ${dep.missing.join(", ")}`);
+    }
+    if (dep.incomplete.length > 0) {
+        warnings.push(`incomplete deps: ${dep.incomplete.join(", ")}`);
+    }
+    return warnings;
+}
 export function buildDependencyState(tasks) {
     const byId = new Map(tasks.map((task) => [task.id, task]));
     const state = new Map();

package/dist/commands/task/shared/docs.d.ts

@@ -5,6 +5,7 @@ export declare const VERIFY_STEPS_PLACEHOLDER = "<!-- TODO: REPLACE WITH TASK-SP
 export declare const VERIFICATION_RESULTS_BEGIN = "<!-- BEGIN VERIFICATION RESULTS -->";
 export declare const VERIFICATION_RESULTS_END = "<!-- END VERIFICATION RESULTS -->";
 export type TaskDocVersion = 2 | 3;
+export declare function decodeEscapedTaskTextNewlines(text: string): string;
 export declare function extractDocSection(doc: string, sectionName: string): string | null;
 export declare function isVerifyStepsFilled(sectionText: string | null): boolean;
 export declare function normalizeTaskDocVersion(value: unknown, fallback?: TaskDocVersion): TaskDocVersion;

package/dist/commands/task/shared/docs.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"docs.d.ts","sourceRoot":"","sources":["../../../../src/commands/task/shared/docs.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qBAAqB,CAAC;AAI5D,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AAElE,wBAAgB,MAAM,IAAI,MAAM,CAE/B;AAED,eAAO,MAAM,wBAAwB,+DACyB,CAAC;AAC/D,eAAO,MAAM,0BAA0B,wCAAwC,CAAC;AAChF,eAAO,MAAM,wBAAwB,sCAAsC,CAAC;AAC5E,MAAM,MAAM,cAAc,GAAG,CAAC,GAAG,CAAC,CAAC;AAEnC,wBAAgB,iBAAiB,CAAC,GAAG,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAiBjF;AAED,wBAAgB,mBAAmB,CAAC,WAAW,EAAE,MAAM,GAAG,IAAI,GAAG,OAAO,CAKvE;AAED,wBAAgB,uBAAuB,CACrC,KAAK,EAAE,OAAO,EACd,QAAQ,GAAE,cAAkB,GAC3B,cAAc,CAEhB;AAED,wBAAgB,kCAAkC,CAChD,WAAW,EAAE,MAAM,GAAG,IAAI,EAC1B,OAAO,EAAE,cAAc,GACtB,MAAM,CAyCR;AAED,wBAAgB,0BAA0B,CAAC,OAAO,EAAE,cAAc,GAAG,OAAO,GAAG,UAAU,CAExF;AAED,wBAAgB,6BAA6B,CAAC,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,cAAc,GAAG,MAAM,GAAG,IAAI,CAIjG;AAMD,wBAAgB,kBAAkB,CAAC,WAAW,EAAE,MAAM,GAAG,IAAI,GAAG,OAAO,CAKtE;AAED,wBAAgB,oCAAoC,CAAC,IAAI,EAAE;IACzD,IAAI,EAAE,IAAI,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC;IAC3B,MAAM,EAAE,gBAAgB,CAAC;IACzB,GAAG,EAAE,MAAM,CAAC;IACZ,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,IAAI,CA6BP"}
+{"version":3,"file":"docs.d.ts","sourceRoot":"","sources":["../../../../src/commands/task/shared/docs.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qBAAqB,CAAC;AAI5D,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AAElE,wBAAgB,MAAM,IAAI,MAAM,CAE/B;AAED,eAAO,MAAM,wBAAwB,+DACyB,CAAC;AAC/D,eAAO,MAAM,0BAA0B,wCAAwC,CAAC;AAChF,eAAO,MAAM,wBAAwB,sCAAsC,CAAC;AAC5E,MAAM,MAAM,cAAc,GAAG,CAAC,GAAG,CAAC,CAAC;AAEnC,wBAAgB,6BAA6B,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAMlE;AAED,wBAAgB,iBAAiB,CAAC,GAAG,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAiBjF;AAED,wBAAgB,mBAAmB,CAAC,WAAW,EAAE,MAAM,GAAG,IAAI,GAAG,OAAO,CAKvE;AAED,wBAAgB,uBAAuB,CACrC,KAAK,EAAE,OAAO,EACd,QAAQ,GAAE,cAAkB,GAC3B,cAAc,CAEhB;AAED,wBAAgB,kCAAkC,CAChD,WAAW,EAAE,MAAM,GAAG,IAAI,EAC1B,OAAO,EAAE,cAAc,GACtB,MAAM,CAyCR;AAED,wBAAgB,0BAA0B,CAAC,OAAO,EAAE,cAAc,GAAG,OAAO,GAAG,UAAU,CAExF;AAED,wBAAgB,6BAA6B,CAAC,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,cAAc,GAAG,MAAM,GAAG,IAAI,CAIjG;AAMD,wBAAgB,kBAAkB,CAAC,WAAW,EAAE,MAAM,GAAG,IAAI,GAAG,OAAO,CAKtE;AAED,wBAAgB,oCAAoC,CAAC,IAAI,EAAE;IACzD,IAAI,EAAE,IAAI,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC;IAC3B,MAAM,EAAE,gBAAgB,CAAC;IACzB,GAAG,EAAE,MAAM,CAAC;IACZ,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,IAAI,CA6BP"}

package/dist/commands/task/shared/docs.js

@@ -6,6 +6,13 @@ export function nowIso() {
 export const VERIFY_STEPS_PLACEHOLDER = "<!-- TODO: REPLACE WITH TASK-SPECIFIC ACCEPTANCE STEPS -->";
 export const VERIFICATION_RESULTS_BEGIN = "<!-- BEGIN VERIFICATION RESULTS -->";
 export const VERIFICATION_RESULTS_END = "<!-- END VERIFICATION RESULTS -->";
+export function decodeEscapedTaskTextNewlines(text) {
+    const normalized = text.replaceAll("\r\n", "\n");
+    if (!normalized.includes(String.raw `\n`) && !normalized.includes(String.raw `\r\n`)) {
+        return normalized;
+    }
+    return normalized.replaceAll(String.raw `\r\n`, "\n").replaceAll(String.raw `\n`, "\n");
+}
 export function extractDocSection(doc, sectionName) {
     const lines = doc.replaceAll("\r\n", "\n").split("\n");
     let capturing = false;
@@ -33,7 +40,7 @@ export function isVerifyStepsFilled(sectionText) {
         return false;
     return true;
 }
-export function normalizeTaskDocVersion(value, fallback = 2) {
+export function normalizeTaskDocVersion(value, fallback = 3) {
     return value === 3 ? 3 : value === 2 ? 2 : fallback;
 }
 export function normalizeVerificationSectionLayout(sectionText, version) {