@sfdxy/mule-lint 1.20.0 → 1.22.0

package/dist/src/rules/error-handling/ErrorResponseStructureRule.js

@@ -0,0 +1,73 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ErrorResponseStructureRule = void 0;
+const BaseRule_1 = require("../base/BaseRule");
+const XPathHelper_1 = require("../../core/XPathHelper");
+/**
+ * ERR-003: Error Response Structure
+ *
+ * Error responses should include both a correlationId and a message field
+ * for effective debugging and traceability. Real-world accelerator projects
+ * consistently use a JSON envelope like:
+ *   { correlationId: "...", error: "...", message: "..." }
+ *
+ * This rule checks that ee:set-payload elements inside error handlers
+ * contain both correlationId and message in their DataWeave body.
+ */
+class ErrorResponseStructureRule extends BaseRule_1.BaseRule {
+    id = 'ERR-003';
+    name = 'Error Response Structure';
+    description = 'Error responses should include correlationId and message fields';
+    severity = 'info';
+    category = 'error-handling';
+    issueType = 'bug';
+    validate(doc, _context) {
+        const issues = [];
+        // Find on-error blocks that contain ee:set-payload (i.e., they build an error response)
+        const onErrorBlocks = [
+            ...this.select('//mule:on-error-continue', doc),
+            ...this.select('//mule:on-error-propagate', doc),
+        ];
+        for (const block of onErrorBlocks) {
+            // Find ee:set-payload within this error block
+            const setPayloads = this.select('.//*[local-name()="set-payload" and namespace-uri()="http://www.mulesoft.org/schema/mule/ee/core"]', block);
+            if (setPayloads.length === 0) {
+                continue; // No payload transformation — skip (might be logging-only)
+            }
+            for (const sp of setPayloads) {
+                const content = (0, XPathHelper_1.getTextContent)(sp).toLowerCase();
+                // Also check resource attribute — if present, we can't inspect inline
+                const resourceAttr = this.getAttribute(sp, 'resource');
+                if (resourceAttr) {
+                    continue; // External DWL — inspected by MULE-007 (CorrelationIdRule)
+                }
+                // Skip non-DataWeave content (e.g., static strings)
+                if (!content.includes('dw 2.0') && !content.includes('---')) {
+                    continue;
+                }
+                const hasCorrelationId = content.includes('correlationid');
+                const hasMessage = content.includes('message') || content.includes('detaileddescription');
+                if (!hasCorrelationId && !hasMessage) {
+                    issues.push(this.createIssue(sp, 'Error response payload is missing both correlationId and message fields', {
+                        suggestion: 'Include correlationId and message in the error response for traceability: { correlationId: vars.correlationId, message: error.detailedDescription }',
+                    }));
+                }
+                else if (!hasCorrelationId) {
+                    issues.push(this.createIssue(sp, 'Error response payload is missing correlationId field', {
+                        severity: 'info',
+                        suggestion: 'Add correlationId to the error response: correlationId: vars.correlationId default ""',
+                    }));
+                }
+                else if (!hasMessage) {
+                    issues.push(this.createIssue(sp, 'Error response payload is missing a message or description field', {
+                        severity: 'info',
+                        suggestion: 'Add a message field: message: error.detailedDescription default ""',
+                    }));
+                }
+            }
+        }
+        return issues;
+    }
+}
+exports.ErrorResponseStructureRule = ErrorResponseStructureRule;
+//# sourceMappingURL=ErrorResponseStructureRule.js.map

package/dist/src/rules/error-handling/ErrorResponseStructureRule.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ErrorResponseStructureRule.js","sourceRoot":"","sources":["../../../../src/rules/error-handling/ErrorResponseStructureRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAC5C,wDAAwD;AAExD;;;;;;;;;;GAUG;AACH,MAAa,0BAA2B,SAAQ,mBAAQ;IACtD,EAAE,GAAG,SAAS,CAAC;IACf,IAAI,GAAG,0BAA0B,CAAC;IAClC,WAAW,GAAG,iEAAiE,CAAC;IAChF,QAAQ,GAAG,MAAe,CAAC;IAC3B,QAAQ,GAAG,gBAAyB,CAAC;IACrC,SAAS,GAAc,KAAK,CAAC;IAE7B,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,wFAAwF;QACxF,MAAM,aAAa,GAAG;YACpB,GAAG,IAAI,CAAC,MAAM,CAAC,0BAA0B,EAAE,GAAG,CAAC;YAC/C,GAAG,IAAI,CAAC,MAAM,CAAC,2BAA2B,EAAE,GAAG,CAAC;SACjD,CAAC;QAEF,KAAK,MAAM,KAAK,IAAI,aAAa,EAAE,CAAC;YAClC,8CAA8C;YAC9C,MAAM,WAAW,GAAG,IAAI,CAAC,MAAM,CAC7B,oGAAoG,EACpG,KAAK,CACN,CAAC;YAEF,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;gBAC7B,SAAS,CAAC,2DAA2D;YACvE,CAAC;YAED,KAAK,MAAM,EAAE,IAAI,WAAW,EAAE,CAAC;gBAC7B,MAAM,OAAO,GAAG,IAAA,4BAAc,EAAC,EAAE,CAAC,CAAC,WAAW,EAAE,CAAC;gBAEjD,sEAAsE;gBACtE,MAAM,YAAY,GAAG,IAAI,CAAC,YAAY,CAAC,EAAE,EAAE,UAAU,CAAC,CAAC;gBACvD,IAAI,YAAY,EAAE,CAAC;oBACjB,SAAS,CAAC,2DAA2D;gBACvE,CAAC;gBAED,oDAAoD;gBACpD,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;oBAC5D,SAAS;gBACX,CAAC;gBAED,MAAM,gBAAgB,GAAG,OAAO,CAAC,QAAQ,CAAC,eAAe,CAAC,CAAC;gBAC3D,MAAM,UAAU,GAAG,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAC,IAAI,OAAO,CAAC,QAAQ,CAAC,qBAAqB,CAAC,CAAC;gBAE1F,IAAI,CAAC,gBAAgB,IAAI,CAAC,UAAU,EAAE,CAAC;oBACrC,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,EAAE,EACF,yEAAyE,EACzE;wBACE,UAAU,EACR,qJAAqJ;qBACxJ,CACF,CACF,CAAC;gBACJ,CAAC;qBAAM,IAAI,CAAC,gBAAgB,EAAE,CAAC;oBAC7B,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CAAC,EAAE,EAAE,uDAAuD,EAAE;wBAC5E,QAAQ,EAAE,MAAM;wBAChB,UAAU,EACR,uFAAuF;qBAC1F,CAAC,CACH,CAAC;gBACJ,CAAC;qBAAM,IAAI,CAAC,UAAU,EAAE,CAAC;oBACvB,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,EAAE,EACF,kEAAkE,EAClE;wBACE,QAAQ,EAAE,MAAM;wBAChB,UAAU,EAAE,oEAAoE;qBACjF,CACF,CACF,CAAC;gBACJ,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;CACF;AAjFD,gEAiFC"}

package/dist/src/rules/error-handling/GenericErrorRule.d.ts

@@ -3,8 +3,13 @@ import { BaseRule } from '../base/BaseRule';
 /**
  * MULE-009: Generic Error Type
  *
- * Avoid catching type="ANY" in error handlers.
- * Be specific about error types to handle them appropriately.
+ * Avoid catching type="ANY" in error handlers — unless it is the LAST
+ * on-error block in the error-handler, where it serves as a catch-all
+ * fallback (the accelerator pattern maps this to HTTP 500).
+ *
+ * Enhanced: only flag type="ANY" when it is NOT the last sibling in
+ * its parent error-handler, since a catch-all as the final block is
+ * an accepted MuleSoft best practice.
  */
 export declare class GenericErrorRule extends BaseRule {
     id: string;
@@ -16,8 +21,15 @@ export declare class GenericErrorRule extends BaseRule {
     private readonly GENERIC_TYPES;
     validate(doc: Document, _context: ValidationContext): Issue[];
     /**
-     * Check if handler uses generic error type
+     * Check if handler uses generic error type.
+     * Skip if this handler is the LAST on-error block in its parent
+     * error-handler (catch-all fallback is an accepted pattern).
      */
     private checkGenericType;
+    /**
+     * Returns true if the given on-error node is the last on-error child
+     * of its parent error-handler element.
+     */
+    private isLastOnErrorBlock;
 }
 //# sourceMappingURL=GenericErrorRule.d.ts.map

package/dist/src/rules/error-handling/GenericErrorRule.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"GenericErrorRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/error-handling/GenericErrorRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAClE,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C;;;;;GAKG;AACH,qBAAa,gBAAiB,SAAQ,QAAQ;IAC5C,EAAE,SAAc;IAChB,IAAI,SAAwB;IAC5B,WAAW,SAA+D;IAC1E,QAAQ,EAAG,SAAS,CAAU;IAC9B,QAAQ,EAAG,gBAAgB,CAAU;IACrC,SAAS,EAAE,SAAS,CAAS;IAG7B,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAuB;IAErD,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,iBAAiB,GAAG,KAAK,EAAE;IAoB7D;;OAEG;IACH,OAAO,CAAC,gBAAgB;CAmBzB"}
+{"version":3,"file":"GenericErrorRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/error-handling/GenericErrorRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAClE,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C;;;;;;;;;;GAUG;AACH,qBAAa,gBAAiB,SAAQ,QAAQ;IAC5C,EAAE,SAAc;IAChB,IAAI,SAAwB;IAC5B,WAAW,SAA+D;IAC1E,QAAQ,EAAG,SAAS,CAAU;IAC9B,QAAQ,EAAG,gBAAgB,CAAU;IACrC,SAAS,EAAE,SAAS,CAAS;IAG7B,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAuB;IAErD,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,iBAAiB,GAAG,KAAK,EAAE;IAsB7D;;;;OAIG;IACH,OAAO,CAAC,gBAAgB;IA2BxB;;;OAGG;IACH,OAAO,CAAC,kBAAkB;CAuB3B"}

package/dist/src/rules/error-handling/GenericErrorRule.js

@@ -5,8 +5,13 @@ const BaseRule_1 = require("../base/BaseRule");
 /**
  * MULE-009: Generic Error Type
  *
- * Avoid catching type="ANY" in error handlers.
- * Be specific about error types to handle them appropriately.
+ * Avoid catching type="ANY" in error handlers — unless it is the LAST
+ * on-error block in the error-handler, where it serves as a catch-all
+ * fallback (the accelerator pattern maps this to HTTP 500).
+ *
+ * Enhanced: only flag type="ANY" when it is NOT the last sibling in
+ * its parent error-handler, since a catch-all as the final block is
+ * an accepted MuleSoft best practice.
  */
 class GenericErrorRule extends BaseRule_1.BaseRule {
     id = 'MULE-009';
@@ -19,30 +24,65 @@ class GenericErrorRule extends BaseRule_1.BaseRule {
     GENERIC_TYPES = ['ANY', 'MULE:ANY'];
     validate(doc, _context) {
         const issues = [];
-        // Find on-error-continue with type="ANY"
-        const onErrorContinue = this.select('//mule:on-error-continue[@type]', doc);
-        for (const handler of onErrorContinue) {
-            this.checkGenericType(handler, 'on-error-continue', issues);
-        }
-        // Find on-error-propagate with type="ANY"
-        const onErrorPropagate = this.select('//mule:on-error-propagate[@type]', doc);
-        for (const handler of onErrorPropagate) {
-            this.checkGenericType(handler, 'on-error-propagate', issues);
+        // Find all on-error-continue and on-error-propagate with type attribute
+        const allHandlers = [
+            ...this.select('//mule:on-error-continue[@type]', doc).map((h) => ({
+                node: h,
+                type: 'on-error-continue',
+            })),
+            ...this.select('//mule:on-error-propagate[@type]', doc).map((h) => ({
+                node: h,
+                type: 'on-error-propagate',
+            })),
+        ];
+        for (const handler of allHandlers) {
+            this.checkGenericType(handler.node, handler.type, issues);
         }
         return issues;
     }
     /**
-     * Check if handler uses generic error type
+     * Check if handler uses generic error type.
+     * Skip if this handler is the LAST on-error block in its parent
+     * error-handler (catch-all fallback is an accepted pattern).
      */
     checkGenericType(handler, handlerType, issues) {
         const errorType = this.getAttribute(handler, 'type');
-        if (errorType && this.GENERIC_TYPES.includes(errorType.toUpperCase())) {
-            const docName = this.getDocName(handler);
-            const displayName = docName ? `"${docName}"` : '';
-            issues.push(this.createIssue(handler, `${handlerType} ${displayName} uses generic type="${errorType}"`, {
-                suggestion: 'Catch specific error types (e.g., HTTP:CONNECTIVITY, DB:CONNECTIVITY, VALIDATION:INVALID_JSON) for better error handling',
-            }));
+        if (!errorType || !this.GENERIC_TYPES.includes(errorType.toUpperCase())) {
+            return;
+        }
+        // Check if this is the last on-error sibling in its parent error-handler
+        if (this.isLastOnErrorBlock(handler)) {
+            return; // Catch-all as last block is fine
+        }
+        const docName = this.getDocName(handler);
+        const displayName = docName ? `"${docName}"` : '';
+        issues.push(this.createIssue(handler, `${handlerType} ${displayName} uses generic type="${errorType}" but is not the last error handler — move it to the end or use specific types`, {
+            suggestion: 'If this is a catch-all, place it as the last on-error block. Otherwise, catch specific error types (e.g., HTTP:CONNECTIVITY, DB:CONNECTIVITY)',
+        }));
+    }
+    /**
+     * Returns true if the given on-error node is the last on-error child
+     * of its parent error-handler element.
+     */
+    isLastOnErrorBlock(handler) {
+        const parent = handler.parentNode;
+        if (!parent) {
+            return true; // No parent, can't determine position — assume last
+        }
+        // Collect on-error siblings (continue and propagate)
+        const siblings = parent.childNodes;
+        let lastOnError = null;
+        for (let i = 0; i < siblings.length; i++) {
+            const sibling = siblings[i];
+            if (sibling.nodeType === 1 /* ELEMENT_NODE */) {
+                const el = sibling;
+                const localName = el.localName ?? el.tagName ?? '';
+                if (localName === 'on-error-continue' || localName === 'on-error-propagate') {
+                    lastOnError = sibling;
+                }
+            }
         }
+        return lastOnError === handler;
     }
 }
 exports.GenericErrorRule = GenericErrorRule;

package/dist/src/rules/error-handling/GenericErrorRule.js.map

@@ -1 +1 @@
-{"version":3,"file":"GenericErrorRule.js","sourceRoot":"","sources":["../../../../src/rules/error-handling/GenericErrorRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;GAKG;AACH,MAAa,gBAAiB,SAAQ,mBAAQ;IAC5C,EAAE,GAAG,UAAU,CAAC;IAChB,IAAI,GAAG,oBAAoB,CAAC;IAC5B,WAAW,GAAG,2DAA2D,CAAC;IAC1E,QAAQ,GAAG,SAAkB,CAAC;IAC9B,QAAQ,GAAG,gBAAyB,CAAC;IACrC,SAAS,GAAc,KAAK,CAAC;IAE7B,wBAAwB;IACP,aAAa,GAAG,CAAC,KAAK,EAAE,UAAU,CAAC,CAAC;IAErD,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,yCAAyC;QACzC,MAAM,eAAe,GAAG,IAAI,CAAC,MAAM,CAAC,iCAAiC,EAAE,GAAG,CAAC,CAAC;QAE5E,KAAK,MAAM,OAAO,IAAI,eAAe,EAAE,CAAC;YACtC,IAAI,CAAC,gBAAgB,CAAC,OAAO,EAAE,mBAAmB,EAAE,MAAM,CAAC,CAAC;QAC9D,CAAC;QAED,0CAA0C;QAC1C,MAAM,gBAAgB,GAAG,IAAI,CAAC,MAAM,CAAC,kCAAkC,EAAE,GAAG,CAAC,CAAC;QAE9E,KAAK,MAAM,OAAO,IAAI,gBAAgB,EAAE,CAAC;YACvC,IAAI,CAAC,gBAAgB,CAAC,OAAO,EAAE,oBAAoB,EAAE,MAAM,CAAC,CAAC;QAC/D,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;IAED;;OAEG;IACK,gBAAgB,CAAC,OAAa,EAAE,WAAmB,EAAE,MAAe;QAC1E,MAAM,SAAS,GAAG,IAAI,CAAC,YAAY,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QAErD,IAAI,SAAS,IAAI,IAAI,CAAC,aAAa,CAAC,QAAQ,CAAC,SAAS,CAAC,WAAW,EAAE,CAAC,EAAE,CAAC;YACtE,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC;YACzC,MAAM,WAAW,GAAG,OAAO,CAAC,CAAC,CAAC,IAAI,OAAO,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;YAElD,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,OAAO,EACP,GAAG,WAAW,IAAI,WAAW,uBAAuB,SAAS,GAAG,EAChE;gBACE,UAAU,EACR,0HAA0H;aAC7H,CACF,CACF,CAAC;QACJ,CAAC;IACH,CAAC;CACF;AArDD,4CAqDC"}
+{"version":3,"file":"GenericErrorRule.js","sourceRoot":"","sources":["../../../../src/rules/error-handling/GenericErrorRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;;;;;;GAUG;AACH,MAAa,gBAAiB,SAAQ,mBAAQ;IAC5C,EAAE,GAAG,UAAU,CAAC;IAChB,IAAI,GAAG,oBAAoB,CAAC;IAC5B,WAAW,GAAG,2DAA2D,CAAC;IAC1E,QAAQ,GAAG,SAAkB,CAAC;IAC9B,QAAQ,GAAG,gBAAyB,CAAC;IACrC,SAAS,GAAc,KAAK,CAAC;IAE7B,wBAAwB;IACP,aAAa,GAAG,CAAC,KAAK,EAAE,UAAU,CAAC,CAAC;IAErD,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,wEAAwE;QACxE,MAAM,WAAW,GAAG;YAClB,GAAG,IAAI,CAAC,MAAM,CAAC,iCAAiC,EAAE,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;gBACjE,IAAI,EAAE,CAAC;gBACP,IAAI,EAAE,mBAAmB;aAC1B,CAAC,CAAC;YACH,GAAG,IAAI,CAAC,MAAM,CAAC,kCAAkC,EAAE,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;gBAClE,IAAI,EAAE,CAAC;gBACP,IAAI,EAAE,oBAAoB;aAC3B,CAAC,CAAC;SACJ,CAAC;QAEF,KAAK,MAAM,OAAO,IAAI,WAAW,EAAE,CAAC;YAClC,IAAI,CAAC,gBAAgB,CAAC,OAAO,CAAC,IAAI,EAAE,OAAO,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;QAC5D,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;IAED;;;;OAIG;IACK,gBAAgB,CAAC,OAAa,EAAE,WAAmB,EAAE,MAAe;QAC1E,MAAM,SAAS,GAAG,IAAI,CAAC,YAAY,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QAErD,IAAI,CAAC,SAAS,IAAI,CAAC,IAAI,CAAC,aAAa,CAAC,QAAQ,CAAC,SAAS,CAAC,WAAW,EAAE,CAAC,EAAE,CAAC;YACxE,OAAO;QACT,CAAC;QAED,yEAAyE;QACzE,IAAI,IAAI,CAAC,kBAAkB,CAAC,OAAO,CAAC,EAAE,CAAC;YACrC,OAAO,CAAC,kCAAkC;QAC5C,CAAC;QAED,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC;QACzC,MAAM,WAAW,GAAG,OAAO,CAAC,CAAC,CAAC,IAAI,OAAO,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;QAElD,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,OAAO,EACP,GAAG,WAAW,IAAI,WAAW,uBAAuB,SAAS,gFAAgF,EAC7I;YACE,UAAU,EACR,+IAA+I;SAClJ,CACF,CACF,CAAC;IACJ,CAAC;IAED;;;OAGG;IACK,kBAAkB,CAAC,OAAa;QACtC,MAAM,MAAM,GAAG,OAAO,CAAC,UAAU,CAAC;QAClC,IAAI,CAAC,MAAM,EAAE,CAAC;YACZ,OAAO,IAAI,CAAC,CAAC,oDAAoD;QACnE,CAAC;QAED,qDAAqD;QACrD,MAAM,QAAQ,GAAG,MAAM,CAAC,UAAU,CAAC;QACnC,IAAI,WAAW,GAAgB,IAAI,CAAC;QAEpC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,QAAQ,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YACzC,MAAM,OAAO,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC;YAC5B,IAAI,OAAO,CAAC,QAAQ,KAAK,CAAC,CAAC,kBAAkB,EAAE,CAAC;gBAC9C,MAAM,EAAE,GAAG,OAAkB,CAAC;gBAC9B,MAAM,SAAS,GAAG,EAAE,CAAC,SAAS,IAAI,EAAE,CAAC,OAAO,IAAI,EAAE,CAAC;gBACnD,IAAI,SAAS,KAAK,mBAAmB,IAAI,SAAS,KAAK,oBAAoB,EAAE,CAAC;oBAC5E,WAAW,GAAG,OAAO,CAAC;gBACxB,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO,WAAW,KAAK,OAAO,CAAC;IACjC,CAAC;CACF;AA5FD,4CA4FC"}

package/dist/src/rules/error-handling/GlobalErrorHandlerRule.d.ts

@@ -1,30 +1,29 @@
 import { ValidationContext, Issue, IssueType } from '../../types';
-import { BaseRule } from '../base/BaseRule';
+import { ProjectRule } from '../base/ProjectRule';
 /**
  * MULE-001: Global Error Handler Exists
  *
  * Every Mule project should have a global error handler: either a dedicated
- * file (default: global-error-handler.xml) OR any scanned XML file that
- * contains a named <error-handler> element.
+ * file (default: global-error-handler.xml) OR any XML file in the project
+ * that contains a named <error-handler> element.
  *
- * The rule fires on every scanned XML file that:
- *   1. Contains at least one <flow> or <sub-flow> (is a flow file), AND
- *   2. Does not itself define an <error-handler> element
- *
- * If neither the expected file exists NOR any named <error-handler> has been
- * found in the current document, an issue is raised.
- *
- * Note: the old guard `context.relativePath.includes('global')` has been
- * removed.  Restricting reports to only "global" files hid the rule for
- * projects that did not follow that naming convention.
+ * This is a ProjectRule — it runs once per scan (not per file). It scans all
+ * XML files in the Mule source directory looking for a named error-handler
+ * definition. If none is found and the expected file doesn't exist, it reports
+ * a single project-level issue.
  */
-export declare class GlobalErrorHandlerRule extends BaseRule {
+export declare class GlobalErrorHandlerRule extends ProjectRule {
     id: string;
     name: string;
     description: string;
     severity: "warning";
     category: "error-handling";
     issueType: IssueType;
-    validate(doc: Document, context: ValidationContext): Issue[];
+    protected validateProject(context: ValidationContext): Issue[];
+    /**
+     * Recursively scan a directory for XML files containing a named
+     * <error-handler> or <error-handler ref="..."> element.
+     */
+    private scanForErrorHandler;
 }
 //# sourceMappingURL=GlobalErrorHandlerRule.d.ts.map

package/dist/src/rules/error-handling/GlobalErrorHandlerRule.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"GlobalErrorHandlerRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/error-handling/GlobalErrorHandlerRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAClE,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAI5C;;;;;;;;;;;;;;;;;GAiBG;AACH,qBAAa,sBAAuB,SAAQ,QAAQ;IAClD,EAAE,SAAc;IAChB,IAAI,SAAiC;IACrC,WAAW,SACgF;IAC3F,QAAQ,EAAG,SAAS,CAAU;IAC9B,QAAQ,EAAG,gBAAgB,CAAU;IACrC,SAAS,EAAE,SAAS,CAAS;IAE7B,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,OAAO,EAAE,iBAAiB,GAAG,KAAK,EAAE;CAoD7D"}
+{"version":3,"file":"GlobalErrorHandlerRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/error-handling/GlobalErrorHandlerRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAClE,OAAO,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AAKlD;;;;;;;;;;;GAWG;AACH,qBAAa,sBAAuB,SAAQ,WAAW;IACrD,EAAE,SAAc;IAChB,IAAI,SAAiC;IACrC,WAAW,SACgF;IAC3F,QAAQ,EAAG,SAAS,CAAU;IAC9B,QAAQ,EAAG,gBAAgB,CAAU;IACrC,SAAS,EAAE,SAAS,CAAS;IAE7B,SAAS,CAAC,eAAe,CAAC,OAAO,EAAE,iBAAiB,GAAG,KAAK,EAAE;IAqC9D;;;OAGG;IACH,OAAO,CAAC,mBAAmB;CA8B5B"}

package/dist/src/rules/error-handling/GlobalErrorHandlerRule.js

@@ -34,66 +34,87 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.GlobalErrorHandlerRule = void 0;
-const BaseRule_1 = require("../base/BaseRule");
+const ProjectRule_1 = require("../base/ProjectRule");
 const FileScanner_1 = require("../../core/FileScanner");
 const path = __importStar(require("path"));
+const fs = __importStar(require("fs"));
 /**
  * MULE-001: Global Error Handler Exists
  *
  * Every Mule project should have a global error handler: either a dedicated
- * file (default: global-error-handler.xml) OR any scanned XML file that
- * contains a named <error-handler> element.
+ * file (default: global-error-handler.xml) OR any XML file in the project
+ * that contains a named <error-handler> element.
  *
- * The rule fires on every scanned XML file that:
- *   1. Contains at least one <flow> or <sub-flow> (is a flow file), AND
- *   2. Does not itself define an <error-handler> element
- *
- * If neither the expected file exists NOR any named <error-handler> has been
- * found in the current document, an issue is raised.
- *
- * Note: the old guard `context.relativePath.includes('global')` has been
- * removed.  Restricting reports to only "global" files hid the rule for
- * projects that did not follow that naming convention.
+ * This is a ProjectRule — it runs once per scan (not per file). It scans all
+ * XML files in the Mule source directory looking for a named error-handler
+ * definition. If none is found and the expected file doesn't exist, it reports
+ * a single project-level issue.
  */
-class GlobalErrorHandlerRule extends BaseRule_1.BaseRule {
+class GlobalErrorHandlerRule extends ProjectRule_1.ProjectRule {
     id = 'MULE-001';
     name = 'Global Error Handler Exists';
     description = 'Project should have a global error handler configuration for consistent error handling';
     severity = 'warning';
     category = 'error-handling';
     issueType = 'bug';
-    validate(doc, context) {
-        const issues = [];
+    validateProject(context) {
         // Get configurable expected file path
         const expectedFile = this.getOption(context, 'filePath', 'src/main/mule/global-error-handler.xml');
         const fullPath = path.join(context.projectRoot, expectedFile);
-        // If the dedicated global error handler file exists, the project satisfies
-        // the rule regardless of what is in the current file.
+        // 1. If the dedicated global error handler file exists, the project
+        //    satisfies the rule.
         if ((0, FileScanner_1.fileExists)(fullPath)) {
-            return issues;
+            return [];
         }
-        // The expected file does not exist.  Check whether the current document
-        // itself provides a global error handler via:
-        //   (a) a named <error-handler> element, OR
-        //   (b) a flow that references an error handler by ref attribute
-        const hasNamedErrorHandler = this.exists('//*[local-name()="error-handler"][@name]', doc);
-        if (hasNamedErrorHandler) {
-            return issues;
+        // 2. Scan all XML files in src/main/mule for a named <error-handler>
+        //    element or an <error-handler ref="..."> reference.
+        const muleDir = path.join(context.projectRoot, 'src/main/mule');
+        if (fs.existsSync(muleDir)) {
+            const hasErrorHandler = this.scanForErrorHandler(muleDir);
+            if (hasErrorHandler) {
+                return [];
+            }
         }
-        const hasErrorHandlerRef = this.exists('//*[local-name()="error-handler"][@ref]', doc);
-        if (hasErrorHandlerRef) {
-            return issues;
+        return [
+            this.createProjectIssue(`Global error handler configuration not found. Expected "${expectedFile}" or a named <error-handler> element in any Mule XML file.`, {
+                suggestion: 'Create a global-error-handler.xml file with a named <error-handler> element, or add an <error-handler name="..."> to an existing configuration file',
+            }),
+        ];
+    }
+    /**
+     * Recursively scan a directory for XML files containing a named
+     * <error-handler> or <error-handler ref="..."> element.
+     */
+    scanForErrorHandler(dir) {
+        try {
+            const entries = fs.readdirSync(dir, { withFileTypes: true });
+            for (const entry of entries) {
+                const fullPath = path.join(dir, entry.name);
+                if (entry.isDirectory()) {
+                    if (this.scanForErrorHandler(fullPath)) {
+                        return true;
+                    }
+                }
+                else if (entry.name.endsWith('.xml')) {
+                    try {
+                        const content = fs.readFileSync(fullPath, 'utf-8');
+                        // Quick string check before full parse — avoids parsing every file
+                        if (content.includes('error-handler') &&
+                            (/<error-handler\s[^>]*name\s*=/.test(content) ||
+                                /<error-handler\s[^>]*ref\s*=/.test(content))) {
+                            return true;
+                        }
+                    }
+                    catch {
+                        // Unreadable file, skip
+                    }
+                }
+            }
         }
-        // Only report for files that actually contain flows / sub-flows so that
-        // pure configuration files (e.g. global.xml without flows) are excluded.
-        const hasFlows = this.exists('//*[local-name()="flow" or local-name()="sub-flow"]', doc);
-        if (!hasFlows) {
-            return issues;
+        catch {
+            // Unreadable directory
         }
-        issues.push(this.createFileIssue(`Global error handler configuration not found. Expected "${expectedFile}" or a named <error-handler> element in any flow file.`, {
-            suggestion: 'Create a global-error-handler.xml file with a named <error-handler> element, or add an <error-handler name="..."> to an existing configuration file',
-        }));
-        return issues;
+        return false;
     }
 }
 exports.GlobalErrorHandlerRule = GlobalErrorHandlerRule;

package/dist/src/rules/error-handling/GlobalErrorHandlerRule.js.map

@@ -1 +1 @@
-{"version":3,"file":"GlobalErrorHandlerRule.js","sourceRoot":"","sources":["../../../../src/rules/error-handling/GlobalErrorHandlerRule.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,+CAA4C;AAC5C,wDAAoD;AACpD,2CAA6B;AAE7B;;;;;;;;;;;;;;;;;GAiBG;AACH,MAAa,sBAAuB,SAAQ,mBAAQ;IAClD,EAAE,GAAG,UAAU,CAAC;IAChB,IAAI,GAAG,6BAA6B,CAAC;IACrC,WAAW,GACT,wFAAwF,CAAC;IAC3F,QAAQ,GAAG,SAAkB,CAAC;IAC9B,QAAQ,GAAG,gBAAyB,CAAC;IACrC,SAAS,GAAc,KAAK,CAAC;IAE7B,QAAQ,CAAC,GAAa,EAAE,OAA0B;QAChD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,sCAAsC;QACtC,MAAM,YAAY,GAAG,IAAI,CAAC,SAAS,CACjC,OAAO,EACP,UAAU,EACV,wCAAwC,CACzC,CAAC;QAEF,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC;QAE9D,2EAA2E;QAC3E,sDAAsD;QACtD,IAAI,IAAA,wBAAU,EAAC,QAAQ,CAAC,EAAE,CAAC;YACzB,OAAO,MAAM,CAAC;QAChB,CAAC;QAED,wEAAwE;QACxE,8CAA8C;QAC9C,4CAA4C;QAC5C,iEAAiE;QACjE,MAAM,oBAAoB,GAAG,IAAI,CAAC,MAAM,CAAC,0CAA0C,EAAE,GAAG,CAAC,CAAC;QAC1F,IAAI,oBAAoB,EAAE,CAAC;YACzB,OAAO,MAAM,CAAC;QAChB,CAAC;QAED,MAAM,kBAAkB,GAAG,IAAI,CAAC,MAAM,CAAC,yCAAyC,EAAE,GAAG,CAAC,CAAC;QACvF,IAAI,kBAAkB,EAAE,CAAC;YACvB,OAAO,MAAM,CAAC;QAChB,CAAC;QAED,wEAAwE;QACxE,yEAAyE;QACzE,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,qDAAqD,EAAE,GAAG,CAAC,CAAC;QAEzF,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,OAAO,MAAM,CAAC;QAChB,CAAC;QAED,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,eAAe,CAClB,2DAA2D,YAAY,wDAAwD,EAC/H;YACE,UAAU,EACR,qJAAqJ;SACxJ,CACF,CACF,CAAC;QAEF,OAAO,MAAM,CAAC;IAChB,CAAC;CACF;AA7DD,wDA6DC"}
+{"version":3,"file":"GlobalErrorHandlerRule.js","sourceRoot":"","sources":["../../../../src/rules/error-handling/GlobalErrorHandlerRule.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,qDAAkD;AAClD,wDAAoD;AACpD,2CAA6B;AAC7B,uCAAyB;AAEzB;;;;;;;;;;;GAWG;AACH,MAAa,sBAAuB,SAAQ,yBAAW;IACrD,EAAE,GAAG,UAAU,CAAC;IAChB,IAAI,GAAG,6BAA6B,CAAC;IACrC,WAAW,GACT,wFAAwF,CAAC;IAC3F,QAAQ,GAAG,SAAkB,CAAC;IAC9B,QAAQ,GAAG,gBAAyB,CAAC;IACrC,SAAS,GAAc,KAAK,CAAC;IAEnB,eAAe,CAAC,OAA0B;QAClD,sCAAsC;QACtC,MAAM,YAAY,GAAG,IAAI,CAAC,SAAS,CACjC,OAAO,EACP,UAAU,EACV,wCAAwC,CACzC,CAAC;QAEF,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC;QAE9D,oEAAoE;QACpE,yBAAyB;QACzB,IAAI,IAAA,wBAAU,EAAC,QAAQ,CAAC,EAAE,CAAC;YACzB,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,qEAAqE;QACrE,wDAAwD;QACxD,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,eAAe,CAAC,CAAC;QAChE,IAAI,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,CAAC;YAC3B,MAAM,eAAe,GAAG,IAAI,CAAC,mBAAmB,CAAC,OAAO,CAAC,CAAC;YAC1D,IAAI,eAAe,EAAE,CAAC;gBACpB,OAAO,EAAE,CAAC;YACZ,CAAC;QACH,CAAC;QAED,OAAO;YACL,IAAI,CAAC,kBAAkB,CACrB,2DAA2D,YAAY,4DAA4D,EACnI;gBACE,UAAU,EACR,qJAAqJ;aACxJ,CACF;SACF,CAAC;IACJ,CAAC;IAED;;;OAGG;IACK,mBAAmB,CAAC,GAAW;QACrC,IAAI,CAAC;YACH,MAAM,OAAO,GAAG,EAAE,CAAC,WAAW,CAAC,GAAG,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC,CAAC;YAC7D,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;gBAC5B,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,CAAC,CAAC;gBAC5C,IAAI,KAAK,CAAC,WAAW,EAAE,EAAE,CAAC;oBACxB,IAAI,IAAI,CAAC,mBAAmB,CAAC,QAAQ,CAAC,EAAE,CAAC;wBACvC,OAAO,IAAI,CAAC;oBACd,CAAC;gBACH,CAAC;qBAAM,IAAI,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;oBACvC,IAAI,CAAC;wBACH,MAAM,OAAO,GAAG,EAAE,CAAC,YAAY,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;wBACnD,mEAAmE;wBACnE,IACE,OAAO,CAAC,QAAQ,CAAC,eAAe,CAAC;4BACjC,CAAC,+BAA+B,CAAC,IAAI,CAAC,OAAO,CAAC;gCAC5C,8BAA8B,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,EAC/C,CAAC;4BACD,OAAO,IAAI,CAAC;wBACd,CAAC;oBACH,CAAC;oBAAC,MAAM,CAAC;wBACP,wBAAwB;oBAC1B,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QAAC,MAAM,CAAC;YACP,uBAAuB;QACzB,CAAC;QACD,OAAO,KAAK,CAAC;IACf,CAAC;CACF;AAhFD,wDAgFC"}

package/dist/src/rules/error-handling/TryScopeRule.d.ts

@@ -5,6 +5,10 @@ import { BaseRule } from '../base/BaseRule';
  *
  * Complex operations (DB calls, HTTP requests) should use Try scope
  * for granular error isolation and handling.
+ *
+ * Enhanced to also check sub-flows containing http:request without
+ * Try scope — sub-flows are often called from multiple places and
+ * should handle their own errors for isolation.
  */
 export declare class TryScopeRule extends BaseRule {
     id: string;
@@ -14,5 +18,6 @@ export declare class TryScopeRule extends BaseRule {
     category: "error-handling";
     issueType: IssueType;
     validate(doc: Document, _context: ValidationContext): Issue[];
+    private countRiskyOperations;
 }
 //# sourceMappingURL=TryScopeRule.d.ts.map

package/dist/src/rules/error-handling/TryScopeRule.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"TryScopeRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/error-handling/TryScopeRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAClE,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C;;;;;GAKG;AACH,qBAAa,YAAa,SAAQ,QAAQ;IACxC,EAAE,SAAa;IACf,IAAI,SAA6B;IACjC,WAAW,SAAiE;IAC5E,QAAQ,EAAG,MAAM,CAAU;IAC3B,QAAQ,EAAG,gBAAgB,CAAU;IACrC,SAAS,EAAE,SAAS,CAAS;IAE7B,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,iBAAiB,GAAG,KAAK,EAAE;CAyC9D"}
+{"version":3,"file":"TryScopeRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/error-handling/TryScopeRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAClE,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C;;;;;;;;;GASG;AACH,qBAAa,YAAa,SAAQ,QAAQ;IACxC,EAAE,SAAa;IACf,IAAI,SAA6B;IACjC,WAAW,SAAiE;IAC5E,QAAQ,EAAG,MAAM,CAAU;IAC3B,QAAQ,EAAG,gBAAgB,CAAU;IACrC,SAAS,EAAE,SAAS,CAAS;IAE7B,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,iBAAiB,GAAG,KAAK,EAAE;IA8D7D,OAAO,CAAC,oBAAoB;CAe7B"}

package/dist/src/rules/error-handling/TryScopeRule.js

@@ -7,6 +7,10 @@ const BaseRule_1 = require("../base/BaseRule");
  *
  * Complex operations (DB calls, HTTP requests) should use Try scope
  * for granular error isolation and handling.
+ *
+ * Enhanced to also check sub-flows containing http:request without
+ * Try scope — sub-flows are often called from multiple places and
+ * should handle their own errors for isolation.
  */
 class TryScopeRule extends BaseRule_1.BaseRule {
     id = 'ERR-001';
@@ -17,26 +21,45 @@ class TryScopeRule extends BaseRule_1.BaseRule {
     issueType = 'bug';
     validate(doc, _context) {
         const issues = [];
-        // Find flows with multiple risky operations
-        const flows = this.select('//*[local-name()="flow"]', doc);
+        // Find flows and sub-flows with risky operations
+        const flows = this.select('//mule:flow', doc);
+        const subFlows = this.select('//mule:sub-flow', doc);
         for (const flow of flows) {
             const flowElement = flow;
             const flowName = flowElement.getAttribute('name') ?? 'unnamed';
             // Count risky operations (DB, HTTP, external calls)
-            const dbOperations = this.select('.//*[namespace-uri()="http://www.mulesoft.org/schema/mule/db"]', flow);
-            const httpRequests = this.select('.//*[local-name()="request" and namespace-uri()="http://www.mulesoft.org/schema/mule/http"]', flow);
-            const externalCalls = [...dbOperations, ...httpRequests];
+            const riskyOps = this.countRiskyOperations(flow);
             // Check if Try scope exists
             const tryScopes = this.select('.//*[local-name()="try"]', flow);
             // If multiple external calls but no Try scope
-            if (externalCalls.length >= 2 && tryScopes.length === 0) {
-                issues.push(this.createIssue(flow, `Flow "${flowName}" has ${externalCalls.length} external calls without Try scope isolation`, {
+            if (riskyOps >= 2 && tryScopes.length === 0) {
+                issues.push(this.createIssue(flow, `Flow "${flowName}" has ${String(riskyOps)} external calls without Try scope isolation`, {
                     suggestion: 'Wrap risky operations in Try scope for granular error handling and isolation',
                 }));
             }
         }
+        // Sub-flows with http:request should also use Try scope
+        for (const subFlow of subFlows) {
+            const subFlowElement = subFlow;
+            const subFlowName = subFlowElement.getAttribute('name') ?? 'unnamed';
+            const httpRequests = this.select('.//*[local-name()="request" and namespace-uri()="http://www.mulesoft.org/schema/mule/http"]', subFlow);
+            if (httpRequests.length > 0) {
+                const tryScopes = this.select('.//*[local-name()="try"]', subFlow);
+                if (tryScopes.length === 0) {
+                    issues.push(this.createIssue(subFlow, `Sub-flow "${subFlowName}" contains http:request without Try scope — errors will propagate to all callers`, {
+                        suggestion: 'Wrap the http:request in a Try scope within the sub-flow for isolated error handling',
+                    }));
+                }
+            }
+        }
         return issues;
     }
+    countRiskyOperations(node) {
+        const dbOps = this.select('.//*[namespace-uri()="http://www.mulesoft.org/schema/mule/db"]', node);
+        const httpReqs = this.select('.//*[local-name()="request" and namespace-uri()="http://www.mulesoft.org/schema/mule/http"]', node);
+        const wsConsumers = this.select('.//*[local-name()="consume" and namespace-uri()="http://www.mulesoft.org/schema/mule/wsc"]', node);
+        return dbOps.length + httpReqs.length + wsConsumers.length;
+    }
 }
 exports.TryScopeRule = TryScopeRule;
 //# sourceMappingURL=TryScopeRule.js.map

package/dist/src/rules/error-handling/TryScopeRule.js.map

@@ -1 +1 @@
-{"version":3,"file":"TryScopeRule.js","sourceRoot":"","sources":["../../../../src/rules/error-handling/TryScopeRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;GAKG;AACH,MAAa,YAAa,SAAQ,mBAAQ;IACxC,EAAE,GAAG,SAAS,CAAC;IACf,IAAI,GAAG,yBAAyB,CAAC;IACjC,WAAW,GAAG,6DAA6D,CAAC;IAC5E,QAAQ,GAAG,MAAe,CAAC;IAC3B,QAAQ,GAAG,gBAAyB,CAAC;IACrC,SAAS,GAAc,KAAK,CAAC;IAE7B,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,4CAA4C;QAC5C,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,CAAC,0BAA0B,EAAE,GAAG,CAAC,CAAC;QAE3D,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,MAAM,WAAW,GAAG,IAAe,CAAC;YACpC,MAAM,QAAQ,GAAG,WAAW,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,SAAS,CAAC;YAE/D,oDAAoD;YACpD,MAAM,YAAY,GAAG,IAAI,CAAC,MAAM,CAC9B,gEAAgE,EAChE,IAAI,CACL,CAAC;YACF,MAAM,YAAY,GAAG,IAAI,CAAC,MAAM,CAC9B,6FAA6F,EAC7F,IAAI,CACL,CAAC;YACF,MAAM,aAAa,GAAG,CAAC,GAAG,YAAY,EAAE,GAAG,YAAY,CAAC,CAAC;YAEzD,4BAA4B;YAC5B,MAAM,SAAS,GAAG,IAAI,CAAC,MAAM,CAAC,0BAA0B,EAAE,IAAI,CAAC,CAAC;YAEhE,8CAA8C;YAC9C,IAAI,aAAa,CAAC,MAAM,IAAI,CAAC,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;gBACxD,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,IAAI,EACJ,SAAS,QAAQ,SAAS,aAAa,CAAC,MAAM,6CAA6C,EAC3F;oBACE,UAAU,EACR,8EAA8E;iBACjF,CACF,CACF,CAAC;YACJ,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;CACF;AAjDD,oCAiDC"}
+{"version":3,"file":"TryScopeRule.js","sourceRoot":"","sources":["../../../../src/rules/error-handling/TryScopeRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;;;;;GASG;AACH,MAAa,YAAa,SAAQ,mBAAQ;IACxC,EAAE,GAAG,SAAS,CAAC;IACf,IAAI,GAAG,yBAAyB,CAAC;IACjC,WAAW,GAAG,6DAA6D,CAAC;IAC5E,QAAQ,GAAG,MAAe,CAAC;IAC3B,QAAQ,GAAG,gBAAyB,CAAC;IACrC,SAAS,GAAc,KAAK,CAAC;IAE7B,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,iDAAiD;QACjD,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,CAAC,aAAa,EAAE,GAAG,CAAC,CAAC;QAC9C,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,iBAAiB,EAAE,GAAG,CAAC,CAAC;QAErD,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,MAAM,WAAW,GAAG,IAAe,CAAC;YACpC,MAAM,QAAQ,GAAG,WAAW,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,SAAS,CAAC;YAE/D,oDAAoD;YACpD,MAAM,QAAQ,GAAG,IAAI,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAC;YAEjD,4BAA4B;YAC5B,MAAM,SAAS,GAAG,IAAI,CAAC,MAAM,CAAC,0BAA0B,EAAE,IAAI,CAAC,CAAC;YAEhE,8CAA8C;YAC9C,IAAI,QAAQ,IAAI,CAAC,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;gBAC5C,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,IAAI,EACJ,SAAS,QAAQ,SAAS,MAAM,CAAC,QAAQ,CAAC,6CAA6C,EACvF;oBACE,UAAU,EACR,8EAA8E;iBACjF,CACF,CACF,CAAC;YACJ,CAAC;QACH,CAAC;QAED,wDAAwD;QACxD,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;YAC/B,MAAM,cAAc,GAAG,OAAkB,CAAC;YAC1C,MAAM,WAAW,GAAG,cAAc,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,SAAS,CAAC;YAErE,MAAM,YAAY,GAAG,IAAI,CAAC,MAAM,CAC9B,6FAA6F,EAC7F,OAAO,CACR,CAAC;YAEF,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBAC5B,MAAM,SAAS,GAAG,IAAI,CAAC,MAAM,CAAC,0BAA0B,EAAE,OAAO,CAAC,CAAC;gBACnE,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;oBAC3B,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,OAAO,EACP,aAAa,WAAW,kFAAkF,EAC1G;wBACE,UAAU,EACR,sFAAsF;qBACzF,CACF,CACF,CAAC;gBACJ,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,oBAAoB,CAAC,IAAU;QACrC,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,CACvB,gEAAgE,EAChE,IAAI,CACL,CAAC;QACF,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAC1B,6FAA6F,EAC7F,IAAI,CACL,CAAC;QACF,MAAM,WAAW,GAAG,IAAI,CAAC,MAAM,CAC7B,4FAA4F,EAC5F,IAAI,CACL,CAAC;QACF,OAAO,KAAK,CAAC,MAAM,GAAG,QAAQ,CAAC,MAAM,GAAG,WAAW,CAAC,MAAM,CAAC;IAC7D,CAAC;CACF;AArFD,oCAqFC"}

package/dist/src/rules/http/ConnectionIdleTimeoutRule.d.ts

@@ -0,0 +1,27 @@
+import { ValidationContext, Issue } from '../../types';
+import { BaseRule } from '../base/BaseRule';
+/**
+ * HTTP-004: Connection Idle Timeout
+ *
+ * HTTP request-connection elements should configure an idle timeout
+ * to prevent stale connections from accumulating in the pool.
+ * Without idle timeout, connections may linger and consume resources
+ * even when not actively in use.
+ *
+ * Accelerator pattern:
+ *   <http:request-connection>
+ *     <http:client-socket-properties>
+ *       <sockets:tcp-client-socket-properties connectionTimeout="10000"
+ *                                              clientTimeout="30000"/>
+ *     </http:client-socket-properties>
+ *   </http:request-connection>
+ */
+export declare class ConnectionIdleTimeoutRule extends BaseRule {
+    id: string;
+    name: string;
+    description: string;
+    severity: "info";
+    category: "http";
+    validate(doc: Document, _context: ValidationContext): Issue[];
+}
+//# sourceMappingURL=ConnectionIdleTimeoutRule.d.ts.map

package/dist/src/rules/http/ConnectionIdleTimeoutRule.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"ConnectionIdleTimeoutRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/http/ConnectionIdleTimeoutRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AACvD,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C;;;;;;;;;;;;;;;GAeG;AACH,qBAAa,yBAA0B,SAAQ,QAAQ;IACrD,EAAE,SAAc;IAChB,IAAI,SAA6B;IACjC,WAAW,SACkF;IAC7F,QAAQ,EAAG,MAAM,CAAU;IAC3B,QAAQ,EAAG,MAAM,CAAU;IAE3B,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,iBAAiB,GAAG,KAAK,EAAE;CAiC9D"}

package/dist/src/rules/http/ConnectionIdleTimeoutRule.js

@@ -0,0 +1,46 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ConnectionIdleTimeoutRule = void 0;
+const BaseRule_1 = require("../base/BaseRule");
+/**
+ * HTTP-004: Connection Idle Timeout
+ *
+ * HTTP request-connection elements should configure an idle timeout
+ * to prevent stale connections from accumulating in the pool.
+ * Without idle timeout, connections may linger and consume resources
+ * even when not actively in use.
+ *
+ * Accelerator pattern:
+ *   <http:request-connection>
+ *     <http:client-socket-properties>
+ *       <sockets:tcp-client-socket-properties connectionTimeout="10000"
+ *                                              clientTimeout="30000"/>
+ *     </http:client-socket-properties>
+ *   </http:request-connection>
+ */
+class ConnectionIdleTimeoutRule extends BaseRule_1.BaseRule {
+    id = 'HTTP-004';
+    name = 'Connection Idle Timeout';
+    description = 'HTTP request configs should configure connection/idle timeouts to prevent resource leaks';
+    severity = 'info';
+    category = 'http';
+    validate(doc, _context) {
+        const issues = [];
+        // Find HTTP request-config elements
+        const requestConfigs = this.select('//*[local-name()="request-config"]', doc);
+        for (const config of requestConfigs) {
+            const configName = this.getNameAttribute(config) ?? 'HTTP Request Config';
+            // Check for useSendBuffer, connectionIdleTimeout, or socket properties
+            const hasIdleTimeout = this.getAttribute(config, 'connectionIdleTimeout') !== null;
+            const hasSocketProps = this.exists('.//*[local-name()="tcp-client-socket-properties" or local-name()="client-socket-properties"]', config);
+            if (!hasIdleTimeout && !hasSocketProps) {
+                issues.push(this.createIssue(config, `HTTP request config "${configName}" has no connection idle timeout or socket properties configured`, {
+                    suggestion: 'Add connectionIdleTimeout="30000" on the config, or configure <http:client-socket-properties> with tcp-client-socket-properties for fine-grained control',
+                }));
+            }
+        }
+        return issues;
+    }
+}
+exports.ConnectionIdleTimeoutRule = ConnectionIdleTimeoutRule;
+//# sourceMappingURL=ConnectionIdleTimeoutRule.js.map

package/dist/src/rules/http/ConnectionIdleTimeoutRule.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ConnectionIdleTimeoutRule.js","sourceRoot":"","sources":["../../../../src/rules/http/ConnectionIdleTimeoutRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;;;;;;;;;;;GAeG;AACH,MAAa,yBAA0B,SAAQ,mBAAQ;IACrD,EAAE,GAAG,UAAU,CAAC;IAChB,IAAI,GAAG,yBAAyB,CAAC;IACjC,WAAW,GACT,0FAA0F,CAAC;IAC7F,QAAQ,GAAG,MAAe,CAAC;IAC3B,QAAQ,GAAG,MAAe,CAAC;IAE3B,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,oCAAoC;QACpC,MAAM,cAAc,GAAG,IAAI,CAAC,MAAM,CAAC,oCAAoC,EAAE,GAAG,CAAC,CAAC;QAE9E,KAAK,MAAM,MAAM,IAAI,cAAc,EAAE,CAAC;YACpC,MAAM,UAAU,GAAG,IAAI,CAAC,gBAAgB,CAAC,MAAM,CAAC,IAAI,qBAAqB,CAAC;YAE1E,uEAAuE;YACvE,MAAM,cAAc,GAAG,IAAI,CAAC,YAAY,CAAC,MAAM,EAAE,uBAAuB,CAAC,KAAK,IAAI,CAAC;YAEnF,MAAM,cAAc,GAAG,IAAI,CAAC,MAAM,CAChC,8FAA8F,EAC9F,MAAM,CACP,CAAC;YAEF,IAAI,CAAC,cAAc,IAAI,CAAC,cAAc,EAAE,CAAC;gBACvC,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,MAAM,EACN,wBAAwB,UAAU,kEAAkE,EACpG;oBACE,UAAU,EACR,0JAA0J;iBAC7J,CACF,CACF,CAAC;YACJ,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;CACF;AAzCD,8DAyCC"}

package/dist/src/rules/index.d.ts

@@ -28,7 +28,7 @@ export { AsyncErrorHandlerRule } from './performance/AsyncErrorHandlerRule';
 export { LargeChoiceBlockRule } from './performance/LargeChoiceBlockRule';
 /**
  * All available rules - instantiated and ready to use
- * Total: 56 rules (including operations, resilience, and hygiene rules)
+ * Total: 82 rules (including operations, resilience, hygiene, API-led, connector, and code quality rules)
  */
 export declare const ALL_RULES: Rule[];
 /**