@harness-engineering/eslint-plugin 0.1.0 → 0.1.2

package/README.md

@@ -81,6 +81,14 @@ Create `harness.config.json` in your project root:
 | --------------------- | ------------------------ | ------- |
 | `enforce-doc-exports` | Require JSDoc on exports | warn    |
 
+### Performance Rules
+
+| Rule                          | Description                                      | Default |
+| ----------------------------- | ------------------------------------------------ | ------- |
+| `no-nested-loops-in-critical` | Disallow nested loops in critical path functions | warn    |
+| `no-sync-io-in-async`         | Disallow synchronous I/O in async functions      | warn    |
+| `no-unbounded-array-chains`   | Disallow unbounded array method chains           | warn    |
+
 ## Configs
 
 - **recommended**: Architecture rules as errors, others as warnings

package/dist/index.d.ts

@@ -20,6 +20,15 @@ declare const plugin: {
         'no-layer-violation': import("@typescript-eslint/utils/ts-eslint").RuleModule<"layerViolation", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
             name: string;
         };
+        'no-nested-loops-in-critical': import("@typescript-eslint/utils/ts-eslint").RuleModule<"nestedLoopInCritical", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+            name: string;
+        };
+        'no-sync-io-in-async': import("@typescript-eslint/utils/ts-eslint").RuleModule<"syncIoInAsync", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+            name: string;
+        };
+        'no-unbounded-array-chains': import("@typescript-eslint/utils/ts-eslint").RuleModule<"unboundedArrayChain", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+            name: string;
+        };
         'require-boundary-schema': import("@typescript-eslint/utils/ts-eslint").RuleModule<"missingSchema", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
             name: string;
         };
@@ -77,6 +86,15 @@ export declare const configs: {
                     'no-layer-violation': import("@typescript-eslint/utils/ts-eslint").RuleModule<"layerViolation", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
+                    'no-nested-loops-in-critical': import("@typescript-eslint/utils/ts-eslint").RuleModule<"nestedLoopInCritical", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                        name: string;
+                    };
+                    'no-sync-io-in-async': import("@typescript-eslint/utils/ts-eslint").RuleModule<"syncIoInAsync", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                        name: string;
+                    };
+                    'no-unbounded-array-chains': import("@typescript-eslint/utils/ts-eslint").RuleModule<"unboundedArrayChain", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                        name: string;
+                    };
                     'require-boundary-schema': import("@typescript-eslint/utils/ts-eslint").RuleModule<"missingSchema", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
@@ -115,6 +133,15 @@ export declare const configs: {
                     'no-layer-violation': import("@typescript-eslint/utils/ts-eslint").RuleModule<"layerViolation", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
+                    'no-nested-loops-in-critical': import("@typescript-eslint/utils/ts-eslint").RuleModule<"nestedLoopInCritical", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                        name: string;
+                    };
+                    'no-sync-io-in-async': import("@typescript-eslint/utils/ts-eslint").RuleModule<"syncIoInAsync", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                        name: string;
+                    };
+                    'no-unbounded-array-chains': import("@typescript-eslint/utils/ts-eslint").RuleModule<"unboundedArrayChain", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                        name: string;
+                    };
                     'require-boundary-schema': import("@typescript-eslint/utils/ts-eslint").RuleModule<"missingSchema", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,KAAK,EAAE,MAAM,SAAS,CAAC;AAGhC,QAAA,MAAM,MAAM;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAoCX,CAAC;AAGF,eAAe,MAAM,CAAC;AAGtB,OAAO,EAAE,KAAK,EAAE,CAAC;AACjB,eAAO,MAAM,OAAO;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAAiB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,KAAK,EAAE,MAAM,SAAS,CAAC;AAGhC,QAAA,MAAM,MAAM;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAoCX,CAAC;AAGF,eAAe,MAAM,CAAC;AAGtB,OAAO,EAAE,KAAK,EAAE,CAAC;AACjB,eAAO,MAAM,OAAO;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAAiB,CAAC"}

package/dist/rules/index.d.ts

@@ -14,6 +14,15 @@ export declare const rules: {
     'no-layer-violation': import("@typescript-eslint/utils/ts-eslint").RuleModule<"layerViolation", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
         name: string;
     };
+    'no-nested-loops-in-critical': import("@typescript-eslint/utils/ts-eslint").RuleModule<"nestedLoopInCritical", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+        name: string;
+    };
+    'no-sync-io-in-async': import("@typescript-eslint/utils/ts-eslint").RuleModule<"syncIoInAsync", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+        name: string;
+    };
+    'no-unbounded-array-chains': import("@typescript-eslint/utils/ts-eslint").RuleModule<"unboundedArrayChain", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+        name: string;
+    };
     'require-boundary-schema': import("@typescript-eslint/utils/ts-eslint").RuleModule<"missingSchema", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
         name: string;
     };

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/rules/index.ts"],"names":[],"mappings":"AAOA,eAAO,MAAM,KAAK;;;;;;;;;;;;;;;;;;;CAMjB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/rules/index.ts"],"names":[],"mappings":"AAUA,eAAO,MAAM,KAAK;;;;;;;;;;;;;;;;;;;;;;;;;;;;CASjB,CAAC"}

package/dist/rules/index.js

@@ -3,12 +3,18 @@ import enforceDocExports from './enforce-doc-exports';
 import noCircularDeps from './no-circular-deps';
 import noForbiddenImports from './no-forbidden-imports';
 import noLayerViolation from './no-layer-violation';
+import noNestedLoopsInCritical from './no-nested-loops-in-critical';
+import noSyncIoInAsync from './no-sync-io-in-async';
+import noUnboundedArrayChains from './no-unbounded-array-chains';
 import requireBoundarySchema from './require-boundary-schema';
 export const rules = {
     'enforce-doc-exports': enforceDocExports,
     'no-circular-deps': noCircularDeps,
     'no-forbidden-imports': noForbiddenImports,
     'no-layer-violation': noLayerViolation,
+    'no-nested-loops-in-critical': noNestedLoopsInCritical,
+    'no-sync-io-in-async': noSyncIoInAsync,
+    'no-unbounded-array-chains': noUnboundedArrayChains,
     'require-boundary-schema': requireBoundarySchema,
 };
 //# sourceMappingURL=index.js.map

package/dist/rules/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/rules/index.ts"],"names":[],"mappings":"AAAA,qBAAqB;AACrB,OAAO,iBAAiB,MAAM,uBAAuB,CAAC;AACtD,OAAO,cAAc,MAAM,oBAAoB,CAAC;AAChD,OAAO,kBAAkB,MAAM,wBAAwB,CAAC;AACxD,OAAO,gBAAgB,MAAM,sBAAsB,CAAC;AACpD,OAAO,qBAAqB,MAAM,2BAA2B,CAAC;AAE9D,MAAM,CAAC,MAAM,KAAK,GAAG;IACnB,qBAAqB,EAAE,iBAAiB;IACxC,kBAAkB,EAAE,cAAc;IAClC,sBAAsB,EAAE,kBAAkB;IAC1C,oBAAoB,EAAE,gBAAgB;IACtC,yBAAyB,EAAE,qBAAqB;CACjD,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/rules/index.ts"],"names":[],"mappings":"AAAA,qBAAqB;AACrB,OAAO,iBAAiB,MAAM,uBAAuB,CAAC;AACtD,OAAO,cAAc,MAAM,oBAAoB,CAAC;AAChD,OAAO,kBAAkB,MAAM,wBAAwB,CAAC;AACxD,OAAO,gBAAgB,MAAM,sBAAsB,CAAC;AACpD,OAAO,uBAAuB,MAAM,+BAA+B,CAAC;AACpE,OAAO,eAAe,MAAM,uBAAuB,CAAC;AACpD,OAAO,sBAAsB,MAAM,6BAA6B,CAAC;AACjE,OAAO,qBAAqB,MAAM,2BAA2B,CAAC;AAE9D,MAAM,CAAC,MAAM,KAAK,GAAG;IACnB,qBAAqB,EAAE,iBAAiB;IACxC,kBAAkB,EAAE,cAAc;IAClC,sBAAsB,EAAE,kBAAkB;IAC1C,oBAAoB,EAAE,gBAAgB;IACtC,6BAA6B,EAAE,uBAAuB;IACtD,qBAAqB,EAAE,eAAe;IACtC,2BAA2B,EAAE,sBAAsB;IACnD,yBAAyB,EAAE,qBAAqB;CACjD,CAAC"}

package/dist/rules/no-nested-loops-in-critical.d.ts

@@ -0,0 +1,6 @@
+import { ESLintUtils } from '@typescript-eslint/utils';
+declare const _default: ESLintUtils.RuleModule<"nestedLoopInCritical", [], unknown, ESLintUtils.RuleListener> & {
+    name: string;
+};
+export default _default;
+//# sourceMappingURL=no-nested-loops-in-critical.d.ts.map

package/dist/rules/no-nested-loops-in-critical.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-nested-loops-in-critical.d.ts","sourceRoot":"","sources":["../../src/rules/no-nested-loops-in-critical.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,WAAW,EAAiB,MAAM,0BAA0B,CAAC;;;;AAQtE,wBAoGG"}

package/dist/rules/no-nested-loops-in-critical.js

@@ -0,0 +1,87 @@
+// src/rules/no-nested-loops-in-critical.ts
+import { ESLintUtils } from '@typescript-eslint/utils';
+const createRule = ESLintUtils.RuleCreator((name) => `https://github.com/harness-engineering/eslint-plugin/blob/main/docs/rules/${name}.md`);
+export default createRule({
+    name: 'no-nested-loops-in-critical',
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Disallow nested loops in @perf-critical functions',
+        },
+        messages: {
+            nestedLoopInCritical: 'Nested loop in @perf-critical code — consider alternative algorithm',
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        const sourceText = context.sourceCode.getText();
+        // Quick check: if the file has no @perf-critical at all, skip entirely
+        if (!sourceText.includes('@perf-critical')) {
+            return {};
+        }
+        // Stack tracks whether each nested function scope is @perf-critical
+        const criticalStack = [];
+        let loopDepth = 0;
+        function isCritical() {
+            return criticalStack.length > 0 && criticalStack[criticalStack.length - 1] === true;
+        }
+        function hasCriticalAnnotation(node) {
+            // Check the source lines immediately before this function for @perf-critical.
+            // For JSDoc: `/** @perf-critical */` is typically 1 line before.
+            // For line comment: `// @perf-critical` is 1 line before.
+            const target = node.parent?.type === 'ExportNamedDeclaration' ||
+                node.parent?.type === 'VariableDeclaration'
+                ? node.parent
+                : node;
+            const startLine = target.loc.start.line; // 1-indexed
+            const lines = sourceText.split('\n');
+            // Only check the line immediately before the function (or same line for inline)
+            for (let i = Math.max(0, startLine - 2); i < startLine; i++) {
+                if (lines[i]?.includes('@perf-critical'))
+                    return true;
+            }
+            return false;
+        }
+        function enterFunction(node) {
+            criticalStack.push(hasCriticalAnnotation(node));
+            loopDepth = 0;
+        }
+        function exitFunction() {
+            criticalStack.pop();
+            loopDepth = 0;
+        }
+        function enterLoop(node) {
+            if (!isCritical())
+                return;
+            loopDepth++;
+            if (loopDepth > 1) {
+                context.report({ node, messageId: 'nestedLoopInCritical' });
+            }
+        }
+        function exitLoop() {
+            if (!isCritical())
+                return;
+            loopDepth--;
+        }
+        return {
+            FunctionDeclaration: enterFunction,
+            'FunctionDeclaration:exit': exitFunction,
+            FunctionExpression: enterFunction,
+            'FunctionExpression:exit': exitFunction,
+            ArrowFunctionExpression: enterFunction,
+            'ArrowFunctionExpression:exit': exitFunction,
+            ForStatement: enterLoop,
+            'ForStatement:exit': exitLoop,
+            ForInStatement: enterLoop,
+            'ForInStatement:exit': exitLoop,
+            ForOfStatement: enterLoop,
+            'ForOfStatement:exit': exitLoop,
+            WhileStatement: enterLoop,
+            'WhileStatement:exit': exitLoop,
+            DoWhileStatement: enterLoop,
+            'DoWhileStatement:exit': exitLoop,
+        };
+    },
+});
+//# sourceMappingURL=no-nested-loops-in-critical.js.map

package/dist/rules/no-nested-loops-in-critical.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-nested-loops-in-critical.js","sourceRoot":"","sources":["../../src/rules/no-nested-loops-in-critical.ts"],"names":[],"mappings":"AAAA,2CAA2C;AAC3C,OAAO,EAAE,WAAW,EAAiB,MAAM,0BAA0B,CAAC;AAEtE,MAAM,UAAU,GAAG,WAAW,CAAC,WAAW,CACxC,CAAC,IAAI,EAAE,EAAE,CAAC,6EAA6E,IAAI,KAAK,CACjG,CAAC;AAIF,eAAe,UAAU,CAAiB;IACxC,IAAI,EAAE,6BAA6B;IACnC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,IAAI,EAAE;YACJ,WAAW,EAAE,mDAAmD;SACjE;QACD,QAAQ,EAAE;YACR,oBAAoB,EAAE,qEAAqE;SAC5F;QACD,MAAM,EAAE,EAAE;KACX;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACZ,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,CAAC,OAAO,EAAE,CAAC;QAEhD,uEAAuE;QACvE,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,gBAAgB,CAAC,EAAE,CAAC;YAC3C,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,oEAAoE;QACpE,MAAM,aAAa,GAAc,EAAE,CAAC;QACpC,IAAI,SAAS,GAAG,CAAC,CAAC;QAElB,SAAS,UAAU;YACjB,OAAO,aAAa,CAAC,MAAM,GAAG,CAAC,IAAI,aAAa,CAAC,aAAa,CAAC,MAAM,GAAG,CAAC,CAAC,KAAK,IAAI,CAAC;QACtF,CAAC;QAED,SAAS,qBAAqB,CAC5B,IAGoC;YAEpC,8EAA8E;YAC9E,iEAAiE;YACjE,0DAA0D;YAC1D,MAAM,MAAM,GACV,IAAI,CAAC,MAAM,EAAE,IAAI,KAAK,wBAAwB;gBAC9C,IAAI,CAAC,MAAM,EAAE,IAAI,KAAK,qBAAqB;gBACzC,CAAC,CAAC,IAAI,CAAC,MAAM;gBACb,CAAC,CAAC,IAAI,CAAC;YACX,MAAM,SAAS,GAAG,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,YAAY;YACrD,MAAM,KAAK,GAAG,UAAU,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;YACrC,gFAAgF;YAChF,KAAK,IAAI,CAAC,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,SAAS,GAAG,CAAC,CAAC,EAAE,CAAC,GAAG,SAAS,EAAE,CAAC,EAAE,EAAE,CAAC;gBAC5D,IAAI,KAAK,CAAC,CAAC,CAAC,EAAE,QAAQ,CAAC,gBAAgB,CAAC;oBAAE,OAAO,IAAI,CAAC;YACxD,CAAC;YACD,OAAO,KAAK,CAAC;QACf,CAAC;QAED,SAAS,aAAa,CACpB,IAGoC;YAEpC,aAAa,CAAC,IAAI,CAAC,qBAAqB,CAAC,IAAI,CAAC,CAAC,CAAC;YAChD,SAAS,GAAG,CAAC,CAAC;QAChB,CAAC;QAED,SAAS,YAAY;YACnB,aAAa,CAAC,GAAG,EAAE,CAAC;YACpB,SAAS,GAAG,CAAC,CAAC;QAChB,CAAC;QAED,SAAS,SAAS,CAAC,IAAmB;YACpC,IAAI,CAAC,UAAU,EAAE;gBAAE,OAAO;YAC1B,SAAS,EAAE,CAAC;YACZ,IAAI,SAAS,GAAG,CAAC,EAAE,CAAC;gBAClB,OAAO,CAAC,MAAM,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,sBAAsB,EAAE,CAAC,CAAC;YAC9D,CAAC;QACH,CAAC;QAED,SAAS,QAAQ;YACf,IAAI,CAAC,UAAU,EAAE;gBAAE,OAAO;YAC1B,SAAS,EAAE,CAAC;QACd,CAAC;QAED,OAAO;YACL,mBAAmB,EAAE,aAAa;YAClC,0BAA0B,EAAE,YAAY;YACxC,kBAAkB,EAAE,aAAa;YACjC,yBAAyB,EAAE,YAAY;YACvC,uBAAuB,EAAE,aAAa;YACtC,8BAA8B,EAAE,YAAY;YAE5C,YAAY,EAAE,SAAS;YACvB,mBAAmB,EAAE,QAAQ;YAC7B,cAAc,EAAE,SAAS;YACzB,qBAAqB,EAAE,QAAQ;YAC/B,cAAc,EAAE,SAAS;YACzB,qBAAqB,EAAE,QAAQ;YAC/B,cAAc,EAAE,SAAS;YACzB,qBAAqB,EAAE,QAAQ;YAC/B,gBAAgB,EAAE,SAAS;YAC3B,uBAAuB,EAAE,QAAQ;SAClC,CAAC;IACJ,CAAC;CACF,CAAC,CAAC"}

package/dist/rules/no-sync-io-in-async.d.ts

@@ -0,0 +1,6 @@
+import { ESLintUtils } from '@typescript-eslint/utils';
+declare const _default: ESLintUtils.RuleModule<"syncIoInAsync", [], unknown, ESLintUtils.RuleListener> & {
+    name: string;
+};
+export default _default;
+//# sourceMappingURL=no-sync-io-in-async.d.ts.map

package/dist/rules/no-sync-io-in-async.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-sync-io-in-async.d.ts","sourceRoot":"","sources":["../../src/rules/no-sync-io-in-async.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,WAAW,EAAiB,MAAM,0BAA0B,CAAC;;;;AAqBtE,wBA2EG"}

package/dist/rules/no-sync-io-in-async.js

@@ -0,0 +1,73 @@
+// src/rules/no-sync-io-in-async.ts
+import { ESLintUtils } from '@typescript-eslint/utils';
+const createRule = ESLintUtils.RuleCreator((name) => `https://github.com/harness-engineering/eslint-plugin/blob/main/docs/rules/${name}.md`);
+const SYNC_FS_METHODS = new Set([
+    'readFileSync',
+    'writeFileSync',
+    'existsSync',
+    'readdirSync',
+    'statSync',
+    'mkdirSync',
+    'unlinkSync',
+    'copyFileSync',
+    'renameSync',
+    'accessSync',
+]);
+export default createRule({
+    name: 'no-sync-io-in-async',
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Disallow synchronous fs operations inside async functions',
+        },
+        messages: {
+            syncIoInAsync: "Use async fs methods instead of '{{name}}' in async functions",
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        let asyncDepth = 0;
+        function enterFunction(node) {
+            if (node.async) {
+                asyncDepth++;
+            }
+        }
+        function exitFunction(node) {
+            if (node.async) {
+                asyncDepth--;
+            }
+        }
+        return {
+            FunctionDeclaration: enterFunction,
+            'FunctionDeclaration:exit': exitFunction,
+            FunctionExpression: enterFunction,
+            'FunctionExpression:exit': exitFunction,
+            ArrowFunctionExpression: enterFunction,
+            'ArrowFunctionExpression:exit': exitFunction,
+            CallExpression(node) {
+                if (asyncDepth === 0)
+                    return;
+                let name;
+                // Handle: readFileSync(...)
+                if (node.callee.type === 'Identifier' && SYNC_FS_METHODS.has(node.callee.name)) {
+                    name = node.callee.name;
+                }
+                // Handle: fs.readFileSync(...)
+                if (node.callee.type === 'MemberExpression' &&
+                    node.callee.property.type === 'Identifier' &&
+                    SYNC_FS_METHODS.has(node.callee.property.name)) {
+                    name = node.callee.property.name;
+                }
+                if (name) {
+                    context.report({
+                        node,
+                        messageId: 'syncIoInAsync',
+                        data: { name },
+                    });
+                }
+            },
+        };
+    },
+});
+//# sourceMappingURL=no-sync-io-in-async.js.map

package/dist/rules/no-sync-io-in-async.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-sync-io-in-async.js","sourceRoot":"","sources":["../../src/rules/no-sync-io-in-async.ts"],"names":[],"mappings":"AAAA,mCAAmC;AACnC,OAAO,EAAE,WAAW,EAAiB,MAAM,0BAA0B,CAAC;AAEtE,MAAM,UAAU,GAAG,WAAW,CAAC,WAAW,CACxC,CAAC,IAAI,EAAE,EAAE,CAAC,6EAA6E,IAAI,KAAK,CACjG,CAAC;AAIF,MAAM,eAAe,GAAG,IAAI,GAAG,CAAC;IAC9B,cAAc;IACd,eAAe;IACf,YAAY;IACZ,aAAa;IACb,UAAU;IACV,WAAW;IACX,YAAY;IACZ,cAAc;IACd,YAAY;IACZ,YAAY;CACb,CAAC,CAAC;AAEH,eAAe,UAAU,CAAiB;IACxC,IAAI,EAAE,qBAAqB;IAC3B,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EAAE,2DAA2D;SACzE;QACD,QAAQ,EAAE;YACR,aAAa,EAAE,+DAA+D;SAC/E;QACD,MAAM,EAAE,EAAE;KACX;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACZ,IAAI,UAAU,GAAG,CAAC,CAAC;QAEnB,SAAS,aAAa,CACpB,IAGoC;YAEpC,IAAI,IAAI,CAAC,KAAK,EAAE,CAAC;gBACf,UAAU,EAAE,CAAC;YACf,CAAC;QACH,CAAC;QAED,SAAS,YAAY,CACnB,IAGoC;YAEpC,IAAI,IAAI,CAAC,KAAK,EAAE,CAAC;gBACf,UAAU,EAAE,CAAC;YACf,CAAC;QACH,CAAC;QAED,OAAO;YACL,mBAAmB,EAAE,aAAa;YAClC,0BAA0B,EAAE,YAAY;YACxC,kBAAkB,EAAE,aAAa;YACjC,yBAAyB,EAAE,YAAY;YACvC,uBAAuB,EAAE,aAAa;YACtC,8BAA8B,EAAE,YAAY;YAE5C,cAAc,CAAC,IAA6B;gBAC1C,IAAI,UAAU,KAAK,CAAC;oBAAE,OAAO;gBAE7B,IAAI,IAAwB,CAAC;gBAE7B,4BAA4B;gBAC5B,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,YAAY,IAAI,eAAe,CAAC,GAAG,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC/E,IAAI,GAAG,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC;gBAC1B,CAAC;gBAED,+BAA+B;gBAC/B,IACE,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,kBAAkB;oBACvC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,YAAY;oBAC1C,eAAe,CAAC,GAAG,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC,EAC9C,CAAC;oBACD,IAAI,GAAG,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC;gBACnC,CAAC;gBAED,IAAI,IAAI,EAAE,CAAC;oBACT,OAAO,CAAC,MAAM,CAAC;wBACb,IAAI;wBACJ,SAAS,EAAE,eAAe;wBAC1B,IAAI,EAAE,EAAE,IAAI,EAAE;qBACf,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC"}

package/dist/rules/no-unbounded-array-chains.d.ts

@@ -0,0 +1,6 @@
+import { ESLintUtils } from '@typescript-eslint/utils';
+declare const _default: ESLintUtils.RuleModule<"unboundedArrayChain", [], unknown, ESLintUtils.RuleListener> & {
+    name: string;
+};
+export default _default;
+//# sourceMappingURL=no-unbounded-array-chains.d.ts.map

package/dist/rules/no-unbounded-array-chains.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-unbounded-array-chains.d.ts","sourceRoot":"","sources":["../../src/rules/no-unbounded-array-chains.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,WAAW,EAAiB,MAAM,0BAA0B,CAAC;;;;AAoBtE,wBA+DG"}

package/dist/rules/no-unbounded-array-chains.js

@@ -0,0 +1,71 @@
+// src/rules/no-unbounded-array-chains.ts
+import { ESLintUtils } from '@typescript-eslint/utils';
+const createRule = ESLintUtils.RuleCreator((name) => `https://github.com/harness-engineering/eslint-plugin/blob/main/docs/rules/${name}.md`);
+const ARRAY_METHODS = new Set([
+    'filter',
+    'map',
+    'reduce',
+    'sort',
+    'flatMap',
+    'find',
+    'some',
+    'every',
+    'forEach',
+]);
+export default createRule({
+    name: 'no-unbounded-array-chains',
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Disallow 3+ chained array operations',
+        },
+        messages: {
+            unboundedArrayChain: '3+ chained array operations — consider a single-pass approach',
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        return {
+            'CallExpression > MemberExpression.callee'(node) {
+                // Only care about array method calls
+                if (node.property.type !== 'Identifier' || !ARRAY_METHODS.has(node.property.name)) {
+                    return;
+                }
+                // Skip if this call is itself the object of another array method call
+                // (i.e., not the outermost in the chain). Only report on the outermost.
+                const callExpr = node.parent;
+                if (callExpr.parent &&
+                    callExpr.parent.type === 'MemberExpression' &&
+                    callExpr.parent.property.type === 'Identifier' &&
+                    ARRAY_METHODS.has(callExpr.parent.property.name)) {
+                    return;
+                }
+                // Count the chain length by walking down through the object
+                let chainLength = 1;
+                let current = node;
+                while (true) {
+                    const memberExpr = current;
+                    const obj = memberExpr.object;
+                    if (obj.type === 'CallExpression' &&
+                        obj.callee.type === 'MemberExpression' &&
+                        obj.callee.property.type === 'Identifier' &&
+                        ARRAY_METHODS.has(obj.callee.property.name)) {
+                        chainLength++;
+                        current = obj.callee;
+                    }
+                    else {
+                        break;
+                    }
+                }
+                if (chainLength >= 3) {
+                    context.report({
+                        node: callExpr,
+                        messageId: 'unboundedArrayChain',
+                    });
+                }
+            },
+        };
+    },
+});
+//# sourceMappingURL=no-unbounded-array-chains.js.map

package/dist/rules/no-unbounded-array-chains.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-unbounded-array-chains.js","sourceRoot":"","sources":["../../src/rules/no-unbounded-array-chains.ts"],"names":[],"mappings":"AAAA,yCAAyC;AACzC,OAAO,EAAE,WAAW,EAAiB,MAAM,0BAA0B,CAAC;AAEtE,MAAM,UAAU,GAAG,WAAW,CAAC,WAAW,CACxC,CAAC,IAAI,EAAE,EAAE,CAAC,6EAA6E,IAAI,KAAK,CACjG,CAAC;AAIF,MAAM,aAAa,GAAG,IAAI,GAAG,CAAC;IAC5B,QAAQ;IACR,KAAK;IACL,QAAQ;IACR,MAAM;IACN,SAAS;IACT,MAAM;IACN,MAAM;IACN,OAAO;IACP,SAAS;CACV,CAAC,CAAC;AAEH,eAAe,UAAU,CAAiB;IACxC,IAAI,EAAE,2BAA2B;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,IAAI,EAAE;YACJ,WAAW,EAAE,sCAAsC;SACpD;QACD,QAAQ,EAAE;YACR,mBAAmB,EAAE,+DAA+D;SACrF;QACD,MAAM,EAAE,EAAE;KACX;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACZ,OAAO;YACL,0CAA0C,CAAC,IAA+B;gBACxE,qCAAqC;gBACrC,IAAI,IAAI,CAAC,QAAQ,CAAC,IAAI,KAAK,YAAY,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;oBAClF,OAAO;gBACT,CAAC;gBAED,sEAAsE;gBACtE,wEAAwE;gBACxE,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAO,CAAC;gBAC9B,IACE,QAAQ,CAAC,MAAM;oBACf,QAAQ,CAAC,MAAM,CAAC,IAAI,KAAK,kBAAkB;oBAC3C,QAAQ,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,YAAY;oBAC9C,aAAa,CAAC,GAAG,CAAC,QAAQ,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC,EAChD,CAAC;oBACD,OAAO;gBACT,CAAC;gBAED,4DAA4D;gBAC5D,IAAI,WAAW,GAAG,CAAC,CAAC;gBACpB,IAAI,OAAO,GAAkB,IAAI,CAAC;gBAElC,OAAO,IAAI,EAAE,CAAC;oBACZ,MAAM,UAAU,GAAG,OAAoC,CAAC;oBACxD,MAAM,GAAG,GAAG,UAAU,CAAC,MAAM,CAAC;oBAE9B,IACE,GAAG,CAAC,IAAI,KAAK,gBAAgB;wBAC7B,GAAG,CAAC,MAAM,CAAC,IAAI,KAAK,kBAAkB;wBACtC,GAAG,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,YAAY;wBACzC,aAAa,CAAC,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC,EAC3C,CAAC;wBACD,WAAW,EAAE,CAAC;wBACd,OAAO,GAAG,GAAG,CAAC,MAAM,CAAC;oBACvB,CAAC;yBAAM,CAAC;wBACN,MAAM;oBACR,CAAC;gBACH,CAAC;gBAED,IAAI,WAAW,IAAI,CAAC,EAAE,CAAC;oBACrB,OAAO,CAAC,MAAM,CAAC;wBACb,IAAI,EAAE,QAAQ;wBACd,SAAS,EAAE,qBAAqB;qBACjC,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@harness-engineering/eslint-plugin",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "ESLint plugin for harness engineering architectural constraints",
   "type": "module",
   "main": "./dist/index.js",
@@ -17,16 +17,16 @@
   ],
   "dependencies": {
     "@typescript-eslint/utils": "^8.0.0",
-    "minimatch": "^9.0.0",
+    "minimatch": "^10.2.4",
     "zod": "^3.22.0"
   },
   "devDependencies": {
-    "@types/node": "^20.0.0",
+    "@types/node": "^22.0.0",
     "@typescript-eslint/parser": "^8.57.0",
     "@typescript-eslint/rule-tester": "^8.0.0",
     "eslint": "^10.0.0",
-    "typescript": "^5.0.0",
-    "vitest": "^2.0.0"
+    "typescript": "^5.3.3",
+    "vitest": "^4.0.18"
   },
   "peerDependencies": {
     "eslint": "^8.0.0 || ^9.0.0 || ^10.0.0",
@@ -57,6 +57,7 @@
     "test": "vitest run",
     "test:watch": "vitest",
     "lint": "eslint src",
+    "typecheck": "tsc --noEmit",
     "clean": "rm -rf dist"
   }
 }