agentplane 0.3.8 → 0.3.10

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;AA4GrD,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,CAcjF"}

package/dist/commands/pr/internal/review-template.js

@@ -1,62 +1,217 @@
-export function renderPrReviewTemplate(opts) {
+import { extractTaskSuffix } from "@agentplaneorg/core";
+const AUTO_SUMMARY_START = "<!-- BEGIN AUTO SUMMARY -->";
+const AUTO_SUMMARY_END = "<!-- END AUTO SUMMARY -->";
+const SUMMARY_SECTION = "## Summary";
+const SCOPE_SECTION = "## Scope";
+const VERIFICATION_SECTION = "## Verification";
+const RISKS_SECTION = "## Risks";
+const HANDOFF_NOTES_MARKER = "## Handoff Notes";
+function sectionText(task, name, fallback) {
+    const value = typeof task.sections?.[name] === "string" ? task.sections[name].trim() : "";
+    return value || fallback;
+}
+function normalizeOneLine(value, maxChars) {
+    const trimmed = value.trim().replaceAll(/\s+/g, " ");
+    if (!trimmed)
+        return "";
+    return trimmed.length > maxChars ? `${trimmed.slice(0, Math.max(1, maxChars - 3))}...` : trimmed;
+}
+function informativeTags(task) {
+    const seen = new Set();
+    const out = [];
+    for (const rawTag of Array.isArray(task.tags) ? task.tags : []) {
+        const tag = rawTag.trim().toLowerCase();
+        if (!tag || tag === "code" || seen.has(tag))
+            continue;
+        seen.add(tag);
+        out.push(tag);
+    }
+    return out;
+}
+function renderVerificationSummary(task) {
+    const state = task.verification?.state ?? "pending";
+    const note = typeof task.verification?.note === "string" ? task.verification.note.trim() : "";
+    const commands = Array.isArray(task.verify)
+        ? task.verify.filter((value) => typeof value === "string" && value.trim().length > 0)
+        : [];
+    const plan = sectionText(task, "Verify Steps", commands.length > 0
+        ? commands.map((command) => `- ${command.trim()}`).join("\n")
+        : "- Not recorded.");
+    const statusLine = state === "ok"
+        ? note || "Recorded as passed."
+        : state === "needs_rework"
+            ? note || "Recorded as needs rework."
+            : note || "Not recorded yet.";
     return [
-        "# PR Review",
+        "### Plan",
+        "",
+        plan,
+        "",
+        "### Current Status",
+        "",
+        `- State: ${state}`,
+        `- Note: ${statusLine}`,
+    ].join("\n");
+}
+function renderRiskSummary(task) {
+    const riskLevel = typeof task.risk_level === "string" ? task.risk_level : "not recorded";
+    const rollbackPlan = sectionText(task, "Rollback Plan", "- Revert task-related commit(s) if rollback is required.");
+    return [
+        `- Risk level: ${riskLevel}`,
+        `- Breaking change: ${task.breaking === true ? "yes" : "no"}`,
         "",
-        `Opened by ${opts.author} on ${opts.createdAt}`,
-        `Branch: ${opts.branch}`,
+        "### Rollback",
         "",
-        "## Summary",
+        rollbackPlan,
+    ].join("\n");
+}
+function renderSharedPrSections(opts) {
+    return [
+        SUMMARY_SECTION,
+        "",
+        sectionText(opts.task, "Summary", opts.task.title.trim() || "- Not recorded."),
+        "",
+        SCOPE_SECTION,
+        "",
+        sectionText(opts.task, "Scope", "- Not recorded."),
         "",
-        "- ",
+        VERIFICATION_SECTION,
         "",
-        "## Checklist",
+        renderVerificationSummary(opts.task),
         "",
-        "- [ ] Tests added/updated",
-        "- [ ] Lint/format passes",
-        "- [ ] Verify passed",
-        "- [ ] Docs updated (if needed)",
+        RISKS_SECTION,
         "",
-        "## Handoff Notes",
+        renderRiskSummary(opts.task),
+        "",
+        HANDOFF_NOTES_MARKER,
+        "",
+        ...renderPrHandoffNotes(opts.handoffNotes),
+        "",
+    ];
+}
+export function buildGithubPrTitle(task) {
+    const suffix = extractTaskSuffix(task.id);
+    const scope = informativeTags(task).slice(0, 2).join("/");
+    const title = normalizeOneLine(task.title, scope ? 72 : 84);
+    return scope ? `${scope}: ${title} (${suffix})` : `${title} (${suffix})`;
+}
+export function renderPrAutoSummary(opts) {
+    return [
+        "<details>",
+        "<summary>Raw evidence</summary>",
         "",
-        "<!-- Add review notes here. -->",
+        `- Updated: ${opts.updatedAt}`,
+        `- Branch: ${opts.branch}`,
+        `- Head: ${opts.headSha ? opts.headSha.slice(0, 12) : "No commits yet"}`,
         "",
-        "<!-- BEGIN AUTO SUMMARY -->",
-        "<!-- END AUTO SUMMARY -->",
+        "```text",
+        opts.diffstat || "No changes detected.",
+        "```",
         "",
+        "</details>",
     ].join("\n");
 }
 export function updateAutoSummaryBlock(text, summary) {
-    const start = "<!-- BEGIN AUTO SUMMARY -->";
-    const end = "<!-- END AUTO SUMMARY -->";
-    const startIdx = text.indexOf(start);
-    const endIdx = text.indexOf(end);
+    const startIdx = text.indexOf(AUTO_SUMMARY_START);
+    const endIdx = text.indexOf(AUTO_SUMMARY_END);
     if (startIdx === -1 || endIdx === -1 || endIdx < startIdx) {
-        return `${text.trimEnd()}\n\n${start}\n${summary}\n${end}\n`;
+        return `${text.trimEnd()}\n\n${AUTO_SUMMARY_START}\n${summary}\n${AUTO_SUMMARY_END}\n`;
     }
-    const before = text.slice(0, startIdx + start.length);
+    const before = text.slice(0, startIdx + AUTO_SUMMARY_START.length);
     const after = text.slice(endIdx);
     return `${before}\n${summary}\n${after}`;
 }
-export function appendHandoffNote(review, note) {
-    const marker = "## Handoff Notes";
-    const idx = review.indexOf(marker);
-    if (idx === -1)
-        return `${review.trimEnd()}\n\n${marker}\n\n- ${note}\n`;
-    const head = review.slice(0, idx + marker.length);
-    const tail = review.slice(idx + marker.length);
-    const trimmedTail = tail.startsWith("\n") ? tail.slice(1) : tail;
-    return `${head}\n\n- ${note}\n${trimmedTail}`;
+export function extractAutoSummaryBlock(text) {
+    const startIdx = text.indexOf(AUTO_SUMMARY_START);
+    const endIdx = text.indexOf(AUTO_SUMMARY_END);
+    if (startIdx === -1 || endIdx === -1 || endIdx < startIdx)
+        return null;
+    return text.slice(startIdx + AUTO_SUMMARY_START.length, endIdx).trim();
+}
+function formatNoteTimestamp(value) {
+    return value.replace(/\.\d{3}Z$/, "Z");
+}
+export function renderPrHandoffNotes(notes) {
+    if (notes.length === 0) {
+        return ["- No handoff notes recorded yet. Use `agentplane pr note ...` to append one."];
+    }
+    return notes.map((note) => `- ${formatNoteTimestamp(note.created_at)} ${note.author}: ${note.body}`);
+}
+export function updateHandoffNotesBlock(text, notes) {
+    const markerIdx = text.indexOf(HANDOFF_NOTES_MARKER);
+    const nextIdx = markerIdx === -1
+        ? text.indexOf(AUTO_SUMMARY_START)
+        : text.indexOf(AUTO_SUMMARY_START, markerIdx);
+    const renderedNotes = renderPrHandoffNotes(notes).join("\n");
+    if (markerIdx === -1) {
+        if (nextIdx === -1) {
+            return `${text.trimEnd()}\n\n${HANDOFF_NOTES_MARKER}\n\n${renderedNotes}\n`;
+        }
+        return `${text.slice(0, nextIdx).trimEnd()}\n\n${HANDOFF_NOTES_MARKER}\n\n${renderedNotes}\n\n${text.slice(nextIdx)}`;
+    }
+    const before = text.slice(0, markerIdx + HANDOFF_NOTES_MARKER.length);
+    const after = nextIdx === -1 ? "" : text.slice(nextIdx);
+    return `${before}\n\n${renderedNotes}\n\n${after}`.trimEnd() + "\n";
+}
+export function renderPrReviewDocument(opts) {
+    return [
+        "# PR Review",
+        "",
+        `Created: ${opts.createdAt || "UNKNOWN"}`,
+        `Branch: ${opts.branch || "UNKNOWN"}`,
+        "",
+        ...renderSharedPrSections({
+            task: opts.task,
+            handoffNotes: opts.handoffNotes ?? [],
+        }),
+        AUTO_SUMMARY_START,
+        opts.autoSummary,
+        AUTO_SUMMARY_END,
+        "",
+    ].join("\n");
+}
+export function renderGithubPrBody(opts) {
+    return [
+        ...renderSharedPrSections({
+            task: opts.task,
+            handoffNotes: opts.handoffNotes ?? [],
+        }),
+        opts.autoSummary,
+        "",
+    ].join("\n");
 }
 export function validateReviewContents(review, errors) {
-    const requiredSections = ["## Summary", "## Checklist", "## Handoff Notes"];
+    const requiredSections = [
+        SUMMARY_SECTION,
+        SCOPE_SECTION,
+        VERIFICATION_SECTION,
+        RISKS_SECTION,
+        HANDOFF_NOTES_MARKER,
+    ];
     for (const section of requiredSections) {
         if (!review.includes(section))
             errors.push(`Missing section: ${section}`);
     }
-    if (!review.includes("<!-- BEGIN AUTO SUMMARY -->")) {
+    if (!review.includes(AUTO_SUMMARY_START)) {
         errors.push("Missing auto summary start marker");
     }
-    if (!review.includes("<!-- END AUTO SUMMARY -->")) {
+    if (!review.includes(AUTO_SUMMARY_END)) {
         errors.push("Missing auto summary end marker");
     }
 }
+export function validateGithubPrBodyContents(body, errors) {
+    const requiredSections = [
+        SUMMARY_SECTION,
+        SCOPE_SECTION,
+        VERIFICATION_SECTION,
+        RISKS_SECTION,
+        HANDOFF_NOTES_MARKER,
+    ];
+    for (const section of requiredSections) {
+        if (!body.includes(section))
+            errors.push(`Missing section: ${section}`);
+    }
+    if (!body.includes("<details>")) {
+        errors.push("Missing raw evidence details block");
+    }
+}

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

@@ -0,0 +1,32 @@
+import { type CommandContext } from "../../shared/task-backend.js";
+type PrSyncMode = "open" | "update";
+export declare function ensurePrArtifactsSynced(opts: {
+    ctx?: CommandContext;
+    cwd: string;
+    rootOverride?: string;
+    taskId: string;
+    author?: string;
+    branch?: string;
+}): Promise<{
+    branch: string;
+    prDir: string;
+    resolved: {
+        gitRoot: string;
+    };
+} | null>;
+export declare function syncPrArtifacts(opts: {
+    ctx?: CommandContext;
+    cwd: string;
+    rootOverride?: string;
+    taskId: string;
+    mode: PrSyncMode;
+    author?: string;
+    branch?: string;
+}): Promise<{
+    prDir: string;
+    resolved: {
+        gitRoot: string;
+    };
+}>;
+export {};
+//# sourceMappingURL=sync.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"sync.d.ts","sourceRoot":"","sources":["../../../../src/commands/pr/internal/sync.ts"],"names":[],"mappings":"AAoBA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,8BAA8B,CAAC;AAoCtC,KAAK,UAAU,GAAG,MAAM,GAAG,QAAQ,CAAC;AAmCpC,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,CA8CR;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;CACjB,GAAG,OAAO,CAAC;IACV,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;CAC/B,CAAC,CAkLD"}

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

@@ -0,0 +1,258 @@
+import { mkdir, readFile } from "node:fs/promises";
+import path from "node:path";
+import { resolveBaseBranch } from "@agentplaneorg/core";
+import { mapBackendError } from "../../../cli/error-map.js";
+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 { writeJsonStableIfChanged, writeTextIfChanged } from "../../../shared/write-if-changed.js";
+import { execFileAsync, gitEnv } from "../../shared/git.js";
+import { gitCurrentBranch } from "../../shared/git-ops.js";
+import { parseTaskIdFromBranch } from "../../shared/git-worktree.js";
+import { buildOpenedPrMeta, buildUpdatedPrMeta, parsePrMeta, } from "../../shared/pr-meta.js";
+import { loadBackendTask, loadCommandContext, } from "../../shared/task-backend.js";
+import { resolvePrPaths } from "./pr-paths.js";
+import { readPrHandoffNotes } from "./note-store.js";
+import { buildGithubPrTitle, renderGithubPrBody, renderPrAutoSummary, renderPrReviewDocument, } from "./review-template.js";
+function nowIso() {
+    return new Date().toISOString();
+}
+function isUnknownRevisionError(err) {
+    const message = err instanceof Error ? err.message : String(err);
+    return /unknown revision or path not in the working tree/i.test(message);
+}
+async function resolveBranchHeadSha(opts) {
+    try {
+        const { stdout } = await execFileAsync("git", ["rev-parse", opts.branch], {
+            cwd: opts.gitRoot,
+            env: gitEnv(),
+        });
+        return stdout.trim() || null;
+    }
+    catch (err) {
+        if (!isUnknownRevisionError(err))
+            throw err;
+        return null;
+    }
+}
+async function resolvePrSyncBranch(opts) {
+    const explicitBranch = opts.branch?.trim() ?? "";
+    if (explicitBranch) {
+        return { branch: explicitBranch, source: "explicit" };
+    }
+    if (await fileExists(opts.metaPath)) {
+        const metaBranch = parsePrMeta(await readFile(opts.metaPath, "utf8"), opts.taskId).branch?.trim() ?? "";
+        if (metaBranch) {
+            return { branch: metaBranch, source: "meta" };
+        }
+    }
+    const currentBranchValue = await gitCurrentBranch(opts.resolved.gitRoot);
+    const currentBranch = currentBranchValue.trim();
+    if (currentBranch) {
+        return { branch: currentBranch, source: "current" };
+    }
+    return { branch: null, source: "none" };
+}
+export async function ensurePrArtifactsSynced(opts) {
+    const ctx = opts.ctx ??
+        (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
+    const { resolved, config, metaPath } = await resolvePrPaths({ ...opts, ctx });
+    if (config.workflow_mode !== "branch_pr")
+        return null;
+    const resolvedBranch = await resolvePrSyncBranch({
+        resolved,
+        metaPath,
+        taskId: opts.taskId,
+        branch: opts.branch,
+    });
+    const branch = resolvedBranch.branch?.trim() ?? "";
+    if (!branch)
+        return null;
+    if (resolvedBranch.source === "current" &&
+        parseTaskIdFromBranch(config.branch.task_prefix, branch) !== opts.taskId) {
+        return null;
+    }
+    const baseBranch = await resolveBaseBranch({
+        cwd: opts.cwd,
+        rootOverride: opts.rootOverride ?? null,
+        cliBaseOpt: null,
+        mode: config.workflow_mode,
+    });
+    if (resolvedBranch.source === "current" && baseBranch && branch === baseBranch) {
+        return null;
+    }
+    await syncPrArtifacts({
+        ...opts,
+        ctx,
+        mode: "open",
+        author: opts.author,
+        branch,
+    });
+    const result = await syncPrArtifacts({
+        ...opts,
+        ctx,
+        mode: "update",
+        branch,
+    });
+    return { ...result, branch };
+}
+export async function syncPrArtifacts(opts) {
+    try {
+        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, config, prDir, metaPath, diffstatPath, notesPath, verifyLogPath, reviewPath, githubTitlePath, githubBodyPath, } = await resolvePrPaths({ ...opts, ctx });
+        if (config.workflow_mode !== "branch_pr") {
+            throw new CliError({
+                exitCode: exitCodeForError("E_USAGE"),
+                code: "E_USAGE",
+                message: workflowModeMessage(config.workflow_mode, "branch_pr"),
+            });
+        }
+        const resolvedBranch = await resolvePrSyncBranch({
+            resolved,
+            metaPath,
+            taskId: task.id,
+            branch: opts.branch,
+        });
+        const branch = resolvedBranch.branch?.trim() ?? "";
+        if (!branch) {
+            throw new CliError({
+                exitCode: exitCodeForError("E_USAGE"),
+                code: "E_USAGE",
+                message: "Branch could not be resolved (use --branch).",
+            });
+        }
+        const metaExists = await fileExists(metaPath);
+        const reviewExists = await fileExists(reviewPath);
+        if (opts.mode === "update" && (!metaExists || !reviewExists)) {
+            const missing = [];
+            if (!metaExists)
+                missing.push(path.relative(resolved.gitRoot, metaPath));
+            if (!reviewExists)
+                missing.push(path.relative(resolved.gitRoot, reviewPath));
+            throw new CliError({
+                exitCode: exitCodeForError("E_VALIDATION"),
+                code: "E_VALIDATION",
+                message: `PR artifacts missing: ${missing.join(", ")} (run \`agentplane pr open\`)`,
+            });
+        }
+        await mkdir(prDir, { recursive: true });
+        const existingMeta = metaExists && (await fileExists(metaPath))
+            ? parsePrMeta(await readFile(metaPath, "utf8"), task.id)
+            : null;
+        const handoffNotes = await readPrHandoffNotes(notesPath);
+        const now = nowIso();
+        const createdAt = existingMeta?.created_at ?? now;
+        const baseBranch = await resolveBaseBranch({
+            cwd: opts.cwd,
+            rootOverride: opts.rootOverride ?? null,
+            cliBaseOpt: null,
+            mode: config.workflow_mode,
+        });
+        const headSha = await resolveBranchHeadSha({ gitRoot: resolved.gitRoot, branch });
+        if (opts.mode === "open") {
+            const nextMeta = buildOpenedPrMeta({
+                taskId: task.id,
+                branch,
+                at: now,
+                previousMeta: existingMeta,
+                base: baseBranch,
+                headSha,
+            });
+            const nextAutoSummary = renderPrAutoSummary({
+                updatedAt: nextMeta.updated_at,
+                branch,
+                headSha,
+                diffstat: "",
+            });
+            const nextReview = renderPrReviewDocument({
+                task,
+                author: opts.author,
+                createdAt,
+                branch,
+                handoffNotes,
+                autoSummary: nextAutoSummary,
+            });
+            const githubTitle = buildGithubPrTitle(task);
+            const githubBody = renderGithubPrBody({
+                task,
+                handoffNotes,
+                autoSummary: nextAutoSummary,
+            });
+            await writeJsonStableIfChanged(metaPath, nextMeta);
+            if (!(await fileExists(diffstatPath))) {
+                await writeTextIfChanged(diffstatPath, "");
+            }
+            if (!(await fileExists(notesPath))) {
+                await writeTextIfChanged(notesPath, "");
+            }
+            if (!(await fileExists(verifyLogPath))) {
+                await writeTextIfChanged(verifyLogPath, "");
+            }
+            await writeTextIfChanged(reviewPath, nextReview);
+            await writeTextIfChanged(githubTitlePath, `${githubTitle}\n`);
+            await writeTextIfChanged(githubBodyPath, githubBody);
+            return { prDir, resolved };
+        }
+        if (!baseBranch) {
+            throw new CliError({
+                exitCode: exitCodeForError("E_USAGE"),
+                code: "E_USAGE",
+                message: "Base branch could not be resolved (use `agentplane branch base set`).",
+            });
+        }
+        let diffstat = "";
+        try {
+            const { stdout: diffStatOut } = await execFileAsync("git", ["diff", "--stat", `${baseBranch}...${branch}`], { cwd: resolved.gitRoot, env: gitEnv() });
+            diffstat = diffStatOut.trimEnd();
+        }
+        catch (err) {
+            if (!isUnknownRevisionError(err))
+                throw err;
+        }
+        const nextMeta = buildUpdatedPrMeta({
+            meta: existingMeta,
+            branch,
+            at: now,
+            base: baseBranch,
+            headSha,
+        });
+        const nextAutoSummary = renderPrAutoSummary({
+            updatedAt: nextMeta.updated_at,
+            branch,
+            headSha,
+            diffstat,
+        });
+        const nextReview = renderPrReviewDocument({
+            task,
+            createdAt,
+            branch,
+            handoffNotes,
+            autoSummary: nextAutoSummary,
+        });
+        const githubTitle = buildGithubPrTitle(task);
+        const githubBody = renderGithubPrBody({
+            task,
+            handoffNotes,
+            autoSummary: nextAutoSummary,
+        });
+        await writeTextIfChanged(diffstatPath, diffstat ? `${diffstat}\n` : "");
+        await writeTextIfChanged(reviewPath, nextReview);
+        await writeTextIfChanged(githubTitlePath, `${githubTitle}\n`);
+        await writeTextIfChanged(githubBodyPath, githubBody);
+        await writeJsonStableIfChanged(metaPath, nextMeta);
+        return { prDir, resolved };
+    }
+    catch (err) {
+        if (err instanceof CliError)
+            throw err;
+        throw mapBackendError(err, { command: "pr sync", root: opts.rootOverride ?? null });
+    }
+}

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

@@ -1 +1 @@
-{"version":3,"file":"note.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/note.ts"],"names":[],"mappings":"AAUA,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAKpF,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,IAAI,EAAE,MAAM,CAAC;CACd,GAAG,OAAO,CAAC,MAAM,CAAC,CAmDlB"}
+{"version":3,"file":"note.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/note.ts"],"names":[],"mappings":"AAUA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AAgBnC,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,IAAI,EAAE,MAAM,CAAC;CACd,GAAG,OAAO,CAAC,MAAM,CAAC,CAmFlB"}

package/dist/commands/pr/note.js

@@ -6,9 +6,11 @@ import { exitCodeForError } from "../../cli/exit-codes.js";
 import { fileExists } from "../../cli/fs-utils.js";
 import { createCliEmitter, workflowModeMessage } from "../../cli/output.js";
 import { CliError } from "../../shared/errors.js";
-import { loadCommandContext } from "../shared/task-backend.js";
+import { loadBackendTask, loadCommandContext, } from "../shared/task-backend.js";
 import { resolvePrPaths } from "./internal/pr-paths.js";
-import { appendHandoffNote } from "./internal/review-template.js";
+import { appendPrHandoffNote, buildPrHandoffNote, readPrHandoffNotes, } from "./internal/note-store.js";
+import { buildGithubPrTitle, extractAutoSummaryBlock, renderGithubPrBody, renderPrReviewDocument, } from "./internal/review-template.js";
+import { parsePrMeta } from "../shared/pr-meta.js";
 export async function cmdPrNote(opts) {
     try {
         const output = createCliEmitter();
@@ -30,7 +32,13 @@ export async function cmdPrNote(opts) {
         }
         const ctx = opts.ctx ??
             (await loadCommandContext({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null }));
-        const { config, reviewPath, resolved } = await resolvePrPaths({ ...opts, ctx });
+        const { task } = await loadBackendTask({
+            ctx,
+            cwd: opts.cwd,
+            rootOverride: opts.rootOverride,
+            taskId: opts.taskId,
+        });
+        const { config, reviewPath, notesPath, githubTitlePath, githubBodyPath, metaPath, resolved } = await resolvePrPaths({ ...opts, ctx });
         if (config.workflow_mode !== "branch_pr") {
             throw new CliError({
                 exitCode: exitCodeForError("E_USAGE"),
@@ -47,8 +55,33 @@ export async function cmdPrNote(opts) {
             });
         }
         const review = await readFile(reviewPath, "utf8");
-        const updated = appendHandoffNote(review, `${author}: ${body}`);
+        const meta = parsePrMeta(await readFile(metaPath, "utf8"), opts.taskId);
+        await appendPrHandoffNote({
+            notesPath,
+            note: buildPrHandoffNote({
+                createdAt: new Date().toISOString(),
+                author,
+                body,
+            }),
+        });
+        const handoffNotes = await readPrHandoffNotes(notesPath);
+        const autoSummary = extractAutoSummaryBlock(review) ?? "";
+        const updated = renderPrReviewDocument({
+            task,
+            createdAt: meta.created_at ?? "",
+            branch: meta.branch ?? "",
+            handoffNotes,
+            autoSummary,
+        });
+        const githubTitle = buildGithubPrTitle(task);
+        const githubBody = renderGithubPrBody({
+            task,
+            handoffNotes,
+            autoSummary,
+        });
         await atomicWriteFile(reviewPath, updated, "utf8");
+        await atomicWriteFile(githubTitlePath, `${githubTitle}\n`, "utf8");
+        await atomicWriteFile(githubBodyPath, githubBody, "utf8");
         output.success("pr note", opts.taskId);
         return 0;
     }

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":"AAaA,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,2BAA2B,CAAC;AASnC,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;CACjB,GAAG,OAAO,CAAC,MAAM,CAAC,CAuElB"}
+{"version":3,"file":"open.d.ts","sourceRoot":"","sources":["../../../src/commands/pr/open.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,KAAK,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAIhE,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;CACjB,GAAG,OAAO,CAAC,MAAM,CAAC,CA4BlB"}