opencode-magi 0.0.0-dev-20260702001022 → 0.0.0-dev-20260714064227

package/dist/tools/review/review.js

@@ -27,11 +27,6 @@ export class Review {
         this.state = state;
     }
     static async init(number, magi, config, context, options) {
-        if (!options.sync) {
-            const controller = new AbortController();
-            context = { ...context, abort: controller.signal };
-            magi.registerBackground(number, controller);
-        }
         const url = `${config.github.url}/pull/${number}`;
         const octokit = await magi.createOctokit(config, context.abort);
         const graphql = magi.createGraphql(octokit);
@@ -47,8 +42,8 @@ export class Review {
             repo: quote(`${config.github.owner}/${config.github.repo}`),
             reviewers,
             sessionId: context.sessionID,
-            text: `Started reviewing [#${number}](${url}).`,
         });
+        await magi.updateEvent(state.output, `Started reviewing.`);
         const exec = createExecWithGitHubApiRetry(magi.exec, config.github.retryApiAttempts);
         return new Review(number, magi, config, context, octokit, graphql, exec, state);
     }
@@ -64,17 +59,18 @@ export class Review {
     postReviews = postReviews;
     automate = automate;
     createReport = createReport;
-    async notify(text) {
-        if (!this.state.sync)
-            await this.magi.notify(this.state.sessionId, text);
-    }
     async cleanup() {
         if (this.state.worktree?.path)
             await this.magi.deleteWorktree(this.state.worktree.path);
-        this.magi.unregisterBackground(this.number, this.context.abort);
     }
-    getLink() {
-        return `[#${this.state.pr.number}](${this.state.pr.url})`;
+    async updateState(next) {
+        this.state = await this.magi.updateState(this.state.output, next);
+    }
+    async updateEvent(message) {
+        await this.magi.updateEvent(this.state.output, message);
+    }
+    async getEvents() {
+        return await this.magi.getEvents(this.state.output);
     }
     async createAgentFile(phase, id, content, attempt, cycle) {
         await this.magi.createAgentFile(this.state.output, phase, id, content, attempt, cycle);
@@ -97,21 +93,14 @@ export class Review {
                 permissions: this.state.operator.permissions,
             }),
         };
-        this.state = await this.magi.updateState(this.state.output, {
-            operator,
-            reviewers,
-        });
+        await this.updateState({ operator, reviewers });
     }
     async createWorktree() {
         this.context.abort.throwIfAborted();
-        this.state = await this.magi.updateState(this.state.output, {
-            text: `Creating worktree for ${this.getLink()}.`,
-        });
+        await this.updateEvent(`Creating worktree.`);
         const worktree = await this.magi.createWorktree(this.config.review.worktree, this.number, this.state.id, this.context.abort);
-        this.state = await this.magi.updateState(this.state.output, {
-            text: `Finished creating worktree for ${this.getLink()}.`,
-            worktree,
-        });
+        await this.updateState({ worktree });
+        await this.updateEvent(`Finished creating worktree.`);
     }
     async resolveVerdict() {
         if (!this.config.review.reviewers?.length)
@@ -138,10 +127,8 @@ export class Review {
                 (majority && counts.APPROVED >= threshold)
                 ? "APPROVED"
                 : "CHANGES_REQUESTED";
-        this.state = await this.magi.updateState(this.state.output, {
-            pr: { verdict },
-            text: `Final verdict for ${this.getLink()} is ${verdict}.`,
-        });
+        await this.updateState({ pr: { verdict } });
+        await this.updateEvent(`Final verdict is ${verdict}.`);
         return verdict;
     }
 }

package/dist/tools/review/reviewer.js

@@ -3,9 +3,7 @@ import { Prompt } from "@/prompts";
 import { filterEmpty, marker, omitNullish, retry, Worker } from "@/utils";
 export async function review() {
     this.context.abort.throwIfAborted();
-    this.state = await this.magi.updateState(this.state.output, {
-        text: `Reviewing ${this.getLink()}.`,
-    });
+    await this.updateEvent(`Reviewing.`);
     if (!this.config.review.reviewers?.length)
         throw new MagiError("blocked", "No reviewers configured.");
     const worker = new Worker(this.config.review.concurrency.reviewers);
@@ -22,7 +20,7 @@ export async function review() {
             throw new MagiError("blocked", "Reviewers not found.");
         const { outputs, review, sessionId, status } = this.state.reviewers[id];
         if (status === "skip") {
-            await this.notify(`Skipping review for ${this.getLink()} with reviewer ${id}.`);
+            await this.updateEvent(`Skipping review with reviewer ${id}.`);
             return [id, {}];
         }
         else {
@@ -33,7 +31,7 @@ export async function review() {
             const rereview = status !== "initial";
             const label = rereview ? "rereview" : "review";
             const cycle = (outputs?.length ?? 0) + 1;
-            await this.notify(`Running ${label} for ${this.getLink()} with reviewer ${id}.`);
+            await this.updateEvent(`Running ${label} with reviewer ${id}.`);
             const sha = rereview
                 ? (review?.commit_id ?? this.state.pr.metadata.base.sha)
                 : this.state.pr.metadata.base.sha;
@@ -106,7 +104,7 @@ export async function review() {
                 validateInlineCommentTargets(status, parsed, inlineCommentTargets);
                 return parsed;
             }, {
-                error: (_, count) => this.notify(`Attempt ${count} failed to ${label} for ${this.getLink()} with reviewer ${id}. Retrying...`),
+                error: (_, count) => this.updateEvent(`Attempt ${count} failed to ${label} with reviewer ${id}. Retrying...`),
                 retries: this.config.output.repairAttempts,
             });
             if (!output)
@@ -118,10 +116,8 @@ export async function review() {
             return [id, { outputs: [...(outputs ?? []), output] }];
         }
     }))));
-    this.state = await this.magi.updateState(this.state.output, {
-        reviewers,
-        text: `Finished reviewing ${this.getLink()}.`,
-    });
+    await this.updateState({ reviewers });
+    await this.updateEvent(`Finished reviewing.`);
 }
 export async function validateFindings() {
     this.context.abort.throwIfAborted();
@@ -137,19 +133,15 @@ export async function validateFindings() {
     });
     if (!findings.length)
         return;
-    this.state = await this.magi.updateState(this.state.output, {
-        text: `Validating review findings for ${this.getLink()}.`,
-    });
+    await this.updateEvent(`Validating review findings.`);
     const accepted = await collectAcceptedFindings.call(this, findings);
     const reviewers = Object.fromEntries(Object.entries(this.state.reviewers ?? {}).map(([id, reviewer]) => [
         id,
         transformState(reviewer, id, accepted),
     ]));
     await notifyVerdictChanges.call(this, this.state.reviewers ?? {}, reviewers, "after majority finding validation");
-    this.state = await this.magi.updateState(this.state.output, {
-        reviewers,
-        text: `Finished validating review findings for ${this.getLink()}.`,
-    });
+    await this.updateState({ reviewers });
+    await this.updateEvent(`Finished validating review findings.`);
 }
 export async function reconsiderClose() {
     this.context.abort.throwIfAborted();
@@ -164,9 +156,7 @@ export async function reconsiderClose() {
     const count = targetReviewers.length;
     if (!count || count >= threshold)
         return;
-    this.state = await this.magi.updateState(this.state.output, {
-        text: `Reconsidering close verdicts for ${this.getLink()}.`,
-    });
+    await this.updateEvent(`Reconsidering close verdicts.`);
     const worker = new Worker(this.config.review.concurrency.reviewers);
     const prompt = await Prompt.init(this.magi, this.config, "review/close-reconsideration");
     const reviewers = Object.fromEntries(await Promise.all(targetReviewers.map(([id, { outputs, review, sessionId, status }]) => worker.run(async () => {
@@ -187,7 +177,7 @@ export async function reconsiderClose() {
             repo: this.config.github.repo,
         });
         const repairMessage = await prompt.repair();
-        await this.notify(`Reconsidering close verdict for ${this.getLink()} with reviewer ${id}.`);
+        await this.updateEvent(`Reconsidering close verdict with reviewer ${id}.`);
         const output = await retry(async (count) => {
             const raw = await this.magi.promptSession(sessionId, count === 1 ? taskMessage : repairMessage);
             await this.createAgentFile("close-reconsideration", id, raw, count, cycle);
@@ -197,7 +187,7 @@ export async function reconsiderClose() {
             validateInlineCommentTargets("initial", parsed, inlineCommentTargets);
             return parsed;
         }, {
-            error: (_, count) => this.notify(`Attempt ${count} failed to reconsider close verdict for ${this.getLink()} with reviewer ${id}. Retrying...`),
+            error: (_, count) => this.updateEvent(`Attempt ${count} failed to reconsider close verdict with reviewer ${id}. Retrying...`),
             retries: this.config.output.repairAttempts,
         });
         if (!output)
@@ -225,10 +215,8 @@ export async function reconsiderClose() {
         transformState(reviewer, id, accepted),
     ]));
     await notifyVerdictChanges.call(this, reviewers, validatedReviewers, "after majority finding validation");
-    this.state = await this.magi.updateState(this.state.output, {
-        reviewers: validatedReviewers,
-        text: `Finished reconsidering close verdicts for ${this.getLink()}.`,
-    });
+    await this.updateState({ reviewers: validatedReviewers });
+    await this.updateEvent(`Finished reconsidering close verdicts.`);
 }
 async function collectAcceptedFindings(findings) {
     const accepted = new Set();
@@ -244,7 +232,7 @@ async function collectAcceptedFindings(findings) {
         const { sessionId } = this.state.reviewers[id];
         if (!sessionId)
             throw new MagiError("blocked", `No session ID found for reviewer ${id}.`);
-        await this.notify(`Validating review findings for ${this.getLink()} with reviewer ${id}.`);
+        await this.updateEvent(`Validating review findings with reviewer ${id}.`);
         const targetFindings = findings.filter((target) => target.reviewer !== id);
         const expectedKeys = new Set(targetFindings.map(({ index, reviewer }) => `${reviewer}:${index}`));
         const taskMessage = await prompt.create(this.config.review.prompts?.findingValidation, [
@@ -278,7 +266,7 @@ async function collectAcceptedFindings(findings) {
                     throw new Error(`Missing finding vote: ${key}.`);
             return parsed;
         }, {
-            error: (_, count) => this.notify(`Attempt ${count} failed to validate review findings for ${this.getLink()} with reviewer ${id}. Retrying...`),
+            error: (_, count) => this.updateEvent(`Attempt ${count} failed to validate review findings with reviewer ${id}. Retrying...`),
             retries: this.config.output.repairAttempts,
         });
         if (!output)
@@ -303,8 +291,8 @@ async function collectAcceptedFindings(findings) {
             .map(({ validator, vote }) => `- ${validator}: ${vote.comment}`);
         if (agrees >= threshold)
             accepted.add(key);
-        await this.notify(filterEmpty([
-            `Finding ${reviewer} #${index + 1} for ${this.getLink()} was ${agrees >= threshold ? "accepted" : "rejected"} by majority vote.`,
+        await this.updateEvent(filterEmpty([
+            `Finding ${reviewer} #${index + 1} was ${agrees >= threshold ? "accepted" : "rejected"} by majority vote.`,
             `Finding: ${finding.path}:${finding.line}\n${finding.body}`,
             acceptedComments.length
                 ? `Accepted by:\n${acceptedComments.join("\n")}`
@@ -343,7 +331,7 @@ async function notifyVerdictChanges(prev, next, reason) {
         const nextVerdict = reviewer.outputs?.at(-1)?.verdict;
         if (!prevVerdict || !nextVerdict || prevVerdict === nextVerdict)
             return;
-        await this.notify(`Reviewer ${id} verdict changed from ${prevVerdict} to ${nextVerdict} for ${this.getLink()} ${reason}.`);
+        await this.updateEvent(`Reviewer ${id} verdict changed from ${prevVerdict} to ${nextVerdict} ${reason}.`);
     }));
 }
 function validateInlineCommentTargets(status, output, inlineCommentTargets) {

package/dist/tools/triage/index.js

@@ -5,10 +5,9 @@ export const triage = function (magi) {
             args: {
                 dryRun: tool.schema.boolean().optional(),
                 issues: tool.schema.string(),
-                sync: tool.schema.boolean().optional(),
             },
             description: "Triage one or more issues with configured Magi triage voters.",
-            async execute({ dryRun: _dryRun = false, issues: _issues, sync: _sync = false, }) {
+            async execute({ dryRun: _dryRun = false, issues: _issues }) {
                 const _config = await magi.getConfig({ creator: true, voters: true });
                 return "";
             },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-magi",
-  "version": "0.0.0-dev-20260702001022",
+  "version": "0.0.0-dev-20260714064227",
   "description": "Multi-agent PR review and merge orchestration plugin for OpenCode.",
   "license": "MIT",
   "author": "Hirotomo Yamada <hirotomo.yamada@avap.co.jp>",
@@ -64,18 +64,18 @@
     "vitest": "^4.1.7"
   },
   "scripts": {
+    "clean": "rimraf node_modules dist coverage",
     "postbuild": "node scripts/copy-data.ts",
     "build": "tsgo -p tsconfig.build.json",
-    "clean": "rimraf node_modules dist coverage",
+    "graphql:codegen": "graphql-codegen --config codegen.json",
     "format:check": "oxfmt --check .",
     "format:write": "oxfmt --write .",
-    "graphql:codegen": "graphql-codegen --config codegen.json",
     "lint:check": "oxlint . --max-warnings=0",
     "lint:fix": "oxlint . --max-warnings=0 --fix",
-    "quality": "pnpm format:check && pnpm lint:check && pnpm typecheck && pnpm test",
+    "type:check": "tsgo --noEmit",
     "test": "vitest run --passWithNoTests",
     "test:dev": "vitest --watch --ui --passWithNoTests",
-    "typecheck": "tsgo --noEmit",
+    "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/dist/tools/cancel/index.js

@@ -1,30 +0,0 @@
-import { tool } from "@opencode-ai/plugin";
-export const cancel = function (magi) {
-    return {
-        magi_cancel: tool({
-            args: {
-                numbers: tool.schema.number().array().optional(),
-            },
-            description: "Cancel active background Magi runs by pull request or issue number, or all active background runs when no target is provided.",
-            async execute({ numbers }) {
-                const result = magi.cancelBackgrounds(numbers);
-                if (!result.cancelled.length && !result.missing.length)
-                    return await Promise.resolve("No active background runs.");
-                return await Promise.resolve([
-                    ...(result.cancelled.length
-                        ? [
-                            `Cancelled: ${result.cancelled.length}`,
-                            ...result.cancelled.map((number) => `- ${number}`),
-                        ]
-                        : []),
-                    ...(result.missing.length
-                        ? [
-                            `Missing: ${result.missing.length}`,
-                            ...result.missing.map((number) => `- ${number}`),
-                        ]
-                        : []),
-                ].join("\n"));
-            },
-        }),
-    };
-};