opencode-magi 0.1.0 → 0.3.0

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/{close-reconsideration.md → review/close-reconsideration.md}

@@ -1,6 +1,5 @@
 You requested CLOSE for pull request #{pr} in {owner}/{repo}, but the other reviewers did not.
-Reconsider your decision and choose MERGE or CHANGES_REQUESTED instead.
-Use the existing review session context.
+Reconsider your decision using the existing session context and choose MERGE or CHANGES_REQUESTED instead.
 Original close reason:
 {closeReason}
 Do not edit files or perform write operations.

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

@@ -0,0 +1,13 @@
+Review pull request #{pr} for {owner}/{repo}.
+The PR worktree is {worktreePath}.
+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/action.md

@@ -0,0 +1,5 @@
+Decide the next action for issue #{issue} in {owner}/{repo} from the provided triage result and allowed actions.
+
+<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/dist/prompts/templates/triage/comment-classification.md

@@ -0,0 +1,7 @@
+Classify allowed mention replies for issue #{issue} in {owner}/{repo}.
+
+Use OBJECTION for disagreement or reconsideration requests. Use NEW_EVIDENCE for new logs, reproduction steps, screenshots, links, or use cases. Use CLARIFICATION for answers to questions or ambiguity reduction. Use ACKNOWLEDGEMENT for acceptance or thanks. Use UNRELATED for unrelated content.
+
+<context>
+{context}
+</context>

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

@@ -0,0 +1,5 @@
+Compose one concise final triage comment for issue #{issue} in {owner}/{repo}. Mention @{author}. Do not include markdown fences.
+
+<context>
+{context}
+</context>

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

@@ -0,0 +1,7 @@
+Create an implementation pull request for issue #{issue} in {owner}/{repo}.
+
+Use the checked-out worktree at {worktreePath} and commit your changes. Return the required structured output.
+
+<context>
+{context}
+</context>

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

@@ -0,0 +1,7 @@
+Decide whether issue #{issue} in {owner}/{repo} duplicates one of the provided duplicate candidates.
+
+Use only the provided context. Return DUPLICATE only when the target issue is clearly the same underlying report or request.
+
+<context>
+{context}
+</context>

package/dist/prompts/templates/triage/existing-pr.md

@@ -0,0 +1,7 @@
+Decide whether an existing related pull request already handles issue #{issue} in {owner}/{repo}.
+
+Use only the provided context. Return HANDLE only when the PR clearly addresses the issue.
+
+<context>
+{context}
+</context>

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

@@ -0,0 +1,5 @@
+Compose concrete, actionable questions for issue #{issue} in {owner}/{repo}. Mention @{author}. Do not include markdown fences.
+
+<context>
+{context}
+</context>

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

@@ -0,0 +1,5 @@
+Reconsider issue #{issue} in {owner}/{repo} using the provided previous triage result and triggering comments.
+
+<context>
+{context}
+</context>

package/package.json

@@ -1,15 +1,15 @@
 {
   "name": "opencode-magi",
-  "version": "0.1.0",
+  "version": "0.3.0",
   "description": "Multi-agent PR review and merge orchestration plugin for OpenCode.",
   "license": "MIT",
   "author": "Hirotomo Yamada <hirotomo.yamada@avap.co.jp>",
   "repository": {
     "type": "git",
-    "url": "https://github.com/hirotomoyamada/opencode-magi"
+    "url": "https://github.com/magi-ai/opencode-magi"
   },
   "bugs": {
-    "url": "https://github.com/hirotomoyamada/opencode-magi/issues"
+    "url": "https://github.com/magi-ai/opencode-magi/issues"
   },
   "keywords": [
     "opencode",
@@ -27,24 +27,8 @@
     "dist",
     "schema.json"
   ],
-  "packageManager": "pnpm@10.33.0",
-  "scripts": {
-    "prebuild": "node scripts/copy-data.ts",
-    "build": "tsgo -p tsconfig.build.json",
-    "clean": "rimraf node_modules dist coverage",
-    "format:check": "oxfmt --check .",
-    "format:write": "oxfmt --write .",
-    "lint:check": "oxlint . --max-warnings=0",
-    "lint:fix": "oxlint . --max-warnings=0 --fix",
-    "prepare": "git rev-parse --is-inside-work-tree >/dev/null 2>&1 && lefthook install || true",
-    "quality": "pnpm format:check && pnpm lint:check && pnpm typecheck && pnpm test",
-    "test": "vitest run",
-    "typecheck": "tsgo --noEmit",
-    "release": "changeset publish",
-    "release:version": "changeset version"
-  },
   "dependencies": {
-    "@opencode-ai/plugin": "latest",
+    "@opencode-ai/plugin": "^1.15.5",
     "ajv": "^8.20.0",
     "picomatch": "^4.0.4",
     "valibot": "^1.4.0"
@@ -54,16 +38,33 @@
     "@changesets/cli": "^2.30.0",
     "@commitlint/cli": "^21.0.1",
     "@commitlint/config-conventional": "^21.0.1",
-    "@types/node": "^25.7.0",
+    "@types/node": "^25.9.0",
     "@types/picomatch": "^4.0.3",
-    "@typescript/native-preview": "7.0.0-dev.20260513.1",
+    "@typescript/native-preview": "7.0.0-dev.20260518.1",
+    "@vitest/coverage-v8": "^4.1.7",
+    "@vitest/ui": "^4.1.7",
     "eslint-plugin-perfectionist": "^5.9.0",
     "eslint-plugin-unused-imports": "^4.4.1",
     "lefthook": "^2.1.6",
-    "oxfmt": "^0.49.0",
-    "oxlint": "^1.64.0",
-    "oxlint-tsgolint": "^0.22.1",
+    "oxfmt": "^0.50.0",
+    "oxlint": "^1.65.0",
+    "oxlint-tsgolint": "^0.23.0",
     "rimraf": "^6.1.3",
-    "vitest": "^4.1.5"
+    "vitest": "^4.1.7"
+  },
+  "scripts": {
+    "prebuild": "node scripts/copy-data.ts",
+    "build": "tsgo -p tsconfig.build.json",
+    "clean": "rimraf node_modules dist coverage",
+    "format:check": "oxfmt --check .",
+    "format:write": "oxfmt --write .",
+    "lint:check": "oxlint . --max-warnings=0",
+    "lint:fix": "oxlint . --max-warnings=0 --fix",
+    "quality": "pnpm format:check && pnpm lint:check && pnpm typecheck && pnpm test",
+    "test": "vitest run",
+    "test:dev": "vitest --watch --ui",
+    "typecheck": "tsgo --noEmit",
+    "release": "changeset publish",
+    "release:dev": "changeset version --snapshot dev && changeset publish --tag dev"
   }
-}
+}