eslint-plugin-maintainability 3.0.13 → 3.0.15

package/src/rules/maintainability/nested-complexity-hotspots.js

@@ -1,137 +1 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.nestedComplexityHotspots = void 0;
-const eslint_devkit_1 = require("@interlace/eslint-devkit");
-const eslint_devkit_2 = require("@interlace/eslint-devkit");
-exports.nestedComplexityHotspots = (0, eslint_devkit_2.createRule)({
-    name: 'nested-complexity-hotspots',
-    meta: {
-        type: 'suggestion',
-        docs: {
-            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-maintainability/docs/rules/nested-complexity-hotspots.md',
-            description: 'Identifies nested control structures that harm readability',
-        },
-        messages: {
-            nestedComplexity: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.COMPLEXITY,
-                issueName: 'Nested complexity hotspot',
-                description: 'Nesting depth {{depth}} exceeds maximum {{max}}',
-                severity: 'MEDIUM',
-                fix: 'Use early returns, guard clauses, or extract methods',
-                documentationLink: 'https://en.wikipedia.org/wiki/Cyclomatic_complexity',
-            }),
-            useEarlyReturn: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.INFO,
-                issueName: 'Use Early Return',
-                description: 'Use early return to reduce nesting',
-                severity: 'LOW',
-                fix: 'if (!condition) return; // Continue with main logic',
-                documentationLink: 'https://refactoring.guru/smells/long-method',
-            }),
-            useGuardClauses: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.INFO,
-                issueName: 'Use Guard Clauses',
-                description: 'Use guard clauses for validation',
-                severity: 'LOW',
-                fix: 'if (!isValid(input)) throw new Error()',
-                documentationLink: 'https://refactoring.guru/replace-nested-conditional-with-guard-clauses',
-            }),
-            extractMethod: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.INFO,
-                issueName: 'Extract Method',
-                description: 'Extract nested logic to separate method',
-                severity: 'LOW',
-                fix: 'Extract to private method',
-                documentationLink: 'https://refactoring.guru/extract-method',
-            }),
-        },
-        schema: [
-            {
-                type: 'object',
-                properties: {
-                    maxDepth: {
-                        type: 'number',
-                        default: 4,
-                        minimum: 1,
-                        description: 'Maximum nesting depth',
-                    },
-                    countConditionals: {
-                        type: 'boolean',
-                        default: true,
-                        description: 'Count nested conditionals',
-                    },
-                    countLoops: {
-                        type: 'boolean',
-                        default: true,
-                        description: 'Count nested loops',
-                    },
-                },
-                additionalProperties: false,
-            },
-        ],
-    },
-    defaultOptions: [
-        {
-            maxDepth: 4,
-            countConditionals: true,
-            countLoops: true,
-        },
-    ],
-    create(context, [options = {}]) {
-        const { maxDepth = 4, countConditionals = true, countLoops = true, } = options || {};
-        function checkControlStructure(node) {
-            let depth = 0;
-            let current = node;
-            const maxDepthCheck = 20;
-            while (current && depth < maxDepthCheck) {
-                const parent = current
-                    .parent;
-                if (!parent)
-                    break;
-                if (parent.type === 'IfStatement' ||
-                    parent.type === 'ForStatement' ||
-                    parent.type === 'ForInStatement' ||
-                    parent.type === 'ForOfStatement' ||
-                    parent.type === 'WhileStatement' ||
-                    parent.type === 'DoWhileStatement' ||
-                    parent.type === 'SwitchStatement' ||
-                    parent.type === 'TryStatement') {
-                    depth++;
-                }
-                current = parent;
-            }
-            if (depth >= maxDepth) {
-                context.report({
-                    node,
-                    messageId: 'nestedComplexity',
-                    data: {
-                        depth: String(depth + 1),
-                        max: String(maxDepth),
-                    },
-                    suggest: [
-                        {
-                            messageId: 'useEarlyReturn',
-                            fix: () => null,
-                        },
-                        {
-                            messageId: 'useGuardClauses',
-                            fix: () => null,
-                        },
-                        {
-                            messageId: 'extractMethod',
-                            fix: () => null,
-                        },
-                    ],
-                });
-            }
-        }
-        return {
-            IfStatement: countConditionals ? checkControlStructure : undefined,
-            ForStatement: countLoops ? checkControlStructure : undefined,
-            ForInStatement: countLoops ? checkControlStructure : undefined,
-            ForOfStatement: countLoops ? checkControlStructure : undefined,
-            WhileStatement: countLoops ? checkControlStructure : undefined,
-            SwitchStatement: countConditionals ? checkControlStructure : undefined,
-        };
-    },
-});
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});exports.nestedComplexityHotspots=void 0;const eslint_devkit_1=require("@interlace/eslint-devkit");const eslint_devkit_2=require("@interlace/eslint-devkit");exports.nestedComplexityHotspots=(0,eslint_devkit_2.createRule)({name:"nested-complexity-hotspots",meta:{type:"suggestion",docs:{url:"https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-maintainability/docs/rules/nested-complexity-hotspots.md",description:"Identifies nested control structures that harm readability"},hasSuggestions:true,messages:{nestedComplexity:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.COMPLEXITY,issueName:"Nested complexity hotspot",description:"Nesting depth {{depth}} exceeds maximum {{max}}",severity:"MEDIUM",fix:"Use early returns, guard clauses, or extract methods",documentationLink:"https://en.wikipedia.org/wiki/Cyclomatic_complexity"}),useEarlyReturn:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.INFO,issueName:"Use Early Return",description:"Use early return to reduce nesting",severity:"LOW",fix:"if (!condition) return; // Continue with main logic",documentationLink:"https://refactoring.guru/smells/long-method"}),useGuardClauses:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.INFO,issueName:"Use Guard Clauses",description:"Use guard clauses for validation",severity:"LOW",fix:"if (!isValid(input)) throw new Error()",documentationLink:"https://refactoring.guru/replace-nested-conditional-with-guard-clauses"}),extractMethod:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.INFO,issueName:"Extract Method",description:"Extract nested logic to separate method",severity:"LOW",fix:"Extract to private method",documentationLink:"https://refactoring.guru/extract-method"})},schema:[{type:"object",properties:{maxDepth:{type:"number",default:4,minimum:1,description:"Maximum nesting depth"},countConditionals:{type:"boolean",default:true,description:"Count nested conditionals"},countLoops:{type:"boolean",default:true,description:"Count nested loops"}},additionalProperties:false}]},defaultOptions:[{maxDepth:4,countConditionals:true,countLoops:true}],create(context,[options={}]){const{maxDepth=4,countConditionals=true,countLoops=true}=options||{};function checkControlStructure(node){let depth=0;let current=node;const maxDepthCheck=20;while(current&&depth<maxDepthCheck){const parent=current.parent;if(!parent)break;if(parent.type==="IfStatement"||parent.type==="ForStatement"||parent.type==="ForInStatement"||parent.type==="ForOfStatement"||parent.type==="WhileStatement"||parent.type==="DoWhileStatement"||parent.type==="SwitchStatement"||parent.type==="TryStatement"){depth++}current=parent}if(depth>=maxDepth){context.report({node,messageId:"nestedComplexity",data:{depth:String(depth+1),max:String(maxDepth)},suggest:[{messageId:"useEarlyReturn",fix:()=>null},{messageId:"useGuardClauses",fix:()=>null},{messageId:"extractMethod",fix:()=>null}]})}}return{IfStatement:countConditionals?checkControlStructure:void 0,ForStatement:countLoops?checkControlStructure:void 0,ForInStatement:countLoops?checkControlStructure:void 0,ForOfStatement:countLoops?checkControlStructure:void 0,WhileStatement:countLoops?checkControlStructure:void 0,SwitchStatement:countConditionals?checkControlStructure:void 0}}});

package/src/rules/maintainability/no-lonely-if.js

@@ -1,107 +1 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.noLonelyIf = void 0;
-const eslint_devkit_1 = require("@interlace/eslint-devkit");
-const eslint_devkit_2 = require("@interlace/eslint-devkit");
-exports.noLonelyIf = (0, eslint_devkit_1.createRule)({
-    name: 'no-lonely-if',
-    meta: {
-        type: 'suggestion',
-        docs: {
-            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-maintainability/docs/rules/no-lonely-if.md',
-            description: 'Prevent lone if statements inside else blocks - use else if instead',
-        },
-        hasSuggestions: true,
-        messages: {
-            noLonelyIf: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.WARNING,
-                issueName: 'Lonely If',
-                description: 'Unexpected if statement inside else block',
-                severity: 'MEDIUM',
-                fix: 'Replace with else if for better readability',
-                documentationLink: 'https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-lonely-if.md',
-            }),
-        },
-        schema: [
-            {
-                type: 'object',
-                properties: {
-                    allow: {
-                        type: 'array',
-                        items: { type: 'string' },
-                        default: [],
-                    },
-                },
-                additionalProperties: false,
-            },
-        ],
-    },
-    defaultOptions: [{ allow: [] }],
-    create(context) {
-        const [options] = context.options;
-        const { allow = [] } = options || {};
-        const allowedContexts = new Set(allow);
-        function isInAllowedContext() {
-            for (const allowedContext of allowedContexts) {
-                const sourceCode = context.sourceCode;
-                const sourceText = sourceCode.getText();
-                if (sourceText.includes(allowedContext)) {
-                    return true;
-                }
-            }
-            return false;
-        }
-        function isLonelyIf(node) {
-            const parent = node.parent;
-            if (parent?.type === 'BlockStatement') {
-                const grandParent = parent.parent;
-                return (grandParent?.type === 'IfStatement' &&
-                    grandParent.alternate === parent);
-            }
-            return false;
-        }
-        return {
-            IfStatement(node) {
-                if (isLonelyIf(node) && !isInAllowedContext()) {
-                    context.report({
-                        node,
-                        messageId: 'noLonelyIf',
-                        data: {
-                            current: 'if statement in else block',
-                            fix: 'else if',
-                        },
-                        suggest: [
-                            {
-                                messageId: 'noLonelyIf',
-                                fix(fixer) {
-                                    const sourceCode = context.sourceCode;
-                                    let elseToken = null;
-                                    const tokens = sourceCode.getTokensBefore(node, 10);
-                                    for (let i = tokens.length - 1; i >= 0; i--) {
-                                        if (tokens[i].value === 'else') {
-                                            elseToken = tokens[i];
-                                            break;
-                                        }
-                                    }
-                                    if (elseToken) {
-                                        const ifToken = sourceCode.getTokenAfter(elseToken);
-                                        if (ifToken && ifToken.value === 'if') {
-                                            return [
-                                                fixer.removeRange([
-                                                    elseToken.range[0],
-                                                    ifToken.range[1],
-                                                ]),
-                                                fixer.insertTextBefore(ifToken, 'else '),
-                                            ];
-                                        }
-                                    }
-                                    return null;
-                                },
-                            },
-                        ],
-                    });
-                }
-            },
-        };
-    },
-});
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});exports.noLonelyIf=void 0;const eslint_devkit_1=require("@interlace/eslint-devkit");const eslint_devkit_2=require("@interlace/eslint-devkit");exports.noLonelyIf=(0,eslint_devkit_1.createRule)({name:"no-lonely-if",meta:{type:"suggestion",docs:{url:"https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-maintainability/docs/rules/no-lonely-if.md",description:"Prevent lone if statements inside else blocks - use else if instead"},hasSuggestions:true,messages:{noLonelyIf:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.WARNING,issueName:"Lonely If",description:"Unexpected if statement inside else block",severity:"MEDIUM",fix:"Replace with else if for better readability",documentationLink:"https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-lonely-if.md"})},schema:[{type:"object",properties:{allow:{type:"array",items:{type:"string"},default:[]}},additionalProperties:false}]},defaultOptions:[{allow:[]}],create(context){const[options]=context.options;const{allow=[]}=options||{};const allowedContexts=new Set(allow);function isInAllowedContext(){for(const allowedContext of allowedContexts){const sourceCode=context.sourceCode;const sourceText=sourceCode.getText();if(sourceText.includes(allowedContext)){return true}}return false}function isLonelyIf(node){const parent=node.parent;if(parent?.type==="BlockStatement"){const grandParent=parent.parent;return grandParent?.type==="IfStatement"&&grandParent.alternate===parent}return false}return{IfStatement(node){if(isLonelyIf(node)&&!isInAllowedContext()){context.report({node,messageId:"noLonelyIf",data:{current:"if statement in else block",fix:"else if"},suggest:[{messageId:"noLonelyIf",fix(fixer){const sourceCode=context.sourceCode;let elseToken=null;const tokens=sourceCode.getTokensBefore(node,10);for(let i=tokens.length-1;i>=0;i--){if(tokens[i].value==="else"){elseToken=tokens[i];break}}if(elseToken){const ifToken=sourceCode.getTokenAfter(elseToken);if(ifToken&&ifToken.value==="if"){return[fixer.removeRange([elseToken.range[0],ifToken.range[1]]),fixer.insertTextBefore(ifToken,"else ")]}}return null}}]})}}}}});

package/src/rules/maintainability/no-nested-ternary.js

@@ -1,139 +1 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.noNestedTernary = void 0;
-const eslint_devkit_1 = require("@interlace/eslint-devkit");
-const eslint_devkit_2 = require("@interlace/eslint-devkit");
-exports.noNestedTernary = (0, eslint_devkit_1.createRule)({
-    name: 'no-nested-ternary',
-    meta: {
-        type: 'suggestion',
-        docs: {
-            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-maintainability/docs/rules/no-nested-ternary.md',
-            description: 'Prevent nested ternary expressions for better readability',
-        },
-        hasSuggestions: true,
-        messages: {
-            noNestedTernary: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.WARNING,
-                issueName: 'Nested Ternary',
-                description: 'Avoid nested ternary expressions',
-                severity: 'MEDIUM',
-                fix: 'Extract to helper variable or use if-else',
-                documentationLink: 'https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-nested-ternary.md',
-            }),
-        },
-        schema: [
-            {
-                type: 'object',
-                properties: {
-                    allow: {
-                        type: 'array',
-                        items: { type: 'string' },
-                        default: [],
-                    },
-                },
-                additionalProperties: false,
-            },
-        ],
-    },
-    defaultOptions: [{ allow: [] }],
-    create(context) {
-        const [options] = context.options;
-        const { allow = [] } = options || {};
-        function isInAllowedContext(node) {
-            if (allow.length === 0) {
-                return false;
-            }
-            let current = node.parent;
-            while (current) {
-                if (allow.includes('jsx') &&
-                    (current.type === 'JSXExpressionContainer' ||
-                        current.type === 'JSXElement' ||
-                        current.type === 'JSXFragment')) {
-                    return true;
-                }
-                if (allow.includes('variable') &&
-                    current.type === 'VariableDeclarator') {
-                    return true;
-                }
-                if (allow.includes('return') && current.type === 'ReturnStatement') {
-                    return true;
-                }
-                if (allow.includes('argument') && current.type === 'CallExpression') {
-                    return true;
-                }
-                current = current.parent;
-            }
-            return false;
-        }
-        function hasNestedTernary(node) {
-            function containsTernary(expr) {
-                if (expr.type === 'ConditionalExpression') {
-                    return true;
-                }
-                switch (expr.type) {
-                    case 'ArrayExpression':
-                        return expr.elements.some((element) => element &&
-                            element.type !== 'SpreadElement' &&
-                            containsTernary(element));
-                    case 'ObjectExpression':
-                        return expr.properties.some((prop) => prop.type === 'Property' &&
-                            prop.value &&
-                            containsTernary(prop.value));
-                    case 'CallExpression':
-                        return (expr.arguments.some((arg) => arg.type !== 'SpreadElement' && containsTernary(arg)) ||
-                            (expr.callee.type !== 'Super' && containsTernary(expr.callee)));
-                    case 'MemberExpression':
-                        return (containsTernary(expr.object) ||
-                            (expr.property.type !== 'Identifier' &&
-                                expr.property.type !== 'PrivateIdentifier' &&
-                                containsTernary(expr.property)));
-                    case 'BinaryExpression':
-                    case 'LogicalExpression':
-                        return containsTernary(expr.left) || containsTernary(expr.right);
-                    case 'UnaryExpression':
-                    case 'UpdateExpression':
-                        return containsTernary(expr.argument);
-                    case 'AssignmentExpression':
-                        return containsTernary(expr.right);
-                    case 'Literal':
-                    case 'Identifier':
-                    case 'ThisExpression':
-                    case 'Super':
-                    case 'MetaProperty':
-                        return false;
-                    case 'TemplateLiteral':
-                        return expr.expressions.some((exp) => containsTernary(exp));
-                    case 'TaggedTemplateExpression':
-                        return (containsTernary(expr.tag) ||
-                            expr.quasi.expressions.some((exp) => containsTernary(exp)));
-                    default:
-                        return false;
-                }
-            }
-            return (containsTernary(node.consequent) || containsTernary(node.alternate));
-        }
-        return {
-            ConditionalExpression(node) {
-                if (hasNestedTernary(node) && !isInAllowedContext(node)) {
-                    context.report({
-                        node,
-                        messageId: 'noNestedTernary',
-                        data: {
-                            current: 'nested ternary expression',
-                            fix: 'extract or use if-else',
-                        },
-                        suggest: [
-                            {
-                                messageId: 'noNestedTernary',
-                                fix() {
-                                    return null;
-                                },
-                            },
-                        ],
-                    });
-                }
-            },
-        };
-    },
-});
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});exports.noNestedTernary=void 0;const eslint_devkit_1=require("@interlace/eslint-devkit");const eslint_devkit_2=require("@interlace/eslint-devkit");exports.noNestedTernary=(0,eslint_devkit_1.createRule)({name:"no-nested-ternary",meta:{type:"suggestion",docs:{url:"https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-maintainability/docs/rules/no-nested-ternary.md",description:"Prevent nested ternary expressions for better readability"},hasSuggestions:true,messages:{noNestedTernary:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.WARNING,issueName:"Nested Ternary",description:"Avoid nested ternary expressions",severity:"MEDIUM",fix:"Extract to helper variable or use if-else",documentationLink:"https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-nested-ternary.md"})},schema:[{type:"object",properties:{allow:{type:"array",items:{type:"string"},default:[]}},additionalProperties:false}]},defaultOptions:[{allow:[]}],create(context){const[options]=context.options;const{allow=[]}=options||{};function isInAllowedContext(node){if(allow.length===0){return false}let current=node.parent;while(current){if(allow.includes("jsx")&&(current.type==="JSXExpressionContainer"||current.type==="JSXElement"||current.type==="JSXFragment")){return true}if(allow.includes("variable")&&current.type==="VariableDeclarator"){return true}if(allow.includes("return")&&current.type==="ReturnStatement"){return true}if(allow.includes("argument")&&current.type==="CallExpression"){return true}current=current.parent}return false}function hasNestedTernary(node){function containsTernary(expr){if(expr.type==="ConditionalExpression"){return true}switch(expr.type){case"ArrayExpression":return expr.elements.some(element=>element&&element.type!=="SpreadElement"&&containsTernary(element));case"ObjectExpression":return expr.properties.some(prop=>prop.type==="Property"&&prop.value&&containsTernary(prop.value));case"CallExpression":return expr.arguments.some(arg=>arg.type!=="SpreadElement"&&containsTernary(arg))||expr.callee.type!=="Super"&&containsTernary(expr.callee);case"MemberExpression":return containsTernary(expr.object)||expr.property.type!=="Identifier"&&expr.property.type!=="PrivateIdentifier"&&containsTernary(expr.property);case"BinaryExpression":case"LogicalExpression":return containsTernary(expr.left)||containsTernary(expr.right);case"UnaryExpression":case"UpdateExpression":return containsTernary(expr.argument);case"AssignmentExpression":return containsTernary(expr.right);case"Literal":case"Identifier":case"ThisExpression":case"Super":case"MetaProperty":return false;case"TemplateLiteral":return expr.expressions.some(exp=>containsTernary(exp));case"TaggedTemplateExpression":return containsTernary(expr.tag)||expr.quasi.expressions.some(exp=>containsTernary(exp));default:return false}}return containsTernary(node.consequent)||containsTernary(node.alternate)}return{ConditionalExpression(node){if(hasNestedTernary(node)&&!isInAllowedContext(node)){context.report({node,messageId:"noNestedTernary",data:{current:"nested ternary expression",fix:"extract or use if-else"},suggest:[{messageId:"noNestedTernary",fix(){return null}}]})}}}}});

package/src/rules/maintainability/no-unreadable-iife.js

@@ -1,206 +1 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.noUnreadableIife = void 0;
-const eslint_devkit_1 = require("@interlace/eslint-devkit");
-const eslint_devkit_2 = require("@interlace/eslint-devkit");
-exports.noUnreadableIife = (0, eslint_devkit_1.createRule)({
-    name: 'no-unreadable-iife',
-    meta: {
-        type: 'suggestion',
-        docs: {
-            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-maintainability/docs/rules/no-unreadable-iife.md',
-            description: 'Prevent unreadable Immediately Invoked Function Expressions that harm code clarity',
-        },
-        hasSuggestions: true,
-        messages: {
-            unreadableIIFE: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.WARNING,
-                issueName: 'Unreadable IIFE',
-                description: 'Complex IIFE reduces code readability',
-                severity: 'MEDIUM',
-                fix: 'Extract to named function or simplify the IIFE structure',
-                documentationLink: 'https://developer.mozilla.org/en-US/docs/Glossary/IIFE',
-            }),
-            suggestNamedFunction: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.INFO,
-                issueName: 'Extract Function',
-                description: 'Extract complex logic to named function',
-                severity: 'LOW',
-                fix: 'function processData() { /* logic */ } processData();',
-                documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Functions',
-            }),
-            suggestBlockScope: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.INFO,
-                issueName: 'Use Block Scope',
-                description: 'Use block scope instead of IIFE',
-                severity: 'LOW',
-                fix: '{ const isolated = value; /* logic */ }',
-                documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/block',
-            }),
-            complexIIFE: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.INFO,
-                issueName: 'Simplify IIFE',
-                description: 'Complex IIFE detected',
-                severity: 'LOW',
-                fix: 'Simplify or extract to separate function',
-                documentationLink: 'https://developer.mozilla.org/en-US/docs/Glossary/IIFE',
-            }),
-        },
-        schema: [
-            {
-                type: 'object',
-                properties: {
-                    maxStatements: {
-                        type: 'number',
-                        minimum: 1,
-                        default: 3,
-                    },
-                    maxDepth: {
-                        type: 'number',
-                        minimum: 1,
-                        default: 2,
-                    },
-                    allowReturningIIFE: {
-                        type: 'boolean',
-                        default: true,
-                    },
-                },
-                additionalProperties: false,
-            },
-        ],
-    },
-    defaultOptions: [{ maxStatements: 3, maxDepth: 2, allowReturningIIFE: true }],
-    create(context) {
-        const [options] = context.options;
-        const { maxStatements = 3, maxDepth = 2, allowReturningIIFE = true } = options || {};
-        function getIifeFunctionNode(node) {
-            if (node.callee.type === 'FunctionExpression' ||
-                node.callee.type === 'ArrowFunctionExpression') {
-                return node.callee;
-            }
-            if (node.callee.type === 'UnaryExpression' &&
-                (node.callee.argument.type === 'FunctionExpression' ||
-                    node.callee.argument.type === 'ArrowFunctionExpression')) {
-                return node.callee.argument;
-            }
-            return null;
-        }
-        function countStatements(body) {
-            if (body.type === 'BlockStatement') {
-                return body.body.length;
-            }
-            return 1;
-        }
-        function calculateDepth(node) {
-            const CONTROL_FLOW_TYPES = new Set([
-                'IfStatement', 'ForStatement', 'ForInStatement', 'ForOfStatement',
-                'WhileStatement', 'DoWhileStatement', 'SwitchStatement', 'TryStatement',
-            ]);
-            const SKIP_KEYS = new Set(['parent', 'loc', 'range', 'tokens', 'comments', 'leadingComments', 'trailingComments']);
-            function isASTNode(value) {
-                return !!value && typeof value === 'object' && 'type' in value && 'loc' in value;
-            }
-            let maxNestingDepth = 0;
-            function walk(current, depth) {
-                const isControlFlow = CONTROL_FLOW_TYPES.has(current.type);
-                const newDepth = isControlFlow ? depth + 1 : depth;
-                maxNestingDepth = Math.max(maxNestingDepth, newDepth);
-                Object.entries(current).forEach(([key, value]) => {
-                    if (SKIP_KEYS.has(key))
-                        return;
-                    if (Array.isArray(value)) {
-                        value.forEach(item => { if (isASTNode(item))
-                            walk(item, newDepth); });
-                    }
-                    else if (isASTNode(value)) {
-                        walk(value, newDepth);
-                    }
-                });
-            }
-            walk(node, 0);
-            return maxNestingDepth;
-        }
-        function hasComplexLogic(body) {
-            if (body.type !== 'BlockStatement') {
-                return false;
-            }
-            if (body.body.length > maxStatements) {
-                return true;
-            }
-            for (const statement of body.body) {
-                if (statement.type === 'IfStatement' ||
-                    statement.type === 'ForStatement' ||
-                    statement.type === 'WhileStatement' ||
-                    statement.type === 'SwitchStatement' ||
-                    statement.type === 'TryStatement') {
-                    return true;
-                }
-                if (statement.type === 'FunctionDeclaration' ||
-                    statement.type === 'VariableDeclaration' &&
-                        statement.declarations.some((decl) => decl.init?.type === 'FunctionExpression' ||
-                            decl.init?.type === 'ArrowFunctionExpression')) {
-                    return true;
-                }
-            }
-            return false;
-        }
-        function analyzeIIFE(node, functionNode) {
-            const body = functionNode.body;
-            const statementCount = countStatements(body);
-            const nestingDepth = calculateDepth(body);
-            const hasComplex = hasComplexLogic(body);
-            const returnsValue = body.type === 'BlockStatement' &&
-                body.body.some((stmt) => stmt.type === 'ReturnStatement');
-            if (allowReturningIIFE && returnsValue && !hasComplex) {
-                return;
-            }
-            const issues = [];
-            if (statementCount > maxStatements) {
-                issues.push(`too many statements (${statementCount} > ${maxStatements})`);
-            }
-            if (nestingDepth > maxDepth) {
-                issues.push(`too deeply nested (${nestingDepth} > ${maxDepth})`);
-            }
-            if (hasComplex) {
-                issues.push('contains complex control flow');
-            }
-            if (functionNode.params.length > 2) {
-                issues.push('too many parameters');
-            }
-            if (issues.length > 0) {
-                context.report({
-                    node,
-                    messageId: 'unreadableIIFE',
-                    data: {
-                        issues: issues.join(', '),
-                        statementCount,
-                        nestingDepth,
-                        suggestion: hasComplex ? 'extract to named function' : 'simplify or use block scope',
-                    },
-                    suggest: [
-                        {
-                            messageId: 'suggestNamedFunction',
-                            fix(fixer) {
-                                return fixer.insertTextBefore(node, '// TODO: Extract complex IIFE to named function\n');
-                            },
-                        },
-                        {
-                            messageId: 'suggestBlockScope',
-                            fix(fixer) {
-                                return fixer.insertTextBefore(node, '// TODO: Consider using block scope { const x = ...; }\n');
-                            },
-                        },
-                    ],
-                });
-            }
-        }
-        return {
-            CallExpression(node) {
-                const functionNode = getIifeFunctionNode(node);
-                if (functionNode) {
-                    analyzeIIFE(node, functionNode);
-                }
-            },
-        };
-    },
-});
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});exports.noUnreadableIife=void 0;const eslint_devkit_1=require("@interlace/eslint-devkit");const eslint_devkit_2=require("@interlace/eslint-devkit");exports.noUnreadableIife=(0,eslint_devkit_1.createRule)({name:"no-unreadable-iife",meta:{type:"suggestion",docs:{url:"https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-maintainability/docs/rules/no-unreadable-iife.md",description:"Prevent unreadable Immediately Invoked Function Expressions that harm code clarity"},hasSuggestions:true,messages:{unreadableIIFE:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.WARNING,issueName:"Unreadable IIFE",description:"Complex IIFE reduces code readability",severity:"MEDIUM",fix:"Extract to named function or simplify the IIFE structure",documentationLink:"https://developer.mozilla.org/en-US/docs/Glossary/IIFE"}),suggestNamedFunction:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.INFO,issueName:"Extract Function",description:"Extract complex logic to named function",severity:"LOW",fix:"function processData() { /* logic */ } processData();",documentationLink:"https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Functions"}),suggestBlockScope:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.INFO,issueName:"Use Block Scope",description:"Use block scope instead of IIFE",severity:"LOW",fix:"{ const isolated = value; /* logic */ }",documentationLink:"https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/block"}),complexIIFE:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.INFO,issueName:"Simplify IIFE",description:"Complex IIFE detected",severity:"LOW",fix:"Simplify or extract to separate function",documentationLink:"https://developer.mozilla.org/en-US/docs/Glossary/IIFE"})},schema:[{type:"object",properties:{maxStatements:{type:"number",minimum:1,default:3},maxDepth:{type:"number",minimum:1,default:2},allowReturningIIFE:{type:"boolean",default:true}},additionalProperties:false}]},defaultOptions:[{maxStatements:3,maxDepth:2,allowReturningIIFE:true}],create(context){const[options]=context.options;const{maxStatements=3,maxDepth=2,allowReturningIIFE=true}=options||{};function getIifeFunctionNode(node){if(node.callee.type==="FunctionExpression"||node.callee.type==="ArrowFunctionExpression"){return node.callee}if(node.callee.type==="UnaryExpression"&&(node.callee.argument.type==="FunctionExpression"||node.callee.argument.type==="ArrowFunctionExpression")){return node.callee.argument}return null}function countStatements(body){if(body.type==="BlockStatement"){return body.body.length}return 1}function calculateDepth(node){const CONTROL_FLOW_TYPES=new Set(["IfStatement","ForStatement","ForInStatement","ForOfStatement","WhileStatement","DoWhileStatement","SwitchStatement","TryStatement"]);const SKIP_KEYS=new Set(["parent","loc","range","tokens","comments","leadingComments","trailingComments"]);function isASTNode(value){return!!value&&typeof value==="object"&&"type"in value&&"loc"in value}let maxNestingDepth=0;function walk(current,depth){const isControlFlow=CONTROL_FLOW_TYPES.has(current.type);const newDepth=isControlFlow?depth+1:depth;maxNestingDepth=Math.max(maxNestingDepth,newDepth);Object.entries(current).forEach(([key,value])=>{if(SKIP_KEYS.has(key))return;if(Array.isArray(value)){value.forEach(item=>{if(isASTNode(item))walk(item,newDepth)})}else if(isASTNode(value)){walk(value,newDepth)}})}walk(node,0);return maxNestingDepth}function hasComplexLogic(body){if(body.type!=="BlockStatement"){return false}if(body.body.length>maxStatements){return true}for(const statement of body.body){if(statement.type==="IfStatement"||statement.type==="ForStatement"||statement.type==="WhileStatement"||statement.type==="SwitchStatement"||statement.type==="TryStatement"){return true}if(statement.type==="FunctionDeclaration"||statement.type==="VariableDeclaration"&&statement.declarations.some(decl=>decl.init?.type==="FunctionExpression"||decl.init?.type==="ArrowFunctionExpression")){return true}}return false}function analyzeIIFE(node,functionNode){const body=functionNode.body;const statementCount=countStatements(body);const nestingDepth=calculateDepth(body);const hasComplex=hasComplexLogic(body);const returnsValue=body.type==="BlockStatement"&&body.body.some(stmt=>stmt.type==="ReturnStatement");if(allowReturningIIFE&&returnsValue&&!hasComplex){return}const issues=[];if(statementCount>maxStatements){issues.push(`too many statements (${statementCount} > ${maxStatements})`)}if(nestingDepth>maxDepth){issues.push(`too deeply nested (${nestingDepth} > ${maxDepth})`)}if(hasComplex){issues.push("contains complex control flow")}if(functionNode.params.length>2){issues.push("too many parameters")}if(issues.length>0){context.report({node,messageId:"unreadableIIFE",data:{issues:issues.join(", "),statementCount,nestingDepth,suggestion:hasComplex?"extract to named function":"simplify or use block scope"},suggest:[{messageId:"suggestNamedFunction",fix(fixer){return fixer.insertTextBefore(node,"// TODO: Extract complex IIFE to named function\n")}},{messageId:"suggestBlockScope",fix(fixer){return fixer.insertTextBefore(node,"// TODO: Consider using block scope { const x = ...; }\n")}}]})}}return{CallExpression(node){const functionNode=getIifeFunctionNode(node);if(functionNode){analyzeIIFE(node,functionNode)}}}}});

package/src/types/index.js

@@ -1,2 +1 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});