@skaleagents/swarm 0.6.10 → 0.6.11

package/README.md

@@ -62,7 +62,9 @@ review-brief builder.
 ## Infrastructure scanning
 
 `scan_iac` parses Terraform HCL/JSON, CloudFormation YAML/JSON, and Kubernetes
-manifests, including multi-document YAML and Kubernetes Lists. It returns a
+manifests, including multi-document YAML and Kubernetes Lists. It also reads
+Dockerfiles, GitHub Actions workflows, Helm templates, Ansible playbooks,
+Bicep files, and ARM templates. It returns a
 resource inventory and findings with stable rule IDs, severity, property paths,
 line locations, and remediation. Findings never include matched secret values.
 

package/dist/iac/parse.d.ts

@@ -1,4 +1,4 @@
-export type IacFormat = "terraform" | "cloudformation" | "kubernetes" | "pulumi" | "compose" | "dockerfile" | "github" | "helm" | "ansible";
+export type IacFormat = "terraform" | "cloudformation" | "kubernetes" | "pulumi" | "compose" | "dockerfile" | "github" | "helm" | "ansible" | "bicep" | "arm";
 export type Path = (string | number)[];
 export type Location = {
     line: number;
@@ -28,5 +28,8 @@ export declare function looksLikeDockerfile(content: string): boolean;
 export declare function looksLikeAnsible(content: string): boolean;
 export declare function looksLikeHelm(content: string): boolean;
 export declare function looksLikeGithubWorkflow(content: string): boolean;
+export declare function looksLikeTerraformHcl(content: string): boolean;
+export declare function looksLikeBicep(content: string): boolean;
+export declare function looksLikeArm(content: string): boolean;
 export declare function looksLikePulumiYaml(content: string): boolean;
 export declare function parseIac(content: string, requested?: IacFormat | "auto"): ParsedIac;

package/dist/iac/parse.js

@@ -44,7 +44,9 @@ export function looksLikeIac(content) {
         looksLikeDockerfile(content) ||
         looksLikeGithubWorkflow(content) ||
         looksLikeHelm(content) ||
-        looksLikeAnsible(content));
+        looksLikeAnsible(content) ||
+        looksLikeBicep(content) ||
+        looksLikeArm(content));
 }
 export function looksLikePulumiProgram(content) {
     return /(?:^|\n)\s*(?:import\s+\*\s+as\s+pulumi\b|import\s+pulumi\b|from\s+["']@pulumi\/)/.test(content);
@@ -89,6 +91,44 @@ export function looksLikeGithubWorkflow(content) {
         return false;
     return topOn && topLevelKey(content, "jobs");
 }
+export function looksLikeTerraformHcl(content) {
+    return /^\s*(?:(?:resource|module|variable|provider|data|output)\s+"|(?:terraform|locals)\s*\{)/m.test(content);
+}
+export function looksLikeBicep(content) {
+    const visible = maskIaCComments(content);
+    if (looksLikeTerraformHcl(visible))
+        return false;
+    return /^\s*resource\s+[A-Za-z_][A-Za-z0-9_]*\s+['"]Microsoft\./m.test(visible);
+}
+export function looksLikeArm(content) {
+    const visible = maskIaCComments(content).trim();
+    if (!visible.startsWith("{"))
+        return false;
+    let value;
+    try {
+        value = JSON.parse(visible);
+    }
+    catch {
+        return false;
+    }
+    const root = object(value);
+    if (typeof root.AWSTemplateFormatVersion === "string")
+        return false;
+    if (Object.values(object(root.Resources)).some((item) => String(object(item).Type ?? "").startsWith("AWS::")))
+        return false;
+    if ("resource" in root && !Array.isArray(root.resources))
+        return false;
+    if ("format_version" in root ||
+        "planned_values" in root ||
+        "resource_changes" in root)
+        return false;
+    if (typeof root.$schema === "string" &&
+        root.$schema.includes("deploymentTemplate"))
+        return true;
+    if (typeof root.contentVersion !== "string" || !Array.isArray(root.resources))
+        return false;
+    return root.resources.some((item) => String(object(item).type ?? "").startsWith("Microsoft."));
+}
 export function looksLikePulumiYaml(content) {
     return (/^\s*resources\s*:/m.test(content) &&
         /^\s+type\s*:\s*[A-Za-z0-9_.-]+:\S+/m.test(content));
@@ -114,6 +154,16 @@ export function parseIac(content, requested = "auto") {
         (requested === "auto" && looksLikeHelm(content))) {
         return parseHelm(content);
     }
+    if (requested === "bicep" || (requested === "auto" && looksLikeBicep(content))) {
+        if (!looksLikeBicep(content))
+            throw new ScanInputError("A Bicep file requires a Microsoft resource declaration.");
+        return parseAzure(content, "bicep");
+    }
+    if (requested === "arm" || (requested === "auto" && looksLikeArm(content))) {
+        if (!looksLikeArm(content))
+            throw new ScanInputError("An ARM template requires a deploymentTemplate schema or contentVersion with Microsoft resources.");
+        return parseAzure(content, "arm");
+    }
     if (requested !== "terraform" &&
         requested !== "compose" &&
         looksLikePulumiProgram(content) &&
@@ -123,8 +173,7 @@ export function parseIac(content, requested = "auto") {
     const warnings = [];
     const resources = [];
     const isHcl = (requested === "terraform" && !content.trimStart().startsWith("{")) ||
-        (requested === "auto" &&
-            /^\s*(?:(?:resource|module|variable|provider|data|output)\s+"|(?:terraform|locals)\s*\{)/m.test(content));
+        (requested === "auto" && looksLikeTerraformHcl(content));
     if (isHcl) {
         let data;
         try {
@@ -330,7 +379,7 @@ export function parseIac(content, requested = "auto") {
         }
     }
     if (!format)
-        throw new ScanInputError("No IaC document found. Choose Terraform, CloudFormation, Kubernetes, Pulumi, Docker Compose, a Dockerfile, a GitHub Actions workflow, a Helm template, or an Ansible playbook.");
+        throw new ScanInputError("No IaC document found. Choose Terraform, CloudFormation, Kubernetes, Pulumi, Docker Compose, a Dockerfile, a GitHub Actions workflow, a Helm template, an Ansible playbook, Bicep, or an ARM template.");
     if (!resources.length)
         warnings.push("No resources found in the submitted document.");
     if (/(?:!(?:Ref|Sub|GetAtt|If)\b|"(?:Ref|Fn::\w+)"\s*:|\$\{)/.test(content))
@@ -517,6 +566,53 @@ function sourceIsSecret(source) {
         return true;
     return base === "credentials";
 }
+function secretVariableName(name) {
+    const normalized = name.replace(/-/g, "_");
+    return (/(?:^|_)(?:password|passwd|secret|token)$/i.test(normalized) ||
+        /(?:^|_)(?:api_?key|auth_token|access_key|private_key)$/i.test(normalized));
+}
+function isShellReference(value) {
+    return (/^\$[A-Za-z_][A-Za-z0-9_]*$/.test(value) ||
+        /^\$\{[A-Za-z_][A-Za-z0-9_]*\}$/.test(value));
+}
+function parseImageAssignments(kind, rest) {
+    const trimmed = rest.trim();
+    if (!trimmed)
+        return [];
+    if (kind === "ARG" && /^[A-Za-z_][A-Za-z0-9_]*$/.test(trimmed))
+        return [{ name: trimmed, value: "" }];
+    if (!trimmed.includes("=")) {
+        const spaced = trimmed.match(/^([A-Za-z_][A-Za-z0-9_]*)\s+([\s\S]*)$/);
+        if (!spaced)
+            return [];
+        return [{ name: spaced[1], value: unquoteAssignment(spaced[2].trim()) }];
+    }
+    const pairs = [];
+    const pattern = /([A-Za-z_][A-Za-z0-9_]*)=(?:"([^"]*)"|'([^']*)'|([^\s]*))/g;
+    for (const match of trimmed.matchAll(pattern)) {
+        pairs.push({
+            name: match[1],
+            value: match[2] ?? match[3] ?? match[4] ?? "",
+        });
+    }
+    return pairs;
+}
+function unquoteAssignment(value) {
+    if ((value.startsWith('"') && value.endsWith('"') && value.length >= 2) ||
+        (value.startsWith("'") && value.endsWith("'") && value.length >= 2))
+        return value.slice(1, -1);
+    return value;
+}
+function workflowEnvIsLiteral(value) {
+    const trimmed = value.trim();
+    if (!trimmed)
+        return false;
+    if (/^\$\{\{\s*secrets\.[^}]+\}\}$/i.test(trimmed))
+        return false;
+    if (/^\$\{\{\s*github\.token\s*\}\}$/i.test(trimmed))
+        return false;
+    return true;
+}
 function lineLocation(line, path) {
     return { line, column: 1, path };
 }
@@ -576,6 +672,22 @@ function parseDockerfile(content) {
             locate: () => lineLocation(instruction.line, "copy"),
         });
     });
+    instructions.forEach((instruction, index) => {
+        const header = instruction.text.match(/^(ENV|ARG)\b\s*([\s\S]*)$/i);
+        if (!header)
+            return;
+        const kind = header[1].toUpperCase() === "ARG" ? "ARG" : "ENV";
+        const assignments = parseImageAssignments(kind, header[2] ?? "");
+        const literal = assignments.some((item) => secretVariableName(item.name) &&
+            item.value.trim() !== "" &&
+            !isShellReference(item.value.trim()));
+        resources.push({
+            id: `dockerfile.env.${index + 1}`,
+            type: "dockerfile:env",
+            value: { literal },
+            locate: () => lineLocation(instruction.line, "env"),
+        });
+    });
     return {
         format: "dockerfile",
         resources,
@@ -678,7 +790,27 @@ function parseGithubWorkflow(content) {
             value: { broad },
             locate: () => position(root.permissions != null ? ["permissions"] : ["on"]),
         });
+        const addEnv = (envValue, path) => {
+            for (const [key, raw] of Object.entries(object(envValue))) {
+                if (typeof raw !== "string" && raw != null)
+                    continue;
+                if (!secretVariableName(key))
+                    continue;
+                const value = typeof raw === "string" ? raw : "";
+                resources.push({
+                    id: `env.${[...path, key].join(".")}`,
+                    type: "github:env",
+                    value: { literal: workflowEnvIsLiteral(value) },
+                    locate: () => position([...path, key]),
+                });
+            }
+        };
+        addEnv(root.env, ["env"]);
         for (const [jobName, job] of Object.entries(object(root.jobs))) {
+            addEnv(object(job).env, ["jobs", jobName, "env"]);
+            array(object(job).steps).forEach((step, index) => {
+                addEnv(object(step).env, ["jobs", jobName, "steps", index, "env"]);
+            });
             array(object(job).steps).forEach((step, index) => {
                 const uses = object(step).uses;
                 if (typeof uses !== "string")
@@ -836,3 +968,105 @@ function parseAnsible(content) {
         warnings: [ansibleFactsWarning],
     };
 }
+function maskIaCComments(content) {
+    const withoutBlock = content.replace(/\/\*[\s\S]*?\*\//g, (match) => match.replace(/[^\n]/g, " "));
+    return withoutBlock
+        .split("\n")
+        .map((line) => {
+        let out = "";
+        let quote;
+        for (let index = 0; index < line.length; index++) {
+            const char = line[index];
+            const next = line[index + 1];
+            if (quote) {
+                out += char;
+                if (char === "\\" && quote === '"') {
+                    if (next)
+                        out += next;
+                    index += 1;
+                    continue;
+                }
+                if (char === quote)
+                    quote = undefined;
+                continue;
+            }
+            if (char === '"' || char === "'") {
+                quote = char;
+                out += char;
+                continue;
+            }
+            if ((char === "/" && next === "/") ||
+                char === "#") {
+                return out + " ".repeat(line.length - out.length);
+            }
+            out += char;
+        }
+        return out;
+    })
+        .join("\n");
+}
+function addressValueIsOpen(after) {
+    return /(?:^|[\s\[,])(?:"\*"|"0\.0\.0\.0\/0"|'\*'|'0\.0\.0\.0\/0'|\*|0\.0\.0\.0\/0)(?=$|[\s,\]}"'])/.test(after);
+}
+function nearestScopeIsEgress(content, index) {
+    const windowStart = Math.max(0, index - 800);
+    const windowEnd = Math.min(content.length, index + 400);
+    const slice = content.slice(windowStart, windowEnd);
+    const relative = index - windowStart;
+    let nearest;
+    for (const match of slice.matchAll(/\b(Inbound|Outbound|ingress|egress)\b/gi)) {
+        const at = match.index ?? 0;
+        const distance = Math.abs(at - relative);
+        const egress = /^(?:outbound|egress)$/i.test(match[1] ?? "");
+        if (!nearest || distance < nearest.distance)
+            nearest = { distance, egress };
+    }
+    return nearest?.egress ?? false;
+}
+function firstOpenAddress(content) {
+    const pattern = /(source|destination)AddressPrefix(es)?["']?\s*[:=]/gi;
+    for (const match of content.matchAll(pattern)) {
+        const kind = (match[1] ?? "").toLowerCase();
+        const index = match.index ?? 0;
+        const after = content.slice(index + match[0].length, index + match[0].length + 240);
+        if (!addressValueIsOpen(after))
+            continue;
+        if (kind === "destination" && nearestScopeIsEgress(content, index))
+            continue;
+        return { present: true, line: lineAt(content, index) };
+    }
+    return { present: false, line: 1 };
+}
+function enclosingAzureType(content, index) {
+    const before = content.slice(Math.max(0, index - 4000), index);
+    const declarations = [
+        ...before.matchAll(/resource\s+[A-Za-z_][\w]*\s+['"](Microsoft\.[^'"]+)['"]|(?:["']type["']|type)\s*:\s*["'](Microsoft\.[^"']+)["']/g),
+    ];
+    const last = declarations.at(-1);
+    return last?.[1] ?? last?.[2] ?? "";
+}
+function firstPublicNetwork(content) {
+    const pattern = /publicNetworkAccess["']?\s*[:=]\s*(["']?)(Enabled|true|Disabled|false)\1(?![\w.-])/gi;
+    for (const match of content.matchAll(pattern)) {
+        if (!/^(?:enabled|true)$/i.test(match[2] ?? ""))
+            continue;
+        const index = match.index ?? 0;
+        const type = enclosingAzureType(content, index);
+        if (!/(?:database|storage|server)/i.test(type))
+            continue;
+        return { present: true, line: lineAt(content, index) };
+    }
+    return { present: false, line: 1 };
+}
+const azureExpressionWarning = "Bicep modules and ARM expressions are not evaluated.";
+function parseAzure(content, format) {
+    const visible = maskIaCComments(content);
+    return {
+        format,
+        resources: [
+            literalResource("azure.exposure", "azure:exposure", firstOpenAddress(visible), "sourceAddressPrefix"),
+            literalResource("azure.publicNetwork", "azure:publicNetwork", firstPublicNetwork(visible), "publicNetworkAccess"),
+        ],
+        warnings: [azureExpressionWarning],
+    };
+}

package/dist/iac/rules.js

@@ -253,6 +253,34 @@ export const rules = {
         detail: "A run script echoes or prints a secrets expression.",
         remediation: "Remove the printed secret from the script. Pass secrets through an environment variable that the step does not print.",
     },
+    DF005: {
+        category: "security",
+        severity: "high",
+        title: "Secret assigned in an image variable",
+        detail: "A Dockerfile ENV or ARG assigns a non-empty literal to a secret-like name.",
+        remediation: "Pass the value at runtime or from a secret mount. Do not bake a literal secret into ENV or ARG.",
+    },
+    GH005: {
+        category: "security",
+        severity: "high",
+        title: "Secret assigned in workflow env",
+        detail: "A workflow env mapping assigns a non-empty literal to a secret-like name.",
+        remediation: "Reference secrets or github.token in the env value. Do not commit a literal secret.",
+    },
+    AZ001: {
+        category: "security",
+        severity: "high",
+        title: "Broad network exposure",
+        detail: "A source address prefix allows every address, or a destination prefix is open outside an egress scope.",
+        remediation: "Restrict source address prefixes to approved ranges. Keep open destination prefixes on egress rules only.",
+    },
+    AZ002: {
+        category: "security",
+        severity: "high",
+        title: "Database or storage allows public network access",
+        detail: "publicNetworkAccess is Enabled or true on a database, storage, or server resource.",
+        remediation: "Set publicNetworkAccess to Disabled and reach the resource through a private endpoint.",
+    },
     HELM001: {
         category: "security",
         severity: "critical",
@@ -326,6 +354,10 @@ export function resourceFindings(resources, format) {
             checkAnsible(resource, check);
             continue;
         }
+        if (format === "bicep" || format === "arm") {
+            checkAzure(resource, check);
+            continue;
+        }
         walk(resource.value, (key, value, path, parent) => {
             if (typeof value !== "string")
                 return;
@@ -355,6 +387,8 @@ function checkDockerfile(resource, check) {
         check("DF003", resource.value.piped === true, ["run"]);
     else if (resource.type === "dockerfile:copy")
         check("DF004", resource.value.secret === true, ["copy"]);
+    else if (resource.type === "dockerfile:env")
+        check("DF005", resource.value.literal === true, ["env"]);
 }
 function checkGithub(resource, check) {
     if (resource.type === "github:trigger")
@@ -366,6 +400,8 @@ function checkGithub(resource, check) {
         check("GH003", resource.value.floating === true, ["uses"]);
     else if (resource.type === "github:run")
         check("GH004", resource.value.printsSecret === true, ["run"]);
+    else if (resource.type === "github:env")
+        check("GH005", resource.value.literal === true, ["env"]);
 }
 function checkHelm(resource, check) {
     if (resource.type === "helm:privileged")
@@ -383,6 +419,12 @@ function checkAnsible(resource, check) {
     else if (resource.type === "ansible:cidr")
         check("ANS002", resource.value.present === true, ["cidr"]);
 }
+function checkAzure(resource, check) {
+    if (resource.type === "azure:exposure")
+        check("AZ001", resource.value.present === true, ["sourceAddressPrefix"]);
+    else if (resource.type === "azure:publicNetwork")
+        check("AZ002", resource.value.present === true, ["publicNetworkAccess"]);
+}
 function walk(value, visit, path = []) {
     if (Array.isArray(value))
         value.forEach((child, index) => walk(child, visit, [...path, index]));

package/dist/server.js

@@ -77,12 +77,14 @@ export function createServer(remote = false) {
                 "github",
                 "helm",
                 "ansible",
+                "bicep",
+                "arm",
                 "application",
                 "auto",
             ])
                 .optional()
                 .default("auto")
-                .describe("Content format: terraform, cloudformation, kubernetes, pulumi, compose, dockerfile, github, helm, ansible, application, auto."),
+                .describe("Content format: terraform, cloudformation, kubernetes, pulumi, compose, dockerfile, github, helm, ansible, bicep, arm, application, auto."),
         },
     }, async ({ content, focus, format, minSeverity, maxFindings }) => {
         const botHints = fetchPublicBotHints();
@@ -130,7 +132,7 @@ export function createServer(remote = false) {
             description: (name === "scan_iac_stub"
                 ? "Compatibility alias for scan_iac; runs the full scanner. "
                 : "") +
-                "Parse Terraform HCL/JSON, CloudFormation YAML/JSON, Kubernetes manifests, Pulumi YAML, Pulumi programs, or Docker Compose. Includes Dockerfile, GitHub Actions, Helm templates, and Ansible playbooks. Helm templates and Ansible facts are not executed. Check security, reliability and cost rules with resource locations and remediation. Pulumi programs are scanned as text.",
+                "Parse Terraform HCL/JSON, CloudFormation YAML/JSON, Kubernetes manifests, Pulumi YAML, Pulumi programs, or Docker Compose. Includes Dockerfile, GitHub Actions, Helm templates, Ansible playbooks, Bicep files, and ARM templates. Helm templates, Ansible facts, Bicep modules, and ARM expressions are not executed. Check security, reliability and cost rules with resource locations and remediation. Pulumi programs are scanned as text.",
             annotations: {
                 readOnlyHint: true,
                 destructiveHint: false,
@@ -138,7 +140,7 @@ export function createServer(remote = false) {
             },
             _meta: { securitySchemes: [{ type: "oauth2", scopes: ["mcp"] }] },
             inputSchema: {
-                content: contentSchema.describe("Terraform HCL/JSON, CloudFormation YAML/JSON, Kubernetes YAML/JSON, Pulumi YAML or program text, or Docker Compose. Includes Dockerfile, GitHub Actions, Helm templates, and Ansible playbooks."),
+                content: contentSchema.describe("Terraform HCL/JSON, CloudFormation YAML/JSON, Kubernetes YAML/JSON, Pulumi YAML or program text, or Docker Compose. Includes Dockerfile, GitHub Actions, Helm templates, Ansible playbooks, Bicep files, and ARM templates."),
                 format: z
                     .enum([
                     "terraform",
@@ -150,6 +152,8 @@ export function createServer(remote = false) {
                     "github",
                     "helm",
                     "ansible",
+                    "bicep",
+                    "arm",
                     "auto",
                 ])
                     .default("auto"),

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "0.6.10";
+export declare const VERSION = "0.6.11";

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = "0.6.10";
+export const VERSION = "0.6.11";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skaleagents/swarm",
-  "version": "0.6.10",
+  "version": "0.6.11",
   "private": false,
   "publishConfig": {
     "access": "public"