agentplane 0.3.9 → 0.3.11

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

@@ -1,62 +1,250 @@
-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,
         "",
-        `Opened by ${opts.author} on ${opts.createdAt}`,
-        `Branch: ${opts.branch}`,
+        "### 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"}`,
+        "",
+        "### Rollback",
+        "",
+        rollbackPlan,
+    ].join("\n");
+}
+function renderReviewSections(opts) {
+    return [
+        SUMMARY_SECTION,
         "",
-        "## Summary",
+        sectionText(opts.task, "Summary", opts.task.title.trim() || "- Not recorded."),
         "",
-        "- ",
+        SCOPE_SECTION,
         "",
-        "## Checklist",
+        sectionText(opts.task, "Scope", "- Not recorded."),
         "",
-        "- [ ] Tests added/updated",
-        "- [ ] Lint/format passes",
-        "- [ ] Verify passed",
-        "- [ ] Docs updated (if needed)",
+        VERIFICATION_SECTION,
         "",
-        "## Handoff Notes",
+        renderVerificationSummary(opts.task),
         "",
-        "<!-- Add review notes here. -->",
+        RISKS_SECTION,
         "",
-        "<!-- BEGIN AUTO SUMMARY -->",
-        "<!-- END AUTO SUMMARY -->",
+        renderRiskSummary(opts.task),
         "",
+        HANDOFF_NOTES_MARKER,
+        "",
+        ...renderPrHandoffNotes(opts.handoffNotes),
+        "",
+    ];
+}
+function renderGithubVerificationSummary(task) {
+    const state = task.verification?.state ?? "pending";
+    const note = typeof task.verification?.note === "string" ? task.verification.note.trim() : "";
+    const statusLine = state === "ok"
+        ? note || "Recorded as passed."
+        : state === "needs_rework"
+            ? note || "Recorded as needs rework."
+            : note || "Not recorded yet.";
+    return [
+        `- State: ${state}`,
+        `- Note: ${statusLine}`,
+        "- Full verification checklist lives in local review.md.",
+    ].join("\n");
+}
+function renderGithubBodySections(opts) {
+    return [
+        SUMMARY_SECTION,
+        "",
+        sectionText(opts.task, "Summary", opts.task.title.trim() || "- Not recorded."),
+        "",
+        SCOPE_SECTION,
+        "",
+        sectionText(opts.task, "Scope", "- Not recorded."),
+        "",
+        VERIFICATION_SECTION,
+        "",
+        renderGithubVerificationSummary(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>",
+        "",
+        `- Updated: ${opts.updatedAt}`,
+        `- Branch: ${opts.branch}`,
+        `- Head: ${opts.headSha ? opts.headSha.slice(0, 12) : "No commits yet"}`,
+        "",
+        "```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"}`,
+        "",
+        ...renderReviewSections({
+            task: opts.task,
+            handoffNotes: opts.handoffNotes ?? [],
+        }),
+        AUTO_SUMMARY_START,
+        opts.autoSummary,
+        AUTO_SUMMARY_END,
+        "",
+    ].join("\n");
+}
+export function renderGithubPrBody(opts) {
+    return [
+        ...renderGithubBodySections({
+            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,
+        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,41 @@
+import { type PrMeta } from "../../shared/pr-meta.js";
+import { type CommandContext } from "../../shared/task-backend.js";
+export type PrRemoteMode = "auto" | "sync-only";
+export type PrOpenOutcome = {
+    action: "linked-existing" | "created" | "sync-only" | "staged";
+    message: string;
+};
+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;
+    remoteMode?: PrRemoteMode;
+}): Promise<{
+    meta: PrMeta;
+    prDir: string;
+    resolved: {
+        gitRoot: string;
+    };
+    openOutcome?: PrOpenOutcome;
+}>;
+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":"AAqBA,OAAO,EAML,KAAK,MAAM,EACZ,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAGL,KAAK,cAAc,EACpB,MAAM,8BAA8B,CAAC;AAsFtC,MAAM,MAAM,YAAY,GAAG,MAAM,GAAG,WAAW,CAAC;AAEhD,MAAM,MAAM,aAAa,GAAG;IAC1B,MAAM,EAAE,iBAAiB,GAAG,SAAS,GAAG,WAAW,GAAG,QAAQ,CAAC;IAC/D,OAAO,EAAE,MAAM,CAAC;CACjB,CAAC;AAyOF,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,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,CA6SD"}