@intentius/chant-lexicon-k8s 0.46.0 → 0.50.0

package/dist/rules/wk8042.ts

@@ -7,7 +7,7 @@
  */
 
 import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
-import { getPrimaryOutput, parseK8sManifests } from "./k8s-helpers";
+import { docsToManifests } from "./k8s-helpers";
 
 const PRIVATE_KEY_PATTERN = /-----BEGIN\s+[\w\s]*PRIVATE KEY-----/;
 
@@ -18,27 +18,22 @@ export const wk8042: PostSynthCheck = {
   check(ctx: PostSynthContext): PostSynthDiagnostic[] {
     const diagnostics: PostSynthDiagnostic[] = [];
 
-    for (const [, output] of ctx.outputs) {
-      const yaml = getPrimaryOutput(output);
-      const manifests = parseK8sManifests(yaml);
-
-      for (const manifest of manifests) {
-        if (manifest.kind !== "ConfigMap") continue;
-
-        const resourceName = manifest.metadata?.name ?? "ConfigMap";
-        const data = manifest.data;
-        if (typeof data !== "object" || data === null) continue;
-
-        for (const [key, value] of Object.entries(data)) {
-          if (typeof value === "string" && PRIVATE_KEY_PATTERN.test(value)) {
-            diagnostics.push({
-              checkId: "WK8042",
-              severity: "error",
-              message: `ConfigMap "${resourceName}" contains a private key in data field "${key}" — use a Secret resource instead`,
-              entity: resourceName,
-              lexicon: "k8s",
-            });
-          }
+    for (const manifest of docsToManifests(ctx)) {
+      if (manifest.kind !== "ConfigMap") continue;
+
+      const resourceName = manifest.metadata?.name ?? "ConfigMap";
+      const data = manifest.data;
+      if (typeof data !== "object" || data === null) continue;
+
+      for (const [key, value] of Object.entries(data)) {
+        if (typeof value === "string" && PRIVATE_KEY_PATTERN.test(value)) {
+          diagnostics.push({
+            checkId: "WK8042",
+            severity: "error",
+            message: `ConfigMap "${resourceName}" contains a private key in data field "${key}" — use a Secret resource instead`,
+            entity: resourceName,
+            lexicon: "k8s",
+          });
         }
       }
     }

package/dist/rules/wk8101.ts

@@ -7,7 +7,7 @@
  */
 
 import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
-import { getPrimaryOutput, parseK8sManifests } from "./k8s-helpers";
+import { docsToManifests } from "./k8s-helpers";
 
 export const wk8101: PostSynthCheck = {
   id: "WK8101",
@@ -16,46 +16,41 @@ export const wk8101: PostSynthCheck = {
   check(ctx: PostSynthContext): PostSynthDiagnostic[] {
     const diagnostics: PostSynthDiagnostic[] = [];
 
-    for (const [, output] of ctx.outputs) {
-      const yaml = getPrimaryOutput(output);
-      const manifests = parseK8sManifests(yaml);
-
-      for (const manifest of manifests) {
-        if (manifest.kind !== "Deployment") continue;
-
-        const resourceName = manifest.metadata?.name ?? "Deployment";
-        const spec = manifest.spec;
-        if (!spec) continue;
-
-        const selector = spec.selector as Record<string, unknown> | undefined;
-        const matchLabels = selector?.matchLabels as Record<string, string> | undefined;
-        if (!matchLabels || typeof matchLabels !== "object") continue;
-
-        const template = spec.template as Record<string, unknown> | undefined;
-        const templateMetadata = template?.metadata as Record<string, unknown> | undefined;
-        const templateLabels = templateMetadata?.labels as Record<string, string> | undefined;
+    for (const manifest of docsToManifests(ctx)) {
+      if (manifest.kind !== "Deployment") continue;
+
+      const resourceName = manifest.metadata?.name ?? "Deployment";
+      const spec = manifest.spec;
+      if (!spec) continue;
+
+      const selector = spec.selector as Record<string, unknown> | undefined;
+      const matchLabels = selector?.matchLabels as Record<string, string> | undefined;
+      if (!matchLabels || typeof matchLabels !== "object") continue;
+
+      const template = spec.template as Record<string, unknown> | undefined;
+      const templateMetadata = template?.metadata as Record<string, unknown> | undefined;
+      const templateLabels = templateMetadata?.labels as Record<string, string> | undefined;
+
+      if (!templateLabels || typeof templateLabels !== "object") {
+        diagnostics.push({
+          checkId: "WK8101",
+          severity: "error",
+          message: `Deployment "${resourceName}" has spec.selector.matchLabels but no spec.template.metadata.labels`,
+          entity: resourceName,
+          lexicon: "k8s",
+        });
+        continue;
+      }
 
-        if (!templateLabels || typeof templateLabels !== "object") {
+      for (const [key, value] of Object.entries(matchLabels)) {
+        if (templateLabels[key] !== value) {
           diagnostics.push({
             checkId: "WK8101",
             severity: "error",
-            message: `Deployment "${resourceName}" has spec.selector.matchLabels but no spec.template.metadata.labels`,
+            message: `Deployment "${resourceName}" selector label "${key}=${value}" does not match template label "${key}=${templateLabels[key] ?? '(missing)'}"`,
             entity: resourceName,
             lexicon: "k8s",
           });
-          continue;
-        }
-
-        for (const [key, value] of Object.entries(matchLabels)) {
-          if (templateLabels[key] !== value) {
-            diagnostics.push({
-              checkId: "WK8101",
-              severity: "error",
-              message: `Deployment "${resourceName}" selector label "${key}=${value}" does not match template label "${key}=${templateLabels[key] ?? '(missing)'}"`,
-              entity: resourceName,
-              lexicon: "k8s",
-            });
-          }
         }
       }
     }

package/dist/rules/wk8102.ts

@@ -6,7 +6,7 @@
  */
 
 import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
-import { getPrimaryOutput, parseK8sManifests } from "./k8s-helpers";
+import { docsToManifests } from "./k8s-helpers";
 
 export const wk8102: PostSynthCheck = {
   id: "WK8102",
@@ -15,25 +15,20 @@ export const wk8102: PostSynthCheck = {
   check(ctx: PostSynthContext): PostSynthDiagnostic[] {
     const diagnostics: PostSynthDiagnostic[] = [];
 
-    for (const [, output] of ctx.outputs) {
-      const yaml = getPrimaryOutput(output);
-      const manifests = parseK8sManifests(yaml);
+    for (const manifest of docsToManifests(ctx)) {
+      if (!manifest.kind || !manifest.metadata) continue;
 
-      for (const manifest of manifests) {
-        if (!manifest.kind || !manifest.metadata) continue;
+      const resourceName = manifest.metadata.name ?? manifest.kind;
+      const labels = manifest.metadata.labels;
 
-        const resourceName = manifest.metadata.name ?? manifest.kind;
-        const labels = manifest.metadata.labels;
-
-        if (!labels || typeof labels !== "object" || Object.keys(labels).length === 0) {
-          diagnostics.push({
-            checkId: "WK8102",
-            severity: "warning",
-            message: `${manifest.kind} "${resourceName}" has no metadata.labels — add labels for organizational purposes`,
-            entity: resourceName,
-            lexicon: "k8s",
-          });
-        }
+      if (!labels || typeof labels !== "object" || Object.keys(labels).length === 0) {
+        diagnostics.push({
+          checkId: "WK8102",
+          severity: "warning",
+          message: `${manifest.kind} "${resourceName}" has no metadata.labels — add labels for organizational purposes`,
+          entity: resourceName,
+          lexicon: "k8s",
+        });
       }
     }
 

package/dist/rules/wk8103.ts

@@ -6,7 +6,7 @@
  */
 
 import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
-import { getPrimaryOutput, parseK8sManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
+import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
 
 export const wk8103: PostSynthCheck = {
   id: "WK8103",
@@ -15,27 +15,22 @@ export const wk8103: PostSynthCheck = {
   check(ctx: PostSynthContext): PostSynthDiagnostic[] {
     const diagnostics: PostSynthDiagnostic[] = [];
 
-    for (const [, output] of ctx.outputs) {
-      const yaml = getPrimaryOutput(output);
-      const manifests = parseK8sManifests(yaml);
-
-      for (const manifest of manifests) {
-        if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
-
-        const containers = extractContainers(manifest);
-        const resourceName = manifest.metadata?.name ?? manifest.kind;
-
-        for (let i = 0; i < containers.length; i++) {
-          const container = containers[i];
-          if (!container.name || typeof container.name !== "string" || container.name.trim() === "") {
-            diagnostics.push({
-              checkId: "WK8103",
-              severity: "error",
-              message: `${manifest.kind} "${resourceName}" has a container at index ${i} without a name — the name field is required`,
-              entity: resourceName,
-              lexicon: "k8s",
-            });
-          }
+    for (const manifest of docsToManifests(ctx)) {
+      if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
+
+      const containers = extractContainers(manifest);
+      const resourceName = manifest.metadata?.name ?? manifest.kind;
+
+      for (let i = 0; i < containers.length; i++) {
+        const container = containers[i];
+        if (!container.name || typeof container.name !== "string" || container.name.trim() === "") {
+          diagnostics.push({
+            checkId: "WK8103",
+            severity: "error",
+            message: `${manifest.kind} "${resourceName}" has a container at index ${i} without a name — the name field is required`,
+            entity: resourceName,
+            lexicon: "k8s",
+          });
         }
       }
     }

package/dist/rules/wk8104.ts

@@ -7,7 +7,7 @@
  */
 
 import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
-import { getPrimaryOutput, parseK8sManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
+import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
 
 export const wk8104: PostSynthCheck = {
   id: "WK8104",
@@ -16,45 +16,20 @@ export const wk8104: PostSynthCheck = {
   check(ctx: PostSynthContext): PostSynthDiagnostic[] {
     const diagnostics: PostSynthDiagnostic[] = [];
 
-    for (const [, output] of ctx.outputs) {
-      const yaml = getPrimaryOutput(output);
-      const manifests = parseK8sManifests(yaml);
+    for (const manifest of docsToManifests(ctx)) {
+      const resourceName = manifest.metadata?.name ?? manifest.kind ?? "(unknown)";
 
-      for (const manifest of manifests) {
-        const resourceName = manifest.metadata?.name ?? manifest.kind ?? "(unknown)";
-
-        // Check container ports in workloads
-        if (manifest.kind && WORKLOAD_KINDS.has(manifest.kind)) {
-          const containers = extractContainers(manifest);
-          for (const container of containers) {
-            if (!Array.isArray(container.ports)) continue;
-            for (const port of container.ports) {
-              if (typeof port === "object" && port !== null && !port.name) {
-                diagnostics.push({
-                  checkId: "WK8104",
-                  severity: "warning",
-                  message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" has an unnamed port (containerPort: ${port.containerPort ?? "?"}) — add a name for clarity`,
-                  entity: resourceName,
-                  lexicon: "k8s",
-                });
-              }
-            }
-          }
-        }
-
-        // Check Service ports
-        if (manifest.kind === "Service") {
-          const spec = manifest.spec;
-          if (!spec) continue;
-          const ports = spec.ports as Array<Record<string, unknown>> | undefined;
-          if (!Array.isArray(ports)) continue;
-
-          for (const port of ports) {
+      // Check container ports in workloads
+      if (manifest.kind && WORKLOAD_KINDS.has(manifest.kind)) {
+        const containers = extractContainers(manifest);
+        for (const container of containers) {
+          if (!Array.isArray(container.ports)) continue;
+          for (const port of container.ports) {
             if (typeof port === "object" && port !== null && !port.name) {
               diagnostics.push({
                 checkId: "WK8104",
                 severity: "warning",
-                message: `Service "${resourceName}" has an unnamed port (port: ${port.port ?? "?"}) — add a name for clarity`,
+                message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" has an unnamed port (containerPort: ${port.containerPort ?? "?"}) — add a name for clarity`,
                 entity: resourceName,
                 lexicon: "k8s",
               });
@@ -62,6 +37,26 @@ export const wk8104: PostSynthCheck = {
           }
         }
       }
+
+      // Check Service ports
+      if (manifest.kind === "Service") {
+        const spec = manifest.spec;
+        if (!spec) continue;
+        const ports = spec.ports as Array<Record<string, unknown>> | undefined;
+        if (!Array.isArray(ports)) continue;
+
+        for (const port of ports) {
+          if (typeof port === "object" && port !== null && !port.name) {
+            diagnostics.push({
+              checkId: "WK8104",
+              severity: "warning",
+              message: `Service "${resourceName}" has an unnamed port (port: ${port.port ?? "?"}) — add a name for clarity`,
+              entity: resourceName,
+              lexicon: "k8s",
+            });
+          }
+        }
+      }
     }
 
     return diagnostics;

package/dist/rules/wk8105.ts

@@ -7,7 +7,7 @@
  */
 
 import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
-import { getPrimaryOutput, parseK8sManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
+import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
 
 export const wk8105: PostSynthCheck = {
   id: "WK8105",
@@ -16,26 +16,21 @@ export const wk8105: PostSynthCheck = {
   check(ctx: PostSynthContext): PostSynthDiagnostic[] {
     const diagnostics: PostSynthDiagnostic[] = [];
 
-    for (const [, output] of ctx.outputs) {
-      const yaml = getPrimaryOutput(output);
-      const manifests = parseK8sManifests(yaml);
-
-      for (const manifest of manifests) {
-        if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
-
-        const containers = extractContainers(manifest);
-        const resourceName = manifest.metadata?.name ?? manifest.kind;
-
-        for (const container of containers) {
-          if (!container.imagePullPolicy) {
-            diagnostics.push({
-              checkId: "WK8105",
-              severity: "warning",
-              message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" has no explicit imagePullPolicy — set it to "Always", "IfNotPresent", or "Never"`,
-              entity: resourceName,
-              lexicon: "k8s",
-            });
-          }
+    for (const manifest of docsToManifests(ctx)) {
+      if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
+
+      const containers = extractContainers(manifest);
+      const resourceName = manifest.metadata?.name ?? manifest.kind;
+
+      for (const container of containers) {
+        if (!container.imagePullPolicy) {
+          diagnostics.push({
+            checkId: "WK8105",
+            severity: "warning",
+            message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" has no explicit imagePullPolicy — set it to "Always", "IfNotPresent", or "Never"`,
+            entity: resourceName,
+            lexicon: "k8s",
+          });
         }
       }
     }

package/dist/rules/wk8201.ts

@@ -7,7 +7,7 @@
  */
 
 import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
-import { getPrimaryOutput, parseK8sManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
+import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
 
 export const wk8201: PostSynthCheck = {
   id: "WK8201",
@@ -16,36 +16,31 @@ export const wk8201: PostSynthCheck = {
   check(ctx: PostSynthContext): PostSynthDiagnostic[] {
     const diagnostics: PostSynthDiagnostic[] = [];
 
-    for (const [, output] of ctx.outputs) {
-      const yaml = getPrimaryOutput(output);
-      const manifests = parseK8sManifests(yaml);
-
-      for (const manifest of manifests) {
-        if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
-
-        const containers = extractContainers(manifest);
-        const resourceName = manifest.metadata?.name ?? manifest.kind;
-
-        for (const container of containers) {
-          const limits = container.resources?.limits;
-          const missing: string[] = [];
-
-          if (!limits || typeof limits !== "object") {
-            missing.push("cpu", "memory");
-          } else {
-            if (!limits.cpu) missing.push("cpu");
-            if (!limits.memory) missing.push("memory");
-          }
-
-          if (missing.length > 0) {
-            diagnostics.push({
-              checkId: "WK8201",
-              severity: "warning",
-              message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" is missing resource limits for ${missing.join(", ")}`,
-              entity: resourceName,
-              lexicon: "k8s",
-            });
-          }
+    for (const manifest of docsToManifests(ctx)) {
+      if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
+
+      const containers = extractContainers(manifest);
+      const resourceName = manifest.metadata?.name ?? manifest.kind;
+
+      for (const container of containers) {
+        const limits = container.resources?.limits;
+        const missing: string[] = [];
+
+        if (!limits || typeof limits !== "object") {
+          missing.push("cpu", "memory");
+        } else {
+          if (!limits.cpu) missing.push("cpu");
+          if (!limits.memory) missing.push("memory");
+        }
+
+        if (missing.length > 0) {
+          diagnostics.push({
+            checkId: "WK8201",
+            severity: "warning",
+            message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" is missing resource limits for ${missing.join(", ")}`,
+            entity: resourceName,
+            lexicon: "k8s",
+          });
         }
       }
     }

package/dist/rules/wk8202.ts

@@ -7,7 +7,7 @@
  */
 
 import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
-import { getPrimaryOutput, parseK8sManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
+import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
 
 export const wk8202: PostSynthCheck = {
   id: "WK8202",
@@ -16,27 +16,22 @@ export const wk8202: PostSynthCheck = {
   check(ctx: PostSynthContext): PostSynthDiagnostic[] {
     const diagnostics: PostSynthDiagnostic[] = [];
 
-    for (const [, output] of ctx.outputs) {
-      const yaml = getPrimaryOutput(output);
-      const manifests = parseK8sManifests(yaml);
-
-      for (const manifest of manifests) {
-        if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
-
-        const containers = extractContainers(manifest);
-        const resourceName = manifest.metadata?.name ?? manifest.kind;
-
-        for (const container of containers) {
-          const secCtx = container.securityContext;
-          if (secCtx && secCtx.privileged === true) {
-            diagnostics.push({
-              checkId: "WK8202",
-              severity: "error",
-              message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" runs in privileged mode — this grants full host access and should be avoided`,
-              entity: resourceName,
-              lexicon: "k8s",
-            });
-          }
+    for (const manifest of docsToManifests(ctx)) {
+      if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
+
+      const containers = extractContainers(manifest);
+      const resourceName = manifest.metadata?.name ?? manifest.kind;
+
+      for (const container of containers) {
+        const secCtx = container.securityContext;
+        if (secCtx && secCtx.privileged === true) {
+          diagnostics.push({
+            checkId: "WK8202",
+            severity: "error",
+            message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" runs in privileged mode — this grants full host access and should be avoided`,
+            entity: resourceName,
+            lexicon: "k8s",
+          });
         }
       }
     }

package/dist/rules/wk8203.ts

@@ -7,7 +7,7 @@
  */
 
 import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
-import { getPrimaryOutput, parseK8sManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
+import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
 
 export const wk8203: PostSynthCheck = {
   id: "WK8203",
@@ -16,27 +16,22 @@ export const wk8203: PostSynthCheck = {
   check(ctx: PostSynthContext): PostSynthDiagnostic[] {
     const diagnostics: PostSynthDiagnostic[] = [];
 
-    for (const [, output] of ctx.outputs) {
-      const yaml = getPrimaryOutput(output);
-      const manifests = parseK8sManifests(yaml);
-
-      for (const manifest of manifests) {
-        if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
-
-        const containers = extractContainers(manifest);
-        const resourceName = manifest.metadata?.name ?? manifest.kind;
-
-        for (const container of containers) {
-          const secCtx = container.securityContext;
-          if (!secCtx || secCtx.readOnlyRootFilesystem !== true) {
-            diagnostics.push({
-              checkId: "WK8203",
-              severity: "warning",
-              message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" does not set readOnlyRootFilesystem: true — set it to reduce the attack surface`,
-              entity: resourceName,
-              lexicon: "k8s",
-            });
-          }
+    for (const manifest of docsToManifests(ctx)) {
+      if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
+
+      const containers = extractContainers(manifest);
+      const resourceName = manifest.metadata?.name ?? manifest.kind;
+
+      for (const container of containers) {
+        const secCtx = container.securityContext;
+        if (!secCtx || secCtx.readOnlyRootFilesystem !== true) {
+          diagnostics.push({
+            checkId: "WK8203",
+            severity: "warning",
+            message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" does not set readOnlyRootFilesystem: true — set it to reduce the attack surface`,
+            entity: resourceName,
+            lexicon: "k8s",
+          });
         }
       }
     }