opencode-magi 0.2.0 → 0.3.0

package/dist/permissions/editor.json

@@ -1,7 +1,14 @@
 {
     "bash": {
+        "bun *": "allow",
+        "bunx *": "allow",
+        "corepack *": "allow",
         "git add*": "allow",
-        "git commit*": "allow"
+        "git commit*": "allow",
+        "npm *": "allow",
+        "npx *": "allow",
+        "pnpm *": "allow",
+        "yarn *": "allow"
     },
     "edit": "allow"
 }

package/dist/prompts/compose.js

@@ -1,7 +1,7 @@
 import { readFile } from "node:fs/promises";
 import { homedir } from "node:os";
 import { isAbsolute, join } from "node:path";
-import { ciClassificationAfterEditOutputContract, ciClassificationOutputContract, closeReconsiderationOutputContract, editOutputContract, findingValidationOutputContract, rereviewCloseReconsiderationOutputContract, rereviewOutputContract, reviewOutputContract, } from "./contracts";
+import { ciClassificationAfterEditOutputContract, ciClassificationOutputContract, closeReconsiderationOutputContract, editOutputContract, findingValidationOutputContract, rereviewCloseReconsiderationOutputContract, rereviewOutputContract, reviewOutputContract, triageActionOutputContract, triageCommentClassificationOutputContract, triageCreatePrOutputContract, triageDuplicateOutputContract, triageVoteOutputContract, } from "./contracts";
 async function readOptionalPrompt(directory, path, values = {}) {
     if (!path)
         return "";
@@ -45,6 +45,7 @@ function reviewValues(input) {
         headSha: input.headSha,
         jsonEncodedWorktreePath: JSON.stringify(input.worktreePath),
         pr: String(input.pr),
+        reviewContext: input.reviewContext ?? "",
         worktreePath: input.worktreePath,
     };
 }
@@ -65,6 +66,22 @@ function editValues(input) {
         worktreePath: input.worktreePath,
     };
 }
+function triageValues(input) {
+    const categories = input.repository.triage?.categories ?? [];
+    const categoryOptions = categories
+        .map((category) => category.description
+        ? `- ${category.id}: ${category.description}`
+        : `- ${category.id}`)
+        .join("\n");
+    return {
+        ...repositoryValues(input.repository),
+        author: input.author ?? "",
+        categoryOptions,
+        context: input.context,
+        issue: String(input.issue),
+        worktreePath: input.worktreePath ?? "",
+    };
+}
 function personaBlock(persona) {
     return persona ? `<persona>\n${persona}\n</persona>` : "";
 }
@@ -76,6 +93,9 @@ function previousReviewBlock(previousReview) {
         ? `<previous_review>\n${previousReview.trim()}\n</previous_review>`
         : "";
 }
+function reviewContextBlock(reviewContext) {
+    return reviewContext?.trim() ? reviewContext.trim() : "";
+}
 async function reviewGuidelinesBlock(input) {
     const body = (await readOptionalPrompt(input.directory, input.path, input.values)).trim();
     return body ? `<review_guidelines>\n${body}\n</review_guidelines>` : "";
@@ -84,8 +104,15 @@ async function editGuidelinesBlock(input) {
     const body = (await readOptionalPrompt(input.directory, input.path, input.values)).trim();
     return body ? `<edit_guidelines>\n${body}\n</edit_guidelines>` : "";
 }
+async function createGuidelinesBlock(input) {
+    const body = (await readOptionalPrompt(input.directory, input.path, input.values)).trim();
+    return body ? `<create_guidelines>\n${body}\n</create_guidelines>` : "";
+}
 async function sessionContextBlocks(input) {
     return [
+        input.includeSessionContext
+            ? reviewContextBlock(input.values.reviewContext)
+            : "",
         input.includeSessionContext ? languageBlock(input.repository.language) : "",
         input.includeSessionContext ? personaBlock(input.reviewer.persona) : "",
         input.includeReviewGuidelines
@@ -107,6 +134,7 @@ export async function composeReviewPrompt(input) {
     });
     return [
         task,
+        reviewContextBlock(input.reviewContext),
         languageBlock(input.repository.language),
         personaBlock(input.reviewer.persona),
         await reviewGuidelinesBlock({
@@ -129,6 +157,7 @@ export async function composeRereviewPrompt(input) {
     });
     return [
         task,
+        reviewContextBlock(input.reviewContext),
         input.includeSessionContext === false
             ? ""
             : languageBlock(input.repository.language),
@@ -292,3 +321,135 @@ export async function composeCiClassificationAfterEditPrompt(input) {
         .filter(Boolean)
         .join("\n\n");
 }
+async function composeTriageVotePrompt(input) {
+    const values = triageValues(input);
+    const task = await taskBlock({
+        builtin: `triage/${input.builtin}`,
+        customPath: input.customPath,
+        directory: input.directory,
+        values,
+    });
+    return [
+        task,
+        languageBlock(input.repository.language),
+        personaBlock(input.reviewer.persona),
+        input.outputContract,
+    ]
+        .filter(Boolean)
+        .join("\n\n");
+}
+export async function composeTriageActionPrompt(input) {
+    return composeTriageVotePrompt({
+        ...input,
+        builtin: "action",
+        customPath: input.repository.triage?.prompts.action,
+        outputContract: triageActionOutputContract,
+    });
+}
+export async function composeTriageCommentPrompt(input) {
+    const values = triageValues(input);
+    const task = await taskBlock({
+        builtin: "triage/comment",
+        customPath: input.repository.triage?.prompts.comment,
+        directory: input.directory,
+        values,
+    });
+    return [
+        task,
+        languageBlock(input.repository.language),
+        `<context>\n${input.context}\n</context>`,
+    ]
+        .filter(Boolean)
+        .join("\n\n");
+}
+export async function composeTriageQuestionPrompt(input) {
+    const values = triageValues(input);
+    const task = await taskBlock({
+        builtin: "triage/question",
+        customPath: input.repository.triage?.prompts.question,
+        directory: input.directory,
+        values,
+    });
+    return [
+        task,
+        languageBlock(input.repository.language),
+        `<context>\n${input.context}\n</context>`,
+    ]
+        .filter(Boolean)
+        .join("\n\n");
+}
+export async function composeTriageCreatePrPrompt(input) {
+    const values = triageValues(input);
+    const task = await taskBlock({
+        builtin: "triage/create",
+        customPath: input.repository.triage?.prompts.create,
+        directory: input.directory,
+        values,
+    });
+    const persona = input.repository.agents.triageCreator?.persona;
+    return [
+        task,
+        languageBlock(input.repository.language),
+        personaBlock(persona),
+        await createGuidelinesBlock({
+            directory: input.directory,
+            path: input.repository.triage?.prompts.createGuidelines,
+            values,
+        }),
+        triageCreatePrOutputContract,
+    ]
+        .filter(Boolean)
+        .join("\n\n");
+}
+export async function composeTriageExistingPrPrompt(input) {
+    return composeTriageVotePrompt({
+        ...input,
+        builtin: "existing-pr",
+        customPath: input.repository.triage?.prompts.existingPr,
+        outputContract: triageVoteOutputContract('"RELATED_PR_HANDLES_ISSUE" | "RELATED_PR_DOES_NOT_HANDLE_ISSUE"'),
+    });
+}
+export async function composeTriageDuplicatePrompt(input) {
+    return composeTriageVotePrompt({
+        ...input,
+        builtin: "duplicate",
+        customPath: input.repository.triage?.prompts.duplicate,
+        outputContract: triageDuplicateOutputContract,
+    });
+}
+export async function composeTriageCategoryPrompt(input) {
+    const categories = input.repository.triage?.categories ?? [];
+    const votes = ["ASK", ...categories.map((category) => category.id)]
+        .map((vote) => JSON.stringify(vote))
+        .join(" | ");
+    return composeTriageVotePrompt({
+        ...input,
+        builtin: "category",
+        customPath: input.repository.triage?.prompts.category,
+        outputContract: triageVoteOutputContract(votes),
+    });
+}
+export async function composeTriageAcceptancePrompt(input) {
+    return composeTriageVotePrompt({
+        ...input,
+        builtin: "acceptance",
+        customPath: input.repository.triage?.prompts.acceptance,
+        outputContract: triageVoteOutputContract('"YES" | "NO" | "ASK"'),
+    });
+}
+export async function composeTriageCommentClassificationPrompt(input) {
+    return composeTriageVotePrompt({
+        ...input,
+        builtin: "comment-classification",
+        customPath: input.repository.triage?.prompts.commentClassification,
+        outputContract: triageCommentClassificationOutputContract,
+    });
+}
+export async function composeTriageReconsiderPrompt(input) {
+    return composeTriageVotePrompt({
+        ...input,
+        builtin: "reconsider",
+        customPath: input.repository.triage?.prompts.reconsider,
+        outputContract: triageVoteOutputContract('"YES" | "NO" | "ASK"'),
+    });
+}

package/dist/prompts/contracts.js

@@ -15,16 +15,25 @@ The object must match this shape:
       "perspective": "Optional review perspective."
     }
   ],
+  "requirementFindings": [
+    {
+      "issueNumber": 47,
+      "requirement": "Required closing-issue behavior that is missing.",
+      "evidence": "Why the PR does not satisfy the requirement.",
+      "fix": "How to satisfy the requirement."
+    }
+  ],
   "reason": "Required only for CLOSE."
 }
 
 Rules:
-- MERGE requires an empty findings array.
-- CHANGES_REQUESTED requires at least one finding.
-- CLOSE requires a reason and an empty findings array.
+- MERGE requires empty findings and requirementFindings arrays.
+- CHANGES_REQUESTED requires at least one finding or requirementFinding.
+- CLOSE requires a reason and empty findings and requirementFindings arrays.
 - path must be repository-relative.
 - line and startLine must refer to lines inside the PR diff hunk.
 - Omit startLine for single-line findings.
+- Use requirementFindings for missing closing-issue requirements that do not map cleanly to a diff line.
 </output_contract>`.trim();
 export const rereviewOutputContract = `
 <output_contract>
@@ -36,15 +45,17 @@ The object must match this shape:
   "resolve": [{ "commentId": 123, "threadId": "..." }],
   "followUps": [{ "commentId": 123, "body": "..." }],
   "newFindings": [{ "path": "relative/path.ext", "line": 123, "startLine": 120, "body": "..." }],
+  "requirementFindings": [{ "issueNumber": 47, "requirement": "Missing requirement.", "evidence": "Why it is missing.", "fix": "How to fix it." }],
   "reason": "Required only for CLOSE."
 }
 
 Rules:
-- MERGE requires empty followUps and newFindings arrays.
-- CHANGES_REQUESTED requires at least one followUp or newFinding.
-- CLOSE requires a reason and empty followUps and newFindings arrays.
+- MERGE requires empty followUps, newFindings, and requirementFindings arrays.
+- CHANGES_REQUESTED requires at least one followUp, newFinding, or requirementFinding.
+- CLOSE requires a reason and empty followUps, newFindings, and requirementFindings arrays.
 - line and startLine must refer to lines inside the latest PR diff hunk.
 - Omit startLine for single-line findings.
+- Use requirementFindings for missing closing-issue requirements that do not map cleanly to a diff line.
 </output_contract>`.trim();
 export const findingValidationOutputContract = `
 <output_contract>
@@ -83,12 +94,13 @@ The object must match this shape:
       "issue": "What is wrong.",
       "fix": "How to fix it."
     }
-  ]
+  ],
+  "requirementFindings": [{ "issueNumber": 47, "requirement": "Missing requirement.", "evidence": "Why it is missing.", "fix": "How to fix it." }]
 }
 
 Rules:
-- MERGE requires an empty findings array.
-- CHANGES_REQUESTED requires at least one finding.
+- MERGE requires empty findings and requirementFindings arrays.
+- CHANGES_REQUESTED requires at least one finding or requirementFinding.
 - CLOSE is not allowed in this reconsideration step.
 - Omit startLine for single-line findings.
 </output_contract>`.trim();
@@ -101,12 +113,13 @@ The object must match this shape:
   "verdict": "MERGE" | "CHANGES_REQUESTED",
   "resolve": [{ "commentId": 123, "threadId": "..." }],
   "followUps": [{ "commentId": 123, "body": "..." }],
-  "newFindings": [{ "path": "relative/path.ext", "line": 123, "startLine": 120, "body": "..." }]
+  "newFindings": [{ "path": "relative/path.ext", "line": 123, "startLine": 120, "body": "..." }],
+  "requirementFindings": [{ "issueNumber": 47, "requirement": "Missing requirement.", "evidence": "Why it is missing.", "fix": "How to fix it." }]
 }
 
 Rules:
-- MERGE requires empty followUps and newFindings arrays.
-- CHANGES_REQUESTED requires at least one followUp or newFinding.
+- MERGE requires empty followUps, newFindings, and requirementFindings arrays.
+- CHANGES_REQUESTED requires at least one followUp, newFinding, or requirementFinding.
 - CLOSE is not allowed in this reconsideration step.
 - Omit startLine for single-line findings.
 </output_contract>`.trim();
@@ -135,6 +148,32 @@ Rules:
 - responses must include a reply for each thread you addressed.
 - REPLIED requires filesTouched to be empty and at least one DISAGREE or ASK response.
 </output_contract>`.trim();
+export const triageCreatePrOutputContract = `
+<output_contract>
+Return exactly one JSON object and nothing else. Do not wrap it in markdown.
+
+The object must match this shape:
+{
+  "mode": "EDITED" | "REPLIED",
+  "commitSha": "full sha, required only when mode is EDITED; omit when mode is REPLIED",
+  "commitMessage": "fix(scope): short description, required only when mode is EDITED; omit when mode is REPLIED",
+  "filesTouched": ["relative/path.ext"],
+  "pullRequest": {
+    "title": "PR title, required only when mode is EDITED; omit when mode is REPLIED",
+    "body": "PR body, required only when mode is EDITED; omit when mode is REPLIED"
+  },
+  "responses": [{ "commentId": 123, "action": "FIXED" | "DISAGREE" | "ASK", "body": "Fixed." }]
+}
+
+Rules:
+- Use EDITED only when you edited files, staged changes, and committed.
+- Use REPLIED when you only replied without code changes.
+- For EDITED, pullRequest.title and pullRequest.body must be non-empty and follow the repository's PR conventions.
+- Do not push or create the PR. The orchestrator pushes and creates the PR using pullRequest exactly as provided.
+- filesTouched must include every final changed file.
+- responses may be empty when no review threads were addressed.
+- REPLIED requires filesTouched to be empty and at least one DISAGREE or ASK response.
+</output_contract>`.trim();
 export const ciClassificationOutputContract = `
 <output_contract>
 Return exactly one JSON object and nothing else. Do not wrap it in markdown.
@@ -171,6 +210,66 @@ Rules:
 - SCOPE_OUT means the failure is likely flaky, external, or infrastructure-related and may be rerun.
 - If uncertain, choose SCOPE_IN.
 </output_contract>`.trim();
+export function triageVoteOutputContract(votes) {
+    return `
+<output_contract>
+Return exactly one JSON object and nothing else. Do not wrap it in markdown.
+
+The object must match this shape:
+{
+  "vote": ${votes},
+  "reason": "Short rationale."
+}
+</output_contract>`.trim();
+}
+export const triageDuplicateOutputContract = `
+<output_contract>
+Return exactly one JSON object and nothing else. Do not wrap it in markdown.
+
+The object must match this shape:
+{
+  "vote": "DUPLICATE" | "NOT_DUPLICATE",
+  "duplicateOf": 123,
+  "reason": "Short rationale."
+}
+
+Rules:
+- duplicateOf is required only when vote is DUPLICATE.
+- duplicateOf must be one of the provided duplicate candidate issue numbers.
+</output_contract>`.trim();
+export const triageCommentClassificationOutputContract = `
+<output_contract>
+Return exactly one JSON object and nothing else. Do not wrap it in markdown.
+
+The object must match this shape:
+{
+  "comments": [
+    {
+      "commentId": 123,
+      "classification": "OBJECTION" | "NEW_EVIDENCE" | "CLARIFICATION" | "ACKNOWLEDGEMENT" | "UNRELATED",
+      "reason": "Short rationale."
+    }
+  ]
+}
+</output_contract>`.trim();
+export const triageActionOutputContract = `
+<output_contract>
+Return exactly one JSON object and nothing else. Do not wrap it in markdown.
+
+The object must match this shape:
+{
+  "action": "ASK" | "COMMENT" | "CLOSE" | "PR" | "CLEAR_ONLY",
+  "reason": "Short rationale."
+}
+
+Rules:
+- Choose only an action listed as allowed in the task context.
+- ASK means post an author-mentioned question and do not close, create a PR, or clear labels.
+- COMMENT means post a decision comment only.
+- CLOSE means post a decision comment and close the issue.
+- PR means post a decision comment and create an implementation PR.
+- CLEAR_ONLY means clear labels without posting a comment.
+</output_contract>`.trim();
 const outputContractsBySchemaName = {
     "CI classification": ciClassificationOutputContract,
     "close reconsideration": closeReconsiderationOutputContract,
@@ -179,6 +278,14 @@ const outputContractsBySchemaName = {
     rereview: rereviewOutputContract,
     "rereview close reconsideration": rereviewCloseReconsiderationOutputContract,
     review: reviewOutputContract,
+    "triage action": triageActionOutputContract,
+    "triage acceptance": triageVoteOutputContract('"YES" | "NO" | "ASK"'),
+    "triage category": triageVoteOutputContract('"ASK" or one of the configured category IDs'),
+    "triage create PR": triageCreatePrOutputContract,
+    "triage comment classification": triageCommentClassificationOutputContract,
+    "triage duplicate": triageDuplicateOutputContract,
+    "triage existing PR": triageVoteOutputContract('"RELATED_PR_HANDLES_ISSUE" | "RELATED_PR_DOES_NOT_HANDLE_ISSUE"'),
+    "triage reconsider": triageVoteOutputContract('"YES" | "NO" | "ASK"'),
 };
 export function repairPrompt(schemaName) {
     const outputContract = outputContractsBySchemaName[schemaName];

package/dist/prompts/output.js

@@ -71,6 +71,100 @@ function requireNumber(value, path) {
         throw new Error(`${path} must be an integer`);
     return value;
 }
+function parseRequirementFindings(value) {
+    return (value == null ? [] : requireArray(value, "requirementFindings")).map((finding, index) => {
+        const item = finding;
+        return {
+            evidence: requireString(item.evidence, `requirementFindings[${index}].evidence`),
+            fix: requireString(item.fix, `requirementFindings[${index}].fix`),
+            issueNumber: requireNumber(item.issueNumber, `requirementFindings[${index}].issueNumber`),
+            requirement: requireString(item.requirement, `requirementFindings[${index}].requirement`),
+        };
+    });
+}
+function requireOneOf(value, path, values) {
+    const text = requireString(value, path);
+    if (!values.includes(text)) {
+        throw new Error(`${path} must be ${values.join(", ")}`);
+    }
+    return text;
+}
+function parseTriageVote(text, votes) {
+    const data = extractJson(text);
+    if (!data || typeof data !== "object")
+        throw new Error("triage vote output must be an object");
+    return {
+        reason: requireString(data.reason, "reason"),
+        vote: requireOneOf(data.vote, "vote", votes),
+    };
+}
+export function parseTriageExistingPrOutput(text) {
+    return parseTriageVote(text, [
+        "RELATED_PR_DOES_NOT_HANDLE_ISSUE",
+        "RELATED_PR_HANDLES_ISSUE",
+    ]);
+}
+export function parseTriageCategoryOutput(text, categories) {
+    return parseTriageVote(text, ["ASK", ...categories]);
+}
+export function parseTriageBinaryOutput(text) {
+    return parseTriageVote(text, ["ASK", "NO", "YES"]);
+}
+export function parseTriageDuplicateOutput(text) {
+    const data = extractJson(text);
+    if (!data || typeof data !== "object")
+        throw new Error("triage duplicate output must be an object");
+    const vote = requireOneOf(data.vote, "vote", [
+        "DUPLICATE",
+        "NOT_DUPLICATE",
+    ]);
+    const duplicateOf = data.duplicateOf == null
+        ? undefined
+        : requireNumber(data.duplicateOf, "duplicateOf");
+    if (vote === "DUPLICATE" && duplicateOf == null)
+        throw new Error("DUPLICATE requires duplicateOf");
+    return {
+        duplicateOf,
+        reason: requireString(data.reason, "reason"),
+        vote,
+    };
+}
+export function parseTriageCommentClassificationOutput(text) {
+    const data = extractJson(text);
+    if (!data || typeof data !== "object")
+        throw new Error("triage comment classification output must be an object");
+    return {
+        comments: requireArray(data.comments, "comments").map((item, index) => {
+            const value = item;
+            return {
+                classification: requireOneOf(value.classification, `comments[${index}].classification`, [
+                    "ACKNOWLEDGEMENT",
+                    "CLARIFICATION",
+                    "NEW_EVIDENCE",
+                    "OBJECTION",
+                    "UNRELATED",
+                ]),
+                commentId: requireNumber(value.commentId, `comments[${index}].commentId`),
+                reason: requireString(value.reason, `comments[${index}].reason`),
+            };
+        }),
+    };
+}
+export function parseTriageActionOutput(text) {
+    const data = extractJson(text);
+    if (!data || typeof data !== "object")
+        throw new Error("triage action output must be an object");
+    return {
+        action: requireOneOf(data.action, "action", [
+            "ASK",
+            "CLEAR_ONLY",
+            "CLOSE",
+            "COMMENT",
+            "PR",
+        ]),
+        reason: requireString(data.reason, "reason"),
+    };
+}
 export function parseReviewOutput(text) {
     const data = extractJson(text);
     if (!data || typeof data !== "object")
@@ -92,12 +186,17 @@ export function parseReviewOutput(text) {
                 : requireNumber(item.startLine, `findings[${index}].startLine`),
         };
     });
-    if (data.verdict === "MERGE" && findings.length)
-        throw new Error("MERGE requires no findings");
-    if (data.verdict === "CHANGES_REQUESTED" && !findings.length)
-        throw new Error("CHANGES_REQUESTED requires findings");
-    if (data.verdict === "CLOSE" && findings.length)
-        throw new Error("CLOSE requires no findings");
+    const requirementFindings = parseRequirementFindings(data.requirementFindings);
+    if (data.verdict === "MERGE" &&
+        (findings.length || requirementFindings.length))
+        throw new Error("MERGE requires no findings or requirementFindings");
+    if (data.verdict === "CHANGES_REQUESTED" &&
+        !findings.length &&
+        !requirementFindings.length)
+        throw new Error("CHANGES_REQUESTED requires findings or requirementFindings");
+    if (data.verdict === "CLOSE" &&
+        (findings.length || requirementFindings.length))
+        throw new Error("CLOSE requires no findings or requirementFindings");
     const reason = typeof data.reason === "string" && data.reason.trim()
         ? data.reason
         : undefined;
@@ -106,6 +205,7 @@ export function parseReviewOutput(text) {
     return {
         findings,
         reason,
+        requirementFindings,
         verdict: data.verdict,
     };
 }
@@ -138,24 +238,29 @@ export function parseRereviewOutput(text) {
                 : requireNumber(value.startLine, `newFindings[${index}].startLine`),
         };
     });
-    if (data.verdict === "MERGE" && (followUps.length || newFindings.length)) {
-        throw new Error("MERGE requires no followUps or newFindings");
+    const requirementFindings = parseRequirementFindings(data.requirementFindings);
+    if (data.verdict === "MERGE" &&
+        (followUps.length || newFindings.length || requirementFindings.length)) {
+        throw new Error("MERGE requires no followUps, newFindings, or requirementFindings");
     }
-    if (data.verdict === "CLOSE" && (followUps.length || newFindings.length)) {
-        throw new Error("CLOSE requires no followUps or newFindings");
+    if (data.verdict === "CLOSE" &&
+        (followUps.length || newFindings.length || requirementFindings.length)) {
+        throw new Error("CLOSE requires no followUps, newFindings, or requirementFindings");
     }
     if (data.verdict === "CLOSE" && !data.reason) {
         throw new Error("CLOSE requires reason");
     }
     if (data.verdict === "CHANGES_REQUESTED" &&
         !followUps.length &&
-        !newFindings.length) {
-        throw new Error("CHANGES_REQUESTED requires followUps or newFindings");
+        !newFindings.length &&
+        !requirementFindings.length) {
+        throw new Error("CHANGES_REQUESTED requires followUps, newFindings, or requirementFindings");
     }
     return {
         followUps,
         newFindings,
         reason: data.reason == null ? undefined : requireString(data.reason, "reason"),
+        requirementFindings,
         resolve,
         verdict: data.verdict,
     };
@@ -212,7 +317,21 @@ export function parseCiClassificationOutput(text) {
         }),
     };
 }
-export function parseEditOutput(text) {
+function parsePullRequest(value, options) {
+    if (value == null) {
+        if (options.required)
+            throw new Error("pullRequest is required");
+        return undefined;
+    }
+    if (typeof value !== "object")
+        throw new Error("pullRequest must be an object");
+    const pullRequest = value;
+    return {
+        body: requireString(pullRequest.body, "pullRequest.body"),
+        title: requireString(pullRequest.title, "pullRequest.title"),
+    };
+}
+function parseEditOutputWithOptions(text, options) {
     const data = extractJson(text);
     if (!data || typeof data !== "object")
         throw new Error("edit output must be an object");
@@ -230,16 +349,20 @@ export function parseEditOutput(text) {
             commentId: requireNumber(value.commentId, `responses[${index}].commentId`),
         };
     });
-    if (!responses.length)
+    if (options.requireResponses && !responses.length)
         throw new Error("responses must not be empty");
     if (data.mode === "EDITED") {
         if (!filesTouched.length)
             throw new Error("EDITED requires filesTouched");
+        const pullRequest = parsePullRequest(data.pullRequest, {
+            required: options.requirePullRequest,
+        });
         return {
             commitMessage: requireString(data.commitMessage, "commitMessage"),
             commitSha: requireString(data.commitSha, "commitSha"),
             filesTouched,
             mode: data.mode,
+            ...(pullRequest ? { pullRequest } : {}),
             responses,
         };
     }
@@ -258,3 +381,15 @@ export function parseEditOutput(text) {
         responses,
     };
 }
+export function parseEditOutput(text) {
+    return parseEditOutputWithOptions(text, {
+        requirePullRequest: false,
+        requireResponses: true,
+    });
+}
+export function parseTriageCreatePrOutput(text) {
+    return parseEditOutputWithOptions(text, {
+        requirePullRequest: true,
+        requireResponses: false,
+    });
+}

package/dist/prompts/templates/review/review.md

@@ -4,4 +4,10 @@ Review only the diff from {baseSha} to {headSha}.
 Use: git -C {jsonEncodedWorktreePath} diff {baseSha}...{headSha}
 Do not edit files or perform write operations.
 
+This PR may include closing issue references.
+For each closing issue, review whether the PR fully satisfies the issue body, acceptance criteria, required behavior, required tests, required documentation, and bounded issue comments.
+Request changes if a closing issue requirement is missing, only documented, only schema-exposed, or not wired into runtime behavior.
+Do not approve solely because the PR improves the codebase if it claims to close an issue that remains incomplete.
+For referenced non-closing issues, use them as context only unless the PR body explicitly claims to complete them.
+
 {ciFailureContextBlock}

package/dist/prompts/templates/triage/acceptance.md

@@ -0,0 +1,7 @@
+Evaluate issue #{issue} in {owner}/{repo} for the selected category.
+
+Choose YES when the issue should be accepted for the project. Choose NO when it should be rejected, is not actionable, or is not appropriate for this project. Choose ASK when specific missing information is required before deciding.
+
+<context>
+{context}
+</context>