opencode-magi 0.10.0 → 0.12.0

package/dist/utils/object.js

@@ -0,0 +1,21 @@
+import { isObject } from "./assertion";
+export function merge(base, override) {
+    const merged = { ...base };
+    for (const [key, value] of Object.entries(override)) {
+        const existing = merged[key];
+        merged[key] =
+            isObject(existing) && isObject(value) ? merge(existing, value) : value;
+    }
+    return merged;
+}
+export function filterObject(obj, func) {
+    const result = {};
+    Object.entries(obj).forEach(([key, value]) => {
+        if (func(key, value, obj))
+            result[key] = value;
+    });
+    return result;
+}
+export function omitNullish(obj) {
+    return filterObject(obj, (_, val) => val != null);
+}

package/dist/utils/opencode.js

@@ -0,0 +1,25 @@
+import { isArray } from "./assertion";
+import { createExec } from "./exec";
+export async function getModels(input) {
+    const providers = await input.client.config
+        .providers({ directory: input.directory })
+        .catch(() => input.client.provider.list({ directory: input.directory }));
+    const data = "data" in providers ? providers.data : undefined;
+    const all = data && "providers" in data ? data.providers : data?.all;
+    const models = isArray(all)
+        ? all.flatMap((provider) => {
+            const models = Object.keys(provider.models);
+            return provider.id ? models.map((id) => `${provider.id}/${id}`) : models;
+        })
+        : [];
+    if (models.length)
+        return models;
+    else
+        try {
+            const output = await createExec(input.directory)("opencode models");
+            return output.split("\n").filter((model) => model.includes("/"));
+        }
+        catch {
+            return [];
+        }
+}

package/dist/utils/string.js

@@ -0,0 +1,38 @@
+import { filterEmpty } from "./array";
+export function quote(value) {
+    return `'${value.replaceAll("'", "'\\''")}'`;
+}
+export function split(value) {
+    return filterEmpty(value.split(/[\s,]+/));
+}
+export function toTitleCase(value) {
+    return value
+        .replace(/([A-Z])/g, " $1")
+        .replace(/[_-](.)/g, (_, val) => ` ${val.toUpperCase()}`)
+        .replace(/^./, (str) => str.toUpperCase())
+        .trim();
+}
+export function command(...values) {
+    return filterEmpty(values).join(" ");
+}
+export const marker = {
+    parse(body) {
+        const matchAll = body.matchAll(/<!--\s*opencode-magi\s+([^>]*)-->/g);
+        return [...matchAll].map((match) => Object.fromEntries(match[1]
+            ?.trim()
+            .split(/\s+/)
+            .flatMap((part) => {
+            const [key, ...rest] = part.split("=");
+            if (!key || !rest.length)
+                return [];
+            return [[key, rest.join("=")]];
+        }) ?? []));
+    },
+    stringify(...values) {
+        return values
+            .map((value) => `<!-- opencode-magi ${Object.entries(value)
+            .map(([key, value]) => `${key}=${value}`)
+            .join(" ")} -->`)
+            .join("\n");
+    },
+};

package/dist/utils/worker.js

@@ -0,0 +1,30 @@
+export class Worker {
+    active = 0;
+    tasks = [];
+    limit;
+    constructor(limit) {
+        this.limit = Math.max(1, Math.floor(limit));
+    }
+    async run(run) {
+        const task = new Promise((resolve, reject) => {
+            this.tasks.push({ reject, resolve, run });
+        });
+        this.drain();
+        return task;
+    }
+    drain() {
+        while (this.active < this.limit) {
+            const task = this.tasks.shift();
+            if (!task)
+                return;
+            this.active += 1;
+            void task
+                .run()
+                .then(task.resolve, task.reject)
+                .finally(() => {
+                this.active -= 1;
+                this.drain();
+            });
+        }
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-magi",
-  "version": "0.10.0",
+  "version": "0.12.0",
   "description": "Multi-agent PR review and merge orchestration plugin for OpenCode.",
   "license": "MIT",
   "author": "Hirotomo Yamada <hirotomo.yamada@avap.co.jp>",
@@ -28,42 +28,55 @@
     "schema.json"
   ],
   "dependencies": {
-    "@opencode-ai/plugin": "^1.15.10",
+    "@opencode-ai/plugin": "^1.18.4",
+    "@opencode-ai/sdk": "^1.18.4",
     "ajv": "^8.20.0",
-    "picomatch": "^4.0.4",
-    "valibot": "^1.4.1"
+    "graphql": "^17.0.2",
+    "graphql-tag": "^2.12.7",
+    "octokit": "^5.0.5",
+    "picomatch": "^4.0.5",
+    "valibot": "^1.4.2"
   },
   "devDependencies": {
     "@changesets/changelog-github": "^0.7.0",
-    "@changesets/cli": "^2.30.0",
-    "@commitlint/cli": "^21.0.1",
-    "@commitlint/config-conventional": "^21.0.1",
-    "@types/node": "^25.9.1",
+    "@changesets/cli": "^2.31.1",
+    "@commitlint/cli": "^21.2.1",
+    "@commitlint/config-conventional": "^21.2.0",
+    "@graphql-codegen/cli": "^7.2.0",
+    "@graphql-codegen/typescript-generic-sdk": "^5.0.1",
+    "@graphql-codegen/typescript-operations": "^6.1.1",
+    "@limegrass/eslint-plugin-import-alias": "^1.6.1",
+    "@octokit/graphql-schema": "^15.26.1",
+    "@octokit/plugin-throttling": "^11.0.3",
+    "@stylistic/eslint-plugin": "^5.10.0",
+    "@types/node": "^26.1.1",
     "@types/picomatch": "^4.0.3",
-    "@typescript/native-preview": "7.0.0-dev.20260525.1",
-    "@vitest/coverage-v8": "^4.1.7",
-    "@vitest/ui": "^4.1.7",
-    "eslint-plugin-perfectionist": "^5.9.0",
+    "@typescript/native-preview": "7.0.0-dev.20260707.2",
+    "@vitest/coverage-v8": "^4.1.10",
+    "@vitest/ui": "^4.1.10",
+    "eslint-plugin-perfectionist": "^5.10.0",
     "eslint-plugin-unused-imports": "^4.4.1",
-    "lefthook": "^2.1.8",
-    "oxfmt": "^0.51.0",
-    "oxlint": "^1.66.0",
-    "oxlint-tsgolint": "^0.23.0",
+    "lefthook": "^2.1.10",
+    "oxfmt": "^0.60.0",
+    "oxlint": "^1.75.0",
+    "oxlint-tsgolint": "^7.0.2001",
     "rimraf": "^6.1.3",
-    "vitest": "^4.1.7"
+    "vitest": "^4.1.10"
   },
   "scripts": {
-    "prebuild": "node scripts/copy-data.ts",
-    "build": "tsgo -p tsconfig.build.json",
     "clean": "rimraf node_modules dist coverage",
+    "postbuild": "node scripts/copy-data.ts",
+    "build": "tsgo -p tsconfig.build.json",
+    "graphql:codegen": "graphql-codegen --config codegen.json",
     "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",
+    "type:check": "tsgo --noEmit",
+    "test": "vitest run --passWithNoTests",
+    "test:coverage": "vitest run --coverage --coverage.reporter=html && open coverage/index.html",
+    "test:dev": "vitest --watch --ui --passWithNoTests",
+    "quality": "pnpm format:check && pnpm lint:check && pnpm type:check && pnpm test",
     "release": "changeset publish",
     "release:dev": "changeset version --snapshot dev && changeset publish --tag dev"
   }

package/schema.json

@@ -33,14 +33,15 @@
       "additionalProperties": false,
       "properties": {
         "host": { "type": "string", "default": "github.com" },
-        "apiRetryAttempts": {
+        "retryApiAttempts": {
           "type": "integer",
           "minimum": 0,
           "default": 3,
           "description": "Number of retry attempts for GitHub CLI API calls that fail with rate limit errors."
         },
         "owner": { "type": "string", "minLength": 1 },
-        "repo": { "type": "string", "minLength": 1 }
+        "repo": { "type": "string", "minLength": 1 },
+        "url": { "type": "string" }
       }
     },
     "language": { "type": "string" },
@@ -170,24 +171,128 @@
       "additionalProperties": false,
       "properties": {
         "id": { "type": "string", "minLength": 1 },
-        "options": { "type": "object", "additionalProperties": true }
+        "variant": { "type": "string" }
+      }
+    },
+    "safetyFilter": {
+      "type": "object",
+      "additionalProperties": false,
+      "minProperties": 1,
+      "properties": {
+        "exclude": { "type": "array", "items": { "type": "string" } },
+        "include": { "type": "array", "items": { "type": "string" } }
+      }
+    },
+    "safetyBranchFilter": {
+      "type": "object",
+      "additionalProperties": false,
+      "minProperties": 1,
+      "properties": {
+        "base": { "$ref": "#/$defs/safetyFilter" },
+        "head": { "$ref": "#/$defs/safetyFilter" }
+      }
+    },
+    "safety": {
+      "type": "object",
+      "additionalProperties": false,
+      "properties": {
+        "authors": { "$ref": "#/$defs/safetyFilter" },
+        "branches": { "$ref": "#/$defs/safetyBranchFilter" },
+        "labels": { "$ref": "#/$defs/safetyFilter" },
+        "maxChangedFiles": { "type": "integer", "minimum": 0 },
+        "paths": { "$ref": "#/$defs/safetyFilter" }
+      }
+    },
+    "conditions": {
+      "oneOf": [
+        { "type": "boolean" },
+        {
+          "type": "array",
+          "items": {
+            "type": "array",
+            "minItems": 2,
+            "maxItems": 2,
+            "prefixItems": [{ "type": "boolean" }, { "$ref": "#/$defs/safety" }]
+          }
+        }
+      ]
+    },
+    "automationSafety": {
+      "type": "object",
+      "additionalProperties": false,
+      "properties": {
+        "authors": { "$ref": "#/$defs/safetyFilter" },
+        "branches": { "$ref": "#/$defs/safetyBranchFilter" },
+        "labels": { "$ref": "#/$defs/safetyFilter" },
+        "paths": { "$ref": "#/$defs/safetyFilter" }
+      }
+    },
+    "automationConditions": {
+      "oneOf": [
+        { "type": "boolean" },
+        {
+          "type": "array",
+          "items": {
+            "type": "array",
+            "minItems": 2,
+            "maxItems": 2,
+            "prefixItems": [
+              { "type": "boolean" },
+              { "$ref": "#/$defs/automationSafety" }
+            ]
+          }
+        }
+      ]
+    },
+    "mergeAutomationConditions": {
+      "oneOf": [
+        { "type": "boolean" },
+        {
+          "type": "array",
+          "items": {
+            "type": "array",
+            "minItems": 2,
+            "maxItems": 2,
+            "prefixItems": [
+              { "type": "boolean" },
+              { "$ref": "#/$defs/mergeAutomationSafety" }
+            ]
+          }
+        }
+      ]
+    },
+    "mergeAutomationSafety": {
+      "type": "object",
+      "additionalProperties": false,
+      "properties": {
+        "authors": { "$ref": "#/$defs/safetyFilter" },
+        "branches": { "$ref": "#/$defs/safetyBranchFilter" },
+        "edited": { "type": "boolean" },
+        "labels": { "$ref": "#/$defs/safetyFilter" },
+        "paths": { "$ref": "#/$defs/safetyFilter" }
       }
     },
     "automation": {
       "type": "object",
       "additionalProperties": false,
       "properties": {
-        "merge": { "type": "boolean", "default": true },
-        "close": { "type": "boolean" }
+        "merge": { "$ref": "#/$defs/automationConditions", "default": true },
+        "close": { "$ref": "#/$defs/automationConditions", "default": false }
       }
     },
     "mergeAutomation": {
       "type": "object",
       "additionalProperties": false,
       "properties": {
-        "merge": { "type": "boolean", "default": true },
-        "close": { "type": "boolean", "default": false },
-        "conflict": { "type": "boolean", "default": false }
+        "merge": {
+          "$ref": "#/$defs/mergeAutomationConditions",
+          "default": true
+        },
+        "close": {
+          "$ref": "#/$defs/mergeAutomationConditions",
+          "default": false
+        },
+        "conflict": { "$ref": "#/$defs/automationConditions", "default": false }
       }
     },
     "reviewChecks": {
@@ -221,16 +326,6 @@
         "runs": { "type": "integer", "minimum": 1, "default": 3 }
       }
     },
-    "safety": {
-      "type": "object",
-      "additionalProperties": false,
-      "properties": {
-        "allowAuthors": { "type": "array", "items": { "type": "string" } },
-        "blockedPaths": { "type": "array", "items": { "type": "string" } },
-        "maxChangedFiles": { "type": "integer", "minimum": 0 },
-        "requiredLabels": { "type": "array", "items": { "type": "string" } }
-      }
-    },
     "reviewPrompts": {
       "type": "object",
       "additionalProperties": false,
@@ -375,10 +470,11 @@
         },
         "prompts": { "$ref": "#/$defs/reviewPrompts" },
         "checks": { "$ref": "#/$defs/reviewChecks" },
-        "safety": { "$ref": "#/$defs/safety" },
+        "safety": { "$ref": "#/$defs/conditions" },
         "automation": { "$ref": "#/$defs/automation" },
         "concurrency": { "$ref": "#/$defs/concurrency" },
         "merge": { "$ref": "#/$defs/reviewMerge" },
+        "operator": { "type": "string", "minLength": 1 },
         "output": { "type": "string" },
         "worktree": { "type": "string" }
       }
@@ -421,7 +517,7 @@
         "safety": { "$ref": "#/$defs/triageSafety" },
         "concurrency": { "$ref": "#/$defs/triageConcurrency" },
         "prompts": { "$ref": "#/$defs/triagePrompts" },
-        "reporter": { "type": "string", "minLength": 1 },
+        "operator": { "type": "string", "minLength": 1 },
         "output": { "type": "string" },
         "worktree": { "type": "string" }
       }

package/dist/config/load.js

@@ -1,62 +0,0 @@
-import { readFile } from "node:fs/promises";
-import { homedir } from "node:os";
-import { isAbsolute, join } from "node:path";
-const GLOBAL_CONFIG = join(homedir(), ".config", "opencode", "magi.json");
-const PROJECT_CONFIG = join(".opencode", "magi.json");
-function isPlainObject(value) {
-    return (!!value &&
-        typeof value === "object" &&
-        !Array.isArray(value) &&
-        Object.getPrototypeOf(value) === Object.prototype);
-}
-export function mergeMagiConfig(base, override) {
-    const merged = { ...base };
-    for (const [key, value] of Object.entries(override)) {
-        const existing = merged[key];
-        merged[key] =
-            isPlainObject(existing) && isPlainObject(value)
-                ? mergeMagiConfig(existing, value)
-                : value;
-    }
-    return merged;
-}
-async function readConfig(path) {
-    try {
-        return JSON.parse(await readFile(path, "utf8"));
-    }
-    catch (error) {
-        const code = error.code;
-        if (code === "ENOENT")
-            return null;
-        throw error;
-    }
-}
-export async function loadConfig(directory, configPath) {
-    if (configPath) {
-        const path = isAbsolute(configPath)
-            ? configPath
-            : join(directory, configPath);
-        const config = await readConfig(path);
-        if (!config)
-            throw new Error(`Magi config not found: ${path}`);
-        return { config: config, path };
-    }
-    const projectPath = join(directory, PROJECT_CONFIG);
-    const configs = await Promise.all([
-        readConfig(GLOBAL_CONFIG),
-        readConfig(projectPath),
-    ]);
-    const loaded = configs
-        .map((config, index) => ({
-        config,
-        path: index === 0 ? GLOBAL_CONFIG : projectPath,
-    }))
-        .filter((item) => Boolean(item.config));
-    if (!loaded.length)
-        throw new Error(`Magi config not found. Tried: ${GLOBAL_CONFIG}, ${projectPath}`);
-    const config = loaded.reduce((merged, item) => mergeMagiConfig(merged, item.config), {});
-    return {
-        config: config,
-        path: loaded.map((item) => item.path).join(", "),
-    };
-}

package/dist/config/output.js

@@ -1,25 +0,0 @@
-import { isAbsolute, join } from "node:path";
-const DEFAULT_OUTPUT_DIRS = {
-    issue: ".magi/runs/issue",
-    pr: ".magi/runs/pr",
-};
-function resolvePath(directory, path) {
-    return isAbsolute(path) ? path : join(directory, path);
-}
-export function outputBaseDir(directory, config, kind) {
-    return resolvePath(directory, kind === "issue"
-        ? (config.triage?.output ?? DEFAULT_OUTPUT_DIRS[kind])
-        : (config.review?.output ?? DEFAULT_OUTPUT_DIRS[kind]));
-}
-export function outputBaseDirs(directory, config) {
-    return [
-        outputBaseDir(directory, config, "pr"),
-        outputBaseDir(directory, config, "issue"),
-    ];
-}
-export function prRunOutputDir(input) {
-    return join(outputBaseDir(input.directory, input.config, "pr"), String(input.pr), ...(input.runId ? [input.runId] : []));
-}
-export function issueRunOutputDir(input) {
-    return join(outputBaseDir(input.directory, input.config, "issue"), String(input.issue), ...(input.runId ? [input.runId] : []));
-}

package/dist/config/worktree.js

@@ -1,25 +0,0 @@
-import { isAbsolute, join } from "node:path";
-const DEFAULT_WORKTREE_DIRS = {
-    issue: ".magi/worktrees/issue",
-    pr: ".magi/worktrees/pr",
-};
-function resolvePath(directory, path) {
-    return isAbsolute(path) ? path : join(directory, path);
-}
-export function worktreeBaseDir(directory, config, kind) {
-    return resolvePath(directory, kind === "issue"
-        ? (config.triage?.worktree ?? DEFAULT_WORKTREE_DIRS[kind])
-        : (config.review?.worktree ?? DEFAULT_WORKTREE_DIRS[kind]));
-}
-export function worktreeBaseDirs(directory, config = {}) {
-    return [
-        worktreeBaseDir(directory, config, "pr"),
-        worktreeBaseDir(directory, config, "issue"),
-    ];
-}
-export function prRunWorktreeDir(input) {
-    return join(worktreeBaseDir(input.directory, input.config, "pr"), String(input.pr), input.runId);
-}
-export function issueRunWorktreeDir(input) {
-    return join(worktreeBaseDir(input.directory, input.config, "issue"), String(input.issue), input.runId);
-}