@j0hanz/code-review-analyst-mcp 1.3.0 → 1.4.1

package/dist/tools/inspect-code-quality.js

@@ -1,6 +1,7 @@
 import { validateContextBudget } from '../lib/context-budget.js';
 import { validateDiffBudget } from '../lib/diff-budget.js';
 import { computeDiffStatsAndSummaryFromFiles, parseDiffFiles, } from '../lib/diff-parser.js';
+import { createNoDiffError } from '../lib/diff-store.js';
 import { requireToolContract } from '../lib/tool-contracts.js';
 import { registerStructuredToolTask } from '../lib/tool-factory.js';
 import { InspectCodeQualityInputSchema } from '../schemas/inputs.js';
@@ -49,31 +50,11 @@ ${sanitizeContent(file.content)}
     }
     return `${FILE_CONTEXT_HEADING}${fileBlocks.join('\n')}`;
 }
-function buildInspectPrompt(input) {
-    const parsedFiles = parseDiffFiles(input.diff);
-    const { summary: fileSummary } = computeDiffStatsAndSummaryFromFiles(parsedFiles);
-    const fileContext = formatFileContext(input.files);
-    const languageLine = formatOptionalLine('Language', input.language);
-    const maxFindingsLine = formatOptionalLine('Max Findings', input.maxFindings);
-    const noFilesNote = !input.files?.length
-        ? '\nNote: No file context provided. Leave contextualInsights empty.'
-        : '';
-    return `
-Repository: ${input.repository}${languageLine}
-Focus Areas: ${input.focusAreas?.join(', ') ?? DEFAULT_FOCUS_AREAS}${maxFindingsLine}${noFilesNote}
-Changed Files:
-${fileSummary}
-
-Diff:
-${input.diff}
-${fileContext}
-`;
-}
 export function registerInspectCodeQualityTool(server) {
     registerStructuredToolTask(server, {
         name: 'inspect_code_quality',
         title: 'Inspect Code Quality',
-        description: 'Deep-dive code review with optional file context.',
+        description: 'Deep code review. Prerequisite: generate_diff. Auto-infer repo/language/focus. Provide file content for best results.',
         inputSchema: InspectCodeQualityInputSchema,
         fullInputSchema: InspectCodeQualityInputSchema,
         resultSchema: CodeQualityOutputSchema,
@@ -89,13 +70,16 @@ export function registerInspectCodeQualityTool(server) {
             const fileCount = input.files?.length;
             return fileCount ? `+${fileCount} files` : '';
         },
-        formatOutcome: (result) => `${result.findings.length} findings, risk: ${result.overallRisk}`,
-        validateInput: (input) => {
-            const diffError = validateDiffBudget(input.diff);
+        validateInput: (input, ctx) => {
+            const slot = ctx.diffSlot;
+            if (!slot)
+                return createNoDiffError();
+            const diffError = validateDiffBudget(slot.diff);
             if (diffError)
                 return diffError;
-            return validateContextBudget(input.diff, input.files);
+            return validateContextBudget(slot.diff, input.files);
         },
+        formatOutcome: (result) => `${result.findings.length} findings, risk: ${result.overallRisk}`,
         formatOutput: (result) => {
             const count = result.findings.length;
             const total = result.totalFindings ?? count;
@@ -109,9 +93,29 @@ export function registerInspectCodeQualityTool(server) {
             const cappedFindings = capFindings(result.findings, input.maxFindings);
             return { ...result, findings: cappedFindings, totalFindings };
         },
-        buildPrompt: (input) => ({
-            systemInstruction: SYSTEM_INSTRUCTION,
-            prompt: buildInspectPrompt(input),
-        }),
+        buildPrompt: (input, ctx) => {
+            const diff = ctx.diffSlot?.diff ?? '';
+            const parsedFiles = parseDiffFiles(diff);
+            const { summary: fileSummary } = computeDiffStatsAndSummaryFromFiles(parsedFiles);
+            const fileContext = formatFileContext(input.files);
+            const languageLine = formatOptionalLine('Language', input.language);
+            const maxFindingsLine = formatOptionalLine('Max Findings', input.maxFindings);
+            const noFilesNote = !input.files?.length
+                ? '\nNote: No file context provided. Leave contextualInsights empty.'
+                : '';
+            return {
+                systemInstruction: SYSTEM_INSTRUCTION,
+                prompt: `
+Repository: ${input.repository}${languageLine}
+Focus Areas: ${input.focusAreas?.join(', ') ?? DEFAULT_FOCUS_AREAS}${maxFindingsLine}${noFilesNote}
+Changed Files:
+${fileSummary}
+
+Diff:
+${diff}
+${fileContext}
+`,
+            };
+        },
     });
 }

package/dist/tools/suggest-search-replace.js

@@ -1,5 +1,6 @@
 import { validateDiffBudget } from '../lib/diff-budget.js';
 import { extractChangedPathsFromFiles, parseDiffFiles, } from '../lib/diff-parser.js';
+import { createNoDiffError } from '../lib/diff-store.js';
 import { requireToolContract } from '../lib/tool-contracts.js';
 import { registerStructuredToolTask } from '../lib/tool-factory.js';
 import { SuggestSearchReplaceInputSchema } from '../schemas/inputs.js';
@@ -14,23 +15,11 @@ const TOOL_CONTRACT = requireToolContract('suggest_search_replace');
 function formatPatchCount(count) {
     return `${count} ${count === 1 ? 'patch' : 'patches'}`;
 }
-function buildSuggestSearchReplacePrompt(input) {
-    const files = parseDiffFiles(input.diff);
-    const paths = extractChangedPathsFromFiles(files);
-    return `
-Finding: ${input.findingTitle}
-Details: ${input.findingDetails}
-Changed Files: ${paths.join(', ')}
-
-Diff:
-${input.diff}
-`;
-}
 export function registerSuggestSearchReplaceTool(server) {
     registerStructuredToolTask(server, {
         name: 'suggest_search_replace',
         title: 'Suggest Search & Replace',
-        description: 'Generate search-and-replace blocks to fix a finding.',
+        description: 'Generate fix patches. Prerequisite: inspect_code_quality findings. Pass verbatim finding title/details.',
         inputSchema: SuggestSearchReplaceInputSchema,
         fullInputSchema: SuggestSearchReplaceInputSchema,
         resultSchema: SearchReplaceResultSchema,
@@ -41,16 +30,33 @@ export function registerSuggestSearchReplaceTool(server) {
         ...(TOOL_CONTRACT.thinkingBudget !== undefined
             ? { thinkingBudget: TOOL_CONTRACT.thinkingBudget }
             : undefined),
-        validateInput: (input) => validateDiffBudget(input.diff),
+        validateInput: (_input, ctx) => {
+            const slot = ctx.diffSlot;
+            if (!slot)
+                return createNoDiffError();
+            return validateDiffBudget(slot.diff);
+        },
         formatOutcome: (result) => formatPatchCount(result.blocks.length),
         formatOutput: (result) => {
             const count = result.blocks.length;
             const patches = formatPatchCount(count);
             return `${result.summary}\n${patches} • Checklist: ${result.validationChecklist.join(' | ')}`;
         },
-        buildPrompt: (input) => ({
-            systemInstruction: SYSTEM_INSTRUCTION,
-            prompt: buildSuggestSearchReplacePrompt(input),
-        }),
+        buildPrompt: (input, ctx) => {
+            const diff = ctx.diffSlot?.diff ?? '';
+            const files = parseDiffFiles(diff);
+            const paths = extractChangedPathsFromFiles(files);
+            return {
+                systemInstruction: SYSTEM_INSTRUCTION,
+                prompt: `
+Finding: ${input.findingTitle}
+Details: ${input.findingDetails}
+Changed Files: ${paths.join(', ')}
+
+Diff:
+${diff}
+`,
+            };
+        },
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@j0hanz/code-review-analyst-mcp",
-  "version": "1.3.0",
+  "version": "1.4.1",
   "mcpName": "io.github.j0hanz/code-review-analyst",
   "description": "Gemini-powered MCP server for code review analysis.",
   "type": "module",
@@ -32,7 +32,6 @@
   },
   "scripts": {
     "clean": "node scripts/tasks.mjs clean",
-    "validate:instructions": "node scripts/tasks.mjs validate:instructions",
     "build": "node scripts/tasks.mjs build",
     "copy:assets": "node scripts/tasks.mjs copy:assets",
     "prepare": "npm run build",

package/dist/instructions.md

@@ -1,7 +0,0 @@
-# CODE REVIEW ANALYST MCP INSTRUCTIONS
-
-Runtime instructions are generated from `src/lib/tool-contracts.ts` by
-`buildServerInstructions()` in `src/resources/instructions.ts`.
-
-This file is kept as a packaging placeholder for build pipelines that expect an
-`instructions.md` artifact in `dist/`.