@warpmetrics/coder 0.2.6 → 0.2.8

package/package.json

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

package/src/revise.js

@@ -12,7 +12,7 @@ import { reflect } from './reflect.js';
 
 const CONFIG_DIR = '.warp-coder';
 
-export async function revise(item, { board, config, log, refActId }) {
+export async function revise(item, { board, config, log, refActId, since }) {
   const prNumber = item._prNumber || item.content?.number;
   const repo = config.repo;
   const repoName = repo.replace(/\.git$/, '').replace(/^.*github\.com[:\/]/, '');
@@ -36,13 +36,13 @@ export async function revise(item, { board, config, log, refActId }) {
   // Check revision limit
   if (config.warpmetricsApiKey) {
     try {
-      const count = await warp.countRevisions(config.warpmetricsApiKey, { prNumber, repo: repoName });
-      if (count >= maxRevisions) {
-        log(`  revision limit reached (${count}/${maxRevisions}) — moving to Blocked`);
+      const revisionCount = await warp.countRevisions(config.warpmetricsApiKey, { prNumber, repo: repoName, since });
+      if (revisionCount >= maxRevisions) {
+        log(`  revision limit reached (${revisionCount}/${maxRevisions}) — moving to Blocked`);
         try { await board.moveToBlocked(item); } catch {}
-        return false;
+        return { success: false, reason: 'max_retries', count: revisionCount };
       }
-      log(`  revision ${count + 1}/${maxRevisions}`);
+      log(`  revision ${revisionCount + 1}/${maxRevisions}`);
     } catch (err) {
       log(`  warning: revision check failed: ${err.message}`);
     }
@@ -268,5 +268,5 @@ export async function revise(item, { board, config, log, refActId }) {
     rmSync(workdir, { recursive: true, force: true });
   }
 
-  return success;
+  return { success, reason: success ? 'ok' : 'error' };
 }

package/src/warp.js

@@ -186,13 +186,14 @@ export async function emitAct(apiKey, { outcomeId, actId, name, opts }) {
   });
 }
 
-export async function countRevisions(apiKey, { prNumber, repo }) {
+export async function countRevisions(apiKey, { prNumber, repo, since }) {
   try {
     const runs = await findRuns(apiKey, 'agent-pipeline');
     return runs.filter(r =>
       r.opts?.step === 'revise' &&
       r.opts?.pr_number === String(prNumber) &&
-      r.opts?.repo === repo
+      r.opts?.repo === repo &&
+      (!since || new Date(r.createdAt) >= new Date(since))
     ).length;
   } catch {
     return 0;

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 });
+            issueRuns.set(issueNumber, { runId: issue.runId, blockedAt: null });
             implementActId = issue.actId;
             log(`  issue run: ${issue.runId}`);
           } catch (err) {
@@ -73,8 +73,42 @@ export async function watch() {
       const reviewItems = await board.listInReview();
       for (const item of reviewItems) {
         if (!running) break;
+        const issueNumber = item.content?.number;
+        const issueCtx = issueNumber ? issueRuns.get(issueNumber) : null;
         log(`Found review feedback: PR #${item._prNumber || item.content?.number}`);
-        await revise(item, { board, config, log, refActId: item._reviewActId });
+
+        // Detect resume: item was previously blocked, human moved it back to In Review
+        let since = null;
+        if (issueCtx?.blockedAt) {
+          since = issueCtx.blockedAt;
+          issueCtx.blockedAt = null;
+          log(`  resumed (counter reset)`);
+          if (config.warpmetricsApiKey) {
+            try {
+              await warp.closeIssueRun(config.warpmetricsApiKey, {
+                runId: issueCtx.runId, name: 'Resumed',
+              });
+            } catch {}
+          }
+        }
+
+        const result = await revise(item, { board, config, log, refActId: item._reviewActId, since });
+
+        // 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();
+          try {
+            await warp.closeIssueRun(config.warpmetricsApiKey, {
+              runId: issueCtx.runId,
+              name,
+              opts: { pr_number: String(item._prNumber || ''), revisions: String(result.count || '') },
+            });
+            log(`  issue run: ${name}`);
+          } catch (err) {
+            log(`  warning: issue run outcome failed: ${err.message}`);
+          }
+        }
       }
 
       // 3. Merge approved PRs