flowlint 0.8.2 → 0.8.3

package/dist/cli.js

@@ -23532,6 +23532,30 @@ var metadata7 = {
   description: "Ensures critical paths include logging or alerting steps.",
   details: "For example, a failed payment processing branch should trigger an alert for monitoring."
 };
+function isPathHandled(graph, startNodeId) {
+  const queue = [startNodeId];
+  const visited = /* @__PURE__ */ new Set([startNodeId]);
+  let head = 0;
+  while (head < queue.length) {
+    const currentId = queue[head++];
+    const currentNode = graph.nodes.find((n) => n.id === currentId);
+    if (!currentNode) continue;
+    if (isNotificationNode(currentNode.type) || isErrorHandlerNode(currentNode.type, currentNode.name)) {
+      return true;
+    }
+    if (isRejoinNode(graph, currentId)) {
+      continue;
+    }
+    const outgoing = graph.edges.filter((e) => e.from === currentId);
+    for (const outEdge of outgoing) {
+      if (!visited.has(outEdge.to)) {
+        visited.add(outEdge.to);
+        queue.push(outEdge.to);
+      }
+    }
+  }
+  return false;
+}
 function r7AlertLogEnforcement(graph, ctx) {
   const cfg = ctx.cfg.rules.alert_log_enforcement;
   if (!cfg?.enabled) return [];
@@ -23539,29 +23563,7 @@ function r7AlertLogEnforcement(graph, ctx) {
   const errorEdges = graph.edges.filter((edge) => edge.on === "error");
   for (const edge of errorEdges) {
     const fromNode = graph.nodes.find((n) => n.id === edge.from);
-    let isHandled = false;
-    const queue = [edge.to];
-    const visited = /* @__PURE__ */ new Set([edge.to]);
-    let head = 0;
-    while (head < queue.length) {
-      const currentId = queue[head++];
-      const currentNode = graph.nodes.find((n) => n.id === currentId);
-      if (isNotificationNode(currentNode.type) || isErrorHandlerNode(currentNode.type, currentNode.name)) {
-        isHandled = true;
-        break;
-      }
-      if (isRejoinNode(graph, currentId)) {
-        continue;
-      }
-      const outgoing = graph.edges.filter((e) => e.from === currentId);
-      for (const outEdge of outgoing) {
-        if (!visited.has(outEdge.to)) {
-          visited.add(outEdge.to);
-          queue.push(outEdge.to);
-        }
-      }
-    }
-    if (!isHandled) {
+    if (!isPathHandled(graph, edge.to)) {
       findings.push({
         rule: metadata7.id,
         severity: metadata7.severity,
@@ -23772,7 +23774,7 @@ var r14RetryAfterCompliance = createNodeRule(metadata14.id, metadata14.name, (no
   const waitBetweenTries = node.flags?.waitBetweenTries;
   if (waitBetweenTries !== void 0 && waitBetweenTries !== null) {
     if (typeof waitBetweenTries === "number") return null;
-    if (typeof waitBetweenTries === "string" && !isNaN(Number(waitBetweenTries)) && !waitBetweenTries.includes("{{")) {
+    if (typeof waitBetweenTries === "string" && !Number.isNaN(Number(waitBetweenTries)) && !waitBetweenTries.includes("{{")) {
       return null;
     }
   }
@@ -24249,7 +24251,7 @@ var initCommand = new Command("init").description("Initialize FlowLint configura
 // src/cli.ts
 var import_meta = {};
 var program2 = new Command();
-program2.name("flowlint").description("Static analysis tool for n8n workflows").version("0.8.2");
+program2.name("flowlint").description("Static analysis tool for n8n workflows").version("0.8.3");
 program2.addCommand(scanCommand);
 program2.addCommand(initCommand);
 if (import_meta.url === `file://${process.argv[1]}`) {