@warpmetrics/coder 0.2.8 → 0.2.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@warpmetrics/coder",
-  "version": "0.2.8",
+  "version": "0.2.10",
   "type": "module",
   "description": "Local agent loop for implementing GitHub issues with Claude Code. Powered by WarpMetrics.",
   "bin": {

package/src/git.js

@@ -17,6 +17,10 @@ export function createBranch(dir, name) {
   run(`git checkout -b ${name}`, { cwd: dir });
 }
 
+export function getHead(dir) {
+  return run(`git rev-parse HEAD`, { cwd: dir });
+}
+
 export function hasNewCommits(dir, base = 'main') {
   const log = run(`git log ${base}..HEAD --oneline`, { cwd: dir });
   return log.length > 0;
@@ -59,6 +63,10 @@ export function getReviewComments(prNumber, { repo }) {
   return JSON.parse(out);
 }
 
+export function dismissReview(prNumber, reviewId, { repo, message }) {
+  run(`gh api repos/${repo}/pulls/${prNumber}/reviews/${reviewId}/dismissals -X PUT -f message=${JSON.stringify(message)}`);
+}
+
 export function updatePRBody(prNumber, { repo, body }) {
   run(`gh pr edit ${prNumber} --repo ${repo} --body ${JSON.stringify(body)}`);
 }

package/src/revise.js

@@ -158,6 +158,8 @@ export async function revise(item, { board, config, log, refActId, since }) {
 
     const prompt = promptParts.join('\n');
 
+    const headBefore = git.getHead(workdir);
+
     log('  running claude...');
     claudeResult = await claude.run({
       prompt,
@@ -182,32 +184,61 @@ export async function revise(item, { board, config, log, refActId, since }) {
       git.commitAll(workdir, 'Address review feedback');
     }
 
-    // Push
-    log('  pushing...');
-    git.push(workdir, branch);
+    // Check if Claude actually made changes
+    const headAfter = git.getHead(workdir);
+    if (headAfter === headBefore) {
+      log('  no changes needed — review feedback already addressed');
 
-    // Update PR body with new act ID for next review cycle
-    if (actId) {
+      // Dismiss active CHANGES_REQUESTED reviews (they're stale)
       try {
-        let body = git.getPRBody(prNumber, { repo: repoName });
-        body = body.replace(/<!-- wm:act:wm_act_\w+ -->/, `<!-- wm:act:${actId} -->`);
-        if (!body.includes(`<!-- wm:act:${actId} -->`)) {
-          body += `\n\n<!-- wm:act:${actId} -->`;
+        const reviews = git.getReviews(prNumber, { repo: repoName });
+        for (const r of reviews) {
+          if (r.state === 'CHANGES_REQUESTED') {
+            git.dismissReview(prNumber, r.id, {
+              repo: repoName,
+              message: 'Code verified correct by warp-coder — no changes needed.',
+            });
+            log(`  dismissed stale review ${r.id}`);
+          }
         }
-        git.updatePRBody(prNumber, { repo: repoName, body });
       } catch (err) {
-        log(`  warning: could not update PR body with act ID: ${err.message}`);
+        log(`  warning: could not dismiss stale reviews: ${err.message}`);
       }
-    }
 
-    // Move back to In Review
-    try {
-      await board.moveToReview(item);
-    } catch (err) {
-      log(`  warning: could not move to In Review: ${err.message}`);
-    }
+      // Move back to In Review (no active CHANGES_REQUESTED, so won't be picked up again)
+      try {
+        await board.moveToReview(item);
+      } catch (err) {
+        log(`  warning: could not move to In Review: ${err.message}`);
+      }
+      success = true;
+    } else {
+      // Push
+      log('  pushing...');
+      git.push(workdir, branch);
+
+      // Update PR body with new act ID for next review cycle
+      if (actId) {
+        try {
+          let body = git.getPRBody(prNumber, { repo: repoName });
+          body = body.replace(/<!-- wm:act:wm_act_\w+ -->/, `<!-- wm:act:${actId} -->`);
+          if (!body.includes(`<!-- wm:act:${actId} -->`)) {
+            body += `\n\n<!-- wm:act:${actId} -->`;
+          }
+          git.updatePRBody(prNumber, { repo: repoName, body });
+        } catch (err) {
+          log(`  warning: could not update PR body with act ID: ${err.message}`);
+        }
+      }
 
-    success = true;
+      // Move back to In Review
+      try {
+        await board.moveToReview(item);
+      } catch (err) {
+        log(`  warning: could not move to In Review: ${err.message}`);
+      }
+      success = true;
+    }
   } catch (err) {
     taskError = err.message;
     log(`  failed: ${err.message}`);

package/src/warp.js

@@ -186,6 +186,27 @@ export async function emitAct(apiKey, { outcomeId, actId, name, opts }) {
   });
 }
 
+export async function findIssueRun(apiKey, { repo, issueNumber }) {
+  const runs = await findRuns(apiKey, 'issue');
+  const match = runs.find(r =>
+    r.opts?.repo === repo &&
+    r.opts?.issue === String(issueNumber)
+  );
+  if (!match) return null;
+
+  const res = await fetch(`${API_URL}/v1/runs/${match.id}`, {
+    headers: { Authorization: `Bearer ${apiKey}` },
+  });
+  if (!res.ok) return null;
+  const { data } = await res.json();
+
+  const outcomes = data.outcomes || [];
+  const lastOutcome = outcomes[outcomes.length - 1];
+  const blockedAt = lastOutcome?.name === 'Max Retries' ? lastOutcome.timestamp : null;
+
+  return { runId: match.id, blockedAt, countSince: null };
+}
+
 export async function countRevisions(apiKey, { prNumber, repo, since }) {
   try {
     const runs = await findRuns(apiKey, 'agent-pipeline');

package/src/watch.js

@@ -58,7 +58,7 @@ export async function watch() {
             const issue = await warp.createIssueRun(config.warpmetricsApiKey, {
               repo: repoName, issueNumber, issueTitle,
             });
-            issueRuns.set(issueNumber, { runId: issue.runId, blockedAt: null });
+            issueRuns.set(issueNumber, { runId: issue.runId, blockedAt: null, countSince: null });
             implementActId = issue.actId;
             log(`  issue run: ${issue.runId}`);
           } catch (err) {
@@ -74,15 +74,28 @@ export async function watch() {
       for (const item of reviewItems) {
         if (!running) break;
         const issueNumber = item.content?.number;
-        const issueCtx = issueNumber ? issueRuns.get(issueNumber) : null;
+        let issueCtx = issueNumber ? issueRuns.get(issueNumber) : null;
         log(`Found review feedback: PR #${item._prNumber || item.content?.number}`);
 
+        // Recover issue run from WM if watcher was restarted
+        if (!issueCtx && issueNumber && config.warpmetricsApiKey) {
+          try {
+            const recovered = await warp.findIssueRun(config.warpmetricsApiKey, { repo: repoName, issueNumber });
+            if (recovered) {
+              issueRuns.set(issueNumber, recovered);
+              issueCtx = recovered;
+              log(`  recovered issue run: ${recovered.runId}`);
+            }
+          } catch (err) {
+            log(`  warning: could not recover issue run: ${err.message}`);
+          }
+        }
+
         // Detect resume: item was previously blocked, human moved it back to In Review
-        let since = null;
         if (issueCtx?.blockedAt) {
-          since = issueCtx.blockedAt;
+          issueCtx.countSince = issueCtx.blockedAt;
           issueCtx.blockedAt = null;
-          log(`  resumed (counter reset)`);
+          log(`  resumed from blocked (counter reset, since: ${issueCtx.countSince})`);
           if (config.warpmetricsApiKey) {
             try {
               await warp.closeIssueRun(config.warpmetricsApiKey, {
@@ -92,12 +105,13 @@ export async function watch() {
           }
         }
 
-        const result = await revise(item, { board, config, log, refActId: item._reviewActId, since });
+        const result = await revise(item, { board, config, log, refActId: item._reviewActId, since: issueCtx?.countSince });
 
         // Record outcome on the issue run if revision failed terminally
         if (!result.success && issueCtx && config.warpmetricsApiKey) {
           const name = result.reason === 'max_retries' ? 'Max Retries' : 'Revision Failed';
           issueCtx.blockedAt = new Date().toISOString();
+          issueCtx.countSince = null;
           try {
             await warp.closeIssueRun(config.warpmetricsApiKey, {
               runId: issueCtx.runId,