opencode-magi 0.0.0-dev-20260525164734 → 0.0.0-dev-20260525205150

package/README.md

@@ -64,20 +64,18 @@ Add the following content to the configuration file.
   "agents": {
     "refs": {
       "account-1": {
-        "model": "openai/gpt-5.5",
-        "account": "account-1"
+        "model": "openai/gpt-5.5"
       },
       "account-2": {
-        "model": "anthropic/claude-opus-4-7",
-        "account": "account-2"
+        "model": "anthropic/claude-opus-4-7"
       },
       "account-3": {
-        "model": "opencode/kimi-k2-6",
-        "account": "account-3"
+        "model": "opencode/kimi-k2-6"
       }
     }
   },
   "review": {
+    "account": "your-account",
     "reviewers": [
       { "ref": "account-1" },
       { "ref": "account-2" },
@@ -87,19 +85,22 @@ Add the following content to the configuration file.
 }
 ```
 
-After `refs` are expanded, `review.reviewers[].account` is the GitHub account used to post reviews and approvals in the default `review.mode: "multi"`. Must be authenticated with `gh auth token --user <account>` and must be unique.
+By default, `review.mode` is `"single"`. Magi uses one `review.account` to post reviewer-originated GitHub mutations while still running multiple logical reviewer agents and preserving majority voting, finding validation, and close reconsideration. The account must be authenticated with `gh auth token --user <account>`.
 
-For individual setups, use `review.mode: "single"` with one `review.account`. Magi still runs an odd number of at least 3 logical reviewer agents and keeps majority voting, finding validation, and close reconsideration, but GitHub branch protection sees only the one configured review account.
+For team setups that need separate GitHub review identities, set `review.mode: "multi"` and configure a unique account for each reviewer.
 
 ```json
 {
   "review": {
-    "mode": "single",
-    "account": "your-account",
+    "mode": "multi",
     "reviewers": [
-      { "id": "general", "model": "openai/gpt-5.5" },
-      { "id": "security", "model": "anthropic/claude-opus-4-7" },
-      { "id": "compat", "model": "opencode/kimi-k2-6" }
+      { "id": "general", "model": "openai/gpt-5.5", "account": "account-1" },
+      {
+        "id": "security",
+        "model": "anthropic/claude-opus-4-7",
+        "account": "account-2"
+      },
+      { "id": "compat", "model": "opencode/kimi-k2-6", "account": "account-3" }
     ]
   }
 }

package/dist/config/resolve.js

@@ -113,7 +113,7 @@ export function resolveAgents(config) {
     const agents = config.agents ?? {};
     const editor = config.merge?.editor;
     const creator = config.triage?.creator;
-    const singleReviewAccount = config.review?.mode === "single" ? config.review.account : undefined;
+    const singleReviewAccount = config.review && config.mode !== "multi" ? config.account : undefined;
     return {
         editor: editor
             ? {
@@ -186,6 +186,8 @@ export function resolveRepository(config) {
             repo: config.github.repo,
         },
         language: config.language,
+        account: config.account,
+        mode: config.mode ?? "single",
         merge: {
             approvalPolicy: config.review?.merge?.approvalPolicy ?? "majority",
             method: config.review?.merge?.method ?? "squash",
@@ -206,10 +208,6 @@ export function resolveRepository(config) {
             review: config.review?.prompts?.review,
             reviewGuidelines: config.review?.prompts?.reviewGuidelines,
         },
-        review: {
-            account: config.review?.account,
-            mode: config.review?.mode ?? "multi",
-        },
         reviewAutomation: {
             close: config.review?.automation?.close ?? false,
             merge: config.review?.automation?.merge ?? true,

package/dist/config/validate.js

@@ -13,11 +13,13 @@ const TRIAGE_CATEGORY_ID_PATTERN = /^[A-Za-z0-9_-]+$/;
 const RESERVED_TRIAGE_CATEGORY_IDS = new Set(["ASK", "none"]);
 const CONFIG_KEYS = new Set([
     "$schema",
+    "account",
     "agents",
     "clear",
     "github",
     "language",
     "merge",
+    "mode",
     "output",
     "review",
     "triage",
@@ -54,12 +56,10 @@ const TRIAGE_CREATOR_KEYS = new Set([
 const AUTHOR_KEYS = new Set(["email", "name"]);
 const GITHUB_KEYS = new Set(["apiRetryAttempts", "host", "owner", "repo"]);
 const REVIEW_KEYS = new Set([
-    "account",
     "automation",
     "checks",
     "concurrency",
     "merge",
-    "mode",
     "output",
     "prompts",
     "reviewers",
@@ -386,7 +386,7 @@ function validateAndNormalizeModel(target, path, errors, catalog) {
     }
     errors.push(`${path} must contain at least one usable OpenCode model candidate${candidateErrors.length ? ` (${candidateErrors.join("; ")})` : ""}`);
 }
-function validateReviewerList(reviewers, path, errors, catalog, mode = "multi") {
+function validateReviewerList(reviewers, path, errors, catalog, mode = "single") {
     if (reviewers == null)
         return;
     if (!Array.isArray(reviewers)) {
@@ -456,7 +456,7 @@ function validateTriageAgentList(voters, path, errors, catalog) {
         }
     });
 }
-function validateResolvedReviewers(reviewers, path, errors, mode = "multi") {
+function validateResolvedReviewers(reviewers, path, errors, mode = "single") {
     const keys = new Set();
     const accounts = new Set();
     for (const reviewer of reviewers) {
@@ -469,16 +469,15 @@ function validateResolvedReviewers(reviewers, path, errors, mode = "multi") {
     }
 }
 function reviewMode(config) {
-    return config.review?.mode === "single" ? "single" : "multi";
+    return config.mode === "multi" ? "multi" : "single";
 }
 function validateReviewIdentity(config, errors) {
-    const mode = config.review?.mode;
+    const mode = config.mode;
     if (mode != null && mode !== "multi" && mode !== "single") {
-        errors.push("review.mode must be multi or single");
+        errors.push("mode must be multi or single");
     }
-    validateString(config.review?.account, "review.account", errors);
-    if (mode === "single" && !config.review?.account) {
-        errors.push("review.account is required when review.mode is single");
+    if ((mode == null || mode === "single") && !config.account) {
+        errors.push("account is required when mode is single");
     }
 }
 function validateResolvedTriageAgents(agents, path, errors) {
@@ -924,8 +923,8 @@ async function validateAuth(config, exec, errors) {
     const accounts = new Set();
     const agents = resolveAgents(config);
     if (reviewMode(config) === "single") {
-        if (config.review?.account)
-            accounts.add(config.review.account);
+        if (config.account)
+            accounts.add(config.account);
     }
     else {
         for (const reviewer of agents.reviewers)
@@ -978,8 +977,8 @@ async function validateRepositoryPermissions(config, exec, errors, warnings) {
         return;
     const agents = resolveAgents(config);
     const reviewAccounts = reviewMode(config) === "single"
-        ? config.review?.account
-            ? [config.review.account]
+        ? config.account
+            ? [config.account]
             : []
         : agents.reviewers.map((reviewer) => reviewer.account);
     await Promise.all(reviewAccounts.map(async (account) => {
@@ -1040,6 +1039,7 @@ export async function validateConfig(config, options = {}) {
         validateJsonSchema(config, errors);
     validateKnownKeys(config, "config", CONFIG_KEYS, errors);
     validateString(config.$schema, "$schema", errors);
+    validateString(config.account, "account", errors);
     validateString(config.language, "language", errors);
     if (config.agents != null && !isPlainObject(config.agents)) {
         errors.push("agents must be an object");

package/dist/orchestrator/merge.js

@@ -213,7 +213,7 @@ async function runRereview(input, worktreePath, previousHeadSha, cycle, sessionI
         worktreePath,
     });
     const artifactDir = outputDir(input);
-    const singleReviewMode = input.repository.review?.mode === "single";
+    const singleReviewMode = input.repository.mode !== "multi";
     const reviewerKeys = input.repository.agents.reviewers.map((reviewer) => reviewer.key);
     const singleModeThreads = singleReviewMode
         ? assignThreadsByReviewFindingMarker({
@@ -221,7 +221,7 @@ async function runRereview(input, worktreePath, previousHeadSha, cycle, sessionI
             pr: input.pr,
             reviewerKeys,
             threads: options.dryRunThreads == null
-                ? await fetchUnresolvedThreads(input.exec, input.repository, input.pr, input.repository.review?.account ?? "")
+                ? await fetchUnresolvedThreads(input.exec, input.repository, input.pr, input.repository.account ?? "")
                 : Object.values(options.dryRunThreads).flat(),
         })
         : undefined;
@@ -412,7 +412,7 @@ async function runRereview(input, worktreePath, previousHeadSha, cycle, sessionI
             ? { consensus: `dry-run:would-post-single-review:${verdict}` }
             : {
                 consensus: await (async () => {
-                    const account = input.repository.review?.account ?? "";
+                    const account = input.repository.account ?? "";
                     await Promise.all(entries.flatMap((entry) => entry.output.resolve.map((item) => resolveThread(input.exec, input.repository, account, item.threadId))));
                     await Promise.all(entries.flatMap((entry) => entry.output.followUps.map((item) => postReply(input.exec, input.repository, input.pr, account, item.commentId, [
                         `**Reviewer:** ${entry.reviewer}`,

package/dist/orchestrator/review.js

@@ -16,12 +16,14 @@ import { formatReviewReport } from "./report";
 import { buildReviewContextSnapshot, renderReviewContext, } from "./review-context";
 import { checkSafetyGate, hasSafetyGate } from "./safety";
 function resolvedReviewMode(repository) {
-    return repository.review?.mode === "single" ? "single" : "multi";
+    return repository.mode === "multi" ? "multi" : "single";
 }
 export function reviewPostingAccount(repository, reviewer) {
-    return resolvedReviewMode(repository) === "single"
-        ? (repository.review?.account ?? reviewer.account)
-        : reviewer.account;
+    if (resolvedReviewMode(repository) !== "single")
+        return reviewer.account;
+    if (repository.account)
+        return repository.account;
+    throw new Error("account is required for single review mode");
 }
 function reviewAssignmentKey(repository, reviewer) {
     return resolvedReviewMode(repository) === "single"
@@ -579,9 +581,9 @@ function singleFindingBody(input) {
     ].join("\n");
 }
 export async function postSingleConsensusReview(input) {
-    const account = input.repository.review?.account;
+    const account = input.repository.account;
     if (!account)
-        throw new Error("review.account is required for single review mode");
+        throw new Error("account is required for single review mode");
     const body = singleReviewBody(input);
     if (input.verdict === "MERGE") {
         return postApproval(input.exec, input.repository, input.pr, account, body);
@@ -927,7 +929,7 @@ export async function runReview(input) {
     const reviewerAccounts = input.repository.agents.reviewers.map((reviewer) => reviewer.account);
     const preliminaryMode = singleReviewMode
         ? resolveSingleAccountReviewMode({
-            account: input.repository.review?.account ?? "",
+            account: input.repository.account ?? "",
             current: freshnessTarget,
             pr: input.pr,
             reviewerKeys,
@@ -942,7 +944,7 @@ export async function runReview(input) {
         return (preliminaryMode.assignments.get(reviewAssignmentKey(input.repository, reviewer))?.type === "skip");
     });
     if (singleReviewMode) {
-        const account = input.repository.review?.account ?? "";
+        const account = input.repository.account ?? "";
         const threads = await fetchUnresolvedThreads(exec, input.repository, input.pr, account);
         const assigned = assignThreadsByReviewFindingMarker({
             fallbackReviewerKeys: reviewerKeys,
@@ -973,7 +975,7 @@ export async function runReview(input) {
     const mode = singleReviewMode
         ? pendingThreadReplyReviewers.size
             ? resolveSingleAccountReviewMode({
-                account: input.repository.review?.account ?? "",
+                account: input.repository.account ?? "",
                 current: freshnessTarget,
                 pendingReviewers: pendingThreadReplyReviewers,
                 pr: input.pr,
@@ -1368,7 +1370,7 @@ export async function runReview(input) {
                         consensus: input.dryRun
                             ? `dry-run:would-post-single-review:${verdict}`
                             : await (async () => {
-                                const account = input.repository.review?.account ?? "";
+                                const account = input.repository.account ?? "";
                                 await Promise.all(Object.values(activeOutputs).flatMap((output) => {
                                     if (!("resolve" in output))
                                         return [];
@@ -1414,7 +1416,7 @@ export async function runReview(input) {
                 ]))),
             };
         const automationAccount = singleReviewMode
-            ? input.repository.review?.account
+            ? input.repository.account
             : input.repository.agents.reviewers[0]?.account;
         const enableReviewAutomation = input.enableReviewAutomation ?? true;
         if (enableReviewAutomation &&

package/package.json

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

package/schema.json

@@ -5,6 +5,7 @@
   "additionalProperties": false,
   "properties": {
     "$schema": { "type": "string" },
+    "account": { "type": "string", "minLength": 1 },
     "agents": {
       "type": "object",
       "additionalProperties": false,
@@ -44,6 +45,7 @@
     },
     "language": { "type": "string" },
     "merge": { "$ref": "#/$defs/merge" },
+    "mode": { "enum": ["multi", "single"], "default": "single" },
     "output": {
       "type": "object",
       "additionalProperties": false,
@@ -366,8 +368,6 @@
       "type": "object",
       "additionalProperties": false,
       "properties": {
-        "mode": { "enum": ["multi", "single"], "default": "multi" },
-        "account": { "type": "string", "minLength": 1 },
         "reviewers": {
           "type": "array",
           "minItems": 3,