opencode-magi 0.0.0-dev-20260525141349 → 0.0.0-dev-20260525154011

package/dist/orchestrator/merge.js

@@ -218,7 +218,6 @@ async function runRereview(input, worktreePath, previousHeadSha, cycle, sessionI
     const singleModeThreads = singleReviewMode
         ? assignThreadsByReviewFindingMarker({
             fallbackReviewerKeys: reviewerKeys,
-            headSha,
             pr: input.pr,
             reviewerKeys,
             threads: options.dryRunThreads == null

package/dist/orchestrator/review.js

@@ -531,13 +531,29 @@ function outputFindings(reviewer, output) {
     }));
 }
 function singleReviewBody(input) {
+    const outputs = Object.entries(input.outputs).sort(([a], [b]) => a.localeCompare(b));
+    const closeReasons = outputs.flatMap(([reviewer, output]) => output.verdict === "CLOSE"
+        ? [`- ${reviewer}: ${output.reason ?? "Close requested."}`]
+        : []);
+    const acceptedFindings = outputs.flatMap(([reviewer, output]) => outputFindings(reviewer, output).map(({ finding, index }) => {
+        const line = finding.startLine == null || finding.startLine === finding.line
+            ? String(finding.line)
+            : `${finding.startLine}-${finding.line}`;
+        return `- ${reviewer} #${index + 1} ${finding.path}:${line}: ${finding.issue} Fix: ${finding.fix}`;
+    }));
     const lines = [
         `Magi single-account review result: ${input.verdict}.`,
         "",
         "Logical reviewer verdicts:",
-        ...Object.entries(input.outputs).map(([reviewer, output]) => `- ${reviewer}: ${output.verdict}`),
+        ...outputs.map(([reviewer, output]) => `- ${reviewer}: ${output.verdict}`),
+        ...(input.verdict === "CLOSE" && closeReasons.length
+            ? ["", "Close reasons:", ...closeReasons]
+            : []),
+        ...(input.verdict === "CHANGES_REQUESTED" && acceptedFindings.length
+            ? ["", "Accepted change requests:", ...acceptedFindings]
+            : []),
         "",
-        ...Object.entries(input.outputs).map(([reviewer, output]) => formatReviewMarker({
+        ...outputs.map(([reviewer, output]) => formatReviewMarker({
             head: input.headSha,
             pr: input.pr,
             reviewer,
@@ -925,19 +941,21 @@ export async function runReview(input) {
     const skippedReviewers = input.repository.agents.reviewers.filter((reviewer) => {
         return (preliminaryMode.assignments.get(reviewAssignmentKey(input.repository, reviewer))?.type === "skip");
     });
-    if (singleReviewMode && skippedReviewers.length) {
+    if (singleReviewMode) {
         const account = input.repository.review?.account ?? "";
         const threads = await fetchUnresolvedThreads(exec, input.repository, input.pr, account);
         const assigned = assignThreadsByReviewFindingMarker({
             fallbackReviewerKeys: reviewerKeys,
-            headSha: meta.headRefOid,
             pr: input.pr,
             reviewerKeys,
             threads,
         });
-        for (const reviewer of skippedReviewers) {
+        for (const reviewer of input.repository.agents.reviewers) {
             const reviewerThreads = assigned[reviewer.key] ?? [];
             unresolvedThreadsByReviewer.set(reviewer.key, reviewerThreads);
+            if (preliminaryMode.assignments.get(reviewer.key)?.type !== "skip") {
+                continue;
+            }
             if (hasPendingThreadReply(reviewerThreads, account)) {
                 pendingThreadReplyReviewers.add(reviewer.key);
             }

package/package.json

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