openairev 0.3.0 → 0.3.1

package/bin/openairev.js

@@ -1,6 +1,7 @@
 #!/usr/bin/env node
 
 import { Command } from 'commander';
+import { VERSION } from '../src/version.js';
 import { initCommand } from '../src/cli/init.js';
 import { reviewCommand } from '../src/cli/review.js';
 import { resumeCommand } from '../src/cli/resume.js';
@@ -12,7 +13,7 @@ const program = new Command();
 program
   .name('openairev')
   .description('OpenAIRev — cross-model AI code reviewer')
-  .version('0.3.0');
+  .version(VERSION);
 
 program
   .command('init')

package/package.json

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

package/src/agents/codex.js

@@ -31,8 +31,11 @@ export class CodexAdapter {
       args.push('--output-schema', schemaPath);
     }
 
-    const onData = stream ? createCodexSummarizer({ reviewerName: typeof stream === 'string' ? stream : 'codex' }) : undefined;
-    const result = await exec(this.cmd, args, { onData });
+    const summarizer = stream ? createCodexSummarizer({
+      reviewerName: stream.reviewerName || 'codex',
+      tty: stream.tty !== false,
+    }) : undefined;
+    const result = await exec(this.cmd, args, { onData: summarizer });
 
     try {
       const lines = result.stdout.trim().split('\n');
@@ -62,15 +65,17 @@ export class CodexAdapter {
         this.sessionId = sessionId;
       }
 
+      const progress = summarizer?.getProgress() || [];
+
       if (agentMessage) {
         try {
-          return { result: JSON.parse(agentMessage), raw_output: result.stdout, session_id: this.sessionId };
+          return { result: JSON.parse(agentMessage), raw_output: result.stdout, progress, session_id: this.sessionId };
         } catch {
-          return { result: agentMessage, raw_output: result.stdout, session_id: this.sessionId };
+          return { result: agentMessage, raw_output: result.stdout, progress, session_id: this.sessionId };
         }
       }
 
-      return { raw: result.stdout, raw_output: result.stdout, session_id: this.sessionId };
+      return { raw: result.stdout, raw_output: result.stdout, progress, session_id: this.sessionId };
     } catch {
       return { raw: result.stdout, raw_output: result.stdout, error: 'Failed to parse output' };
     }

package/src/agents/stream-summarizer.js

@@ -2,51 +2,70 @@ import chalk from 'chalk';
 
 /**
  * Creates a summarizer callback for Codex NDJSON event streams.
- * Prints concise progress lines to stderr instead of raw JSON.
+ * tty=true: prints colored progress to stderr.
+ * tty=false: collects plain-text progress lines silently.
+ * Both modes always collect lines for the final summary.
  */
-export function createCodexSummarizer({ reviewerName } = {}) {
+export function createCodexSummarizer({ reviewerName, tty = true } = {}) {
   const seenFiles = new Set();
+  const progressLines = [];
   let buffer = '';
   let started = false;
 
-  return (chunk) => {
+  const summarizer = (chunk) => {
     if (!started) {
       started = true;
-      if (reviewerName) log(chalk.cyan(`  reviewer: ${reviewerName}`));
+      if (reviewerName) emit(`reviewer: ${reviewerName}`, 'cyan');
     }
     buffer += chunk;
     const lines = buffer.split('\n');
-    buffer = lines.pop(); // keep incomplete last line
+    buffer = lines.pop();
 
     for (const line of lines) {
       if (!line.trim()) continue;
       try {
         const event = JSON.parse(line);
-        summarizeCodexEvent(event, seenFiles);
+        summarizeCodexEvent(event, seenFiles, { emit });
       } catch {
         // skip non-JSON
       }
     }
   };
+
+  summarizer.getProgress = () => progressLines;
+
+  function emit(msg, color) {
+    if (progressLines.length < 200) progressLines.push(msg);
+    if (tty) {
+      const colorFn = color === 'cyan' ? chalk.cyan
+        : color === 'green' ? chalk.green
+        : color === 'yellow' ? chalk.yellow
+        : color === 'red' ? chalk.red
+        : chalk.dim;
+      process.stderr.write(`  ${colorFn(msg)}\n`);
+    }
+  }
+
+  return summarizer;
 }
 
-function summarizeCodexEvent(event, seenFiles) {
+function summarizeCodexEvent(event, seenFiles, { emit }) {
   const type = event.type;
 
   if (type === 'thread.started') {
-    log(chalk.dim(`  session: ${event.thread_id}`));
+    emit(`session: ${event.thread_id}`);
   }
 
   if (type === 'item.started' && event.item?.type === 'todo_list') {
     const items = event.item.items?.map(i => i.text) || [];
-    log(chalk.cyan('  plan:'));
-    for (const item of items) log(chalk.dim(`    • ${item}`));
+    emit('plan:', 'cyan');
+    for (const item of items) emit(`  • ${item}`);
   }
 
   if (type === 'item.completed' && event.item?.type === 'todo_list') {
     const items = event.item.items || [];
     const done = items.filter(i => i.completed).length;
-    log(chalk.dim(`  progress: ${done}/${items.length} tasks done`));
+    emit(`progress: ${done}/${items.length} tasks done`);
   }
 
   if (type === 'item.started' && event.item?.type === 'command_execution') {
@@ -55,10 +74,10 @@ function summarizeCodexEvent(event, seenFiles) {
     const file = extractFileFromCmd(cmd);
     if (file && !seenFiles.has(file)) {
       seenFiles.add(file);
-      log(chalk.dim(`  reading: ${file}`));
+      emit(`reading: ${file}`);
     } else if (!file) {
       const short = summarizeCmd(cmd);
-      if (short) log(chalk.dim(`  running: ${short}`));
+      if (short) emit(`running: ${short}`);
     }
   }
 
@@ -66,27 +85,27 @@ function summarizeCodexEvent(event, seenFiles) {
     const exit = event.item.exit_code;
     if (exit !== null && exit !== 0) {
       const cmd = summarizeCmd(event.item.command || '');
-      log(chalk.yellow(`  command failed (exit ${exit}): ${cmd}`));
+      emit(`command failed (exit ${exit}): ${cmd}`, 'yellow');
     }
   }
 
   if (type === 'item.started' && event.item?.type === 'agent_message') {
-    log(chalk.cyan('  generating verdict...'));
+    emit('generating verdict...', 'cyan');
   }
 
   if (type === 'item.completed' && event.item?.type === 'agent_message') {
-    log(chalk.green('  verdict ready'));
+    emit('verdict ready', 'green');
   }
 
   if (type === 'turn.completed' && event.usage) {
     const { input_tokens, output_tokens } = event.usage;
     const total = input_tokens + output_tokens;
-    log(chalk.dim(`  tokens: ${fmt(total)} total (${fmt(input_tokens)} in / ${fmt(output_tokens)} out)`));
+    emit(`tokens: ${fmt(total)} total (${fmt(input_tokens)} in / ${fmt(output_tokens)} out)`);
   }
 
   if (type === 'error' || type === 'turn.failed') {
     const msg = event.message || event.error?.message || 'unknown error';
-    log(chalk.red(`  error: ${msg}`));
+    emit(`error: ${msg}`, 'red');
   }
 }
 
@@ -118,7 +137,3 @@ function fmt(n) {
   if (n >= 1_000) return (n / 1_000).toFixed(1) + 'k';
   return String(n);
 }
-
-function log(msg) {
-  process.stderr.write(msg + '\n');
-}

package/src/mcp/mcp-server.js

@@ -8,13 +8,14 @@ import { getDiff } from '../tools/git-tools.js';
 import { runToolGates } from '../tools/tool-runner.js';
 import { runReview } from '../review/review-runner.js';
 import { createSession, saveSession } from '../session/session-manager.js';
+import { VERSION } from '../version.js';
 
 const cwd = process.cwd();
 const config = loadConfig(cwd);
 
 const server = new McpServer({
   name: 'openairev',
-  version: '0.3.0',
+  version: VERSION,
 });
 
 server.tool(
@@ -52,6 +53,7 @@ server.tool(
       reviewerName,
       taskDescription: task_description,
       cwd,
+      stream: 'silent',
     });
 
     const session = createSession({ executor: execAgent, reviewer: reviewerName });
@@ -60,8 +62,16 @@ server.tool(
     session.status = 'completed';
     saveSession(session, cwd);
 
-    const text = review.executor_feedback || JSON.stringify(review.verdict || review, null, 2);
-    return { content: [{ type: 'text', text }] };
+    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 };
   }
 );
 

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 : false,
+    stream: stream ? { reviewerName, tty: stream === true } : false,
   });
 
   const verdict = extractVerdict(result);
@@ -54,6 +54,7 @@ export async function runReview(content, {
     verdict,
     executor_feedback: executorFeedback,
     reviewer_output: rawOutput,
+    progress: result?.progress || [],
     session_id: adapter.sessionName || adapter.sessionId,
   };
 }

package/src/version.js

@@ -0,0 +1,8 @@
+import { readFileSync } from 'fs';
+import { join, dirname } from 'path';
+import { fileURLToPath } from 'url';
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const pkg = JSON.parse(readFileSync(join(__dirname, '..', 'package.json'), 'utf-8'));
+
+export const VERSION = pkg.version;