@codacy/analysis-runner 0.2.2 → 0.3.0

package/dist/index.d.ts

@@ -109,7 +109,11 @@ declare function formatTextSummary(result: AnalysisResult, isInspect?: boolean):
  * 3. Default (empty patterns, no local config)
  */
 
-/** Reads a CodacyConfig from disk. Returns null if the file doesn't exist. */
+/**
+ * Reads a CodacyConfig from disk.
+ * Returns null if the file doesn't exist. Throws on parse/permission errors
+ * so callers can distinguish "no config" from "broken config".
+ */
 declare function readCodacyConfig(repositoryRoot: string): Promise<CodacyConfig | null>;
 /** Writes a CodacyConfig to disk, creating the .codacy/ directory if needed. */
 declare function writeCodacyConfig(repositoryRoot: string, config: CodacyConfig): Promise<string>;

package/dist/index.js

@@ -61,6 +61,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");
 
 // src/registry.ts
 var adapters = /* @__PURE__ */ new Map();
@@ -93,14 +94,18 @@ var CODACY_DIR = ".codacy";
 var CONFIG_FILE = "codacy.config.json";
 async function readCodacyConfig(repositoryRoot) {
   const configPath = import_path.default.join(repositoryRoot, CODACY_DIR, CONFIG_FILE);
+  let raw;
   try {
-    const raw = await import_promises.default.readFile(configPath, "utf-8");
-    const config = JSON.parse(raw);
-    coerceParameterValues(config);
-    return config;
-  } catch {
-    return null;
+    raw = await import_promises.default.readFile(configPath, "utf-8");
+  } catch (err) {
+    if (err instanceof Error && "code" in err && err.code === "ENOENT") {
+      return null;
+    }
+    throw err;
   }
+  const config = JSON.parse(raw);
+  coerceParameterValues(config);
+  return config;
 }
 function coerceParameterValues(config) {
   for (const tool of config.tools) {
@@ -440,6 +445,76 @@ async function analyze(options) {
   const toolTimeout = options.toolTimeout || 3e5;
   const parallelTools = options.parallelTools || 1;
   const logger = options.logger ?? nullLogger;
+  const emptyCapability = {
+    ready: [],
+    unavailable: [],
+    checkedAt: (/* @__PURE__ */ new Date()).toISOString()
+  };
+  try {
+    (0, import_child_process2.execSync)("git rev-parse --is-inside-work-tree", {
+      cwd: options.repositoryRoot,
+      stdio: "pipe"
+    });
+  } catch {
+    return buildResult(
+      startedAt,
+      options.repositoryRoot,
+      mode,
+      emptyCapability,
+      [],
+      [
+        {
+          toolId: "runner",
+          phase: "requirementCheck",
+          kind: "NotAGitRepository",
+          message: `${options.repositoryRoot} is not a git repository. Codacy analysis requires a git repository for file discovery. Run \`git init\` to initialize one.`,
+          level: "error"
+        }
+      ],
+      []
+    );
+  }
+  let codacyConfig;
+  try {
+    codacyConfig = await readCodacyConfig(options.repositoryRoot);
+  } catch (err) {
+    return buildResult(
+      startedAt,
+      options.repositoryRoot,
+      mode,
+      emptyCapability,
+      [],
+      [
+        {
+          toolId: "runner",
+          phase: "toolConfig",
+          kind: "InvalidConfig",
+          message: `.codacy/codacy.config.json is invalid: ${err instanceof Error ? err.message : String(err)}`,
+          level: "error"
+        }
+      ],
+      []
+    );
+  }
+  if (!codacyConfig) {
+    return buildResult(
+      startedAt,
+      options.repositoryRoot,
+      mode,
+      emptyCapability,
+      [],
+      [
+        {
+          toolId: "runner",
+          phase: "toolConfig",
+          kind: "MissingConfig",
+          message: "No .codacy/codacy.config.json found. Run `codacy-analysis init` to initialize the repository.",
+          level: "error"
+        }
+      ],
+      []
+    );
+  }
   const adapters2 = getRegisteredAdapters(options.tools);
   logger.info("Pipeline started", { mode, toolCount: adapters2.length, parallelTools, toolTimeout });
   const globalDir = import_path3.default.join(import_os.default.homedir(), ".codacy");
@@ -512,20 +587,15 @@ async function analyze(options) {
     unavailable,
     checkedAt: (/* @__PURE__ */ new Date()).toISOString()
   };
-  const codacyConfig = await readCodacyConfig(options.repositoryRoot);
-  if (codacyConfig) {
-    const readyBefore = capability.ready.length;
-    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
-    });
-  } else {
-    logger.debug("No codacy config found, running all available tools");
-  }
+  const readyBefore = capability.ready.length;
+  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
+  });
   if (mode === "auto-install" && capability.unavailable.length > 0) {
     const installErrors = [];
     for (let i = capability.unavailable.length - 1; i >= 0; i--) {
@@ -650,7 +720,7 @@ async function analyze(options) {
       []
     );
   }
-  const globalExclude = codacyConfig?.exclude || [];
+  const globalExclude = codacyConfig.exclude || [];
   let allFiles = listRepoFiles(options.repositoryRoot);
   logger.debug("File discovery", { totalFiles: allFiles.length });
   if (options.gitScope) {
@@ -675,7 +745,7 @@ async function analyze(options) {
   const toolRuns = [];
   for (const adapter of readyAdapters) {
     const toolConfig = getToolConfig(codacyConfig, adapter.id);
-    if (codacyConfig && !toolConfig) continue;
+    if (!toolConfig) continue;
     const toolExclude = toolConfig?.exclude || [];
     const filesAfterExclusion = applyExclusions(allFiles, globalExclude, toolExclude);
     const toolFiles = routeFilesToTool(filesAfterExclusion, adapter, toolConfig);
@@ -683,19 +753,9 @@ async function analyze(options) {
     if (readyEntryForFiles) {
       readyEntryForFiles.filesRouted = toolFiles.length;
     }
-    if (toolFiles.length === 0) {
-      logger.debug("No files routed, skipping", { toolId: adapter.id });
-      continue;
-    }
-    logger.debug("Files routed to tool", { toolId: adapter.id, fileCount: toolFiles.length });
-    const ctx = makeToolContext(baseCtx, adapter, toolFiles);
-    const configCheck = await adapter.checkLocalConfigurationFile(ctx);
+    const configCtx = makeToolContext(baseCtx, adapter, toolFiles);
+    const configCheck = await adapter.checkLocalConfigurationFile(configCtx);
     const codacyToolConfig = buildToolConfig(codacyConfig, adapter, configCheck);
-    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) {
@@ -705,7 +765,17 @@ async function analyze(options) {
         readyEntry.configSource = "codacy";
       }
     }
-    toolRuns.push({ adapter, ctx, codacyToolConfig });
+    if (toolFiles.length === 0) {
+      logger.debug("No files routed, skipping", { toolId: adapter.id });
+      continue;
+    }
+    logger.debug("Files routed to tool", { toolId: adapter.id, fileCount: toolFiles.length });
+    logger.debug("Tool config resolved", {
+      toolId: adapter.id,
+      configSource: codacyToolConfig.useLocalConfigurationFile ? "local" : "codacy",
+      patternCount: codacyToolConfig.patterns?.length ?? 0
+    });
+    toolRuns.push({ adapter, ctx: configCtx, codacyToolConfig });
   }
   logger.info("Starting tool execution", {
     toolCount: toolRuns.length,
@@ -1135,7 +1205,7 @@ function colorSeverity(severity) {
 // src/init.ts
 var import_promises3 = __toESM(require("fs/promises"));
 var import_path6 = __toESM(require("path"));
-var import_child_process2 = require("child_process");
+var import_child_process3 = require("child_process");
 var import_tooling = require("@codacy/tooling");
 
 // src/codacy-yaml.ts
@@ -4730,7 +4800,7 @@ async function listRepositoryToolPatterns(provider, org, repo, toolUuid) {
 async function detectLanguages(repositoryRoot) {
   let files = [];
   try {
-    const output = (0, import_child_process2.execSync)("git ls-files", {
+    const output = (0, import_child_process3.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.2.2",
+  "version": "0.3.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.2.2"
+    "@codacy/tooling": "0.3.0"
   },
   "devDependencies": {
     "@codacy/openapi-typescript-codegen": "0.0.9",
-    "@codacy/tools-dummy-1": "0.2.2"
+    "@codacy/tools-dummy-1": "0.3.0"
   },
   "license": "MIT",
   "author": "Codacy <support@codacy.com> (https://www.codacy.com)",