cc-reviewer 1.8.4 → 1.8.5

package/dist/adapters/codex.js

@@ -76,10 +76,11 @@ export class CodexAdapter {
             // Select role based on focus areas
             const role = selectRole(request.focusAreas);
             // Build prompt with retry context if needed
+            // Use 'schema-enforced' since Codex gets --output-schema flag (avoids redundant inline JSON template)
             let prompt = buildHandoffPrompt({
                 handoff,
                 role,
-                outputFormat: 'json',
+                outputFormat: 'schema-enforced',
             });
             // Add retry context if this is a retry attempt
             if (attempt > 0) {
@@ -139,9 +140,14 @@ export class CodexAdapter {
                 };
             }
             // Check for empty/minimal data on any parse path
+            // A valid review may have findings, agreements, disagreements, alternatives,
+            // or a non-default risk assessment. Only retry if truly empty across all fields.
             const hasMinimalData = output.findings.length === 0 &&
                 output.agreements.length === 0 &&
-                output.disagreements.length === 0;
+                output.disagreements.length === 0 &&
+                output.alternatives.length === 0 &&
+                output.risk_assessment.overall_level === 'medium' &&
+                output.risk_assessment.score === 50;
             if (hasMinimalData) {
                 if (attempt < MAX_RETRIES) {
                     console.error(`[codex] Received empty output, retrying...`);
@@ -231,6 +237,7 @@ export class CodexAdapter {
     }
     async runPeerWithRetry(request, attempt, startTime, previousError, previousOutput) {
         try {
+            // Use 'schema-enforced' since Codex gets --output-schema flag (avoids redundant inline JSON template)
             let prompt = buildPeerPrompt({
                 workingDir: request.workingDir,
                 prompt: request.prompt,
@@ -239,7 +246,7 @@ export class CodexAdapter {
                 context: request.context,
                 focusAreas: request.focusAreas,
                 customInstructions: request.customPrompt,
-                outputFormat: 'json',
+                outputFormat: 'schema-enforced',
             });
             if (attempt > 0) {
                 prompt += `\n\n---\n\n# RETRY ATTEMPT ${attempt + 1}\n\n` +
@@ -320,9 +327,8 @@ export class CodexAdapter {
                 schemaFile = null;
             }
             const args = [
-                '--search',
                 'exec',
-                '-m', 'gpt-5.3-codex',
+                '-m', 'gpt-5.4',
                 '-c', `model_reasoning_effort=${reasoningEffort}`,
                 '-c', 'model_reasoning_summary_format=experimental',
                 '--dangerously-bypass-approvals-and-sandbox',

package/dist/adapters/gemini.js

@@ -137,9 +137,14 @@ export class GeminiAdapter {
                 };
             }
             // If output has no substantive data, retry or fail
+            // A valid review may have findings, agreements, disagreements, alternatives,
+            // or a non-default risk assessment. Only retry if truly empty across all fields.
             const hasMinimalData = output.findings.length === 0 &&
                 output.agreements.length === 0 &&
-                output.disagreements.length === 0;
+                output.disagreements.length === 0 &&
+                output.alternatives.length === 0 &&
+                output.risk_assessment.overall_level === 'medium' &&
+                output.risk_assessment.score === 50;
             if (hasMinimalData) {
                 if (attempt < MAX_RETRIES) {
                     console.error(`[gemini] Received empty output, retrying...`);

package/dist/handoff.d.ts

@@ -215,7 +215,7 @@ export declare function selectRole(focusAreas?: FocusArea[]): ReviewerRole;
 export interface PromptOptions {
     handoff: Handoff;
     role?: ReviewerRole;
-    outputFormat: 'json' | 'markdown';
+    outputFormat: 'json' | 'markdown' | 'schema-enforced';
 }
 /**
  * Build the review prompt using minimal, targeted context
@@ -238,7 +238,7 @@ export interface PeerPromptOptions {
     context?: string;
     focusAreas?: FocusArea[];
     customInstructions?: string;
-    outputFormat: 'json';
+    outputFormat: 'json' | 'schema-enforced';
 }
 /**
  * Build a prompt for general-purpose peer assistance (not review).

package/dist/handoff.js

@@ -372,7 +372,23 @@ ${handoff.priorityFiles.map(f => `- \`${f}\``).join('\n')}`);
     // ==========================================================================
     // SECTION 7: OUTPUT FORMAT
     // ==========================================================================
-    if (outputFormat === 'json') {
+    if (outputFormat === 'schema-enforced') {
+        // Schema is enforced externally (e.g. via --output-schema flag).
+        // Only include behavioral rules, skip the redundant JSON template.
+        sections.push(`
+---
+
+# OUTPUT FORMAT
+
+Respond with valid JSON matching the provided output schema.
+
+**Rules:**
+- Use \`git diff\` and file reading to verify before claiming issues
+- Include evidence (code snippets) for findings
+- Confidence reflects how sure YOU are (not CC)
+- Answer CC's uncertainties and questions explicitly`);
+    }
+    else if (outputFormat === 'json') {
         sections.push(`
 ---
 
@@ -481,7 +497,7 @@ export function enhanceHandoff(handoff, uncertainties, questions, decisions) {
  * The peer acts as a collaborative coworker, not a critic.
  */
 export function buildPeerPrompt(options) {
-    const { workingDir, prompt, taskType, relevantFiles, context, focusAreas, customInstructions } = options;
+    const { workingDir, prompt, taskType, relevantFiles, context, focusAreas, customInstructions, outputFormat } = options;
     // Select role based on focus areas (reuse existing role selection)
     const role = selectRole(focusAreas);
     const sections = [];
@@ -549,7 +565,25 @@ ${customInstructions}`);
 5. Reference specific files and line numbers
 6. Suggest concrete next steps`);
     // SECTION 7: OUTPUT FORMAT
-    sections.push(`
+    if (outputFormat === 'schema-enforced') {
+        // Schema is enforced externally (e.g. via --output-schema flag).
+        // Only include behavioral rules, skip the redundant JSON template.
+        sections.push(`
+---
+
+# OUTPUT FORMAT
+
+Respond with valid JSON matching the provided output schema.
+
+**Rules:**
+- Read files before making claims
+- Reference specific file paths and line numbers
+- Be concrete and actionable — no vague suggestions
+- Confidence reflects how sure YOU are about your answer
+- Include alternatives when there are meaningful tradeoffs`);
+    }
+    else {
+        sections.push(`
 ---
 
 # OUTPUT FORMAT
@@ -596,5 +630,6 @@ Respond with valid JSON:
 - Be concrete and actionable — no vague suggestions
 - Confidence reflects how sure YOU are about your answer
 - Include alternatives when there are meaningful tradeoffs`);
+    }
     return sections.join('\n');
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cc-reviewer",
-  "version": "1.8.4",
+  "version": "1.8.5",
   "description": "MCP server for Claude Code - Get second-opinion feedback from Codex/Gemini CLIs",
   "type": "module",
   "main": "dist/index.js",