@skaleagents/swarm 0.6.7 → 0.6.9

package/dist/iac/parse.d.ts

@@ -1,4 +1,4 @@
-export type IacFormat = "terraform" | "cloudformation" | "kubernetes" | "pulumi" | "compose";
+export type IacFormat = "terraform" | "cloudformation" | "kubernetes" | "pulumi" | "compose" | "dockerfile" | "github";
 export type Path = (string | number)[];
 export type Location = {
     line: number;
@@ -24,5 +24,7 @@ export declare function array(value: unknown): unknown[];
 export declare function looksLikeIac(content: string): boolean;
 export declare function looksLikePulumiProgram(content: string): boolean;
 export declare function looksLikeCompose(content: string): boolean;
+export declare function looksLikeDockerfile(content: string): boolean;
+export declare function looksLikeGithubWorkflow(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

@@ -40,7 +40,9 @@ export function looksLikeIac(content) {
         /^\s*\{\s*"(?:resource|Resources|apiVersion|AWSTemplateFormatVersion)"\s*:/.test(content) ||
         looksLikePulumiYaml(content) ||
         looksLikePulumiProgram(content) ||
-        looksLikeCompose(content));
+        looksLikeCompose(content) ||
+        looksLikeDockerfile(content) ||
+        looksLikeGithubWorkflow(content));
 }
 export function looksLikePulumiProgram(content) {
     return /(?:^|\n)\s*(?:import\s+\*\s+as\s+pulumi\b|import\s+pulumi\b|from\s+["']@pulumi\/)/.test(content);
@@ -48,6 +50,23 @@ export function looksLikePulumiProgram(content) {
 export function looksLikeCompose(content) {
     return /^\s*services\s*:/m.test(content) && /^\s+image\s*:/m.test(content);
 }
+export function looksLikeDockerfile(content) {
+    return /^\s*FROM\s+\S+/m.test(content);
+}
+function topLevelKey(content, name) {
+    return new RegExp(`^(?:"${name}"|'${name}'|${name})\\s*:`, "m").test(content);
+}
+export function looksLikeGithubWorkflow(content) {
+    if (topLevelKey(content, "apiVersion") || topLevelKey(content, "kind"))
+        return false;
+    const topOn = topLevelKey(content, "on");
+    const composeOnly = topLevelKey(content, "services") &&
+        /^\s+image\s*:/m.test(content) &&
+        !topOn;
+    if (composeOnly)
+        return false;
+    return topOn && topLevelKey(content, "jobs");
+}
 export function looksLikePulumiYaml(content) {
     return (/^\s*resources\s*:/m.test(content) &&
         /^\s+type\s*:\s*[A-Za-z0-9_.-]+:\S+/m.test(content));
@@ -55,6 +74,16 @@ export function looksLikePulumiYaml(content) {
 export function parseIac(content, requested = "auto") {
     if (!content.trim())
         throw new ScanInputError("Content must not be empty or whitespace.");
+    // Detect these before HCL or a failing YAML parse so a Dockerfile is not classified as Terraform.
+    if (requested === "dockerfile" ||
+        ((requested === "auto" || requested === "terraform") &&
+            looksLikeDockerfile(content))) {
+        return parseDockerfile(content);
+    }
+    if (requested === "github" ||
+        (requested === "auto" && looksLikeGithubWorkflow(content))) {
+        return parseGithubWorkflow(content);
+    }
     if (requested !== "terraform" &&
         requested !== "compose" &&
         looksLikePulumiProgram(content) &&
@@ -271,7 +300,7 @@ export function parseIac(content, requested = "auto") {
         }
     }
     if (!format)
-        throw new ScanInputError("No IaC document found. Choose Terraform, CloudFormation, Kubernetes, Pulumi, or Docker Compose.");
+        throw new ScanInputError("No IaC document found. Choose Terraform, CloudFormation, Kubernetes, Pulumi, Docker Compose, a Dockerfile, or a GitHub Actions workflow.");
     if (!resources.length)
         warnings.push("No resources found in the submitted document.");
     if (/(?:!(?:Ref|Sub|GetAtt|If)\b|"(?:Ref|Fn::\w+)"\s*:|\$\{)/.test(content))
@@ -352,6 +381,244 @@ function parsePulumiProgram(content) {
         ],
     };
 }
+const submittedTextWarning = "The scan reads the submitted text and does not execute the image or the workflow.";
+function logicalDockerfileLines(content) {
+    const raw = content.split(/\r?\n/);
+    const lines = [];
+    let buffer = "";
+    let start = 1;
+    for (let index = 0; index < raw.length; index++) {
+        const current = raw[index].replace(/\s+$/, "");
+        if (!buffer)
+            start = index + 1;
+        if (current.endsWith("\\")) {
+            buffer += `${current.slice(0, -1)} `;
+            continue;
+        }
+        buffer += current;
+        const text = buffer.trim();
+        buffer = "";
+        if (!text || text.startsWith("#"))
+            continue;
+        lines.push({ line: start, text });
+    }
+    return lines;
+}
+function parseFromInstruction(text) {
+    if (!/^FROM\b/i.test(text))
+        return undefined;
+    const parts = text.split(/\s+/);
+    let index = 1;
+    while (index < parts.length && parts[index].startsWith("--")) {
+        if (!parts[index].includes("="))
+            index += 1;
+        index += 1;
+    }
+    const image = parts[index];
+    if (!image)
+        return undefined;
+    const stage = /^as$/i.test(parts[index + 1] ?? "")
+        ? parts[index + 2]
+        : undefined;
+    return stage ? { image, stage } : { image };
+}
+function imageIsUnpinned(image, stages) {
+    if (image.includes("${") ||
+        image.includes("@") ||
+        stages.has(image) ||
+        stages.has(image.toLowerCase()) ||
+        image.toLowerCase() === "scratch")
+        return false;
+    const slash = image.lastIndexOf("/");
+    const colon = image.lastIndexOf(":");
+    if (colon <= slash)
+        return true;
+    return image.slice(colon + 1) === "latest";
+}
+function parseUserName(text) {
+    if (!/^USER\b/i.test(text))
+        return undefined;
+    const parts = text
+        .split(/\s+/)
+        .slice(1)
+        .filter((part) => part && !part.startsWith("--"));
+    const raw = parts[0];
+    if (!raw)
+        return undefined;
+    return raw.replace(/^["']|["']$/g, "").split(":")[0];
+}
+function runPipesToShell(text) {
+    return (/^RUN\b/i.test(text) &&
+        /\b(?:curl|wget)\b/i.test(text) &&
+        /\|\s*(?:sudo\s+)?(?:\S*\/)?(?:ba)?sh\b/i.test(text));
+}
+function lineLocation(line, path) {
+    return { line, column: 1, path };
+}
+function parseDockerfile(content) {
+    const instructions = logicalDockerfileLines(content);
+    const froms = instructions.flatMap((instruction) => {
+        const parsed = parseFromInstruction(instruction.text);
+        return parsed ? [{ ...instruction, ...parsed }] : [];
+    });
+    if (!froms.length)
+        throw new ScanInputError("A Dockerfile requires a FROM instruction.");
+    const stages = new Set();
+    const resources = [];
+    froms.forEach((from, index) => {
+        resources.push({
+            id: `dockerfile.from.${index + 1}`,
+            type: "dockerfile:from",
+            value: { image: from.image, unpinned: imageIsUnpinned(from.image, stages) },
+            locate: () => lineLocation(from.line, "image"),
+        });
+        if (from.stage) {
+            stages.add(from.stage);
+            stages.add(from.stage.toLowerCase());
+        }
+    });
+    let lastUser;
+    for (const instruction of instructions) {
+        const name = parseUserName(instruction.text);
+        if (name !== undefined)
+            lastUser = { line: instruction.line, name };
+    }
+    const root = !lastUser || lastUser.name.toLowerCase() === "root" || lastUser.name === "0";
+    resources.push({
+        id: "dockerfile.user",
+        type: "dockerfile:user",
+        value: { user: lastUser?.name ?? "", root },
+        locate: () => lineLocation(lastUser?.line ?? froms[0].line, "user"),
+    });
+    instructions.forEach((instruction, index) => {
+        if (!/^RUN\b/i.test(instruction.text))
+            return;
+        resources.push({
+            id: `dockerfile.run.${index + 1}`,
+            type: "dockerfile:run",
+            value: { piped: runPipesToShell(instruction.text) },
+            locate: () => lineLocation(instruction.line, "run"),
+        });
+    });
+    return {
+        format: "dockerfile",
+        resources,
+        warnings: [submittedTextWarning],
+    };
+}
+function triggersPullRequestTarget(value) {
+    if (value === "pull_request_target")
+        return true;
+    if (Array.isArray(value))
+        return value.some((item) => item === "pull_request_target");
+    return Object.prototype.hasOwnProperty.call(object(value), "pull_request_target");
+}
+function workflowChecksOutHead(jobs) {
+    for (const job of Object.values(object(jobs))) {
+        for (const step of array(object(job).steps)) {
+            const inputs = object(object(step).with);
+            const text = [inputs.ref, inputs.sha]
+                .filter((item) => typeof item === "string")
+                .join("\n");
+            if (text.includes("github.event.pull_request.head"))
+                return true;
+        }
+    }
+    return false;
+}
+function permissionsAreBroad(top, jobs, pullRequestTarget) {
+    const values = [
+        top,
+        ...Object.values(object(jobs)).map((job) => object(job).permissions),
+    ];
+    if (values.some((value) => value === "write-all"))
+        return true;
+    if (!pullRequestTarget)
+        return false;
+    return values.some((value) => object(value).contents === "write");
+}
+function parseGithubWorkflow(content) {
+    const lines = new LineCounter();
+    let documents;
+    try {
+        documents = parseAllDocuments(content, {
+            lineCounter: lines,
+            prettyErrors: false,
+        });
+    }
+    catch {
+        throw new ScanInputError("Invalid GitHub Actions workflow YAML.");
+    }
+    const resources = [];
+    let found = false;
+    for (const [documentIndex, doc] of documents.entries()) {
+        if (doc.errors.length)
+            throw new ScanInputError("Invalid GitHub Actions workflow YAML.");
+        let data;
+        try {
+            data = doc.toJS({ maxAliasCount: 0 });
+        }
+        catch {
+            throw new ScanInputError("YAML aliases are not supported. Expand anchors before scanning.");
+        }
+        if (data == null)
+            continue;
+        checkShape(data);
+        const root = object(data);
+        if (!("on" in root) || !("jobs" in root))
+            continue;
+        found = true;
+        const pullRequestTarget = triggersPullRequestTarget(root.on);
+        const checksOutHead = workflowChecksOutHead(root.jobs);
+        const broad = permissionsAreBroad(root.permissions, root.jobs, pullRequestTarget);
+        const position = (path) => {
+            let node = doc.getIn(path, true);
+            if (!node?.range)
+                node = doc.getIn(["on"], true);
+            const at = lines.linePos(node?.range?.[0] ?? doc.range?.[0] ?? 0);
+            return {
+                line: at.line,
+                column: at.col,
+                path: [documentIndex, ...path].join("."),
+            };
+        };
+        resources.push({
+            id: "workflow.on",
+            type: "github:trigger",
+            value: { pullRequestTarget, checksOutHead },
+            locate: () => position(["on"]),
+        });
+        resources.push({
+            id: "workflow.permissions",
+            type: "github:permissions",
+            value: { broad },
+            locate: () => position(root.permissions != null ? ["permissions"] : ["on"]),
+        });
+        for (const [jobName, job] of Object.entries(object(root.jobs))) {
+            array(object(job).steps).forEach((step, index) => {
+                const uses = object(step).uses;
+                if (typeof uses !== "string")
+                    return;
+                const ref = uses.includes("@")
+                    ? uses.slice(uses.lastIndexOf("@") + 1)
+                    : "";
+                resources.push({
+                    id: `jobs.${jobName}.steps.${index}`,
+                    type: "github:action",
+                    value: { uses, floating: ref === "main" || ref === "master" },
+                    locate: () => position(["jobs", jobName, "steps", index, "uses"]),
+                });
+            });
+        }
+    }
+    if (!found)
+        throw new ScanInputError("A GitHub Actions workflow requires top-level on and jobs keys.");
+    return {
+        format: "github",
+        resources,
+        warnings: [submittedTextWarning],
+    };
+}
 function composeServiceMap(root) {
     if (root.apiVersion || root.kind || root.Resources || root.resources)
         return undefined;

package/dist/iac/rules.js

@@ -197,6 +197,48 @@ export const rules = {
         detail: "A role grants wildcard verbs, resources, or API groups.",
         remediation: "Scope the role to the required API groups, resource names, and verbs.",
     },
+    DF001: {
+        category: "security",
+        severity: "high",
+        title: "Dockerfile runs as root",
+        detail: "The last USER is root or 0, or the Dockerfile has no USER instruction.",
+        remediation: "Add a final USER instruction for a non-root account that the image supports.",
+    },
+    DF002: {
+        category: "security",
+        severity: "medium",
+        title: "Unpinned container image",
+        detail: "A FROM image uses the latest tag, or it has no tag and no digest.",
+        remediation: "Pin a release tag or an image digest. Scratch and named stages are not image pins.",
+    },
+    DF003: {
+        category: "security",
+        severity: "high",
+        title: "Remote script piped to a shell",
+        detail: "A RUN instruction downloads a script with curl or wget and pipes it to sh or bash.",
+        remediation: "Download the script, review it, and pin a checksum before running it. Do not pipe a remote script to a shell.",
+    },
+    GH001: {
+        category: "security",
+        severity: "critical",
+        title: "Pull request target checks out PR code",
+        detail: "The workflow triggers on pull_request_target and checks out the pull request head ref or head sha.",
+        remediation: "Avoid checking out pull request head code from pull_request_target. Use pull_request for untrusted code, or check out the base repository only.",
+    },
+    GH002: {
+        category: "security",
+        severity: "high",
+        title: "Workflow permissions are broad",
+        detail: "permissions is write-all, or a pull_request_target workflow sets contents to write.",
+        remediation: "Set the narrowest permissions the workflow needs. Keep contents read-only on pull_request_target.",
+    },
+    GH003: {
+        category: "security",
+        severity: "medium",
+        title: "Action ref is a floating branch",
+        detail: "A uses value is pinned to @main or @master.",
+        remediation: "Pin the action to a release tag or a full commit SHA.",
+    },
 };
 export function resourceFindings(resources, format) {
     const findings = [];
@@ -212,6 +254,14 @@ export function resourceFindings(resources, format) {
                     location: resource.locate(path),
                 });
         };
+        if (format === "dockerfile") {
+            checkDockerfile(resource, check);
+            continue;
+        }
+        if (format === "github") {
+            checkGithub(resource, check);
+            continue;
+        }
         walk(resource.value, (key, value, path, parent) => {
             if (typeof value !== "string")
                 return;
@@ -232,6 +282,23 @@ export function resourceFindings(resources, format) {
     }
     return { findings, checked };
 }
+function checkDockerfile(resource, check) {
+    if (resource.type === "dockerfile:user")
+        check("DF001", resource.value.root === true, ["user"]);
+    else if (resource.type === "dockerfile:from")
+        check("DF002", resource.value.unpinned === true, ["image"]);
+    else if (resource.type === "dockerfile:run")
+        check("DF003", resource.value.piped === true, ["run"]);
+}
+function checkGithub(resource, check) {
+    if (resource.type === "github:trigger")
+        check("GH001", resource.value.pullRequestTarget === true &&
+            resource.value.checksOutHead === true, ["on"]);
+    else if (resource.type === "github:permissions")
+        check("GH002", resource.value.broad === true, ["permissions"]);
+    else if (resource.type === "github:action")
+        check("GH003", resource.value.floating === true, ["uses"]);
+}
 function walk(value, visit, path = []) {
     if (Array.isArray(value))
         value.forEach((child, index) => walk(child, visit, [...path, index]));

package/dist/server.js

@@ -73,12 +73,14 @@ export function createServer(remote = false) {
                 "kubernetes",
                 "pulumi",
                 "compose",
+                "dockerfile",
+                "github",
                 "application",
                 "auto",
             ])
                 .optional()
                 .default("auto")
-                .describe("Content format: terraform, cloudformation, kubernetes, pulumi, compose, application, auto"),
+                .describe("Content format: terraform, cloudformation, kubernetes, pulumi, compose, application, auto. Includes Dockerfile and GitHub Actions."),
         },
     }, async ({ content, focus, format, minSeverity, maxFindings }) => {
         const botHints = fetchPublicBotHints();
@@ -126,7 +128,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. 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 and GitHub Actions. Check security, reliability and cost rules with resource locations and remediation. Pulumi programs are scanned as text.",
             annotations: {
                 readOnlyHint: true,
                 destructiveHint: false,
@@ -134,7 +136,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"),
+                content: contentSchema.describe("Terraform HCL/JSON, CloudFormation YAML/JSON, Kubernetes YAML/JSON, Pulumi YAML or program text, or Docker Compose. Includes Dockerfile and GitHub Actions."),
                 format: z
                     .enum([
                     "terraform",
@@ -142,6 +144,8 @@ export function createServer(remote = false) {
                     "kubernetes",
                     "pulumi",
                     "compose",
+                    "dockerfile",
+                    "github",
                     "auto",
                 ])
                     .default("auto"),

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "0.6.7";
+export declare const VERSION = "0.6.9";

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = "0.6.7";
+export const VERSION = "0.6.9";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skaleagents/swarm",
-  "version": "0.6.7",
+  "version": "0.6.9",
   "private": false,
   "publishConfig": {
     "access": "public"
@@ -15,6 +15,10 @@
     "./scan": {
       "types": "./dist/iac/scan.d.ts",
       "import": "./dist/iac/scan.js"
+    },
+    "./review": {
+      "types": "./dist/review.d.ts",
+      "import": "./dist/review.js"
     }
   },
   "bin": {