@eventcatalog/cli 0.5.4 → 0.5.5

package/dist/cli/index.mjs

@@ -1248,7 +1248,15 @@ var loadGovernanceConfig = (catalogDir) => {
   }
   const content = fs.readFileSync(configPath, "utf-8");
   const parsed = yaml.load(content);
-  return { rules: parsed?.rules || [] };
+  const rules = parsed?.rules || [];
+  for (const rule of rules) {
+    for (const action of rule.actions) {
+      if (action.type === "fail" && action.message !== void 0 && typeof action.message !== "string") {
+        throw new Error(`Invalid "message" in fail action for rule "${rule.name}". Must be a string.`);
+      }
+    }
+  }
+  return { rules };
 };
 var TRIGGER_FILTERS = {
   consumer_added: (c2) => c2.direction === "receives" && c2.changeType === "added",
@@ -1716,6 +1724,17 @@ var formatGovernanceOutput = (results) => {
   }
   return lines.join("\n");
 };
+var formatFailureOutput = (failures) => {
+  if (failures.length === 0) return "";
+  const lines = [];
+  for (const f of failures) {
+    lines.push(`FAILED: ${f.ruleName}`);
+    for (const msg of f.messages) {
+      lines.push(`  ${msg}`);
+    }
+  }
+  return lines.join("\n");
+};
 
 // src/cli/governance/check.ts
 import path3 from "path";
@@ -1773,9 +1792,16 @@ var governanceCheck = async (opts) => {
     const diff = await baseSDK.diffSnapshots(baseResult.filePath, targetResult.filePath);
     const config = loadGovernanceConfig(dir);
     if (config.rules.length === 0) {
-      return "No governance.yaml (or governance.yml) found or no rules defined.";
+      return { output: "No governance.yaml (or governance.yml) found or no rules defined.", exitCode: 0, failures: [] };
     }
     const results = evaluateGovernanceRules(diff, config, targetResult.snapshot, baseResult.snapshot);
+    for (const result of results) {
+      const failActions = result.rule.actions.filter((a) => a.type === "fail");
+      if (failActions.length > 0) {
+        result.failed = true;
+        result.failMessages = failActions.map((a) => "message" in a && a.message ? resolveEnvVars(a.message) : void 0).filter((m) => m !== void 0);
+      }
+    }
     await enrichSchemaContent(results, baseTmpDir, targetCatalogDir);
     const messageTypes = buildMessageTypeMap(targetResult.snapshot);
     const serviceOwners = buildServiceOwnersMap(targetResult.snapshot);
@@ -1786,8 +1812,20 @@ var governanceCheck = async (opts) => {
       baseRef: baseBranch,
       targetRef: opts.target || "working-directory"
     });
+    const failures = results.filter((r) => r.failed).map((r) => ({ ruleName: r.rule.name, messages: r.failMessages || [] }));
     if (opts.format === "json") {
-      return JSON.stringify({ baseBranch, target: opts.target || "working directory", results, diff: diff.summary }, null, 2);
+      const jsonOutput = {
+        baseBranch,
+        target: opts.target || "working directory",
+        results,
+        summary: {
+          rulesTriggered: results.length,
+          failures: failures.length,
+          passed: failures.length === 0
+        },
+        diff: diff.summary
+      };
+      return { output: JSON.stringify(jsonOutput, null, 2), exitCode: failures.length > 0 ? 1 : 0, failures };
     }
     const targetLabel = opts.target || "working directory";
     const lines = [`Governance check: comparing ${targetLabel} against ${baseBranch}`, ""];
@@ -1803,7 +1841,12 @@ var governanceCheck = async (opts) => {
       lines.push("");
       lines.push(parts.join(", ") + ".");
     }
-    return lines.join("\n");
+    const failureOutput = formatFailureOutput(failures);
+    if (failureOutput) {
+      lines.push("");
+      lines.push(failureOutput);
+    }
+    return { output: lines.join("\n"), exitCode: failures.length > 0 ? 1 : 0, failures };
   } finally {
     for (const d of tempDirs) {
       rmSync2(d, { recursive: true, force: true });
@@ -1929,7 +1972,8 @@ governance.command("check").description("Compare catalog against a base branch a
       status: opts.status,
       dir
     });
-    console.log(result);
+    console.log(result.output);
+    process.exitCode = result.exitCode;
   } catch (error) {
     console.error(error instanceof Error ? error.message : String(error));
     process.exit(1);