@j0hanz/code-review-analyst-mcp 1.5.3 → 1.6.0

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

@@ -1,6 +1,6 @@
 import { validateContextBudget } from '../lib/context-budget.js';
 import { computeDiffStatsAndSummaryFromFiles } from '../lib/diff-parser.js';
-import { requireToolContract } from '../lib/tool-contracts.js';
+import { buildStructuredToolRuntimeOptions, requireToolContract, } from '../lib/tool-contracts.js';
 import { registerStructuredToolTask } from '../lib/tool-factory.js';
 import { InspectCodeQualityInputSchema } from '../schemas/inputs.js';
 import { CodeQualityOutputSchema, CodeQualityResultSchema, } from '../schemas/outputs.js';
@@ -10,12 +10,16 @@ const PATH_ESCAPE_REPLACEMENTS = {
     '"': '\\"',
     '\n': ' ',
     '\r': ' ',
+    '<': '&lt;',
+    '>': '&gt;',
 };
-const PATH_ESCAPE_PATTERN = /["\n\r]/g;
+const PATH_ESCAPE_PATTERN = /["\n\r<>]/g;
 const SYSTEM_INSTRUCTION = `
-You are a principal engineer performing a deep code review. Identify bugs, security vulnerabilities, performance issues, and maintainability risks from the diff and file context.
-Ignore style issues unless they cause runtime risk. Prioritize correctness and failure modes.
-Return strict JSON only.
+Principal Engineer Code Review.
+Source: Unified diff (primary), File excerpts (supplementary context).
+Goal: Identify bugs, security, performance, maintainability.
+Ignore style. Prioritize correctness/failure modes.
+Return strict JSON.
 `;
 const TOOL_CONTRACT = requireToolContract('inspect_code_quality');
 export function sanitizePath(path) {
@@ -38,21 +42,20 @@ function formatFileContext(files) {
     if (!files || files.length === 0) {
         return '';
     }
-    const fileBlocks = [];
-    for (const file of files) {
-        fileBlocks.push(`
+    return (FILE_CONTEXT_HEADING +
+        files
+            .map((file) => `
 <<FILE path="${sanitizePath(file.path)}">>
 ${sanitizeContent(file.content)}
 <<END_FILE>>
-`);
-    }
-    return `${FILE_CONTEXT_HEADING}${fileBlocks.join('\n')}`;
+`)
+            .join('\n'));
 }
 export function registerInspectCodeQualityTool(server) {
     registerStructuredToolTask(server, {
         name: 'inspect_code_quality',
         title: 'Inspect Code Quality',
-        description: 'Deep code review. Prerequisite: generate_diff. Auto-infer repo/language/focus. Provide file content for best results.',
+        description: 'Deep code review. Prerequisite: generate_diff. Auto-infer repo/language/focus. Operates primarily on the diff; files are optional supplementary excerpts only.',
         inputSchema: InspectCodeQualityInputSchema,
         fullInputSchema: InspectCodeQualityInputSchema,
         resultSchema: CodeQualityOutputSchema,
@@ -61,15 +64,7 @@ export function registerInspectCodeQualityTool(server) {
         model: TOOL_CONTRACT.model,
         timeoutMs: TOOL_CONTRACT.timeoutMs,
         maxOutputTokens: TOOL_CONTRACT.maxOutputTokens,
-        ...(TOOL_CONTRACT.thinkingLevel !== undefined
-            ? { thinkingLevel: TOOL_CONTRACT.thinkingLevel }
-            : undefined),
-        ...(TOOL_CONTRACT.temperature !== undefined
-            ? { temperature: TOOL_CONTRACT.temperature }
-            : undefined),
-        ...(TOOL_CONTRACT.deterministicJson !== undefined
-            ? { deterministicJson: TOOL_CONTRACT.deterministicJson }
-            : undefined),
+        ...buildStructuredToolRuntimeOptions(TOOL_CONTRACT),
         progressContext: (input) => {
             const fileCount = input.files?.length;
             return fileCount ? `+${fileCount} files` : '';
@@ -83,10 +78,8 @@ export function registerInspectCodeQualityTool(server) {
         formatOutput: (result) => {
             const count = result.findings.length;
             const total = result.totalFindings ?? count;
-            const findingsSuffix = count < total
-                ? `${count} of ${total} findings reported.`
-                : `${count} findings reported.`;
-            return `Code Quality Inspection: ${result.summary}\n${findingsSuffix}`;
+            const findingsSuffix = count < total ? `${count} of ${total} findings.` : `${count} findings.`;
+            return `${result.summary}\n${findingsSuffix}`;
         },
         transformResult: (input, result) => {
             const totalFindings = result.findings.length;
@@ -101,7 +94,7 @@ export function registerInspectCodeQualityTool(server) {
             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.'
+                ? '\nNote: No file excerpts provided. Review based on diff only; leave contextualInsights empty.'
                 : '';
             return {
                 systemInstruction: SYSTEM_INSTRUCTION,

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

@@ -1,13 +1,14 @@
 import { extractChangedPathsFromFiles } from '../lib/diff-parser.js';
-import { requireToolContract } from '../lib/tool-contracts.js';
+import { buildStructuredToolRuntimeOptions, requireToolContract, } from '../lib/tool-contracts.js';
 import { registerStructuredToolTask } from '../lib/tool-factory.js';
 import { SuggestSearchReplaceInputSchema } from '../schemas/inputs.js';
 import { SearchReplaceResultSchema } from '../schemas/outputs.js';
 const SYSTEM_INSTRUCTION = `
-You are a code remediation expert. Generate minimal search-and-replace blocks to fix exactly the described issue.
-CRITICAL: 'search' must be verbatim — character-exact whitespace and indentation.
-Never modify code outside the fix scope. If the target cannot be located precisely, omit the block rather than guessing.
-Return strict JSON only.
+Code Remediation Expert.
+Generate minimal search/replace blocks for described issue.
+CRITICAL: 'search' must be verbatim (exact whitespace/indentation).
+No out-of-scope changes. Omit if imprecise.
+Return strict JSON.
 `;
 const TOOL_CONTRACT = requireToolContract('suggest_search_replace');
 function formatPatchCount(count) {
@@ -25,21 +26,13 @@ export function registerSuggestSearchReplaceTool(server) {
         model: TOOL_CONTRACT.model,
         timeoutMs: TOOL_CONTRACT.timeoutMs,
         maxOutputTokens: TOOL_CONTRACT.maxOutputTokens,
-        ...(TOOL_CONTRACT.thinkingLevel !== undefined
-            ? { thinkingLevel: TOOL_CONTRACT.thinkingLevel }
-            : undefined),
-        ...(TOOL_CONTRACT.temperature !== undefined
-            ? { temperature: TOOL_CONTRACT.temperature }
-            : undefined),
-        ...(TOOL_CONTRACT.deterministicJson !== undefined
-            ? { deterministicJson: TOOL_CONTRACT.deterministicJson }
-            : undefined),
+        ...buildStructuredToolRuntimeOptions(TOOL_CONTRACT),
         requiresDiff: true,
         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(' | ')}`;
+            return `${result.summary}\n${patches} • ${result.validationChecklist.join(' | ')}`;
         },
         buildPrompt: (input, ctx) => {
             const diff = ctx.diffSlot?.diff ?? '';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@j0hanz/code-review-analyst-mcp",
-  "version": "1.5.3",
+  "version": "1.6.0",
   "mcpName": "io.github.j0hanz/code-review-analyst",
   "description": "Gemini-powered MCP server for code review analysis.",
   "type": "module",