openairev 0.3.1 → 0.3.2

package/README.md

@@ -212,12 +212,22 @@ Restart your agent CLI after adding.
 
 | Tool | Description |
 |------|-------------|
-| `openairev_review` | Send diff to reviewer, get structured verdict |
-| `openairev_status` | Check most recent review result |
+| `openairev_review` | Start a review in the background. Returns immediately. |
+| `openairev_status` | Check review progress and get the verdict when ready. |
 | `openairev_run_tests` | Run project test suite |
 | `openairev_run_lint` | Run linter |
 | `openairev_get_diff` | Get current git diff |
 
+### Async Review Flow
+
+The review runs asynchronously so you can see progress:
+
+1. Call `openairev_review` → returns *"Review started. Call openairev_status."*
+2. Call `openairev_status` → shows live progress (files read, commands run, tokens)
+3. When done, `openairev_status` returns the full verdict and feedback
+
+This lets the executor AI (and user) see what the reviewer is doing instead of waiting blindly.
+
 ## Config
 
 Generated by `openairev init` at `.openairev/config.yaml`:

package/package.json

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

package/src/agents/codex.js

@@ -34,6 +34,7 @@ export class CodexAdapter {
     const summarizer = stream ? createCodexSummarizer({
       reviewerName: stream.reviewerName || 'codex',
       tty: stream.tty !== false,
+      onProgress: stream.onProgress,
     }) : undefined;
     const result = await exec(this.cmd, args, { onData: summarizer });
 

package/src/agents/stream-summarizer.js

@@ -6,7 +6,7 @@ import chalk from 'chalk';
  * tty=false: collects plain-text progress lines silently.
  * Both modes always collect lines for the final summary.
  */
-export function createCodexSummarizer({ reviewerName, tty = true } = {}) {
+export function createCodexSummarizer({ reviewerName, tty = true, onProgress } = {}) {
   const seenFiles = new Set();
   const progressLines = [];
   let buffer = '';
@@ -35,7 +35,10 @@ export function createCodexSummarizer({ reviewerName, tty = true } = {}) {
   summarizer.getProgress = () => progressLines;
 
   function emit(msg, color) {
-    if (progressLines.length < 200) progressLines.push(msg);
+    if (progressLines.length < 200) {
+      progressLines.push(msg);
+      if (onProgress) onProgress(progressLines);
+    }
     if (tty) {
       const colorFn = color === 'cyan' ? chalk.cyan
         : color === 'green' ? chalk.green

package/src/cli/init.js

@@ -237,11 +237,13 @@ function setupClaudeCode(cwd) {
 ${marker}
 ## OpenAIRev — Cross-Model Code Review
 
-This project uses OpenAIRev for independent AI code review. When the user asks to "review", "get a review", or "check my changes", use the \`openairev_review\` MCP tool.
+This project uses OpenAIRev for independent AI code review. When the user asks to "review", "get a review", or "check my changes":
 
-- You are the **executor**. Set \`executor\` to \`"claude_code"\` when calling the tool.
-- A different AI model will review your code independently.
-- **IMPORTANT**: Pass only the diff for files you changed using \`diff_cmd\`, e.g. \`"git diff HEAD -- src/auth.ts src/routes.ts"\`. Do NOT let it auto-detect — the full repo diff may be too large.
+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. Call \`openairev_status\` to check progress. It shows what the reviewer is reading and doing. If the review is still running, call it again after a few seconds.
+3. When the review is complete, \`openairev_status\` returns the full verdict with feedback.
+
+- **IMPORTANT**: 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.
 - The review verdict includes \`critical_issues\`, \`repair_instructions\`, and a \`confidence\` score. Focus on high-confidence critical issues.
 ${marker}
@@ -297,11 +299,13 @@ function setupCodex(cwd) {
 ${marker}
 ## OpenAIRev — Cross-Model Code Review
 
-This project uses OpenAIRev for independent AI code review. When the user asks to "review", "get a review", or "check my changes", use the \`openairev_review\` MCP tool.
+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. Call \`openairev_status\` to check progress. It shows what the reviewer is reading and doing. If the review is still running, call it again after a few seconds.
+3. When the review is complete, \`openairev_status\` returns the full verdict with feedback.
 
-- You are the **executor**. Set \`executor\` to \`"codex"\` when calling the tool.
-- A different AI model will review your code independently.
-- **IMPORTANT**: Pass only the diff for files you changed using \`diff_cmd\`, e.g. \`"git diff HEAD -- src/auth.ts src/routes.ts"\`. Do NOT let it auto-detect — the full repo diff may be too large.
+- **IMPORTANT**: 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.
 - The review verdict includes \`critical_issues\`, \`repair_instructions\`, and a \`confidence\` score. Focus on high-confidence critical issues.
 ${marker}

package/src/mcp/mcp-server.js

@@ -1,5 +1,7 @@
 #!/usr/bin/env node
 
+import { mkdirSync, writeFileSync, readFileSync, existsSync } from 'fs';
+import { join } from 'path';
 import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
 import { z } from 'zod';
@@ -12,6 +14,9 @@ import { VERSION } from '../version.js';
 
 const cwd = process.cwd();
 const config = loadConfig(cwd);
+const PROGRESS_FILE = join(cwd, '.openairev', 'progress.json');
+
+let activeReview = null;
 
 const server = new McpServer({
   name: 'openairev',
@@ -20,7 +25,7 @@ const server = new McpServer({
 
 server.tool(
   'openairev_review',
-  'TRIGGER: Use this tool when the user says "review", "review my code", "get a review", "check my changes", "openairev", or asks for independent/cross-model code review. Sends current code changes to a DIFFERENT AI model for independent review. Returns a structured verdict with critical issues, test gaps, risk level, confidence score, and repair instructions.',
+  'TRIGGER: Use this tool when the user says "review", "review my code", "get a review", "check my changes", "openairev", or asks for independent/cross-model code review. Sends current code changes to a DIFFERENT AI model for independent review. The review runs in the background — call openairev_status to check progress and get the verdict when ready.',
   {
     executor: z.string().optional().describe('Which agent wrote the code (claude_code or codex). If you are Claude Code, set this to "claude_code". If you are Codex, set this to "codex".'),
     diff: z.string().optional().describe('The diff to review. IMPORTANT: Pass only the diff for files YOU changed, not the entire repo. Use `git diff HEAD -- file1 file2` to scope it. If omitted, auto-detects from git which may be too large.'),
@@ -48,46 +53,78 @@ server.tool(
       return { content: [{ type: 'text', text: 'No changes found to review.' }] };
     }
 
-    const review = await runReview(diffContent, {
+    mkdirSync(join(cwd, '.openairev'), { recursive: true });
+    writeProgress({ status: 'running', reviewer: reviewerName, started: new Date().toISOString(), progress: [], verdict: null });
+
+    const onProgress = (lines) => {
+      writeProgress({ status: 'running', reviewer: reviewerName, started: new Date().toISOString(), progress: lines, verdict: null });
+    };
+
+    activeReview = runReview(diffContent, {
       config,
       reviewerName,
       taskDescription: task_description,
       cwd,
-      stream: 'silent',
+      stream: { onProgress },
+    }).then((review) => {
+      const session = createSession({ executor: execAgent, reviewer: reviewerName });
+      session.iterations.push({ round: 1, review, timestamp: new Date().toISOString() });
+      session.final_verdict = review.verdict;
+      session.status = 'completed';
+      saveSession(session, cwd);
+
+      writeProgress({
+        status: 'completed',
+        reviewer: reviewerName,
+        progress: review.progress || [],
+        verdict: review.verdict,
+        executor_feedback: review.executor_feedback,
+      });
+      activeReview = null;
+      return review;
+    }).catch((err) => {
+      writeProgress({ status: 'error', reviewer: reviewerName, error: err.message, progress: [], verdict: null });
+      activeReview = null;
     });
 
-    const session = createSession({ executor: execAgent, reviewer: reviewerName });
-    session.iterations.push({ round: 1, review, timestamp: new Date().toISOString() });
-    session.final_verdict = review.verdict;
-    session.status = 'completed';
-    saveSession(session, cwd);
-
-    const parts = [];
-
-    if (review.progress?.length > 0) {
-      parts.push({ type: 'text', text: `Review progress:\n${review.progress.map(l => `  ${l}`).join('\n')}` });
-    }
-
-    const feedback = review.executor_feedback || JSON.stringify(review.verdict || review, null, 2);
-    parts.push({ type: 'text', text: feedback });
-
-    return { content: parts };
+    return {
+      content: [{
+        type: 'text',
+        text: `Review started. Reviewer: ${reviewerName}\n\nCall openairev_status to check progress and get the verdict when ready.`,
+      }],
+    };
   }
 );
 
 server.tool(
   'openairev_status',
-  'Get the status and verdict of the most recent OpenAIRev review session.',
+  'Check the progress and result of the current or most recent OpenAIRev review. Call this after openairev_review to see what the reviewer is doing and get the verdict when ready.',
   {},
   async () => {
-    const { listSessions } = await import('../session/session-manager.js');
-    const sessions = listSessions(cwd, 1);
-    if (sessions.length === 0) {
-      return { content: [{ type: 'text', text: 'No review sessions found.' }] };
+    const progress = readProgress();
+    if (!progress) {
+      return { content: [{ type: 'text', text: 'No review in progress. Call openairev_review first.' }] };
     }
-    const last = sessions[0];
-    const text = JSON.stringify({ id: last.id, status: last.status, verdict: last.final_verdict, created: last.created }, null, 2);
-    return { content: [{ type: 'text', text }] };
+
+    if (progress.status === 'running') {
+      const lines = progress.progress || [];
+      const text = lines.length > 0
+        ? `Review in progress (reviewer: ${progress.reviewer}):\n${lines.map(l => `  ${l}`).join('\n')}\n\nStill running... Call openairev_status again in a few seconds.`
+        : `Review in progress (reviewer: ${progress.reviewer}). Started: ${progress.started}\n\nCall openairev_status again in a few seconds.`;
+      return { content: [{ type: 'text', text }] };
+    }
+
+    if (progress.status === 'error') {
+      return { content: [{ type: 'text', text: `Review failed: ${progress.error}` }] };
+    }
+
+    const parts = [];
+    if (progress.progress?.length > 0) {
+      parts.push({ type: 'text', text: `Review complete:\n${progress.progress.map(l => `  ${l}`).join('\n')}` });
+    }
+    const feedback = progress.executor_feedback || JSON.stringify(progress.verdict || {}, null, 2);
+    parts.push({ type: 'text', text: feedback });
+    return { content: parts };
   }
 );
 
@@ -125,6 +162,20 @@ server.tool(
   }
 );
 
-// Start server
+function writeProgress(data) {
+  try {
+    writeFileSync(PROGRESS_FILE, JSON.stringify(data, null, 2));
+  } catch { /* non-critical */ }
+}
+
+function readProgress() {
+  try {
+    if (!existsSync(PROGRESS_FILE)) return null;
+    return JSON.parse(readFileSync(PROGRESS_FILE, 'utf-8'));
+  } catch {
+    return null;
+  }
+}
+
 const transport = new StdioServerTransport();
 await server.connect(transport);

package/src/review/review-runner.js

@@ -40,7 +40,7 @@ export async function runReview(content, {
     schemaFile,
     continueSession: !!sessionId,
     sessionName: sessionId ? undefined : `review-${Date.now()}`,
-    stream: stream ? { reviewerName, tty: stream === true } : false,
+    stream: stream ? { reviewerName, tty: stream === true, onProgress: stream.onProgress } : false,
   });
 
   const verdict = extractVerdict(result);