opencode-magi 0.0.0-dev-20260525025216 → 0.0.0-dev-20260525030452

package/README.md

@@ -78,7 +78,7 @@ Add the following content to the configuration file.
     }
   },
   "review": {
-    "agents": [
+    "reviewers": [
       { "ref": "account-1" },
       { "ref": "account-2" },
       { "ref": "account-3" }
@@ -87,7 +87,7 @@ Add the following content to the configuration file.
 }
 ```
 
-After refs are expanded, `review.agents[].account` is the GitHub account used to post reviews and approvals. Must be authenticated with `gh auth token --user <account>` and must be unique.
+After `refs` are expanded, `review.reviewers[].account` is the GitHub account used to post reviews and approvals. Must be authenticated with `gh auth token --user <account>` and must be unique.
 
 #### Set project config
 
@@ -133,7 +133,7 @@ Add the following content to the configuration file.
     }
   },
   "review": {
-    "agents": [
+    "reviewers": [
       { "ref": "account-1" },
       { "ref": "account-2" },
       { "ref": "account-3" }
@@ -143,7 +143,7 @@ Add the following content to the configuration file.
     "editor": { "ref": "account-4" }
   },
   "triage": {
-    "agents": [
+    "voters": [
       { "ref": "account-1" },
       { "ref": "account-2" },
       { "ref": "account-3" }
@@ -168,7 +168,7 @@ Entries with `ref` are expanded from `agents.refs`. Fields set alongside `ref` o
 }
 ```
 
-After refs are expanded, `review.agents[].account` is the GitHub account used to post reviews and approvals. Must be authenticated with `gh auth token --user <account>` and must be unique. `merge.editor.account` is used by `/magi:merge` to push fixes, close PRs, and merge PRs.
+After `refs` are expanded, `review.reviewers[].account` is the GitHub account used to post reviews and approvals. Must be authenticated with `gh auth token --user <account>` and must be unique. `merge.editor.account` is used by `/magi:merge` to push fixes, close PRs, and merge PRs.
 
 #### Validate config
 

package/dist/config/resolve.js

@@ -96,14 +96,14 @@ export function resolveAgents(config) {
                 permission: resolveEditorPermission(agents, editor),
             }
             : undefined,
-        reviewers: (config.review?.agents ?? []).map((reviewer, index) => ({
+        reviewers: (config.review?.reviewers ?? []).map((reviewer, index) => ({
             ...reviewer,
             key: reviewerKey(reviewer, index),
             index,
             model: normalizedModel(reviewer.model),
             permission: resolveReviewerPermission(agents, reviewer),
         })),
-        triage: (config.triage?.agents ?? []).map((agent, index) => ({
+        triage: (config.triage?.voters ?? []).map((agent, index) => ({
             ...agent,
             key: triageAgentKey(agent, index),
             index,

package/dist/config/validate.js

@@ -54,13 +54,13 @@ 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([
-    "agents",
     "automation",
     "checks",
     "concurrency",
     "merge",
     "output",
     "prompts",
+    "reviewers",
     "safety",
     "worktree",
 ]);
@@ -72,7 +72,6 @@ const MERGE_KEYS = new Set([
     "prompts",
 ]);
 const TRIAGE_KEYS = new Set([
-    "agents",
     "automation",
     "categories",
     "concurrency",
@@ -81,6 +80,7 @@ const TRIAGE_KEYS = new Set([
     "prompts",
     "reporter",
     "safety",
+    "voters",
     "worktree",
 ]);
 const REVIEW_MERGE_KEYS = new Set([
@@ -187,14 +187,14 @@ function expandAgentRefs(config, errors) {
     const refsValue = isPlainObject(agents) ? agents.refs : undefined;
     const refsInvalid = refsValue != null && !isPlainObject(refsValue);
     const refs = isPlainObject(refsValue) ? refsValue : undefined;
-    if (Array.isArray(magiConfig.review?.agents)) {
-        magiConfig.review.agents = magiConfig.review.agents.map((agent, index) => expandAgentRefUse(agent, `review.agents[${index}]`, refs, refsInvalid, errors));
+    if (Array.isArray(magiConfig.review?.reviewers)) {
+        magiConfig.review.reviewers = magiConfig.review.reviewers.map((agent, index) => expandAgentRefUse(agent, `review.reviewers[${index}]`, refs, refsInvalid, errors));
     }
     if (isPlainObject(magiConfig.merge?.editor)) {
         magiConfig.merge.editor = expandAgentRefUse(magiConfig.merge.editor, "merge.editor", refs, refsInvalid, errors);
     }
-    if (Array.isArray(magiConfig.triage?.agents)) {
-        magiConfig.triage.agents = magiConfig.triage.agents.map((agent, index) => expandAgentRefUse(agent, `triage.agents[${index}]`, refs, refsInvalid, errors));
+    if (Array.isArray(magiConfig.triage?.voters)) {
+        magiConfig.triage.voters = magiConfig.triage.voters.map((agent, index) => expandAgentRefUse(agent, `triage.voters[${index}]`, refs, refsInvalid, errors));
     }
     if (isPlainObject(magiConfig.triage?.creator)) {
         magiConfig.triage.creator = expandAgentRefUse(magiConfig.triage.creator, "triage.creator", refs, refsInvalid, errors);
@@ -387,18 +387,18 @@ function validateReviewerList(reviewers, path, errors, catalog) {
         }
     });
 }
-function validateTriageAgentList(agents, path, errors, catalog) {
-    if (agents == null)
+function validateTriageAgentList(voters, path, errors, catalog) {
+    if (voters == null)
         return;
-    if (!Array.isArray(agents)) {
+    if (!Array.isArray(voters)) {
         errors.push(`${path} must be an array`);
         return;
     }
-    if (agents.length < 3)
-        errors.push(`${path} must contain at least 3 agents`);
-    if (agents.length % 2 === 0)
-        errors.push(`${path} must contain an odd number of agents`);
-    agents.forEach((agent, index) => {
+    if (voters.length < 3)
+        errors.push(`${path} must contain at least 3 voters`);
+    if (voters.length % 2 === 0)
+        errors.push(`${path} must contain an odd number of voters`);
+    voters.forEach((agent, index) => {
         if (!agent || typeof agent !== "object") {
             errors.push(`${path}[${index}] must be an object`);
             return;
@@ -722,11 +722,11 @@ function validateTriage(config, errors, options) {
     const creator = triage.creator;
     const reporter = typeof triage.reporter === "string" ? triage.reporter : undefined;
     const safety = triage.safety;
-    if (!triage.agents)
-        errors.push("triage.agents is required");
-    validateTriageAgentList(triage.agents, "triage.agents", errors, options.modelCatalog);
-    if (Array.isArray(triage.agents)) {
-        const resolvedTriageAgents = triage.agents.map((agent, index) => ({
+    if (!triage.voters)
+        errors.push("triage.voters is required");
+    validateTriageAgentList(triage.voters, "triage.voters", errors, options.modelCatalog);
+    if (Array.isArray(triage.voters)) {
+        const resolvedTriageAgents = triage.voters.map((agent, index) => ({
             account: agent && typeof agent === "object" && typeof agent.account === "string"
                 ? agent.account
                 : "",
@@ -735,7 +735,7 @@ function validateTriage(config, errors, options) {
         validateResolvedTriageAgents(resolvedTriageAgents, "triage.resolvedAgents", errors);
         if (reporter != null &&
             !resolvedTriageAgents.some((agent) => agent.key === reporter)) {
-            errors.push(`triage.reporter must match a triage agent key: ${reporter}`);
+            errors.push(`triage.reporter must match a triage voter key: ${reporter}`);
         }
     }
     validateString(triage.reporter, "triage.reporter", errors);
@@ -926,11 +926,11 @@ export async function validateConfig(config, options = {}) {
         else {
             validateKnownKeys(config.review, "review", REVIEW_KEYS, errors);
         }
-        if (!config.review.agents)
-            errors.push("review.agents is required");
-        validateReviewerList(config.review.agents, "review.agents", errors, options.modelCatalog);
-        if (Array.isArray(config.review.agents)) {
-            validateResolvedReviewers(config.review.agents.map((reviewer, index) => ({
+        if (!config.review.reviewers)
+            errors.push("review.reviewers is required");
+        validateReviewerList(config.review.reviewers, "review.reviewers", errors, options.modelCatalog);
+        if (Array.isArray(config.review.reviewers)) {
+            validateResolvedReviewers(config.review.reviewers.map((reviewer, index) => ({
                 account: reviewer &&
                     typeof reviewer === "object" &&
                     typeof reviewer.account === "string"
@@ -939,7 +939,7 @@ export async function validateConfig(config, options = {}) {
                 key: reviewer && typeof reviewer === "object"
                     ? reviewerKey(reviewer, index)
                     : "",
-            })), "review.resolvedAgents", errors);
+            })), "review.resolvedReviewers", errors);
         }
     }
     if (options.requireTriage && !config.triage) {

package/dist/index.js

@@ -381,7 +381,7 @@ export async function validateMagiConfigFiles(directory, options = {}) {
                 ? withGitHubApiRetry(options.exec, mergedConfig.github?.apiRetryAttempts ?? 3)
                 : undefined,
             modelCatalog: options.modelCatalog,
-            requireGithub: hasProjectConfig && Boolean(mergedConfig.review?.agents),
+            requireGithub: hasProjectConfig && Boolean(mergedConfig.review?.reviewers),
             requireModelCatalog: true,
             requireWorktreeConfig: true,
         });
@@ -547,7 +547,7 @@ export const MagiPlugin = async ({ client, directory }) => {
                 },
             }),
             magi_triage: tool({
-                description: "Triage one or more GitHub issues with configured Magi triage agents.",
+                description: "Triage one or more GitHub issues with configured Magi triage voters.",
                 args: {
                     issues: tool.schema.string(),
                     dryRun: tool.schema.boolean().optional(),

package/dist/orchestrator/run-manager.js

@@ -1616,16 +1616,16 @@ export class MagiRunManager {
             }));
         }
         if (progress.type === "triage_agent_started") {
-            await this.notify(state, `**Triage agent ${progress.voter}** started ${progress.phase} for ${issue}.`);
+            await this.notify(state, `**Triage voter ${progress.voter}** started ${progress.phase} for ${issue}.`);
         }
         if (progress.type === "triage_agent_repair") {
-            await this.notify(state, `**Triage agent ${progress.voter}** started JSON regeneration for ${issue}.`);
+            await this.notify(state, `**Triage voter ${progress.voter}** started JSON regeneration for ${issue}.`);
         }
         if (progress.type === "triage_agent_completed") {
-            await this.notify(state, `**Triage agent ${progress.voter}** completed ${progress.phase} for ${issue}: ${progress.vote}.`);
+            await this.notify(state, `**Triage voter ${progress.voter}** completed ${progress.phase} for ${issue}: ${progress.vote}.`);
         }
         if (progress.type === "triage_agent_failed") {
-            await this.notify(state, `**Triage agent ${progress.voter}** failed ${progress.phase} for ${issue}: ${redactSecrets(progress.error)}`);
+            await this.notify(state, `**Triage voter ${progress.voter}** failed ${progress.phase} for ${issue}: ${redactSecrets(progress.error)}`);
         }
         if (progress.type === "comment_posting") {
             await this.notify(state, `Posting triage comment for ${issue}.`);

package/dist/orchestrator/triage.js

@@ -217,7 +217,7 @@ export function chooseDuplicateOutput(input) {
 async function runDuplicateVote(input) {
     const agents = input.input.repository.agents.triage;
     if (!agents?.length)
-        throw new Error("triage.agents is required");
+        throw new Error("triage.voters is required");
     await emitProgress(input.input, { phase: "duplicate", type: "phase" });
     const outputs = await Promise.all(agents.map((agent) => runVote({
         agent,
@@ -259,7 +259,7 @@ async function runDuplicateVote(input) {
 async function runPhaseVote(input) {
     const agents = input.input.repository.agents.triage;
     if (!agents?.length)
-        throw new Error("triage.agents is required");
+        throw new Error("triage.voters is required");
     await emitProgress(input.input, { phase: input.phase, type: "phase" });
     const promptTexts = await Promise.all(agents.map((agent) => input.prompt({
         context: input.context,
@@ -525,7 +525,7 @@ async function runReconsiderationVote(input) {
 function triageReporter(repository, issue) {
     const agents = repository.agents.triage ?? [];
     if (!agents.length)
-        throw new Error("triage.agents is required");
+        throw new Error("triage.voters is required");
     const configured = repository.triage?.reporter;
     const reporter = configured
         ? agents.find((agent) => agent.key === configured)
@@ -563,7 +563,7 @@ function decisionCommentFallback(input) {
 function agentForKey(repository, key) {
     const agent = repository.agents.triage?.find((item) => item.key === key);
     if (!agent)
-        throw new Error(`Unknown triage agent: ${key}`);
+        throw new Error(`Unknown triage voter: ${key}`);
     return agent;
 }
 function askOutputs(outputs) {
@@ -905,7 +905,7 @@ export async function runTriage(input) {
         throw new Error("triage configuration is required");
     const agents = input.repository.agents.triage;
     if (!agents?.length)
-        throw new Error("triage.agents is required");
+        throw new Error("triage.voters is required");
     const runId = input.runId ?? `run-${Date.now().toString(36)}`;
     const outputDir = issueRunOutputDir({
         config: input.config,

package/package.json

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

package/schema.json

@@ -316,7 +316,7 @@
       "type": "object",
       "additionalProperties": false,
       "properties": {
-        "agents": {
+        "reviewers": {
           "type": "array",
           "minItems": 3,
           "items": { "$ref": "#/$defs/reviewer" }
@@ -351,7 +351,7 @@
       "type": "object",
       "additionalProperties": false,
       "properties": {
-        "agents": {
+        "voters": {
           "type": "array",
           "minItems": 3,
           "items": { "$ref": "#/$defs/triageAgent" }