opencode-magi 0.0.0-dev-20260520165753 → 0.0.0-dev-20260520173258

package/dist/prompts/output.js

@@ -71,6 +71,17 @@ 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)) {
@@ -93,18 +104,8 @@ export function parseTriageExistingPrOutput(text) {
         "RELATED_PR_HANDLES_ISSUE",
     ]);
 }
-export function parseTriageKindOutput(text) {
-    return parseTriageVote(text, ["ASK", "BUG", "FEATURE"]);
-}
-export function parseTriageFinalOutput(text) {
-    return parseTriageVote(text, [
-        "ASK",
-        "BUG_ACCEPTED",
-        "BUG_REJECTED",
-        "DUPLICATE",
-        "FEATURE_ACCEPTED",
-        "FEATURE_REJECTED",
-    ]);
+export function parseTriageCategoryOutput(text, categories) {
+    return parseTriageVote(text, ["ASK", ...categories]);
 }
 export function parseTriageBinaryOutput(text) {
     return parseTriageVote(text, ["ASK", "NO", "YES"]);
@@ -185,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;
@@ -199,6 +205,7 @@ export function parseReviewOutput(text) {
     return {
         findings,
         reason,
+        requirementFindings,
         verdict: data.verdict,
     };
 }
@@ -231,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,
     };

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>

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

@@ -0,0 +1,10 @@
+Classify issue #{issue} in {owner}/{repo}.
+
+Choose ASK when more information is required to classify what should be done. Otherwise choose exactly one configured category ID.
+
+Configured categories:
+{categoryOptions}
+
+<context>
+{context}
+</context>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-magi",
-  "version": "0.0.0-dev-20260520165753",
+  "version": "0.0.0-dev-20260520173258",
   "description": "Multi-agent PR review and merge orchestration plugin for OpenCode.",
   "license": "MIT",
   "author": "Hirotomo Yamada <hirotomo.yamada@avap.co.jp>",

package/schema.json

@@ -194,9 +194,8 @@
       "properties": {
         "existingPr": { "type": "string" },
         "duplicate": { "type": "string" },
-        "kind": { "type": "string" },
-        "bug": { "type": "string" },
-        "feature": { "type": "string" },
+        "category": { "type": "string" },
+        "acceptance": { "type": "string" },
         "action": { "type": "string" },
         "question": { "type": "string" },
         "comment": { "type": "string" },
@@ -205,20 +204,19 @@
         "createPr": { "type": "string" }
       }
     },
-    "triageKindRule": {
+    "triageCategory": {
       "type": "object",
       "additionalProperties": false,
+      "required": ["id"],
       "properties": {
-        "label": { "type": "array", "items": { "type": "string" } },
-        "type": { "type": "array", "items": { "type": "string" } }
-      }
-    },
-    "triageKind": {
-      "type": "object",
-      "additionalProperties": false,
-      "properties": {
-        "bug": { "$ref": "#/$defs/triageKindRule" },
-        "feature": { "$ref": "#/$defs/triageKindRule" }
+        "id": {
+          "type": "string",
+          "pattern": "^[A-Za-z0-9_-]+$",
+          "not": { "enum": ["ASK", "none"] }
+        },
+        "labels": { "type": "array", "items": { "type": "string" } },
+        "types": { "type": "array", "items": { "type": "string" } },
+        "description": { "type": "string" }
       }
     },
     "triageAutomation": {
@@ -308,7 +306,10 @@
           "items": { "$ref": "#/$defs/triageAgent" }
         },
         "creator": { "$ref": "#/$defs/triageCreator" },
-        "kind": { "$ref": "#/$defs/triageKind" },
+        "categories": {
+          "type": "array",
+          "items": { "$ref": "#/$defs/triageCategory" }
+        },
         "automation": { "$ref": "#/$defs/triageAutomation" },
         "safety": { "$ref": "#/$defs/triageSafety" },
         "concurrency": { "$ref": "#/$defs/triageConcurrency" },

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

@@ -1,7 +0,0 @@
-Evaluate bug issue #{issue} in {owner}/{repo}.
-
-Choose YES when the bug is reproduced or otherwise valid based on strong evidence. Choose NO when it is not reproduced, invalid, a misunderstanding, or not actionable as a bug. Choose ASK when specific missing information is required.
-
-<context>
-{context}
-</context>

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

@@ -1,7 +0,0 @@
-Evaluate feature issue #{issue} in {owner}/{repo}.
-
-Choose YES when the feature should be implemented. Choose NO when it should not be implemented for this project. Choose ASK when specific missing information is required.
-
-<context>
-{context}
-</context>

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

@@ -1,7 +0,0 @@
-Classify issue #{issue} in {owner}/{repo}.
-
-Choose BUG when it reports broken existing behavior. Choose FEATURE when it requests new behavior or enhancement. Choose ASK when more information is required to classify it.
-
-<context>
-{context}
-</context>