agentplane 0.3.15 → 0.3.17

package/dist/commands/hooks/run.d.ts

@@ -0,0 +1,9 @@
+import type { HookName } from "./shared.js";
+export type HooksRunOptions = {
+    cwd: string;
+    rootOverride?: string;
+    hook: HookName;
+    args: string[];
+};
+export declare function cmdHooksRun(opts: HooksRunOptions): Promise<number>;
+//# sourceMappingURL=run.d.ts.map

package/dist/commands/hooks/run.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"run.d.ts","sourceRoot":"","sources":["../../../src/commands/hooks/run.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AAM5C,MAAM,MAAM,eAAe,GAAG;IAC5B,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,IAAI,EAAE,QAAQ,CAAC;IACf,IAAI,EAAE,MAAM,EAAE,CAAC;CAChB,CAAC;AAmBF,wBAAsB,WAAW,CAAC,IAAI,EAAE,eAAe,GAAG,OAAO,CAAC,MAAM,CAAC,CAkBxE"}

package/dist/commands/hooks/run.js

@@ -0,0 +1,45 @@
+import { mapBackendError } from "../../cli/error-map.js";
+import { CliError } from "../../shared/errors.js";
+import { runCommitMsgHook } from "./run.commit-msg.js";
+import { runPostMergeHook } from "./run.post-merge.js";
+import { runPreCommitHook } from "./run.pre-commit.js";
+import { runPrePushHook } from "./run.pre-push.js";
+async function runHook(opts) {
+    switch (opts.hook) {
+        case "commit-msg": {
+            return await runCommitMsgHook(opts);
+        }
+        case "pre-commit": {
+            return await runPreCommitHook(opts);
+        }
+        case "pre-push": {
+            return await runPrePushHook(opts);
+        }
+        case "post-merge": {
+            return await runPostMergeHook(opts);
+        }
+    }
+}
+export async function cmdHooksRun(opts) {
+    try {
+        return await runHook(opts);
+    }
+    catch (err) {
+        const status = err?.status;
+        const stdout = err?.stdout;
+        const stderr = err?.stderr;
+        if (typeof stdout === "string" && stdout.length > 0)
+            process.stdout.write(stdout);
+        if (typeof stderr === "string" && stderr.length > 0)
+            process.stderr.write(stderr);
+        if (typeof status === "number" && Number.isInteger(status) && status >= 0) {
+            return status;
+        }
+        if (err instanceof CliError)
+            throw err;
+        throw mapBackendError(err, {
+            command: `hooks run ${opts.hook}`,
+            root: opts.rootOverride ?? null,
+        });
+    }
+}

package/dist/commands/hooks/run.post-merge.d.ts

@@ -0,0 +1,3 @@
+import type { HooksRunOptions } from "./run.js";
+export declare function runPostMergeHook(opts: HooksRunOptions): Promise<number>;
+//# sourceMappingURL=run.post-merge.d.ts.map

package/dist/commands/hooks/run.post-merge.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"run.post-merge.d.ts","sourceRoot":"","sources":["../../../src/commands/hooks/run.post-merge.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,UAAU,CAAC;AAEhD,wBAAsB,gBAAgB,CAAC,IAAI,EAAE,eAAe,GAAG,OAAO,CAAC,MAAM,CAAC,CAuC7E"}

package/dist/commands/hooks/run.post-merge.js

@@ -0,0 +1,44 @@
+import { resolveBaseBranch } from "@agentplaneorg/core";
+import { cmdCleanupMerged } from "../branch/index.js";
+import { gitCurrentBranch } from "../shared/git-ops.js";
+import { loadCommandContext } from "../shared/task-backend.js";
+export async function runPostMergeHook(opts) {
+    try {
+        const ctx = await loadCommandContext({
+            cwd: opts.cwd,
+            rootOverride: opts.rootOverride ?? null,
+        });
+        if (ctx.config.workflow_mode !== "branch_pr")
+            return 0;
+        const baseBranch = await resolveBaseBranch({
+            cwd: opts.cwd,
+            rootOverride: opts.rootOverride ?? null,
+            cliBaseOpt: null,
+            mode: ctx.config.workflow_mode,
+        });
+        if (!baseBranch)
+            return 0;
+        const currentBranch = await gitCurrentBranch(ctx.resolvedProject.gitRoot);
+        if (currentBranch !== baseBranch)
+            return 0;
+        return await cmdCleanupMerged({
+            ctx,
+            cwd: opts.cwd,
+            rootOverride: opts.rootOverride,
+            base: baseBranch,
+            yes: true,
+            archive: false,
+            deleteRemoteBranches: false,
+            fetch: false,
+            quiet: true,
+            skipUnsafeWorktrees: true,
+        });
+    }
+    catch (error) {
+        const message = error instanceof Error && error.message.trim().length > 0
+            ? error.message.trim()
+            : String(error);
+        process.stderr.write(`warning: post-merge cleanup skipped: ${message}\n`);
+        return 0;
+    }
+}

package/dist/commands/hooks/run.pre-commit.d.ts

@@ -0,0 +1,3 @@
+import type { HooksRunOptions } from "./run.js";
+export declare function runPreCommitHook(opts: HooksRunOptions): Promise<number>;
+//# sourceMappingURL=run.pre-commit.d.ts.map

package/dist/commands/hooks/run.pre-commit.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"run.pre-commit.d.ts","sourceRoot":"","sources":["../../../src/commands/hooks/run.pre-commit.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,UAAU,CAAC;AAMhD,wBAAsB,gBAAgB,CAAC,IAAI,EAAE,eAAe,GAAG,OAAO,CAAC,MAAM,CAAC,CAwC7E"}

package/dist/commands/hooks/run.pre-commit.js

@@ -0,0 +1,48 @@
+import { loadConfig, resolveBaseBranch, resolveProject } from "@agentplaneorg/core";
+import { evaluatePolicy } from "../../policy/evaluate.js";
+import { GitContext } from "../shared/git-context.js";
+import { gitCurrentBranch } from "../shared/git-ops.js";
+import { throwIfPolicyDenied } from "../shared/policy-deny.js";
+function envFlag(name) {
+    return (process.env[name] ?? "").trim() === "1";
+}
+export async function runPreCommitHook(opts) {
+    const resolved = await resolveProject({
+        cwd: opts.cwd,
+        rootOverride: opts.rootOverride ?? null,
+    });
+    const staged = await new GitContext({ gitRoot: resolved.gitRoot }).statusStagedPaths();
+    if (staged.length === 0)
+        return 0;
+    const loaded = await loadConfig(resolved.agentplaneDir);
+    const inBranchPr = loaded.config.workflow_mode === "branch_pr";
+    const baseBranch = inBranchPr
+        ? await resolveBaseBranch({
+            cwd: opts.cwd,
+            rootOverride: opts.rootOverride ?? null,
+            cliBaseOpt: null,
+            mode: loaded.config.workflow_mode,
+        })
+        : null;
+    const currentBranch = inBranchPr ? await gitCurrentBranch(resolved.gitRoot) : undefined;
+    const res = evaluatePolicy({
+        action: "hook_pre_commit",
+        config: loaded.config,
+        taskId: (process.env.AGENTPLANE_TASK_ID ?? "").trim(),
+        git: {
+            stagedPaths: staged,
+            currentBranch,
+            baseBranch,
+        },
+        allow: {
+            allowTasks: envFlag("AGENTPLANE_ALLOW_TASKS"),
+            allowBase: envFlag("AGENTPLANE_ALLOW_BASE"),
+            allowPolicy: envFlag("AGENTPLANE_ALLOW_POLICY"),
+            allowConfig: envFlag("AGENTPLANE_ALLOW_CONFIG"),
+            allowHooks: envFlag("AGENTPLANE_ALLOW_HOOKS"),
+            allowCI: envFlag("AGENTPLANE_ALLOW_CI"),
+        },
+    });
+    throwIfPolicyDenied(res);
+    return 0;
+}

package/dist/commands/hooks/run.pre-push.d.ts

@@ -0,0 +1,6 @@
+import type { HooksRunOptions } from "./run.js";
+export declare function resolvePrePushHookScriptPath(gitRoot: string, opts?: {
+    bundledScriptPath?: string;
+}): Promise<string | null>;
+export declare function runPrePushHook(opts: HooksRunOptions): Promise<number>;
+//# sourceMappingURL=run.pre-push.d.ts.map

package/dist/commands/hooks/run.pre-push.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"run.pre-push.d.ts","sourceRoot":"","sources":["../../../src/commands/hooks/run.pre-push.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,UAAU,CAAC;AAMhD,wBAAsB,4BAA4B,CAChD,OAAO,EAAE,MAAM,EACf,IAAI,GAAE;IAAE,iBAAiB,CAAC,EAAE,MAAM,CAAA;CAAO,GACxC,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAMxB;AA4CD,wBAAsB,cAAc,CAAC,IAAI,EAAE,eAAe,GAAG,OAAO,CAAC,MAAM,CAAC,CAgC3E"}

package/dist/commands/hooks/run.pre-push.js

@@ -0,0 +1,88 @@
+import { runProcessSync, resolveProject } from "@agentplaneorg/core";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+import { fileExists } from "../../cli/fs-utils.js";
+import { CliError } from "../../shared/errors.js";
+function resolveBundledPrePushHookScriptPath() {
+    return fileURLToPath(new URL("../../../../../scripts/run-pre-push-hook.mjs", import.meta.url));
+}
+export async function resolvePrePushHookScriptPath(gitRoot, opts = {}) {
+    const repoScriptPath = path.join(gitRoot, "scripts", "run-pre-push-hook.mjs");
+    if (await fileExists(repoScriptPath))
+        return repoScriptPath;
+    const bundledScriptPath = opts.bundledScriptPath ?? resolveBundledPrePushHookScriptPath();
+    if (await fileExists(bundledScriptPath))
+        return bundledScriptPath;
+    return null;
+}
+async function readHookStdinUtf8(timeoutMs = 25) {
+    if (process.stdin.isTTY)
+        return "";
+    const chunks = [];
+    const consume = () => {
+        let chunk = process.stdin.read();
+        while (chunk !== null) {
+            chunks.push(Buffer.isBuffer(chunk) ? chunk : Buffer.from(String(chunk)));
+            chunk = process.stdin.read();
+        }
+    };
+    consume();
+    if (chunks.length > 0 || process.stdin.readableEnded) {
+        return Buffer.concat(chunks).toString("utf8");
+    }
+    await new Promise((resolve) => {
+        const finish = () => {
+            clearTimeout(timer);
+            process.stdin.off("readable", onReadable);
+            process.stdin.off("end", onEnd);
+            resolve();
+        };
+        const onReadable = () => {
+            consume();
+            finish();
+        };
+        const onEnd = () => {
+            consume();
+            finish();
+        };
+        const timer = setTimeout(finish, timeoutMs);
+        process.stdin.on("readable", onReadable);
+        process.stdin.on("end", onEnd);
+        process.stdin.resume();
+    });
+    consume();
+    return Buffer.concat(chunks).toString("utf8");
+}
+export async function runPrePushHook(opts) {
+    const resolved = await resolveProject({
+        cwd: opts.cwd,
+        rootOverride: opts.rootOverride ?? null,
+    });
+    const scriptPath = await resolvePrePushHookScriptPath(resolved.gitRoot);
+    if (!scriptPath) {
+        throw new CliError({
+            exitCode: 2,
+            code: "E_USAGE",
+            message: [
+                "Missing pre-push hook script: scripts/run-pre-push-hook.mjs",
+                "The pre-push hook needs a repository-local script or an installed CLI bundle that ships the fallback.",
+                "Fix:",
+                "  1) Restore scripts/run-pre-push-hook.mjs in this repository, or",
+                "  2) Run `agentplane hooks uninstall` if this repository should not use the agentplane pre-push gate.",
+            ].join("\n"),
+        });
+    }
+    const result = runProcessSync({
+        command: "node",
+        args: [scriptPath],
+        cwd: resolved.gitRoot,
+        env: process.env,
+        encoding: "utf8",
+        input: await readHookStdinUtf8(),
+        stdin: "pipe",
+        stdout: "inherit",
+        stderr: "inherit",
+        reject: false,
+    });
+    return result.exitCode ?? (result.signal ? 1 : 0);
+}

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

@@ -0,0 +1,7 @@
+export declare const HOOK_MARKER = "agentplane-hook";
+export declare const SHIM_MARKER = "agentplane-hook-shim";
+export declare const HOOK_NAMES: readonly ["commit-msg", "pre-commit", "pre-push", "post-merge"];
+export type HookName = (typeof HOOK_NAMES)[number];
+export declare function resolveGitHooksDir(cwd: string): Promise<string>;
+export declare function fileIsManaged(filePath: string, marker: string): Promise<boolean>;
+//# sourceMappingURL=shared.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"shared.d.ts","sourceRoot":"","sources":["../../../src/commands/hooks/shared.ts"],"names":[],"mappings":"AAOA,eAAO,MAAM,WAAW,oBAAoB,CAAC;AAC7C,eAAO,MAAM,WAAW,yBAAyB,CAAC;AAClD,eAAO,MAAM,UAAU,iEAAkE,CAAC;AAC1F,MAAM,MAAM,QAAQ,GAAG,CAAC,OAAO,UAAU,CAAC,CAAC,MAAM,CAAC,CAAC;AAEnD,wBAAsB,kBAAkB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CA4BrE;AAED,wBAAsB,aAAa,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAOtF"}

package/dist/commands/hooks/shared.js

@@ -0,0 +1,41 @@
+import { readFile } from "node:fs/promises";
+import path from "node:path";
+import { gitRevParse } from "../shared/git-ops.js";
+import { isPathWithin } from "../shared/path.js";
+import { CliError } from "../../shared/errors.js";
+export const HOOK_MARKER = "agentplane-hook";
+export const SHIM_MARKER = "agentplane-hook-shim";
+export const HOOK_NAMES = ["commit-msg", "pre-commit", "pre-push", "post-merge"];
+export async function resolveGitHooksDir(cwd) {
+    const repoRoot = await gitRevParse(cwd, ["--show-toplevel"]);
+    const commonDirRaw = await gitRevParse(cwd, ["--git-common-dir"]);
+    const hooksRaw = await gitRevParse(cwd, ["--git-path", "hooks"]);
+    const commonDir = path.resolve(path.isAbsolute(commonDirRaw) ? commonDirRaw : path.join(repoRoot, commonDirRaw));
+    const hooksDir = path.resolve(path.isAbsolute(hooksRaw) ? hooksRaw : path.join(repoRoot, hooksRaw));
+    const resolvedRoot = path.resolve(repoRoot);
+    if (!isPathWithin(resolvedRoot, hooksDir) && !isPathWithin(commonDir, hooksDir)) {
+        throw new CliError({
+            exitCode: 5,
+            code: "E_GIT",
+            message: [
+                "Refusing to manage git hooks outside the repository.",
+                `hooks_path=${hooksDir}`,
+                `repo_root=${resolvedRoot}`,
+                `common_dir=${commonDir}`,
+                "Fix:",
+                "  1) Use a repo-relative core.hooksPath (e.g., .git/hooks)",
+                "  2) Re-run `agentplane hooks install`",
+            ].join("\n"),
+        });
+    }
+    return hooksDir;
+}
+export async function fileIsManaged(filePath, marker) {
+    try {
+        const content = await readFile(filePath, "utf8");
+        return content.includes(marker);
+    }
+    catch {
+        return false;
+    }
+}

package/dist/commands/pr/integrate/cmd.js

@@ -2,7 +2,7 @@ import path from "node:path";
 import { mapBackendError } from "../../../cli/error-map.js";
 import { exitCodeForError } from "../../../cli/exit-codes.js";
 import { createCliEmitter } from "../../../cli/output.js";
-import { withDiagnosticContext } from "../../../shared/diagnostics.js";
+import { withDiagnosticContext } from "../../shared/diagnostics.js";
 import { CliError } from "../../../shared/errors.js";
 import { cleanupIntegratedBranch } from "./internal/cleanup.js";
 import { renderPostIntegrateBootstrapFailureGuidance, renderPostIntegrateBootstrapGuidance, shouldRecommendPostIntegrateBootstrap, } from "./internal/bootstrap-guidance.js";

package/dist/commands/pr/integrate/internal/prepare.js

@@ -4,7 +4,7 @@ import { resolveBaseBranch } from "@agentplaneorg/core";
 import { fileExists } from "../../../../cli/fs-utils.js";
 import { exitCodeForError } from "../../../../cli/exit-codes.js";
 import { unknownEntityMessage, workflowModeMessage } from "../../../../cli/output.js";
-import { withDiagnosticContext } from "../../../../shared/diagnostics.js";
+import { withDiagnosticContext } from "../../../shared/diagnostics.js";
 import { CliError } from "../../../../shared/errors.js";
 import { ensureGitClean } from "../../../guard/index.js";
 import { gitDiffNames } from "../../../shared/git-diff.js";

package/dist/commands/pr/internal/auto-commit.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"auto-commit.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/auto-commit.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAiCnE,wBAAsB,8BAA8B,CAAC,IAAI,EAAE;IACzD,GAAG,EAAE,cAAc,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,OAAO,CAAC,CA+CnB"}
+{"version":3,"file":"auto-commit.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/auto-commit.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AA6BnE,wBAAsB,8BAA8B,CAAC,IAAI,EAAE;IACzD,GAAG,EAAE,cAAc,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,OAAO,CAAC,CA+CnB"}

package/dist/commands/pr/internal/auto-commit.js

@@ -1,5 +1,5 @@
 import path from "node:path";
-import { extractTaskSuffix } from "@agentplaneorg/core";
+import { buildTaskArtifactRefreshCommitSubject } from "@agentplaneorg/core";
 import { buildGitCommitEnv, resolveCanonicalGitIdentity } from "../../guard/impl/env.js";
 import { toGitPath } from "../../shared/git-diff.js";
 import { execFileAsync, gitEnv } from "../../shared/git.js";
@@ -25,9 +25,6 @@ async function readCachedPaths(gitRoot) {
         .map((line) => line.trim())
         .filter((line) => line.length > 0);
 }
-function taskPrArtifactRefreshMessage(taskId) {
-    return `📝 ${extractTaskSuffix(taskId)} task: refresh PR artifacts`;
-}
 export async function maybeAutoCommitTaskPrArtifacts(opts) {
     if (opts.ctx.config.workflow_mode !== "branch_pr")
         return false;
@@ -53,7 +50,7 @@ export async function maybeAutoCommitTaskPrArtifacts(opts) {
     }
     await opts.ctx.git.stage(taskPacketPaths);
     await opts.ctx.git.commit({
-        message: taskPrArtifactRefreshMessage(opts.taskId),
+        message: buildTaskArtifactRefreshCommitSubject({ taskId: opts.taskId }),
         env: buildGitCommitEnv({
             taskId: opts.taskId,
             allowTasks: true,

package/dist/commands/pr/internal/gh-api.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"gh-api.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/gh-api.ts"],"names":[],"mappings":"AAqBA,wBAAgB,KAAK,IAAI,MAAM,CAAC,UAAU,CAiBzC;AAED,wBAAsB,wBAAwB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAe3E;AAED,wBAAsB,YAAY,CAAC,CAAC,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,CAAC,CAAC,CAa7E;AAED,wBAAsB,gBAAgB,CAAC,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAYjF;AAED,wBAAgB,YAAY,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CAElD"}
+{"version":3,"file":"gh-api.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/gh-api.ts"],"names":[],"mappings":"AAiBA,wBAAgB,KAAK,IAAI,MAAM,CAAC,UAAU,CAiBzC;AAED,wBAAsB,wBAAwB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAkB3E;AAED,wBAAsB,YAAY,CAAC,CAAC,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,CAAC,CAAC,CAgB7E;AAED,wBAAsB,gBAAgB,CAAC,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAejF;AAED,wBAAgB,YAAY,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CAElD"}

package/dist/commands/pr/internal/gh-api.js

@@ -1,11 +1,9 @@
-import { execFile } from "node:child_process";
-import { promisify } from "node:util";
+import { runProcess } from "@agentplaneorg/core";
 import { exitCodeForError } from "../../../cli/exit-codes.js";
 import { CliError } from "../../../shared/errors.js";
 import { isDotEnvLoadedKey } from "../../../shared/env.js";
 import { gitEnv } from "../../shared/git.js";
 import { normalizeGhTransportError, withGhTransportRetry } from "../../shared/gh-transport.js";
-const execFileAsyncRaw = promisify(execFile);
 function parseGithubRepoFromRemoteUrl(remoteUrl) {
     const trimmed = remoteUrl.trim();
     if (!trimmed)
@@ -41,12 +39,15 @@ export function ghEnv() {
     return env;
 }
 export async function resolveDefaultGithubRepo(cwd) {
-    const { stdout } = await execFileAsyncRaw("git", ["remote", "get-url", "origin"], {
+    const { stdout } = await runProcess({
+        command: "git",
+        args: ["remote", "get-url", "origin"],
         cwd,
         env: gitEnv(),
+        encoding: "utf8",
         maxBuffer: 10 * 1024 * 1024,
     });
-    const repo = parseGithubRepoFromRemoteUrl(stdout);
+    const repo = parseGithubRepoFromRemoteUrl(String(stdout));
     if (!repo) {
         throw new CliError({
             exitCode: exitCodeForError("E_VALIDATION"),
@@ -57,19 +58,25 @@ export async function resolveDefaultGithubRepo(cwd) {
     return repo;
 }
 export async function runGhApiJson(cwd, args) {
-    const { stdout } = await withGhTransportRetry(() => execFileAsyncRaw("gh", ["api", ...args], {
+    const { stdout } = await withGhTransportRetry(() => runProcess({
+        command: "gh",
+        args: ["api", ...args],
         cwd,
         env: ghEnv(),
+        encoding: "utf8",
         maxBuffer: 10 * 1024 * 1024,
     }), {
         label: `running gh api ${args[0] ?? ""}`,
     });
-    return JSON.parse(stdout);
+    return JSON.parse(String(stdout));
 }
 export async function runGhApiNoOutput(cwd, args) {
-    await withGhTransportRetry(() => execFileAsyncRaw("gh", ["api", ...args], {
+    await withGhTransportRetry(() => runProcess({
+        command: "gh",
+        args: ["api", ...args],
         cwd,
         env: ghEnv(),
+        encoding: "utf8",
         maxBuffer: 10 * 1024 * 1024,
     }), {
         label: `running gh api ${args[0] ?? ""}`,

package/dist/commands/pr/internal/sync-branch.d.ts

@@ -3,7 +3,6 @@ export type ResolvedPrSyncBranch = {
     source: "explicit" | "meta" | "current" | "none";
 };
 export declare function isUnknownRevisionError(err: unknown): boolean;
-export declare function taskPrArtifactRefreshMessage(taskId: string): string;
 export declare function resolveBranchHeadSha(opts: {
     gitRoot: string;
     branch: string;

package/dist/commands/pr/internal/sync-branch.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"sync-branch.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/sync-branch.ts"],"names":[],"mappings":"AAYA,MAAM,MAAM,oBAAoB,GAAG;IACjC,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,MAAM,EAAE,UAAU,GAAG,MAAM,GAAG,SAAS,GAAG,MAAM,CAAC;CAClD,CAAC;AAEF,wBAAgB,sBAAsB,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CAO5D;AAED,wBAAgB,4BAA4B,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAEnE;AAED,wBAAsB,oBAAoB,CAAC,IAAI,EAAE;IAC/C,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAWzB;AAED,wBAAsB,2BAA2B,CAAC,IAAI,EAAE;IACtD,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC;IAAE,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAAC,eAAe,EAAE,OAAO,CAAA;CAAE,CAAC,CAwBhE;AAED,wBAAsB,oBAAoB,CAAC,IAAI,EAAE;IAC/C,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,CAAC;CACpB,GAAG,OAAO,CAAC,MAAM,CAAC,CAclB;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;CACf,GAAG,OAAO,CAAC,MAAM,CAAC,CAiBlB;AAED,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,GAAG,OAAO,CAAC,oBAAoB,CAAC,CAqBhC;AAED,wBAAgB,wBAAwB,CACtC,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,MAAM,GACb,OAAO,CAET"}
+{"version":3,"file":"sync-branch.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/sync-branch.ts"],"names":[],"mappings":"AAYA,MAAM,MAAM,oBAAoB,GAAG;IACjC,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,MAAM,EAAE,UAAU,GAAG,MAAM,GAAG,SAAS,GAAG,MAAM,CAAC;CAClD,CAAC;AAEF,wBAAgB,sBAAsB,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CAO5D;AAED,wBAAsB,oBAAoB,CAAC,IAAI,EAAE;IAC/C,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAWzB;AAED,wBAAsB,2BAA2B,CAAC,IAAI,EAAE;IACtD,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC;IAAE,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAAC,eAAe,EAAE,OAAO,CAAA;CAAE,CAAC,CAwBhE;AAED,wBAAsB,oBAAoB,CAAC,IAAI,EAAE;IAC/C,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,CAAC;CACpB,GAAG,OAAO,CAAC,MAAM,CAAC,CAclB;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE;IAC5C,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;CACf,GAAG,OAAO,CAAC,MAAM,CAAC,CAiBlB;AAED,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,GAAG,OAAO,CAAC,oBAAoB,CAAC,CAqBhC;AAED,wBAAgB,wBAAwB,CACtC,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,MAAM,GACb,OAAO,CAET"}

package/dist/commands/pr/internal/sync-branch.js

@@ -1,6 +1,6 @@
 import { readFile } from "node:fs/promises";
 import path from "node:path";
-import { extractTaskSuffix } from "@agentplaneorg/core";
+import { isTaskArtifactRefreshCommitSubject } from "@agentplaneorg/core";
 import { fileExists } from "../../../cli/fs-utils.js";
 import { execFileAsync, gitEnv } from "../../shared/git.js";
 import { gitBranchUpstream, gitCurrentBranch } from "../../shared/git-ops.js";
@@ -13,9 +13,6 @@ export function isUnknownRevisionError(err) {
         /bad revision/i.test(message) ||
         /ambiguous argument/i.test(message));
 }
-export function taskPrArtifactRefreshMessage(taskId) {
-    return `📝 ${extractTaskSuffix(taskId)} task: refresh PR artifacts`;
-}
 export async function resolveBranchHeadSha(opts) {
     try {
         const { stdout } = await execFileAsync("git", ["rev-parse", opts.branch], {
@@ -45,7 +42,7 @@ export async function resolveRenderableBranchHead(opts) {
         const subject = subjectStdout.trim();
         return {
             headSha: branchHeadSha,
-            artifactRefresh: subject === taskPrArtifactRefreshMessage(opts.taskId),
+            artifactRefresh: isTaskArtifactRefreshCommitSubject({ subject, taskId: opts.taskId }),
         };
     }
     catch {

package/dist/commands/pr/internal/sync.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"sync.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/sync.ts"],"names":[],"mappings":"AAYA,OAAO,EAAyC,KAAK,MAAM,EAAE,MAAM,yBAAyB,CAAC;AAE7F,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,8BAA8B,CAAC;AAQtC,OAAO,KAAK,EACV,aAAa,EACb,YAAY,EAGb,MAAM,iBAAiB,CAAC;AAKzB,KAAK,UAAU,GAAG,MAAM,GAAG,QAAQ,CAAC;AACpC,YAAY,EAAE,aAAa,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AA+EnE,wBAAsB,uBAAuB,CAAC,IAAI,EAAE;IAClD,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,GAAG,OAAO,CAAC;IACV,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;CAC/B,GAAG,IAAI,CAAC,CAmDR;AAED,wBAAsB,eAAe,CAAC,IAAI,EAAE;IAC1C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,UAAU,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,UAAU,CAAC,EAAE,YAAY,CAAC;CAC3B,GAAG,OAAO,CAAC;IACV,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IAC9B,WAAW,CAAC,EAAE,aAAa,CAAC;CAC7B,CAAC,CAkHD"}
+{"version":3,"file":"sync.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/sync.ts"],"names":[],"mappings":"AAaA,OAAO,EAAyC,KAAK,MAAM,EAAE,MAAM,yBAAyB,CAAC;AAE7F,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,8BAA8B,CAAC;AAQtC,OAAO,KAAK,EACV,aAAa,EACb,YAAY,EAGb,MAAM,iBAAiB,CAAC;AAKzB,KAAK,UAAU,GAAG,MAAM,GAAG,QAAQ,CAAC;AACpC,YAAY,EAAE,aAAa,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AA+EnE,wBAAsB,uBAAuB,CAAC,IAAI,EAAE;IAClD,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,GAAG,OAAO,CAAC;IACV,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;CAC/B,GAAG,IAAI,CAAC,CAmDR;AAED,wBAAsB,eAAe,CAAC,IAAI,EAAE;IAC1C,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,UAAU,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,UAAU,CAAC,EAAE,YAAY,CAAC;CAC3B,GAAG,OAAO,CAAC;IACV,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IAC9B,WAAW,CAAC,EAAE,aAAa,CAAC;CAC7B,CAAC,CA+ID"}

package/dist/commands/pr/internal/sync.js

@@ -6,6 +6,7 @@ import { exitCodeForError } from "../../../cli/exit-codes.js";
 import { fileExists } from "../../../cli/fs-utils.js";
 import { workflowModeMessage } from "../../../cli/output.js";
 import { CliError } from "../../../shared/errors.js";
+import { emitTraceEvent } from "../../../shared/trace-events.js";
 import { INCIDENTS_POLICY_PATH } from "../../incidents/shared.js";
 import { resolvePrArtifactHeadSha, parsePrMeta } from "../../shared/pr-meta.js";
 import { isTaskLocalOnlyAdvance } from "../../shared/task-local-freshness.js";
@@ -126,6 +127,11 @@ export async function syncPrArtifacts(opts) {
     try {
         const ctx = opts.ctx ??
             (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
+        emitTraceEvent({
+            component: "pr-sync",
+            event: "sync_started",
+            details: { task_id: opts.taskId, mode: opts.mode, backend: ctx.backendId },
+        });
         const { task } = await loadBackendTask({
             ctx,
             cwd: opts.cwd,
@@ -203,9 +209,24 @@ export async function syncPrArtifacts(opts) {
                     author: opts.author,
                     remoteMode,
                 });
+                emitTraceEvent({
+                    component: "pr-sync",
+                    event: "sync_completed",
+                    details: {
+                        task_id: task.id,
+                        mode: opts.mode,
+                        branch,
+                        action: openOutcome?.action ?? null,
+                    },
+                });
                 return { meta, prDir, resolved, openOutcome };
             }
             const { meta } = await runPrUpdateSync(common);
+            emitTraceEvent({
+                component: "pr-sync",
+                event: "sync_completed",
+                details: { task_id: task.id, mode: opts.mode, branch, action: "updated" },
+            });
             return { meta, prDir, resolved };
         }
         finally {
@@ -216,6 +237,15 @@ export async function syncPrArtifacts(opts) {
         }
     }
     catch (err) {
+        emitTraceEvent({
+            component: "pr-sync",
+            event: "sync_failed",
+            details: {
+                task_id: opts.taskId,
+                mode: opts.mode,
+                error: err instanceof Error ? err.name : "UnknownError",
+            },
+        });
         if (err instanceof CliError)
             throw err;
         throw mapBackendError(err, { command: "pr sync", root: opts.rootOverride ?? null });

package/dist/commands/pr/open.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"open.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/open.ts"],"names":[],"mappings":"AAQA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAuDpF,wBAAsB,SAAS,CAAC,IAAI,EAAE;IACpC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB,GAAG,OAAO,CAAC,MAAM,CAAC,CAgElB"}
+{"version":3,"file":"open.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/open.ts"],"names":[],"mappings":"AAQA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAoKpF,wBAAsB,SAAS,CAAC,IAAI,EAAE;IACpC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB,GAAG,OAAO,CAAC,MAAM,CAAC,CAgElB"}