opencode-magi 0.0.0-dev-20260525165911 → 0.0.0-dev-20260525205150

package/dist/config/resolve.js

@@ -113,9 +113,7 @@ export function resolveAgents(config) {
     const agents = config.agents ?? {};
     const editor = config.merge?.editor;
     const creator = config.triage?.creator;
-    const singleReviewAccount = config.review && config.review.mode !== "multi"
-        ? config.review.account
-        : undefined;
+    const singleReviewAccount = config.review && config.mode !== "multi" ? config.account : undefined;
     return {
         editor: editor
             ? {
@@ -188,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",
@@ -208,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 ?? "single",
-        },
         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",
@@ -469,16 +469,15 @@ function validateResolvedReviewers(reviewers, path, errors, mode = "single") {
     }
 }
 function reviewMode(config) {
-    return config.review?.mode === "multi" ? "multi" : "single";
+    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 == null || 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 !== "multi";
+    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 === "multi" ? "multi" : "single";
+    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-20260525165911",
+  "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": "single" },
-        "account": { "type": "string", "minLength": 1 },
         "reviewers": {
           "type": "array",
           "minItems": 3,