@sfdxy/mule-lint 1.19.0 → 1.21.0

package/dist/src/rules/logging/LoggerPayloadRule.js

@@ -6,6 +6,14 @@ const BaseRule_1 = require("../base/BaseRule");
  * MULE-301: Logger Payload Reference
  *
  * Loggers should not directly reference #[payload] for security/performance.
+ * This includes:
+ *   - Direct payload reference: #[payload]
+ *   - DataWeave write of full payload: write(payload, 'application/json')
+ *   - output application/json --- payload (full payload serialization)
+ *
+ * Logging the entire payload risks exposing PII/sensitive customer data
+ * (names, addresses, SSNs, credit cards) and can degrade performance for
+ * large payloads.
  */
 class LoggerPayloadRule extends BaseRule_1.BaseRule {
     id = 'MULE-301';
@@ -15,24 +23,60 @@ class LoggerPayloadRule extends BaseRule_1.BaseRule {
     category = 'logging';
     validate(doc, _context) {
         const issues = [];
-        const loggers = this.select('//mule:logger[@message]', doc);
+        // Check standard logger elements with message attribute
+        const loggers = this.select('//*[local-name()="logger"]', doc);
         for (const logger of loggers) {
             const message = this.getAttribute(logger, 'message') ?? '';
-            // Check for direct payload logging
             if (this.hasDirectPayloadReference(message)) {
                 const docName = this.getDocName(logger) ?? 'Logger';
                 issues.push(this.createIssue(logger, `Logger "${docName}" logs entire payload - security/performance risk`, {
-                    suggestion: 'Log specific fields instead: #[payload.orderId]',
+                    suggestion: 'Log specific fields instead: #[payload.orderId] or use a masking DataWeave module',
                 }));
             }
         }
+        // Check ee:transform set-payload inside logger contexts
+        // (Some projects put write(payload,...) in transform message elements)
+        const transforms = this.select('//*[local-name()="transform"]', doc);
+        for (const transform of transforms) {
+            // Check all text content in set-payload / set-variable / message elements
+            const payloadSetters = this.select('.//*[local-name()="set-payload" or local-name()="set-variable"]', transform);
+            for (const setter of payloadSetters) {
+                const content = setter.textContent ?? '';
+                if (this.hasPayloadSerialization(content)) {
+                    const docName = this.getDocName(transform) ?? 'Transform';
+                    issues.push(this.createIssue(transform, `Transform "${docName}" serializes entire payload (write(payload,...)) - PII exposure risk`, {
+                        severity: 'warning',
+                        suggestion: 'Serialize only specific fields or use a masking function to redact sensitive data before logging',
+                    }));
+                    break; // One issue per transform
+                }
+            }
+        }
         return issues;
     }
     hasDirectPayloadReference(message) {
         // Match #[payload] but not #[payload.something]
         return (/#\[payload\s*\]/.test(message) ||
             /#\[\s*payload\s*\]/.test(message) ||
-            message === '#[payload]');
+            message === '#[payload]' ||
+            this.hasPayloadSerialization(message));
+    }
+    /**
+     * Detect DataWeave patterns that serialize the full payload:
+     *   - write(payload, 'application/json')
+     *   - write(payload, "application/json")
+     *   - output application/json --- payload
+     */
+    hasPayloadSerialization(content) {
+        // write(payload, ...) — serializes entire payload to string
+        if (/write\s*\(\s*payload\s*,/.test(content)) {
+            return true;
+        }
+        // output ... --- payload (entire payload as output body)
+        if (/---\s*payload\s*$/.test(content.trim())) {
+            return true;
+        }
+        return false;
     }
 }
 exports.LoggerPayloadRule = LoggerPayloadRule;

package/dist/src/rules/logging/LoggerPayloadRule.js.map

@@ -1 +1 @@
-{"version":3,"file":"LoggerPayloadRule.js","sourceRoot":"","sources":["../../../../src/rules/logging/LoggerPayloadRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;GAIG;AACH,MAAa,iBAAkB,SAAQ,mBAAQ;IAC7C,EAAE,GAAG,UAAU,CAAC;IAChB,IAAI,GAAG,0BAA0B,CAAC;IAClC,WAAW,GAAG,gDAAgD,CAAC;IAC/D,QAAQ,GAAG,SAAkB,CAAC;IAC9B,QAAQ,GAAG,SAAkB,CAAC;IAE9B,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,MAAM,OAAO,GAAG,IAAI,CAAC,MAAM,CAAC,yBAAyB,EAAE,GAAG,CAAC,CAAC;QAE5D,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;YAC7B,MAAM,OAAO,GAAG,IAAI,CAAC,YAAY,CAAC,MAAM,EAAE,SAAS,CAAC,IAAI,EAAE,CAAC;YAE3D,mCAAmC;YACnC,IAAI,IAAI,CAAC,yBAAyB,CAAC,OAAO,CAAC,EAAE,CAAC;gBAC5C,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,MAAM,CAAC,IAAI,QAAQ,CAAC;gBACpD,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,MAAM,EACN,WAAW,OAAO,mDAAmD,EACrE;oBACE,UAAU,EAAE,iDAAiD;iBAC9D,CACF,CACF,CAAC;YACJ,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,yBAAyB,CAAC,OAAe;QAC/C,gDAAgD;QAChD,OAAO,CACL,iBAAiB,CAAC,IAAI,CAAC,OAAO,CAAC;YAC/B,oBAAoB,CAAC,IAAI,CAAC,OAAO,CAAC;YAClC,OAAO,KAAK,YAAY,CACzB,CAAC;IACJ,CAAC;CACF;AAzCD,8CAyCC"}
+{"version":3,"file":"LoggerPayloadRule.js","sourceRoot":"","sources":["../../../../src/rules/logging/LoggerPayloadRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;;;;;;;;GAYG;AACH,MAAa,iBAAkB,SAAQ,mBAAQ;IAC7C,EAAE,GAAG,UAAU,CAAC;IAChB,IAAI,GAAG,0BAA0B,CAAC;IAClC,WAAW,GAAG,gDAAgD,CAAC;IAC/D,QAAQ,GAAG,SAAkB,CAAC;IAC9B,QAAQ,GAAG,SAAkB,CAAC;IAE9B,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,wDAAwD;QACxD,MAAM,OAAO,GAAG,IAAI,CAAC,MAAM,CAAC,4BAA4B,EAAE,GAAG,CAAC,CAAC;QAC/D,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;YAC7B,MAAM,OAAO,GAAG,IAAI,CAAC,YAAY,CAAC,MAAM,EAAE,SAAS,CAAC,IAAI,EAAE,CAAC;YAE3D,IAAI,IAAI,CAAC,yBAAyB,CAAC,OAAO,CAAC,EAAE,CAAC;gBAC5C,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,MAAM,CAAC,IAAI,QAAQ,CAAC;gBACpD,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,MAAM,EACN,WAAW,OAAO,mDAAmD,EACrE;oBACE,UAAU,EACR,mFAAmF;iBACtF,CACF,CACF,CAAC;YACJ,CAAC;QACH,CAAC;QAED,wDAAwD;QACxD,uEAAuE;QACvE,MAAM,UAAU,GAAG,IAAI,CAAC,MAAM,CAAC,+BAA+B,EAAE,GAAG,CAAC,CAAC;QACrE,KAAK,MAAM,SAAS,IAAI,UAAU,EAAE,CAAC;YACnC,0EAA0E;YAC1E,MAAM,cAAc,GAAG,IAAI,CAAC,MAAM,CAChC,iEAAiE,EACjE,SAAqB,CACtB,CAAC;YACF,KAAK,MAAM,MAAM,IAAI,cAAc,EAAE,CAAC;gBACpC,MAAM,OAAO,GAAG,MAAM,CAAC,WAAW,IAAI,EAAE,CAAC;gBACzC,IAAI,IAAI,CAAC,uBAAuB,CAAC,OAAO,CAAC,EAAE,CAAC;oBAC1C,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,SAAS,CAAC,IAAI,WAAW,CAAC;oBAC1D,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,SAAS,EACT,cAAc,OAAO,sEAAsE,EAC3F;wBACE,QAAQ,EAAE,SAAS;wBACnB,UAAU,EACR,kGAAkG;qBACrG,CACF,CACF,CAAC;oBACF,MAAM,CAAC,0BAA0B;gBACnC,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,yBAAyB,CAAC,OAAe;QAC/C,gDAAgD;QAChD,OAAO,CACL,iBAAiB,CAAC,IAAI,CAAC,OAAO,CAAC;YAC/B,oBAAoB,CAAC,IAAI,CAAC,OAAO,CAAC;YAClC,OAAO,KAAK,YAAY;YACxB,IAAI,CAAC,uBAAuB,CAAC,OAAO,CAAC,CACtC,CAAC;IACJ,CAAC;IAED;;;;;OAKG;IACK,uBAAuB,CAAC,OAAe;QAC7C,4DAA4D;QAC5D,IAAI,0BAA0B,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC;YAC7C,OAAO,IAAI,CAAC;QACd,CAAC;QACD,yDAAyD;QACzD,IAAI,mBAAmB,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,EAAE,CAAC;YAC7C,OAAO,IAAI,CAAC;QACd,CAAC;QACD,OAAO,KAAK,CAAC;IACf,CAAC;CACF;AAzFD,8CAyFC"}

package/dist/src/rules/operations/FlowRefTargetExistsRule.d.ts

@@ -0,0 +1,23 @@
+import { ValidationContext, Issue, IssueType } from '../../types';
+import { BaseRule } from '../base/BaseRule';
+/**
+ * HYG-004: Flow-Ref Target Exists
+ *
+ * Every <flow-ref name="X"/> must have a corresponding <flow name="X"/> or
+ * <sub-flow name="X"/> somewhere in the project.  When the engine provides
+ * `context.allFlowRefs` we rely on the pre-scanned flow name map; otherwise
+ * we fall back to intra-file validation only.
+ *
+ * This rule catches broken wiring at lint time, preventing runtime
+ * MULE:ROUTING errors in production.
+ */
+export declare class FlowRefTargetExistsRule extends BaseRule {
+    id: string;
+    name: string;
+    description: string;
+    severity: "error";
+    category: "operations";
+    issueType: IssueType;
+    validate(doc: Document, context: ValidationContext): Issue[];
+}
+//# sourceMappingURL=FlowRefTargetExistsRule.d.ts.map

package/dist/src/rules/operations/FlowRefTargetExistsRule.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"FlowRefTargetExistsRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/operations/FlowRefTargetExistsRule.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,uBAAwB,SAAQ,QAAQ;IACnD,EAAE,SAAa;IACf,IAAI,SAA4B;IAChC,WAAW,SAAgE;IAC3E,QAAQ,EAAG,OAAO,CAAU;IAC5B,QAAQ,EAAG,YAAY,CAAU;IACjC,SAAS,EAAE,SAAS,CAAS;IAE7B,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,OAAO,EAAE,iBAAiB,GAAG,KAAK,EAAE;CA0C7D"}

package/dist/src/rules/operations/FlowRefTargetExistsRule.js

@@ -0,0 +1,58 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.FlowRefTargetExistsRule = void 0;
+const BaseRule_1 = require("../base/BaseRule");
+/**
+ * HYG-004: Flow-Ref Target Exists
+ *
+ * Every <flow-ref name="X"/> must have a corresponding <flow name="X"/> or
+ * <sub-flow name="X"/> somewhere in the project.  When the engine provides
+ * `context.allFlowRefs` we rely on the pre-scanned flow name map; otherwise
+ * we fall back to intra-file validation only.
+ *
+ * This rule catches broken wiring at lint time, preventing runtime
+ * MULE:ROUTING errors in production.
+ */
+class FlowRefTargetExistsRule extends BaseRule_1.BaseRule {
+    id = 'HYG-004';
+    name = 'Flow-Ref Target Exists';
+    description = 'Every flow-ref must reference an existing flow or sub-flow';
+    severity = 'error';
+    category = 'operations';
+    issueType = 'bug';
+    validate(doc, context) {
+        const issues = [];
+        // Collect all flow/sub-flow names defined in this file
+        const localFlowNames = new Set();
+        const flows = this.select('//*[local-name()="flow" or local-name()="sub-flow"]', doc);
+        for (const flow of flows) {
+            const name = this.getNameAttribute(flow);
+            if (name) {
+                localFlowNames.add(name);
+            }
+        }
+        // Collect all flow names across project if available
+        const allFlowNames = context.allFlowNames ?? localFlowNames;
+        // Check every flow-ref
+        const flowRefs = this.select('//*[local-name()="flow-ref"]', doc);
+        for (const ref of flowRefs) {
+            const targetName = this.getNameAttribute(ref);
+            if (!targetName) {
+                continue;
+            }
+            // Skip dynamic flow-refs (DataWeave expressions)
+            if (targetName.includes('#[') || targetName.includes('${')) {
+                continue;
+            }
+            if (!allFlowNames.has(targetName)) {
+                const docName = this.getDocName(ref) ?? targetName;
+                issues.push(this.createIssue(ref, `Flow-ref "${docName}" targets non-existent flow "${targetName}"`, {
+                    suggestion: 'Verify the target flow or sub-flow exists. Check for typos or missing XML files in src/main/mule/',
+                }));
+            }
+        }
+        return issues;
+    }
+}
+exports.FlowRefTargetExistsRule = FlowRefTargetExistsRule;
+//# sourceMappingURL=FlowRefTargetExistsRule.js.map

package/dist/src/rules/operations/FlowRefTargetExistsRule.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"FlowRefTargetExistsRule.js","sourceRoot":"","sources":["../../../../src/rules/operations/FlowRefTargetExistsRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;;;;;;GAUG;AACH,MAAa,uBAAwB,SAAQ,mBAAQ;IACnD,EAAE,GAAG,SAAS,CAAC;IACf,IAAI,GAAG,wBAAwB,CAAC;IAChC,WAAW,GAAG,4DAA4D,CAAC;IAC3E,QAAQ,GAAG,OAAgB,CAAC;IAC5B,QAAQ,GAAG,YAAqB,CAAC;IACjC,SAAS,GAAc,KAAK,CAAC;IAE7B,QAAQ,CAAC,GAAa,EAAE,OAA0B;QAChD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,uDAAuD;QACvD,MAAM,cAAc,GAAG,IAAI,GAAG,EAAU,CAAC;QACzC,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,CAAC,qDAAqD,EAAE,GAAG,CAAC,CAAC;QACtF,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,MAAM,IAAI,GAAG,IAAI,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC;YACzC,IAAI,IAAI,EAAE,CAAC;gBACT,cAAc,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YAC3B,CAAC;QACH,CAAC;QAED,qDAAqD;QACrD,MAAM,YAAY,GAAG,OAAO,CAAC,YAAY,IAAI,cAAc,CAAC;QAE5D,uBAAuB;QACvB,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,8BAA8B,EAAE,GAAG,CAAC,CAAC;QAClE,KAAK,MAAM,GAAG,IAAI,QAAQ,EAAE,CAAC;YAC3B,MAAM,UAAU,GAAG,IAAI,CAAC,gBAAgB,CAAC,GAAG,CAAC,CAAC;YAC9C,IAAI,CAAC,UAAU,EAAE,CAAC;gBAChB,SAAS;YACX,CAAC;YAED,iDAAiD;YACjD,IAAI,UAAU,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,UAAU,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;gBAC3D,SAAS;YACX,CAAC;YAED,IAAI,CAAC,YAAY,CAAC,GAAG,CAAC,UAAU,CAAC,EAAE,CAAC;gBAClC,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,UAAU,CAAC;gBACnD,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CAAC,GAAG,EAAE,aAAa,OAAO,gCAAgC,UAAU,GAAG,EAAE;oBACvF,UAAU,EACR,mGAAmG;iBACtG,CAAC,CACH,CAAC;YACJ,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;CACF;AAlDD,0DAkDC"}

package/dist/src/rules/operations/UnusedFlowRule.d.ts

@@ -4,6 +4,19 @@ import { BaseRule } from '../base/BaseRule';
  * HYG-003: Unused Flow Detection
  *
  * Detects flows that are never referenced by flow-ref.
+ *
+ * Cross-file detection: when the LintEngine provides context.allFlowRefs
+ * (populated during the pre-scan phase), the rule checks across all project
+ * files.  When scanning a standalone file (allFlowRefs is undefined), only
+ * intra-file references are checked.
+ *
+ * The rule recognises several categories of "externally referenced" flows:
+ *   - APIKit-generated flows: verb:\path:config  (routed by apikit:router)
+ *   - Flows with external triggers: http:listener, scheduler, vm:listener,
+ *     salesforce:* listeners (replay-channel-listener, subscribe-channel-listener,
+ *     replay-topic-listener, subscribe-topic-listener, modified-object-listener,
+ *     new-object-listener)
+ *   - Common naming conventions: *-main, *-api, api-*, *-console, *-error-handler, global*
  */
 export declare class UnusedFlowRule extends BaseRule {
     id: string;
@@ -11,7 +24,19 @@ export declare class UnusedFlowRule extends BaseRule {
     description: string;
     severity: "warning";
     category: "standards";
-    validate(doc: Document, _context: ValidationContext): Issue[];
+    validate(doc: Document, context: ValidationContext): Issue[];
+    /**
+     * Check if a flow name matches the APIKit auto-generated naming convention.
+     * Pattern: verb:\path:config-name  (e.g. "get:\orders:api-config")
+     * Also matches with (type) suffix: "get:\orders:api-config(application\json)"
+     */
+    private isApikitFlow;
+    /**
+     * Check if a flow has an external trigger (a source component as the first
+     * processor). This covers HTTP listeners, schedulers, VM listeners,
+     * Salesforce connectors, and any other connector listener.
+     */
+    private hasExternalTrigger;
     private isExternallyReferenced;
 }
 //# sourceMappingURL=UnusedFlowRule.d.ts.map

package/dist/src/rules/operations/UnusedFlowRule.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"UnusedFlowRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/operations/UnusedFlowRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AACvD,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C;;;;GAIG;AACH,qBAAa,cAAe,SAAQ,QAAQ;IAC1C,EAAE,SAAa;IACf,IAAI,SAA2B;IAC/B,WAAW,SAA6C;IACxD,QAAQ,EAAG,SAAS,CAAU;IAC9B,QAAQ,EAAG,WAAW,CAAU;IAEhC,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,iBAAiB,GAAG,KAAK,EAAE;IAmE7D,OAAO,CAAC,sBAAsB;CAY/B"}
+{"version":3,"file":"UnusedFlowRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/operations/UnusedFlowRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AACvD,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C;;;;;;;;;;;;;;;;;GAiBG;AACH,qBAAa,cAAe,SAAQ,QAAQ;IAC1C,EAAE,SAAa;IACf,IAAI,SAA2B;IAC/B,WAAW,SAA6C;IACxD,QAAQ,EAAG,SAAS,CAAU;IAC9B,QAAQ,EAAG,WAAW,CAAU;IAEhC,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,OAAO,EAAE,iBAAiB,GAAG,KAAK,EAAE;IAwE5D;;;;OAIG;IACH,OAAO,CAAC,YAAY;IAKpB;;;;OAIG;IACH,OAAO,CAAC,kBAAkB;IAgD1B,OAAO,CAAC,sBAAsB;CAY/B"}

package/dist/src/rules/operations/UnusedFlowRule.js

@@ -6,6 +6,19 @@ const BaseRule_1 = require("../base/BaseRule");
  * HYG-003: Unused Flow Detection
  *
  * Detects flows that are never referenced by flow-ref.
+ *
+ * Cross-file detection: when the LintEngine provides context.allFlowRefs
+ * (populated during the pre-scan phase), the rule checks across all project
+ * files.  When scanning a standalone file (allFlowRefs is undefined), only
+ * intra-file references are checked.
+ *
+ * The rule recognises several categories of "externally referenced" flows:
+ *   - APIKit-generated flows: verb:\path:config  (routed by apikit:router)
+ *   - Flows with external triggers: http:listener, scheduler, vm:listener,
+ *     salesforce:* listeners (replay-channel-listener, subscribe-channel-listener,
+ *     replay-topic-listener, subscribe-topic-listener, modified-object-listener,
+ *     new-object-listener)
+ *   - Common naming conventions: *-main, *-api, api-*, *-console, *-error-handler, global*
  */
 class UnusedFlowRule extends BaseRule_1.BaseRule {
     id = 'HYG-003';
@@ -13,18 +26,27 @@ class UnusedFlowRule extends BaseRule_1.BaseRule {
     description = 'Detects flows that are never referenced';
     severity = 'warning';
     category = 'standards';
-    validate(doc, _context) {
+    validate(doc, context) {
         const issues = [];
         // Get all flow names in this document
         const flows = this.select('//*[local-name()="flow"]', doc);
         const subflows = this.select('//*[local-name()="sub-flow"]', doc);
-        // Get all flow-ref targets
-        const flowRefs = this.select('//*[local-name()="flow-ref"]', doc);
-        const referencedFlows = new Set();
-        for (const ref of flowRefs) {
-            const name = this.getNameAttribute(ref);
-            if (name) {
-                referencedFlows.add(name);
+        // Build the set of referenced flow names.
+        // When context.allFlowRefs is available (project-wide pre-scan), use it.
+        // Fall back to intra-file refs only for standalone scans.
+        let referencedFlows;
+        if (context.allFlowRefs) {
+            referencedFlows = context.allFlowRefs;
+        }
+        else {
+            // Intra-file only (standalone scan)
+            const flowRefs = this.select('//*[local-name()="flow-ref"]', doc);
+            referencedFlows = new Set();
+            for (const ref of flowRefs) {
+                const name = this.getNameAttribute(ref);
+                if (name) {
+                    referencedFlows.add(name);
+                }
             }
         }
         // Check sub-flows (they should always be referenced)
@@ -33,25 +55,26 @@ class UnusedFlowRule extends BaseRule_1.BaseRule {
             if (name && !referencedFlows.has(name)) {
                 // Exclude common patterns that are referenced externally
                 if (!this.isExternallyReferenced(name)) {
-                    issues.push(this.createIssue(subflow, `Sub-flow "${name}" is never referenced within this file`, {
+                    issues.push(this.createIssue(subflow, `Sub-flow "${name}" is never referenced`, {
                         severity: 'info',
                         suggestion: 'Consider removing unused sub-flows or verify cross-file references',
                     }));
                 }
             }
         }
-        // Check private flows (not triggered by HTTP/scheduler)
+        // Check private flows (not triggered by HTTP/scheduler/connector listeners)
         for (const flow of flows) {
             const name = this.getNameAttribute(flow);
             if (!name) {
                 continue;
             }
-            // Skip if it has an external trigger
-            const hasHttpListener = this.exists('.//*[local-name()="listener"]', flow);
-            const hasScheduler = this.exists('.//*[local-name()="scheduler"]', flow);
-            const hasVmListener = this.exists('.//*[local-name()="listener" and contains(@config-ref, "vm")]', flow);
-            if (hasHttpListener || hasScheduler || hasVmListener) {
-                continue; // Entry point flow
+            // Skip APIKit-generated flows (verb:\path:config pattern)
+            if (this.isApikitFlow(name)) {
+                continue;
+            }
+            // Skip if it has an external trigger (listener / scheduler)
+            if (this.hasExternalTrigger(flow)) {
+                continue;
             }
             // Check if referenced
             if (!referencedFlows.has(name) && !this.isExternallyReferenced(name)) {
@@ -63,6 +86,63 @@ class UnusedFlowRule extends BaseRule_1.BaseRule {
         }
         return issues;
     }
+    /**
+     * Check if a flow name matches the APIKit auto-generated naming convention.
+     * Pattern: verb:\path:config-name  (e.g. "get:\orders:api-config")
+     * Also matches with (type) suffix: "get:\orders:api-config(application\json)"
+     */
+    isApikitFlow(name) {
+        // APIKit flow names contain backslash-separated segments starting with an HTTP verb
+        return /^(get|post|put|patch|delete|head|options|trace):\\.+:.+$/i.test(name);
+    }
+    /**
+     * Check if a flow has an external trigger (a source component as the first
+     * processor). This covers HTTP listeners, schedulers, VM listeners,
+     * Salesforce connectors, and any other connector listener.
+     */
+    hasExternalTrigger(flow) {
+        // http:listener or any namespace listener
+        const hasHttpListener = this.exists('.//*[local-name()="listener"]', flow);
+        if (hasHttpListener) {
+            return true;
+        }
+        // scheduler
+        const hasScheduler = this.exists('.//*[local-name()="scheduler"]', flow);
+        if (hasScheduler) {
+            return true;
+        }
+        // Salesforce-specific listeners (platform events, CDC, polling)
+        const sfListenerPatterns = [
+            'replay-channel-listener',
+            'subscribe-channel-listener',
+            'replay-topic-listener',
+            'subscribe-topic-listener',
+            'modified-object-listener',
+            'new-object-listener',
+        ];
+        for (const pattern of sfListenerPatterns) {
+            if (this.exists(`.//*[local-name()="${pattern}"]`, flow)) {
+                return true;
+            }
+        }
+        // JMS, AMQP, Anypoint MQ, File, FTP, SFTP, Email, DB polling listeners
+        const otherListenerPatterns = [
+            'subscriber', // jms:subscriber, amqp:subscriber
+            'consume', // anypoint-mq:subscriber is actually "subscriber" but some use "consume"
+            'on-new-file', // file:listener
+            'on-new-or-updated-file', // sftp/ftp
+            'listener-imap', // email
+            'listener-pop3', // email
+            'on-new-message', // anypoint-mq
+            'on-table-row', // db polling
+        ];
+        for (const pattern of otherListenerPatterns) {
+            if (this.exists(`.//*[local-name()="${pattern}"]`, flow)) {
+                return true;
+            }
+        }
+        return false;
+    }
     isExternallyReferenced(name) {
         // Common patterns that are typically referenced externally
         const externalPatterns = [

package/dist/src/rules/operations/UnusedFlowRule.js.map

@@ -1 +1 @@
-{"version":3,"file":"UnusedFlowRule.js","sourceRoot":"","sources":["../../../../src/rules/operations/UnusedFlowRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;GAIG;AACH,MAAa,cAAe,SAAQ,mBAAQ;IAC1C,EAAE,GAAG,SAAS,CAAC;IACf,IAAI,GAAG,uBAAuB,CAAC;IAC/B,WAAW,GAAG,yCAAyC,CAAC;IACxD,QAAQ,GAAG,SAAkB,CAAC;IAC9B,QAAQ,GAAG,WAAoB,CAAC;IAEhC,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,sCAAsC;QACtC,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,CAAC,0BAA0B,EAAE,GAAG,CAAC,CAAC;QAC3D,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,8BAA8B,EAAE,GAAG,CAAC,CAAC;QAElE,2BAA2B;QAC3B,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,8BAA8B,EAAE,GAAG,CAAC,CAAC;QAClE,MAAM,eAAe,GAAG,IAAI,GAAG,EAAU,CAAC;QAE1C,KAAK,MAAM,GAAG,IAAI,QAAQ,EAAE,CAAC;YAC3B,MAAM,IAAI,GAAG,IAAI,CAAC,gBAAgB,CAAC,GAAG,CAAC,CAAC;YACxC,IAAI,IAAI,EAAE,CAAC;gBACT,eAAe,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YAC5B,CAAC;QACH,CAAC;QAED,qDAAqD;QACrD,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;YAC/B,MAAM,IAAI,GAAG,IAAI,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC;YAC5C,IAAI,IAAI,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;gBACvC,yDAAyD;gBACzD,IAAI,CAAC,IAAI,CAAC,sBAAsB,CAAC,IAAI,CAAC,EAAE,CAAC;oBACvC,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CAAC,OAAO,EAAE,aAAa,IAAI,wCAAwC,EAAE;wBACnF,QAAQ,EAAE,MAAM;wBAChB,UAAU,EAAE,oEAAoE;qBACjF,CAAC,CACH,CAAC;gBACJ,CAAC;YACH,CAAC;QACH,CAAC;QAED,wDAAwD;QACxD,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,MAAM,IAAI,GAAG,IAAI,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC;YACzC,IAAI,CAAC,IAAI,EAAE,CAAC;gBACV,SAAS;YACX,CAAC;YAED,qCAAqC;YACrC,MAAM,eAAe,GAAG,IAAI,CAAC,MAAM,CAAC,+BAA+B,EAAE,IAAI,CAAC,CAAC;YAC3E,MAAM,YAAY,GAAG,IAAI,CAAC,MAAM,CAAC,gCAAgC,EAAE,IAAI,CAAC,CAAC;YACzE,MAAM,aAAa,GAAG,IAAI,CAAC,MAAM,CAC/B,+DAA+D,EAC/D,IAAI,CACL,CAAC;YAEF,IAAI,eAAe,IAAI,YAAY,IAAI,aAAa,EAAE,CAAC;gBACrD,SAAS,CAAC,mBAAmB;YAC/B,CAAC;YAED,sBAAsB;YACtB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,sBAAsB,CAAC,IAAI,CAAC,EAAE,CAAC;gBACrE,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CAAC,IAAI,EAAE,SAAS,IAAI,0CAA0C,EAAE;oBAC9E,QAAQ,EAAE,MAAM;oBAChB,UAAU,EAAE,qEAAqE;iBAClF,CAAC,CACH,CAAC;YACJ,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,sBAAsB,CAAC,IAAY;QACzC,2DAA2D;QAC3D,MAAM,gBAAgB,GAAG;YACvB,QAAQ;YACR,OAAO;YACP,OAAO;YACP,WAAW;YACX,iBAAiB;YACjB,SAAS;SACV,CAAC;QACF,OAAO,gBAAgB,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;IAChE,CAAC;CACF;AAtFD,wCAsFC"}
+{"version":3,"file":"UnusedFlowRule.js","sourceRoot":"","sources":["../../../../src/rules/operations/UnusedFlowRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;;;;;;;;;;;;;GAiBG;AACH,MAAa,cAAe,SAAQ,mBAAQ;IAC1C,EAAE,GAAG,SAAS,CAAC;IACf,IAAI,GAAG,uBAAuB,CAAC;IAC/B,WAAW,GAAG,yCAAyC,CAAC;IACxD,QAAQ,GAAG,SAAkB,CAAC;IAC9B,QAAQ,GAAG,WAAoB,CAAC;IAEhC,QAAQ,CAAC,GAAa,EAAE,OAA0B;QAChD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,sCAAsC;QACtC,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,CAAC,0BAA0B,EAAE,GAAG,CAAC,CAAC;QAC3D,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,8BAA8B,EAAE,GAAG,CAAC,CAAC;QAElE,0CAA0C;QAC1C,yEAAyE;QACzE,0DAA0D;QAC1D,IAAI,eAA4B,CAAC;QACjC,IAAI,OAAO,CAAC,WAAW,EAAE,CAAC;YACxB,eAAe,GAAG,OAAO,CAAC,WAAW,CAAC;QACxC,CAAC;aAAM,CAAC;YACN,oCAAoC;YACpC,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,8BAA8B,EAAE,GAAG,CAAC,CAAC;YAClE,eAAe,GAAG,IAAI,GAAG,EAAU,CAAC;YACpC,KAAK,MAAM,GAAG,IAAI,QAAQ,EAAE,CAAC;gBAC3B,MAAM,IAAI,GAAG,IAAI,CAAC,gBAAgB,CAAC,GAAG,CAAC,CAAC;gBACxC,IAAI,IAAI,EAAE,CAAC;oBACT,eAAe,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;gBAC5B,CAAC;YACH,CAAC;QACH,CAAC;QAED,qDAAqD;QACrD,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;YAC/B,MAAM,IAAI,GAAG,IAAI,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC;YAC5C,IAAI,IAAI,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;gBACvC,yDAAyD;gBACzD,IAAI,CAAC,IAAI,CAAC,sBAAsB,CAAC,IAAI,CAAC,EAAE,CAAC;oBACvC,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CAAC,OAAO,EAAE,aAAa,IAAI,uBAAuB,EAAE;wBAClE,QAAQ,EAAE,MAAM;wBAChB,UAAU,EAAE,oEAAoE;qBACjF,CAAC,CACH,CAAC;gBACJ,CAAC;YACH,CAAC;QACH,CAAC;QAED,4EAA4E;QAC5E,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,MAAM,IAAI,GAAG,IAAI,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC;YACzC,IAAI,CAAC,IAAI,EAAE,CAAC;gBACV,SAAS;YACX,CAAC;YAED,0DAA0D;YAC1D,IAAI,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,EAAE,CAAC;gBAC5B,SAAS;YACX,CAAC;YAED,4DAA4D;YAC5D,IAAI,IAAI,CAAC,kBAAkB,CAAC,IAAI,CAAC,EAAE,CAAC;gBAClC,SAAS;YACX,CAAC;YAED,sBAAsB;YACtB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,sBAAsB,CAAC,IAAI,CAAC,EAAE,CAAC;gBACrE,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CAAC,IAAI,EAAE,SAAS,IAAI,0CAA0C,EAAE;oBAC9E,QAAQ,EAAE,MAAM;oBAChB,UAAU,EAAE,qEAAqE;iBAClF,CAAC,CACH,CAAC;YACJ,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;IAED;;;;OAIG;IACK,YAAY,CAAC,IAAY;QAC/B,oFAAoF;QACpF,OAAO,2DAA2D,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAChF,CAAC;IAED;;;;OAIG;IACK,kBAAkB,CAAC,IAAU;QACnC,0CAA0C;QAC1C,MAAM,eAAe,GAAG,IAAI,CAAC,MAAM,CAAC,+BAA+B,EAAE,IAAI,CAAC,CAAC;QAC3E,IAAI,eAAe,EAAE,CAAC;YACpB,OAAO,IAAI,CAAC;QACd,CAAC;QAED,YAAY;QACZ,MAAM,YAAY,GAAG,IAAI,CAAC,MAAM,CAAC,gCAAgC,EAAE,IAAI,CAAC,CAAC;QACzE,IAAI,YAAY,EAAE,CAAC;YACjB,OAAO,IAAI,CAAC;QACd,CAAC;QAED,gEAAgE;QAChE,MAAM,kBAAkB,GAAG;YACzB,yBAAyB;YACzB,4BAA4B;YAC5B,uBAAuB;YACvB,0BAA0B;YAC1B,0BAA0B;YAC1B,qBAAqB;SACtB,CAAC;QACF,KAAK,MAAM,OAAO,IAAI,kBAAkB,EAAE,CAAC;YACzC,IAAI,IAAI,CAAC,MAAM,CAAC,sBAAsB,OAAO,IAAI,EAAE,IAAI,CAAC,EAAE,CAAC;gBACzD,OAAO,IAAI,CAAC;YACd,CAAC;QACH,CAAC;QAED,uEAAuE;QACvE,MAAM,qBAAqB,GAAG;YAC5B,YAAY,EAAE,kCAAkC;YAChD,SAAS,EAAE,yEAAyE;YACpF,aAAa,EAAE,gBAAgB;YAC/B,wBAAwB,EAAE,WAAW;YACrC,eAAe,EAAE,QAAQ;YACzB,eAAe,EAAE,QAAQ;YACzB,gBAAgB,EAAE,cAAc;YAChC,cAAc,EAAE,aAAa;SAC9B,CAAC;QACF,KAAK,MAAM,OAAO,IAAI,qBAAqB,EAAE,CAAC;YAC5C,IAAI,IAAI,CAAC,MAAM,CAAC,sBAAsB,OAAO,IAAI,EAAE,IAAI,CAAC,EAAE,CAAC;gBACzD,OAAO,IAAI,CAAC;YACd,CAAC;QACH,CAAC;QAED,OAAO,KAAK,CAAC;IACf,CAAC;IAEO,sBAAsB,CAAC,IAAY;QACzC,2DAA2D;QAC3D,MAAM,gBAAgB,GAAG;YACvB,QAAQ;YACR,OAAO;YACP,OAAO;YACP,WAAW;YACX,iBAAiB;YACjB,SAAS;SACV,CAAC;QACF,OAAO,gBAAgB,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;IAChE,CAAC;CACF;AA1JD,wCA0JC"}

package/dist/src/rules/operations/UnusedVariableRule.d.ts

@@ -0,0 +1,31 @@
+import { ValidationContext, Issue, IssueType } from '../../types';
+import { BaseRule } from '../base/BaseRule';
+/**
+ * HYG-005: Unused Variable
+ *
+ * Detects variables that are set in a flow but never referenced later
+ * in the same flow. This is a common code smell that indicates dead code
+ * or incomplete refactoring.
+ *
+ * Checks for set-variable elements whose variableName is not referenced
+ * by any subsequent expression in the same flow via `vars.variableName`
+ * or `#[vars.variableName]` patterns.
+ *
+ * Note: This is a best-effort heuristic — variables consumed outside the
+ * flow (via flow-ref caller, for example) cannot be detected.
+ */
+export declare class UnusedVariableRule extends BaseRule {
+    id: string;
+    name: string;
+    description: string;
+    severity: "info";
+    category: "operations";
+    issueType: IssueType;
+    /** Well-known variables that are always considered "used" (consumed by connectors/listeners) */
+    private readonly WELL_KNOWN_VARS;
+    validate(doc: Document, _context: ValidationContext): Issue[];
+    private serializeNode;
+    private collectTextContent;
+    private escapeRegex;
+}
+//# sourceMappingURL=UnusedVariableRule.d.ts.map

package/dist/src/rules/operations/UnusedVariableRule.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"UnusedVariableRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/operations/UnusedVariableRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAClE,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C;;;;;;;;;;;;;GAaG;AACH,qBAAa,kBAAmB,SAAQ,QAAQ;IAC9C,EAAE,SAAa;IACf,IAAI,SAAqB;IACzB,WAAW,SAAuE;IAClF,QAAQ,EAAG,MAAM,CAAU;IAC3B,QAAQ,EAAG,YAAY,CAAU;IACjC,SAAS,EAAE,SAAS,CAAgB;IAEpC,gGAAgG;IAChG,OAAO,CAAC,QAAQ,CAAC,eAAe,CAK7B;IAEH,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,iBAAiB,GAAG,KAAK,EAAE;IAuD7D,OAAO,CAAC,aAAa;IAOrB,OAAO,CAAC,kBAAkB;IAqB1B,OAAO,CAAC,WAAW;CAGpB"}

package/dist/src/rules/operations/UnusedVariableRule.js

@@ -0,0 +1,103 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.UnusedVariableRule = void 0;
+const BaseRule_1 = require("../base/BaseRule");
+/**
+ * HYG-005: Unused Variable
+ *
+ * Detects variables that are set in a flow but never referenced later
+ * in the same flow. This is a common code smell that indicates dead code
+ * or incomplete refactoring.
+ *
+ * Checks for set-variable elements whose variableName is not referenced
+ * by any subsequent expression in the same flow via `vars.variableName`
+ * or `#[vars.variableName]` patterns.
+ *
+ * Note: This is a best-effort heuristic — variables consumed outside the
+ * flow (via flow-ref caller, for example) cannot be detected.
+ */
+class UnusedVariableRule extends BaseRule_1.BaseRule {
+    id = 'HYG-005';
+    name = 'Unused Variable';
+    description = 'Variables set in a flow should be referenced within the same flow';
+    severity = 'info';
+    category = 'operations';
+    issueType = 'code-smell';
+    /** Well-known variables that are always considered "used" (consumed by connectors/listeners) */
+    WELL_KNOWN_VARS = new Set([
+        'httpStatus',
+        'outboundHeaders',
+        'statusCode',
+        'correlationId',
+    ]);
+    validate(doc, _context) {
+        const issues = [];
+        const flows = this.select('//mule:flow | //mule:sub-flow', doc);
+        for (const flow of flows) {
+            const flowName = this.getAttribute(flow, 'name') ?? '';
+            // Collect all set-variable declarations in this flow
+            const setVars = this.select('.//*[local-name()="set-variable"]', flow);
+            const variables = [];
+            for (const setVar of setVars) {
+                const varName = this.getAttribute(setVar, 'variableName');
+                if (varName && !this.WELL_KNOWN_VARS.has(varName)) {
+                    variables.push({ name: varName, node: setVar });
+                }
+            }
+            if (variables.length === 0) {
+                continue;
+            }
+            // Get the full text content of the flow to search for references
+            const flowText = this.serializeNode(flow);
+            for (const { name, node } of variables) {
+                // Check for common reference patterns:
+                // vars.name, vars['name'], vars["name"], vars.name
+                const patterns = [`vars.${name}`, `vars['${name}']`, `vars["${name}"]`, `#[vars.${name}]`];
+                const isReferenced = patterns.some((pattern) => {
+                    // Count occurrences — must appear beyond just the set-variable itself
+                    const regex = new RegExp(this.escapeRegex(pattern), 'g');
+                    const matches = flowText.match(regex);
+                    return matches !== null && matches.length > 0;
+                });
+                if (!isReferenced) {
+                    issues.push(this.createIssue(node, `Variable "${name}" is set in flow "${flowName}" but never referenced within the same flow`, {
+                        suggestion: `Remove the unused variable or verify it is consumed by a downstream flow via flow-ref`,
+                    }));
+                }
+            }
+        }
+        return issues;
+    }
+    serializeNode(node) {
+        // Get all text content and attribute values from the node subtree
+        const parts = [];
+        this.collectTextContent(node, parts);
+        return parts.join(' ');
+    }
+    collectTextContent(node, parts) {
+        if (node.nodeType === 3 /* TEXT_NODE */) {
+            parts.push(node.textContent ?? '');
+        }
+        else if (node.nodeType === 1 /* ELEMENT_NODE */) {
+            // Include attribute values (expressions live in attributes)
+            const element = node;
+            if (element.attributes) {
+                for (let i = 0; i < element.attributes.length; i++) {
+                    parts.push(element.attributes[i].value);
+                }
+            }
+            // Recurse into children
+            const children = node.childNodes;
+            if (children) {
+                for (let i = 0; i < children.length; i++) {
+                    this.collectTextContent(children[i], parts);
+                }
+            }
+        }
+    }
+    escapeRegex(str) {
+        return str.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+    }
+}
+exports.UnusedVariableRule = UnusedVariableRule;
+//# sourceMappingURL=UnusedVariableRule.js.map

package/dist/src/rules/operations/UnusedVariableRule.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"UnusedVariableRule.js","sourceRoot":"","sources":["../../../../src/rules/operations/UnusedVariableRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;;;;;;;;;GAaG;AACH,MAAa,kBAAmB,SAAQ,mBAAQ;IAC9C,EAAE,GAAG,SAAS,CAAC;IACf,IAAI,GAAG,iBAAiB,CAAC;IACzB,WAAW,GAAG,mEAAmE,CAAC;IAClF,QAAQ,GAAG,MAAe,CAAC;IAC3B,QAAQ,GAAG,YAAqB,CAAC;IACjC,SAAS,GAAc,YAAY,CAAC;IAEpC,gGAAgG;IAC/E,eAAe,GAAG,IAAI,GAAG,CAAC;QACzC,YAAY;QACZ,iBAAiB;QACjB,YAAY;QACZ,eAAe;KAChB,CAAC,CAAC;IAEH,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,CAAC,+BAA+B,EAAE,GAAG,CAAC,CAAC;QAEhE,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,MAAM,QAAQ,GAAG,IAAI,CAAC,YAAY,CAAC,IAAI,EAAE,MAAM,CAAC,IAAI,EAAE,CAAC;YAEvD,qDAAqD;YACrD,MAAM,OAAO,GAAG,IAAI,CAAC,MAAM,CAAC,mCAAmC,EAAE,IAAI,CAAC,CAAC;YACvE,MAAM,SAAS,GAAwC,EAAE,CAAC;YAE1D,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;gBAC7B,MAAM,OAAO,GAAG,IAAI,CAAC,YAAY,CAAC,MAAM,EAAE,cAAc,CAAC,CAAC;gBAC1D,IAAI,OAAO,IAAI,CAAC,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;oBAClD,SAAS,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC,CAAC;gBAClD,CAAC;YACH,CAAC;YAED,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;gBAC3B,SAAS;YACX,CAAC;YAED,iEAAiE;YACjE,MAAM,QAAQ,GAAG,IAAI,CAAC,aAAa,CAAC,IAAI,CAAC,CAAC;YAE1C,KAAK,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,SAAS,EAAE,CAAC;gBACvC,uCAAuC;gBACvC,mDAAmD;gBACnD,MAAM,QAAQ,GAAG,CAAC,QAAQ,IAAI,EAAE,EAAE,SAAS,IAAI,IAAI,EAAE,SAAS,IAAI,IAAI,EAAE,UAAU,IAAI,GAAG,CAAC,CAAC;gBAE3F,MAAM,YAAY,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE;oBAC7C,sEAAsE;oBACtE,MAAM,KAAK,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,WAAW,CAAC,OAAO,CAAC,EAAE,GAAG,CAAC,CAAC;oBACzD,MAAM,OAAO,GAAG,QAAQ,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;oBACtC,OAAO,OAAO,KAAK,IAAI,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC;gBAChD,CAAC,CAAC,CAAC;gBAEH,IAAI,CAAC,YAAY,EAAE,CAAC;oBAClB,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,IAAI,EACJ,aAAa,IAAI,qBAAqB,QAAQ,6CAA6C,EAC3F;wBACE,UAAU,EAAE,uFAAuF;qBACpG,CACF,CACF,CAAC;gBACJ,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,aAAa,CAAC,IAAU;QAC9B,kEAAkE;QAClE,MAAM,KAAK,GAAa,EAAE,CAAC;QAC3B,IAAI,CAAC,kBAAkB,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;QACrC,OAAO,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACzB,CAAC;IAEO,kBAAkB,CAAC,IAAU,EAAE,KAAe;QACpD,IAAI,IAAI,CAAC,QAAQ,KAAK,CAAC,CAAC,eAAe,EAAE,CAAC;YACxC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,IAAI,EAAE,CAAC,CAAC;QACrC,CAAC;aAAM,IAAI,IAAI,CAAC,QAAQ,KAAK,CAAC,CAAC,kBAAkB,EAAE,CAAC;YAClD,4DAA4D;YAC5D,MAAM,OAAO,GAAG,IAA0B,CAAC;YAC3C,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;gBACvB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,OAAO,CAAC,UAAU,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;oBACnD,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;gBAC1C,CAAC;YACH,CAAC;YACD,wBAAwB;YACxB,MAAM,QAAQ,GAAG,IAAI,CAAC,UAAU,CAAC;YACjC,IAAI,QAAQ,EAAE,CAAC;gBACb,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,QAAQ,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;oBACzC,IAAI,CAAC,kBAAkB,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC;gBAC9C,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;IAEO,WAAW,CAAC,GAAW;QAC7B,OAAO,GAAG,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,CAAC;IACpD,CAAC;CACF;AAtGD,gDAsGC"}

package/dist/src/rules/performance/ConnectionPoolingRule.d.ts

@@ -5,6 +5,11 @@ import { BaseRule } from '../base/BaseRule';
  *
  * DB and HTTP connectors should configure connection pools
  * for optimal performance and resource management.
+ *
+ * Per the Mule HTTP connector XSD, `maxConnections` and `connectionIdleTimeout`
+ * are valid on BOTH `<http:request-config>` (older style) and on the nested
+ * `<http:request-connection>` child element (current XSD-correct placement).
+ * The rule must accept either location to avoid false positives.
  */
 export declare class ConnectionPoolingRule extends BaseRule {
     id: string;

package/dist/src/rules/performance/ConnectionPoolingRule.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ConnectionPoolingRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/performance/ConnectionPoolingRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AACvD,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C;;;;;GAKG;AACH,qBAAa,qBAAsB,SAAQ,QAAQ;IACjD,EAAE,SAAc;IAChB,IAAI,SAAwB;IAC5B,WAAW,SAA0D;IACrE,QAAQ,EAAG,SAAS,CAAU;IAC9B,QAAQ,EAAG,aAAa,CAAU;IAElC,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,iBAAiB,GAAG,KAAK,EAAE;CA8D9D"}
+{"version":3,"file":"ConnectionPoolingRule.d.ts","sourceRoot":"","sources":["../../../../src/rules/performance/ConnectionPoolingRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AACvD,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C;;;;;;;;;;GAUG;AACH,qBAAa,qBAAsB,SAAQ,QAAQ;IACjD,EAAE,SAAc;IAChB,IAAI,SAAwB;IAC5B,WAAW,SAA0D;IACrE,QAAQ,EAAG,SAAS,CAAU;IAC9B,QAAQ,EAAG,aAAa,CAAU;IAElC,QAAQ,CAAC,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,iBAAiB,GAAG,KAAK,EAAE;CA2E9D"}

package/dist/src/rules/performance/ConnectionPoolingRule.js

@@ -7,6 +7,11 @@ const BaseRule_1 = require("../base/BaseRule");
  *
  * DB and HTTP connectors should configure connection pools
  * for optimal performance and resource management.
+ *
+ * Per the Mule HTTP connector XSD, `maxConnections` and `connectionIdleTimeout`
+ * are valid on BOTH `<http:request-config>` (older style) and on the nested
+ * `<http:request-connection>` child element (current XSD-correct placement).
+ * The rule must accept either location to avoid false positives.
  */
 class ConnectionPoolingRule extends BaseRule_1.BaseRule {
     id = 'PERF-002';
@@ -21,13 +26,21 @@ class ConnectionPoolingRule extends BaseRule_1.BaseRule {
         for (const config of httpConfigs) {
             const element = config;
             const name = element.getAttribute('name') ?? 'unnamed';
-            // Check for connection pooling attributes
-            const hasPooling = element.hasAttribute('maxConnections') ||
-                element.hasAttribute('connectionIdleTimeout') ||
-                this.select('.//*[local-name()="pooling-profile"]', config).length > 0;
+            // Check for connection pooling attributes on the request-config element itself
+            const hasPoolingOnConfig = element.hasAttribute('maxConnections') || element.hasAttribute('connectionIdleTimeout');
+            // Also check the nested http:request-connection child element (XSD-correct placement).
+            // Per the Mule HTTP connector XSD, maxConnections and connectionIdleTimeout belong on
+            // http:request-connection, not http:request-config. Both placements must be accepted.
+            const connectionChild = this.selectFirst('.//*[local-name()="request-connection"]', config);
+            const hasPoolingOnConnection = connectionChild !== null &&
+                (connectionChild.hasAttribute('maxConnections') ||
+                    connectionChild.hasAttribute('connectionIdleTimeout'));
+            // Also accept a pooling-profile child (db-style config)
+            const hasPoolingProfile = this.select('.//*[local-name()="pooling-profile"]', config).length > 0;
+            const hasPooling = hasPoolingOnConfig || hasPoolingOnConnection || hasPoolingProfile;
             if (!hasPooling) {
                 issues.push(this.createIssue(config, `HTTP request config "${name}" has no connection pooling configured`, {
-                    suggestion: 'Add maxConnections and connectionIdleTimeout for optimal connection management',
+                    suggestion: 'Add maxConnections and connectionIdleTimeout on <http:request-connection> for optimal connection management',
                 }));
             }
         }

package/dist/src/rules/performance/ConnectionPoolingRule.js.map

@@ -1 +1 @@
-{"version":3,"file":"ConnectionPoolingRule.js","sourceRoot":"","sources":["../../../../src/rules/performance/ConnectionPoolingRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;GAKG;AACH,MAAa,qBAAsB,SAAQ,mBAAQ;IACjD,EAAE,GAAG,UAAU,CAAC;IAChB,IAAI,GAAG,oBAAoB,CAAC;IAC5B,WAAW,GAAG,sDAAsD,CAAC;IACrE,QAAQ,GAAG,SAAkB,CAAC;IAC9B,QAAQ,GAAG,aAAsB,CAAC;IAElC,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,oCAAoC;QACpC,MAAM,WAAW,GAAG,IAAI,CAAC,MAAM,CAC7B,mGAAmG,EACnG,GAAG,CACJ,CAAC;QAEF,KAAK,MAAM,MAAM,IAAI,WAAW,EAAE,CAAC;YACjC,MAAM,OAAO,GAAG,MAAiB,CAAC;YAClC,MAAM,IAAI,GAAG,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,SAAS,CAAC;YAEvD,0CAA0C;YAC1C,MAAM,UAAU,GACd,OAAO,CAAC,YAAY,CAAC,gBAAgB,CAAC;gBACtC,OAAO,CAAC,YAAY,CAAC,uBAAuB,CAAC;gBAC7C,IAAI,CAAC,MAAM,CAAC,sCAAsC,EAAE,MAAM,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC;YAEzE,IAAI,CAAC,UAAU,EAAE,CAAC;gBAChB,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,MAAM,EACN,wBAAwB,IAAI,wCAAwC,EACpE;oBACE,UAAU,EACR,gFAAgF;iBACnF,CACF,CACF,CAAC;YACJ,CAAC;QACH,CAAC;QAED,gCAAgC;QAChC,MAAM,SAAS,GAAG,IAAI,CAAC,MAAM,CAC3B,yFAAyF,EACzF,GAAG,CACJ,CAAC;QAEF,KAAK,MAAM,MAAM,IAAI,SAAS,EAAE,CAAC;YAC/B,MAAM,OAAO,GAAG,MAAiB,CAAC;YAClC,MAAM,IAAI,GAAG,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,SAAS,CAAC;YAEvD,4BAA4B;YAC5B,MAAM,UAAU,GAAG,IAAI,CAAC,MAAM,CAAC,sCAAsC,EAAE,MAAM,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC;YAE1F,IAAI,CAAC,UAAU,EAAE,CAAC;gBAChB,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,MAAM,EACN,oBAAoB,IAAI,wCAAwC,EAChE;oBACE,UAAU,EACR,iFAAiF;iBACpF,CACF,CACF,CAAC;YACJ,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;CACF;AArED,sDAqEC"}
+{"version":3,"file":"ConnectionPoolingRule.js","sourceRoot":"","sources":["../../../../src/rules/performance/ConnectionPoolingRule.ts"],"names":[],"mappings":";;;AACA,+CAA4C;AAE5C;;;;;;;;;;GAUG;AACH,MAAa,qBAAsB,SAAQ,mBAAQ;IACjD,EAAE,GAAG,UAAU,CAAC;IAChB,IAAI,GAAG,oBAAoB,CAAC;IAC5B,WAAW,GAAG,sDAAsD,CAAC;IACrE,QAAQ,GAAG,SAAkB,CAAC;IAC9B,QAAQ,GAAG,aAAsB,CAAC;IAElC,QAAQ,CAAC,GAAa,EAAE,QAA2B;QACjD,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,oCAAoC;QACpC,MAAM,WAAW,GAAG,IAAI,CAAC,MAAM,CAC7B,mGAAmG,EACnG,GAAG,CACJ,CAAC;QAEF,KAAK,MAAM,MAAM,IAAI,WAAW,EAAE,CAAC;YACjC,MAAM,OAAO,GAAG,MAAiB,CAAC;YAClC,MAAM,IAAI,GAAG,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,SAAS,CAAC;YAEvD,+EAA+E;YAC/E,MAAM,kBAAkB,GACtB,OAAO,CAAC,YAAY,CAAC,gBAAgB,CAAC,IAAI,OAAO,CAAC,YAAY,CAAC,uBAAuB,CAAC,CAAC;YAE1F,uFAAuF;YACvF,sFAAsF;YACtF,sFAAsF;YACtF,MAAM,eAAe,GAAG,IAAI,CAAC,WAAW,CAAC,yCAAyC,EAAE,MAAM,CAAC,CAAC;YAC5F,MAAM,sBAAsB,GAC1B,eAAe,KAAK,IAAI;gBACxB,CAAE,eAA2B,CAAC,YAAY,CAAC,gBAAgB,CAAC;oBACzD,eAA2B,CAAC,YAAY,CAAC,uBAAuB,CAAC,CAAC,CAAC;YAExE,wDAAwD;YACxD,MAAM,iBAAiB,GACrB,IAAI,CAAC,MAAM,CAAC,sCAAsC,EAAE,MAAM,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC;YAEzE,MAAM,UAAU,GAAG,kBAAkB,IAAI,sBAAsB,IAAI,iBAAiB,CAAC;YAErF,IAAI,CAAC,UAAU,EAAE,CAAC;gBAChB,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,MAAM,EACN,wBAAwB,IAAI,wCAAwC,EACpE;oBACE,UAAU,EACR,6GAA6G;iBAChH,CACF,CACF,CAAC;YACJ,CAAC;QACH,CAAC;QAED,gCAAgC;QAChC,MAAM,SAAS,GAAG,IAAI,CAAC,MAAM,CAC3B,yFAAyF,EACzF,GAAG,CACJ,CAAC;QAEF,KAAK,MAAM,MAAM,IAAI,SAAS,EAAE,CAAC;YAC/B,MAAM,OAAO,GAAG,MAAiB,CAAC;YAClC,MAAM,IAAI,GAAG,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,SAAS,CAAC;YAEvD,4BAA4B;YAC5B,MAAM,UAAU,GAAG,IAAI,CAAC,MAAM,CAAC,sCAAsC,EAAE,MAAM,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC;YAE1F,IAAI,CAAC,UAAU,EAAE,CAAC;gBAChB,MAAM,CAAC,IAAI,CACT,IAAI,CAAC,WAAW,CACd,MAAM,EACN,oBAAoB,IAAI,wCAAwC,EAChE;oBACE,UAAU,EACR,iFAAiF;iBACpF,CACF,CACF,CAAC;YACJ,CAAC;QACH,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;CACF;AAlFD,sDAkFC"}