@contextrail/code-review-agent 0.1.1 → 0.1.2-alpha.1

package/dist/review-inputs/index.js

@@ -1,7 +1,8 @@
-import { access, mkdir, writeFile } from 'node:fs/promises';
+import { access, mkdir, readFile, writeFile } from 'node:fs/promises';
 import path from 'node:path';
 import { gitDiffProvider } from './git-diff-provider.js';
-import { filterDiffInputs, filterFiles } from './filtering.js';
+import { DEFAULT_EXCLUDE_PATTERNS } from '../config/defaults.js';
+import { filterDiffInputs, filterFiles, parseGitignoreContent } from './filtering.js';
 export { triagePr } from './triage.js';
 export const isDiffInputs = (inputs) => inputs.mode === 'diff';
 export const isFileListInputs = (inputs) => inputs.mode === 'file-list';
@@ -45,6 +46,43 @@ const validateFileList = async (files, basePath) => {
 const normalizeFiles = (files) => {
     return files.map((file) => file.trim()).filter(Boolean);
 };
+const mergeFilteringWithRootGitignore = async (filtering, repoRoot) => {
+    if (!repoRoot) {
+        return filtering;
+    }
+    const gitignorePath = path.join(repoRoot, '.gitignore');
+    try {
+        const gitignoreContents = await readFile(gitignorePath, 'utf-8');
+        if (typeof gitignoreContents !== 'string' || gitignoreContents.length === 0) {
+            return filtering;
+        }
+        const parsed = parseGitignoreContent(gitignoreContents);
+        if (parsed.excludePatterns.length === 0 && parsed.includePatterns.length === 0) {
+            return filtering;
+        }
+        const excludePatterns = filtering?.excludePatterns
+            ? [...filtering.excludePatterns, ...parsed.excludePatterns]
+            : [...DEFAULT_EXCLUDE_PATTERNS, ...parsed.excludePatterns];
+        const includePatterns = [...(filtering?.includePatterns ?? []), ...parsed.includePatterns];
+        return {
+            ...filtering,
+            excludePatterns,
+            includePatterns,
+        };
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        const code = typeof error === 'object' && error !== null && 'code' in error
+            ? String(error.code)
+            : undefined;
+        const isMissingFileMessage = message.toLowerCase().includes('no such file') || message.toLowerCase().includes('file not found');
+        // Missing .gitignore is expected in many repos; keep defaults without failing.
+        if (code === 'ENOENT' || isMissingFileMessage) {
+            return filtering;
+        }
+        throw new Error(`Failed to read root .gitignore at "${gitignorePath}": ${message}`);
+    }
+};
 export const buildReviewInputs = async (options, deps) => {
     if (options.mode === 'diff') {
         const diffProvider = deps?.diffProvider ?? gitDiffProvider;
@@ -53,8 +91,9 @@ export const buildReviewInputs = async (options, deps) => {
             from: options.from,
             to: options.to,
         });
+        const mergedFiltering = await mergeFilteringWithRootGitignore(options.filtering, options.repoPath);
         const normalizedFiles = normalizeFiles(result.files);
-        const { files, diffs } = filterDiffInputs(normalizedFiles, result.diffs, options.filtering);
+        const { files, diffs } = filterDiffInputs(normalizedFiles, result.diffs, mergedFiltering);
         if (files.length === 0) {
             throw new Error('Diff mode produced no files to review after filtering.');
         }
@@ -76,8 +115,9 @@ export const buildReviewInputs = async (options, deps) => {
         }
         return inputs;
     }
+    const mergedFiltering = await mergeFilteringWithRootGitignore(options.filtering, options.basePath);
     const normalizedFiles = normalizeFiles(options.files);
-    const files = filterFiles(normalizedFiles, options.filtering);
+    const files = filterFiles(normalizedFiles, mergedFiltering);
     if (files.length === 0) {
         throw new Error('File list produced no files to review after filtering.');
     }
@@ -108,15 +148,21 @@ export const buildReviewInputs = async (options, deps) => {
     return inputs;
 };
 export const persistReviewInputs = async (inputs, outputDir) => {
-    await mkdir(outputDir, { recursive: true });
-    const filesPayload = JSON.stringify({ mode: inputs.mode, files: inputs.files }, null, 2);
-    await writeFile(path.join(outputDir, 'files.json'), filesPayload);
-    if (inputs.mode !== 'diff') {
-        return;
+    try {
+        await mkdir(outputDir, { recursive: true });
+        const filesPayload = JSON.stringify({ mode: inputs.mode, files: inputs.files }, null, 2);
+        await writeFile(path.join(outputDir, 'files.json'), filesPayload);
+        if (inputs.mode !== 'diff') {
+            return;
+        }
+        for (const [filePath, diff] of Object.entries(inputs.diffs)) {
+            const diffPath = path.join(outputDir, 'diff', `${filePath}.diff`);
+            await mkdir(path.dirname(diffPath), { recursive: true });
+            await writeFile(diffPath, diff);
+        }
     }
-    for (const [filePath, diff] of Object.entries(inputs.diffs)) {
-        const diffPath = path.join(outputDir, 'diff', `${filePath}.diff`);
-        await mkdir(path.dirname(diffPath), { recursive: true });
-        await writeFile(diffPath, diff);
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        throw new Error(`Failed to persist review inputs to "${outputDir}": ${message}`);
     }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@contextrail/code-review-agent",
-    "version": "0.1.1",
+    "version": "0.1.2-alpha.1",
     "description": "CLI tool for orchestrating ContextRail-powered code reviews",
     "homepage": "https://contextrail.app",
     "repository": {