agentplane 0.3.9 → 0.3.11

package/dist/commands/task/findings-add.command.js

@@ -0,0 +1,165 @@
+import { usageError } from "../../cli/spec/errors.js";
+import { cmdTaskFindingsAdd } from "./findings.js";
+export const taskFindingsAddSpec = {
+    id: ["task", "findings", "add"],
+    group: "Task",
+    summary: "Append a structured Findings/Notes block; incident promotion is default unless --local-only.",
+    synopsis: [
+        "agentplane task findings add <task-id> --observation <text> --impact <text> --resolution <text> [--local-only] [--repo-fixable] [--incident-scope <text>] [--incident-tag <tag>] [--incident-match <term>] [--incident-advice <text>] [--incident-rule <text>] [--updated-by <id>]",
+    ],
+    args: [{ name: "task-id", required: true, valueHint: "<task-id>" }],
+    options: [
+        {
+            kind: "string",
+            name: "observation",
+            valueHint: "<text>",
+            description: "Required. Observation field.",
+        },
+        { kind: "string", name: "impact", valueHint: "<text>", description: "Required. Impact field." },
+        {
+            kind: "string",
+            name: "resolution",
+            valueHint: "<text>",
+            description: "Required. Resolution field.",
+        },
+        {
+            kind: "boolean",
+            name: "promote",
+            description: "Compatibility flag. Promotion is the default unless `--local-only` is set.",
+        },
+        {
+            kind: "boolean",
+            name: "external",
+            description: "Compatibility flag. External incident metadata is the default unless `--local-only` is set.",
+        },
+        {
+            kind: "boolean",
+            name: "repo-fixable",
+            description: "Mark the structured finding as repo-fixable so it can promote into incidents.md.",
+        },
+        {
+            kind: "boolean",
+            name: "local-only",
+            default: false,
+            description: "Keep the entry task-local only; omit `Promotion: incident-candidate` and any explicit fixability marker.",
+        },
+        {
+            kind: "string",
+            name: "incident-scope",
+            valueHint: "<text>",
+            description: "Optional. IncidentScope override.",
+        },
+        {
+            kind: "string",
+            name: "incident-tag",
+            valueHint: "<tag>",
+            repeatable: true,
+            description: "Repeatable. IncidentTags entry.",
+        },
+        {
+            kind: "string",
+            name: "incident-match",
+            valueHint: "<term>",
+            repeatable: true,
+            description: "Repeatable. IncidentMatch entry.",
+        },
+        {
+            kind: "string",
+            name: "incident-advice",
+            valueHint: "<text>",
+            description: "Optional. IncidentAdvice field.",
+        },
+        {
+            kind: "string",
+            name: "incident-rule",
+            valueHint: "<text>",
+            description: "Optional. IncidentRule field.",
+        },
+        {
+            kind: "string",
+            name: "updated-by",
+            valueHint: "<id>",
+            description: "Optional. Override doc_updated_by metadata (must be non-empty).",
+        },
+    ],
+    examples: [
+        {
+            cmd: 'agentplane task findings add 202602030608-F1Q8AB --observation "GitHub API EOF retries were manual." --impact "Operators repeated the reconcile loop." --resolution "Switch the retry path to REST polling." --incident-scope "GitHub PR reconciliation" --incident-tag workflow --incident-tag github',
+            why: "Append a promotable external incident candidate without hand-editing the README or remembering hidden flags.",
+        },
+        {
+            cmd: 'agentplane task findings add 202602030608-F1Q8AB --observation "One manual follow-up remains." --impact "Task notes should stay local." --resolution "Track it in Findings only." --local-only',
+            why: "Keep an observation task-local when it should not feed incidents collection.",
+        },
+    ],
+    validateRaw: (raw) => {
+        for (const key of ["observation", "impact", "resolution"]) {
+            const value = raw.opts[key];
+            if (typeof value !== "string" || value.trim() === "") {
+                throw usageError({
+                    spec: taskFindingsAddSpec,
+                    message: `Missing required option: --${key}.`,
+                });
+            }
+        }
+        const updatedBy = raw.opts["updated-by"];
+        if (typeof updatedBy === "string" && updatedBy.trim() === "") {
+            throw usageError({ spec: taskFindingsAddSpec, message: "--updated-by must be non-empty." });
+        }
+        if (raw.opts["local-only"] === true &&
+            (raw.opts.promote === true || raw.opts.external === true || raw.opts["repo-fixable"] === true)) {
+            throw usageError({
+                spec: taskFindingsAddSpec,
+                message: "--local-only cannot be combined with --promote, --external, or --repo-fixable.",
+            });
+        }
+        if (raw.opts.external === true && raw.opts["repo-fixable"] === true) {
+            throw usageError({
+                spec: taskFindingsAddSpec,
+                message: "--external and --repo-fixable are mutually exclusive.",
+            });
+        }
+    },
+    parse: (raw) => {
+        const localOnly = raw.opts["local-only"] === true;
+        const repoFixable = raw.opts["repo-fixable"] === true;
+        const fixability = localOnly ? null : repoFixable ? "repo-fixable" : "external";
+        return {
+            taskId: String(raw.args["task-id"]),
+            observation: String(raw.opts.observation),
+            impact: String(raw.opts.impact),
+            resolution: String(raw.opts.resolution),
+            promote: !localOnly,
+            external: !localOnly,
+            fixability,
+            incidentScope: typeof raw.opts["incident-scope"] === "string" ? raw.opts["incident-scope"] : undefined,
+            incidentTags: raw.opts["incident-tag"] ?? [],
+            incidentMatch: raw.opts["incident-match"] ?? [],
+            incidentAdvice: typeof raw.opts["incident-advice"] === "string" ? raw.opts["incident-advice"] : undefined,
+            incidentRule: typeof raw.opts["incident-rule"] === "string" ? raw.opts["incident-rule"] : undefined,
+            updatedBy: typeof raw.opts["updated-by"] === "string" ? raw.opts["updated-by"] : undefined,
+        };
+    },
+};
+export function makeRunTaskFindingsAddHandler(getCtx) {
+    return async (ctx, p) => {
+        return await cmdTaskFindingsAdd({
+            ctx: await getCtx("task findings add"),
+            cwd: ctx.cwd,
+            rootOverride: ctx.rootOverride,
+            taskId: p.taskId,
+            observation: p.observation,
+            impact: p.impact,
+            resolution: p.resolution,
+            promote: p.promote,
+            external: p.external,
+            fixability: p.fixability === "repo-fixable" ? "repo-fixable" : null,
+            incidentScope: p.incidentScope,
+            incidentTags: p.incidentTags,
+            incidentMatch: p.incidentMatch,
+            incidentAdvice: p.incidentAdvice,
+            incidentRule: p.incidentRule,
+            updatedBy: p.updatedBy,
+        });
+    };
+}

package/dist/commands/task/findings.command.d.ts

@@ -0,0 +1,7 @@
+import type { CommandCtx, CommandSpec } from "../../cli/spec/spec.js";
+export type TaskFindingsParsed = {
+    subcommand?: string;
+};
+export declare const taskFindingsSpec: CommandSpec<TaskFindingsParsed>;
+export declare function runTaskFindings(_ctx: CommandCtx, p: TaskFindingsParsed): Promise<number>;
+//# sourceMappingURL=findings.command.d.ts.map

package/dist/commands/task/findings.command.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"findings.command.d.ts","sourceRoot":"","sources":["../../../src/commands/task/findings.command.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAGtE,MAAM,MAAM,kBAAkB,GAAG;IAC/B,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB,CAAC;AAEF,eAAO,MAAM,gBAAgB,EAAE,WAAW,CAAC,kBAAkB,CAY5D,CAAC;AAEF,wBAAsB,eAAe,CAAC,IAAI,EAAE,UAAU,EAAE,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,MAAM,CAAC,CAM9F"}

package/dist/commands/task/findings.command.js

@@ -0,0 +1,20 @@
+import { loadDirectSubcommandNames, throwGroupCommandUsage } from "../../cli/group-command.js";
+export const taskFindingsSpec = {
+    id: ["task", "findings"],
+    group: "Task",
+    summary: "Structured Findings/Notes commands (incident promotion is default unless --local-only).",
+    synopsis: [
+        "agentplane task findings add <task-id> --observation <text> --impact <text> --resolution <text> [--local-only] [--incident-scope <text>] [--incident-tag <tag>] [--incident-match <term>] [--incident-advice <text>] [--incident-rule <text>] [--updated-by <id>]",
+    ],
+    args: [{ name: "subcommand", required: false, valueHint: "<add>" }],
+    parse: (raw) => ({
+        subcommand: typeof raw.args.subcommand === "string" ? raw.args.subcommand : undefined,
+    }),
+};
+export async function runTaskFindings(_ctx, p) {
+    throwGroupCommandUsage({
+        spec: taskFindingsSpec,
+        cmd: p.subcommand ? [p.subcommand] : [],
+        subcommands: await loadDirectSubcommandNames(["task", "findings"]),
+    });
+}

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

@@ -0,0 +1,63 @@
+import { type AgentplaneConfig } from "@agentplaneorg/core";
+import type { TaskData } from "../../backends/task-backend.js";
+import { type CommandContext } from "../shared/task-backend.js";
+import { type TaskStoreIntent } from "../shared/task-store.js";
+export declare function renderStructuredFindingBlock(opts: {
+    observation: string;
+    impact: string;
+    resolution: string;
+    promote: boolean;
+    external: boolean;
+    fixability?: "repo-fixable" | null;
+    incidentScope?: string;
+    incidentTags: readonly string[];
+    incidentMatch: readonly string[];
+    incidentAdvice?: string;
+    incidentRule?: string;
+}): string;
+export type StructuredFindingMutationPlan = {
+    targetSection: string;
+    expectedCurrentText: string | null;
+    intents: readonly TaskStoreIntent[];
+};
+export declare function buildStructuredFindingMutationPlan(opts: {
+    current: TaskData;
+    config: AgentplaneConfig;
+    observation: string;
+    impact: string;
+    resolution: string;
+    promote: boolean;
+    external: boolean;
+    fixability?: "repo-fixable" | null;
+    incidentScope?: string;
+    incidentTags: readonly string[];
+    incidentMatch: readonly string[];
+    incidentAdvice?: string;
+    incidentRule?: string;
+    updatedBy?: string;
+}): StructuredFindingMutationPlan | null;
+export declare function renderFindingsAddRegistryNote(opts: {
+    promote: boolean;
+    external: boolean;
+    taskId: string;
+    branchPr?: boolean;
+}): string;
+export declare function cmdTaskFindingsAdd(opts: {
+    ctx?: CommandContext;
+    cwd: string;
+    rootOverride?: string;
+    taskId: string;
+    observation: string;
+    impact: string;
+    resolution: string;
+    promote: boolean;
+    external: boolean;
+    fixability?: "repo-fixable" | null;
+    incidentScope?: string;
+    incidentTags: string[];
+    incidentMatch: string[];
+    incidentAdvice?: string;
+    incidentRule?: string;
+    updatedBy?: string;
+}): Promise<number>;
+//# sourceMappingURL=findings.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"findings.d.ts","sourceRoot":"","sources":["../../../src/commands/task/findings.ts"],"names":[],"mappings":"AAAA,OAAO,EAIL,KAAK,gBAAgB,EACtB,MAAM,qBAAqB,CAAC;AAE7B,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,gCAAgC,CAAC;AAI/D,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAEpF,OAAO,EAGL,KAAK,eAAe,EACrB,MAAM,yBAAyB,CAAC;AAoCjC,wBAAgB,4BAA4B,CAAC,IAAI,EAAE;IACjD,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;IACnB,OAAO,EAAE,OAAO,CAAC;IACjB,QAAQ,EAAE,OAAO,CAAC;IAClB,UAAU,CAAC,EAAE,cAAc,GAAG,IAAI,CAAC;IACnC,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,YAAY,EAAE,SAAS,MAAM,EAAE,CAAC;IAChC,aAAa,EAAE,SAAS,MAAM,EAAE,CAAC;IACjC,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB,GAAG,MAAM,CAkBT;AAQD,MAAM,MAAM,6BAA6B,GAAG;IAC1C,aAAa,EAAE,MAAM,CAAC;IACtB,mBAAmB,EAAE,MAAM,GAAG,IAAI,CAAC;IACnC,OAAO,EAAE,SAAS,eAAe,EAAE,CAAC;CACrC,CAAC;AAEF,wBAAgB,kCAAkC,CAAC,IAAI,EAAE;IACvD,OAAO,EAAE,QAAQ,CAAC;IAClB,MAAM,EAAE,gBAAgB,CAAC;IACzB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;IACnB,OAAO,EAAE,OAAO,CAAC;IACjB,QAAQ,EAAE,OAAO,CAAC;IAClB,UAAU,CAAC,EAAE,cAAc,GAAG,IAAI,CAAC;IACnC,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,YAAY,EAAE,SAAS,MAAM,EAAE,CAAC;IAChC,aAAa,EAAE,SAAS,MAAM,EAAE,CAAC;IACjC,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,GAAG,6BAA6B,GAAG,IAAI,CA0CvC;AAED,wBAAgB,6BAA6B,CAAC,IAAI,EAAE;IAClD,OAAO,EAAE,OAAO,CAAC;IACjB,QAAQ,EAAE,OAAO,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB,GAAG,MAAM,CAcT;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,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;IACnB,OAAO,EAAE,OAAO,CAAC;IACjB,QAAQ,EAAE,OAAO,CAAC;IAClB,UAAU,CAAC,EAAE,cAAc,GAAG,IAAI,CAAC;IACnC,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,aAAa,EAAE,MAAM,EAAE,CAAC;IACxB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,GAAG,OAAO,CAAC,MAAM,CAAC,CA4ElB"}

package/dist/commands/task/findings.js

@@ -0,0 +1,188 @@
+import { ensureDocSections, normalizeTaskDoc, setMarkdownSection, } from "@agentplaneorg/core";
+import { mapBackendError } from "../../cli/error-map.js";
+import { infoMessage } from "../../cli/output.js";
+import { CliError } from "../../shared/errors.js";
+import { loadCommandContext } from "../shared/task-backend.js";
+import { applyTaskMutation } from "../shared/task-mutation.js";
+import { setTaskSectionIntent, touchTaskDocMetaIntent, } from "../shared/task-store.js";
+import { resolveWritableDocSections } from "./shared.js";
+import { extractDocSection, extractTaskObservationSection, normalizeTaskDocVersion, taskObservationSectionName, } from "./shared/docs.js";
+function ensureNonEmptyFlag(name, value) {
+    const trimmed = value.trim();
+    if (!trimmed) {
+        throw new CliError({
+            exitCode: 2,
+            code: "E_USAGE",
+            message: `Invalid value for --${name}: empty.`,
+        });
+    }
+    return trimmed;
+}
+function dedupeTrimmed(values) {
+    const seen = new Set();
+    const out = [];
+    for (const raw of values) {
+        const trimmed = raw.trim();
+        if (!trimmed)
+            continue;
+        const key = trimmed.toLowerCase();
+        if (seen.has(key))
+            continue;
+        seen.add(key);
+        out.push(trimmed);
+    }
+    return out;
+}
+export function renderStructuredFindingBlock(opts) {
+    const fixability = opts.fixability ?? (opts.external ? "external" : null);
+    const lines = [
+        `- Observation: ${ensureNonEmptyFlag("observation", opts.observation)}`,
+        `  Impact: ${ensureNonEmptyFlag("impact", opts.impact)}`,
+        `  Resolution: ${ensureNonEmptyFlag("resolution", opts.resolution)}`,
+    ];
+    if (opts.promote)
+        lines.push("  Promotion: incident-candidate");
+    if (fixability === "external")
+        lines.push("  Fixability: external");
+    if (fixability === "repo-fixable")
+        lines.push("  Fixability: repo-fixable");
+    if (opts.incidentScope?.trim())
+        lines.push(`  IncidentScope: ${opts.incidentScope.trim()}`);
+    const tags = dedupeTrimmed(opts.incidentTags);
+    if (tags.length > 0)
+        lines.push(`  IncidentTags: ${tags.join(", ")}`);
+    const match = dedupeTrimmed(opts.incidentMatch);
+    if (match.length > 0)
+        lines.push(`  IncidentMatch: ${match.join(", ")}`);
+    if (opts.incidentAdvice?.trim())
+        lines.push(`  IncidentAdvice: ${opts.incidentAdvice.trim()}`);
+    if (opts.incidentRule?.trim())
+        lines.push(`  IncidentRule: ${opts.incidentRule.trim()}`);
+    return lines.join("\n");
+}
+function appendFindingBlock(existingSection, block) {
+    const current = (existingSection ?? "").trim();
+    if (!current)
+        return block;
+    return `${current}\n\n${block}`;
+}
+export function buildStructuredFindingMutationPlan(opts) {
+    const currentDocRaw = typeof opts.current.doc === "string" ? opts.current.doc : "";
+    const docVersion = normalizeTaskDocVersion(opts.current.doc_version);
+    const targetSection = taskObservationSectionName(docVersion);
+    const sectionOrder = resolveWritableDocSections({
+        allowedSections: opts.config.tasks.doc.sections,
+        requiredSections: opts.config.tasks.doc.required_sections,
+        targetSection,
+    });
+    const existingSection = extractTaskObservationSection(currentDocRaw, docVersion);
+    const block = renderStructuredFindingBlock({
+        observation: opts.observation,
+        impact: opts.impact,
+        resolution: opts.resolution,
+        promote: opts.promote,
+        external: opts.external,
+        fixability: opts.fixability ?? null,
+        incidentScope: opts.incidentScope,
+        incidentTags: opts.incidentTags,
+        incidentMatch: opts.incidentMatch,
+        incidentAdvice: opts.incidentAdvice,
+        incidentRule: opts.incidentRule,
+    });
+    const nextSectionText = appendFindingBlock(existingSection, block);
+    const nextDoc = ensureDocSections(setMarkdownSection(currentDocRaw, targetSection, nextSectionText), sectionOrder);
+    if (normalizeTaskDoc(currentDocRaw) === normalizeTaskDoc(nextDoc) && !opts.updatedBy) {
+        return null;
+    }
+    const expectedCurrentText = extractDocSection(currentDocRaw, targetSection);
+    const intents = [
+        setTaskSectionIntent({
+            section: targetSection,
+            text: nextSectionText,
+            requiredSections: sectionOrder,
+            expectedCurrentText,
+        }),
+        ...(opts.updatedBy ? [touchTaskDocMetaIntent({ updatedBy: opts.updatedBy })] : []),
+    ];
+    return { targetSection, expectedCurrentText, intents };
+}
+export function renderFindingsAddRegistryNote(opts) {
+    if (opts.promote && opts.external) {
+        if (opts.branchPr) {
+            return (`incident candidate recorded for ${opts.taskId}; ` +
+                "incidents.md updates later during finish, `verify --collect-incidents`, or `agentplane incidents collect <task-id>`; task-local until base-branch promotion in the current task worktree");
+        }
+        return (`incident candidate recorded for ${opts.taskId}; ` +
+            "incidents.md updates later during finish, `verify --collect-incidents`, or `agentplane incidents collect <task-id>`; task-local in the current checkout until promotion");
+    }
+    return `task-local finding recorded for ${opts.taskId}; incidents.md unchanged in the current checkout`;
+}
+export async function cmdTaskFindingsAdd(opts) {
+    let updatedBy;
+    if (opts.updatedBy !== undefined) {
+        updatedBy = ensureNonEmptyFlag("updated-by", opts.updatedBy);
+    }
+    try {
+        const ctx = opts.ctx ??
+            (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
+        const backend = ctx.taskBackend;
+        const config = ctx.config;
+        const resolved = ctx.resolvedProject;
+        if (!backend.getTaskDoc || !backend.writeTask) {
+            throw new CliError({
+                exitCode: 2,
+                code: "E_USAGE",
+                message: "Configured task backend does not support task docs.",
+            });
+        }
+        let targetSection = "Findings";
+        await applyTaskMutation({
+            ctx,
+            taskId: opts.taskId,
+            build: (current) => {
+                const plan = buildStructuredFindingMutationPlan({
+                    current,
+                    config,
+                    observation: opts.observation,
+                    impact: opts.impact,
+                    resolution: opts.resolution,
+                    promote: opts.promote,
+                    external: opts.external,
+                    fixability: opts.fixability ?? null,
+                    incidentScope: opts.incidentScope,
+                    incidentTags: opts.incidentTags,
+                    incidentMatch: opts.incidentMatch,
+                    incidentAdvice: opts.incidentAdvice,
+                    incidentRule: opts.incidentRule,
+                    updatedBy,
+                });
+                if (!plan) {
+                    return null;
+                }
+                targetSection = plan.targetSection;
+                return {
+                    intents: plan.intents,
+                    writeOptions: {
+                        expectedCurrentText: plan.expectedCurrentText,
+                        expectedSection: targetSection,
+                    },
+                };
+            },
+        });
+        const tasksDir = `${resolved.gitRoot}/${config.paths.workflow_dir}`;
+        process.stdout.write(`${tasksDir}/${opts.taskId}/README.md\n`);
+        process.stderr.write(`${infoMessage(`task findings add outcome=entry-appended section=${targetSection}`)}\n`);
+        process.stderr.write(`${infoMessage(renderFindingsAddRegistryNote({
+            promote: opts.promote,
+            external: opts.external,
+            taskId: opts.taskId,
+            branchPr: config.workflow_mode === "branch_pr",
+        }))}\n`);
+        return 0;
+    }
+    catch (err) {
+        if (err instanceof CliError)
+            throw err;
+        throw mapBackendError(err, { command: "task findings add", root: opts.rootOverride ?? null });
+    }
+}

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

@@ -56,5 +56,6 @@ export declare function createTaskCloseCommit(opts: {
     baseBranchOverride?: string;
     quiet: boolean;
     closeUnstageOthers?: boolean;
+    allowPolicy?: boolean;
 }): Promise<void>;
 //# sourceMappingURL=finish-shared.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"finish-shared.d.ts","sourceRoot":"","sources":["../../../src/commands/task/finish-shared.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,gCAAgC,CAAC;AAG/D,OAAO,EAAuB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AACrF,OAAO,EAA6B,YAAY,EAAmB,MAAM,yBAAyB,CAAC;AAWnG,MAAM,MAAM,kBAAkB,GAAG;IAC/B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;CACjB,CAAC;AAEF,MAAM,MAAM,gBAAgB,GAAG;IAC7B,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,QAAQ,CAAC;CAChB,CAAC;AAEF,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,QAAQ,GAAG,kBAAkB,GAAG,IAAI,CAK5E;AAED,wBAAgB,mBAAmB,CAAC,IAAI,EAAE;IACxC,IAAI,EAAE,QAAQ,CAAC;IACf,MAAM,EAAE,cAAc,CAAC,QAAQ,CAAC,CAAC;IACjC,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,OAAO,CAAC;IACpB,cAAc,EAAE,OAAO,CAAC;IACxB,aAAa,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,IAAI,CAyCP;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,GAAG,EAAE,cAAc,CAAC;IACpB,KAAK,EAAE,UAAU,CAAC,OAAO,YAAY,CAAC,GAAG,IAAI,CAAC;IAC9C,QAAQ,EAAE,OAAO,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,cAAc,EAAE,OAAO,CAAC;IACxB,aAAa,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;IACf,2BAA2B,EAAE,OAAO,CAAC;CACtC,GAAG,OAAO,CAAC;IACV,MAAM,EAAE,gBAAgB,CAAC;IACzB,iBAAiB,EAAE,MAAM,GAAG,IAAI,CAAC;IACjC,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;CAC3B,CAAC,CAuDD;AAED,wBAAsB,kBAAkB,CAAC,IAAI,EAAE;IAC7C,GAAG,EAAE,cAAc,CAAC;IACpB,WAAW,EAAE,gBAAgB,EAAE,CAAC;IAChC,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,OAAO,CAAC;IACf,cAAc,EAAE,OAAO,CAAC;IACxB,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,CAAC,EAAE,KAAK,GAAG,KAAK,GAAG,MAAM,CAAC;IACnC,QAAQ,EAAE,OAAO,CAAC;IAClB,cAAc,EAAE,kBAAkB,GAAG,IAAI,CAAC;CAC3C,GAAG,OAAO,CAAC,IAAI,CAAC,CAoDhB;AAED,wBAAsB,qBAAqB,CAAC,IAAI,EAAE;IAChD,GAAG,EAAE,cAAc,CAAC;IACpB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,KAAK,EAAE,OAAO,CAAC;IACf,kBAAkB,CAAC,EAAE,OAAO,CAAC;CAC9B,GAAG,OAAO,CAAC,IAAI,CAAC,CAuBhB"}
+{"version":3,"file":"finish-shared.d.ts","sourceRoot":"","sources":["../../../src/commands/task/finish-shared.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,gCAAgC,CAAC;AAG/D,OAAO,EAAuB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AACrF,OAAO,EAA6B,YAAY,EAAmB,MAAM,yBAAyB,CAAC;AAWnG,MAAM,MAAM,kBAAkB,GAAG;IAC/B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;CACjB,CAAC;AAEF,MAAM,MAAM,gBAAgB,GAAG;IAC7B,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,QAAQ,CAAC;CAChB,CAAC;AA0CF,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,QAAQ,GAAG,kBAAkB,GAAG,IAAI,CAK5E;AAED,wBAAgB,mBAAmB,CAAC,IAAI,EAAE;IACxC,IAAI,EAAE,QAAQ,CAAC;IACf,MAAM,EAAE,cAAc,CAAC,QAAQ,CAAC,CAAC;IACjC,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,OAAO,CAAC;IACpB,cAAc,EAAE,OAAO,CAAC;IACxB,aAAa,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,IAAI,CAyCP;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,GAAG,EAAE,cAAc,CAAC;IACpB,KAAK,EAAE,UAAU,CAAC,OAAO,YAAY,CAAC,GAAG,IAAI,CAAC;IAC9C,QAAQ,EAAE,OAAO,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,cAAc,EAAE,OAAO,CAAC;IACxB,aAAa,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;IACf,2BAA2B,EAAE,OAAO,CAAC;CACtC,GAAG,OAAO,CAAC;IACV,MAAM,EAAE,gBAAgB,CAAC;IACzB,iBAAiB,EAAE,MAAM,GAAG,IAAI,CAAC;IACjC,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;CAC3B,CAAC,CAuDD;AAED,wBAAsB,kBAAkB,CAAC,IAAI,EAAE;IAC7C,GAAG,EAAE,cAAc,CAAC;IACpB,WAAW,EAAE,gBAAgB,EAAE,CAAC;IAChC,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,OAAO,CAAC;IACf,cAAc,EAAE,OAAO,CAAC;IACxB,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,CAAC,EAAE,KAAK,GAAG,KAAK,GAAG,MAAM,CAAC;IACnC,QAAQ,EAAE,OAAO,CAAC;IAClB,cAAc,EAAE,kBAAkB,GAAG,IAAI,CAAC;CAC3C,GAAG,OAAO,CAAC,IAAI,CAAC,CAuEhB;AAED,wBAAsB,qBAAqB,CAAC,IAAI,EAAE;IAChD,GAAG,EAAE,cAAc,CAAC;IACpB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,KAAK,EAAE,OAAO,CAAC;IACf,kBAAkB,CAAC,EAAE,OAAO,CAAC;IAC7B,WAAW,CAAC,EAAE,OAAO,CAAC;CACvB,GAAG,OAAO,CAAC,IAAI,CAAC,CAuBhB"}

package/dist/commands/task/finish-shared.js

@@ -4,6 +4,46 @@ import { cmdCommit } from "../guard/index.js";
 import { loadTaskFromContext } from "../shared/task-backend.js";
 import { backendIsLocalFileBackend, getTaskStore, mutateTaskStore } from "../shared/task-store.js";
 import { ensureAgentFilledRequiredDocSections, executeTaskStatusTransitionRequest, ensureVerificationSatisfiedIfRequired, nowIso, resolvePrimaryTag, toStringArray, } from "./shared.js";
+function normalizeCommentBody(value) {
+    return typeof value === "string" ? value.trim() : "";
+}
+function commitInfoMatches(current, requested) {
+    if (!requested)
+        return true;
+    const existing = existingCommitInfo(current);
+    if (!existing)
+        return false;
+    return existing.hash === requested.hash && existing.message === requested.message;
+}
+function isIdempotentDoneRetry(opts) {
+    if (String(opts.task.status ?? "TODO").toUpperCase() !== "DONE")
+        return false;
+    const lastComment = opts.task.comments?.at(-1) ?? null;
+    const lastEvent = opts.task.events?.at(-1) ?? null;
+    if (lastComment?.author !== opts.author)
+        return false;
+    if (normalizeCommentBody(lastComment.body) !== normalizeCommentBody(opts.body))
+        return false;
+    if (lastEvent?.type !== "status" || lastEvent.author !== opts.author)
+        return false;
+    if (String(lastEvent.from ?? "").toUpperCase() !== "DOING")
+        return false;
+    if (String(lastEvent.to ?? "").toUpperCase() !== "DONE")
+        return false;
+    if (normalizeCommentBody(lastEvent.note) !== normalizeCommentBody(opts.body))
+        return false;
+    if (!commitInfoMatches(opts.task, opts.taskCommitInfo))
+        return false;
+    if (opts.taskId === opts.metaTaskId) {
+        if ((opts.task.result_summary ?? "") !== opts.resultSummary)
+            return false;
+        if ((opts.task.risk_level ?? undefined) !== opts.riskLevel)
+            return false;
+        if (Boolean(opts.task.breaking) !== opts.breaking)
+            return false;
+    }
+    return true;
+}
 export function existingCommitInfo(task) {
     const hash = typeof task.commit?.hash === "string" ? task.commit.hash.trim() : "";
     if (!hash)
@@ -107,7 +147,8 @@ export async function writeFinishedTasks(opts) {
     const useStore = backendIsLocalFileBackend(opts.ctx);
     const store = useStore ? getTaskStore(opts.ctx) : null;
     const taskCount = opts.loadedTasks.length;
-    for (const { taskId, task } of opts.loadedTasks) {
+    for (const loaded of opts.loadedTasks) {
+        const { taskId, task } = loaded;
         const at = nowIso();
         const applyTransition = async (currentTask) => {
             assertTaskCanFinish({
@@ -119,6 +160,20 @@ export async function writeFinishedTasks(opts) {
                 resultSummary: opts.resultSummary,
                 force: opts.force,
             });
+            if (opts.force &&
+                isIdempotentDoneRetry({
+                    task: currentTask,
+                    author: opts.author,
+                    body: opts.body,
+                    resultSummary: opts.resultSummary,
+                    metaTaskId: opts.metaTaskId,
+                    taskId,
+                    riskLevel: opts.riskLevel,
+                    breaking: opts.breaking,
+                    taskCommitInfo: opts.taskCommitInfo,
+                })) {
+                return { intents: [], nextTask: currentTask };
+            }
             return await executeTaskStatusTransitionRequest({
                 task: currentTask,
                 backend: opts.ctx.taskBackend,
@@ -144,14 +199,16 @@ export async function writeFinishedTasks(opts) {
             });
         };
         if (useStore) {
-            await mutateTaskStore(store, taskId, async (currentTask) => {
+            const result = await mutateTaskStore(store, taskId, async (currentTask) => {
                 const execution = await applyTransition(currentTask);
                 return execution.intents;
             });
+            loaded.task = result.task;
         }
         else {
             const execution = await applyTransition(task);
             await opts.ctx.taskBackend.writeTask(execution.nextTask);
+            loaded.task = execution.nextTask;
         }
     }
 }
@@ -168,7 +225,7 @@ export async function createTaskCloseCommit(opts) {
         autoAllow: false,
         allowTasks: true,
         allowBase: opts.ctx.config.workflow_mode === "branch_pr",
-        allowPolicy: false,
+        allowPolicy: opts.allowPolicy === true,
         allowConfig: false,
         allowHooks: false,
         allowCI: false,

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

@@ -28,6 +28,16 @@ export declare function cmdFinish(opts: {
     noCloseCommit?: boolean;
     closeUnstageOthers?: boolean;
     baseBranchOverride?: string;
+    observation?: string;
+    impact?: string;
+    resolution?: string;
+    localOnly?: boolean;
+    repoFixable?: boolean;
+    incidentScope?: string;
+    incidentTags?: string[];
+    incidentMatch?: string[];
+    incidentAdvice?: string;
+    incidentRule?: string;
     quiet: boolean;
 }): Promise<number>;
 //# sourceMappingURL=finish.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"finish.d.ts","sourceRoot":"","sources":["../../../src/commands/task/finish.ts"],"names":[],"mappings":"AAQA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAqCpF,wBAAsB,SAAS,CAAC,IAAI,EAAE;IACpC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,KAAK,GAAG,KAAK,GAAG,MAAM,CAAC;IAC9B,QAAQ,EAAE,OAAO,CAAC;IAClB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,OAAO,CAAC;IACf,GAAG,CAAC,EAAE,OAAO,CAAC;IACd,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,YAAY,EAAE,OAAO,CAAC;IACtB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,iBAAiB,EAAE,MAAM,EAAE,CAAC;IAC5B,qBAAqB,EAAE,OAAO,CAAC;IAC/B,wBAAwB,EAAE,OAAO,CAAC;IAClC,mBAAmB,EAAE,OAAO,CAAC;IAC7B,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,kBAAkB,CAAC,EAAE,OAAO,CAAC;IAC7B,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CA4SlB"}
+{"version":3,"file":"finish.d.ts","sourceRoot":"","sources":["../../../src/commands/task/finish.ts"],"names":[],"mappings":"AAYA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAsHpF,wBAAsB,SAAS,CAAC,IAAI,EAAE;IACpC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,KAAK,GAAG,KAAK,GAAG,MAAM,CAAC;IAC9B,QAAQ,EAAE,OAAO,CAAC;IAClB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,OAAO,CAAC;IACf,GAAG,CAAC,EAAE,OAAO,CAAC;IACd,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,YAAY,EAAE,OAAO,CAAC;IACtB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,iBAAiB,EAAE,MAAM,EAAE,CAAC;IAC5B,qBAAqB,EAAE,OAAO,CAAC;IAC/B,wBAAwB,EAAE,OAAO,CAAC;IAClC,mBAAmB,EAAE,OAAO,CAAC;IAC7B,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,kBAAkB,CAAC,EAAE,OAAO,CAAC;IAC7B,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB,aAAa,CAAC,EAAE,MAAM,EAAE,CAAC;IACzB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CA0YlB"}