korekt-cli 0.11.0 → 0.11.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "korekt-cli",
-  "version": "0.11.0",
+  "version": "0.11.2",
   "description": "AI-powered code review CLI - Keep your kode korekt",
   "main": "src/index.js",
   "bin": {

package/src/git-logic.js

@@ -215,6 +215,14 @@ export async function runUncommittedReview(mode = 'unstaged', fileRulesConfig =
       }
     }
 
+    // Check if file count exceeds max_files_for_content limit
+    if (config.max_files_for_content && fileList.length > config.max_files_for_content) {
+      throw new Error(
+        `Too many files: ${fileList.length} files exceeds the limit of ${config.max_files_for_content}. ` +
+          `Please reduce the number of changed files before reviewing.`
+      );
+    }
+
     // Check if this is a large change set (only if config has large_pr_threshold)
     const isLargePr = config.large_pr_threshold && fileList.length > config.large_pr_threshold;
     if (isLargePr) {
@@ -548,6 +556,14 @@ export async function runLocalReview(
       }
     }
 
+    // Check if file count exceeds max_files_for_content limit
+    if (config.max_files_for_content && filteredFileList.length > config.max_files_for_content) {
+      throw new Error(
+        `Too many files: ${filteredFileList.length} files exceeds the limit of ${config.max_files_for_content}. ` +
+          `Please split this PR into smaller reviews or use --ignore to exclude some files.`
+      );
+    }
+
     // Check if this is a large PR (only if config has large_pr_threshold)
     const isLargePr =
       config.large_pr_threshold && filteredFileList.length > config.large_pr_threshold;

package/src/index.js

@@ -22,6 +22,28 @@ export { detectCIProvider, truncateFileData, formatErrorOutput, getPrUrl } from
 const require = createRequire(import.meta.url);
 const { version } = require('../package.json');
 
+/**
+ * Fetch file processing rules from the API.
+ * These rules control which files to skip, which to show diff-only, etc.
+ * @param {string} apiEndpoint - The API endpoint URL
+ * @param {string} apiKey - The API key for authentication
+ * @returns {Promise<Object|null>} - Config object or null if fetch fails
+ */
+async function fetchFileRulesConfig(apiEndpoint, apiKey) {
+  try {
+    const configEndpoint = apiEndpoint.replace(/\/review\/?$/, '/config/file-rules');
+    const response = await axios.get(configEndpoint, {
+      headers: { Authorization: `Bearer ${apiKey}` },
+      timeout: 3000,
+    });
+    return response.data;
+  } catch {
+    // Non-fatal: continue without filtering if config unavailable
+    log(chalk.yellow('Warning: Could not fetch file rules config.'));
+    return null;
+  }
+}
+
 /**
  * Helper functions for clean output separation:
  * - log() writes to stderr (progress, info, errors)
@@ -157,8 +179,11 @@ program
       process.exit(1);
     }
 
+    // Fetch file rules config from API
+    const fileRulesConfig = await fetchFileRulesConfig(apiEndpoint, apiKey);
+
     // Gather all data using our git logic module
-    const payload = await runLocalReview(targetBranch, options.ignore);
+    const payload = await runLocalReview(targetBranch, options.ignore, fileRulesConfig);
 
     if (!payload) {
       log(chalk.red('Could not proceed with review due to errors during analysis.'));
@@ -334,8 +359,11 @@ async function reviewUncommitted(mode, options) {
     process.exit(1);
   }
 
+  // Fetch file rules config from API
+  const fileRulesConfig = await fetchFileRulesConfig(apiEndpoint, apiKey);
+
   const { runUncommittedReview } = await import('./git-logic.js');
-  const payload = await runUncommittedReview(mode);
+  const payload = await runUncommittedReview(mode, fileRulesConfig);
 
   if (!payload) {
     log(chalk.red('No changes found or error occurred during analysis.'));

package/src/index.test.js

@@ -153,6 +153,36 @@ describe('CLI JSON output mode', () => {
       expect(displayFile.diff).not.toContain('truncated');
       expect(displayFile.content).not.toContain('truncated');
     });
+
+    it('should handle diff-only files without content', () => {
+      const file = {
+        path: 'test.js',
+        status: 'M',
+        diff: 'some diff here',
+        // no content field - this is a diff-only file
+      };
+
+      const displayFile = truncateFileData(file);
+
+      expect(displayFile.path).toBe('test.js');
+      expect(displayFile.status).toBe('M');
+      expect(displayFile.diff).toBe('some diff here');
+      expect(displayFile.content).toBeUndefined();
+    });
+
+    it('should handle files with neither diff nor content', () => {
+      const file = {
+        path: 'deleted.js',
+        status: 'D',
+      };
+
+      const displayFile = truncateFileData(file);
+
+      expect(displayFile.path).toBe('deleted.js');
+      expect(displayFile.status).toBe('D');
+      expect(displayFile.diff).toBeUndefined();
+      expect(displayFile.content).toBeUndefined();
+    });
   });
 
   describe('error formatting for JSON mode', () => {

package/src/utils.js

@@ -73,19 +73,27 @@ export function getPrUrl() {
  * @returns {Object} File object with truncated diff and content
  */
 export function truncateFileData(file, maxLength = 500) {
-  return {
+  const result = {
     path: file.path,
     status: file.status,
     ...(file.old_path && { old_path: file.old_path }),
-    diff:
+  };
+
+  if (file.diff) {
+    result.diff =
       file.diff.length > maxLength
         ? `${file.diff.substring(0, maxLength)}... [truncated ${file.diff.length - maxLength} chars]`
-        : file.diff,
-    content:
+        : file.diff;
+  }
+
+  if (file.content) {
+    result.content =
       file.content.length > maxLength
         ? `${file.content.substring(0, maxLength)}... [truncated ${file.content.length - maxLength} chars]`
-        : file.content,
-  };
+        : file.content;
+  }
+
+  return result;
 }
 
 /**