openairev 0.3.4 → 0.3.5

package/README.md

@@ -213,7 +213,7 @@ Restart your agent CLI after adding.
 | Tool | Description |
 |------|-------------|
 | `openairev_review` | Start a review in the background. Returns immediately. |
-| `openairev_status` | Check review progress and get the verdict when ready. |
+| `openairev_status` | Check review progress (prefer `openairev wait` instead). |
 | `openairev_run_tests` | Run project test suite |
 | `openairev_run_lint` | Run linter |
 | `openairev_get_diff` | Get current git diff |
@@ -223,10 +223,9 @@ Restart your agent CLI after adding.
 The review runs asynchronously so you can see progress:
 
 1. Call `openairev_review` → starts review in background, returns immediately
-2. Read `.openairev/progress.json` → shows live progress (files read, commands run, tokens)
-3. When `status` is `"completed"`, the verdict and feedback are in the same file
+2. Run `openairev wait` via Bash → streams progress (files read, commands run, tokens) and outputs the verdict when done
 
-The executor AI can read the progress file directly (no MCP round-trip needed), or use `openairev_status` as an alternative. The AI can also launch the review in a sub-agent and continue other work while it runs.
+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.
 
 ## Config
 

package/bin/openairev.js

@@ -7,6 +7,7 @@ import { reviewCommand } from '../src/cli/review.js';
 import { resumeCommand } from '../src/cli/resume.js';
 import { statusCommand } from '../src/cli/status.js';
 import { historyCommand } from '../src/cli/history.js';
+import { waitCommand } from '../src/cli/wait.js';
 
 const program = new Command();
 
@@ -54,4 +55,9 @@ program
   .option('--chains', 'Show chains instead of sessions')
   .action(historyCommand);
 
+program
+  .command('wait')
+  .description('Wait for a background review to finish, streaming progress')
+  .action(waitCommand);
+
 program.parse();

package/package.json

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

package/src/cli/init.js

@@ -258,8 +258,7 @@ ${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. To check progress, read \`.openairev/progress.json\` — it updates in real-time showing what the reviewer is reading and doing.
-3. When \`progress.json\` shows \`"status": "completed"\`, the verdict and feedback are in the same file.
+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.
@@ -313,8 +312,7 @@ ${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. To check progress, read \`.openairev/progress.json\` — it updates in real-time showing what the reviewer is reading and doing.
-3. When \`progress.json\` shows \`"status": "completed"\`, the verdict and feedback are in the same file.
+2. Run \`openairev wait\` 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.

package/src/cli/wait.js

@@ -0,0 +1,68 @@
+import { readFileSync, existsSync, watchFile, unwatchFile } from 'fs';
+import { join } from 'path';
+
+export async function waitCommand() {
+  const cwd = process.cwd();
+  const progressFile = join(cwd, '.openairev', 'progress.json');
+
+  if (!existsSync(progressFile)) {
+    console.log('No review in progress. Call openairev_review first.');
+    process.exit(1);
+  }
+
+  // Check if already done
+  const initial = readProgress(progressFile);
+  if (initial?.status === 'completed' || initial?.status === 'error') {
+    printResult(initial);
+    return;
+  }
+
+  // Watch for changes
+  let lastLen = 0;
+  return new Promise((resolve) => {
+    const check = () => {
+      const data = readProgress(progressFile);
+      if (!data) return;
+
+      // Print new progress lines
+      const lines = data.progress || [];
+      if (lines.length > lastLen) {
+        for (let i = lastLen; i < lines.length; i++) {
+          console.log(`  ${lines[i]}`);
+        }
+        lastLen = lines.length;
+      }
+
+      if (data.status === 'completed' || data.status === 'error') {
+        unwatchFile(progressFile);
+        printResult(data);
+        resolve();
+      }
+    };
+
+    watchFile(progressFile, { interval: 1000 }, check);
+    check(); // initial check
+  });
+}
+
+function readProgress(path) {
+  try {
+    return JSON.parse(readFileSync(path, 'utf-8'));
+  } catch {
+    return null;
+  }
+}
+
+function printResult(data) {
+  if (data.status === 'error') {
+    console.log(`\nReview failed: ${data.error}`);
+    process.exit(1);
+  }
+
+  console.log('');
+  if (data.executor_feedback) {
+    console.log(data.executor_feedback);
+  } else if (data.verdict) {
+    console.log(JSON.stringify(data.verdict, null, 2));
+  }
+}

package/src/mcp/mcp-server.js

@@ -25,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. The review runs in the background and returns immediately. To check progress, read .openairev/progress.json — it updates in real-time. When status is "completed", the verdict is in the same file.',
+  '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 starts in the background and returns immediately. After calling this, run `openairev wait` via Bash to stream progress and get the verdict — one blocking call, no polling needed.',
   {
     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.'),
@@ -90,7 +90,7 @@ server.tool(
     return {
       content: [{
         type: 'text',
-        text: `Review started. Reviewer: ${reviewerName}\n\nCall openairev_status to check progress and get the verdict when ready.`,
+        text: `Review started. Reviewer: ${reviewerName}\n\nRun \`openairev wait\` via Bash to stream progress and get the verdict when done.`,
       }],
     };
   }
@@ -98,7 +98,7 @@ server.tool(
 
 server.tool(
   'openairev_status',
-  'Check the progress and result of the current or most recent OpenAIRev review. Alternative to reading .openairev/progress.json directly.',
+  'Check the progress and result of the current or most recent OpenAIRev review. Prefer running `openairev wait` via Bash instead — it streams progress and blocks until done.',
   {},
   async () => {
     const progress = readProgress();
@@ -109,8 +109,8 @@ server.tool(
     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.`;
+        ? `Review in progress (reviewer: ${progress.reviewer}):\n${lines.map(l => `  ${l}`).join('\n')}\n\nStill running. Run \`openairev wait\` via Bash to stream progress until done.`
+        : `Review in progress (reviewer: ${progress.reviewer}). Started: ${progress.started}\n\nRun \`openairev wait\` via Bash to stream progress until done.`;
       return { content: [{ type: 'text', text }] };
     }