@blumintinc/eslint-plugin-blumint 1.13.0 → 1.15.0

package/lib/rules/no-inline-component-prop.js

@@ -0,0 +1,465 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.noInlineComponentProp = void 0;
+const utils_1 = require("@typescript-eslint/utils");
+const ts = __importStar(require("typescript"));
+const createRule_1 = require("../utils/createRule");
+const ASTHelpers_1 = require("../utils/ASTHelpers");
+const DEFAULT_PROP_PATTERNS = ['CatalogWrapper', '*Wrapper', '*Component'];
+const DEFAULT_OPTIONS = {
+    props: DEFAULT_PROP_PATTERNS,
+    allowRenderProps: true,
+    allowModuleScopeFactories: true,
+};
+const INLINE_COMPONENT_NAME = 'inline component';
+function isPascalCase(name) {
+    return /^[A-Z][A-Za-z0-9]*$/.test(name);
+}
+function matchesPattern(name, pattern) {
+    if (pattern === '*')
+        return true;
+    if (pattern.includes('*')) {
+        const wildcardCount = (pattern.match(/\*/g) || []).length;
+        if (wildcardCount > 2) {
+            return false;
+        }
+        const escaped = pattern
+            .replace(/[-/\\^$+?.()|[\]{}]/g, '\\$&')
+            .replace(/\*/g, '.*');
+        return new RegExp(`^${escaped}$`).test(name);
+    }
+    return name === pattern;
+}
+function isRenderPropName(name) {
+    if (name === 'children' || name === 'child' || name === 'render') {
+        return true;
+    }
+    if (name.startsWith('render')) {
+        return true;
+    }
+    return (name === 'rowRenderer' ||
+        name === 'cellRenderer' ||
+        name === 'itemRenderer' ||
+        name === 'renderItem' ||
+        name === 'renderRow');
+}
+function unwrapExpression(expr) {
+    let current = expr;
+    while (current) {
+        if (current.type === utils_1.AST_NODE_TYPES.TSAsExpression ||
+            current.type === utils_1.AST_NODE_TYPES.TSSatisfiesExpression) {
+            current = current.expression;
+            continue;
+        }
+        if (current.type === utils_1.AST_NODE_TYPES.ChainExpression) {
+            current = current.expression;
+            continue;
+        }
+        break;
+    }
+    return current ?? null;
+}
+function isReactCreateElementCall(expr) {
+    if (expr.type !== utils_1.AST_NODE_TYPES.CallExpression)
+        return false;
+    const callee = expr.callee;
+    if (callee.type === utils_1.AST_NODE_TYPES.Identifier &&
+        callee.name === 'createElement') {
+        return true;
+    }
+    if (callee.type === utils_1.AST_NODE_TYPES.MemberExpression &&
+        !callee.computed &&
+        callee.property.type === utils_1.AST_NODE_TYPES.Identifier &&
+        callee.property.name === 'createElement') {
+        if (callee.object.type === utils_1.AST_NODE_TYPES.Identifier &&
+            callee.object.name === 'React') {
+            return true;
+        }
+    }
+    return false;
+}
+function returnsCreateElement(node) {
+    if (node.type === utils_1.AST_NODE_TYPES.CallExpression &&
+        isReactCreateElementCall(node)) {
+        return true;
+    }
+    if (node.type === utils_1.AST_NODE_TYPES.BlockStatement) {
+        for (const statement of node.body) {
+            if (statement.type === utils_1.AST_NODE_TYPES.ReturnStatement &&
+                statement.argument &&
+                statement.argument.type === utils_1.AST_NODE_TYPES.CallExpression &&
+                isReactCreateElementCall(statement.argument)) {
+                return true;
+            }
+        }
+    }
+    return false;
+}
+function isFunctionNode(node) {
+    if (!node)
+        return false;
+    return (node.type === utils_1.AST_NODE_TYPES.ArrowFunctionExpression ||
+        node.type === utils_1.AST_NODE_TYPES.FunctionExpression ||
+        node.type === utils_1.AST_NODE_TYPES.FunctionDeclaration);
+}
+function getCalleeName(callee) {
+    const unwrapped = unwrapExpression(callee);
+    if (!unwrapped)
+        return null;
+    if (unwrapped.type === utils_1.AST_NODE_TYPES.Identifier) {
+        return unwrapped.name;
+    }
+    if (unwrapped.type === utils_1.AST_NODE_TYPES.MemberExpression &&
+        !unwrapped.computed &&
+        unwrapped.property.type === utils_1.AST_NODE_TYPES.Identifier) {
+        const prop = unwrapped.property.name;
+        if (unwrapped.object.type === utils_1.AST_NODE_TYPES.Identifier &&
+            unwrapped.object.name) {
+            return `${unwrapped.object.name}.${prop}`;
+        }
+        return prop;
+    }
+    return null;
+}
+function getFunctionFromCall(call) {
+    const calleeName = getCalleeName(call.callee);
+    const firstArg = unwrapExpression(call.arguments[0] ?? null);
+    if (!firstArg || !isFunctionNode(firstArg)) {
+        return undefined;
+    }
+    if (calleeName === 'useCallback' ||
+        calleeName === 'React.useCallback' ||
+        calleeName === 'useMemo' ||
+        calleeName === 'React.useMemo' ||
+        calleeName === 'memo' ||
+        calleeName === 'React.memo' ||
+        calleeName === 'forwardRef' ||
+        calleeName === 'React.forwardRef') {
+        return firstArg;
+    }
+    return undefined;
+}
+function getFunctionFromInit(init) {
+    const unwrapped = unwrapExpression(init);
+    if (!unwrapped)
+        return undefined;
+    if (isFunctionNode(unwrapped)) {
+        return unwrapped;
+    }
+    if (unwrapped.type === utils_1.AST_NODE_TYPES.CallExpression) {
+        return getFunctionFromCall(unwrapped);
+    }
+    return undefined;
+}
+function isInModuleScope(node) {
+    let current = node.parent;
+    while (current) {
+        if (current.type === utils_1.AST_NODE_TYPES.FunctionDeclaration ||
+            current.type === utils_1.AST_NODE_TYPES.FunctionExpression ||
+            current.type === utils_1.AST_NODE_TYPES.ArrowFunctionExpression ||
+            current.type === utils_1.AST_NODE_TYPES.MethodDefinition) {
+            return false;
+        }
+        if (current.type === utils_1.AST_NODE_TYPES.Program) {
+            return true;
+        }
+        current = current.parent;
+    }
+    return true;
+}
+function isComponentLikeFunction(fn, context, displayName) {
+    const body = fn.body;
+    const hasJSX = !!body && ASTHelpers_1.ASTHelpers.returnsJSX(body, context);
+    const expressionBody = fn.type === utils_1.AST_NODE_TYPES.ArrowFunctionExpression &&
+        fn.body.type !== utils_1.AST_NODE_TYPES.BlockStatement
+        ? fn.body
+        : null;
+    const hasCreateElement = (expressionBody && isReactCreateElementCall(expressionBody)) ||
+        (body ? returnsCreateElement(body) : false);
+    const looksLikeComponent = displayName ? isPascalCase(displayName) : false;
+    return hasJSX || hasCreateElement || looksLikeComponent;
+}
+function findVariableInScopes(context, identifier) {
+    const sourceCode = context.sourceCode;
+    let scope = null;
+    try {
+        // Tolerate ESLint API variations across versions/configurations.
+        // Some contexts may not provide getScope or may throw unexpectedly.
+        scope = sourceCode.getScope?.(identifier) ?? context.getScope();
+    }
+    catch {
+        scope = null;
+    }
+    while (scope) {
+        const variable = scope.variables.find((v) => v.name === identifier.name);
+        if (variable)
+            return variable;
+        scope = scope.upper;
+    }
+    return undefined;
+}
+function findObjectPropertyFunction(objExpr, propertyName) {
+    for (const prop of objExpr.properties) {
+        if (prop.type !== utils_1.AST_NODE_TYPES.Property)
+            continue;
+        if (prop.computed)
+            continue;
+        if (prop.key.type === utils_1.AST_NODE_TYPES.Identifier) {
+            if (prop.key.name !== propertyName)
+                continue;
+        }
+        else if (prop.key.type === utils_1.AST_NODE_TYPES.Literal &&
+            prop.key.value === propertyName) {
+            // match
+        }
+        else {
+            continue;
+        }
+        const value = unwrapExpression(prop.value);
+        if (value && isFunctionNode(value)) {
+            return value;
+        }
+    }
+    return undefined;
+}
+exports.noInlineComponentProp = (0, createRule_1.createRule)({
+    name: 'no-inline-component-prop',
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Prevent inline function components defined in render from being passed to component-type props like CatalogWrapper to avoid remounts and UI flashes.',
+            recommended: 'error',
+        },
+        fixable: undefined,
+        schema: [
+            {
+                type: 'object',
+                properties: {
+                    props: {
+                        type: 'array',
+                        items: { type: 'string' },
+                        default: DEFAULT_PROP_PATTERNS,
+                    },
+                    allowRenderProps: {
+                        type: 'boolean',
+                        default: true,
+                    },
+                    allowModuleScopeFactories: {
+                        type: 'boolean',
+                        default: true,
+                    },
+                },
+                additionalProperties: false,
+            },
+        ],
+        messages: {
+            inlineComponentProp: 'Inline component "{{componentName}}" is created inside a render scope and passed to component-type prop "{{propName}}". React treats it as a new component whenever the scope re-runs, remounting its subtree and causing UI flashes. Move the wrapper to module scope (optionally memoize with React.memo) and pass changing data via props or context instead.',
+        },
+    },
+    defaultOptions: [DEFAULT_OPTIONS],
+    create(context, [options]) {
+        const resolvedOptions = { ...DEFAULT_OPTIONS, ...(options ?? {}) };
+        const propPatterns = resolvedOptions.props ?? DEFAULT_PROP_PATTERNS;
+        const parserServices = context.parserServices;
+        const checker = parserServices?.program?.getTypeChecker();
+        const esTreeNodeToTSNodeMap = parserServices?.esTreeNodeToTSNodeMap;
+        function isComponentTypeByTypeInfo(attribute) {
+            if (!checker || !esTreeNodeToTSNodeMap)
+                return false;
+            const tsNode = esTreeNodeToTSNodeMap.get(attribute);
+            if (!tsNode)
+                return false;
+            let type;
+            if (ts.isExpression(tsNode)) {
+                type =
+                    checker.getContextualType(tsNode) ??
+                        checker.getTypeAtLocation(tsNode);
+            }
+            else {
+                type = checker.getTypeAtLocation(tsNode);
+            }
+            if (!type)
+                return false;
+            const typeText = checker.typeToString(type);
+            return (typeText.includes('ComponentType') ||
+                typeText.includes('FunctionComponent') ||
+                typeText.includes('ReactElement') ||
+                typeText.includes('FC'));
+        }
+        function isTargetProp(attribute, name) {
+            if (resolvedOptions.allowRenderProps && isRenderPropName(name)) {
+                return false;
+            }
+            const patternMatch = propPatterns.some((pattern) => matchesPattern(name, pattern));
+            const looksComponent = isPascalCase(name) ||
+                name.endsWith('Wrapper') ||
+                name.endsWith('Component');
+            if (patternMatch) {
+                return true;
+            }
+            if (looksComponent) {
+                return isComponentTypeByTypeInfo(attribute);
+            }
+            return false;
+        }
+        function shouldReportDefinition(definition, displayName) {
+            if (definition.type === 'ImportBinding' ||
+                definition.type === 'Parameter' ||
+                definition.type === 'Type') {
+                return false;
+            }
+            const defNode = definition.node;
+            const moduleScoped = isInModuleScope(defNode);
+            if (moduleScoped && resolvedOptions.allowModuleScopeFactories) {
+                return false;
+            }
+            let fnNode;
+            if (defNode.type === utils_1.AST_NODE_TYPES.FunctionDeclaration) {
+                fnNode = defNode;
+            }
+            else if (defNode.type === utils_1.AST_NODE_TYPES.VariableDeclarator) {
+                fnNode = getFunctionFromInit(defNode.init);
+            }
+            else {
+                return false;
+            }
+            if (!fnNode)
+                return false;
+            return isComponentLikeFunction(fnNode, context, displayName);
+        }
+        function report(node, propName, componentName) {
+            context.report({
+                node,
+                messageId: 'inlineComponentProp',
+                data: { propName, componentName },
+            });
+        }
+        function handleIdentifierExpression(identifier, propName) {
+            const variable = findVariableInScopes(context, identifier);
+            if (!variable)
+                return;
+            const definition = variable.defs.find((def) => (def.node.type === utils_1.AST_NODE_TYPES.FunctionDeclaration ||
+                def.node.type === utils_1.AST_NODE_TYPES.VariableDeclarator) &&
+                def.type !== 'Parameter' &&
+                def.type !== 'ImportBinding');
+            if (!definition)
+                return;
+            if (shouldReportDefinition(definition, identifier.name)) {
+                report(identifier, propName, identifier.name);
+            }
+        }
+        function handleMemberExpression(member, propName) {
+            if (member.computed)
+                return;
+            if (member.property.type !== utils_1.AST_NODE_TYPES.Identifier)
+                return;
+            const objectId = member.object;
+            if (objectId.type !== utils_1.AST_NODE_TYPES.Identifier)
+                return;
+            const variable = findVariableInScopes(context, objectId);
+            if (!variable)
+                return;
+            const definition = variable.defs.find((def) => def.node.type === utils_1.AST_NODE_TYPES.VariableDeclarator &&
+                def.node.id.type === utils_1.AST_NODE_TYPES.Identifier &&
+                def.node.id.name === objectId.name &&
+                def.type !== 'ImportBinding' &&
+                def.type !== 'Parameter' &&
+                def.type !== 'Type');
+            if (!definition)
+                return;
+            const defNode = definition.node;
+            if (!defNode ||
+                defNode.type !== utils_1.AST_NODE_TYPES.VariableDeclarator ||
+                !defNode.init ||
+                defNode.init.type !== utils_1.AST_NODE_TYPES.ObjectExpression) {
+                return;
+            }
+            if (isInModuleScope(defNode) &&
+                resolvedOptions.allowModuleScopeFactories) {
+                return;
+            }
+            const fnNode = findObjectPropertyFunction(defNode.init, member.property.name);
+            if (fnNode &&
+                isComponentLikeFunction(fnNode, context, member.property.name)) {
+                report(member, propName, member.property.name);
+            }
+        }
+        function handleInlineFunctionExpression(fn, propName) {
+            const explicitName = fn.type === utils_1.AST_NODE_TYPES.FunctionExpression ? fn.id?.name : undefined;
+            if (!isComponentLikeFunction(fn, context, explicitName)) {
+                return;
+            }
+            const displayName = (fn.type === utils_1.AST_NODE_TYPES.FunctionExpression && fn.id?.name) ||
+                INLINE_COMPONENT_NAME;
+            report(fn, propName, displayName);
+        }
+        function handleCallExpression(call, propName) {
+            const fnNode = getFunctionFromCall(call);
+            if (!fnNode)
+                return;
+            if (isComponentLikeFunction(fnNode, context)) {
+                report(call, propName, INLINE_COMPONENT_NAME);
+            }
+        }
+        return {
+            JSXAttribute(node) {
+                if (!node.name || node.name.type !== utils_1.AST_NODE_TYPES.JSXIdentifier) {
+                    return;
+                }
+                const propName = node.name.name;
+                if (!isTargetProp(node, propName))
+                    return;
+                if (!node.value ||
+                    node.value.type !== utils_1.AST_NODE_TYPES.JSXExpressionContainer) {
+                    return;
+                }
+                if (node.value.expression.type === utils_1.AST_NODE_TYPES.JSXEmptyExpression) {
+                    return;
+                }
+                const expression = unwrapExpression(node.value.expression);
+                if (!expression)
+                    return;
+                if (expression.type === utils_1.AST_NODE_TYPES.ArrowFunctionExpression ||
+                    expression.type === utils_1.AST_NODE_TYPES.FunctionExpression) {
+                    handleInlineFunctionExpression(expression, propName);
+                    return;
+                }
+                if (expression.type === utils_1.AST_NODE_TYPES.CallExpression) {
+                    handleCallExpression(expression, propName);
+                    return;
+                }
+                if (expression.type === utils_1.AST_NODE_TYPES.Identifier) {
+                    handleIdentifierExpression(expression, propName);
+                    return;
+                }
+                if (expression.type === utils_1.AST_NODE_TYPES.MemberExpression) {
+                    handleMemberExpression(expression, propName);
+                }
+            },
+        };
+    },
+});
+//# sourceMappingURL=no-inline-component-prop.js.map

package/lib/rules/no-jsx-in-hooks.js

@@ -135,7 +135,7 @@ exports.noJsxInHooks = (0, createRule_1.createRule)({
         },
         schema: [],
         messages: {
-            noJsxInHooks: 'Hooks should not return JSX. Convert this hook into a component or extract the JSX into a separate component.',
+            noJsxInHooks: 'Hook "{{hookName}}" returns JSX. Hooks are meant to expose data and behavior; embedding JSX turns the hook into a hidden component and forces callers to use a hook where a component should render. Move the JSX into a component and have the hook return the values and callbacks that component needs instead.',
         },
     },
     defaultOptions: [],
@@ -148,6 +148,7 @@ exports.noJsxInHooks = (0, createRule_1.createRule)({
                         context.report({
                             node: node.id,
                             messageId: 'noJsxInHooks',
+                            data: { hookName: node.id.name },
                         });
                         return;
                     }
@@ -157,6 +158,7 @@ exports.noJsxInHooks = (0, createRule_1.createRule)({
                             context.report({
                                 node: node.id,
                                 messageId: 'noJsxInHooks',
+                                data: { hookName: node.id.name },
                             });
                         }
                     }
@@ -173,6 +175,7 @@ exports.noJsxInHooks = (0, createRule_1.createRule)({
                         context.report({
                             node: parent.id,
                             messageId: 'noJsxInHooks',
+                            data: { hookName: parent.id.name },
                         });
                         return;
                     }
@@ -181,6 +184,7 @@ exports.noJsxInHooks = (0, createRule_1.createRule)({
                         context.report({
                             node: parent.id,
                             messageId: 'noJsxInHooks',
+                            data: { hookName: parent.id.name },
                         });
                         return;
                     }
@@ -190,6 +194,7 @@ exports.noJsxInHooks = (0, createRule_1.createRule)({
                             context.report({
                                 node: parent.id,
                                 messageId: 'noJsxInHooks',
+                                data: { hookName: parent.id.name },
                             });
                         }
                     }

package/lib/rules/no-jsx-whitespace-literal.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noJsxWhitespaceLiteral = void 0;
+const utils_1 = require("@typescript-eslint/utils");
 const createRule_1 = require("../utils/createRule");
 exports.noJsxWhitespaceLiteral = (0, createRule_1.createRule)({
     name: 'no-jsx-whitespace-literal',
@@ -12,19 +13,24 @@ exports.noJsxWhitespaceLiteral = (0, createRule_1.createRule)({
         },
         schema: [],
         messages: {
-            noWhitespaceLiteral: 'Avoid using {" "} for spacing in JSX. Use proper text nodes or CSS spacing instead.',
+            noWhitespaceLiteral: 'Whitespace-only JSX expression {{literal}} inserts fragile spacer nodes → React treats the whitespace as a separate text child that shifts, disappears, or duplicates when child elements are reordered, translated, or dynamically rendered → Place spacing inside text content (e.g., "Hello ") or use CSS spacing such as gap, margin, or padding.',
         },
     },
     defaultOptions: [],
     create(context) {
+        const typedContext = context;
+        const sourceCode = typedContext.sourceCode ?? context.getSourceCode();
         return {
             JSXExpressionContainer(node) {
-                if (node.expression.type === 'Literal' &&
+                if (node.expression.type === utils_1.AST_NODE_TYPES.Literal &&
                     typeof node.expression.value === 'string' &&
                     node.expression.value.trim() === '') {
                     context.report({
                         node,
                         messageId: 'noWhitespaceLiteral',
+                        data: {
+                            literal: sourceCode.getText(node),
+                        },
                     });
                 }
             },

package/lib/rules/no-margin-properties.js

@@ -39,7 +39,7 @@ exports.noMarginProperties = (0, createRule_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Discourage using margin properties (margin, marginLeft, marginRight, marginTop, marginBottom, mx, my, etc.) for spacing in MUI components. Instead, prefer defining spacing with padding, gap, or the spacing prop for more predictable layouts.',
+            description: 'Prevent margin properties (margin, marginLeft, marginRight, marginTop, marginBottom, mx, my, etc.) in MUI styling because margins fight container-controlled spacing, double gutters, and misaligned breakpoints; keep spacing centralized with padding, gap, or spacing props instead.',
             recommended: 'warn',
         },
         schema: [
@@ -55,7 +55,7 @@ exports.noMarginProperties = (0, createRule_1.createRule)({
             },
         ],
         messages: {
-            noMarginProperties: 'Avoid using {{property}} for spacing in MUI components. Use padding, gap, or the spacing prop instead for more predictable layouts. See https://www.youtube.com/watch?v=KVQMoEFUee8 for more details.',
+            noMarginProperties: 'Margin property "{{property}}" in MUI styling fights container-controlled spacing (Stack/Grid spacing, gap, responsive gutters) and produces double gutters, misalignment, and overflow as layouts shift. Keep spacing inside the component with padding or let the parent handle separation via gap/spacing so layout remains predictable.',
         },
     },
     defaultOptions: [{ autofix: false }],
@@ -233,7 +233,7 @@ exports.noMarginProperties = (0, createRule_1.createRule)({
                 if (node.init?.type === utils_1.AST_NODE_TYPES.ObjectExpression &&
                     node.id.type === utils_1.AST_NODE_TYPES.Identifier) {
                     const variableName = node.id.name;
-                    const sourceCode = context.getSourceCode().getText();
+                    const sourceText = context.sourceCode.getText();
                     // Check for margin properties in the object
                     node.init.properties.forEach((prop) => {
                         if (prop.type === utils_1.AST_NODE_TYPES.Property) {
@@ -246,9 +246,9 @@ exports.noMarginProperties = (0, createRule_1.createRule)({
                             }
                             if (propertyName && checkProperty(propertyName)) {
                                 // Check if this variable is used in an sx prop
-                                if (sourceCode.includes(`sx={${variableName}}`) ||
-                                    sourceCode.includes(`sx={{ ...${variableName}`) ||
-                                    sourceCode.includes(`sx={Object.assign({}, ${variableName}`)) {
+                                if (sourceText.includes(`sx={${variableName}}`) ||
+                                    sourceText.includes(`sx={{ ...${variableName}`) ||
+                                    sourceText.includes(`sx={Object.assign({}, ${variableName}`)) {
                                     context.report({
                                         node: prop,
                                         messageId: 'noMarginProperties',

package/lib/rules/no-memoize-on-static.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.noMemoizeOnStatic = void 0;
 const utils_1 = require("@typescript-eslint/utils");
 const createRule_1 = require("../utils/createRule");
+const getMethodName_1 = require("../utils/getMethodName");
 exports.noMemoizeOnStatic = (0, createRule_1.createRule)({
     name: 'no-memoize-on-static',
     meta: {
@@ -13,11 +14,12 @@ exports.noMemoizeOnStatic = (0, createRule_1.createRule)({
         },
         schema: [],
         messages: {
-            noMemoizeOnStatic: '@Memoize() decorator should not be used on static methods',
+            noMemoizeOnStatic: 'Static member "{{methodName}}" uses @Memoize(), which shares one cache across every caller instead of per instance. Static memoization can leak stale or cross-tenant data because the cache survives across requests for the lifetime of the process. Remove @Memoize() or move the logic into an instance method so each consumer gets its own cache lifecycle.',
         },
     },
     defaultOptions: [],
     create(context) {
+        const sourceCode = context.getSourceCode();
         // Track renamed imports of Memoize
         const memoizeAliases = new Set(['Memoize']);
         return {
@@ -43,6 +45,12 @@ exports.noMemoizeOnStatic = (0, createRule_1.createRule)({
                             context.report({
                                 node: decorator,
                                 messageId: 'noMemoizeOnStatic',
+                                data: {
+                                    methodName: (0, getMethodName_1.getMethodName)(node, sourceCode, {
+                                        privateIdentifierPrefix: '#',
+                                        computedFallbackToText: false,
+                                    }) || '<unknown>',
+                                },
                             });
                         }
                     }

package/lib/rules/no-misleading-boolean-prefixes.js

@@ -210,7 +210,7 @@ exports.noMisleadingBooleanPrefixes = (0, createRule_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
-            description: 'Reserve boolean-style prefixes (is/has/should) for functions that actually return boolean values.',
+            description: 'Reserve boolean-style prefixes (is/has/should) for functions that actually return boolean values to avoid misleading call sites.',
             recommended: 'error',
         },
         schema: [
@@ -226,7 +226,7 @@ exports.noMisleadingBooleanPrefixes = (0, createRule_1.createRule)({
             },
         ],
         messages: {
-            nonBooleanReturn: 'Function "{{name}}" uses a boolean-style prefix but does not return a boolean value.',
+            nonBooleanReturn: 'Function "{{name}}" uses a boolean-style prefix but its return value is not guaranteed to be boolean. Boolean prefixes promise a yes/no answer; returning strings, objects, or void misleads callers and hides incorrect branching. Return a real boolean (e.g., add an explicit comparison or Boolean(...)) or rename the function to drop the boolean-style prefix (prefixes treated as boolean: {{prefixes}}).',
         },
     },
     defaultOptions: [{ prefixes: DEFAULT_PREFIXES }],
@@ -236,7 +236,11 @@ exports.noMisleadingBooleanPrefixes = (0, createRule_1.createRule)({
             return startsWithBooleanPrefix(name, prefixes);
         }
         function report(node, name) {
-            context.report({ node, messageId: 'nonBooleanReturn', data: { name } });
+            context.report({
+                node,
+                messageId: 'nonBooleanReturn',
+                data: { name, prefixes: prefixes.join(', ') },
+            });
         }
         function checkFunctionLike(node, name, reportNode) {
             if (!shouldCheckName(name))