codecritique 2.0.2 → 2.0.4

package/README.md

@@ -3,7 +3,7 @@
 [![npm version](https://img.shields.io/npm/v/codecritique.svg)](https://www.npmjs.com/package/codecritique)
 [![npm downloads](https://img.shields.io/npm/dm/codecritique.svg)](https://www.npmjs.com/package/codecritique)
 [![License: MIT](https://img.shields.io/badge/License-MIT-blue.svg)](https://opensource.org/licenses/MIT)
-[![Node.js](https://img.shields.io/badge/node-%3E%3D24.15.0-brightgreen.svg)](https://nodejs.org/)
+[![Node.js](https://img.shields.io/badge/node-%3E%3D24-brightgreen.svg)](https://nodejs.org/)
 [![CI](https://github.com/cosmocoder/CodeCritique/actions/workflows/release.yml/badge.svg)](https://github.com/cosmocoder/CodeCritique/actions/workflows/release.yml)
 
 **AI-Powered Code Review. Context-Aware. Privacy-First.**
@@ -96,7 +96,7 @@ This RAG-based approach provides more accurate, project-specific code reviews co
 
 ### Prerequisites
 
-- **Node.js** v24.15.0 or higher
+- **Node.js** 24 or newer
 - **Git** (for diff-based analysis)
 - **Anthropic API key** (for LLM analysis)
 
@@ -199,7 +199,7 @@ For easier integration with non-JavaScript projects, you can use the provided sh
 
 3. **Environment setup** (the script handles this automatically):
    - Creates/uses `.env` file in your project directory
-   - Validates Node.js v24.15.0+ requirement
+   - Validates Node.js 24+ requirement
    - Provides helpful error messages for missing dependencies
 
 ## Quick Start

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codecritique",
-  "version": "2.0.2",
+  "version": "2.0.4",
   "description": "AI-powered code review tool for any programming language",
   "type": "module",
   "main": "src/index.js",
@@ -90,8 +90,8 @@
     "npm": "11.12.1"
   },
   "engines": {
-    "node": ">=24.15.0",
-    "npm": ">=11.12.1 <12"
+    "node": ">=24",
+    "npm": ">=11"
   },
   "engine-strict": true,
   "publishConfig": {

package/src/index.js

@@ -31,6 +31,7 @@ import { reviewFile, reviewFiles, reviewPullRequest } from './rag-review.js';
 import { execGitSafe } from './utils/command.js';
 import { ensureBranchExists, findBaseBranch } from './utils/git.js';
 import { verboseLog } from './utils/logging.js';
+import { collectPRLevelFindings, getFileLevelIssueCount, hasPRLevelFindings } from './utils/review-results.js';
 
 // Create a default embeddings system instance
 const embeddingsSystem = getDefaultEmbeddingsSystem();
@@ -424,7 +425,7 @@ async function runCodeReview(options) {
         // Determine output function based on format option
         const outputFn = options.output === 'json' ? outputJson : options.output === 'markdown' ? outputMarkdown : outputText;
         // Pass the detailed results array to the output function
-        outputFn(reviewResult.results, options);
+        outputFn(reviewResult.results, options, reviewResult);
         console.log(chalk.bold.green(`\nAnalysis complete for ${operationDescription}! (${duration}s)`));
       }
       else {
@@ -435,7 +436,7 @@ async function runCodeReview(options) {
         // Still output empty results if outputFile is specified (for CI/CD pipelines)
         if (options.outputFile) {
           const outputFn = options.output === 'json' ? outputJson : options.output === 'markdown' ? outputMarkdown : outputText;
-          outputFn([], options);
+          outputFn([], options, reviewResult);
           console.log(chalk.yellow(`Empty results written to: ${options.outputFile}`));
         }
 
@@ -1013,14 +1014,20 @@ function getChangedFiles(branch, workingDir = process.cwd()) {
  *
  * @param {Array<Object>} reviewResults - Array of individual file review results
  * @param {Object} options - Command line options
+ * @param {Object} aggregateResult - Full review result with PR-level context
  */
-function outputJson(reviewResults, options) {
+function outputJson(reviewResults, options, aggregateResult = {}) {
+  const prLevelFindings = collectPRLevelFindings(reviewResults, aggregateResult);
+  const fileLevelIssues = getFileLevelIssueCount(reviewResults);
+
   // Structure the output to be informative
   const output = {
     summary: {
       totalFilesReviewed: reviewResults.length,
       filesWithIssues: reviewResults.filter((r) => r.success && !r.skipped && r.results?.issues?.length > 0).length,
-      totalIssues: reviewResults.reduce((sum, r) => sum + (r.results?.issues?.length || 0), 0),
+      totalIssues: fileLevelIssues + prLevelFindings.issues.length,
+      fileLevelIssues,
+      prLevelIssues: prLevelFindings.issues.length,
       issuesWithCodeSuggestions: reviewResults.reduce((sum, r) => {
         if (!r.success || r.skipped || !r.results?.issues) {
           return sum;
@@ -1049,6 +1056,10 @@ function outputJson(reviewResults, options) {
     }),
   };
 
+  if (hasPRLevelFindings(prLevelFindings)) {
+    output.prLevelFindings = prLevelFindings;
+  }
+
   const jsonOutput = JSON.stringify(output, null, 2);
 
   // If output-file is specified, write to file instead of stdout
@@ -1067,13 +1078,15 @@ function outputJson(reviewResults, options) {
  *
  * @param {Array<Object>} reviewResults - Array of individual file review results
  * @param {Object} options - Command line options
+ * @param {Object} aggregateResult - Full review result with PR-level context
  */
-function outputMarkdown(reviewResults, options) {
+function outputMarkdown(reviewResults, options, aggregateResult = {}) {
   const totalFiles = reviewResults.length;
   const filesWithIssues = reviewResults.filter((r) => r.success && !r.skipped && r.results?.issues?.length > 0).length;
-  const totalIssues = reviewResults.reduce((sum, r) => sum + (r.results?.issues?.length || 0), 0);
   const skippedFiles = reviewResults.filter((r) => r.skipped).length;
   const errorFiles = reviewResults.filter((r) => !r.success).length;
+  const prLevelFindings = collectPRLevelFindings(reviewResults, aggregateResult);
+  const totalIssues = getFileLevelIssueCount(reviewResults) + prLevelFindings.issues.length;
 
   const lines = [
     '# AI Code Review Results (RAG Approach)',
@@ -1092,6 +1105,8 @@ function outputMarkdown(reviewResults, options) {
     lines.push(`- **Errors:** ${errorFiles}`);
   }
 
+  appendPRLevelFindingsMarkdown(lines, prLevelFindings);
+
   lines.push('', '## Detailed Review per File', '');
 
   reviewResults.forEach((fileResult) => {
@@ -1157,13 +1172,15 @@ function outputMarkdown(reviewResults, options) {
  *
  * @param {Array<Object>} reviewResults - Array of individual file review results
  * @param {Object} cliOptions - Command line options
+ * @param {Object} aggregateResult - Full review result with PR-level context
  */
-function outputText(reviewResults, cliOptions) {
+function outputText(reviewResults, cliOptions, aggregateResult = {}) {
   const totalFiles = reviewResults.length;
   const filesWithIssues = reviewResults.filter((r) => r.success && !r.skipped && r.results?.issues?.length > 0).length;
-  const totalIssues = reviewResults.reduce((sum, r) => sum + (r.results?.issues?.length || 0), 0);
   const skippedFiles = reviewResults.filter((r) => r.skipped).length;
   const errorFiles = reviewResults.filter((r) => !r.success).length;
+  const prLevelFindings = collectPRLevelFindings(reviewResults, aggregateResult);
+  const totalIssues = getFileLevelIssueCount(reviewResults) + prLevelFindings.issues.length;
 
   console.log(chalk.bold.blue('\n===== AI Code Review Summary ====='));
   console.log(`Files Analyzed: ${chalk.bold(totalFiles)}`);
@@ -1177,6 +1194,8 @@ function outputText(reviewResults, cliOptions) {
   }
   console.log(chalk.bold.blue('================================================'));
 
+  outputPRLevelFindingsText(prLevelFindings);
+
   reviewResults.forEach((fileResult) => {
     if (!fileResult.success) {
       console.log(chalk.bold.red(`\n===== Error reviewing ${fileResult.filePath} =====`));
@@ -1230,6 +1249,68 @@ function outputText(reviewResults, cliOptions) {
   });
 }
 
+function appendPRLevelFindingsMarkdown(lines, prLevelFindings) {
+  if (!hasPRLevelFindings(prLevelFindings)) {
+    return;
+  }
+
+  lines.push('', '## PR-Level Findings', '');
+
+  if (prLevelFindings.summary) {
+    lines.push(`**Summary:** ${prLevelFindings.summary}`, '');
+  }
+
+  for (const issue of prLevelFindings.issues) {
+    const severity = issue.severity || 'info';
+    const severityEmoji = getSeverityEmoji(severity);
+    lines.push(`- **[${severity.toUpperCase()}] ${severityEmoji}** ${issue.description}`);
+    if (issue.files?.length > 0) {
+      lines.push(`  - Affected files: ${issue.files.join(', ')}`);
+    }
+    if (issue.suggestion) {
+      lines.push(`  - Suggestion: ${issue.suggestion}`);
+    }
+  }
+
+  if (prLevelFindings.recommendations.length > 0) {
+    lines.push('', '**Recommendations:**', '');
+    for (const recommendation of prLevelFindings.recommendations) {
+      lines.push(`- ${recommendation}`);
+    }
+  }
+}
+
+function outputPRLevelFindingsText(prLevelFindings) {
+  if (!hasPRLevelFindings(prLevelFindings)) {
+    return;
+  }
+
+  console.log(chalk.bold.underline('\n===== PR-Level Findings ====='));
+  if (prLevelFindings.summary) {
+    console.log(chalk.bold.cyan(`Summary: ${prLevelFindings.summary}`));
+  }
+
+  for (const issue of prLevelFindings.issues) {
+    const severity = issue.severity || 'info';
+    const severityColor = getSeverityColor(severity);
+    console.log(`  ${severityColor(`[${severity.toUpperCase()}]`)} ${issue.description}`);
+    if (issue.files?.length > 0) {
+      console.log(`    Affected files: ${issue.files.join(', ')}`);
+    }
+    if (issue.suggestion) {
+      console.log(`    ${chalk.green(`Suggestion: ${issue.suggestion}`)}`);
+    }
+    console.log('');
+  }
+
+  if (prLevelFindings.recommendations.length > 0) {
+    console.log(chalk.bold.yellow('Recommendations:'));
+    for (const recommendation of prLevelFindings.recommendations) {
+      console.log(`  - ${recommendation}`);
+    }
+  }
+}
+
 // --- Severity Helpers (Remain Unchanged) --- //
 
 /**