openairev 0.3.13 → 0.3.15

package/package.json

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

package/src/agents/claude-code.js

@@ -130,12 +130,18 @@ export function parseClaudeStreamOutput(stdout, { progress = [], fallbackSession
     };
   }
 
-  return {
+  const returnValue = {
     result: verdict || finalText,
     raw_output: stdout,
     progress,
     session_id: sessionId,
   };
+
+  if (!verdict && !finalText) {
+    returnValue.error = 'Claude produced no output. Check .openairev/logs/ for the raw session output.';
+  }
+
+  return returnValue;
 }
 
 function extractAssistantText(event) {

package/src/agents/codex.js

@@ -42,6 +42,8 @@ export class CodexAdapter {
       const lines = result.stdout.trim().split('\n');
       let agentMessage = null;
       let sessionId = null;
+      let lastTurn = null;
+      let lastError = null;
 
       for (const line of lines) {
         try {
@@ -52,12 +54,13 @@ export class CodexAdapter {
           if (event.type === 'item.completed' && event.item?.type === 'agent_message') {
             agentMessage = event.item.text;
           }
+          if (event.type === 'turn.completed') {
+            lastTurn = event;
+          }
           if (event.type === 'error' || event.type === 'turn.failed') {
-            const errMsg = event.message || event.error?.message || JSON.stringify(event);
-            throw new Error(`Codex error: ${errMsg}`);
+            lastError = event.message || event.error?.message || JSON.stringify(event);
           }
-        } catch (e) {
-          if (e.message?.startsWith('Codex error:')) throw e;
+        } catch {
           // skip non-JSON lines
         }
       }
@@ -76,9 +79,19 @@ export class CodexAdapter {
         }
       }
 
-      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' };
+      // No verdict — report what actually happened
+      const cmdCount = lines.filter(l => { try { return JSON.parse(l).type === 'item.completed' && JSON.parse(l).item?.type === 'command_execution'; } catch { return false; } }).length;
+      let error = lastError
+        ? `Codex error: ${lastError}`
+        : `Codex exited after ${cmdCount} commands without producing a verdict or turn completion event.`;
+      if (lastTurn?.usage) {
+        const { input_tokens, output_tokens } = lastTurn.usage;
+        error += ` Tokens: ${input_tokens} in / ${output_tokens} out.`;
+      }
+
+      return { raw: result.stdout, raw_output: result.stdout, progress, session_id: this.sessionId, error };
+    } catch (e) {
+      return { raw: result.stdout, raw_output: result.stdout, error: `Failed to parse Codex output: ${e.message}` };
     }
   }
 }