next-issue 2.4.3 → 2.4.4

package/README.md

@@ -100,13 +100,19 @@ when:
 - the fixer reports that the checks fail for a reason the change did not cause,
   such as a test already broken on the base branch or one that fails only under
   load. It changes nothing and the reason reaches the pull request, rather than
-  the fixer spending its whole budget on a failure it cannot fix;
+  the fixer spending its whole budget on a failure it cannot fix. A commit it
+  makes anyway moves aside, as for a dispute of the findings;
 - a fixer disputes the findings and leaves a change behind, either in the
   worktree or in a commit of its own. A dispute carries a reason, not a diff,
   so the harness throws none of it away: an unclean worktree stays on the
-  disk, and a commit of the fixer stays on the local branch. Neither is
-  pushed, so the pull request shows no such change. It holds the reason of the
-  dispute, and the change is on the machine that ran the harness;
+  disk, and a commit of the fixer moves to a new local
+  `<n>-dispute-<commit>` branch, so the `<n>` branch keeps only what the pull
+  request holds. The worktree drops the change of that commit, unless the fixer
+  edited the worktree after the commit: then the worktree keeps both and stays
+  on the disk. Neither is pushed, so the pull request shows no such change. It
+  holds the reason of the dispute, and the change is on the machine that ran
+  the harness. The `<n>-dispute-<commit>` branches go away with the `<n>`
+  branch when the issue is done;
 - the reviewer repeats a finding set from an earlier round, which shows a
   ping-pong between the fixer and the reviewer, or a finding that the fixer
   disputed and the reviewer holds to. The person then decides;

package/dist/git.mjs

@@ -113,6 +113,18 @@ export async function discardChanges(cwd, target) {
     await must('git', ['clean', '-fd'], { cwd });
     return true;
 }
+export async function setAside(cwd, target, branch) {
+    await must('git', ['branch', branch], { cwd });
+    if (await isDirty(cwd)) {
+        await must('git', ['reset', '--soft', target], { cwd });
+        return;
+    }
+    await discardChanges(cwd, target);
+}
+export async function disputeBranches(repo, branch) {
+    const list = await must('git', ['for-each-ref', '--format=%(refname:short)', `refs/heads/${branch}-dispute-*`], { cwd: repo.root });
+    return list.split('\n').filter((line) => line.length > 0);
+}
 export async function push(cwd, remote, branch, force = false) {
     await must('git', ['push', '-u', ...(force ? ['--force'] : []), remote, branch], { cwd });
 }

package/dist/pipeline.mjs

@@ -1,7 +1,7 @@
 import { CODING, REVIEWING, runAgent } from './agents.mjs';
 import { managedLabels } from './config.mjs';
 import { run } from './exec.mjs';
-import { addWorktree, branchName, commitAll, deleteBranch, diff, discardChanges, findWorktree, isDirty, push, removeWorktree, resetToBase, revision, } from './git.mjs';
+import { addWorktree, branchName, commitAll, deleteBranch, diff, discardChanges, disputeBranches, findWorktree, isDirty, push, removeWorktree, resetToBase, revision, setAside, } from './git.mjs';
 import { commentOnPr, createPr, failedBaseLogs, failedCheckLogs, findPr, issueComments, markPrReady, mergeCommit, mergePr, openBlockers, openIssues, prMerged, reopenIssue, setLabel, viewIssue, waitForBaseChecks, waitForChecks, } from './github.mjs';
 import { message } from './observe.mjs';
 import { capDiff, fixPrompt, hookPrompt, implementPrompt, parseCommitSubject, parseUnrelated, reviewPrompt, } from './prompts.mjs';
@@ -17,6 +17,7 @@ export const PORTS = {
     detectSetupCommand,
     diff,
     discardChanges,
+    disputeBranches,
     failedBaseLogs,
     failedCheckLogs,
     findPr,
@@ -36,6 +37,7 @@ export const PORTS = {
     revision,
     runAgent,
     runSetup: async (command, cwd, timeoutMs) => run('bash', ['-lc', command], { cwd, timeoutMs }),
+    setAside,
     setLabel,
     viewIssue,
     waitForBaseChecks,
@@ -501,8 +503,10 @@ async function finish(job, outcome, reason) {
     const { repo, ports } = context;
     await cleanUp(context, log, branch);
     if (outcome === 'done') {
-        if (!(await ports.deleteBranch(repo, branch))) {
-            log.event('branch.kept', { branch }, 'quiet');
+        for (const name of [branch, ...(await ports.disputeBranches(repo, branch))]) {
+            if (!(await ports.deleteBranch(repo, name))) {
+                log.event('branch.kept', { branch: name }, 'quiet');
+            }
         }
         await dropState(repo, issue.number);
     }
@@ -531,7 +535,14 @@ async function fixRound(job, reason, detail, earlier, kind, force = false, from)
     }));
     const unrelated = parseUnrelated(result.text);
     if (unrelated !== undefined) {
-        const stray = (await ports.revision(worktree)) !== before || (await ports.isDirty(worktree));
+        const disputed = await ports.revision(worktree);
+        const moved = disputed !== before;
+        if (moved) {
+            const aside = `${branch}-dispute-${disputed.slice(0, 7)}`;
+            await ports.setAside(worktree, before, aside);
+            log.event('fix.aside', { branch: aside }, 'quiet');
+        }
+        const stray = moved || (await ports.isDirty(worktree));
         log.event('fix.unrelated', { reason, note: unrelated, stray }, 'quiet');
         return { committed: false, unrelated, stray };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-issue",
-  "version": "2.4.3",
+  "version": "2.4.4",
   "description": "Autonomous issue harness driven by the pi SDK",
   "keywords": [
     "github",