agentplane 0.3.23 → 0.3.26

package/dist/commands/guard/impl/comment-commit.js

@@ -1,127 +0,0 @@
-import { extractTaskSuffix } from "@agentplaneorg/core/commit";
-import { invalidValueMessage, successMessage } from "../../../cli/output.js";
-import { CliError } from "../../../shared/errors.js";
-import { formatCommentBodyForCommit, normalizeCommentBodyForCommit, } from "../../shared/comment-format.js";
-import { loadCommandContext } from "../../shared/task-backend.js";
-import { buildGitCommitEnv, resolveCanonicalGitIdentity } from "./env.js";
-import { stageAllowlist } from "./allow.js";
-import { guardCommitCheck } from "./policy.js";
-function deriveCommitMessageFromComment(opts) {
-    const prefix = opts.emoji.trim();
-    if (!prefix) {
-        throw new CliError({
-            exitCode: 2,
-            code: "E_USAGE",
-            message: "Emoji prefix is required when deriving commit messages from task comments",
-        });
-    }
-    const suffix = extractTaskSuffix(opts.taskId);
-    if (!suffix) {
-        throw new CliError({
-            exitCode: 2,
-            code: "E_USAGE",
-            message: invalidValueMessage("task id", opts.taskId, "valid task id"),
-        });
-    }
-    const primary = opts.primaryTag.trim().toLowerCase();
-    if (!primary) {
-        throw new CliError({
-            exitCode: 2,
-            code: "E_USAGE",
-            message: "Primary tag is required when deriving commit messages from task comments",
-        });
-    }
-    const status = (opts.statusTo?.trim().toLowerCase() ?? "status-transition").replaceAll(/\s+/g, "-");
-    return `${prefix} ${suffix} ${primary}: ${status}`;
-}
-function deriveCommitBodyFromComment(opts) {
-    const lines = [
-        `Task: ${opts.taskId}`,
-        `Primary: ${opts.primaryTag}`,
-        ...(opts.executorAgent ? [`Agent: ${opts.executorAgent}`] : []),
-        ...(opts.author ? [`Author: ${opts.author}`] : []),
-        ...(opts.statusTo ? [`Status: ${opts.statusTo}`] : []),
-        `Comment: ${normalizeCommentBodyForCommit(opts.formattedComment ?? opts.commentBody)}`,
-    ];
-    return lines.join("\n").trimEnd();
-}
-export async function commitFromComment(opts) {
-    const ctx = opts.ctx ??
-        (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
-    if (opts.autoAllow) {
-        throw new CliError({
-            exitCode: 2,
-            code: "E_USAGE",
-            message: "--commit-auto-allow is disabled; pass explicit --commit-allow <path-prefix>.",
-        });
-    }
-    let allowPrefixes = opts.allow.map((prefix) => prefix.trim()).filter(Boolean);
-    if (allowPrefixes.length === 0) {
-        throw new CliError({
-            exitCode: 2,
-            code: "E_USAGE",
-            message: "Provide at least one --commit-allow prefix",
-        });
-    }
-    const staged = await stageAllowlist({
-        ctx,
-        allow: allowPrefixes,
-        allowTasks: opts.allowTasks,
-        tasksPath: opts.config.paths.tasks_path,
-        workflowDir: opts.config.paths.workflow_dir,
-        taskId: opts.taskId,
-    });
-    const message = deriveCommitMessageFromComment({
-        taskId: opts.taskId,
-        primaryTag: opts.primaryTag,
-        statusTo: opts.statusTo,
-        emoji: opts.emoji,
-    });
-    const formattedComment = opts.formattedComment ?? formatCommentBodyForCommit(opts.commentBody, opts.config);
-    const body = deriveCommitBodyFromComment({
-        taskId: opts.taskId,
-        primaryTag: opts.primaryTag,
-        executorAgent: opts.executorAgent,
-        author: opts.author,
-        statusFrom: opts.statusFrom,
-        statusTo: opts.statusTo,
-        commentBody: opts.commentBody,
-        formattedComment,
-    });
-    await guardCommitCheck({
-        ctx,
-        cwd: opts.cwd,
-        rootOverride: opts.rootOverride,
-        taskId: opts.taskId,
-        message,
-        allow: allowPrefixes,
-        allowBase: false,
-        allowTasks: opts.allowTasks,
-        allowPolicy: false,
-        allowConfig: false,
-        allowHooks: false,
-        allowCI: false,
-        requireClean: opts.requireClean,
-        quiet: opts.quiet,
-    });
-    // Never allow base-branch code commits implicitly from comment-driven commits.
-    // Base overrides must be explicit via the `commit` command's --allow-base flag.
-    const env = buildGitCommitEnv({
-        taskId: opts.taskId,
-        agentId: opts.executorAgent,
-        statusTo: opts.statusTo,
-        allowTasks: opts.allowTasks,
-        allowBase: false,
-        allowPolicy: false,
-        allowConfig: false,
-        allowHooks: false,
-        allowCI: false,
-        gitIdentity: await resolveCanonicalGitIdentity(),
-    });
-    await ctx.git.commit({ message, body, env });
-    const { hash, subject } = await ctx.git.headHashSubject();
-    if (!opts.quiet) {
-        process.stdout.write(`${successMessage("committed", `${hash?.slice(0, 12) ?? ""} ${subject ?? ""}`.trim(), `staged=${staged.join(", ")}`)}\n`);
-    }
-    return { hash, message: subject, staged };
-}

package/dist/commands/guard/impl/commit-diagnostics.d.ts

@@ -1,4 +0,0 @@
-import { CliError } from "../../../shared/errors.js";
-export type CommitFailurePhase = "task_commit" | "close_commit";
-export declare function asCommitFailure(err: unknown, phase: CommitFailurePhase): CliError | null;
-//# sourceMappingURL=commit-diagnostics.d.ts.map

package/dist/commands/guard/impl/commit-diagnostics.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"commit-diagnostics.d.ts","sourceRoot":"","sources":["../../../../src/commands/guard/impl/commit-diagnostics.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,QAAQ,EAAE,MAAM,2BAA2B,CAAC;AAWrD,MAAM,MAAM,kBAAkB,GAAG,aAAa,GAAG,cAAc,CAAC;AAwIhE,wBAAgB,eAAe,CAAC,GAAG,EAAE,OAAO,EAAE,KAAK,EAAE,kBAAkB,GAAG,QAAQ,GAAG,IAAI,CAgCxF"}

package/dist/commands/guard/impl/commit-diagnostics.js

@@ -1,150 +0,0 @@
-import { stripAnsi } from "../../../cli/shared/ansi.js";
-import { withDiagnosticContext } from "../../shared/diagnostics.js";
-import { CliError } from "../../../shared/errors.js";
-const COMMIT_FAILURE_SIGNAL_PATTERNS = [
-    /Code style issues found/i,
-    /Run Prettier with --write/i,
-    /\bESLint\b/i,
-    /\b[0-9]+\s+problems?\b/i,
-    /\berror\b/i,
-    /\bfailed\b/i,
-    /✖/,
-];
-const FORMATTER_SIGNAL_PATTERNS = [
-    /Code style issues found/i,
-    /Run Prettier with --write/i,
-];
-const ESLINT_SIGNAL_PATTERNS = [/\bESLint\b/i, /\b[0-9]+\s+problems?\b/i];
-function readText(value) {
-    if (typeof value === "string")
-        return value;
-    if (Buffer.isBuffer(value))
-        return value.toString("utf8");
-    return "";
-}
-function summarizeOutput(raw) {
-    const lines = raw
-        .replaceAll("\r\n", "\n")
-        .split("\n")
-        .map((line) => stripAnsi(line).trimEnd())
-        .filter((line) => line.trim().length > 0)
-        .map((line) => (line.length > 180 ? `${line.slice(0, 180)} [truncated]` : line));
-    if (lines.length <= 12)
-        return lines;
-    const selected = new Set();
-    for (let index = 0; index < Math.min(6, lines.length); index += 1)
-        selected.add(index);
-    for (let index = Math.max(lines.length - 6, 0); index < lines.length; index += 1) {
-        selected.add(index);
-    }
-    for (const [index, line] of lines.entries()) {
-        if (!selected.has(index) &&
-            COMMIT_FAILURE_SIGNAL_PATTERNS.some((pattern) => pattern.test(line))) {
-            selected.add(index);
-        }
-    }
-    const summary = [];
-    let previous = -1;
-    for (const index of [...selected].toSorted((a, b) => a - b)) {
-        if (previous >= 0 && index - previous > 1) {
-            summary.push(`[${index - previous - 1} lines omitted]`);
-        }
-        summary.push(lines[index] ?? "");
-        previous = index;
-    }
-    return summary;
-}
-function detectCommitFailureSignal(output) {
-    if (FORMATTER_SIGNAL_PATTERNS.some((pattern) => pattern.test(output)))
-        return "formatter";
-    if (ESLINT_SIGNAL_PATTERNS.some((pattern) => pattern.test(output)))
-        return "eslint";
-    return null;
-}
-function commitFailureDiagnostic(phase, output) {
-    const signal = detectCommitFailureSignal(output);
-    if (signal === "formatter") {
-        return {
-            state: phase === "close_commit"
-                ? "git rejected the generated close commit"
-                : "git rejected the requested task-scoped commit",
-            likelyCause: phase === "close_commit"
-                ? "a formatting check in the pre-commit path rejected the deterministic close commit after the task README was staged"
-                : "a formatting check in the pre-commit path rejected the staged task-scoped commit",
-            nextAction: {
-                command: "bun run format",
-                reason: "apply formatter fixes before retrying the commit flow",
-                reasonCode: "git_pre_commit_format",
-            },
-        };
-    }
-    if (signal === "eslint") {
-        return {
-            state: phase === "close_commit"
-                ? "git rejected the generated close commit"
-                : "git rejected the requested task-scoped commit",
-            likelyCause: phase === "close_commit"
-                ? "a lint check in the pre-commit path rejected the deterministic close commit after the task README was staged"
-                : "a lint check in the pre-commit path rejected the staged task-scoped commit",
-            nextAction: {
-                command: "bun run lint:core",
-                reason: "rerun lint and fix the reported error before retrying the commit flow",
-                reasonCode: "git_pre_commit_lint",
-            },
-        };
-    }
-    if (phase === "close_commit") {
-        return {
-            state: "git rejected the generated close commit",
-            likelyCause: "a hook or commit policy blocked the deterministic task close commit after the task README was staged",
-            nextAction: {
-                command: "git status --short --untracked-files=no",
-                reason: "inspect the staged close-commit payload before fixing the hook or policy failure",
-                reasonCode: "git_close_commit_blocked",
-            },
-        };
-    }
-    return {
-        state: "git rejected the requested task-scoped commit",
-        likelyCause: "a hook or commit policy blocked the staged changes after guard validation passed",
-        nextAction: {
-            command: "git status --short --untracked-files=no",
-            reason: "inspect the staged task-scoped payload before fixing the hook or policy failure",
-            reasonCode: "git_task_commit_blocked",
-        },
-    };
-}
-export function asCommitFailure(err, phase) {
-    if (!(err instanceof Error))
-        return null;
-    const e = err;
-    const shortMessage = typeof e.shortMessage === "string" ? e.shortMessage : "";
-    const message = typeof e.message === "string" ? e.message : "";
-    const extractedCommand = typeof e.cmd === "string"
-        ? e.cmd
-        : typeof e.escapedCommand === "string"
-            ? e.escapedCommand
-            : (/^Command failed(?: with exit code \d+)?: ([^\n]+)$/m.exec(shortMessage)?.[1] ??
-                /^Command failed(?: with exit code \d+)?: ([^\n]+)$/m.exec(message)?.[1] ??
-                "");
-    const cmd = extractedCommand.trim();
-    if (!cmd.startsWith("git commit"))
-        return null;
-    const output = [readText(e.stderr), readText(e.stdout)]
-        .filter((part) => part.length > 0)
-        .join("\n");
-    const summary = summarizeOutput(output);
-    const code = typeof e.code === "number" ? e.code : null;
-    const lines = ["git commit failed (hook or commit policy).", `command: ${cmd}`];
-    if (typeof code === "number")
-        lines.push(`exit_code: ${code}`);
-    if (summary.length > 0)
-        lines.push("output_summary:");
-    lines.push(...summary.map((line) => `  ${line}`));
-    return new CliError({
-        exitCode: 5,
-        code: "E_GIT",
-        message: lines.join("\n"),
-        context: withDiagnosticContext({ command: "commit" }, commitFailureDiagnostic(phase, output)),
-    });
-}

package/dist/commands/guard/impl/commit.d.ts

@@ -1,25 +0,0 @@
-import { type CommandContext } from "../../shared/task-backend.js";
-export declare function cmdCommit(opts: {
-    ctx?: CommandContext;
-    cwd: string;
-    rootOverride?: string;
-    baseBranchOverride?: string | null;
-    taskId: string;
-    message: string;
-    close: boolean;
-    allow: string[];
-    autoAllow: boolean;
-    allowTasks: boolean;
-    allowBase: boolean;
-    allowPolicy: boolean;
-    allowConfig: boolean;
-    allowHooks: boolean;
-    allowCI: boolean;
-    requireClean: boolean;
-    quiet: boolean;
-    closeUnstageOthers: boolean;
-    closeCheckOnly: boolean;
-    closeStageTaskArtifacts?: boolean;
-    closeRefreshTaskArtifacts?: boolean;
-}): Promise<number>;
-//# sourceMappingURL=commit.d.ts.map

package/dist/commands/guard/impl/commit.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"commit.d.ts","sourceRoot":"","sources":["../../../../src/commands/guard/impl/commit.ts"],"names":[],"mappings":"AAaA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,8BAA8B,CAAC;AAwLtC,wBAAsB,SAAS,CAAC,IAAI,EAAE;IACpC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,kBAAkB,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACnC,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,OAAO,CAAC;IACf,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,SAAS,EAAE,OAAO,CAAC;IACnB,UAAU,EAAE,OAAO,CAAC;IACpB,SAAS,EAAE,OAAO,CAAC;IACnB,WAAW,EAAE,OAAO,CAAC;IACrB,WAAW,EAAE,OAAO,CAAC;IACrB,UAAU,EAAE,OAAO,CAAC;IACpB,OAAO,EAAE,OAAO,CAAC;IACjB,YAAY,EAAE,OAAO,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;IACf,kBAAkB,EAAE,OAAO,CAAC;IAC5B,cAAc,EAAE,OAAO,CAAC;IACxB,uBAAuB,CAAC,EAAE,OAAO,CAAC;IAClC,yBAAyB,CAAC,EAAE,OAAO,CAAC;CACrC,GAAG,OAAO,CAAC,MAAM,CAAC,CAwIlB"}