@codacy/analysis-runner 0.11.0 → 0.12.0

package/dist/index.d.ts

@@ -27,7 +27,7 @@ interface AnalyzeOptions {
     outputFormat?: OutputFormat;
     /** Execution mode. Defaults to "standard". */
     mode?: ExecutionMode;
-    /** Maximum time in ms for each tool invocation. Defaults to 600000 (10 minutes). */
+    /** Maximum time in ms for each tool invocation. Defaults to 900000 (15 minutes). */
     toolTimeout?: number;
     /** Maximum number of tools to run in parallel. 1 = sequential. Defaults to 1. */
     parallelTools?: number;
@@ -601,11 +601,10 @@ interface PatternMetadataLookup {
 declare function computePatternSummary(issues: readonly Issue[], lookup: ReadonlyMap<string, PatternMetadataLookup>): PatternSummary;
 
 /**
- * File discovery and routing — finds files in a repository, applies exclusion
- * layers (.gitignore, global exclude, per-tool exclude), and routes files to
- * the appropriate tool adapters based on file patterns.
+ * Git-scope file resolution — resolves the set of files implied by the
+ * `--staged`, `--diff`, and `--pr` flags. Each helper returns paths relative
+ * to the repository root, excluding deleted files, sorted for stable output.
  */
-
 /**
  * Returns files changed in the git staging area (index), excluding deleted files.
  * Used by `--staged` to analyze only what's about to be committed.

package/dist/index.js

@@ -76,7 +76,7 @@ module.exports = __toCommonJS(index_exports);
 // src/analyze.ts
 var import_os = __toESM(require("os"));
 var import_path3 = __toESM(require("path"));
-var import_child_process2 = require("child_process");
+var import_child_process3 = require("child_process");
 var import_util = require("util");
 var import_tooling3 = require("@codacy/tooling");
 
@@ -238,11 +238,13 @@ function createCodacyConfig(params = {}) {
 }
 
 // src/files.ts
-var import_child_process = require("child_process");
+var import_child_process2 = require("child_process");
 var import_fs = __toESM(require("fs"));
 var import_path2 = __toESM(require("path"));
+var import_minimatch2 = require("minimatch");
+
+// src/glob-matcher.ts
 var import_minimatch = require("minimatch");
-var MAX_FILE_SIZE_BYTES = 5 * 1024 * 1024;
 var PURE_EXTENSION_PATTERN = /^\*\*\/\*(\.[^*/.]+)$/;
 function pureExtensionOf(pattern) {
   const m = PURE_EXTENSION_PATTERN.exec(pattern);
@@ -325,69 +327,9 @@ var CompiledMatcher = class {
     return false;
   }
 };
-function listRepoFiles(repositoryRoot) {
-  try {
-    const tracked = (0, import_child_process.execSync)("git ls-files", {
-      cwd: repositoryRoot,
-      encoding: "utf-8",
-      maxBuffer: 50 * 1024 * 1024
-    }).trim();
-    const untracked = (0, import_child_process.execSync)("git ls-files --others --exclude-standard", {
-      cwd: repositoryRoot,
-      encoding: "utf-8",
-      maxBuffer: 50 * 1024 * 1024
-    }).trim();
-    const files = /* @__PURE__ */ new Set();
-    for (const f of tracked.split("\n")) {
-      if (f) files.add(f);
-    }
-    for (const f of untracked.split("\n")) {
-      if (f) files.add(f);
-    }
-    return Array.from(files).sort();
-  } catch {
-    return [];
-  }
-}
-function filterOversizedFiles(files, repositoryRoot, logger) {
-  const result = [];
-  let skipped = 0;
-  for (const f of files) {
-    try {
-      const stat = import_fs.default.statSync(import_path2.default.join(repositoryRoot, f));
-      if (stat.size > MAX_FILE_SIZE_BYTES) {
-        skipped++;
-        logger?.debug("Skipping oversized file", {
-          file: f,
-          sizeKB: Math.round(stat.size / 1024),
-          limitKB: MAX_FILE_SIZE_BYTES / 1024
-        });
-        continue;
-      }
-    } catch {
-    }
-    result.push(f);
-  }
-  if (skipped > 0) {
-    logger?.info("Filtered oversized files", { skipped, remaining: result.length });
-  }
-  return result;
-}
-function applyExclusions(files, globalExclude, toolExclude) {
-  if (globalExclude.length === 0 && toolExclude.length === 0) return files;
-  const matcher = new CompiledMatcher([...globalExclude, ...toolExclude]);
-  if (matcher.isEmpty) return files;
-  return files.filter((file) => !matcher.matches(file, import_path2.default.basename(file)));
-}
-function routeFilesToTool(files, adapter, toolConfig) {
-  const patterns = [...adapter.filePatterns];
-  if (toolConfig?.extraFilePatterns) {
-    patterns.push(...toolConfig.extraFilePatterns);
-  }
-  const matcher = new CompiledMatcher(patterns);
-  if (matcher.matchAll) return files;
-  return files.filter((file) => matcher.matches(file, import_path2.default.basename(file)));
-}
+
+// src/git-scope.ts
+var import_child_process = require("child_process");
 function listStagedFiles(repositoryRoot) {
   try {
     const output = (0, import_child_process.execSync)("git diff --cached --name-only --diff-filter=d", {
@@ -417,7 +359,7 @@ function listDiffFiles(repositoryRoot, base) {
 }
 function listPrFiles(repositoryRoot) {
   try {
-    const output = (0, import_child_process.execSync)("gh pr view --json files --jq '.files[].path'", {
+    const output = (0, import_child_process.execSync)('gh pr view --json files --jq ".files[].path"', {
       cwd: repositoryRoot,
       encoding: "utf-8",
       maxBuffer: 10 * 1024 * 1024,
@@ -473,11 +415,81 @@ function detectDefaultBranch(repositoryRoot) {
     "Cannot auto-detect default branch: no origin/HEAD, origin/main, or origin/master found. Specify a base explicitly with --diff <base>."
   );
 }
+
+// src/files.ts
+var MAX_FILE_SIZE_BYTES = 150 * 1024;
+function listRepoFiles(repositoryRoot) {
+  try {
+    const tracked = (0, import_child_process2.execSync)("git ls-files", {
+      cwd: repositoryRoot,
+      encoding: "utf-8",
+      maxBuffer: 50 * 1024 * 1024
+    }).trim();
+    const untracked = (0, import_child_process2.execSync)("git ls-files --others --exclude-standard", {
+      cwd: repositoryRoot,
+      encoding: "utf-8",
+      maxBuffer: 50 * 1024 * 1024
+    }).trim();
+    const files = /* @__PURE__ */ new Set();
+    for (const f of tracked.split("\n")) {
+      if (f) files.add(f);
+    }
+    for (const f of untracked.split("\n")) {
+      if (f) files.add(f);
+    }
+    return Array.from(files).sort();
+  } catch {
+    return [];
+  }
+}
+function filterOversizedFiles(files, repositoryRoot, logger) {
+  const result = [];
+  const skipped = [];
+  for (const f of files) {
+    try {
+      const stat = import_fs.default.statSync(import_path2.default.join(repositoryRoot, f));
+      if (stat.size > MAX_FILE_SIZE_BYTES) {
+        skipped.push({ filePath: f, sizeBytes: stat.size });
+        logger?.debug("Skipping oversized file", {
+          file: f,
+          sizeKB: Math.round(stat.size / 1024),
+          limitKB: MAX_FILE_SIZE_BYTES / 1024
+        });
+        continue;
+      }
+    } catch {
+    }
+    result.push(f);
+  }
+  if (skipped.length > 0) {
+    logger?.info("Filtered oversized files", {
+      skipped: skipped.length,
+      remaining: result.length,
+      limitKB: MAX_FILE_SIZE_BYTES / 1024
+    });
+  }
+  return { files: result, skipped };
+}
+function applyExclusions(files, globalExclude, toolExclude) {
+  if (globalExclude.length === 0 && toolExclude.length === 0) return files;
+  const matcher = new CompiledMatcher([...globalExclude, ...toolExclude]);
+  if (matcher.isEmpty) return files;
+  return files.filter((file) => !matcher.matches(file, import_path2.default.basename(file)));
+}
+function routeFilesToTool(files, adapter, toolConfig) {
+  const patterns = [...adapter.filePatterns];
+  if (toolConfig?.extraFilePatterns) {
+    patterns.push(...toolConfig.extraFilePatterns);
+  }
+  const matcher = new CompiledMatcher(patterns);
+  if (matcher.matchAll) return files;
+  return files.filter((file) => matcher.matches(file, import_path2.default.basename(file)));
+}
 function filterByTargetFiles(allFiles, targetPatterns) {
   if (targetPatterns.length === 0) return allFiles;
   const normalized = targetPatterns.map((p) => p.replace(/^\.\//, ""));
   const exact = new Set(normalized);
-  const globs = normalized.map((p) => new import_minimatch.Minimatch(p, { dot: true, matchBase: false })).filter((m) => m.hasMagic());
+  const globs = normalized.map((p) => new import_minimatch2.Minimatch(p, { dot: true, matchBase: false })).filter((m) => m.hasMagic());
   const dirPrefixes = normalized.filter((p) => p.endsWith("/"));
   return allFiles.filter(
     (f) => (
@@ -491,6 +503,7 @@ function filterByTargetFiles(allFiles, targetPatterns) {
 
 // src/executor.ts
 var import_tooling2 = require("@codacy/tooling");
+var HARD_TIMEOUT_GRACE_MS = 2e3;
 async function runTool(adapter, ctx, codacyToolConfig, timeoutMs) {
   const startTime = Date.now();
   ctx.logger.debug("runTool started", {
@@ -498,12 +511,18 @@ async function runTool(adapter, ctx, codacyToolConfig, timeoutMs) {
     fileCount: ctx.targetFiles.length,
     timeoutMs
   });
+  const abortController = new AbortController();
+  ctx.signal = abortController.signal;
+  const abortTimer = setTimeout(() => abortController.abort(), timeoutMs);
+  abortTimer.unref?.();
+  let hardTimer;
   try {
     const timeoutPromise = new Promise((_resolve, reject) => {
-      setTimeout(
+      hardTimer = setTimeout(
         () => reject(new Error(`TIMEOUT: Tool ${adapter.id} exceeded ${(0, import_tooling2.formatDuration)(timeoutMs)}`)),
-        timeoutMs
+        timeoutMs + HARD_TIMEOUT_GRACE_MS
       );
+      hardTimer.unref?.();
     });
     const result = await Promise.race([adapter.analyze(ctx, codacyToolConfig), timeoutPromise]);
     ctx.logger.debug("runTool completed", {
@@ -550,6 +569,9 @@ async function runTool(adapter, ctx, codacyToolConfig, timeoutMs) {
         filesAnalyzed: ctx.targetFiles.length
       }
     };
+  } finally {
+    clearTimeout(abortTimer);
+    if (hardTimer) clearTimeout(hardTimer);
   }
 }
 async function runToolsSequential(tools, timeoutMs, callbacks) {
@@ -708,7 +730,7 @@ function computePatternSummary(issues, lookup) {
 }
 
 // src/analyze.ts
-var execFileAsync = (0, import_util.promisify)(import_child_process2.execFile);
+var execFileAsync = (0, import_util.promisify)(import_child_process3.execFile);
 var nullLogger = {
   debug: () => {
   },
@@ -736,7 +758,7 @@ async function mapWithConcurrency(items, concurrency, task) {
 async function analyze(options) {
   const startedAt = /* @__PURE__ */ new Date();
   const mode = options.mode || "standard";
-  const toolTimeout = options.toolTimeout || 6e5;
+  const toolTimeout = options.toolTimeout || 9e5;
   const parallelTools = options.parallelTools || 1;
   const logger = options.logger ?? nullLogger;
   const phaseTimings = {
@@ -1096,7 +1118,9 @@ async function analyze(options) {
     allFiles = filterByTargetFiles(allFiles, options.files);
     logger.debug("Filtered by target files", { afterFilter: allFiles.length });
   }
-  allFiles = filterOversizedFiles(allFiles, options.repositoryRoot, logger);
+  const oversize = filterOversizedFiles(allFiles, options.repositoryRoot, logger);
+  allFiles = oversize.files;
+  const skippedFiles = oversize.skipped;
   phaseTimings.fileDiscoveryMs = Date.now() - fileDiscoveryStartMs;
   const fileRoutingStartMs = Date.now();
   const readyById = new Map(capability.ready.map((r) => [r.toolId, r]));
@@ -1213,7 +1237,8 @@ async function analyze(options) {
     allErrors,
     allSummaries,
     patternSummary,
-    phaseTimings
+    phaseTimings,
+    skippedFiles
   );
 }
 function makeToolContext(base, adapter, targetFiles) {
@@ -1239,7 +1264,7 @@ function resolveGitScope(repositoryRoot, scope) {
   const base = scope.diff === true ? void 0 : scope.diff;
   return listDiffFiles(repositoryRoot, base);
 }
-function buildResult(startedAt, repositoryRoot, mode, capability, issues, errors, toolResults, patternSummary, phaseTimings) {
+function buildResult(startedAt, repositoryRoot, mode, capability, issues, errors, toolResults, patternSummary, phaseTimings, skippedFiles) {
   const completedAt = /* @__PURE__ */ new Date();
   return {
     metadata: {
@@ -1254,7 +1279,8 @@ function buildResult(startedAt, repositoryRoot, mode, capability, issues, errors
     issues,
     errors,
     toolResults,
-    ...patternSummary ? { patternSummary } : {}
+    ...patternSummary ? { patternSummary } : {},
+    ...skippedFiles && skippedFiles.length > 0 ? { skippedFiles } : {}
   };
 }
 
@@ -1263,6 +1289,7 @@ var import_path4 = __toESM(require("path"));
 var import_tooling4 = require("@codacy/tooling");
 var import_ansis = __toESM(require("ansis"));
 var import_cli_table3 = __toESM(require("cli-table3"));
+var MAX_SKIPPED_FILES_SHOWN = 5;
 function formatOutput(result, format) {
   switch (format) {
     case "json":
@@ -1420,9 +1447,30 @@ function formatText(result) {
     lines.push(...formatErrorSection(result));
     lines.push("");
   }
+  if (result.skippedFiles && result.skippedFiles.length > 0) {
+    lines.push(...formatSkippedFilesSection(result.skippedFiles));
+    lines.push("");
+  }
   lines.push(formatTextSummary(result, isInspect));
   return lines.join("\n");
 }
+function formatSkippedFilesSection(skipped) {
+  const limitKB = Math.round(MAX_FILE_SIZE_BYTES / 1024);
+  const out = [""];
+  out.push(
+    import_ansis.default.yellow(
+      `\u26A0 ${skipped.length} file${skipped.length !== 1 ? "s" : ""} skipped (exceeding the ${limitKB} KB size limit):`
+    )
+  );
+  for (const f of skipped.slice(0, MAX_SKIPPED_FILES_SHOWN)) {
+    out.push(import_ansis.default.dim(`  - ${f.filePath} (${Math.round(f.sizeBytes / 1024)} KB)`));
+  }
+  const remaining = skipped.length - MAX_SKIPPED_FILES_SHOWN;
+  if (remaining > 0) {
+    out.push(import_ansis.default.dim(`  \u2026 and ${remaining} more`));
+  }
+  return out;
+}
 function formatTextSummary(result, isInspect) {
   const lines = [];
   const inspect = isInspect ?? result.metadata.executionMode === "inspect";
@@ -1564,6 +1612,13 @@ function formatTextSummary(result, isInspect) {
   return lines.join("\n");
 }
 var MAX_ERRORS_PER_GROUP = 5;
+var MAX_ERROR_MESSAGE_LENGTH = 240;
+function flattenErrorMessage(message) {
+  return message.replace(/\s+/g, " ").trim();
+}
+function truncateErrorMessage(message) {
+  return message.length > MAX_ERROR_MESSAGE_LENGTH ? `${message.slice(0, MAX_ERROR_MESSAGE_LENGTH)}\u2026` : message;
+}
 function formatErrorSection(result) {
   const lines = [];
   const displayNameMap = /* @__PURE__ */ new Map();
@@ -1585,13 +1640,14 @@ function formatErrorSection(result) {
     const deduped = [];
     const seen = /* @__PURE__ */ new Map();
     for (const err of groupErrors) {
-      const dedupKey = `${err.filePath ?? ""}|||${err.message}`;
+      const flat = flattenErrorMessage(err.message);
+      const dedupKey = `${err.filePath ?? ""}|||${flat}`;
       const idx = seen.get(dedupKey);
       if (idx !== void 0) {
         deduped[idx].count++;
       } else {
         seen.set(dedupKey, deduped.length);
-        deduped.push({ filePath: err.filePath, message: err.message, count: 1 });
+        deduped.push({ filePath: err.filePath, message: truncateErrorMessage(flat), count: 1 });
       }
     }
     const shown = deduped.slice(0, MAX_ERRORS_PER_GROUP);
@@ -1918,7 +1974,7 @@ function applyConfigOperation(op, source, dest) {
 // src/init.ts
 var import_promises4 = __toESM(require("fs/promises"));
 var import_path7 = __toESM(require("path"));
-var import_child_process3 = require("child_process");
+var import_child_process4 = require("child_process");
 var import_tooling5 = require("@codacy/tooling");
 
 // src/discover.ts
@@ -6422,7 +6478,7 @@ async function detectLanguages(repositoryRoot, precomputedFiles) {
     files = precomputedFiles;
   } else {
     try {
-      const output = (0, import_child_process3.execSync)("git ls-files", {
+      const output = (0, import_child_process4.execSync)("git ls-files", {
         cwd: repositoryRoot,
         encoding: "utf-8",
         maxBuffer: 50 * 1024 * 1024

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codacy/analysis-runner",
-  "version": "0.11.0",
+  "version": "0.12.0",
   "description": "Core orchestration engine for Codacy analysis",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -12,11 +12,11 @@
     "cli-table3": "^0.6",
     "minimatch": "10.2.4",
     "yaml": "^2.6.0",
-    "@codacy/tooling": "0.11.0"
+    "@codacy/tooling": "0.12.0"
   },
   "devDependencies": {
     "@codacy/openapi-typescript-codegen": "0.0.9",
-    "@codacy/tools-dummy-1": "0.11.0"
+    "@codacy/tools-dummy-1": "0.12.0"
   },
   "license": "MIT",
   "author": "Codacy <support@codacy.com> (https://www.codacy.com)",