next-issue 2.16.0 → 2.18.0

package/README.md

@@ -7,6 +7,28 @@ the [pi](https://github.com/earendil-works/pi) SDK. Run it inside a git reposito
 npx -y next-issue@latest
 ```
 
+## Overview
+
+The harness claims the oldest open issue with the `status:todo` label and gives it its own git worktree and branch. An implementer agent does the work, and the harness commits, pushes and opens a draft pull request. It then waits for the checks. A fixer agent repairs a red build, and a reviewer agent judges the diff. The fixer deals with any open finding, and the checks and review run again. With no open finding left, the pull request is marked ready and the issue gets `status:done`. When a budget runs out or the work cannot go on, the issue gets `status:needs-human` and goes to a person. Then the harness takes the next issue.
+
+```mermaid
+flowchart TD
+    claim[Claim the oldest status:todo issue] --> worktree[Add a worktree and branch]
+    worktree --> implement[Implementer does the work, harness commits and pushes]
+    implement --> pr[Open a draft pull request]
+    pr --> checks{Checks pass?}
+    checks -- no --> fixCi[Fixer repairs the build]
+    fixCi --> checks
+    checks -- yes --> review{Reviewer finds an open finding?}
+    review -- yes --> fixReview[Fixer deals with the findings]
+    fixReview --> checks
+    review -- no --> done[Mark ready, set status:done]
+    fixCi -. budget runs out .-> human[Hand over to a person: status:needs-human]
+    fixReview -. budget runs out .-> human
+    done --> claim
+    human --> claim
+```
+
 ## What it does
 
 Before each issue, fetch the open issues again and take the oldest one not yet

package/dist/fix.mjs

@@ -4,16 +4,19 @@ import { message } from './observe.mjs';
 import { fixPrompt, hookPrompt, parseCommitSubject, parseUnrelated, } from './prompts.mjs';
 import { writeState } from './state.mjs';
 export async function fixRound(job, reason, detail, earlier, kind, force = false, from) {
-    const { context, issue, log, worktree, branch } = job;
-    const { ports } = context;
+    const { context, issue, log, state, worktree, branch } = job;
+    const { ports, config } = context;
     const before = from ?? (await ports.revision(worktree));
-    const result = await log.step('fix', { reason }, () => ports.runAgent(log, {
+    const budget = kind === 'review'
+        ? { round: state.reviewRounds, limit: config.maxReviewRounds }
+        : { round: state.ciFixes, limit: config.maxCiFixes };
+    const result = await log.step('fix', { reason, cause: kind, ...budget }, () => ports.runAgent(log, {
         name: 'fixer',
         cwd: worktree,
         prompt: fixPrompt(issue, reason, detail, earlier, kind, job.reply),
         profile: CODING,
-        model: context.config.models.fixer,
-        timeoutMs: context.config.agentTimeoutMinutes * 60_000,
+        model: config.models.fixer,
+        timeoutMs: config.agentTimeoutMinutes * 60_000,
     }));
     const unrelated = parseUnrelated(result.text);
     if (unrelated !== undefined) {
@@ -33,7 +36,7 @@ export async function fixRound(job, reason, detail, earlier, kind, force = false
         log.event('commit.empty', { reason }, 'quiet');
         return { committed: false };
     }
-    await ports.push(worktree, context.config.remote, branch, force);
+    await ports.push(worktree, config.remote, branch, force);
     return { committed: true };
 }
 export async function commitWork(job, before, subject, role) {
@@ -59,7 +62,7 @@ export async function commitWork(job, before, subject, role) {
             await writeState(repo, state);
             const output = message(error).slice(-config.logMaxChars);
             log.event('commit.fail', { role, output }, 'quiet');
-            const fix = await log.step('commit-fix', { role }, () => ports.runAgent(log, {
+            const fix = await log.step('commit-fix', { role, round: state.commitFixes, limit: config.maxCommitFixes }, () => ports.runAgent(log, {
                 name: role,
                 cwd: worktree,
                 prompt: hookPrompt(issue, comments, role, output),

package/dist/progress.mjs

@@ -9,6 +9,20 @@ function toolLine(record) {
     const line = typeof command === 'string' ? `${tool} ${command.split('\n')[0]}` : tool;
     return line.length > TOOL_LINE_MAX ? `${line.slice(0, TOOL_LINE_MAX - 1)}…` : line;
 }
+function stageOf(step, record) {
+    const cause = record['cause'];
+    const name = step === 'fix' && typeof cause === 'string'
+        ? `${cause} fix`
+        : step === 'commit-fix'
+            ? 'commit fix'
+            : step;
+    const round = record['round'];
+    const limit = record['limit'];
+    if (typeof round !== 'number') {
+        return name;
+    }
+    return typeof limit === 'number' ? `${name} ${round}/${limit}` : `${name} ${round}`;
+}
 function change(progress, issue, update) {
     return {
         ...progress,
@@ -45,16 +59,15 @@ export function applyEvent(progress, record) {
         const issues = progress.issues.filter((current) => current.issue !== issue);
         return { ...progress, issues, handled };
     }
+    if (kind === 'issues') {
+        return { ...progress, open: Number(record['open']) };
+    }
     if (kind === 'issues.fail') {
         return { ...progress, error: String(record['error']) };
     }
     if (kind.endsWith('.start') && !NOT_STAGES.has(kind)) {
-        const stage = kind.slice(0, -'.start'.length);
-        const round = record['round'];
-        return change(progress, issue, (current) => ({
-            ...current,
-            stage: typeof round === 'number' ? `${stage} ${round}` : stage,
-        }));
+        const stage = stageOf(kind.slice(0, -'.start'.length), record);
+        return change(progress, issue, (current) => ({ ...current, stage }));
     }
     return progress;
 }

package/dist/review.mjs

@@ -22,6 +22,7 @@ export async function reviewRound(job) {
     const patch = capDiff(full, config.diffMaxChars);
     const fields = {
         round: state.reviewRounds,
+        limit: config.maxReviewRounds,
         diffChars: full.length,
         cut: full.length > config.diffMaxChars,
     };

package/package.json

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