openairev 0.3.6 → 0.3.8

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.6",
+  "version": "0.3.8",
   "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

@@ -257,14 +257,16 @@ ${marker}
 
 This project uses OpenAIRev for independent AI code review. When the user asks to "review", "get a review", or "check my changes":
 
-1. Call \`openairev_review\` with \`executor: "claude_code"\` and \`diff_cmd\` scoped to the files you changed (e.g. \`"git diff HEAD -- src/auth.ts src/routes.ts"\`). The review starts in the background and returns immediately.
-2. Run \`openairev wait\` via Bash to stream progress and get the verdict when done. This is a single blocking call — no polling needed.
-
-**Tips**:
-- Always use \`diff_cmd\` to scope the diff to your changed files. Do NOT let it auto-detect — the full repo diff may be too large.
-- You can launch the review in a **sub-agent** and continue other work while it runs.
-- When you receive review feedback, treat it as **peer review** — use your judgment, don't blindly apply every suggestion.
-- Focus on high-confidence \`critical_issues\` and \`repair_instructions\`.
+1. Call \`openairev_review\` (MCP tool) with \`executor: "claude_code"\` and \`diff_cmd\` scoped to the files you changed (e.g. \`"git diff HEAD -- src/auth.ts src/routes.ts"\`). It returns immediately.
+2. Run \`openairev wait\` via Bash. It streams live progress and blocks until the verdict is ready — one command, that's it.
+
+**Rules — you MUST follow these:**
+- 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}
 `;
 
@@ -311,13 +313,15 @@ ${marker}
 
 This project uses OpenAIRev for independent AI code review. When the user asks to "review", "get a review", or "check my changes":
 
-1. Call \`openairev_review\` with \`executor: "codex"\` and \`diff_cmd\` scoped to the files you changed (e.g. \`"git diff HEAD -- src/auth.ts src/routes.ts"\`). The review starts in the background and returns immediately.
-2. Run \`openairev wait\` to stream progress and get the verdict when done. This is a single blocking call — no polling needed.
+1. Call \`openairev_review\` (MCP tool) with \`executor: "codex"\` and \`diff_cmd\` scoped to the files you changed (e.g. \`"git diff HEAD -- src/auth.ts src/routes.ts"\`). It returns immediately.
+2. Run \`openairev wait\` via Bash. It streams live progress and blocks until the verdict is ready — one command, that's it.
 
-**Tips**:
-- Always use \`diff_cmd\` to scope the diff to your changed files. Do NOT let it auto-detect — the full repo diff may be too large.
-- When you receive review feedback, treat it as **peer review** — use your judgment, don't blindly apply every suggestion.
-- Focus on high-confidence \`critical_issues\` and \`repair_instructions\`.
+**Rules — you MUST follow these:**
+- 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

@@ -23,7 +23,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();
@@ -41,6 +41,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);