opencode-magi 0.0.0-dev-20260520163717 → 0.0.0-dev-20260520163847

package/dist/orchestrator/triage.js

@@ -4,7 +4,7 @@ import { issueRunOutputDir } from "../config/output";
 import { worktreeBaseDir } from "../config/worktree";
 import { assignIssue, closeIssue, closePullRequest, configureGitIdentity, createPullRequest, fetchIssue, fetchIssueComments, fetchRelatedPullRequests, postIssueComment, pushHead, removeIssueLabels, removeWorktree, searchDuplicateIssues, shellQuote, updateIssueComment, } from "../github/commands";
 import { composeTriageBugPrompt, composeTriageActionPrompt, composeTriageCommentClassificationPrompt, composeTriageCommentPrompt, composeTriageCreatePrPrompt, composeTriageDuplicatePrompt, composeTriageExistingPrPrompt, composeTriageFeaturePrompt, composeTriageKindPrompt, composeTriageQuestionPrompt, composeTriageReconsiderPrompt, } from "../prompts/compose";
-import { parseEditOutput, parseTriageActionOutput, parseTriageBinaryOutput, parseTriageCommentClassificationOutput, parseTriageDuplicateOutput, parseTriageExistingPrOutput, parseTriageFinalOutput, parseTriageKindOutput, } from "../prompts/output";
+import { parseTriageActionOutput, parseTriageBinaryOutput, parseTriageCommentClassificationOutput, parseTriageCreatePrOutput, parseTriageDuplicateOutput, parseTriageExistingPrOutput, parseTriageFinalOutput, parseTriageKindOutput, } from "../prompts/output";
 import { aggregateStringMajority, majorityThreshold } from "./majority";
 import { runModelText, runModelWithRepair } from "./model";
 const MARKER_PREFIX = "opencode-magi:triage";
@@ -65,6 +65,10 @@ function labelsContain(labels, targets) {
     const set = new Set(labels.map((label) => label.toLowerCase()));
     return targets.some((target) => set.has(target.toLowerCase()));
 }
+function existingClearLabels(issue, labels) {
+    const existing = new Set(issue.labels.map((label) => label.toLowerCase()));
+    return labels.filter((label) => existing.has(label.toLowerCase()));
+}
 export function resolveIssueKind(issue, repository) {
     const triage = repository.triage;
     if (!triage)
@@ -251,6 +255,11 @@ function markerPr(marker) {
     const pr = Number(marker.pr);
     return Number.isInteger(pr) && pr > 0 ? pr : undefined;
 }
+function pullRequestNumberFromUrl(url) {
+    const match = url.match(/\/pull\/(\d+)(?:\D|$)/);
+    const number = match ? Number(match[1]) : undefined;
+    return number && Number.isInteger(number) ? number : undefined;
+}
 export function mentionAllowed(comment, repository) {
     const safety = repository.triage?.safety;
     if (!safety)
@@ -329,6 +338,30 @@ function actionPlan(input) {
         postComment: true,
     };
 }
+function previousAutomationPlan(input) {
+    const base = actionPlan({ result: input.result, triage: input.triage });
+    const clearLabels = base.clearLabels &&
+        existingClearLabels(input.issue, input.triage.automation.clear).length > 0;
+    const closeIssue = input.marker.action === "CLOSE" &&
+        base.closeIssue &&
+        input.issue.state === "OPEN";
+    const createPr = input.marker.action === "PR" &&
+        base.createPr &&
+        !markerPr(input.marker) &&
+        !input.relationship.relatedPullRequests.length;
+    if (!clearLabels && !closeIssue && !createPr)
+        return undefined;
+    const action = closeIssue ? "CLOSE" : createPr ? "PR" : "CLEAR_ONLY";
+    return {
+        ...base,
+        action,
+        allowedActions: [action],
+        clearLabels,
+        closeIssue,
+        createPr,
+        postComment: false,
+    };
+}
 async function runActionPrompt(input) {
     const agent = input.input.repository.agents.triage?.[0];
     if (!agent)
@@ -454,7 +487,7 @@ async function persistProcessedMarker(input) {
         action: input.marker.action ?? input.marker.result ?? "ASK",
         checkpoint: markerCheckpoint(input.marker),
         issue: input.issue.number,
-        pr: markerPr(input.marker),
+        pr: input.pr ?? markerPr(input.marker),
         processed: input.processed,
         result: input.marker.result ?? "ASK",
     });
@@ -471,7 +504,7 @@ async function finishWithResult(input) {
     const triage = input.input.repository.triage;
     if (!triage)
         throw new Error("triage configuration is required");
-    const plan = actionPlan({ result: input.result, triage });
+    const plan = input.plan ?? actionPlan({ result: input.result, triage });
     await runActionPrompt({
         context: input.context,
         input: input.input,
@@ -502,6 +535,12 @@ async function finishWithResult(input) {
                 repository: input.input.repository,
             });
         }
+        if (plan.clearLabels) {
+            const clearLabels = existingClearLabels(input.issue, triage.automation.clear);
+            if (clearLabels.length) {
+                await removeIssueLabels(input.input.exec, input.input.repository, input.issue.number, clearLabels, triage.account ?? "");
+            }
+        }
         if (plan.closeIssue) {
             const closedPrs = [];
             for (const pr of input.relationship.relatedPullRequests.filter((pr) => pr.state === "OPEN")) {
@@ -522,8 +561,18 @@ async function finishWithResult(input) {
             if (prUrl)
                 await writeJson(join(input.outputDir, "pr.json"), { url: prUrl });
         }
-        if (plan.clearLabels) {
-            await removeIssueLabels(input.input.exec, input.input.repository, input.issue.number, triage.automation.clear, triage.account ?? "");
+        if (input.previousMarker && prUrl) {
+            await persistProcessedMarker({
+                account: triage.account ?? "",
+                comments: input.relationship.comments,
+                exec: input.input.exec,
+                issue: input.issue,
+                marker: input.previousMarker,
+                outputDir: input.outputDir,
+                pr: pullRequestNumberFromUrl(prUrl),
+                processed: input.processed ?? input.previousMarker.processed,
+                repository: input.input.repository,
+            });
         }
     }
     const report = [
@@ -589,7 +638,7 @@ async function createImplementationPr(input) {
             client: input.input.client,
             model: creator.model,
             options: creator.options,
-            parse: parseEditOutput,
+            parse: parseTriageCreatePrOutput,
             permission: creator.permission,
             prompt,
             repairAttempts: 3,
@@ -650,6 +699,26 @@ export async function runTriage(input) {
     if (relationship.previousMarker) {
         if (!relationship.mentionReplies.length) {
             const result = finalResultFromMarker(relationship.previousMarker);
+            const plan = previousAutomationPlan({
+                issue,
+                marker: relationship.previousMarker,
+                relationship,
+                result,
+                triage,
+            });
+            if (plan) {
+                return finishWithResult({
+                    context,
+                    input,
+                    issue,
+                    outputDir,
+                    plan,
+                    previousMarker: relationship.previousMarker,
+                    processed,
+                    relationship,
+                    result,
+                });
+            }
             const report = `Magi triage skipped #${issue.number} because no eligible mention replies were found for reconsideration.`;
             await writeFile(join(outputDir, "report.md"), `${report}\n`);
             return { issue: issue.number, outputDir, report, result };
@@ -746,6 +815,10 @@ export async function runTriage(input) {
                         outputDir,
                         repository: input.repository,
                     });
+                    const clearLabels = existingClearLabels(issue, triage.automation.clear);
+                    if (clearLabels.length) {
+                        await removeIssueLabels(input.exec, input.repository, issue.number, clearLabels, triage.account);
+                    }
                     const closedPrs = [];
                     for (const pr of relationship.relatedPullRequests.filter((pr) => pr.state === "OPEN")) {
                         await closePullRequest(input.exec, input.repository, pr.number, triage.account);
@@ -754,7 +827,6 @@ export async function runTriage(input) {
                     if (closedPrs.length)
                         await writeJson(join(outputDir, "closed-prs.json"), closedPrs);
                     await closeIssue(input.exec, input.repository, issue.number, triage.account);
-                    await removeIssueLabels(input.exec, input.repository, issue.number, triage.automation.clear, triage.account);
                 }
                 const report = `Magi triage closed #${issue.number} because a related PR was merged.`;
                 await writeFile(join(outputDir, "report.md"), `${report}\n`);

package/dist/prompts/output.js

@@ -305,7 +305,7 @@ export function parseCiClassificationOutput(text) {
         }),
     };
 }
-export function parseEditOutput(text) {
+function parseEditOutputWithOptions(text, options) {
     const data = extractJson(text);
     if (!data || typeof data !== "object")
         throw new Error("edit output must be an object");
@@ -323,7 +323,7 @@ export function parseEditOutput(text) {
             commentId: requireNumber(value.commentId, `responses[${index}].commentId`),
         };
     });
-    if (!responses.length)
+    if (options.requireResponses && !responses.length)
         throw new Error("responses must not be empty");
     if (data.mode === "EDITED") {
         if (!filesTouched.length)
@@ -351,3 +351,9 @@ export function parseEditOutput(text) {
         responses,
     };
 }
+export function parseEditOutput(text) {
+    return parseEditOutputWithOptions(text, { requireResponses: true });
+}
+export function parseTriageCreatePrOutput(text) {
+    return parseEditOutputWithOptions(text, { requireResponses: false });
+}

package/package.json

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