openairev 0.3.7 → 0.3.9

package/README.md

@@ -94,6 +94,15 @@ Options:
   --dry-run               Show what would happen without executing
 ```
 
+### `openairev wait`
+
+Stream review progress and block until the verdict is ready.
+
+```
+Options:
+  --file <path>  Path to progress.json (auto-detected from .openairev/)
+```
+
 ### `openairev resume`
 
 Resume an active or blocked workflow. If blocked on `awaiting_user`, prompts for answers to pending questions.
@@ -213,6 +222,7 @@ Restart your agent CLI after adding.
 | Tool | Description |
 |------|-------------|
 | `openairev_review` | Start a review in the background. Returns immediately. |
+| `openairev_cancel` | Cancel the currently running review. |
 | `openairev_status` | Check review progress (prefer `openairev wait` instead). |
 | `openairev_run_tests` | Run project test suite |
 | `openairev_run_lint` | Run linter |
@@ -224,6 +234,7 @@ The review runs asynchronously so you can see progress:
 
 1. Call `openairev_review` → starts review in background, returns immediately
 2. Run `openairev wait` via Bash → streams progress (files read, commands run, tokens) and outputs the verdict when done
+3. If stuck, call `openairev_cancel` to kill the running review and retry with a smaller diff
 
 One MCP call + one Bash call. No polling, no sleep loops. The AI can also launch the review in a sub-agent and continue other work while it runs.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openairev",
-  "version": "0.3.7",
+  "version": "0.3.9",
   "description": "Cross-model AI code reviewer — independent review for AI-assisted coding workflows",
   "type": "module",
   "bin": {

package/src/agents/claude-code.js

@@ -23,6 +23,7 @@ export class ClaudeCodeAdapter {
     continueSession = false,
     sessionName = null,
     stream = false,
+    signal,
   } = {}) {
     const args = ['-p', prompt];
 
@@ -51,7 +52,7 @@ export class ClaudeCodeAdapter {
       onProgress: stream.onProgress,
     }) : undefined;
 
-    const result = await exec(this.cmd, args, { onData: summarizer, cwd: this.cwd });
+    const result = await exec(this.cmd, args, { onData: summarizer, cwd: this.cwd, signal });
 
     if (stream) {
       return parseClaudeStreamOutput(result.stdout, {

package/src/agents/codex.js

@@ -16,7 +16,7 @@ export class CodexAdapter {
     this.sessionId = id;
   }
 
-  async run(prompt, { useSchema = false, schemaFile = 'verdict-schema.json', continueSession = false, sessionName = null, stream = false } = {}) {
+  async run(prompt, { useSchema = false, schemaFile = 'verdict-schema.json', continueSession = false, sessionName = null, stream = false, signal } = {}) {
     const args = ['exec'];
 
     if (continueSession && this.sessionId) {
@@ -36,7 +36,7 @@ export class CodexAdapter {
       tty: stream.tty !== false,
       onProgress: stream.onProgress,
     }) : undefined;
-    const result = await exec(this.cmd, args, { onData: summarizer });
+    const result = await exec(this.cmd, args, { onData: summarizer, signal });
 
     try {
       const lines = result.stdout.trim().split('\n');

package/src/agents/exec-helper.js

@@ -2,13 +2,24 @@ import { spawn } from 'child_process';
 
 const MAX_BUFFER = 10 * 1024 * 1024;
 
-export function exec(cmd, args, { onData, cwd } = {}) {
+export function exec(cmd, args, { onData, cwd, signal } = {}) {
   return new Promise((resolve, reject) => {
     const child = spawn(cmd, args, {
       timeout: 300_000,
       cwd,
     });
 
+    if (signal) {
+      if (signal.aborted) {
+        child.kill();
+        return reject(new Error(`${cmd} aborted`));
+      }
+      signal.addEventListener('abort', () => {
+        killed = true;
+        child.kill();
+      }, { once: true });
+    }
+
     const stdoutChunks = [];
     const stderrChunks = [];
     let stdoutLen = 0;
@@ -43,7 +54,9 @@ export function exec(cmd, args, { onData, cwd } = {}) {
     child.on('close', (code) => {
       const stdout = Buffer.concat(stdoutChunks).toString();
       const stderr = Buffer.concat(stderrChunks).toString();
-      if (killed) {
+      if (signal?.aborted) {
+        reject(new Error(`${cmd} aborted`));
+      } else if (killed) {
         reject(new Error(`${cmd} output exceeded ${MAX_BUFFER} bytes`));
       } else if (code !== 0 && !stdout) {
         reject(new Error(`${cmd} failed (exit ${code}): ${stderr}`));

package/src/cli/init.js

@@ -264,6 +264,7 @@ This project uses OpenAIRev for independent AI code review. When the user asks t
 - ALWAYS scope \`diff_cmd\` to your changed files. Never send the full repo diff — the reviewer will run out of budget.
 - NEVER use \`sleep\`, polling loops, or manually read progress.json. \`openairev wait\` handles everything.
 - NEVER call \`openairev_status\`. Use \`openairev wait\` instead.
+- If a review is stuck or you want to retry, call \`openairev_cancel\` to kill the running review, then start a new one.
 - You MAY launch the review + wait in a sub-agent and continue other work in parallel.
 - Treat review feedback as peer review — use your judgment, focus on \`critical_issues\` and \`repair_instructions\`.
 ${marker}
@@ -319,6 +320,7 @@ This project uses OpenAIRev for independent AI code review. When the user asks t
 - ALWAYS scope \`diff_cmd\` to your changed files. Never send the full repo diff — the reviewer will run out of budget.
 - NEVER use \`sleep\`, polling loops, or manually read progress.json. \`openairev wait\` handles everything.
 - NEVER call \`openairev_status\`. Use \`openairev wait\` instead.
+- If a review is stuck or you want to retry, call \`openairev_cancel\` to kill the running review, then start a new one.
 - Treat review feedback as peer review — use your judgment, focus on \`critical_issues\` and \`repair_instructions\`.
 ${marker}
 `;

package/src/cli/wait.js

@@ -5,6 +5,12 @@ import { getConfigDir } from '../config/config-loader.js';
 export async function waitCommand(options) {
   const progressFile = options.file || join(getConfigDir(), 'progress.json');
 
+  // Wait up to 30s for the progress file to appear (race with MCP server)
+  let waited = 0;
+  while (!existsSync(progressFile) && waited < 30_000) {
+    await sleep(1000);
+    waited += 1000;
+  }
   if (!existsSync(progressFile)) {
     console.log('No review in progress. Call openairev_review first.');
     process.exit(1);
@@ -23,7 +29,7 @@ export async function waitCommand(options) {
       }
       lastLen = lines.length;
 
-      if (data.status === 'completed' || data.status === 'error') {
+      if (data.status === 'completed' || data.status === 'error' || data.status === 'cancelled') {
         clearInterval(timer);
         printResult(data);
         resolve();
@@ -32,6 +38,10 @@ export async function waitCommand(options) {
   });
 }
 
+function sleep(ms) {
+  return new Promise((resolve) => setTimeout(resolve, ms));
+}
+
 function readProgress(path) {
   try {
     return JSON.parse(readFileSync(path, 'utf-8'));
@@ -41,6 +51,10 @@ function readProgress(path) {
 }
 
 function printResult(data) {
+  if (data.status === 'cancelled') {
+    console.log('\nReview cancelled.');
+    process.exit(1);
+  }
   if (data.status === 'error') {
     console.log(`\nReview failed: ${data.error}`);
     process.exit(1);

package/src/mcp/mcp-server.js

@@ -17,6 +17,7 @@ const config = loadConfig(cwd);
 const PROGRESS_FILE = join(cwd, '.openairev', 'progress.json');
 
 let activeReview = null;
+let activeAbort = null;
 
 const server = new McpServer({
   name: 'openairev',
@@ -60,12 +61,14 @@ server.tool(
       writeProgress({ status: 'running', reviewer: reviewerName, started: new Date().toISOString(), progress: lines, verdict: null });
     };
 
+    activeAbort = new AbortController();
     activeReview = runReview(diffContent, {
       config,
       reviewerName,
       taskDescription: task_description,
       cwd,
       stream: { onProgress },
+      signal: activeAbort.signal,
     }).then((review) => {
       const session = createSession({ executor: execAgent, reviewer: reviewerName });
       session.iterations.push({ round: 1, review, timestamp: new Date().toISOString() });
@@ -81,10 +84,13 @@ server.tool(
         executor_feedback: review.executor_feedback,
       });
       activeReview = null;
+      activeAbort = null;
       return review;
     }).catch((err) => {
-      writeProgress({ status: 'error', reviewer: reviewerName, error: err.message, progress: [], verdict: null });
+      const status = activeAbort?.signal?.aborted ? 'cancelled' : 'error';
+      writeProgress({ status, reviewer: reviewerName, error: err.message, progress: [], verdict: null });
       activeReview = null;
+      activeAbort = null;
     });
 
     return {
@@ -128,6 +134,19 @@ server.tool(
   }
 );
 
+server.tool(
+  'openairev_cancel',
+  'Cancel the currently running review. Use this when the review is taking too long, the diff was too large, or you want to retry with different parameters.',
+  {},
+  async () => {
+    if (!activeReview || !activeAbort) {
+      return { content: [{ type: 'text', text: 'No review is currently running.' }] };
+    }
+    activeAbort.abort();
+    return { content: [{ type: 'text', text: 'Review cancelled.' }] };
+  }
+);
+
 server.tool(
   'openairev_run_tests',
   'Run the project test suite and return pass/fail results.',

package/src/review/review-runner.js

@@ -13,6 +13,7 @@ export async function runReview(content, {
   cwd = process.cwd(),
   sessionId = null,
   stream = false,
+  signal,
 }) {
   const adapter = createAdapter(reviewerName, config, { cwd });
 
@@ -41,6 +42,7 @@ export async function runReview(content, {
     continueSession: !!sessionId,
     sessionName: sessionId ? undefined : `review-${Date.now()}`,
     stream: stream ? { reviewerName, tty: stream === true, onProgress: stream.onProgress } : false,
+    signal,
   });
 
   const verdict = extractVerdict(result);