cc-reviewer 2.0.0 → 2.1.0

package/dist/schema.js

@@ -154,7 +154,7 @@ export function getReviewOutputJsonSchema() {
     return {
         type: 'object',
         additionalProperties: false,
-        required: ['reviewer', 'findings', 'risk_assessment'],
+        required: ['reviewer', 'findings', 'agreements', 'disagreements', 'alternatives', 'risk_assessment', 'uncertainty_responses', 'question_answers'],
         properties: {
             reviewer: { type: 'string' },
             findings: {

package/dist/tools/feedback.d.ts

@@ -1,9 +1,8 @@
 /**
- * MCP Tool Implementations
+ * MCP Tool Implementations — Review Tools
  *
- * Provides two levels of review:
- * 1. Single model review (codex_review, gemini_review)
- * 2. Multi-model parallel review (multi_review)
+ * Returns raw reviewer text to CC. No JSON parsing, no reformatting.
+ * CC handles interpretation and synthesis.
  */
 import { z } from 'zod';
 export declare const ReviewInputSchema: z.ZodObject<{
@@ -20,8 +19,8 @@ export declare const ReviewInputSchema: z.ZodObject<{
     ccOutput: string;
     outputType: "findings" | "analysis" | "plan" | "proposal";
     focusAreas?: ("performance" | "security" | "testing" | "architecture" | "correctness" | "maintainability" | "scalability" | "documentation")[] | undefined;
-    customPrompt?: string | undefined;
     analyzedFiles?: string[] | undefined;
+    customPrompt?: string | undefined;
     reasoningEffort?: "high" | "xhigh" | undefined;
     serviceTier?: "default" | "fast" | "flex" | undefined;
 }, {
@@ -29,8 +28,8 @@ export declare const ReviewInputSchema: z.ZodObject<{
     ccOutput: string;
     outputType: "findings" | "analysis" | "plan" | "proposal";
     focusAreas?: ("performance" | "security" | "testing" | "architecture" | "correctness" | "maintainability" | "scalability" | "documentation")[] | undefined;
-    customPrompt?: string | undefined;
     analyzedFiles?: string[] | undefined;
+    customPrompt?: string | undefined;
     reasoningEffort?: "high" | "xhigh" | undefined;
     serviceTier?: "default" | "fast" | "flex" | undefined;
 }>;

package/dist/tools/feedback.js

@@ -1,9 +1,8 @@
 /**
- * MCP Tool Implementations
+ * MCP Tool Implementations — Review Tools
  *
- * Provides two levels of review:
- * 1. Single model review (codex_review, gemini_review)
- * 2. Multi-model parallel review (multi_review)
+ * Returns raw reviewer text to CC. No JSON parsing, no reformatting.
+ * CC handles interpretation and synthesis.
  */
 import { z } from 'zod';
 import { getAdapter, getAvailableAdapters, } from '../adapters/index.js';
@@ -24,7 +23,7 @@ export const ReviewInputSchema = z.object({
     serviceTier: z.enum(['default', 'fast', 'flex']).optional().describe('Codex service tier (default: default, fast = priority processing, flex = cheaper/slower)')
 });
 // =============================================================================
-// HELPER FUNCTIONS
+// HELPERS
 // =============================================================================
 function toReviewRequest(input) {
     return {
@@ -38,260 +37,70 @@ function toReviewRequest(input) {
         serviceTier: input.serviceTier,
     };
 }
-function formatSingleReviewResponse(result, modelName) {
+function formatResult(result, modelName) {
     if (!result.success) {
-        return formatErrorResponse(result.error, result.suggestion);
-    }
-    const output = result.output;
-    const lines = [];
-    lines.push(`## ${modelName} Review\n`);
-    lines.push(`**Execution Time:** ${(result.executionTimeMs / 1000).toFixed(1)}s\n`);
-    // Risk Assessment
-    const riskEmoji = {
-        critical: '🔴',
-        high: '🟠',
-        medium: '🟡',
-        low: '🟢',
-        minimal: '✅',
-    };
-    lines.push(`### Risk Assessment ${riskEmoji[output.risk_assessment.overall_level]}`);
-    lines.push(`**${output.risk_assessment.overall_level.toUpperCase()}** (Score: ${output.risk_assessment.score}/100)`);
-    lines.push(`${output.risk_assessment.summary}\n`);
-    // Findings
-    if (output.findings.length > 0) {
-        lines.push(`### New Findings (${output.findings.length})\n`);
-        for (const finding of output.findings) {
-            const severityEmoji = {
-                critical: '🔴', high: '🟠', medium: '🟡', low: '🟢', info: 'ℹ️'
-            };
-            const confidence = Math.round(finding.confidence * 100);
-            lines.push(`${severityEmoji[finding.severity]} **${finding.title}** [${confidence}% confidence]`);
-            if (finding.location) {
-                const loc = finding.location.line_start
-                    ? `${finding.location.file}:${finding.location.line_start}`
-                    : finding.location.file;
-                lines.push(`  📍 ${loc}`);
-            }
-            lines.push(`  ${finding.description}`);
-            if (finding.suggestion) {
-                lines.push(`  💡 ${finding.suggestion}`);
-            }
-            lines.push('');
-        }
-    }
-    // Uncertainty Responses
-    if (output.uncertainty_responses && output.uncertainty_responses.length > 0) {
-        lines.push(`### Uncertainty Responses (${output.uncertainty_responses.length})\n`);
-        for (const ur of output.uncertainty_responses) {
-            const icon = ur.verified ? '✓' : '✗';
-            lines.push(`${icon} **Uncertainty #${ur.uncertainty_index}**: ${ur.finding}`);
-            if (ur.recommendation) {
-                lines.push(`  → ${ur.recommendation}`);
-            }
-            lines.push('');
-        }
-    }
-    // Question Answers
-    if (output.question_answers && output.question_answers.length > 0) {
-        lines.push(`### Question Answers (${output.question_answers.length})\n`);
-        for (const qa of output.question_answers) {
-            const confidence = qa.confidence != null ? ` [${Math.round(qa.confidence * 100)}%]` : '';
-            lines.push(`**Q${qa.question_index}**${confidence}: ${qa.answer}`);
-            lines.push('');
-        }
-    }
-    // Agreements
-    if (output.agreements.length > 0) {
-        lines.push(`### Agreements (${output.agreements.length})\n`);
-        for (const agreement of output.agreements) {
-            const confidence = Math.round(agreement.confidence * 100);
-            lines.push(`✓ **${agreement.original_claim}** [${confidence}%]`);
-            if (agreement.supporting_evidence) {
-                lines.push(`  ${agreement.supporting_evidence}`);
-            }
-            lines.push('');
-        }
-    }
-    // Disagreements
-    if (output.disagreements.length > 0) {
-        lines.push(`### Disagreements (${output.disagreements.length})\n`);
-        for (const disagreement of output.disagreements) {
-            const confidence = Math.round(disagreement.confidence * 100);
-            lines.push(`✗ **${disagreement.original_claim}** [${disagreement.issue}] [${confidence}%]`);
-            lines.push(`  ${disagreement.reason}`);
-            if (disagreement.correction) {
-                lines.push(`  → Correction: ${disagreement.correction}`);
-            }
-            lines.push('');
-        }
-    }
-    // Alternatives
-    if (output.alternatives.length > 0) {
-        lines.push(`### Alternatives (${output.alternatives.length})\n`);
-        for (const alt of output.alternatives) {
-            lines.push(`**${alt.topic}**`);
-            lines.push(`  Current: ${alt.current_approach}`);
-            lines.push(`  Alternative: ${alt.alternative}`);
-            lines.push(`  Recommendation: ${alt.recommendation}`);
-            lines.push('');
-        }
-    }
-    return lines.join('\n');
-}
-function formatErrorResponse(error, suggestion) {
-    const emoji = {
-        cli_not_found: '❌',
-        timeout: '⏱️',
-        rate_limit: '🚫',
-        auth_error: '🔐',
-        parse_error: '⚠️',
-        cli_error: '❌',
-    };
-    let response = `${emoji[error.type] || '❌'} **${error.type}**: ${error.message}`;
-    if (suggestion) {
-        response += `\n\n💡 ${suggestion}`;
+        const emoji = {
+            cli_not_found: '❌', timeout: '⏱️', rate_limit: '🚫',
+            auth_error: '🔐', cli_error: '❌',
+        };
+        let msg = `${emoji[result.error.type] || '❌'} **${result.error.type}**: ${result.error.message}`;
+        if (result.suggestion)
+            msg += `\n\n💡 ${result.suggestion}`;
+        return msg;
     }
-    return response;
+    return `## ${modelName} Review\n\n**Execution Time:** ${(result.executionTimeMs / 1000).toFixed(1)}s\n\n${result.output}`;
 }
 // =============================================================================
 // SINGLE MODEL HANDLERS
 // =============================================================================
 export async function handleCodexReview(input) {
     const adapter = getAdapter('codex');
-    if (!adapter) {
-        return {
-            content: [{
-                    type: 'text',
-                    text: '❌ Codex adapter not registered'
-                }]
-        };
-    }
+    if (!adapter)
+        return { content: [{ type: 'text', text: '❌ Codex adapter not registered' }] };
     const available = await adapter.isAvailable();
-    if (!available) {
-        return {
-            content: [{
-                    type: 'text',
-                    text: '❌ Codex CLI not found.\n\nInstall with: npm install -g @openai/codex\n\nAlternative: Use gemini_review instead'
-                }]
-        };
-    }
-    const request = toReviewRequest(input);
-    const result = await adapter.runReview(request);
-    return {
-        content: [{
-                type: 'text',
-                text: formatSingleReviewResponse(result, 'Codex')
-            }]
-    };
+    if (!available)
+        return { content: [{ type: 'text', text: '❌ Codex CLI not found.\n\nInstall with: npm install -g @openai/codex\n\nAlternative: Use gemini_review instead' }] };
+    const result = await adapter.runReview(toReviewRequest(input));
+    return { content: [{ type: 'text', text: formatResult(result, 'Codex') }] };
 }
 export async function handleGeminiReview(input) {
     const adapter = getAdapter('gemini');
-    if (!adapter) {
-        return {
-            content: [{
-                    type: 'text',
-                    text: '❌ Gemini adapter not registered'
-                }]
-        };
-    }
+    if (!adapter)
+        return { content: [{ type: 'text', text: '❌ Gemini adapter not registered' }] };
     const available = await adapter.isAvailable();
-    if (!available) {
-        return {
-            content: [{
-                    type: 'text',
-                    text: '❌ Gemini CLI not found.\n\nInstall with: npm install -g @google/gemini-cli\n\nAlternative: Use codex_review instead'
-                }]
-        };
-    }
-    const request = toReviewRequest(input);
-    const result = await adapter.runReview(request);
-    return {
-        content: [{
-                type: 'text',
-                text: formatSingleReviewResponse(result, 'Gemini')
-            }]
-    };
+    if (!available)
+        return { content: [{ type: 'text', text: '❌ Gemini CLI not found.\n\nInstall with: npm install -g @google/gemini-cli\n\nAlternative: Use codex_review instead' }] };
+    const result = await adapter.runReview(toReviewRequest(input));
+    return { content: [{ type: 'text', text: formatResult(result, 'Gemini') }] };
 }
 // =============================================================================
 // MULTI-MODEL HANDLER
 // =============================================================================
 export async function handleMultiReview(input) {
     const request = toReviewRequest(input);
-    // Get all available adapters
     const availableAdapters = await getAvailableAdapters();
     if (availableAdapters.length === 0) {
-        return {
-            content: [{
-                    type: 'text',
-                    text: `❌ No AI CLIs found.
-
-Install at least one:
-  - Codex: npm install -g @openai/codex
-  - Gemini: npm install -g @google/gemini-cli`
-                }]
-        };
+        return { content: [{ type: 'text', text: '❌ No AI CLIs found.\n\nInstall at least one:\n  - Codex: npm install -g @openai/codex\n  - Gemini: npm install -g @google/gemini-cli' }] };
     }
-    // Run all available adapters in parallel
-    const promises = availableAdapters.map(async (adapter) => {
-        const adapterRequest = { ...request };
-        const result = await adapter.runReview(adapterRequest);
+    const results = await Promise.all(availableAdapters.map(async (adapter) => {
+        const result = await adapter.runReview({ ...request });
         return { adapter, result };
-    });
-    const results = await Promise.all(promises);
-    // Collect successful and failed results
-    const successful = [];
-    const failed = [];
-    for (const { adapter, result } of results) {
-        if (result.success) {
-            successful.push({ model: adapter.id, output: result.output });
-        }
-        else {
-            failed.push({ model: adapter.id, error: result.error.message });
-        }
-    }
-    // Build response
+    }));
     const lines = [];
-    // Header
-    if (failed.length === results.length) {
+    const allFailed = results.every(r => !r.result.success);
+    const someFailed = results.some(r => !r.result.success);
+    if (allFailed)
         lines.push('## Multi-Model Review ❌ All Failed\n');
-    }
-    else if (failed.length > 0) {
+    else if (someFailed)
         lines.push('## Multi-Model Review ⚠️ Partial Success\n');
-    }
-    else {
+    else
         lines.push('## Multi-Model Review ✓\n');
-    }
-    lines.push(`**Models:** ${availableAdapters.map(a => a.id).join(', ')}`);
-    lines.push('');
-    // Successful reviews
-    for (const { model, output } of successful) {
-        lines.push(`### ${model.charAt(0).toUpperCase() + model.slice(1)} Review\n`);
-        lines.push(formatSingleReviewResponse({ success: true, output, executionTimeMs: 0 }, model));
-        lines.push('');
-    }
-    // Failed reviews
-    if (failed.length > 0) {
-        lines.push('### Failures\n');
-        for (const { model, error } of failed) {
-            lines.push(`**${model}:** ${error}`);
-        }
+    lines.push(`**Models:** ${availableAdapters.map(a => a.id).join(', ')}\n`);
+    for (const { adapter, result } of results) {
+        lines.push(formatResult(result, adapter.getCapabilities().name));
         lines.push('');
     }
-    // Synthesis instructions (only if multiple successful)
-    if (successful.length > 1) {
-        lines.push(`---
-
-**Synthesis Instructions:**
-- ✓✓ Mark agreements where both models concur
-- Resolve conflicts with your own judgment
-- Note unique insights from each model`);
-    }
-    return {
-        content: [{
-                type: 'text',
-                text: lines.join('\n')
-            }]
-    };
+    return { content: [{ type: 'text', text: lines.join('\n') }] };
 }
 // =============================================================================
 // TOOL DEFINITIONS
@@ -303,46 +112,14 @@ export const TOOL_DEFINITIONS = {
         inputSchema: {
             type: 'object',
             properties: {
-                workingDir: {
-                    type: 'string',
-                    description: 'Working directory for the CLI to operate in'
-                },
-                ccOutput: {
-                    type: 'string',
-                    description: "Claude Code's output to review (findings, plan, analysis)"
-                },
-                outputType: {
-                    type: 'string',
-                    enum: ['plan', 'findings', 'analysis', 'proposal'],
-                    description: 'Type of output being reviewed'
-                },
-                analyzedFiles: {
-                    type: 'array',
-                    items: { type: 'string' },
-                    description: 'File paths that CC analyzed'
-                },
-                focusAreas: {
-                    type: 'array',
-                    items: {
-                        type: 'string',
-                        enum: ['security', 'performance', 'architecture', 'correctness', 'maintainability', 'scalability', 'testing', 'documentation']
-                    },
-                    description: 'Areas to focus the review on'
-                },
-                customPrompt: {
-                    type: 'string',
-                    description: 'Custom instructions for the reviewer'
-                },
-                reasoningEffort: {
-                    type: 'string',
-                    enum: ['high', 'xhigh'],
-                    description: 'Codex reasoning effort (default: high, use xhigh for deeper analysis)'
-                },
-                serviceTier: {
-                    type: 'string',
-                    enum: ['default', 'fast', 'flex'],
-                    description: 'Codex service tier (fast = priority processing, flex = cheaper/slower)'
-                }
+                workingDir: { type: 'string', description: 'Working directory for the CLI to operate in' },
+                ccOutput: { type: 'string', description: "Claude Code's output to review (findings, plan, analysis)" },
+                outputType: { type: 'string', enum: ['plan', 'findings', 'analysis', 'proposal'], description: 'Type of output being reviewed' },
+                analyzedFiles: { type: 'array', items: { type: 'string' }, description: 'File paths that CC analyzed' },
+                focusAreas: { type: 'array', items: { type: 'string', enum: ['security', 'performance', 'architecture', 'correctness', 'maintainability', 'scalability', 'testing', 'documentation'] }, description: 'Areas to focus the review on' },
+                customPrompt: { type: 'string', description: 'Custom instructions for the reviewer' },
+                reasoningEffort: { type: 'string', enum: ['high', 'xhigh'], description: 'Codex reasoning effort (default: high, use xhigh for deeper analysis)' },
+                serviceTier: { type: 'string', enum: ['default', 'fast', 'flex'], description: 'Codex service tier (fast = priority processing, flex = cheaper/slower)' }
             },
             required: ['workingDir', 'ccOutput', 'outputType']
         }
@@ -353,36 +130,12 @@ export const TOOL_DEFINITIONS = {
         inputSchema: {
             type: 'object',
             properties: {
-                workingDir: {
-                    type: 'string',
-                    description: 'Working directory for the CLI to operate in'
-                },
-                ccOutput: {
-                    type: 'string',
-                    description: "Claude Code's output to review (findings, plan, analysis)"
-                },
-                outputType: {
-                    type: 'string',
-                    enum: ['plan', 'findings', 'analysis', 'proposal'],
-                    description: 'Type of output being reviewed'
-                },
-                analyzedFiles: {
-                    type: 'array',
-                    items: { type: 'string' },
-                    description: 'File paths that CC analyzed'
-                },
-                focusAreas: {
-                    type: 'array',
-                    items: {
-                        type: 'string',
-                        enum: ['security', 'performance', 'architecture', 'correctness', 'maintainability', 'scalability', 'testing', 'documentation']
-                    },
-                    description: 'Areas to focus the review on'
-                },
-                customPrompt: {
-                    type: 'string',
-                    description: 'Custom instructions for the reviewer'
-                }
+                workingDir: { type: 'string', description: 'Working directory for the CLI to operate in' },
+                ccOutput: { type: 'string', description: "Claude Code's output to review (findings, plan, analysis)" },
+                outputType: { type: 'string', enum: ['plan', 'findings', 'analysis', 'proposal'], description: 'Type of output being reviewed' },
+                analyzedFiles: { type: 'array', items: { type: 'string' }, description: 'File paths that CC analyzed' },
+                focusAreas: { type: 'array', items: { type: 'string', enum: ['security', 'performance', 'architecture', 'correctness', 'maintainability', 'scalability', 'testing', 'documentation'] }, description: 'Areas to focus the review on' },
+                customPrompt: { type: 'string', description: 'Custom instructions for the reviewer' },
             },
             required: ['workingDir', 'ccOutput', 'outputType']
         }
@@ -393,41 +146,13 @@ export const TOOL_DEFINITIONS = {
         inputSchema: {
             type: 'object',
             properties: {
-                workingDir: {
-                    type: 'string',
-                    description: 'Working directory for the CLI to operate in'
-                },
-                ccOutput: {
-                    type: 'string',
-                    description: "Claude Code's output to review (findings, plan, analysis)"
-                },
-                outputType: {
-                    type: 'string',
-                    enum: ['plan', 'findings', 'analysis', 'proposal'],
-                    description: 'Type of output being reviewed'
-                },
-                analyzedFiles: {
-                    type: 'array',
-                    items: { type: 'string' },
-                    description: 'File paths that CC analyzed'
-                },
-                focusAreas: {
-                    type: 'array',
-                    items: {
-                        type: 'string',
-                        enum: ['security', 'performance', 'architecture', 'correctness', 'maintainability', 'scalability', 'testing', 'documentation']
-                    },
-                    description: 'Areas to focus the review on'
-                },
-                customPrompt: {
-                    type: 'string',
-                    description: 'Custom instructions for the reviewer'
-                },
-                serviceTier: {
-                    type: 'string',
-                    enum: ['default', 'fast', 'flex'],
-                    description: 'Codex service tier (fast = priority processing, flex = cheaper/slower). Only applies to Codex.'
-                }
+                workingDir: { type: 'string', description: 'Working directory for the CLI to operate in' },
+                ccOutput: { type: 'string', description: "Claude Code's output to review (findings, plan, analysis)" },
+                outputType: { type: 'string', enum: ['plan', 'findings', 'analysis', 'proposal'], description: 'Type of output being reviewed' },
+                analyzedFiles: { type: 'array', items: { type: 'string' }, description: 'File paths that CC analyzed' },
+                focusAreas: { type: 'array', items: { type: 'string', enum: ['security', 'performance', 'architecture', 'correctness', 'maintainability', 'scalability', 'testing', 'documentation'] }, description: 'Areas to focus the review on' },
+                customPrompt: { type: 'string', description: 'Custom instructions for the reviewer' },
+                serviceTier: { type: 'string', enum: ['default', 'fast', 'flex'], description: 'Codex service tier (fast = priority processing, flex = cheaper/slower). Only applies to Codex.' }
             },
             required: ['workingDir', 'ccOutput', 'outputType']
         }

package/dist/tools/peer.d.ts

@@ -6,7 +6,6 @@
  * 2. ask_gemini - Ask Gemini for help
  * 3. ask_multi - Ask both in parallel
  */
-import { PeerResult } from '../adapters/index.js';
 export type PeerInput = {
     workingDir: string;
     prompt: string;
@@ -18,7 +17,6 @@ export type PeerInput = {
     reasoningEffort?: 'high' | 'xhigh';
     serviceTier?: 'default' | 'fast' | 'flex';
 };
-export declare function formatPeerResponse(result: PeerResult, modelName: string): string;
 export declare function handleAskCodex(input: PeerInput): Promise<{
     content: Array<{
         type: 'text';