security-mcp 1.3.1 → 1.3.4

package/dist/gate/checks/cloud-controls.js

@@ -0,0 +1,69 @@
+import fg from "fast-glob";
+import { readFileSafe } from "../../repo/fs.js";
+import { detectBicep, detectCloudFormation, detectTerraform } from "../cloud-controls/detect.js";
+import { loadCloudRules } from "../cloud-controls/types.js";
+// .tf = Terraform (HCL); .bicep = Bicep; json/yaml/template are CloudFormation/SAM
+// candidates (gated by a content check so arbitrary JSON/YAML is skipped cheaply).
+const GLOBS = ["**/*.tf", "**/*.bicep", "**/*.json", "**/*.yaml", "**/*.yml", "**/*.template"];
+const IGNORE = [
+    "**/node_modules/**",
+    "**/.git/**",
+    "**/dist/**",
+    "**/.claude/**",
+    "src/gate/**"
+];
+function detectForFile(file, text, rules) {
+    if (file.endsWith(".tf"))
+        return detectTerraform(file, text, rules);
+    if (file.endsWith(".bicep"))
+        return detectBicep(file, text, rules);
+    return detectCloudFormation(file, text, rules);
+}
+const MAX_EVIDENCE = 20;
+function toFinding(ruleId, rule, violations) {
+    const files = Array.from(new Set(violations.map((v) => v.file)));
+    return {
+        id: ruleId,
+        title: `${rule.title} — ${rule.threat}`,
+        severity: rule.severity,
+        evidence: violations
+            .slice(0, MAX_EVIDENCE)
+            .map((v) => `${v.file}:${v.line}: ${rule.detect.resourceType} — ${v.reason}`),
+        files,
+        requiredActions: rule.requiredActions
+    };
+}
+/**
+ * Threat-detection pass over the FSBP/CIS cloud-control ruleset. Pure — emits
+ * Findings, never mutates files. Auto-remediation lives in cloud-controls/apply.ts
+ * and is invoked explicitly (CLI `autoharden`), not during the read-only gate.
+ */
+export async function checkCloudControls(opts) {
+    void opts; // matching scans the whole working tree, consistent with checkIac
+    const rules = await loadCloudRules();
+    if (rules.length === 0)
+        return [];
+    const files = await fg(GLOBS, { dot: true, followSymbolicLinks: false, ignore: IGNORE });
+    const byRule = new Map();
+    for (const file of files) {
+        let text = "";
+        try {
+            text = await readFileSafe(file);
+        }
+        catch {
+            continue;
+        }
+        for (const v of detectForFile(file, text, rules)) {
+            const list = byRule.get(v.rule.ruleId);
+            if (list)
+                list.push(v);
+            else
+                byRule.set(v.rule.ruleId, [v]);
+        }
+    }
+    const findings = [];
+    for (const [ruleId, violations] of byRule) {
+        findings.push(toFinding(ruleId, violations[0].rule, violations));
+    }
+    return findings;
+}

package/dist/gate/checks/crypto.js

@@ -186,7 +186,7 @@ async function checkRsaPaddingScheme() {
     }
     return findings;
 }
-async function checkShaUsedForPassword(weakHashHits) {
+async function checkShaUsedForPassword(_weakHashHits) {
     const findings = [];
     // Detect SHA-256/384/512 used in password context
     const shaPasswordHits = await searchRepo({