codecritique 2.0.1 → 2.0.3

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.1",
+  "version": "2.0.3",
   "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/rag-analyzer.js

@@ -722,7 +722,7 @@ function prepareContextForLLM(filePath, content, language, finalCodeExamples, fi
     context: contextSections,
     codeExamples,
     guidelineSnippets,
-    customDocs: relevantCustomDocChunks || customDocs, // Use relevant chunks if available, fallback to full docs
+    customDocs: relevantCustomDocChunks?.length > 0 ? relevantCustomDocChunks : customDocs, // Use relevant chunks if available, fallback to full docs
     feedbackContext: generateFeedbackContext(dismissedPatterns), // Add feedback context for LLM
     projectSummary: projectSummary, // Add project architecture summary
     metadata: {
@@ -1729,7 +1729,12 @@ async function performHolisticPRAnalysis(options) {
           items: unifiedContext.prComments.slice(0, 10),
         },
       ],
-      customDocs: unifiedContext.customDocChunks || options.relevantCustomDocChunks || customDocs, // Use unified chunks first, then relevant chunks, then full docs
+      customDocs:
+        unifiedContext.customDocChunks?.length > 0
+          ? unifiedContext.customDocChunks
+          : options.relevantCustomDocChunks?.length > 0
+            ? options.relevantCustomDocChunks
+            : customDocs, // Use chunks only when available, then fall back to full docs
       projectSummary: projectSummary, // Add project architecture summary
       metadata: {
         hasCodeExamples: unifiedContext.codeExamples.length > 0,

package/src/rag-analyzer.test.js

@@ -762,6 +762,19 @@ describe('rag-analyzer', () => {
       expect(result.success).toBe(true);
     });
 
+    it('should fall back to full custom docs when no relevant chunks are found', async () => {
+      mockEmbeddingsSystem.getExistingCustomDocumentChunks.mockResolvedValue([
+        { id: 'existing', content: 'Existing indexed chunk', document_title: 'Indexed Doc', chunk_index: 0 },
+      ]);
+      mockEmbeddingsSystem.findRelevantCustomDocChunks.mockResolvedValue([]);
+
+      await runAnalysis('/test/file.js', {
+        customDocs: [{ content: 'Mandatory internal review rule', document_title: 'Internal Standards' }],
+      });
+
+      expect(llm.sendPromptToClaude).toHaveBeenCalledWith(expect.stringContaining('Mandatory internal review rule'), expect.any(Object));
+    });
+
     it('should log selected chunks when verbose', async () => {
       mockEmbeddingsSystem.findRelevantCustomDocChunks.mockResolvedValue([
         { id: 'chunk1', content: 'Guidelines', document_title: 'Guide', chunk_index: 0, similarity: 0.85 },

package/src/rag-review.js

@@ -230,7 +230,17 @@ async function reviewPullRequestWithCrossFileContext(filesToReview, options = {}
     verboseLog(verbose, chalk.gray(`Base branch: ${baseBranch}, Target branch: ${targetBranch}`));
 
     const prFiles = [];
-    for (const filePath of filesToReview) {
+    if (options.preloadedPRFiles?.length > 0) {
+      prFiles.push(
+        ...options.preloadedPRFiles.map((file) => ({
+          ...file,
+          baseBranch: file.baseBranch || baseBranch,
+          targetBranch: file.targetBranch || targetBranch,
+        }))
+      );
+    }
+
+    for (const filePath of options.preloadedPRFiles?.length > 0 ? [] : filesToReview) {
       try {
         // Check if the file should be processed before fetching its content from git
         if (!shouldProcessFile(filePath, '', options)) {
@@ -596,6 +606,7 @@ async function reviewPRChunk(chunk, sharedContext, options, chunkNumber, totalCh
 
   // Skip chunking decision for chunk reviews to prevent infinite recursion
   const skipChunkingOptions = { ...chunkOptions, skipChunking: true };
+  skipChunkingOptions.preloadedPRFiles = chunk.files;
 
   return await reviewPullRequestWithCrossFileContext(chunkFilePaths, skipChunkingOptions);
 }