agentplane 0.3.9 → 0.3.11

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

@@ -6,23 +6,20 @@ import { exitCodeForError } from "../../../../cli/exit-codes.js";
 import { unknownEntityMessage, workflowModeMessage } from "../../../../cli/output.js";
 import { CliError } from "../../../../shared/errors.js";
 import { ensureGitClean } from "../../../guard/index.js";
-import { gitDiffNames, gitShowFile, toGitPath } from "../../../shared/git-diff.js";
+import { gitDiffNames } from "../../../shared/git-diff.js";
 import { gitBranchExists, gitCurrentBranch, gitRevParse } from "../../../shared/git-ops.js";
-import { loadBackendTask, loadCommandContext, } from "../../../shared/task-backend.js";
+import { findWorktreeForBranch } from "../../../shared/git-worktree.js";
+import { loadCommandContext, loadTaskFromContext, resolveTaskBranchFromContext, } from "../../../shared/task-backend.js";
 import { ensurePlanApprovedIfRequired, ensureVerificationSatisfiedIfRequired, } from "../../../task/shared.js";
-import { resolvePrPaths } from "../../internal/pr-paths.js";
-import { readAndValidatePrArtifacts } from "../artifacts.js";
+import { readPrArtifact, resolvePrPaths } from "../../internal/pr-paths.js";
+import { ensurePrArtifactsSynced } from "../../internal/sync.js";
+import { readAndValidatePrArtifacts, ensureCommittedPrArtifactsOnBranch } from "../artifacts.js";
 import { computeVerifyState } from "../verify.js";
-import { parsePrMeta } from "../../../shared/pr-meta.js";
+import { parsePrMetaForwardCompatible } from "../../../shared/pr-meta.js";
+import { assessPrArtifactFreshness } from "../../internal/freshness.js";
 export async function prepareIntegrate(opts) {
     const ctx = opts.ctx ??
         (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
-    const { task } = await loadBackendTask({
-        ctx,
-        cwd: opts.cwd,
-        rootOverride: opts.rootOverride,
-        taskId: opts.taskId,
-    });
     const resolved = ctx.resolvedProject;
     const loadedConfig = ctx.config;
     if (loadedConfig.workflow_mode !== "branch_pr") {
@@ -32,8 +29,6 @@ export async function prepareIntegrate(opts) {
             message: workflowModeMessage(loadedConfig.workflow_mode, "branch_pr"),
         });
     }
-    ensurePlanApprovedIfRequired(task, loadedConfig);
-    ensureVerificationSatisfiedIfRequired(task, loadedConfig);
     await ensureGitClean({ cwd: opts.cwd, rootOverride: opts.rootOverride });
     if (opts.base?.trim().length === 0) {
         throw new CliError({
@@ -72,10 +67,16 @@ export async function prepareIntegrate(opts) {
     let meta = null;
     let branch = (opts.branch ?? "").trim();
     if (await fileExists(metaPath)) {
-        meta = parsePrMeta(await readFile(metaPath, "utf8"), task.id);
+        meta = parsePrMetaForwardCompatible(await readFile(metaPath, "utf8"), opts.taskId);
         if (!branch)
             branch = (meta.branch ?? "").trim();
     }
+    if (!branch) {
+        const inferredBranch = await resolveTaskBranchFromContext({ ctx, taskId: opts.taskId });
+        if (inferredBranch) {
+            branch = inferredBranch;
+        }
+    }
     if (!branch) {
         throw new CliError({
             exitCode: 2,
@@ -90,8 +91,27 @@ export async function prepareIntegrate(opts) {
             message: unknownEntityMessage("branch", branch),
         });
     }
-    const metaSource = meta ??
-        parsePrMeta(await gitShowFile(resolved.gitRoot, branch, toGitPath(path.relative(resolved.gitRoot, metaPath))), task.id);
+    await ensureCommittedPrArtifactsOnBranch({
+        resolved,
+        prDir,
+        branch,
+    });
+    const worktreePath = await findWorktreeForBranch(resolved.gitRoot, branch);
+    const metaText = await readPrArtifact({
+        resolved,
+        prDir,
+        fileName: "meta.json",
+        branch,
+        worktreePath,
+    });
+    if (!metaText) {
+        throw new CliError({
+            exitCode: 4,
+            code: "E_IO",
+            message: `PR artifacts missing: ${path.relative(resolved.gitRoot, metaPath)} (run \`agentplane pr open\`)`,
+        });
+    }
+    let metaSource = parsePrMetaForwardCompatible(metaText, opts.taskId);
     const baseCandidate = opts.base ?? metaSource.base ?? baseBranch;
     const base = typeof baseCandidate === "string" && baseCandidate.trim().length > 0
         ? baseCandidate.trim()
@@ -102,10 +122,16 @@ export async function prepareIntegrate(opts) {
         prDir,
         metaPath,
         branch,
-        taskId: task.id,
+        taskId: opts.taskId,
     });
     // readAndValidatePrArtifacts() throws if verify.log is missing; keep this non-null downstream.
-    const verifyLogText = verifyLogTextMaybe;
+    let verifyLogText = verifyLogTextMaybe;
+    const task = await loadTaskFromContext({
+        ctx,
+        taskId: opts.taskId,
+        preferBranchSnapshot: true,
+        branchSnapshotBranch: branch,
+    });
     const changedPaths = await gitDiffNames(resolved.gitRoot, base, branch);
     const tasksPath = loadedConfig.paths.tasks_path;
     if (changedPaths.includes(tasksPath)) {
@@ -116,9 +142,71 @@ export async function prepareIntegrate(opts) {
         });
     }
     const branchHeadSha = await gitRevParse(resolved.gitRoot, [branch]);
+    let freshness = await assessPrArtifactFreshness({
+        gitRoot: resolved.gitRoot,
+        workflowDir: loadedConfig.paths.workflow_dir,
+        taskId: opts.taskId,
+        branchHeadSha,
+        metaHeadSha: metaSource.head_sha ?? null,
+        metaLastVerifiedSha: metaSource.last_verified_sha ?? null,
+        metaVerifyStatus: metaSource.verify?.status ?? null,
+        taskVerificationState: task.verification?.state ?? null,
+        verifyLogText,
+        requiresVerify: Boolean(task.verify && task.verify.length > 0),
+    });
+    if (!freshness.reviewFresh && worktreePath) {
+        await ensurePrArtifactsSynced({
+            cwd: worktreePath,
+            taskId: opts.taskId,
+            branch,
+        });
+        const repairedMetaText = await readPrArtifact({
+            resolved,
+            prDir,
+            fileName: "meta.json",
+            branch,
+            worktreePath,
+        });
+        if (repairedMetaText) {
+            const repairedMetaSource = parsePrMetaForwardCompatible(repairedMetaText, opts.taskId);
+            const repairedArtifacts = await readAndValidatePrArtifacts({
+                ctx,
+                resolved,
+                prDir,
+                metaPath,
+                branch,
+                taskId: opts.taskId,
+            });
+            verifyLogText = repairedArtifacts.verifyLogText ?? verifyLogText;
+            freshness = await assessPrArtifactFreshness({
+                gitRoot: resolved.gitRoot,
+                workflowDir: loadedConfig.paths.workflow_dir,
+                taskId: opts.taskId,
+                branchHeadSha,
+                metaHeadSha: repairedMetaSource.head_sha ?? null,
+                metaLastVerifiedSha: repairedMetaSource.last_verified_sha ?? null,
+                metaVerifyStatus: repairedMetaSource.verify?.status ?? null,
+                taskVerificationState: task.verification?.state ?? null,
+                verifyLogText,
+                requiresVerify: Boolean(task.verify && task.verify.length > 0),
+            });
+            meta = repairedMetaSource;
+            metaSource = repairedMetaSource;
+        }
+    }
+    if (!freshness.reviewFresh) {
+        throw new CliError({
+            exitCode: exitCodeForError("E_VALIDATION"),
+            code: "E_VALIDATION",
+            message: `PR artifacts stale for ${opts.taskId}: meta.head_sha=${metaSource.head_sha ?? "<missing>"} ` +
+                `current_head=${branchHeadSha} (refresh the task branch artifacts before integrate)`,
+        });
+    }
+    ensurePlanApprovedIfRequired(task, loadedConfig);
+    ensureVerificationSatisfiedIfRequired(task, loadedConfig);
     const initialVerifyState = computeVerifyState({
         rawVerify: task.verify,
-        metaLastVerifiedSha: metaSource?.last_verified_sha ?? null,
+        metaLastVerifiedSha: freshness.effectiveVerifiedSha,
         verifyLogText,
         branchHeadSha,
         runVerify: opts.runVerify,

package/dist/commands/pr/internal/freshness.d.ts

@@ -0,0 +1,20 @@
+export type PrArtifactFreshness = {
+    reviewFresh: boolean;
+    verifyFresh: boolean;
+    verifySatisfied: boolean;
+    verifyLogSha: string | null;
+    effectiveVerifiedSha: string | null;
+};
+export declare function assessPrArtifactFreshness(opts: {
+    gitRoot: string;
+    workflowDir: string;
+    taskId: string;
+    branchHeadSha: string;
+    metaHeadSha: unknown;
+    metaLastVerifiedSha: unknown;
+    metaVerifyStatus: unknown;
+    taskVerificationState: unknown;
+    verifyLogText: string | null;
+    requiresVerify: boolean;
+}): Promise<PrArtifactFreshness>;
+//# sourceMappingURL=freshness.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"freshness.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/freshness.ts"],"names":[],"mappings":"AASA,MAAM,MAAM,mBAAmB,GAAG;IAChC,WAAW,EAAE,OAAO,CAAC;IACrB,WAAW,EAAE,OAAO,CAAC;IACrB,eAAe,EAAE,OAAO,CAAC;IACzB,YAAY,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,oBAAoB,EAAE,MAAM,GAAG,IAAI,CAAC;CACrC,CAAC;AAEF,wBAAsB,yBAAyB,CAAC,IAAI,EAAE;IACpD,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,MAAM,CAAC;IACtB,WAAW,EAAE,OAAO,CAAC;IACrB,mBAAmB,EAAE,OAAO,CAAC;IAC7B,gBAAgB,EAAE,OAAO,CAAC;IAC1B,qBAAqB,EAAE,OAAO,CAAC;IAC/B,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,cAAc,EAAE,OAAO,CAAC;CACzB,GAAG,OAAO,CAAC,mBAAmB,CAAC,CAgD/B"}

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

@@ -0,0 +1,50 @@
+import { extractLastVerifiedSha } from "../../shared/pr-meta.js";
+import { isTaskLocalOnlyAdvance } from "../../shared/task-local-freshness.js";
+function normalizeSha(value) {
+    if (typeof value !== "string")
+        return null;
+    const trimmed = value.trim();
+    return trimmed.length > 0 ? trimmed : null;
+}
+export async function assessPrArtifactFreshness(opts) {
+    const metaHeadSha = normalizeSha(opts.metaHeadSha);
+    const metaLastVerifiedSha = normalizeSha(opts.metaLastVerifiedSha);
+    const verifyLogSha = normalizeSha(extractLastVerifiedSha(opts.verifyLogText ?? ""));
+    const metaVerifyPassed = opts.metaVerifyStatus === "pass";
+    const reviewFresh = metaHeadSha === opts.branchHeadSha ||
+        (await isTaskLocalOnlyAdvance({
+            gitRoot: opts.gitRoot,
+            workflowDir: opts.workflowDir,
+            taskId: opts.taskId,
+            fromRef: metaHeadSha,
+            toRef: opts.branchHeadSha,
+        }));
+    if (!opts.requiresVerify) {
+        return {
+            reviewFresh,
+            verifyFresh: true,
+            verifySatisfied: true,
+            verifyLogSha,
+            effectiveVerifiedSha: null,
+        };
+    }
+    const verifiedFromMeta = metaVerifyPassed &&
+        (metaLastVerifiedSha === opts.branchHeadSha ||
+            (await isTaskLocalOnlyAdvance({
+                gitRoot: opts.gitRoot,
+                workflowDir: opts.workflowDir,
+                taskId: opts.taskId,
+                fromRef: metaLastVerifiedSha,
+                toRef: opts.branchHeadSha,
+            })));
+    const verifiedFromLog = verifyLogSha === opts.branchHeadSha;
+    const verifiedFromTaskState = opts.taskVerificationState === "ok" && reviewFresh && metaLastVerifiedSha === null;
+    const verifySatisfied = verifiedFromMeta || verifiedFromLog || verifiedFromTaskState;
+    return {
+        reviewFresh,
+        verifyFresh: verifySatisfied,
+        verifySatisfied,
+        verifyLogSha,
+        effectiveVerifiedSha: verifySatisfied ? opts.branchHeadSha : metaLastVerifiedSha,
+    };
+}

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

@@ -0,0 +1,6 @@
+export declare function ghEnv(): NodeJS.ProcessEnv;
+export declare function resolveDefaultGithubRepo(cwd: string): Promise<string>;
+export declare function runGhApiJson<T>(cwd: string, args: string[]): Promise<T>;
+export declare function runGhApiNoOutput(cwd: string, args: string[]): Promise<void>;
+export declare function isGhNotFound(err: unknown): boolean;
+//# sourceMappingURL=gh-api.d.ts.map

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

@@ -0,0 +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"}

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

@@ -0,0 +1,80 @@
+import { execFile } from "node:child_process";
+import { promisify } from "node:util";
+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)
+        return null;
+    const httpsMatch = /^https?:\/\/github\.com\/([^/]+)\/([^/]+?)(?:\.git)?\/?$/.exec(trimmed);
+    if (httpsMatch)
+        return `${httpsMatch[1]}/${httpsMatch[2]}`;
+    const sshMatch = /^git@github\.com:([^/]+)\/([^/]+?)(?:\.git)?$/.exec(trimmed);
+    if (sshMatch)
+        return `${sshMatch[1]}/${sshMatch[2]}`;
+    return null;
+}
+export function ghEnv() {
+    const env = gitEnv();
+    // Preserve explicit shell/CI auth, but do not let repo-local `.env` tokens override
+    // a valid gh keyring session for nested CLI invocations.
+    if (isDotEnvLoadedKey("GH_TOKEN"))
+        delete env.GH_TOKEN;
+    if (isDotEnvLoadedKey("GITHUB_TOKEN"))
+        delete env.GITHUB_TOKEN;
+    if (typeof process.env.GH_TOKEN === "string" && !isDotEnvLoadedKey("GH_TOKEN")) {
+        env.GH_TOKEN = process.env.GH_TOKEN;
+    }
+    if (typeof process.env.GITHUB_TOKEN === "string" && !isDotEnvLoadedKey("GITHUB_TOKEN")) {
+        env.GITHUB_TOKEN = process.env.GITHUB_TOKEN;
+    }
+    if (typeof process.env.GH_CONFIG_DIR === "string")
+        env.GH_CONFIG_DIR = process.env.GH_CONFIG_DIR;
+    if (typeof process.env.XDG_CONFIG_HOME === "string")
+        env.XDG_CONFIG_HOME = process.env.XDG_CONFIG_HOME;
+    if (typeof process.env.HOME === "string")
+        env.HOME = process.env.HOME;
+    return env;
+}
+export async function resolveDefaultGithubRepo(cwd) {
+    const { stdout } = await execFileAsyncRaw("git", ["remote", "get-url", "origin"], {
+        cwd,
+        env: gitEnv(),
+        maxBuffer: 10 * 1024 * 1024,
+    });
+    const repo = parseGithubRepoFromRemoteUrl(stdout);
+    if (!repo) {
+        throw new CliError({
+            exitCode: exitCodeForError("E_VALIDATION"),
+            code: "E_VALIDATION",
+            message: "Could not derive GitHub owner/repo from git remote origin.",
+        });
+    }
+    return repo;
+}
+export async function runGhApiJson(cwd, args) {
+    const { stdout } = await withGhTransportRetry(() => execFileAsyncRaw("gh", ["api", ...args], {
+        cwd,
+        env: ghEnv(),
+        maxBuffer: 10 * 1024 * 1024,
+    }), {
+        label: `running gh api ${args[0] ?? ""}`,
+    });
+    return JSON.parse(stdout);
+}
+export async function runGhApiNoOutput(cwd, args) {
+    await withGhTransportRetry(() => execFileAsyncRaw("gh", ["api", ...args], {
+        cwd,
+        env: ghEnv(),
+        maxBuffer: 10 * 1024 * 1024,
+    }), {
+        label: `running gh api ${args[0] ?? ""}`,
+    });
+}
+export function isGhNotFound(err) {
+    return /\b404\b/.test(normalizeGhTransportError(err));
+}

package/dist/commands/pr/internal/note-store.d.ts

@@ -0,0 +1,18 @@
+export type PrHandoffNote = {
+    schema_version: 1;
+    created_at: string;
+    author: string;
+    body: string;
+};
+export declare function buildPrHandoffNote(opts: {
+    createdAt: string;
+    author: string;
+    body: string;
+}): PrHandoffNote;
+export declare function parsePrHandoffNotes(raw: string): PrHandoffNote[];
+export declare function readPrHandoffNotes(notesPath: string): Promise<PrHandoffNote[]>;
+export declare function appendPrHandoffNote(opts: {
+    notesPath: string;
+    note: PrHandoffNote;
+}): Promise<void>;
+//# sourceMappingURL=note-store.d.ts.map

package/dist/commands/pr/internal/note-store.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"note-store.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/note-store.ts"],"names":[],"mappings":"AAGA,MAAM,MAAM,aAAa,GAAG;IAC1B,cAAc,EAAE,CAAC,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;CACd,CAAC;AAaF,wBAAgB,kBAAkB,CAAC,IAAI,EAAE;IACvC,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;CACd,GAAG,aAAa,CAOhB;AAED,wBAAgB,mBAAmB,CAAC,GAAG,EAAE,MAAM,GAAG,aAAa,EAAE,CA2BhE;AAED,wBAAsB,kBAAkB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,aAAa,EAAE,CAAC,CAQpF;AAED,wBAAsB,mBAAmB,CAAC,IAAI,EAAE;IAC9C,SAAS,EAAE,MAAM,CAAC;IAClB,IAAI,EAAE,aAAa,CAAC;CACrB,GAAG,OAAO,CAAC,IAAI,CAAC,CAGhB"}

package/dist/commands/pr/internal/note-store.js

@@ -0,0 +1,66 @@
+import { appendFile, mkdir, readFile } from "node:fs/promises";
+import path from "node:path";
+function invalidNote(index, message) {
+    return new Error(`Invalid pr/notes.jsonl entry ${index + 1}: ${message}`);
+}
+function trimRequired(value, field, index) {
+    if (typeof value !== "string")
+        throw invalidNote(index, `${field} must be a string`);
+    const trimmed = value.trim();
+    if (!trimmed)
+        throw invalidNote(index, `${field} must not be empty`);
+    return trimmed;
+}
+export function buildPrHandoffNote(opts) {
+    return {
+        schema_version: 1,
+        created_at: opts.createdAt,
+        author: opts.author.trim(),
+        body: opts.body.trim(),
+    };
+}
+export function parsePrHandoffNotes(raw) {
+    if (!raw.trim())
+        return [];
+    return raw
+        .split(/\r?\n/)
+        .filter((line) => line.trim().length > 0)
+        .map((line, index) => {
+        let parsed;
+        try {
+            parsed = JSON.parse(line);
+        }
+        catch (err) {
+            const message = err instanceof Error ? err.message : String(err);
+            throw invalidNote(index, `invalid JSON (${message})`);
+        }
+        const record = parsed;
+        if (!record || typeof record !== "object") {
+            throw invalidNote(index, "entry must be an object");
+        }
+        if ((record.schema_version ?? 1) !== 1) {
+            throw invalidNote(index, "unsupported schema_version");
+        }
+        return {
+            schema_version: 1,
+            created_at: trimRequired(record.created_at, "created_at", index),
+            author: trimRequired(record.author, "author", index),
+            body: trimRequired(record.body, "body", index),
+        };
+    });
+}
+export async function readPrHandoffNotes(notesPath) {
+    try {
+        return parsePrHandoffNotes(await readFile(notesPath, "utf8"));
+    }
+    catch (err) {
+        const code = err?.code;
+        if (code === "ENOENT")
+            return [];
+        throw err;
+    }
+}
+export async function appendPrHandoffNote(opts) {
+    await mkdir(path.dirname(opts.notesPath), { recursive: true });
+    await appendFile(opts.notesPath, `${JSON.stringify(opts.note)}\n`, "utf8");
+}

package/dist/commands/pr/internal/pr-paths.d.ts

@@ -9,8 +9,11 @@ export type ResolvedPrPaths = {
     prDir: string;
     metaPath: string;
     diffstatPath: string;
+    notesPath: string;
     verifyLogPath: string;
     reviewPath: string;
+    githubTitlePath: string;
+    githubBodyPath: string;
 };
 export declare function resolvePrPaths(opts: {
     ctx?: CommandContext;
@@ -25,5 +28,15 @@ export declare function readPrArtifact(opts: {
     prDir: string;
     fileName: string;
     branch: string;
+    worktreePath?: string | null;
+}): Promise<string | null>;
+export declare function readPrArtifactFromBranch(opts: {
+    resolved: {
+        gitRoot: string;
+    };
+    prDir: string;
+    fileName: string;
+    branch: string;
+    worktreePath?: string | null;
 }): Promise<string | null>;
 //# sourceMappingURL=pr-paths.d.ts.map

package/dist/commands/pr/internal/pr-paths.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pr-paths.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/pr-paths.ts"],"names":[],"mappings":"AAGA,OAAO,EAA8B,KAAK,gBAAgB,EAAE,MAAM,qBAAqB,CAAC;AAIxF,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAEnE,MAAM,MAAM,eAAe,GAAG;IAC5B,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,aAAa,EAAE,MAAM,CAAA;KAAE,CAAC;IACrD,MAAM,EAAE,gBAAgB,CAAC;IACzB,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,MAAM,CAAC;IACrB,aAAa,EAAE,MAAM,CAAC;IACtB,UAAU,EAAE,MAAM,CAAC;CACpB,CAAC;AAEF,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,eAAe,CAAC,CAoB3B;AAED,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IAC9B,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAWzB"}
+{"version":3,"file":"pr-paths.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/pr-paths.ts"],"names":[],"mappings":"AAGA,OAAO,EAA8B,KAAK,gBAAgB,EAAE,MAAM,qBAAqB,CAAC;AAIxF,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAEnE,MAAM,MAAM,eAAe,GAAG;IAC5B,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,aAAa,EAAE,MAAM,CAAA;KAAE,CAAC;IACrD,MAAM,EAAE,gBAAgB,CAAC;IACzB,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,MAAM,CAAC;IACrB,SAAS,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;IACtB,UAAU,EAAE,MAAM,CAAC;IACnB,eAAe,EAAE,MAAM,CAAC;IACxB,cAAc,EAAE,MAAM,CAAC;CACxB,CAAC;AAEF,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,GAAG,CAAC,EAAE,cAAc,CAAC;IACrB,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,OAAO,CAAC,eAAe,CAAC,CAuB3B;AAED,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IAC9B,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAMzB;AAED,wBAAsB,wBAAwB,CAAC,IAAI,EAAE;IACnD,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IAC9B,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAiBzB"}

package/dist/commands/pr/internal/pr-paths.js

@@ -19,8 +19,11 @@ export async function resolvePrPaths(opts) {
         prDir,
         metaPath: path.join(prDir, "meta.json"),
         diffstatPath: path.join(prDir, "diffstat.txt"),
+        notesPath: path.join(prDir, "notes.jsonl"),
         verifyLogPath: path.join(prDir, "verify.log"),
         reviewPath: path.join(prDir, "review.md"),
+        githubTitlePath: path.join(prDir, "github-title.txt"),
+        githubBodyPath: path.join(prDir, "github-body.md"),
     };
 }
 export async function readPrArtifact(opts) {
@@ -28,6 +31,16 @@ export async function readPrArtifact(opts) {
     if (await fileExists(filePath)) {
         return await readFile(filePath, "utf8");
     }
+    return await readPrArtifactFromBranch(opts);
+}
+export async function readPrArtifactFromBranch(opts) {
+    const filePath = path.join(opts.prDir, opts.fileName);
+    if (opts.worktreePath) {
+        const worktreeFilePath = path.join(opts.worktreePath, path.relative(opts.resolved.gitRoot, filePath));
+        if (await fileExists(worktreeFilePath)) {
+            return await readFile(worktreeFilePath, "utf8");
+        }
+    }
     const rel = toGitPath(path.relative(opts.resolved.gitRoot, filePath));
     try {
         return await gitShowFile(opts.resolved.gitRoot, opts.branch, rel);

package/dist/commands/pr/internal/review-template.d.ts

@@ -1,9 +1,29 @@
-export declare function renderPrReviewTemplate(opts: {
-    author: string;
-    createdAt: string;
+import type { TaskData } from "../../../backends/task-backend.js";
+import type { PrHandoffNote } from "./note-store.js";
+export declare function buildGithubPrTitle(task: TaskData): string;
+export declare function renderPrAutoSummary(opts: {
+    updatedAt: string;
     branch: string;
+    headSha: string | null;
+    diffstat: string;
 }): string;
 export declare function updateAutoSummaryBlock(text: string, summary: string): string;
-export declare function appendHandoffNote(review: string, note: string): string;
+export declare function extractAutoSummaryBlock(text: string): string | null;
+export declare function renderPrHandoffNotes(notes: PrHandoffNote[]): string[];
+export declare function updateHandoffNotesBlock(text: string, notes: PrHandoffNote[]): string;
+export declare function renderPrReviewDocument(opts: {
+    task: TaskData;
+    author?: string;
+    createdAt: string;
+    branch: string;
+    handoffNotes?: PrHandoffNote[];
+    autoSummary: string;
+}): string;
+export declare function renderGithubPrBody(opts: {
+    task: TaskData;
+    handoffNotes?: PrHandoffNote[];
+    autoSummary: string;
+}): string;
 export declare function validateReviewContents(review: string, errors: string[]): void;
+export declare function validateGithubPrBodyContents(body: string, errors: string[]): void;
 //# sourceMappingURL=review-template.d.ts.map

package/dist/commands/pr/internal/review-template.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"review-template.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/review-template.ts"],"names":[],"mappings":"AAAA,wBAAgB,sBAAsB,CAAC,IAAI,EAAE;IAC3C,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;CAChB,GAAG,MAAM,CA0BT;AAED,wBAAgB,sBAAsB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAW5E;AAED,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,CAQtE;AAED,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,IAAI,CAW7E"}
+{"version":3,"file":"review-template.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/review-template.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AAClE,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,iBAAiB,CAAC;AAoJrD,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,QAAQ,GAAG,MAAM,CAKzD;AAED,wBAAgB,mBAAmB,CAAC,IAAI,EAAE;IACxC,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,QAAQ,EAAE,MAAM,CAAC;CAClB,GAAG,MAAM,CAeT;AAED,wBAAgB,sBAAsB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAS5E;AAED,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAKnE;AAMD,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,aAAa,EAAE,GAAG,MAAM,EAAE,CAOrE;AAED,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,aAAa,EAAE,GAAG,MAAM,CAgBpF;AAED,wBAAgB,sBAAsB,CAAC,IAAI,EAAE;IAC3C,IAAI,EAAE,QAAQ,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,aAAa,EAAE,CAAC;IAC/B,WAAW,EAAE,MAAM,CAAC;CACrB,GAAG,MAAM,CAgBT;AAED,wBAAgB,kBAAkB,CAAC,IAAI,EAAE;IACvC,IAAI,EAAE,QAAQ,CAAC;IACf,YAAY,CAAC,EAAE,aAAa,EAAE,CAAC;IAC/B,WAAW,EAAE,MAAM,CAAC;CACrB,GAAG,MAAM,CAST;AAED,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,IAAI,CAiB7E;AAED,wBAAgB,4BAA4B,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,IAAI,CAajF"}