agentplane 0.3.6 → 0.3.8

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

@@ -1,8 +1,11 @@
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";
-import { warnMessage } from "../../../cli/output.js";
+import { infoMessage, warnMessage } from "../../../cli/output.js";
+import { formatCommentBodyForCommit } from "../../../shared/comment-format.js";
+import { readDirectWorkLock } from "../../../shared/direct-work-lock.js";
 import { CliError } from "../../../shared/errors.js";
 import { parseGitLogHashSubject } from "../../../shared/git-log.js";
+import { commitFromComment } from "../../guard/index.js";
 import { requiresVerificationByPrimary, toStringArray } from "./tags.js";
 const execFileAsync = promisify(execFile);
 export function appendTaskEvent(task, event) {
@@ -71,8 +74,21 @@ export function ensureStatusTransitionAllowed(opts) {
     });
 }
 export function ensureCommentCommitAllowed(opts) {
-    if (!opts.enabled)
+    const warning = resolveCommentCommitWarning(opts);
+    if (warning) {
+        process.stderr.write(`${warnMessage(warning)}\n`);
+    }
+}
+export function emitTransitionWarnings(warnings, quiet) {
+    if (quiet)
         return;
+    for (const warning of new Set(warnings.filter((item) => item.trim().length > 0))) {
+        process.stderr.write(`${warnMessage(warning)}\n`);
+    }
+}
+export function resolveCommentCommitWarning(opts) {
+    if (!opts.enabled)
+        return null;
     if (opts.config.commit_automation === "finish_only") {
         throw new CliError({
             exitCode: 2,
@@ -81,7 +97,7 @@ export function ensureCommentCommitAllowed(opts) {
                 "(allowed only in finish).",
         });
     }
-    enforceStatusCommitPolicy({
+    return resolveStatusCommitPolicyWarning({
         policy: opts.config.status_commit_policy,
         action: opts.action,
         confirmed: opts.confirmed,
@@ -107,12 +123,69 @@ export function requireStructuredComment(body, prefix, minChars) {
         });
     }
 }
+export function prepareTaskTransitionComment(opts) {
+    if (typeof opts.body !== "string") {
+        return { formattedComment: null, commentBody: undefined };
+    }
+    const formattedComment = opts.enabled ? formatCommentBodyForCommit(opts.body, opts.config) : null;
+    return {
+        formattedComment,
+        commentBody: formattedComment ?? opts.body,
+    };
+}
+export async function resolveTaskTransitionExecutorAgent(opts) {
+    const author = opts.author?.trim() ?? "";
+    if (opts.ctx.config.workflow_mode !== "direct") {
+        return author || undefined;
+    }
+    const lock = await readDirectWorkLock(opts.ctx.resolvedProject.agentplaneDir);
+    const lockAgent = lock?.task_id === opts.taskId ? (lock.agent?.trim() ?? "") : "";
+    return lockAgent || author || undefined;
+}
+export async function runTaskTransitionCommentCommit(opts) {
+    if (opts.progressMessage && !opts.quiet) {
+        process.stdout.write(`${infoMessage(opts.progressMessage)}\n`);
+    }
+    const executorAgent = opts.resolveExecutorAgent
+        ? await resolveTaskTransitionExecutorAgent({
+            ctx: opts.ctx,
+            taskId: opts.taskId,
+            author: opts.author,
+        })
+        : undefined;
+    return await commitFromComment({
+        ctx: opts.ctx,
+        cwd: opts.cwd,
+        rootOverride: opts.rootOverride,
+        taskId: opts.taskId,
+        primaryTag: opts.primaryTag,
+        executorAgent,
+        author: opts.author,
+        statusFrom: opts.statusFrom,
+        statusTo: opts.statusTo,
+        commentBody: opts.commentBody,
+        formattedComment: opts.formattedComment,
+        emoji: opts.emoji,
+        allow: opts.allow,
+        autoAllow: opts.autoAllow,
+        allowTasks: opts.allowTasks,
+        requireClean: opts.requireClean,
+        quiet: opts.quiet,
+        config: opts.ctx.config,
+    });
+}
 export async function readHeadCommit(cwd) {
     const { stdout } = await execFileAsync("git", ["log", "-1", "--pretty=%H%x00%s"], { cwd });
     const { hash, subject } = parseGitLogHashSubject(stdout);
     return { hash, message: subject };
 }
 export function enforceStatusCommitPolicy(opts) {
+    const warning = resolveStatusCommitPolicyWarning(opts);
+    if (warning) {
+        process.stderr.write(`${warnMessage(warning)}\n`);
+    }
+}
+export function resolveStatusCommitPolicyWarning(opts) {
     if (!isMajorStatusCommitTransition(opts.statusFrom, opts.statusTo)) {
         throw new CliError({
             exitCode: 2,
@@ -122,12 +195,12 @@ export function enforceStatusCommitPolicy(opts) {
         });
     }
     if (opts.policy === "off")
-        return;
+        return null;
     if (opts.policy === "warn") {
-        if (!opts.quiet && !opts.confirmed) {
-            process.stderr.write(`${warnMessage(`${opts.action}: status/comment-driven commit requested; policy=warn (pass --confirm-status-commit to acknowledge)`)}\n`);
-        }
-        return;
+        return opts.quiet || opts.confirmed
+            ? null
+            : `${opts.action}: status/comment-driven commit requested; policy=warn ` +
+                "(pass --confirm-status-commit to acknowledge)";
     }
     if (opts.policy === "confirm" && !opts.confirmed) {
         throw new CliError({
@@ -137,6 +210,7 @@ export function enforceStatusCommitPolicy(opts) {
                 "(pass --confirm-status-commit to proceed)",
         });
     }
+    return null;
 }
 const MAJOR_STATUS_COMMIT_TRANSITIONS = new Set([
     "READY->DOING",

package/dist/commands/task/shared/workflow-transition-service.d.ts

@@ -0,0 +1,75 @@
+import { type AgentplaneConfig } from "@agentplaneorg/core";
+import type { TaskBackend, TaskData } from "../../../backends/task-backend.js";
+import { type TaskStoreIntent, type TaskStoreTaskPatch } from "../../shared/task-store.js";
+import { type DependencyState } from "./dependencies.js";
+type TaskComment = NonNullable<TaskData["comments"]>[number];
+type BuildTaskStatusTransitionOptions = {
+    task: TaskData;
+    at: string;
+    toStatus: TaskData["status"];
+    eventAuthor: string;
+    updatedBy: string;
+    note?: string;
+    comment?: TaskComment;
+    commit?: TaskData["commit"] | null;
+    extraFields?: TaskStoreTaskPatch;
+};
+type BuildTaskVerificationTransitionOptions = {
+    task: TaskData;
+    at: string;
+    by: string;
+    note: string;
+    state: "ok" | "needs_rework";
+    verificationSection: string;
+    nextDoc: string;
+    requiredSections: string[];
+};
+export type ExecuteTaskVerificationTransitionRequest = {
+    task: TaskData;
+    at: string;
+    by: string;
+    note: string;
+    state: "ok" | "needs_rework";
+    details?: string | null;
+    doc: string;
+    requiredSections: string[];
+};
+export type TaskVerificationTransitionExecution = TaskTransitionWrite & {
+    verificationSection: string;
+    nextDoc: string;
+};
+export type TaskTransitionWrite = {
+    currentStatus: string;
+    intents: TaskStoreIntent[];
+    nextTask: TaskData;
+};
+export type TaskStatusTransitionDependencyPolicy = {
+    kind: "none";
+} | {
+    kind: "require-ready";
+    failureMessage?: string;
+};
+export type TaskStatusTransitionCommentCommitPolicy = {
+    enabled: boolean;
+    action: string;
+    confirmed: boolean;
+    quiet: boolean;
+};
+export type ExecuteTaskStatusTransitionRequest = BuildTaskStatusTransitionOptions & {
+    backend: Pick<TaskBackend, "getTask" | "getTasks">;
+    config: AgentplaneConfig;
+    force: boolean;
+    dependencyPolicy?: TaskStatusTransitionDependencyPolicy;
+    commentCommitPolicy?: TaskStatusTransitionCommentCommitPolicy;
+};
+export type TaskStatusTransitionExecution = TaskTransitionWrite & {
+    dependencyState: DependencyState | null;
+    deferredWarnings: string[];
+};
+export declare function readDeferredTaskTransitionWarnings(error: unknown): string[];
+export declare function buildTaskStatusTransition(opts: BuildTaskStatusTransitionOptions): TaskTransitionWrite;
+export declare function executeTaskStatusTransitionRequest(opts: ExecuteTaskStatusTransitionRequest): Promise<TaskStatusTransitionExecution>;
+export declare function buildTaskVerificationTransition(opts: BuildTaskVerificationTransitionOptions): TaskTransitionWrite;
+export declare function executeTaskVerificationTransitionRequest(opts: ExecuteTaskVerificationTransitionRequest): TaskVerificationTransitionExecution;
+export {};
+//# sourceMappingURL=workflow-transition-service.d.ts.map

package/dist/commands/task/shared/workflow-transition-service.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"workflow-transition-service.d.ts","sourceRoot":"","sources":["../../../../src/commands/task/shared/workflow-transition-service.ts"],"names":[],"mappings":"AAEA,OAAO,EAAyC,KAAK,gBAAgB,EAAE,MAAM,qBAAqB,CAAC;AAEnG,OAAO,KAAK,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AAE/E,OAAO,EAML,KAAK,eAAe,EACpB,KAAK,kBAAkB,EACxB,MAAM,4BAA4B,CAAC;AACpC,OAAO,EAGL,KAAK,eAAe,EACrB,MAAM,mBAAmB,CAAC;AAW3B,KAAK,WAAW,GAAG,WAAW,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;AAE7D,KAAK,gCAAgC,GAAG;IACtC,IAAI,EAAE,QAAQ,CAAC;IACf,EAAE,EAAE,MAAM,CAAC;IACX,QAAQ,EAAE,QAAQ,CAAC,QAAQ,CAAC,CAAC;IAC7B,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,MAAM,CAAC;IAClB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,WAAW,CAAC;IACtB,MAAM,CAAC,EAAE,QAAQ,CAAC,QAAQ,CAAC,GAAG,IAAI,CAAC;IACnC,WAAW,CAAC,EAAE,kBAAkB,CAAC;CAClC,CAAC;AAEF,KAAK,sCAAsC,GAAG;IAC5C,IAAI,EAAE,QAAQ,CAAC;IACf,EAAE,EAAE,MAAM,CAAC;IACX,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,IAAI,GAAG,cAAc,CAAC;IAC7B,mBAAmB,EAAE,MAAM,CAAC;IAC5B,OAAO,EAAE,MAAM,CAAC;IAChB,gBAAgB,EAAE,MAAM,EAAE,CAAC;CAC5B,CAAC;AAEF,MAAM,MAAM,wCAAwC,GAAG;IACrD,IAAI,EAAE,QAAQ,CAAC;IACf,EAAE,EAAE,MAAM,CAAC;IACX,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,IAAI,GAAG,cAAc,CAAC;IAC7B,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,GAAG,EAAE,MAAM,CAAC;IACZ,gBAAgB,EAAE,MAAM,EAAE,CAAC;CAC5B,CAAC;AAEF,MAAM,MAAM,mCAAmC,GAAG,mBAAmB,GAAG;IACtE,mBAAmB,EAAE,MAAM,CAAC;IAC5B,OAAO,EAAE,MAAM,CAAC;CACjB,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG;IAChC,aAAa,EAAE,MAAM,CAAC;IACtB,OAAO,EAAE,eAAe,EAAE,CAAC;IAC3B,QAAQ,EAAE,QAAQ,CAAC;CACpB,CAAC;AAEF,MAAM,MAAM,oCAAoC,GAC5C;IAAE,IAAI,EAAE,MAAM,CAAA;CAAE,GAChB;IAAE,IAAI,EAAE,eAAe,CAAC;IAAC,cAAc,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC;AAEvD,MAAM,MAAM,uCAAuC,GAAG;IACpD,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,OAAO,CAAC;IACnB,KAAK,EAAE,OAAO,CAAC;CAChB,CAAC;AAEF,MAAM,MAAM,kCAAkC,GAAG,gCAAgC,GAAG;IAClF,OAAO,EAAE,IAAI,CAAC,WAAW,EAAE,SAAS,GAAG,UAAU,CAAC,CAAC;IACnD,MAAM,EAAE,gBAAgB,CAAC;IACzB,KAAK,EAAE,OAAO,CAAC;IACf,gBAAgB,CAAC,EAAE,oCAAoC,CAAC;IACxD,mBAAmB,CAAC,EAAE,uCAAuC,CAAC;CAC/D,CAAC;AAEF,MAAM,MAAM,6BAA6B,GAAG,mBAAmB,GAAG;IAChE,eAAe,EAAE,eAAe,GAAG,IAAI,CAAC;IACxC,gBAAgB,EAAE,MAAM,EAAE,CAAC;CAC5B,CAAC;AAEF,wBAAgB,kCAAkC,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,EAAE,CAS3E;AA4ED,wBAAgB,yBAAyB,CACvC,IAAI,EAAE,gCAAgC,GACrC,mBAAmB,CAkCrB;AAED,wBAAsB,kCAAkC,CACtD,IAAI,EAAE,kCAAkC,GACvC,OAAO,CAAC,6BAA6B,CAAC,CAgDxC;AAED,wBAAgB,+BAA+B,CAC7C,IAAI,EAAE,sCAAsC,GAC3C,mBAAmB,CAyCrB;AAED,wBAAgB,wCAAwC,CACtD,IAAI,EAAE,wCAAwC,GAC7C,mCAAmC,CA8CrC"}

package/dist/commands/task/shared/workflow-transition-service.js

@@ -0,0 +1,226 @@
+import { createHash } from "node:crypto";
+import { ensureDocSections, setMarkdownSection } from "@agentplaneorg/core";
+import { CliError } from "../../../shared/errors.js";
+import { appendTaskCommentIntent, appendTaskEventIntent, setTaskFieldsIntent, setTaskSectionIntent, touchTaskDocMetaIntent, } from "../../shared/task-store.js";
+import { dependencyWarningMessages, resolveTaskDependencyState, } from "./dependencies.js";
+import { appendTaskEvent, extractDocSection, normalizeTaskDocVersion, normalizeVerificationSectionLayout, VERIFICATION_RESULTS_BEGIN, VERIFICATION_RESULTS_END, } from "../shared.js";
+import { ensureStatusTransitionAllowed, resolveCommentCommitWarning } from "./transitions.js";
+export function readDeferredTaskTransitionWarnings(error) {
+    if (!(error instanceof CliError))
+        return [];
+    const warnings = error.context?.deferred_warnings;
+    if (!Array.isArray(warnings))
+        return [];
+    return [
+        ...new Set(warnings.filter((item) => typeof item === "string" && item.length > 0)),
+    ];
+}
+function normalizeComments(task) {
+    return Array.isArray(task.comments)
+        ? task.comments.filter((item) => !!item && typeof item.author === "string" && typeof item.body === "string")
+        : [];
+}
+function appendVerificationEntryBetweenMarkers(sectionText, entryText, version) {
+    const ensured = normalizeVerificationSectionLayout(sectionText, version);
+    const beginIdx = ensured.indexOf(VERIFICATION_RESULTS_BEGIN);
+    const endIdx = ensured.indexOf(VERIFICATION_RESULTS_END);
+    if (beginIdx === -1 || endIdx === -1 || endIdx <= beginIdx) {
+        throw new Error("Verification results markers are malformed");
+    }
+    const beforeEnd = ensured.slice(0, endIdx).trimEnd();
+    const afterEnd = ensured.slice(endIdx).trimStart();
+    const entry = entryText.trimEnd();
+    const parts = [
+        beforeEnd,
+        ...(beforeEnd.endsWith(VERIFICATION_RESULTS_BEGIN) ? [] : [""]),
+        entry,
+        "",
+        afterEnd,
+    ];
+    return parts.join("\n").trimEnd();
+}
+function renderVerificationEntry(opts) {
+    const lines = [
+        `### ${opts.at} — VERIFY — ${opts.state}`,
+        "",
+        `By: ${opts.by}`,
+        "",
+        `Note: ${opts.note}`,
+    ];
+    const verifyStepsRef = (opts.verifyStepsRef ?? "").trim();
+    if (verifyStepsRef) {
+        lines.push("", `VerifyStepsRef: ${verifyStepsRef}`);
+    }
+    const details = (opts.details ?? "").trim();
+    if (details) {
+        lines.push("", "Details:", "", details);
+    }
+    return `${lines.join("\n").trimEnd()}\n`;
+}
+function sha256Hex(text) {
+    return createHash("sha256").update(text, "utf8").digest("hex");
+}
+function buildStatusTaskPatch(opts) {
+    const patch = { status: opts.toStatus };
+    if (opts.extraFields)
+        Object.assign(patch, opts.extraFields);
+    if (opts.commit !== undefined) {
+        patch.commit = opts.commit;
+    }
+    return patch;
+}
+export function buildTaskStatusTransition(opts) {
+    const currentStatus = String(opts.task.status || "TODO").toUpperCase();
+    const patch = buildStatusTaskPatch(opts);
+    const statusEvent = {
+        type: "status",
+        at: opts.at,
+        author: opts.eventAuthor,
+        from: currentStatus,
+        to: opts.toStatus,
+        note: opts.note,
+    };
+    const intents = [setTaskFieldsIntent(patch)];
+    if (opts.comment) {
+        intents.push(appendTaskCommentIntent(opts.comment));
+    }
+    intents.push(appendTaskEventIntent(statusEvent), touchTaskDocMetaIntent({
+        updatedBy: opts.updatedBy,
+        version: normalizeTaskDocVersion(opts.task.doc_version),
+    }));
+    const nextTask = {
+        ...opts.task,
+        ...patch,
+        comments: opts.comment ? [...normalizeComments(opts.task), opts.comment] : opts.task.comments,
+        events: appendTaskEvent(opts.task, statusEvent),
+        doc_version: normalizeTaskDocVersion(opts.task.doc_version),
+        doc_updated_at: opts.at,
+        doc_updated_by: opts.updatedBy,
+    };
+    return { currentStatus, intents, nextTask };
+}
+export async function executeTaskStatusTransitionRequest(opts) {
+    const currentStatus = String(opts.task.status || "TODO").toUpperCase();
+    ensureStatusTransitionAllowed({
+        currentStatus,
+        nextStatus: opts.toStatus,
+        force: opts.force,
+    });
+    let dependencyState = null;
+    const deferredWarnings = [];
+    const dependencyPolicy = opts.dependencyPolicy ?? { kind: "none" };
+    if (dependencyPolicy.kind === "require-ready" && !opts.force) {
+        dependencyState = await resolveTaskDependencyState(opts.task, opts.backend);
+        deferredWarnings.push(...dependencyWarningMessages(dependencyState));
+        if (dependencyState.missing.length > 0 || dependencyState.incomplete.length > 0) {
+            const warnings = [...new Set(deferredWarnings)];
+            throw new CliError({
+                exitCode: 2,
+                code: "E_USAGE",
+                message: dependencyPolicy.failureMessage ??
+                    `Task is not ready: ${opts.task.id} (use --force to override)`,
+                context: {
+                    reason_code: "task_transition_dependencies_not_ready",
+                    deferred_warnings: warnings,
+                },
+            });
+        }
+    }
+    if (opts.commentCommitPolicy) {
+        const warning = resolveCommentCommitWarning({
+            enabled: opts.commentCommitPolicy.enabled,
+            config: opts.config,
+            action: opts.commentCommitPolicy.action,
+            confirmed: opts.commentCommitPolicy.confirmed,
+            quiet: opts.commentCommitPolicy.quiet,
+            statusFrom: currentStatus,
+            statusTo: opts.toStatus,
+        });
+        if (warning)
+            deferredWarnings.push(warning);
+    }
+    return {
+        ...buildTaskStatusTransition(opts),
+        dependencyState,
+        deferredWarnings: [...new Set(deferredWarnings)],
+    };
+}
+export function buildTaskVerificationTransition(opts) {
+    const currentStatus = String(opts.task.status || "TODO").toUpperCase();
+    const verification = {
+        state: opts.state,
+        updated_at: opts.at,
+        updated_by: opts.by,
+        note: opts.note,
+    };
+    const patch = {
+        verification,
+        ...(opts.state === "needs_rework" ? { status: "DOING", commit: null } : {}),
+    };
+    const verifyEvent = {
+        type: "verify",
+        at: opts.at,
+        author: opts.by,
+        state: opts.state,
+        note: opts.note,
+    };
+    const intents = [
+        setTaskFieldsIntent(patch),
+        setTaskSectionIntent({
+            section: "Verification",
+            text: opts.verificationSection,
+            requiredSections: opts.requiredSections,
+        }),
+        appendTaskEventIntent(verifyEvent),
+        touchTaskDocMetaIntent({ updatedBy: opts.by }),
+    ];
+    const nextTask = {
+        ...opts.task,
+        ...patch,
+        doc: opts.nextDoc,
+        events: appendTaskEvent(opts.task, verifyEvent),
+        doc_version: normalizeTaskDocVersion(opts.task.doc_version),
+        doc_updated_at: opts.at,
+        doc_updated_by: opts.by,
+    };
+    return { currentStatus, intents, nextTask };
+}
+export function executeTaskVerificationTransitionRequest(opts) {
+    const baseDoc = ensureDocSections(opts.doc, opts.requiredSections);
+    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(opts.task.doc_version);
+    const verifyStepsRef = [
+        `doc_version=${String(docVersion)}`,
+        `doc_updated_at=${String(opts.task.doc_updated_at ?? "missing")}`,
+        `excerpt_hash=sha256:${verifyStepsHash ?? "missing"}`,
+    ].join(", ");
+    const entry = renderVerificationEntry({
+        at: opts.at,
+        state: opts.state,
+        by: opts.by,
+        note: opts.note,
+        details: opts.details ?? null,
+        verifyStepsRef,
+    });
+    const nextVerification = appendVerificationEntryBetweenMarkers(verificationSection, entry, docVersion);
+    const nextDoc = ensureDocSections(setMarkdownSection(baseDoc, "Verification", nextVerification), opts.requiredSections);
+    const transition = buildTaskVerificationTransition({
+        task: opts.task,
+        at: opts.at,
+        by: opts.by,
+        note: opts.note,
+        state: opts.state,
+        verificationSection: nextVerification,
+        nextDoc,
+        requiredSections: opts.requiredSections,
+    });
+    return {
+        ...transition,
+        verificationSection: nextVerification,
+        nextDoc,
+    };
+}

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

@@ -1,7 +1,9 @@
 export { dedupeStrings } from "../../shared/strings.js";
-export { decodeEscapedTaskTextNewlines, nowIso, normalizeTaskDocVersion, taskObservationSectionName, extractTaskObservationSection, VERIFY_STEPS_PLACEHOLDER, VERIFICATION_RESULTS_BEGIN, VERIFICATION_RESULTS_END, extractDocSection, isVerifyStepsFilled, isDocSectionFilled, ensureAgentFilledRequiredDocSections, normalizeVerificationSectionLayout, } from "./shared/docs.js";
+export { decodeEscapedTaskTextNewlines, nowIso, normalizeTaskDocVersion, taskObservationSectionName, extractTaskObservationSection, VERIFY_STEPS_PLACEHOLDER, VERIFICATION_RESULTS_BEGIN, VERIFICATION_RESULTS_END, extractDocSection, isVerifyStepsFilled, isDocSectionFilled, ensureAgentFilledRequiredDocSections, normalizeVerificationSectionLayout, resolveWritableDocSections, } from "./shared/docs.js";
 export { normalizeDependsOnInput, normalizeTaskStatus, toStringArray, requiresVerify, type PrimaryTagResolution, type TaskTagPolicy, readTaskTagPolicy, resolvePrimaryTagFromConfig, requiresVerifyStepsByPrimary, requiresVerificationByPrimary, resolvePrimaryTag, warnIfUnknownOwner, } from "./shared/tags.js";
-export { type DependencyState, ensureTaskDependsOnGraphIsAcyclic, resolveTaskDependencyState, buildDependencyState, formatTaskLine, } from "./shared/dependencies.js";
-export { appendTaskEvent, ensurePlanApprovedIfRequired, ensureVerificationSatisfiedIfRequired, isTransitionAllowed, ensureStatusTransitionAllowed, ensureCommentCommitAllowed, requireStructuredComment, enforceStatusCommitPolicy, isMajorStatusCommitTransition, readHeadCommit, readCommitInfo, defaultCommitEmojiForStatus, } from "./shared/transitions.js";
-export { type TaskListFilters, parseTaskListFilters, handleTaskListWarnings, taskTextBlob, } from "./shared/listing.js";
+export { type DependencyState, dependencyWarningMessages, ensureTaskDependsOnGraphIsAcyclic, resolveTaskDependencyState, buildDependencyState, formatTaskLine, } from "./shared/dependencies.js";
+export { appendTaskEvent, ensurePlanApprovedIfRequired, ensureVerificationSatisfiedIfRequired, isTransitionAllowed, ensureStatusTransitionAllowed, ensureCommentCommitAllowed, emitTransitionWarnings, resolveCommentCommitWarning, requireStructuredComment, prepareTaskTransitionComment, resolveTaskTransitionExecutorAgent, runTaskTransitionCommentCommit, enforceStatusCommitPolicy, resolveStatusCommitPolicyWarning, isMajorStatusCommitTransition, readHeadCommit, readCommitInfo, defaultCommitEmojiForStatus, } from "./shared/transitions.js";
+export { buildTaskStatusTransition, executeTaskStatusTransitionRequest, buildTaskVerificationTransition, executeTaskVerificationTransitionRequest, readDeferredTaskTransitionWarnings, type TaskTransitionWrite, type TaskStatusTransitionDependencyPolicy, type TaskStatusTransitionCommentCommitPolicy, type ExecuteTaskStatusTransitionRequest, type TaskStatusTransitionExecution, type ExecuteTaskVerificationTransitionRequest, type TaskVerificationTransitionExecution, } from "./shared/workflow-transition-service.js";
+export { applyTaskStatusTransitionCommand } from "./shared/transition-command.js";
+export { type TaskListFilters, parseTaskListFilters, handleTaskListWarnings, queryTaskProjection, type QueryTaskProjectionResult, taskTextBlob, } from "./shared/listing.js";
 //# sourceMappingURL=shared.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"shared.d.ts","sourceRoot":"","sources":["../../../src/commands/task/shared.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,aAAa,EAAE,MAAM,yBAAyB,CAAC;AAExD,OAAO,EACL,6BAA6B,EAC7B,MAAM,EACN,uBAAuB,EACvB,0BAA0B,EAC1B,6BAA6B,EAC7B,wBAAwB,EACxB,0BAA0B,EAC1B,wBAAwB,EACxB,iBAAiB,EACjB,mBAAmB,EACnB,kBAAkB,EAClB,oCAAoC,EACpC,kCAAkC,GACnC,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EACL,uBAAuB,EACvB,mBAAmB,EACnB,aAAa,EACb,cAAc,EACd,KAAK,oBAAoB,EACzB,KAAK,aAAa,EAClB,iBAAiB,EACjB,2BAA2B,EAC3B,4BAA4B,EAC5B,6BAA6B,EAC7B,iBAAiB,EACjB,kBAAkB,GACnB,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EACL,KAAK,eAAe,EACpB,iCAAiC,EACjC,0BAA0B,EAC1B,oBAAoB,EACpB,cAAc,GACf,MAAM,0BAA0B,CAAC;AAClC,OAAO,EACL,eAAe,EACf,4BAA4B,EAC5B,qCAAqC,EACrC,mBAAmB,EACnB,6BAA6B,EAC7B,0BAA0B,EAC1B,wBAAwB,EACxB,yBAAyB,EACzB,6BAA6B,EAC7B,cAAc,EACd,cAAc,EACd,2BAA2B,GAC5B,MAAM,yBAAyB,CAAC;AACjC,OAAO,EACL,KAAK,eAAe,EACpB,oBAAoB,EACpB,sBAAsB,EACtB,YAAY,GACb,MAAM,qBAAqB,CAAC"}
+{"version":3,"file":"shared.d.ts","sourceRoot":"","sources":["../../../src/commands/task/shared.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,aAAa,EAAE,MAAM,yBAAyB,CAAC;AAExD,OAAO,EACL,6BAA6B,EAC7B,MAAM,EACN,uBAAuB,EACvB,0BAA0B,EAC1B,6BAA6B,EAC7B,wBAAwB,EACxB,0BAA0B,EAC1B,wBAAwB,EACxB,iBAAiB,EACjB,mBAAmB,EACnB,kBAAkB,EAClB,oCAAoC,EACpC,kCAAkC,EAClC,0BAA0B,GAC3B,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EACL,uBAAuB,EACvB,mBAAmB,EACnB,aAAa,EACb,cAAc,EACd,KAAK,oBAAoB,EACzB,KAAK,aAAa,EAClB,iBAAiB,EACjB,2BAA2B,EAC3B,4BAA4B,EAC5B,6BAA6B,EAC7B,iBAAiB,EACjB,kBAAkB,GACnB,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EACL,KAAK,eAAe,EACpB,yBAAyB,EACzB,iCAAiC,EACjC,0BAA0B,EAC1B,oBAAoB,EACpB,cAAc,GACf,MAAM,0BAA0B,CAAC;AAClC,OAAO,EACL,eAAe,EACf,4BAA4B,EAC5B,qCAAqC,EACrC,mBAAmB,EACnB,6BAA6B,EAC7B,0BAA0B,EAC1B,sBAAsB,EACtB,2BAA2B,EAC3B,wBAAwB,EACxB,4BAA4B,EAC5B,kCAAkC,EAClC,8BAA8B,EAC9B,yBAAyB,EACzB,gCAAgC,EAChC,6BAA6B,EAC7B,cAAc,EACd,cAAc,EACd,2BAA2B,GAC5B,MAAM,yBAAyB,CAAC;AACjC,OAAO,EACL,yBAAyB,EACzB,kCAAkC,EAClC,+BAA+B,EAC/B,wCAAwC,EACxC,kCAAkC,EAClC,KAAK,mBAAmB,EACxB,KAAK,oCAAoC,EACzC,KAAK,uCAAuC,EAC5C,KAAK,kCAAkC,EACvC,KAAK,6BAA6B,EAClC,KAAK,wCAAwC,EAC7C,KAAK,mCAAmC,GACzC,MAAM,yCAAyC,CAAC;AACjD,OAAO,EAAE,gCAAgC,EAAE,MAAM,gCAAgC,CAAC;AAClF,OAAO,EACL,KAAK,eAAe,EACpB,oBAAoB,EACpB,sBAAsB,EACtB,mBAAmB,EACnB,KAAK,yBAAyB,EAC9B,YAAY,GACb,MAAM,qBAAqB,CAAC"}

package/dist/commands/task/shared.js

@@ -1,6 +1,8 @@
 export { dedupeStrings } from "../../shared/strings.js";
-export { decodeEscapedTaskTextNewlines, nowIso, normalizeTaskDocVersion, taskObservationSectionName, extractTaskObservationSection, VERIFY_STEPS_PLACEHOLDER, VERIFICATION_RESULTS_BEGIN, VERIFICATION_RESULTS_END, extractDocSection, isVerifyStepsFilled, isDocSectionFilled, ensureAgentFilledRequiredDocSections, normalizeVerificationSectionLayout, } from "./shared/docs.js";
+export { decodeEscapedTaskTextNewlines, nowIso, normalizeTaskDocVersion, taskObservationSectionName, extractTaskObservationSection, VERIFY_STEPS_PLACEHOLDER, VERIFICATION_RESULTS_BEGIN, VERIFICATION_RESULTS_END, extractDocSection, isVerifyStepsFilled, isDocSectionFilled, ensureAgentFilledRequiredDocSections, normalizeVerificationSectionLayout, resolveWritableDocSections, } from "./shared/docs.js";
 export { normalizeDependsOnInput, normalizeTaskStatus, toStringArray, requiresVerify, readTaskTagPolicy, resolvePrimaryTagFromConfig, requiresVerifyStepsByPrimary, requiresVerificationByPrimary, resolvePrimaryTag, warnIfUnknownOwner, } from "./shared/tags.js";
-export { ensureTaskDependsOnGraphIsAcyclic, resolveTaskDependencyState, buildDependencyState, formatTaskLine, } from "./shared/dependencies.js";
-export { appendTaskEvent, ensurePlanApprovedIfRequired, ensureVerificationSatisfiedIfRequired, isTransitionAllowed, ensureStatusTransitionAllowed, ensureCommentCommitAllowed, requireStructuredComment, enforceStatusCommitPolicy, isMajorStatusCommitTransition, readHeadCommit, readCommitInfo, defaultCommitEmojiForStatus, } from "./shared/transitions.js";
-export { parseTaskListFilters, handleTaskListWarnings, taskTextBlob, } from "./shared/listing.js";
+export { dependencyWarningMessages, ensureTaskDependsOnGraphIsAcyclic, resolveTaskDependencyState, buildDependencyState, formatTaskLine, } from "./shared/dependencies.js";
+export { appendTaskEvent, ensurePlanApprovedIfRequired, ensureVerificationSatisfiedIfRequired, isTransitionAllowed, ensureStatusTransitionAllowed, ensureCommentCommitAllowed, emitTransitionWarnings, resolveCommentCommitWarning, requireStructuredComment, prepareTaskTransitionComment, resolveTaskTransitionExecutorAgent, runTaskTransitionCommentCommit, enforceStatusCommitPolicy, resolveStatusCommitPolicyWarning, isMajorStatusCommitTransition, readHeadCommit, readCommitInfo, defaultCommitEmojiForStatus, } from "./shared/transitions.js";
+export { buildTaskStatusTransition, executeTaskStatusTransitionRequest, buildTaskVerificationTransition, executeTaskVerificationTransitionRequest, readDeferredTaskTransitionWarnings, } from "./shared/workflow-transition-service.js";
+export { applyTaskStatusTransitionCommand } from "./shared/transition-command.js";
+export { parseTaskListFilters, handleTaskListWarnings, queryTaskProjection, taskTextBlob, } from "./shared/listing.js";

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

@@ -1 +1 @@
-{"version":3,"file":"show.d.ts","sourceRoot":"","sources":["../../../src/commands/task/show.ts"],"names":[],"mappings":"AAKA,OAAO,EAIL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AAEnC,wBAAsB,WAAW,CAAC,IAAI,EAAE;IACtC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CA2BlB"}
+{"version":3,"file":"show.d.ts","sourceRoot":"","sources":["../../../src/commands/task/show.ts"],"names":[],"mappings":"AAOA,OAAO,EAIL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AA4BnC,wBAAsB,WAAW,CAAC,IAAI,EAAE;IACtC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAsClB"}

package/dist/commands/task/show.js

@@ -1,11 +1,35 @@
-import { validateTaskDocMetadata } from "@agentplaneorg/core";
+import { parseTaskReadme, taskReadmePath, validateTaskDocMetadata } from "@agentplaneorg/core";
+import { readFile } from "node:fs/promises";
+import path from "node:path";
 import { mapBackendError } from "../../cli/error-map.js";
 import { CliError } from "../../shared/errors.js";
 import { loadCommandContext, loadTaskFromContext, taskDataToFrontmatter, } from "../shared/task-backend.js";
+async function detectLocalTaskMetadataErrors(ctx, taskId) {
+    if (ctx.backendId !== "local")
+        return null;
+    const readmePath = taskReadmePath(path.join(ctx.resolvedProject.gitRoot, ctx.config.paths.workflow_dir), taskId);
+    try {
+        const text = await readFile(readmePath, "utf8");
+        const parsed = parseTaskReadme(text);
+        const frontmatter = {
+            ...parsed.frontmatter,
+            id: typeof parsed.frontmatter.id === "string" && parsed.frontmatter.id.trim().length > 0
+                ? parsed.frontmatter.id
+                : taskId,
+        };
+        const metadataErrors = validateTaskDocMetadata(frontmatter);
+        return metadataErrors.length > 0 ? metadataErrors : null;
+    }
+    catch {
+        return null;
+    }
+}
 export async function cmdTaskShow(opts) {
+    let ctx = null;
     try {
-        const ctx = opts.ctx ??
-            (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
+        ctx =
+            opts.ctx ??
+                (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
         const task = await loadTaskFromContext({ ctx, taskId: opts.taskId });
         const frontmatter = taskDataToFrontmatter(task);
         if (ctx.backendId === "local") {
@@ -22,6 +46,16 @@ export async function cmdTaskShow(opts) {
         return 0;
     }
     catch (err) {
+        if (!(err instanceof CliError) && ctx) {
+            const metadataErrors = await detectLocalTaskMetadataErrors(ctx, opts.taskId);
+            if (metadataErrors && metadataErrors.length > 0) {
+                throw new CliError({
+                    exitCode: 3,
+                    code: "E_VALIDATION",
+                    message: `Invalid task README metadata: ${metadataErrors.join("; ")}`,
+                });
+            }
+        }
         if (err instanceof CliError)
             throw err;
         throw mapBackendError(err, {

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

@@ -1 +1 @@
-{"version":3,"file":"start.d.ts","sourceRoot":"","sources":["../../../src/commands/task/start.ts"],"names":[],"mappings":"AAQA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AAiEnC,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;IACf,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,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;IACf,GAAG,CAAC,EAAE,OAAO,CAAC;IACd,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CA6KlB"}
+{"version":3,"file":"start.d.ts","sourceRoot":"","sources":["../../../src/commands/task/start.ts"],"names":[],"mappings":"AAMA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AA4DpF,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;IACf,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,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;IACf,GAAG,CAAC,EAAE,OAAO,CAAC;IACd,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAoFlB"}