codecritique 1.2.4 → 2.0.1

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%3D22.14.0-brightgreen.svg)](https://nodejs.org/)
+[![Node.js](https://img.shields.io/badge/node-%3E%3D24.15.0-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** v22.14.0 or higher
+- **Node.js** v24.15.0 or higher
 - **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 v22.14.0+ requirement
+   - Validates Node.js v24.15.0+ requirement
    - Provides helpful error messages for missing dependencies
 
 ## Quick Start

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codecritique",
-  "version": "1.2.4",
+  "version": "2.0.1",
   "description": "AI-powered code review tool for any programming language",
   "type": "module",
   "main": "src/index.js",
@@ -53,43 +53,45 @@
   "dependencies": {
     "@anthropic-ai/sdk": "0.71.0",
     "@huggingface/transformers": "3.8.0",
-    "@lancedb/lancedb": "0.22.3",
+    "@lancedb/lancedb": "0.30.0",
     "@octokit/rest": "22.0.1",
+    "apache-arrow": "18.1.0",
     "chalk": "5.6.2",
     "cli-spinner": "0.2.10",
     "commander": "14.0.1",
-    "dotenv": "17.2.3",
-    "fastembed": "2.0.0",
-    "glob": "13.0.0",
-    "linguist-languages": "9.1.0",
-    "lru-cache": "11.2.2",
-    "minimatch": "10.1.1",
+    "dotenv": "17.4.2",
+    "fastembed": "2.1.0",
+    "glob": "13.0.6",
+    "linguist-languages": "9.3.2",
+    "lru-cache": "11.5.1",
+    "minimatch": "10.2.5",
     "stopwords-iso": "1.1.0"
   },
   "devDependencies": {
     "@eslint/js": "9.39.1",
     "@semantic-release/commit-analyzer": "13.0.1",
-    "@semantic-release/github": "12.0.3",
-    "@semantic-release/npm": "13.1.3",
-    "@semantic-release/release-notes-generator": "14.1.0",
-    "@types/node": "24.9.1",
-    "@vitest/coverage-v8": "4.0.16",
-    "@vitest/eslint-plugin": "1.3.23",
+    "@semantic-release/github": "12.0.8",
+    "@semantic-release/npm": "13.1.5",
+    "@semantic-release/release-notes-generator": "14.1.1",
+    "@types/node": "24.12.4",
+    "@vitest/coverage-v8": "4.1.8",
+    "@vitest/eslint-plugin": "1.6.19",
     "eslint": "9.39.1",
     "eslint-plugin-import": "2.32.0",
     "globals": "16.5.0",
     "knip": "5.70.2",
-    "prettier": "3.7.1",
+    "prettier": "3.8.3",
+    "prettier-plugin-brace-style": "0.10.1",
     "typescript": "5.9.3",
-    "vitest": "4.0.16"
+    "vitest": "4.1.8"
   },
   "volta": {
-    "node": "22.14.0",
-    "npm": "10.9.2"
+    "node": "24.15.0",
+    "npm": "11.12.1"
   },
   "engines": {
-    "node": ">=22.14.0",
-    "npm": "10.x"
+    "node": ">=24.15.0",
+    "npm": ">=11.12.1 <12"
   },
   "engine-strict": true,
   "publishConfig": {

package/src/content-retrieval.js

@@ -23,6 +23,7 @@ import { calculateCosineSimilarity, calculatePathSimilarity } from './embeddings
 import { inferContextFromDocumentContent } from './utils/context-inference.js';
 import { isGenericDocument, getGenericDocumentContext } from './utils/document-detection.js';
 import { isDocumentationFile } from './utils/file-validation.js';
+import { getTableSchema, schemaHasField } from './utils/lancedb.js';
 import { debug, verboseLog } from './utils/logging.js';
 import { isPathWithinProject } from './utils/path-utils.js';
 import { escapeSqlString } from './utils/string-utils.js';
@@ -93,7 +94,8 @@ export class ContentRetriever {
           try {
             await fs.promises.access(absolutePath, fs.constants.F_OK);
             return { index, exists: true };
-          } catch {
+          }
+          catch {
             debug(`Filtering out non-existent project file: ${resultPath}`);
             return { index, exists: false };
           }
@@ -151,12 +153,13 @@ export class ContentRetriever {
 
       const resolvedProjectPath = path.resolve(projectPath);
       try {
-        const tableSchema = await table.schema;
-        if (tableSchema?.fields?.some((field) => field.name === 'project_path')) {
+        const tableSchema = await getTableSchema(table);
+        if (schemaHasField(tableSchema, 'project_path')) {
           query = query.where(`project_path = '${escapeSqlString(resolvedProjectPath)}'`);
           debug(`Filtering documentation by project_path: ${resolvedProjectPath}`);
         }
-      } catch (schemaError) {
+      }
+      catch (schemaError) {
         debug(`Could not check schema for project_path field: ${schemaError.message}`);
       }
 
@@ -174,9 +177,11 @@ export class ContentRetriever {
         let similarity;
         if (result._distance !== undefined) {
           similarity = Math.max(0, Math.min(1, 1 - result._distance));
-        } else if (result._score !== undefined) {
+        }
+        else if (result._score !== undefined) {
           similarity = Math.max(0, Math.min(1, result._score));
-        } else {
+        }
+        else {
           similarity = 0.5;
         }
 
@@ -286,7 +291,8 @@ export class ContentRetriever {
                     // Use pre-computed context for generic documents (README, RUNBOOK, etc.)
                     context = getGenericDocumentContext(originalPath, docH1);
                     debug(`[FAST-PATH] Using pre-computed context for generic document: ${originalPath}`);
-                  } else {
+                  }
+                  else {
                     // Use the expensive inference for non-generic documents
                     context = await inferContextFromDocumentContent(
                       originalPath,
@@ -297,7 +303,8 @@ export class ContentRetriever {
                   }
 
                   return context;
-                } catch (error) {
+                }
+                catch (error) {
                   debug(`[ERROR] Failed to get context for ${originalPath}: ${error.message}`);
                   // Return a fallback context to avoid breaking the pipeline
                   return {
@@ -367,7 +374,8 @@ export class ContentRetriever {
                   contextMatchBonus += MODERATE_BOOST_TECH_MATCH;
                 }
               }
-            } else if (queryContextForReranking.area !== 'GeneralJS_TS') {
+            }
+            else if (queryContextForReranking.area !== 'GeneralJS_TS') {
               contextMatchBonus += HEAVY_PENALTY_AREA_MISMATCH;
             }
           }
@@ -420,7 +428,8 @@ export class ContentRetriever {
       verboseLog(options, chalk.green(`Returning ${finalResults.length} documentation results`));
 
       return finalResults;
-    } catch (error) {
+    }
+    catch (error) {
       console.error(chalk.red(`Error in findRelevantDocs: ${error.message}`), error);
       throw new EmbeddingError(`Documentation search failed: ${error.message}`);
     }
@@ -477,7 +486,8 @@ export class ContentRetriever {
           // Only include test files
           conditions.push(`(path LIKE '%.test.%' OR path LIKE '%.spec.%' OR path LIKE '%_test.py' OR path LIKE 'test_%.py')`);
           verboseLog(options, chalk.blue(`Filtering to include only test files.`));
-        } else {
+        }
+        else {
           // Exclude test files
           conditions.push(
             `(path NOT LIKE '%.test.%' AND path NOT LIKE '%.spec.%' AND path NOT LIKE '%_test.py' AND path NOT LIKE 'test_%.py')`
@@ -509,17 +519,14 @@ export class ContentRetriever {
       // Add project path filtering if the field exists in the schema
       // Check if the table has project_path field
       try {
-        const tableSchema = await table.schema;
-        if (tableSchema && tableSchema.fields) {
-          const hasProjectPathField = tableSchema.fields.some((field) => field.name === 'project_path');
-
-          if (hasProjectPathField) {
-            // Use exact match for project path
-            conditions.push(`project_path = '${escapeSqlString(resolvedProjectPath)}'`);
-            debug(`Filtering by project_path: ${resolvedProjectPath}`);
-          }
+        const tableSchema = await getTableSchema(table);
+        if (schemaHasField(tableSchema, 'project_path')) {
+          // Use exact match for project path
+          conditions.push(`project_path = '${escapeSqlString(resolvedProjectPath)}'`);
+          debug(`Filtering by project_path: ${resolvedProjectPath}`);
         }
-      } catch (schemaError) {
+      }
+      catch (schemaError) {
         debug(`Could not check schema for project_path field: ${schemaError.message}`);
         // Continue without project_path filtering in query
       }
@@ -548,10 +555,12 @@ export class ContentRetriever {
           // Vector search distance (0 = perfect match, higher = less similar)
           // Apply more precise normalization to avoid all scores being 1.000
           similarity = Math.max(0, Math.min(1, Math.exp(-result._distance * 2)));
-        } else if (result._score !== undefined) {
+        }
+        else if (result._score !== undefined) {
           // FTS or hybrid score - normalize to 0-1 range with better scaling
           similarity = Math.max(0, Math.min(1, result._score / Math.max(result._score, 1)));
-        } else {
+        }
+        else {
           // Fallback
           similarity = 0.5;
         }
@@ -612,7 +621,8 @@ export class ContentRetriever {
               }
             }
           }
-        } catch (error) {
+        }
+        catch (error) {
           console.warn(chalk.yellow(`Project structure inclusion failed: ${error.message}`));
         }
       }
@@ -625,7 +635,8 @@ export class ContentRetriever {
 
       verboseLog(options, chalk.green(`Returning ${finalResults.length} optimized hybrid search results`));
       return finalResults;
-    } catch (error) {
+    }
+    catch (error) {
       console.error(chalk.red(`Error in optimized findSimilarCode: ${error.message}`), error);
       return [];
     }
@@ -680,7 +691,8 @@ export class ContentRetriever {
       };
 
       verboseLog({}, chalk.green('ContentRetriever cleanup complete'));
-    } finally {
+    }
+    finally {
       this.cleaningUp = false;
     }
   }

package/src/custom-documents.js

@@ -70,7 +70,8 @@ export class CustomDocumentProcessor {
       const headerMatch = content.match(/^#\s+(.+)$/m);
       if (headerMatch) {
         documentTitle = headerMatch[1].trim();
-      } else {
+      }
+      else {
         // If no header found, try to extract filename from title like "instruction:./FILENAME.md"
         const filePathMatch = title.match(/:\.\/([^/]+)\.([a-zA-Z]+)$/);
         if (filePathMatch) {
@@ -88,7 +89,9 @@ export class CustomDocumentProcessor {
 
       for (let i = 0; i < sections.length; i++) {
         const section = sections[i].trim();
-        if (!section) continue;
+        if (!section) {
+          continue;
+        }
 
         // Check if adding this section would exceed max chunk size
         if (currentChunk.length + section.length > maxChunkSize && currentChunk.length > minChunkSize) {
@@ -108,7 +111,8 @@ export class CustomDocumentProcessor {
 
           chunkIndex++;
           currentChunk = section;
-        } else {
+        }
+        else {
           // Add section to current chunk
           currentChunk += (currentChunk ? '\n\n' : '') + section;
         }
@@ -142,7 +146,8 @@ export class CustomDocumentProcessor {
 
       verboseLog({}, chalk.gray(`  Chunked document "${documentTitle}" into ${chunks.length} chunks`));
       return chunks;
-    } catch (error) {
+    }
+    catch (error) {
       console.error(chalk.red(`Error chunking document: ${error.message}`));
       throw new EmbeddingError(`Document chunking failed: ${error.message}`);
     }
@@ -195,7 +200,8 @@ export class CustomDocumentProcessor {
                 project_path: path.resolve(projectPath),
                 created_at: new Date().toISOString(),
               };
-            } else {
+            }
+            else {
               console.error(chalk.red(`Error generating embedding for chunk ${chunk.id}: batch processing failed`));
               return null;
             }
@@ -207,7 +213,8 @@ export class CustomDocumentProcessor {
 
           verboseLog({}, chalk.gray(`    Generated embeddings for ${validChunks.length}/${chunks.length} chunks`));
           this.performanceMetrics.embeddingsCalculated += validChunks.length;
-        } catch (error) {
+        }
+        catch (error) {
           console.error(chalk.red(`Error in batch embedding generation for document ${doc.title}: ${error.message}`));
           // Fallback to individual processing for this document
           verboseLog({}, chalk.yellow(`    Falling back to individual processing for ${doc.title}`));
@@ -225,7 +232,8 @@ export class CustomDocumentProcessor {
                   project_path: path.resolve(projectPath),
                   created_at: new Date().toISOString(),
                 };
-              } catch (error) {
+              }
+              catch (error) {
                 console.error(chalk.red(`Error generating embedding for chunk ${chunk.id}: ${error.message}`));
                 return null;
               }
@@ -254,7 +262,8 @@ export class CustomDocumentProcessor {
 
       verboseLog({}, chalk.green(`Successfully processed ${allChunks.length} custom document chunks (${Date.now() - startTime}ms)`));
       return allChunks;
-    } catch (error) {
+    }
+    catch (error) {
       console.error(chalk.red(`Error processing custom documents: ${error.message}`));
       throw new EmbeddingError(`Custom document processing failed: ${error.message}`);
     }
@@ -327,7 +336,8 @@ export class CustomDocumentProcessor {
       }
 
       return filteredResults;
-    } catch (error) {
+    }
+    catch (error) {
       console.error(chalk.red(`Error searching custom document chunks: ${error.message}`));
       throw new EmbeddingError(`Custom document search failed: ${error.message}`);
     }
@@ -360,7 +370,8 @@ export class CustomDocumentProcessor {
 
       debug(`[getExistingChunks] No existing chunks found for project: ${resolvedProjectPath}`);
       return [];
-    } catch (error) {
+    }
+    catch (error) {
       debug(`[getExistingChunks] Error checking existing chunks: ${error.message}`);
       return [];
     }
@@ -419,7 +430,8 @@ export class CustomDocumentProcessor {
               contextMatchBonus += MODERATE_BOOST_TECH_MATCH;
             }
           }
-        } else if (queryArea !== 'GeneralJS_TS') {
+        }
+        else if (queryArea !== 'GeneralJS_TS') {
           contextMatchBonus += HEAVY_PENALTY_AREA_MISMATCH;
         }
       }
@@ -505,7 +517,8 @@ export class CustomDocumentProcessor {
             this.h1EmbeddingCache.set(docTitlesToCalculate[i], titleEmbeddings[i]);
           }
         }
-      } catch (error) {
+      }
+      catch (error) {
         debug(`[OPTIMIZATION] Error in batch title embedding calculation: ${error.message}`);
         // Continue without title embeddings
       }
@@ -522,7 +535,8 @@ export class CustomDocumentProcessor {
       this.customDocumentChunks.delete(resolvedProjectPath);
       this.cacheManager.customDocumentChunks.delete(resolvedProjectPath);
       verboseLog({}, chalk.green(`Cleared custom document chunks for project: ${resolvedProjectPath}`));
-    } catch (error) {
+    }
+    catch (error) {
       console.error(chalk.red(`Error clearing project chunks: ${error.message}`));
     }
   }
@@ -590,7 +604,8 @@ export class CustomDocumentProcessor {
       };
 
       verboseLog({}, chalk.green('CustomDocumentProcessor cleanup complete'));
-    } finally {
+    }
+    finally {
       this.cleaningUp = false;
     }
   }

package/src/feedback-loader.js

@@ -68,7 +68,8 @@ export async function loadFeedbackData(feedbackPath, options = {}) {
           totalItems += itemCount;
           verboseLog(verbose, chalk.cyan(`📋 Loaded feedback from ${file}: ${itemCount} items`));
         }
-      } catch (parseError) {
+      }
+      catch (parseError) {
         console.warn(chalk.yellow(`⚠️ Error parsing feedback file ${file}: ${parseError.message}`));
       }
     }
@@ -79,7 +80,8 @@ export async function loadFeedbackData(feedbackPath, options = {}) {
     }
 
     return {};
-  } catch (error) {
+  }
+  catch (error) {
     console.error(chalk.red(`❌ Error loading feedback data: ${error.message}`));
     return {};
   }
@@ -112,7 +114,8 @@ export async function initializeSemanticSimilarity() {
     semanticSimilarityInitialized = true;
     semanticSimilarityAvailable = true;
     verboseLog({}, chalk.green('[FeedbackLoader] Semantic similarity initialized using embeddings system'));
-  } catch (error) {
+  }
+  catch (error) {
     console.warn(chalk.yellow(`[FeedbackLoader] Semantic similarity initialization failed: ${error.message}`));
     semanticSimilarityAvailable = false;
   }
@@ -154,7 +157,8 @@ async function calculateSemanticSimilarity(text1, text2) {
     const similarity = calculateCosineSimilarity(embedding1, embedding2);
     // Cosine similarity ranges from -1 to 1, normalize to 0-1
     return (similarity + 1) / 2;
-  } catch (error) {
+  }
+  catch (error) {
     console.warn(chalk.yellow(`[FeedbackLoader] Semantic similarity calculation failed: ${error.message}`));
     return null;
   }
@@ -207,7 +211,9 @@ export async function shouldSkipSimilarIssue(issueDescription, feedbackData, opt
 
   // Check similarity with dismissed issues
   for (const dismissed of dismissedIssues) {
-    if (!dismissed.originalIssue) continue;
+    if (!dismissed.originalIssue) {
+      continue;
+    }
 
     let similarity;
     let similarityMethod;
@@ -222,7 +228,8 @@ export async function shouldSkipSimilarIssue(issueDescription, feedbackData, opt
         similarity = calculateWordSimilarity(issueDescription, dismissed.originalIssue);
         similarityMethod = 'word-based';
       }
-    } else {
+    }
+    else {
       // Use word-based similarity
       similarity = calculateWordSimilarity(issueDescription, dismissed.originalIssue);
       similarityMethod = 'word-based';
@@ -297,7 +304,9 @@ export async function calculateIssueSimilarity(text1, text2, options = {}) {
  * @returns {number} Similarity score (0-1)
  */
 export function calculateWordSimilarity(text1, text2) {
-  if (!text1 || !text2) return 0;
+  if (!text1 || !text2) {
+    return 0;
+  }
 
   // Normalize and tokenize
   const normalize = (text) =>
@@ -310,7 +319,9 @@ export function calculateWordSimilarity(text1, text2) {
   const words1 = new Set(normalize(text1));
   const words2 = new Set(normalize(text2));
 
-  if (words1.size === 0 || words2.size === 0) return 0;
+  if (words1.size === 0 || words2.size === 0) {
+    return 0;
+  }
 
   // Calculate Jaccard similarity (intersection over union)
   const intersection = [...words1].filter((word) => words2.has(word)).length;