@codacy/analysis-runner 0.1.0 → 0.1.1

package/dist/index.js

@@ -225,16 +225,24 @@ function filterByTargetFiles(allFiles, targetPatterns) {
 // src/executor.ts
 async function runTool(adapter, ctx, codacyToolConfig, timeoutMs) {
   const startTime = Date.now();
-  ctx.logger.debug("runTool started", { toolId: adapter.id, fileCount: ctx.targetFiles.length, timeoutMs });
+  ctx.logger.debug("runTool started", {
+    toolId: adapter.id,
+    fileCount: ctx.targetFiles.length,
+    timeoutMs
+  });
   try {
     const timeoutPromise = new Promise((_resolve, reject) => {
-      setTimeout(() => reject(new Error(`TIMEOUT: Tool ${adapter.id} exceeded ${timeoutMs}ms`)), timeoutMs);
+      setTimeout(
+        () => reject(new Error(`TIMEOUT: Tool ${adapter.id} exceeded ${timeoutMs}ms`)),
+        timeoutMs
+      );
+    });
+    const result = await Promise.race([adapter.analyze(ctx, codacyToolConfig), timeoutPromise]);
+    ctx.logger.debug("runTool completed", {
+      toolId: adapter.id,
+      issueCount: result.summary.issueCount,
+      durationMs: result.summary.durationMs
     });
-    const result = await Promise.race([
-      adapter.analyze(ctx, codacyToolConfig),
-      timeoutPromise
-    ]);
-    ctx.logger.debug("runTool completed", { toolId: adapter.id, issueCount: result.summary.issueCount, durationMs: result.summary.durationMs });
     return result;
   } catch (err) {
     const durationMs = Date.now() - startTime;
@@ -242,7 +250,11 @@ async function runTool(adapter, ctx, codacyToolConfig, timeoutMs) {
     if (isTimeout) {
       ctx.logger.error("Tool timed out", { toolId: adapter.id, timeoutMs, elapsedMs: durationMs });
     } else {
-      ctx.logger.error("Tool execution failed", { toolId: adapter.id, error: err instanceof Error ? err.message : String(err), durationMs });
+      ctx.logger.error("Tool execution failed", {
+        toolId: adapter.id,
+        error: err instanceof Error ? err.message : String(err),
+        durationMs
+      });
     }
     const now = /* @__PURE__ */ new Date();
     return {
@@ -277,7 +289,12 @@ async function runToolsSequential(tools, timeoutMs, callbacks) {
   for (const { adapter, ctx, codacyToolConfig } of tools) {
     callbacks?.onToolStart?.(adapter.id, adapter.displayName);
     const result = await runTool(adapter, ctx, codacyToolConfig, timeoutMs);
-    callbacks?.onToolComplete?.(adapter.id, adapter.displayName, result.summary.issueCount, result.summary.durationMs);
+    callbacks?.onToolComplete?.(
+      adapter.id,
+      adapter.displayName,
+      result.summary.issueCount,
+      result.summary.durationMs
+    );
     results.push(result);
   }
   return results;
@@ -291,7 +308,12 @@ async function runToolsParallel(tools, timeoutMs, concurrency, callbacks) {
       const { adapter, ctx, codacyToolConfig } = tools[index];
       callbacks?.onToolStart?.(adapter.id, adapter.displayName);
       results[index] = await runTool(adapter, ctx, codacyToolConfig, timeoutMs);
-      callbacks?.onToolComplete?.(adapter.id, adapter.displayName, results[index].summary.issueCount, results[index].summary.durationMs);
+      callbacks?.onToolComplete?.(
+        adapter.id,
+        adapter.displayName,
+        results[index].summary.issueCount,
+        results[index].summary.durationMs
+      );
     }
   }
   const workers = Array.from({ length: Math.min(concurrency, tools.length) }, () => worker());
@@ -337,11 +359,19 @@ async function analyze(options) {
       const availability = await Promise.race([
         adapter.checkAvailability(ctx),
         new Promise(
-          (_, reject) => setTimeout(() => reject(new Error(`checkAvailability timed out after ${availabilityTimeout}ms`)), availabilityTimeout)
+          (_, reject) => setTimeout(
+            () => reject(new Error(`checkAvailability timed out after ${availabilityTimeout}ms`)),
+            availabilityTimeout
+          )
         )
       ]);
       if (availability.available) {
-        logger.debug("Tool available", { toolId: adapter.id, version: availability.version, installation: availability.installation, durationMs: Date.now() - checkStart });
+        logger.debug("Tool available", {
+          toolId: adapter.id,
+          version: availability.version,
+          installation: availability.installation,
+          durationMs: Date.now() - checkStart
+        });
         ready.push({
           toolId: adapter.id,
           displayName: adapter.displayName,
@@ -350,7 +380,11 @@ async function analyze(options) {
           runner: "local"
         });
       } else {
-        logger.warn("Tool unavailable", { toolId: adapter.id, reason: availability.reason, durationMs: Date.now() - checkStart });
+        logger.warn("Tool unavailable", {
+          toolId: adapter.id,
+          reason: availability.reason,
+          durationMs: Date.now() - checkStart
+        });
         unavailable.push({
           toolId: adapter.id,
           displayName: adapter.displayName,
@@ -359,7 +393,11 @@ async function analyze(options) {
         });
       }
     } catch (err) {
-      logger.warn("Availability check failed", { toolId: adapter.id, error: err instanceof Error ? err.message : String(err), durationMs: Date.now() - checkStart });
+      logger.warn("Availability check failed", {
+        toolId: adapter.id,
+        error: err instanceof Error ? err.message : String(err),
+        durationMs: Date.now() - checkStart
+      });
       unavailable.push({
         toolId: adapter.id,
         displayName: adapter.displayName,
@@ -378,7 +416,11 @@ async function analyze(options) {
     const configuredIds = new Set(codacyConfig.tools.map((t) => t.toolId));
     capability.ready = capability.ready.filter((t) => configuredIds.has(t.toolId));
     capability.unavailable = capability.unavailable.filter((t) => configuredIds.has(t.toolId));
-    logger.debug("Filtered by config", { readyBefore, readyAfter: capability.ready.length, configuredToolCount: configuredIds.size });
+    logger.debug("Filtered by config", {
+      readyBefore,
+      readyAfter: capability.ready.length,
+      configuredToolCount: configuredIds.size
+    });
   } else {
     logger.debug("No codacy config found, running all available tools");
   }
@@ -407,7 +449,9 @@ async function analyze(options) {
             });
             capability.unavailable.splice(i, 1);
           } else {
-            logger.error(`${adapter.displayName} post-install check failed: ${recheck.reason ?? "unknown"}`);
+            logger.error(
+              `${adapter.displayName} post-install check failed: ${recheck.reason ?? "unknown"}`
+            );
             installErrors.push({
               toolId: adapter.id,
               phase: "toolInstall",
@@ -445,7 +489,15 @@ async function analyze(options) {
       }
     }
     if (installErrors.length > 0) {
-      return buildResult(startedAt, options.repositoryRoot, mode, capability, [], installErrors, []);
+      return buildResult(
+        startedAt,
+        options.repositoryRoot,
+        mode,
+        capability,
+        [],
+        installErrors,
+        []
+      );
     }
   }
   if (mode === "inspect") {
@@ -478,15 +530,23 @@ async function analyze(options) {
   }
   if (mode === "strict" && capability.unavailable.length > 0) {
     const missingTools = capability.unavailable.map((u) => u.toolId).join(", ");
-    return buildResult(startedAt, options.repositoryRoot, mode, capability, [], [
-      {
-        toolId: "runner",
-        phase: "requirementCheck",
-        kind: "UnavailableTools",
-        message: `Strict mode: the following tools are unavailable: ${missingTools}`,
-        level: "error"
-      }
-    ], []);
+    return buildResult(
+      startedAt,
+      options.repositoryRoot,
+      mode,
+      capability,
+      [],
+      [
+        {
+          toolId: "runner",
+          phase: "requirementCheck",
+          kind: "UnavailableTools",
+          message: `Strict mode: the following tools are unavailable: ${missingTools}`,
+          level: "error"
+        }
+      ],
+      []
+    );
   }
   const globalExclude = codacyConfig?.exclude || [];
   let allFiles = listRepoFiles(options.repositoryRoot);
@@ -515,7 +575,11 @@ async function analyze(options) {
     const ctx = makeToolContext(baseCtx, adapter, toolFiles);
     const configCheck = await adapter.checkLocalConfigurationFile(ctx);
     const codacyToolConfig = buildToolConfig(codacyConfig, adapter, configCheck);
-    logger.debug("Tool config resolved", { toolId: adapter.id, configSource: codacyToolConfig.useLocalConfigurationFile ? "local" : "codacy", patternCount: codacyToolConfig.patterns?.length ?? 0 });
+    logger.debug("Tool config resolved", {
+      toolId: adapter.id,
+      configSource: codacyToolConfig.useLocalConfigurationFile ? "local" : "codacy",
+      patternCount: codacyToolConfig.patterns?.length ?? 0
+    });
     const readyEntry = capability.ready.find((r) => r.toolId === adapter.id);
     if (readyEntry) {
       if (configCheck.found && codacyToolConfig.useLocalConfigurationFile) {
@@ -527,7 +591,11 @@ async function analyze(options) {
     }
     toolRuns.push({ adapter, ctx, codacyToolConfig });
   }
-  logger.info("Starting tool execution", { toolCount: toolRuns.length, parallel: parallelTools > 1, parallelTools });
+  logger.info("Starting tool execution", {
+    toolCount: toolRuns.length,
+    parallel: parallelTools > 1,
+    parallelTools
+  });
   const callbacks = {
     onToolStart: options.onToolStart,
     onToolComplete: options.onToolComplete
@@ -541,8 +609,20 @@ async function analyze(options) {
   const allIssues = normalizeIssues(results.flatMap((r) => r.issues));
   const allErrors = results.flatMap((r) => r.errors);
   const allSummaries = results.map((r) => r.summary);
-  logger.info("Pipeline complete", { totalIssues: allIssues.length, totalErrors: allErrors.length, durationMs: Date.now() - startedAt.getTime() });
-  return buildResult(startedAt, options.repositoryRoot, mode, capability, allIssues, allErrors, allSummaries);
+  logger.info("Pipeline complete", {
+    totalIssues: allIssues.length,
+    totalErrors: allErrors.length,
+    durationMs: Date.now() - startedAt.getTime()
+  });
+  return buildResult(
+    startedAt,
+    options.repositoryRoot,
+    mode,
+    capability,
+    allIssues,
+    allErrors,
+    allSummaries
+  );
 }
 function makeToolContext(base, adapter, targetFiles) {
   return {
@@ -792,7 +872,14 @@ function formatTextSummary(result, isInspect) {
         ]);
       }
       for (const tool of unavailableTools) {
-        table.push([import_ansis.default.dim.red(tool.displayName ?? tool.toolId), "-", "-", "-", "-", tool.reason]);
+        table.push([
+          import_ansis.default.dim.red(tool.displayName ?? tool.toolId),
+          "-",
+          "-",
+          "-",
+          "-",
+          tool.reason
+        ]);
       }
       lines.push(table.toString());
     } else {
@@ -4602,7 +4689,11 @@ async function initLocalConfigBase(repositoryRoot, adapters2) {
 }
 async function initLocalConfig(repositoryRoot, adapters2) {
   const baseConfig = await initLocalConfigBase(repositoryRoot, adapters2);
-  const { config, codacyYamlApplied } = await enrichWithCodacyYaml(baseConfig, repositoryRoot, adapters2);
+  const { config, codacyYamlApplied } = await enrichWithCodacyYaml(
+    baseConfig,
+    repositoryRoot,
+    adapters2
+  );
   return { config, stats: { codacyYamlApplied } };
 }
 async function initDefaultConfig(repositoryRoot, adapters2) {
@@ -4618,7 +4709,7 @@ async function initDefaultConfig(repositoryRoot, adapters2) {
   for (const adapter of adapters2) {
     if (toolsWithLocalConfig.has(adapter.id)) continue;
     const apiTool = apiTools.find(
-      (t) => t.prefix ? t.prefix.replace(/_$/, "") === adapter.id : t.name === adapter.id
+      (t) => t.prefix ? t.prefix.replace(/_$/, "") === adapter.id : t.shortName === adapter.id
     );
     if (!apiTool) continue;
     if (!apiTool.enabledByDefault) continue;
@@ -4641,8 +4732,16 @@ async function initDefaultConfig(repositoryRoot, adapters2) {
       addedTools++;
     }
   }
-  const baseConfig = createCodacyConfig({ source: "default", languages: base.metadata.languages, tools: updatedTools });
-  const { config, codacyYamlApplied } = await enrichWithCodacyYaml(baseConfig, repositoryRoot, adapters2);
+  const baseConfig = createCodacyConfig({
+    source: "default",
+    languages: base.metadata.languages,
+    tools: updatedTools
+  });
+  const { config, codacyYamlApplied } = await enrichWithCodacyYaml(
+    baseConfig,
+    repositoryRoot,
+    adapters2
+  );
   return { config, stats: { totalApiTools: apiTools.length, addedTools, codacyYamlApplied } };
 }
 async function initRemoteConfig(repositoryRoot, provider, organization, repositoryName, adapters2, apiToken, onRemoteToolsFound, onToolProcessed) {
@@ -4711,8 +4810,15 @@ async function initRemoteConfig(repositoryRoot, provider, organization, reposito
     repositoryId: repoInfo.repositoryId != null ? String(repoInfo.repositoryId) : void 0,
     repositoryName
   });
-  const { config, codacyYamlApplied } = await enrichWithCodacyYaml(baseConfig, repositoryRoot, adapters2);
-  return { config, stats: { totalRemoteTools: remoteTools.length, matchedAdapters, codacyYamlApplied } };
+  const { config, codacyYamlApplied } = await enrichWithCodacyYaml(
+    baseConfig,
+    repositoryRoot,
+    adapters2
+  );
+  return {
+    config,
+    stats: { totalRemoteTools: remoteTools.length, matchedAdapters, codacyYamlApplied }
+  };
 }
 async function initContainerConfig(repositoryRoot, adapters2) {
   const codacyrcPath = import_path6.default.join(repositoryRoot, ".codacyrc");
@@ -4787,8 +4893,9 @@ function rotateLogFiles(logsDir, maxFiles) {
     }
   }
 }
+var fileSeq = 0;
 function fileTimestamp() {
-  return (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
+  return `${(/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-")}-${fileSeq++}`;
 }
 function enrichMessage(message, fields) {
   if (!fields) return message;
@@ -4799,10 +4906,13 @@ function enrichMessage(message, fields) {
   if (fields.configSource) parts.push(String(fields.configSource));
   if (typeof fields.fileCount === "number") parts.push(`${fields.fileCount} files`);
   if (typeof fields.issueCount === "number") parts.push(`${fields.issueCount} issues`);
-  if (typeof fields.errorCount === "number" && fields.errorCount > 0) parts.push(`${fields.errorCount} errors`);
+  if (typeof fields.errorCount === "number" && fields.errorCount > 0)
+    parts.push(`${fields.errorCount} errors`);
   if (typeof fields.totalIssues === "number") parts.push(`${fields.totalIssues} issues`);
-  if (typeof fields.totalErrors === "number" && fields.totalErrors > 0) parts.push(`${fields.totalErrors} errors`);
-  if (typeof fields.durationMs === "number") parts.push(`${(fields.durationMs / 1e3).toFixed(1)}s`);
+  if (typeof fields.totalErrors === "number" && fields.totalErrors > 0)
+    parts.push(`${fields.totalErrors} errors`);
+  if (typeof fields.durationMs === "number")
+    parts.push(`${(fields.durationMs / 1e3).toFixed(1)}s`);
   if (fields.reason) parts.push(String(fields.reason));
   if (fields.error) parts.push(String(fields.error));
   if (typeof fields.found === "boolean") parts.push(fields.found ? "found" : "not found");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codacy/analysis-runner",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "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.1.0"
+    "@codacy/tooling": "0.1.1"
   },
   "devDependencies": {
     "@codacy/openapi-typescript-codegen": "0.0.9",
-    "@codacy/tools-dummy-1": "0.1.0"
+    "@codacy/tools-dummy-1": "0.1.1"
   },
   "license": "MIT",
   "author": "Codacy <support@codacy.com> (https://www.codacy.com)",