opencode-magi 0.2.0 → 0.4.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,
     };
 }
@@ -61,10 +62,27 @@ function editValues(input) {
     return {
         ...repositoryValues(input.repository),
         pr: String(input.pr),
+        reviewFindings: input.reviewFindings,
         unresolvedThreads: input.unresolvedThreads,
         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 +94,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 +105,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 +135,7 @@ export async function composeReviewPrompt(input) {
     });
     return [
         task,
+        reviewContextBlock(input.reviewContext),
         languageBlock(input.repository.language),
         personaBlock(input.reviewer.persona),
         await reviewGuidelinesBlock({
@@ -129,6 +158,7 @@ export async function composeRereviewPrompt(input) {
     });
     return [
         task,
+        reviewContextBlock(input.reviewContext),
         input.includeSessionContext === false
             ? ""
             : languageBlock(input.repository.language),
@@ -292,3 +322,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,26 @@ 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.
+- line is optional. Include line only when the finding targets a valid line inside the PR diff hunk.
+- startLine is allowed only when line is present and must also refer to a line inside the PR diff hunk.
+- Omit startLine for single-line findings and omit line for file-level or body-only findings.
+- Use requirementFindings only for missing closing-issue requirements; use findings for ordinary file-level issues that do not map cleanly to a diff line.
 </output_contract>`.trim();
 export const rereviewOutputContract = `
 <output_contract>
@@ -36,15 +46,18 @@ 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.
-- line and startLine must refer to lines inside the latest PR diff hunk.
-- Omit startLine for single-line findings.
+- 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 is optional. Include line only when the newFinding targets a valid line inside the latest PR diff hunk.
+- startLine is allowed only when line is present and must also refer to a line inside the latest PR diff hunk.
+- Omit startLine for single-line findings and omit line for file-level or body-only findings.
+- Use requirementFindings only for missing closing-issue requirements; use newFindings for ordinary file-level issues that do not map cleanly to a diff line.
 </output_contract>`.trim();
 export const findingValidationOutputContract = `
 <output_contract>
@@ -83,14 +96,17 @@ 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.
+- line is optional. Include line only when the finding targets a valid line inside the PR diff hunk.
+- startLine is allowed only when line is present.
+- Omit startLine for single-line findings and omit line for file-level or body-only findings.
 </output_contract>`.trim();
 export const rereviewCloseReconsiderationOutputContract = `
 <output_contract>
@@ -101,14 +117,17 @@ 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.
+- line is optional. Include line only when the newFinding targets a valid line inside the latest PR diff hunk.
+- startLine is allowed only when line is present.
+- Omit startLine for single-line findings and omit line for file-level or body-only findings.
 </output_contract>`.trim();
 export const editOutputContract = `
 <output_contract>
@@ -135,6 +154,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 +216,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 +284,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];