codeant-cli 0.1.5 → 0.1.7

package/src/utils/reviewApiHelper.js

@@ -1,139 +0,0 @@
-import CommonApiHelper from './commonApiHelper.js';
-import path from 'path';
-import fs from 'fs';
-
-const MAX_REVIEW_FILES = 10;
-
-const EXCLUDED_FILENAMES = new Set([
-  'package-lock.json',
-  'yarn.lock',
-  'pnpm-lock.yaml',
-  'composer.lock',
-  'Gemfile.lock',
-  'Cargo.lock',
-  'poetry.lock',
-  'Pipfile.lock',
-  'go.sum',
-  '.DS_Store',
-  'Thumbs.db',
-]);
-
-const EXCLUDED_EXTENSIONS = new Set([
-  // Images
-  '.png', '.jpg', '.jpeg', '.gif', '.bmp', '.ico', '.svg', '.webp', '.avif',
-  // Fonts
-  '.woff', '.woff2', '.ttf', '.eot', '.otf',
-  // Binary / compiled
-  '.exe', '.dll', '.so', '.dylib', '.bin', '.o', '.a', '.class', '.pyc', '.pyo',
-  '.wasm', '.jar', '.war', '.ear',
-  // Archives
-  '.zip', '.tar', '.gz', '.bz2', '.7z', '.rar',
-  // Media
-  '.mp3', '.mp4', '.wav', '.avi', '.mov', '.flv', '.wmv', '.ogg', '.webm',
-  // Data / generated
-  '.min.js', '.min.css', '.map',
-  '.pb', '.proto.bin',
-  // PDF / docs
-  '.pdf', '.doc', '.docx', '.xls', '.xlsx', '.ppt', '.pptx',
-]);
-
-function isReviewableFile(filePath) {
-  const basename = path.basename(filePath);
-
-  if (EXCLUDED_FILENAMES.has(basename)) return false;
-
-  const ext = path.extname(basename).toLowerCase();
-  if (EXCLUDED_EXTENSIONS.has(ext)) return false;
-
-  // Check for .min.js / .min.css (double extension)
-  if (basename.endsWith('.min.js') || basename.endsWith('.min.css')) return false;
-
-  return true;
-}
-
-/**
- * Transforms git diff data into the format expected by the agentic review API
- */
-class ReviewApiHelper extends CommonApiHelper {
-  /**
-   * Pass structured diffs through so buildReviewApiRequest can filter them
-   */
-  _transformDiffsToApiFormat(diffs) {
-    return diffs;
-  }
-
-  /**
-   * Build the complete request body for the agentic review API
-   */
-  async buildReviewApiRequest(type = 'staged-only', includePatterns = [], excludePatterns = [], options = {}) {
-    const diffs = await this.getFilesForType(type, options);
-
-    // diffs is either a string (from getAllDiffs) or an array of structured diff objects
-    if (typeof diffs === 'string') {
-      // Extract filenames from diff headers and read their contents
-      const gitRoot = this.getGitRoot() || process.cwd();
-      const fileContents = {};
-      const fileMatches = diffs.matchAll(/^diff --git a\/.+ b\/(.+)$/gm);
-      for (const match of fileMatches) {
-        const f = match[1];
-        if (!isReviewableFile(f)) continue;
-        try {
-          const fullPath = path.resolve(gitRoot, f);
-          if (fs.existsSync(fullPath)) {
-            fileContents[f] = fs.readFileSync(fullPath, 'utf-8');
-          }
-        } catch {
-          // Skip files that can't be read
-        }
-      }
-      return { diff_content: diffs, file_contents: fileContents };
-    }
-
-    // Filter to reviewable files only
-    const reviewable = diffs.filter(d => isReviewableFile(d.filename_str));
-
-    // Dedupe by filename, keeping all hunks, but limit unique files to MAX_REVIEW_FILES
-    const fileOrder = [];
-    const fileHunks = new Map();
-    for (const d of reviewable) {
-      const f = d.filename_str;
-      if (!fileHunks.has(f)) {
-        if (fileOrder.length >= MAX_REVIEW_FILES) continue;
-        fileOrder.push(f);
-        fileHunks.set(f, []);
-      }
-      fileHunks.get(f).push(d);
-    }
-
-    // Reconstruct a combined diff string from the filtered hunks
-    const parts = [];
-    for (const f of fileOrder) {
-      const hunks = fileHunks.get(f);
-      // Build a file-level diff header + all hunk patches
-      const first = hunks[0];
-      const header = `diff --git a/${f} b/${f}\n--- a/${f}\n+++ b/${f}`;
-      const hunkPatches = hunks.map(h => h.patch_str).filter(Boolean);
-      if (hunkPatches.length > 0) {
-        parts.push(header + '\n' + hunkPatches.join('\n'));
-      }
-    }
-
-    // Read full file contents for each changed file
-    const gitRoot = this.getGitRoot() || process.cwd();
-    const fileContents = {};
-    for (const f of fileOrder) {
-      try {
-        const fullPath = path.resolve(gitRoot, f);
-        if (fs.existsSync(fullPath)) {
-          fileContents[f] = fs.readFileSync(fullPath, 'utf-8');
-        }
-      } catch {
-        // Skip files that can't be read
-      }
-    }
-
-    return { diff_content: parts.join('\n'), file_contents: fileContents };
-  }
-}
-
-export default ReviewApiHelper;

package/src/utils/secretsApiHelper.js

@@ -1,134 +0,0 @@
-import CommonApiHelper from './commonApiHelper.js';
-import { minimatch } from 'minimatch';
-
-/**
- * Transforms git diff data into the format expected by the secrets detection API
- *
- * API Input Format:
- * {
- *   "files": [
- *     {
- *       "file_path": str,
- *       "code": str,
- *       "diffs": [{ "start_line": int, "end_line": int }]
- *     }
- *   ]
- * }
- */
-class SecretsApiHelper extends CommonApiHelper {
-  /**
-   * Transform diff info array to API format
-   * Groups by file and extracts diff ranges
-   */
-  _transformDiffsToApiFormat(diffs) {
-    // Group diffs by filename
-    const fileMap = new Map();
-
-    for (const diff of diffs) {
-      const filePath = diff.filename_str;
-
-      // Skip diffs with missing or invalid filename
-      if (!filePath) {
-        continue;
-      }
-
-      if (!fileMap.has(filePath)) {
-        fileMap.set(filePath, {
-          file_path: filePath,
-          code: diff.head_file_str || '',
-          diffs: []
-        });
-      }
-
-      // Add diff range if available
-      if (diff.start_line_str && diff.end_line_str) {
-        const startLine = parseInt(diff.start_line_str, 10);
-        const endLine = parseInt(diff.end_line_str, 10);
-
-        if (!isNaN(startLine) && !isNaN(endLine)) {
-          fileMap.get(filePath).diffs.push({
-            start_line: startLine,
-            end_line: endLine
-          });
-        }
-      }
-    }
-
-    return Array.from(fileMap.values());
-  }
-
-  /**
-   * Filter files based on include and exclude glob patterns
-   * @param {Array} files - Array of file objects with 'file_path' property
-   * @param {Array} includePatterns - Array of glob pattern strings to include
-   * @param {Array} excludePatterns - Array of glob pattern strings to exclude
-   */
-  _filterFiles(files, includePatterns = [], excludePatterns = []) {
-    return files.filter(fileObj => {
-      const filePath = fileObj.file_path;
-
-      // If include patterns are specified, file must match at least one
-      if (includePatterns.length > 0) {
-        const matchesInclude = includePatterns.some(pattern => {
-          try {
-            if (pattern instanceof RegExp) {
-              return pattern.test(filePath);
-            }
-            return minimatch(filePath, pattern, { matchBase: true });
-          } catch (e) {
-            console.warn(`Invalid include pattern: ${pattern}`, e.message);
-            return false;
-          }
-        });
-
-        if (!matchesInclude) {
-          return false;
-        }
-      }
-
-      // If exclude patterns are specified, file must not match any
-      if (excludePatterns.length > 0) {
-        const matchesExclude = excludePatterns.some(pattern => {
-          try {
-            if (pattern instanceof RegExp) {
-              return pattern.test(filePath);
-            }
-            return minimatch(filePath, pattern, { matchBase: true });
-          } catch (e) {
-            console.warn(`Invalid exclude pattern: ${pattern}`, e.message);
-            return false;
-          }
-        });
-
-        if (matchesExclude) {
-          return false;
-        }
-      }
-
-      return true;
-    });
-  }
-
-  /**
-   * Build the complete request body for the secrets API
-   * @param {string} type - Type of scan (staged-only, branch-diff, etc.)
-   * @param {Array} includePatterns - Optional array of glob patterns to include files
-   * @param {Array} excludePatterns - Optional array of glob patterns to exclude files
-   * @param {Object} options - Additional options like lastNCommits
-   */
-  async buildSecretsApiRequest(type = 'staged-only', includePatterns = [], excludePatterns = [], options = {}) {
-    let files = await this.getFilesForType(type, options);
-
-    // Handle null/undefined return
-    if (!files || !Array.isArray(files)) {
-      files = [];
-    }
-
-    // Apply include/exclude filters
-    files = this._filterFiles(files, includePatterns, excludePatterns);
-
-    return { files };
-  }
-}
-
-export default SecretsApiHelper;

package/src/utils/securityAnalysisApiHelper.js

@@ -1,119 +0,0 @@
-import CommonApiHelper from './commonApiHelper.js';
-import { minimatch } from 'minimatch';
-
-/**
- * Transforms git diff data into the format expected by the static analysis API
- *
- * API Input Format:
- * {
- *   "filesData": [
- *     {
- *       "file": str,
- *       "code": str
- *     }
- *   ]
- * }
- */
-class SecurityAnalysisApiHelper extends CommonApiHelper {
-  /**
-   * Transform diff info array to API format
-   * Groups by file (without diff ranges for static analysis)
-   */
-  _transformDiffsToApiFormat(diffs) {
-    // Group diffs by filename
-    const fileMap = new Map();
-
-    for (const diff of diffs) {
-      const filePath = diff.filename_str;
-
-      // Skip diffs with missing or invalid filename
-      if (!filePath) {
-        continue;
-      }
-
-      if (!fileMap.has(filePath)) {
-        fileMap.set(filePath, {
-          file_path: filePath,
-          code: diff.head_file_str || ''
-        });
-      }
-    }
-
-    return Array.from(fileMap.values());
-  }
-
-  /**
-   * Filter files based on include and exclude glob patterns
-   * @param {Array} files - Array of file objects with 'file' property
-   * @param {Array} includePatterns - Array of glob pattern strings to include
-   * @param {Array} excludePatterns - Array of glob pattern strings to exclude
-   */
-  _filterFiles(files, includePatterns = [], excludePatterns = []) {
-    return files.filter(fileObj => {
-      const filePath = fileObj.file;
-
-      // If include patterns are specified, file must match at least one
-      if (includePatterns.length > 0) {
-        const matchesInclude = includePatterns.some(pattern => {
-          try {
-            if (pattern instanceof RegExp) {
-              return pattern.test(filePath);
-            }
-            return minimatch(filePath, pattern, { matchBase: true });
-          } catch (e) {
-            console.warn(`Invalid include pattern: ${pattern}`, e.message);
-            return false;
-          }
-        });
-
-        if (!matchesInclude) {
-          return false;
-        }
-      }
-
-      // If exclude patterns are specified, file must not match any
-      if (excludePatterns.length > 0) {
-        const matchesExclude = excludePatterns.some(pattern => {
-          try {
-            if (pattern instanceof RegExp) {
-              return pattern.test(filePath);
-            }
-            return minimatch(filePath, pattern, { matchBase: true });
-          } catch (e) {
-            console.warn(`Invalid exclude pattern: ${pattern}`, e.message);
-            return false;
-          }
-        });
-
-        if (matchesExclude) {
-          return false;
-        }
-      }
-
-      return true;
-    });
-  }
-
-  /**
-   * Build the complete request body for the static analysis API
-   * @param {string} type - Type of scan (staged-only, branch-diff, etc.)
-   * @param {Array} includePatterns - Optional array of glob patterns to include files
-   * @param {Array} excludePatterns - Optional array of glob patterns to exclude files
-   * @param {Object} options - Additional options like lastNCommits
-   */
-  async buildSecurityAnalysisApiRequest(type = 'staged-only', includePatterns = [], excludePatterns = [], options = {}) {
-    let files = await this.getFilesForType(type, options);
-
-    // Handle null/undefined return
-    if (!files || !Array.isArray(files)) {
-      files = [];
-    }
-
-    // Apply include/exclude filters
-    files = this._filterFiles(files, includePatterns, excludePatterns);
-
-    return { files };
-  }
-}
-
-export default SecurityAnalysisApiHelper;

package/src/utils/staticAnalysisApiHelper.js

@@ -1,119 +0,0 @@
-import CommonApiHelper from './commonApiHelper.js';
-import { minimatch } from 'minimatch';
-
-/**
- * Transforms git diff data into the format expected by the static analysis API
- *
- * API Input Format:
- * {
- *   "filesData": [
- *     {
- *       "file": str,
- *       "code": str
- *     }
- *   ]
- * }
- */
-class StaticAnalysisApiHelper extends CommonApiHelper {
-  /**
-   * Transform diff info array to API format
-   * Groups by file (without diff ranges for static analysis)
-   */
-  _transformDiffsToApiFormat(diffs) {
-    // Group diffs by filename
-    const fileMap = new Map();
-
-    for (const diff of diffs) {
-      const filePath = diff.filename_str;
-
-      // Skip diffs with missing or invalid filename
-      if (!filePath) {
-        continue;
-      }
-
-      if (!fileMap.has(filePath)) {
-        fileMap.set(filePath, {
-          file: filePath,
-          code: diff.head_file_str || ''
-        });
-      }
-    }
-
-    return Array.from(fileMap.values());
-  }
-
-  /**
-   * Filter files based on include and exclude glob patterns
-   * @param {Array} files - Array of file objects with 'file' property
-   * @param {Array} includePatterns - Array of glob pattern strings to include
-   * @param {Array} excludePatterns - Array of glob pattern strings to exclude
-   */
-  _filterFiles(files, includePatterns = [], excludePatterns = []) {
-    return files.filter(fileObj => {
-      const filePath = fileObj.file;
-
-      // If include patterns are specified, file must match at least one
-      if (includePatterns.length > 0) {
-        const matchesInclude = includePatterns.some(pattern => {
-          try {
-            if (pattern instanceof RegExp) {
-              return pattern.test(filePath);
-            }
-            return minimatch(filePath, pattern, { matchBase: true });
-          } catch (e) {
-            console.warn(`Invalid include pattern: ${pattern}`, e.message);
-            return false;
-          }
-        });
-
-        if (!matchesInclude) {
-          return false;
-        }
-      }
-
-      // If exclude patterns are specified, file must not match any
-      if (excludePatterns.length > 0) {
-        const matchesExclude = excludePatterns.some(pattern => {
-          try {
-            if (pattern instanceof RegExp) {
-              return pattern.test(filePath);
-            }
-            return minimatch(filePath, pattern, { matchBase: true });
-          } catch (e) {
-            console.warn(`Invalid exclude pattern: ${pattern}`, e.message);
-            return false;
-          }
-        });
-
-        if (matchesExclude) {
-          return false;
-        }
-      }
-
-      return true;
-    });
-  }
-
-  /**
-   * Build the complete request body for the static analysis API
-   * @param {string} type - Type of scan (staged-only, branch-diff, etc.)
-   * @param {Array} includePatterns - Optional array of glob patterns to include files
-   * @param {Array} excludePatterns - Optional array of glob patterns to exclude files
-   * @param {Object} options - Additional options like lastNCommits
-   */
-  async buildStaticAnalysisApiRequest(type = 'staged-only', includePatterns = [], excludePatterns = [], options = {}) {
-    let files = await this.getFilesForType(type, options);
-
-    // Handle null/undefined return
-    if (!files || !Array.isArray(files)) {
-      files = [];
-    }
-
-    // Apply include/exclude filters
-    files = this._filterFiles(files, includePatterns, excludePatterns);
-
-    return { filesData: files };
-  }
-}
-
-export default StaticAnalysisApiHelper;