agentplane 0.1.8 → 0.2.0

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

@@ -0,0 +1,137 @@
+import { extractTaskSuffix } from "@agentplaneorg/core";
+import { invalidValueMessage, successMessage } from "../../../cli/output.js";
+import { CliError } from "../../../shared/errors.js";
+import { formatCommentBodyForCommit, normalizeCommentBodyForCommit, } from "../../../shared/comment-format.js";
+import { protectedPathKindForFile } from "../../../shared/protected-paths.js";
+import { loadCommandContext } from "../../shared/task-backend.js";
+import { buildGitCommitEnv } from "./env.js";
+import { stageAllowlist, suggestAllowPrefixes } from "./allow.js";
+import { guardCommitCheck } from "./policy.js";
+function deriveCommitMessageFromComment(opts) {
+    const raw = (opts.formattedComment ?? formatCommentBodyForCommit(opts.body, opts.config))
+        .trim()
+        .replaceAll(/\s+/g, " ");
+    if (!raw) {
+        throw new CliError({
+            exitCode: 2,
+            code: "E_USAGE",
+            message: "Comment body is required to build a commit message from the task comment",
+        });
+    }
+    const summary = raw.replace(/^(start|blocked|verified):\s+/i, "").trim();
+    if (!summary) {
+        throw new CliError({
+            exitCode: 2,
+            code: "E_USAGE",
+            message: "Comment body is required to build a commit message from the task comment",
+        });
+    }
+    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"),
+        });
+    }
+    return `${prefix} ${suffix} task: ${summary}`;
+}
+function deriveCommitBodyFromComment(opts) {
+    const lines = [
+        `Task: ${opts.taskId}`,
+        ...(opts.author ? [`Agent: ${opts.author}`] : []),
+        ...(opts.statusFrom && opts.statusTo ? [`Status: ${opts.statusFrom} -> ${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 }));
+    let allowPrefixes = opts.allow.map((prefix) => prefix.trim()).filter(Boolean);
+    if (opts.autoAllow && allowPrefixes.length === 0) {
+        const changed = await ctx.git.statusChangedPaths();
+        const tasksPath = opts.config.paths.tasks_path;
+        // Auto-allow is for ergonomic status commits. It must never silently
+        // broaden into policy/config/CI changes (those require explicit intent).
+        const eligible = changed.filter((filePath) => {
+            const kind = protectedPathKindForFile({ filePath, tasksPath });
+            if (!kind)
+                return true;
+            if (kind === "tasks")
+                return opts.allowTasks;
+            return false;
+        });
+        allowPrefixes = suggestAllowPrefixes(eligible);
+    }
+    if (allowPrefixes.length === 0) {
+        throw new CliError({
+            exitCode: 2,
+            code: "E_USAGE",
+            message: "Provide at least one --commit-allow prefix or enable --commit-auto-allow",
+        });
+    }
+    const staged = await stageAllowlist({
+        ctx,
+        allow: allowPrefixes,
+        allowTasks: opts.allowTasks,
+        tasksPath: opts.config.paths.tasks_path,
+    });
+    const message = deriveCommitMessageFromComment({
+        taskId: opts.taskId,
+        body: opts.commentBody,
+        emoji: opts.emoji,
+        formattedComment: opts.formattedComment,
+        config: opts.config,
+    });
+    const formattedComment = opts.formattedComment ?? formatCommentBodyForCommit(opts.commentBody, opts.config);
+    const body = deriveCommitBodyFromComment({
+        taskId: opts.taskId,
+        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,
+        allowTasks: opts.allowTasks,
+        allowBase: false,
+        allowPolicy: false,
+        allowConfig: false,
+        allowHooks: false,
+        allowCI: false,
+    });
+    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/env.d.ts

@@ -0,0 +1,10 @@
+export declare function buildGitCommitEnv(opts: {
+    taskId: string;
+    allowTasks: boolean;
+    allowBase: boolean;
+    allowPolicy: boolean;
+    allowConfig: boolean;
+    allowHooks: boolean;
+    allowCI: boolean;
+}): NodeJS.ProcessEnv;
+//# sourceMappingURL=env.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"env.d.ts","sourceRoot":"","sources":["../../../../src/commands/guard/impl/env.ts"],"names":[],"mappings":"AAAA,wBAAgB,iBAAiB,CAAC,IAAI,EAAE;IACtC,MAAM,EAAE,MAAM,CAAC;IACf,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;CAClB,GAAG,MAAM,CAAC,UAAU,CAWpB"}

package/dist/commands/guard/impl/env.js

@@ -0,0 +1,12 @@
+export function buildGitCommitEnv(opts) {
+    return {
+        ...process.env,
+        AGENTPLANE_TASK_ID: opts.taskId,
+        AGENTPLANE_ALLOW_TASKS: opts.allowTasks ? "1" : "0",
+        AGENTPLANE_ALLOW_BASE: opts.allowBase ? "1" : "0",
+        AGENTPLANE_ALLOW_POLICY: opts.allowPolicy ? "1" : "0",
+        AGENTPLANE_ALLOW_CONFIG: opts.allowConfig ? "1" : "0",
+        AGENTPLANE_ALLOW_HOOKS: opts.allowHooks ? "1" : "0",
+        AGENTPLANE_ALLOW_CI: opts.allowCI ? "1" : "0",
+    };
+}

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

@@ -0,0 +1,19 @@
+import { type CommandContext } from "../../shared/task-backend.js";
+export type GuardCommitOptions = {
+    ctx?: CommandContext;
+    cwd: string;
+    rootOverride?: string;
+    taskId: string;
+    message: string;
+    allow: string[];
+    allowBase: boolean;
+    allowTasks: boolean;
+    allowPolicy: boolean;
+    allowConfig: boolean;
+    allowHooks: boolean;
+    allowCI: boolean;
+    requireClean: boolean;
+    quiet: boolean;
+};
+export declare function guardCommitCheck(opts: GuardCommitOptions): Promise<void>;
+//# sourceMappingURL=policy.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"policy.d.ts","sourceRoot":"","sources":["../../../../src/commands/guard/impl/policy.ts"],"names":[],"mappings":"AAKA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAEvF,MAAM,MAAM,kBAAkB,GAAG;IAC/B,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,SAAS,EAAE,OAAO,CAAC;IACnB,UAAU,EAAE,OAAO,CAAC;IACpB,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;CAChB,CAAC;AAEF,wBAAsB,gBAAgB,CAAC,IAAI,EAAE,kBAAkB,GAAG,OAAO,CAAC,IAAI,CAAC,CA4C9E"}

package/dist/commands/guard/impl/policy.js

@@ -0,0 +1,46 @@
+import { resolveBaseBranch } from "@agentplaneorg/core";
+import { evaluatePolicy } from "../../../policy/evaluate.js";
+import { gitCurrentBranch } from "../../shared/git-ops.js";
+import { throwIfPolicyDenied } from "../../shared/policy-deny.js";
+import { loadCommandContext } from "../../shared/task-backend.js";
+export async function guardCommitCheck(opts) {
+    const ctx = opts.ctx ??
+        (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
+    const staged = await ctx.git.statusStagedPaths();
+    const unstagedTrackedPaths = opts.requireClean ? await ctx.git.statusUnstagedTrackedPaths() : [];
+    const inBranchPr = ctx.config.workflow_mode === "branch_pr";
+    const baseBranch = inBranchPr
+        ? await resolveBaseBranch({
+            cwd: opts.cwd,
+            rootOverride: opts.rootOverride ?? null,
+            cliBaseOpt: null,
+            mode: ctx.config.workflow_mode,
+        })
+        : null;
+    const currentBranch = inBranchPr
+        ? await gitCurrentBranch(ctx.resolvedProject.gitRoot)
+        : undefined;
+    const res = evaluatePolicy({
+        action: "guard_commit",
+        config: ctx.config,
+        taskId: opts.taskId,
+        git: {
+            stagedPaths: staged,
+            unstagedTrackedPaths: unstagedTrackedPaths,
+            currentBranch,
+            baseBranch,
+        },
+        commit: { subject: opts.message },
+        allow: {
+            prefixes: opts.allow,
+            allowTasks: opts.allowTasks,
+            allowBase: opts.allowBase,
+            allowPolicy: opts.allowPolicy,
+            allowConfig: opts.allowConfig,
+            allowHooks: opts.allowHooks,
+            allowCI: opts.allowCI,
+        },
+        requireClean: opts.requireClean,
+    });
+    throwIfPolicyDenied(res);
+}

package/dist/commands/guard/index.d.ts

@@ -1,67 +1,5 @@
-import { loadConfig } from "@agentplaneorg/core";
-export declare function suggestAllowPrefixes(paths: string[]): string[];
-export declare const GUARD_COMMIT_USAGE = "Usage: agentplane guard commit <task-id> -m <message> --allow <path> [--allow <path>...] [--auto-allow] [--allow-tasks] [--require-clean] [--quiet]";
-export declare const GUARD_COMMIT_USAGE_EXAMPLE = "agentplane guard commit 202602030608-F1Q8AB -m \"\u2728 F1Q8AB update\" --allow packages/agentplane";
-export declare const COMMIT_USAGE = "Usage: agentplane commit <task-id> -m <message>";
-export declare const COMMIT_USAGE_EXAMPLE = "agentplane commit 202602030608-F1Q8AB -m \"\u2728 F1Q8AB update\"";
-type GuardCommitOptions = {
-    cwd: string;
-    rootOverride?: string;
-    taskId: string;
-    message: string;
-    allow: string[];
-    allowTasks: boolean;
-    requireClean: boolean;
-    quiet: boolean;
-};
-export declare function gitStatusChangedPaths(opts: {
-    cwd: string;
-    rootOverride?: string;
-}): Promise<string[]>;
-export declare function ensureGitClean(opts: {
-    cwd: string;
-    rootOverride?: string;
-}): Promise<void>;
-export declare function commitFromComment(opts: {
-    cwd: string;
-    rootOverride?: string;
-    taskId: string;
-    commentBody: string;
-    formattedComment: string | null;
-    emoji: string;
-    allow: string[];
-    autoAllow: boolean;
-    allowTasks: boolean;
-    requireClean: boolean;
-    quiet: boolean;
-    config: Awaited<ReturnType<typeof loadConfig>>["config"];
-}): Promise<{
-    hash: string;
-    message: string;
-    staged: string[];
-}>;
-export declare function cmdGuardClean(opts: {
-    cwd: string;
-    rootOverride?: string;
-    quiet: boolean;
-}): Promise<number>;
-export declare function cmdGuardSuggestAllow(opts: {
-    cwd: string;
-    rootOverride?: string;
-    format: "lines" | "args";
-}): Promise<number>;
-export declare function cmdGuardCommit(opts: GuardCommitOptions): Promise<number>;
-export declare function cmdCommit(opts: {
-    cwd: string;
-    rootOverride?: string;
-    taskId: string;
-    message: string;
-    allow: string[];
-    autoAllow: boolean;
-    allowTasks: boolean;
-    allowBase: boolean;
-    requireClean: boolean;
-    quiet: boolean;
-}): Promise<number>;
-export {};
+export { buildGitCommitEnv } from "./impl/env.js";
+export { ensureGitClean, gitStatusChangedPaths, suggestAllowPrefixes } from "./impl/allow.js";
+export { commitFromComment } from "./impl/comment-commit.js";
+export { cmdCommit, cmdGuardClean, cmdGuardCommit, cmdGuardSuggestAllow } from "./impl/commands.js";
 //# sourceMappingURL=index.d.ts.map

package/dist/commands/guard/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/commands/guard/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAIL,UAAU,EAGX,MAAM,qBAAqB,CAAC;AAkB7B,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,MAAM,EAAE,CAS9D;AAED,eAAO,MAAM,kBAAkB,wJACwH,CAAC;AACxJ,eAAO,MAAM,0BAA0B,wGACyD,CAAC;AACjG,eAAO,MAAM,YAAY,oDAAoD,CAAC;AAC9E,eAAO,MAAM,oBAAoB,sEAA+D,CAAC;AAEjG,KAAK,kBAAkB,GAAG;IACxB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;CAChB,CAAC;AAiEF,wBAAsB,qBAAqB,CAAC,IAAI,EAAE;IAChD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAkBpB;AAED,wBAAsB,cAAc,CAAC,IAAI,EAAE;IAAE,GAAG,EAAE,MAAM,CAAC;IAAC,YAAY,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAgBhG;AAqFD,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,gBAAgB,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,SAAS,EAAE,OAAO,CAAC;IACnB,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,OAAO,CAAC,UAAU,CAAC,OAAO,UAAU,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC;CAC1D,GAAG,OAAO,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,EAAE,CAAA;CAAE,CAAC,CAmE/D;AAED,wBAAsB,aAAa,CAAC,IAAI,EAAE;IACxC,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAkBlB;AAED,wBAAsB,oBAAoB,CAAC,IAAI,EAAE;IAC/C,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,OAAO,GAAG,MAAM,CAAC;CAC1B,GAAG,OAAO,CAAC,MAAM,CAAC,CAqBlB;AAED,wBAAsB,cAAc,CAAC,IAAI,EAAE,kBAAkB,GAAG,OAAO,CAAC,MAAM,CAAC,CAS9E;AAED,wBAAsB,SAAS,CAAC,IAAI,EAAE;IACpC,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,SAAS,EAAE,OAAO,CAAC;IACnB,UAAU,EAAE,OAAO,CAAC;IACpB,SAAS,EAAE,OAAO,CAAC;IACnB,YAAY,EAAE,OAAO,CAAC;IACtB,KAAK,EAAE,OAAO,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,CAAC,CAyDlB"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/commands/guard/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,MAAM,eAAe,CAAC;AAElD,OAAO,EAAE,cAAc,EAAE,qBAAqB,EAAE,oBAAoB,EAAE,MAAM,iBAAiB,CAAC;AAE9F,OAAO,EAAE,iBAAiB,EAAE,MAAM,0BAA0B,CAAC;AAE7D,OAAO,EAAE,SAAS,EAAE,aAAa,EAAE,cAAc,EAAE,oBAAoB,EAAE,MAAM,oBAAoB,CAAC"}