cc-reviewer 2.0.0 → 3.0.0

package/dist/handoff.js

@@ -154,10 +154,11 @@ export function selectRole(focusAreas) {
     return CHANGE_FOCUSED_REVIEWER;
 }
 /**
- * Build the review prompt using minimal, targeted context
+ * Build the review prompt using minimal, targeted context.
+ * No output format constraints — reviewer responds naturally, CC interprets.
  */
 export function buildHandoffPrompt(options) {
-    const { handoff, outputFormat } = options;
+    const { handoff } = options;
     const role = options.role || selectRole(handoff.focusAreas);
     const sections = [];
     // SECTION 1: ROLE
@@ -197,38 +198,6 @@ ${handoff.decisions.map((d, i) => `${i + 1}. **${d.decision}**
     if (handoff.priorityFiles && handoff.priorityFiles.length > 0) {
         sections.push(`## PRIORITY FILES\n\n${handoff.priorityFiles.map(f => `- \`${f}\``).join('\n')}`);
     }
-    // SECTION 7: OUTPUT FORMAT
-    if (outputFormat === 'schema-enforced') {
-        sections.push(`## OUTPUT FORMAT
-Respond with valid JSON matching the schema. Confidence reflects YOUR certainty.`);
-    }
-    else if (outputFormat === 'json') {
-        sections.push(`## OUTPUT FORMAT
-Respond with valid JSON:
-\`\`\`json
-{
-  "findings": [{
-    "id": "string",
-    "category": "security|performance|correctness|architecture|other",
-    "severity": "critical|high|medium|low|info",
-    "confidence": 0.0-1.0,
-    "title": "string",
-    "description": "string",
-    "location": { "file": "string", "line_start": 0 },
-    "evidence": "code snippet",
-    "suggestion": "string"
-  }],
-  "uncertainty_responses": [{"uncertainty_index": 0, "verified": true, "finding": "string", "recommendation": "string"}],
-  "question_answers": [{"question_index": 0, "answer": "string", "confidence": 0.0-1.0}],
-  "agreements": ["string"],
-  "risk_assessment": { "level": "critical|high|medium|low|minimal", "summary": "string" }
-}
-\`\`\``);
-    }
-    else {
-        sections.push(`## OUTPUT FORMAT
-Structure: ## Findings, ## Uncertainty Responses, ## Question Answers, ## Agreements, ## Risk Assessment.`);
-    }
     return sections.join('\n\n');
 }
 // =============================================================================
@@ -258,56 +227,3 @@ export function enhanceHandoff(handoff, uncertainties, questions, decisions) {
         decisions: decisions || handoff.decisions,
     };
 }
-/**
- * Build a prompt for general-purpose peer assistance (not review).
- * The peer acts as a collaborative coworker, not a critic.
- */
-export function buildPeerPrompt(options) {
-    const { workingDir, prompt, taskType, relevantFiles, context, focusAreas, customInstructions, outputFormat } = options;
-    const role = selectRole(focusAreas);
-    const sections = [];
-    // SECTION 1: ROLE
-    sections.push(`# ROLE: ${role.name} — Peer Engineer
-
-${role.systemPrompt}
-Be direct and actionable.`);
-    // SECTION 2: TASK
-    const taskLabel = taskType ? ` [${taskType.toUpperCase()}]` : '';
-    sections.push(`## YOUR TASK${taskLabel}
-
-**Request:** ${prompt}${context ? `\n**Context:** ${context}` : ''}`);
-    // SECTION 3: RELEVANT FILES
-    if (relevantFiles && relevantFiles.length > 0) {
-        sections.push(`## RELEVANT FILES\n${relevantFiles.map(f => `- \`${f}\``).join('\n')}`);
-    }
-    // SECTION 4: FOCUS AREAS
-    if (focusAreas && focusAreas.length > 0) {
-        sections.push(`## FOCUS AREAS\n\n${focusAreas.join(', ')}`);
-    }
-    // SECTION 5: CUSTOM INSTRUCTIONS
-    if (customInstructions) {
-        sections.push(`## ADDITIONAL INSTRUCTIONS\n\n${customInstructions}`);
-    }
-    // SECTION 6: OUTPUT FORMAT
-    if (outputFormat === 'schema-enforced') {
-        sections.push(`## OUTPUT FORMAT
-Respond with valid JSON matching the schema. Confidence reflects YOUR certainty.`);
-    }
-    else {
-        sections.push(`## OUTPUT FORMAT
-Respond with valid JSON:
-\`\`\`json
-{
-  "responder": "string",
-  "answer": "markdown",
-  "confidence": 0.0-1.0,
-  "key_points": ["string"],
-  "suggested_actions": [{ "action": "string", "priority": "high|medium|low", "file": "string", "rationale": "string" }],
-  "file_references": [{ "path": "string", "lines": "string", "relevance": "string" }],
-  "alternatives": [{ "topic": "string", "current_approach": "string", "alternative": "string", "tradeoffs": { "pros": [], "cons": [] }, "recommendation": "string" }],
-  "execution_notes": "string"
-}
-\`\`\``);
-    }
-    return sections.join('\n\n');
-}

package/dist/index.d.ts

@@ -13,6 +13,6 @@
  *
  * Usage:
  * - npx cc-reviewer          # Run MCP server (normal usage)
- * - npx cc-reviewer --setup  # Install slash commands only
+ * - npx cc-reviewer update   # Install/update slash commands
  */
 import './adapters/index.js';

package/dist/index.js

@@ -13,21 +13,31 @@
  *
  * Usage:
  * - npx cc-reviewer          # Run MCP server (normal usage)
- * - npx cc-reviewer --setup  # Install slash commands only
+ * - npx cc-reviewer update   # Install/update slash commands
  */
 import { Server } from '@modelcontextprotocol/sdk/server/index.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
 import { CallToolRequestSchema, ListToolsRequestSchema, } from '@modelcontextprotocol/sdk/types.js';
 import { handleCodexReview, handleGeminiReview, handleMultiReview, ReviewInputSchema, TOOL_DEFINITIONS } from './tools/feedback.js';
-import { handleAskCodex, handleAskGemini, handleAskMulti, PEER_TOOL_DEFINITIONS, } from './tools/peer.js';
-import { PeerInputSchema } from './schema.js';
 import { logCliStatus } from './cli/check.js';
 import { installCommands } from './commands.js';
-// Handle --setup flag: install commands and exit
-if (process.argv.includes('--setup') || process.argv.includes('--commands')) {
+// Read version from package.json
+import { readFileSync } from 'fs';
+import { join, dirname } from 'path';
+import { fileURLToPath } from 'url';
+const __pkgPath = join(dirname(fileURLToPath(import.meta.url)), '..', 'package.json');
+const __pkg = JSON.parse(readFileSync(__pkgPath, 'utf-8'));
+const VERSION = __pkg.version;
+// Handle subcommands
+const subcommand = process.argv[2];
+if (subcommand === 'update' || subcommand === '--setup' || subcommand === '--commands') {
     const result = installCommands();
     if (result.success) {
-        console.log(`✓ Installed ${result.installed.length} slash commands: ${result.installed.join(', ')}`);
+        console.log(`cc-reviewer v${VERSION}`);
+        if (result.removed.length > 0) {
+            console.log(`✓ Removed ${result.removed.length} deprecated commands: ${result.removed.map(c => `/${c}`).join(', ')}`);
+        }
+        console.log(`✓ Installed ${result.installed.length} slash commands: ${result.installed.map(c => `/${c}`).join(', ')}`);
         process.exit(0);
     }
     else {
@@ -40,7 +50,7 @@ import './adapters/index.js';
 // Create the MCP server
 const server = new Server({
     name: 'ai-reviewer',
-    version: '2.0.0',
+    version: VERSION,
 }, {
     capabilities: {
         tools: {},
@@ -53,9 +63,6 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
             TOOL_DEFINITIONS.codex_review,
             TOOL_DEFINITIONS.gemini_review,
             TOOL_DEFINITIONS.multi_review,
-            PEER_TOOL_DEFINITIONS.ask_codex,
-            PEER_TOOL_DEFINITIONS.ask_gemini,
-            PEER_TOOL_DEFINITIONS.ask_multi,
         ],
     };
 });
@@ -76,18 +83,6 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 const input = ReviewInputSchema.parse(args);
                 return await handleMultiReview(input);
             }
-            case 'ask_codex': {
-                const input = PeerInputSchema.parse(args);
-                return await handleAskCodex(input);
-            }
-            case 'ask_gemini': {
-                const input = PeerInputSchema.parse(args);
-                return await handleAskGemini(input);
-            }
-            case 'ask_multi': {
-                const input = PeerInputSchema.parse(args);
-                return await handleAskMulti(input);
-            }
             default:
                 return {
                     content: [{

package/dist/schema.d.ts

@@ -76,8 +76,8 @@ export declare const ReviewFinding: z.ZodObject<{
         column_start?: number | undefined;
         column_end?: number | undefined;
     } | undefined;
-    evidence?: string | undefined;
     suggestion?: string | undefined;
+    evidence?: string | undefined;
     cwe_id?: string | undefined;
     owasp_category?: string | undefined;
     tags?: string[] | undefined;
@@ -95,8 +95,8 @@ export declare const ReviewFinding: z.ZodObject<{
         column_start?: number | undefined;
         column_end?: number | undefined;
     } | undefined;
-    evidence?: string | undefined;
     suggestion?: string | undefined;
+    evidence?: string | undefined;
     cwe_id?: string | undefined;
     owasp_category?: string | undefined;
     tags?: string[] | undefined;
@@ -279,8 +279,8 @@ export declare const ReviewOutput: z.ZodObject<{
             column_start?: number | undefined;
             column_end?: number | undefined;
         } | undefined;
-        evidence?: string | undefined;
         suggestion?: string | undefined;
+        evidence?: string | undefined;
         cwe_id?: string | undefined;
         owasp_category?: string | undefined;
         tags?: string[] | undefined;
@@ -298,8 +298,8 @@ export declare const ReviewOutput: z.ZodObject<{
             column_start?: number | undefined;
             column_end?: number | undefined;
         } | undefined;
-        evidence?: string | undefined;
         suggestion?: string | undefined;
+        evidence?: string | undefined;
         cwe_id?: string | undefined;
         owasp_category?: string | undefined;
         tags?: string[] | undefined;
@@ -444,8 +444,8 @@ export declare const ReviewOutput: z.ZodObject<{
             column_start?: number | undefined;
             column_end?: number | undefined;
         } | undefined;
-        evidence?: string | undefined;
         suggestion?: string | undefined;
+        evidence?: string | undefined;
         cwe_id?: string | undefined;
         owasp_category?: string | undefined;
         tags?: string[] | undefined;
@@ -512,8 +512,8 @@ export declare const ReviewOutput: z.ZodObject<{
             column_start?: number | undefined;
             column_end?: number | undefined;
         } | undefined;
-        evidence?: string | undefined;
         suggestion?: string | undefined;
+        evidence?: string | undefined;
         cwe_id?: string | undefined;
         owasp_category?: string | undefined;
         tags?: string[] | undefined;
@@ -649,192 +649,3 @@ export declare function isSubstantiveReview(output: ReviewOutput): boolean;
  * This provides backwards compatibility during transition.
  */
 export declare function parseLegacyMarkdownOutput(markdown: string, reviewer: string): ReviewOutput | null;
-export declare const SuggestedAction: z.ZodObject<{
-    action: z.ZodString;
-    priority: z.ZodEnum<["high", "medium", "low"]>;
-    file: z.ZodOptional<z.ZodNullable<z.ZodString>>;
-    rationale: z.ZodString;
-}, "strip", z.ZodTypeAny, {
-    rationale: string;
-    action: string;
-    priority: "high" | "medium" | "low";
-    file?: string | null | undefined;
-}, {
-    rationale: string;
-    action: string;
-    priority: "high" | "medium" | "low";
-    file?: string | null | undefined;
-}>;
-export type SuggestedAction = z.infer<typeof SuggestedAction>;
-export declare const FileReference: z.ZodObject<{
-    path: z.ZodString;
-    lines: z.ZodOptional<z.ZodNullable<z.ZodString>>;
-    relevance: z.ZodString;
-}, "strip", z.ZodTypeAny, {
-    path: string;
-    relevance: string;
-    lines?: string | null | undefined;
-}, {
-    path: string;
-    relevance: string;
-    lines?: string | null | undefined;
-}>;
-export type FileReference = z.infer<typeof FileReference>;
-export declare const PeerOutput: z.ZodObject<{
-    responder: z.ZodString;
-    timestamp: z.ZodOptional<z.ZodNullable<z.ZodString>>;
-    answer: z.ZodString;
-    confidence: z.ZodNumber;
-    key_points: z.ZodArray<z.ZodString, "many">;
-    suggested_actions: z.ZodArray<z.ZodObject<{
-        action: z.ZodString;
-        priority: z.ZodEnum<["high", "medium", "low"]>;
-        file: z.ZodOptional<z.ZodNullable<z.ZodString>>;
-        rationale: z.ZodString;
-    }, "strip", z.ZodTypeAny, {
-        rationale: string;
-        action: string;
-        priority: "high" | "medium" | "low";
-        file?: string | null | undefined;
-    }, {
-        rationale: string;
-        action: string;
-        priority: "high" | "medium" | "low";
-        file?: string | null | undefined;
-    }>, "many">;
-    file_references: z.ZodArray<z.ZodObject<{
-        path: z.ZodString;
-        lines: z.ZodOptional<z.ZodNullable<z.ZodString>>;
-        relevance: z.ZodString;
-    }, "strip", z.ZodTypeAny, {
-        path: string;
-        relevance: string;
-        lines?: string | null | undefined;
-    }, {
-        path: string;
-        relevance: string;
-        lines?: string | null | undefined;
-    }>, "many">;
-    alternatives: z.ZodOptional<z.ZodNullable<z.ZodArray<z.ZodObject<{
-        topic: z.ZodString;
-        current_approach: z.ZodString;
-        alternative: z.ZodString;
-        tradeoffs: z.ZodObject<{
-            pros: z.ZodArray<z.ZodString, "many">;
-            cons: z.ZodArray<z.ZodString, "many">;
-        }, "strip", z.ZodTypeAny, {
-            pros: string[];
-            cons: string[];
-        }, {
-            pros: string[];
-            cons: string[];
-        }>;
-        recommendation: z.ZodEnum<["strongly_prefer", "consider", "situational", "informational"]>;
-    }, "strip", z.ZodTypeAny, {
-        topic: string;
-        tradeoffs: {
-            pros: string[];
-            cons: string[];
-        };
-        current_approach: string;
-        alternative: string;
-        recommendation: "strongly_prefer" | "consider" | "situational" | "informational";
-    }, {
-        topic: string;
-        tradeoffs: {
-            pros: string[];
-            cons: string[];
-        };
-        current_approach: string;
-        alternative: string;
-        recommendation: "strongly_prefer" | "consider" | "situational" | "informational";
-    }>, "many">>>;
-    execution_notes: z.ZodOptional<z.ZodNullable<z.ZodString>>;
-}, "strip", z.ZodTypeAny, {
-    confidence: number;
-    answer: string;
-    responder: string;
-    key_points: string[];
-    suggested_actions: {
-        rationale: string;
-        action: string;
-        priority: "high" | "medium" | "low";
-        file?: string | null | undefined;
-    }[];
-    file_references: {
-        path: string;
-        relevance: string;
-        lines?: string | null | undefined;
-    }[];
-    alternatives?: {
-        topic: string;
-        tradeoffs: {
-            pros: string[];
-            cons: string[];
-        };
-        current_approach: string;
-        alternative: string;
-        recommendation: "strongly_prefer" | "consider" | "situational" | "informational";
-    }[] | null | undefined;
-    timestamp?: string | null | undefined;
-    execution_notes?: string | null | undefined;
-}, {
-    confidence: number;
-    answer: string;
-    responder: string;
-    key_points: string[];
-    suggested_actions: {
-        rationale: string;
-        action: string;
-        priority: "high" | "medium" | "low";
-        file?: string | null | undefined;
-    }[];
-    file_references: {
-        path: string;
-        relevance: string;
-        lines?: string | null | undefined;
-    }[];
-    alternatives?: {
-        topic: string;
-        tradeoffs: {
-            pros: string[];
-            cons: string[];
-        };
-        current_approach: string;
-        alternative: string;
-        recommendation: "strongly_prefer" | "consider" | "situational" | "informational";
-    }[] | null | undefined;
-    timestamp?: string | null | undefined;
-    execution_notes?: string | null | undefined;
-}>;
-export type PeerOutput = z.infer<typeof PeerOutput>;
-export declare const TaskType: z.ZodEnum<["plan", "debug", "explain", "question", "fix", "explore", "general"]>;
-export type TaskType = z.infer<typeof TaskType>;
-export declare const PeerInputSchema: z.ZodObject<{
-    workingDir: z.ZodString;
-    prompt: z.ZodString;
-    taskType: z.ZodOptional<z.ZodEnum<["plan", "debug", "explain", "question", "fix", "explore", "general"]>>;
-    relevantFiles: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
-    context: z.ZodOptional<z.ZodString>;
-    focusAreas: z.ZodOptional<z.ZodArray<z.ZodEnum<["security", "performance", "architecture", "correctness", "maintainability", "scalability", "testing", "documentation"]>, "many">>;
-    customPrompt: z.ZodOptional<z.ZodString>;
-}, "strip", z.ZodTypeAny, {
-    workingDir: string;
-    prompt: string;
-    taskType?: "question" | "plan" | "debug" | "explain" | "fix" | "explore" | "general" | undefined;
-    context?: string | undefined;
-    relevantFiles?: string[] | undefined;
-    focusAreas?: ("performance" | "security" | "testing" | "architecture" | "correctness" | "maintainability" | "scalability" | "documentation")[] | undefined;
-    customPrompt?: string | undefined;
-}, {
-    workingDir: string;
-    prompt: string;
-    taskType?: "question" | "plan" | "debug" | "explain" | "fix" | "explore" | "general" | undefined;
-    context?: string | undefined;
-    relevantFiles?: string[] | undefined;
-    focusAreas?: ("performance" | "security" | "testing" | "architecture" | "correctness" | "maintainability" | "scalability" | "documentation")[] | undefined;
-    customPrompt?: string | undefined;
-}>;
-export type PeerInput = z.infer<typeof PeerInputSchema>;
-export declare function getPeerOutputJsonSchema(): object;
-export declare function parsePeerOutput(rawOutput: string): PeerOutput | null;

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: {
@@ -499,192 +499,3 @@ export function parseLegacyMarkdownOutput(markdown, reviewer) {
         return null;
     }
 }
-// =============================================================================
-// PEER OUTPUT SCHEMA (General-purpose coworker responses)
-// =============================================================================
-export const SuggestedAction = z.object({
-    action: z.string().describe('What to do'),
-    priority: z.enum(['high', 'medium', 'low']),
-    file: z.string().nullable().optional().describe('Relevant file path'),
-    rationale: z.string().describe('Why this action is recommended'),
-});
-export const FileReference = z.object({
-    path: z.string().describe('Relative file path'),
-    lines: z.string().nullable().optional().describe('Line range, e.g. "10-25"'),
-    relevance: z.string().describe('Why this file matters'),
-});
-export const PeerOutput = z.object({
-    responder: z.string().describe('"codex" or "gemini"'),
-    timestamp: z.string().nullable().optional(),
-    // Core response
-    answer: z.string().describe('Main response text (markdown)'),
-    confidence: ConfidenceScore.describe('Confidence in the response (0-1)'),
-    // Structured breakdown
-    key_points: z.array(z.string()).describe('Bullet summary of main points'),
-    // Actionable items
-    suggested_actions: z.array(SuggestedAction).describe('Recommended actions'),
-    // File references
-    file_references: z.array(FileReference).describe('Files examined by the peer'),
-    // Optional — nullable because OpenAI strict mode sends null instead of omitting
-    alternatives: z.array(Alternative).nullable().optional().describe('Alternative approaches'),
-    execution_notes: z.string().nullable().optional().describe('Notes about the process'),
-});
-// =============================================================================
-// PEER INPUT SCHEMA
-// =============================================================================
-export const TaskType = z.enum(['plan', 'debug', 'explain', 'question', 'fix', 'explore', 'general']);
-export const PeerInputSchema = z.object({
-    workingDir: z.string().describe('Working directory for filesystem access'),
-    prompt: z.string().describe('The question or request from CC'),
-    taskType: TaskType.optional().describe('Hint about the type of task'),
-    relevantFiles: z.array(z.string()).optional().describe('Files the peer should focus on'),
-    context: z.string().optional().describe('Additional context (error messages, prior analysis)'),
-    focusAreas: z.array(z.enum([
-        'security', 'performance', 'architecture', 'correctness',
-        'maintainability', 'scalability', 'testing', 'documentation'
-    ])).optional().describe('Areas to focus on'),
-    customPrompt: z.string().optional().describe('Additional instructions for the peer'),
-});
-// =============================================================================
-// PEER OUTPUT JSON SCHEMA (for embedding in prompts)
-// =============================================================================
-export function getPeerOutputJsonSchema() {
-    return {
-        type: 'object',
-        additionalProperties: false,
-        required: ['responder', 'answer', 'confidence', 'key_points', 'suggested_actions', 'file_references', 'timestamp', 'alternatives', 'execution_notes'],
-        properties: {
-            responder: { type: 'string' },
-            timestamp: { anyOf: [{ type: 'string' }, { type: 'null' }] },
-            answer: { type: 'string' },
-            confidence: { type: 'number', minimum: 0, maximum: 1 },
-            key_points: {
-                type: 'array',
-                items: { type: 'string' },
-            },
-            suggested_actions: {
-                type: 'array',
-                items: {
-                    type: 'object',
-                    additionalProperties: false,
-                    required: ['action', 'priority', 'rationale', 'file'],
-                    properties: {
-                        action: { type: 'string' },
-                        priority: { type: 'string', enum: ['high', 'medium', 'low'] },
-                        file: { anyOf: [{ type: 'string' }, { type: 'null' }] },
-                        rationale: { type: 'string' },
-                    },
-                },
-            },
-            file_references: {
-                type: 'array',
-                items: {
-                    type: 'object',
-                    additionalProperties: false,
-                    required: ['path', 'relevance', 'lines'],
-                    properties: {
-                        path: { type: 'string' },
-                        lines: { anyOf: [{ type: 'string' }, { type: 'null' }] },
-                        relevance: { type: 'string' },
-                    },
-                },
-            },
-            alternatives: {
-                anyOf: [
-                    {
-                        type: 'array',
-                        items: {
-                            type: 'object',
-                            additionalProperties: false,
-                            required: ['topic', 'current_approach', 'alternative', 'tradeoffs', 'recommendation'],
-                            properties: {
-                                topic: { type: 'string' },
-                                current_approach: { type: 'string' },
-                                alternative: { type: 'string' },
-                                tradeoffs: {
-                                    type: 'object',
-                                    additionalProperties: false,
-                                    required: ['pros', 'cons'],
-                                    properties: {
-                                        pros: { type: 'array', items: { type: 'string' } },
-                                        cons: { type: 'array', items: { type: 'string' } },
-                                    },
-                                },
-                                recommendation: { type: 'string', enum: ['strongly_prefer', 'consider', 'situational', 'informational'] },
-                            },
-                        },
-                    },
-                    { type: 'null' },
-                ],
-            },
-            execution_notes: { anyOf: [{ type: 'string' }, { type: 'null' }] },
-        },
-    };
-}
-// =============================================================================
-// PEER OUTPUT PARSING
-// =============================================================================
-function normalizePeerOutput(parsed) {
-    const normalized = { ...parsed };
-    if (!normalized.responder) {
-        normalized.responder = 'external';
-    }
-    normalized.key_points = normalized.key_points ?? [];
-    normalized.suggested_actions = normalized.suggested_actions ?? [];
-    normalized.file_references = normalized.file_references ?? [];
-    if (normalized.confidence === undefined) {
-        normalized.confidence = 0.5;
-    }
-    if (!normalized.answer && typeof normalized.response === 'string') {
-        normalized.answer = normalized.response;
-    }
-    return normalized;
-}
-export function parsePeerOutput(rawOutput) {
-    try {
-        let jsonStr = rawOutput;
-        // Unwrap Gemini envelope
-        try {
-            const envelope = JSON.parse(rawOutput);
-            if (envelope && typeof envelope.session_id === 'string' && typeof envelope.response === 'string') {
-                jsonStr = envelope.response;
-            }
-        }
-        catch {
-            // Not an envelope
-        }
-        // Extract from ```json ... ``` blocks
-        const jsonBlockMatch = jsonStr.match(/```(?:json)?\s*([\s\S]*?)```/);
-        if (jsonBlockMatch) {
-            jsonStr = jsonBlockMatch[1].trim();
-        }
-        // Find JSON object boundaries
-        const jsonStart = jsonStr.indexOf('{');
-        const jsonEnd = jsonStr.lastIndexOf('}');
-        if (jsonStart !== -1 && jsonEnd !== -1 && jsonEnd > jsonStart) {
-            jsonStr = jsonStr.slice(jsonStart, jsonEnd + 1);
-        }
-        const parsed = JSON.parse(jsonStr);
-        // Try direct parse
-        const result = PeerOutput.safeParse(parsed);
-        if (result.success) {
-            return result.data;
-        }
-        // Normalize and retry
-        const recognizedFields = ['responder', 'answer', 'response', 'key_points', 'suggested_actions', 'file_references', 'confidence'];
-        const hasRecognizedField = typeof parsed === 'object' && parsed !== null && !Array.isArray(parsed) &&
-            recognizedFields.some(f => f in parsed);
-        if (!hasRecognizedField) {
-            return null;
-        }
-        const normalized = normalizePeerOutput(parsed);
-        const retryResult = PeerOutput.safeParse(normalized);
-        if (retryResult.success) {
-            return retryResult.data;
-        }
-        return null;
-    }
-    catch {
-        return null;
-    }
-}

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;
 }>;