@codacy/tools-hadolint-2 0.13.0 → 0.15.0

package/dist/index.js

@@ -642,7 +642,7 @@ var patterns_default = [
     severityLevel: "Info",
     title: "Escape Literal Dollar Signs in Double Quotes",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -651,7 +651,7 @@ var patterns_default = [
     severityLevel: "Info",
     title: "Avoid Unnecessary Escape Sequences in Shell Scripts",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -660,7 +660,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Remove Space After '=' in Variable Assignment",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -669,7 +669,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Enforce Semicolon or Linefeed Before Shell Keywords like done",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -678,7 +678,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Avoid Unicode Non-Breaking and Zero-Width Spaces",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -687,7 +687,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Enforce Space After Negation in Shell Conditionals",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -696,7 +696,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Using Both '&' and ';' to Terminate Commands",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -705,7 +705,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Avoid Declaring Parameters in Shell Functions",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -714,7 +714,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Avoid Using $ on Left Side of Assignments",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -723,7 +723,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Avoid Spaces Around '=' in Shell Variable Assignments",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -732,7 +732,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Enforce Correct Use of Backticks for Command Expansion",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -741,7 +741,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Detect Unclosed Double-Quoted Strings",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -759,7 +759,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Enforce lowercase shell script keywords",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -768,7 +768,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Literal Curly Braces Without Proper Syntax or Quoting",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -777,7 +777,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Avoid Using Dollar Sign on Iterator Variable in For Loops",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -786,7 +786,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Enforce Braces When Expanding Arrays",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -795,7 +795,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Enforce Space Between Function Name and Body in Shell Scripts",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -804,7 +804,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Avoid Using Unexpected '==' in Shell Scripts",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -813,7 +813,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Quote or Escape Special Characters in eval",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -822,7 +822,7 @@ var patterns_default = [
     severityLevel: "Info",
     title: "Enforce Space Before Comment Hash",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -831,7 +831,7 @@ var patterns_default = [
     severityLevel: "Info",
     title: "Avoid Useless Use of cat",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -840,7 +840,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Avoid Using 'A && B || C' Instead of If-Then-Else",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -849,7 +849,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Avoid Unescaped Single Quotes Outside of Quotes in Shell Commands",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -858,7 +858,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Using echo for Escape Sequences; Use printf Instead",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -867,7 +867,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Enforce Safe Glob Usage to Avoid Misinterpretation as Options",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -876,7 +876,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Avoid Using Undefined Features in POSIX sh",
     enabled: false,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -903,7 +903,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Incorrectly Nested or Unnecessary Double Quotes in Shell Scripts",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -912,7 +912,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Detect Unassigned Variable References in Shell Scripts",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -921,7 +921,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Declare and Assign Separately to Avoid Masking Return Values",
     enabled: false,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   },
   {
@@ -930,7 +930,7 @@ var patterns_default = [
     severityLevel: "High",
     title: "Ensure cd Command Failure is Handled",
     enabled: true,
-    languages: ["Dockerfile"],
+    languages: ["Shell"],
     parameters: []
   }
 ];
@@ -1000,7 +1000,10 @@ var PLATFORM_MAP = {
   "darwin-arm64": "macos-arm64",
   "darwin-x64": "macos-x86_64",
   "linux-x64": "linux-x86_64",
-  "linux-arm64": "linux-arm64"
+  "linux-arm64": "linux-arm64",
+  // Windows ships a raw `.exe` (hadolint-Windows-x86_64.exe) — the `.exe`
+  // suffix is added at the download/resolve sites via exeName.
+  "win32-x64": "Windows-x86_64"
 };
 async function getVersionFromBinary(binaryPath) {
   try {
@@ -1015,7 +1018,7 @@ async function getVersionFromBinary(binaryPath) {
   }
 }
 async function resolveBinaryPath(ctx) {
-  const codacyBin = import_path.default.join(ctx.toolInstallDir, BINARY);
+  const codacyBin = import_path.default.join(ctx.toolInstallDir, (0, import_tooling.exeName)(BINARY));
   const version = await getVersionFromBinary(codacyBin);
   if (version) return codacyBin;
   return BINARY;
@@ -1096,12 +1099,13 @@ var adapter = {
       };
     }
     const version = adapter.preferredVersion;
-    const url = `https://github.com/hadolint/hadolint/releases/download/v${version}/hadolint-${platformSuffix}`;
-    const destPath = import_path.default.join(ctx.toolInstallDir, BINARY);
+    const assetExt = import_tooling.IS_WINDOWS ? ".exe" : "";
+    const url = `https://github.com/hadolint/hadolint/releases/download/v${version}/hadolint-${platformSuffix}${assetExt}`;
+    const destPath = import_path.default.join(ctx.toolInstallDir, (0, import_tooling.exeName)(BINARY));
     try {
       ctx.logger.info(`Downloading hadolint v${version} for ${platformKey}...`);
       await (0, import_tooling.downloadFile)(url, destPath, ctx.logger);
-      await import_promises.default.chmod(destPath, 493);
+      if (!import_tooling.IS_WINDOWS) await import_promises.default.chmod(destPath, 493);
       ctx.logger.info(`Installed hadolint v${version} to ${destPath}`);
       return { success: true, installedPath: destPath, installedVersion: version };
     } catch (err) {
@@ -1192,37 +1196,62 @@ var adapter = {
         args.push(flag, code);
       }
     }
-    args.push(...absoluteFiles);
+    const fileChunks = (0, import_tooling.chunkFilesForArgv)(absoluteFiles);
     try {
-      const { stdout, stderr, exitCode } = await (0, import_tooling.spawnTool)(binary, args, {
-        cwd: ctx.repositoryRoot,
-        logger: ctx.logger
-      });
-      if (exitCode >= 2) {
-        errors.push({
-          toolId: TOOL_ID,
-          phase: "toolInvoke",
-          kind: "NonZeroExit",
-          message: `hadolint exited with code ${exitCode}${stderr ? `: ${stderr.trim()}` : ""}`,
-          level: "error"
-        });
-        return (0, import_tooling.buildResult)(TOOL_ID, startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
-      }
-      if (!stdout.trim()) {
-        return (0, import_tooling.buildResult)(TOOL_ID, startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
-      }
-      let findings;
-      try {
-        findings = JSON.parse(stdout);
-      } catch (err) {
-        errors.push({
-          toolId: TOOL_ID,
-          phase: "outputParse",
-          kind: "MalformedOutput",
-          message: `Failed to parse hadolint JSON output: ${err instanceof Error ? err.message : String(err)}`,
-          level: "error"
+      const findings = [];
+      for (const chunk of fileChunks) {
+        const { stdout, stderr, exitCode } = await (0, import_tooling.spawnTool)(binary, [...args, ...chunk], {
+          cwd: ctx.repositoryRoot,
+          logger: ctx.logger
         });
-        return (0, import_tooling.buildResult)(TOOL_ID, startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
+        if (exitCode >= 2) {
+          errors.push({
+            toolId: TOOL_ID,
+            phase: "toolInvoke",
+            kind: "NonZeroExit",
+            message: `hadolint exited with code ${exitCode}${stderr ? `: ${stderr.trim()}` : ""}`,
+            level: "error"
+          });
+          return (0, import_tooling.buildResult)(
+            TOOL_ID,
+            startedAt,
+            /* @__PURE__ */ new Date(),
+            issues,
+            errors,
+            ctx.targetFiles.length
+          );
+        }
+        if (stderr.trim()) {
+          errors.push({
+            toolId: TOOL_ID,
+            phase: "outputParse",
+            kind: "ParseWarning",
+            message: `hadolint stderr: ${stderr.trim()}`,
+            level: "warning"
+          });
+        }
+        if (!stdout.trim()) continue;
+        let chunkFindings;
+        try {
+          chunkFindings = JSON.parse(stdout);
+        } catch (err) {
+          errors.push({
+            toolId: TOOL_ID,
+            phase: "outputParse",
+            kind: "MalformedOutput",
+            message: `Failed to parse hadolint JSON output: ${err instanceof Error ? err.message : String(err)}`,
+            level: "error"
+          });
+          return (0, import_tooling.buildResult)(
+            TOOL_ID,
+            startedAt,
+            /* @__PURE__ */ new Date(),
+            issues,
+            errors,
+            ctx.targetFiles.length
+          );
+        }
+        findings.push(...chunkFindings);
       }
       const enabledPatterns = new Set(codacyConfig.patterns.map((p) => p.patternId));
       const findingsByFile = /* @__PURE__ */ new Map();
@@ -1258,15 +1287,6 @@ var adapter = {
           });
         }
       }
-      if (stderr.trim()) {
-        errors.push({
-          toolId: TOOL_ID,
-          phase: "outputParse",
-          kind: "ParseWarning",
-          message: `hadolint stderr: ${stderr.trim()}`,
-          level: "warning"
-        });
-      }
     } catch (err) {
       errors.push({
         toolId: TOOL_ID,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codacy/tools-hadolint-2",
-  "version": "0.13.0",
+  "version": "0.15.0",
   "description": "Hadolint adapter — CLI-mode Dockerfile linter",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -8,7 +8,7 @@
     "dist"
   ],
   "dependencies": {
-    "@codacy/tooling": "0.13.0"
+    "@codacy/tooling": "0.15.0"
   },
   "license": "MIT",
   "author": "Codacy <support@codacy.com> (https://www.codacy.com)",