opencode-magi 0.0.0-dev-20260720080129 → 0.0.0-dev-20260720123050

package/dist/config/resolve.js

@@ -1,6 +1,7 @@
 import { readFile } from "node:fs/promises";
 import { join } from "node:path";
 import { CONFIG_PATH, DEFAULT_CONFIG } from "@/constant";
+import editPermissions from "@/permissions/editor.json" with { type: "json" };
 import { filterEmpty, getModels, isArray, isObject, isString, merge, } from "@/utils";
 export function resolvePermissions(permissions) {
     if (!permissions)
@@ -83,6 +84,8 @@ export async function getConfig(input) {
         readConfig(projectPath),
     ]);
     const results = filterEmpty(data);
+    if (!models.length)
+        throw new Error("No OpenCode models found.");
     if (!results.length)
         throw new Error(`No Magi config found. Expected ${CONFIG_PATH.GLOBAL} or ${projectPath}.`);
     const config = results.reduce((prev, current) => merge(prev, current), DEFAULT_CONFIG);
@@ -96,8 +99,14 @@ export async function getConfig(input) {
                 model: resolveModel(models, model),
             };
         });
-    config.merge.editor = mergeAgent(config, config.merge.editor);
-    config.merge.editor.model = resolveModel(models, config.merge.editor.model);
+    if (config.merge.editor) {
+        if (config.merge.editor.permissions)
+            config.merge.editor.permissions = mergePermissions(editPermissions, config.merge.editor.permissions);
+        else
+            config.merge.editor.permissions = editPermissions;
+        config.merge.editor = mergeAgent(config, config.merge.editor);
+        config.merge.editor.model = resolveModel(models, config.merge.editor.model);
+    }
     if (config.triage.voters)
         config.triage.voters = config.triage.voters.map((agent, index) => {
             const { id, model, ...rest } = mergeAgent(config, agent);
@@ -107,7 +116,13 @@ export async function getConfig(input) {
                 model: resolveModel(models, model),
             };
         });
-    config.triage.creator = mergeAgent(config, config.triage.creator);
-    config.triage.creator.model = resolveModel(models, config.triage.creator.model);
+    if (config.triage.creator) {
+        if (config.triage.creator.permissions)
+            config.triage.creator.permissions = mergePermissions(editPermissions, config.triage.creator.permissions);
+        else
+            config.triage.creator.permissions = editPermissions;
+        config.triage.creator = mergeAgent(config, config.triage.creator);
+        config.triage.creator.model = resolveModel(models, config.triage.creator.model);
+    }
     return config;
 }

package/dist/config/validate.js

@@ -12,28 +12,32 @@ function schemaErrors(config) {
     return (validateSchema.errors ?? []).map((e) => `schema ${e.instancePath || "config"}: ${e.message ?? "invalid value"}`);
 }
 function requiredErrors(config, { creator = false, editor = false, github = true, reviewers = false, voters = false, } = {}) {
+    creator ||= !!config.triage.creator;
+    editor ||= !!config.merge.editor;
+    reviewers ||= !!config.review.reviewers;
+    voters ||= !!config.triage.voters;
     const errors = [
         required(github, config.github.owner, "github.owner"),
         required(github, config.github.repo, "github.repo"),
         required(reviewers, config.review.reviewers, "review.reviewers"),
-        ...(config.review.reviewers ?? []).map((reviewer, index) => required(reviewers, reviewer.model, `review.reviewers[${index}].model`)),
+        ...(config.review.reviewers ?? []).map((reviewer, index) => required(reviewers || !!reviewer, reviewer.model, `review.reviewers[${index}].model`)),
         required(editor, config.merge.editor, "merge.editor"),
-        required(editor, config.merge.editor.model, "merge.editor.model"),
-        required(editor, config.merge.editor.author, "merge.editor.author"),
+        required(editor, config.merge.editor?.model, "merge.editor.model"),
+        required(editor, config.merge.editor?.author, "merge.editor.author"),
         required(voters, config.triage.voters, "triage.voters"),
-        ...(config.triage.voters ?? []).map((voter, index) => required(voters, voter.model, `triage.voters[${index}].model`)),
+        ...(config.triage.voters ?? []).map((voter, index) => required(voters || !!voter, voter.model, `triage.voters[${index}].model`)),
         required(creator, config.triage.creator, "triage.creator"),
-        required(creator, config.triage.creator.model, "triage.creator.model"),
-        required(creator, config.triage.creator.author, "triage.creator.author"),
+        required(creator, config.triage.creator?.model, "triage.creator.model"),
+        required(creator, config.triage.creator?.author, "triage.creator.author"),
     ];
     if (config.mode === "single") {
         errors.push(required(true, config.account, "account"));
     }
     else {
         errors.push(...(config.review.reviewers ?? []).map((reviewer, index) => required(reviewers, reviewer.account, `review.reviewers[${index}].account`)));
-        errors.push(required(editor, config.merge.editor.account, "merge.editor.account"));
+        errors.push(required(editor, config.merge.editor?.account, "merge.editor.account"));
         errors.push(...(config.triage.voters ?? []).map((voter, index) => required(voters, voter.account, `triage.voters[${index}].account`)));
-        errors.push(required(creator, config.triage.creator.account, "triage.creator.account"));
+        errors.push(required(creator, config.triage.creator?.account, "triage.creator.account"));
     }
     return filterEmpty(errors);
 }
@@ -85,17 +89,19 @@ async function authErrors(config, exec) {
     }
     else {
         const accounts = {
-            ...Object.fromEntries(config.review.reviewers.map(({ account }, index) => [
+            ...Object.fromEntries(config.review.reviewers?.map(({ account }, index) => [
                 account,
                 `review.reviewers[${index}]`,
-            ])),
-            [config.merge.editor.account]: "merge.editor",
-            ...Object.fromEntries(config.triage.voters.map(({ account }, index) => [
+            ]) ?? []),
+            ...Object.fromEntries(config.triage.voters?.map(({ account }, index) => [
                 account,
                 `triage.voters[${index}]`,
-            ])),
-            [config.triage.creator.account]: "triage.creator",
+            ]) ?? []),
         };
+        if (config.merge.editor)
+            accounts[config.merge.editor.account] = "merge.editor";
+        if (config.triage.creator)
+            accounts[config.triage.creator.account] = "triage.creator";
         return filterEmpty([
             ...duplicateErrors(Object.keys(accounts), (value) => `${accounts[value]} has duplicate account: ${value}`),
             ...(await Promise.all(Object.keys(accounts).map((account) => authError(config, exec, account)))),

package/dist/constant.js

@@ -1,7 +1,6 @@
 import { homedir } from "node:os";
 import { join } from "node:path";
 import commonPermissions from "./permissions/common.json" with { type: "json" };
-import editPermissions from "./permissions/editor.json" with { type: "json" };
 export const CONFIG_PATH = {
     GLOBAL: join(homedir(), ".config", "opencode", "magi.json"),
     PROJECT: join(".opencode", "magi.json"),
@@ -33,9 +32,6 @@ export const DEFAULT_CONFIG = {
         checks: {
             wait: true,
         },
-        editor: {
-            permissions: editPermissions,
-        },
         maxThreadResolutionCycles: 5,
     },
     mode: "single",
@@ -126,9 +122,6 @@ export const DEFAULT_CONFIG = {
         concurrency: {
             runs: 3,
         },
-        creator: {
-            permissions: editPermissions,
-        },
         output: ".magi/runs/issue",
         safety: {
             allowAuthors: [],

package/dist/tools/merge/editor.js

@@ -193,7 +193,7 @@ async function push() {
             GIT_CONFIG_KEY_0: "credential.helper",
             GIT_CONFIG_KEY_1: "credential.helper",
             GIT_CONFIG_VALUE_0: "",
-            GIT_CONFIG_VALUE_1: "!f() { echo username=x-access-token; echo password=$GIT_PASSWORD; }; f",
+            GIT_CONFIG_VALUE_1: "!f() { echo username=git; echo password=$GIT_PASSWORD; }; f",
             GIT_PASSWORD: token,
             GIT_TERMINAL_PROMPT: "0",
         },

package/package.json

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