@blumintinc/eslint-plugin-blumint 1.20.37 → 1.20.39

package/lib/index.js

@@ -223,7 +223,7 @@ function noFrontendImportsFromFunctionsPatterns(pattern) {
 module.exports = {
     meta: {
         name: '@blumintinc/eslint-plugin-blumint',
-        version: '1.20.37',
+        version: '1.20.39',
     },
     parseOptions: {
         ecmaVersion: 2020,

package/lib/rules/enforce-callback-memo.js

@@ -22,21 +22,48 @@ exports.default = (0, createRule_1.createRule)({
             return (node.type === utils_1.AST_NODE_TYPES.ArrowFunctionExpression ||
                 node.type === utils_1.AST_NODE_TYPES.FunctionExpression);
         }
+        // Matches both the bare hook (`useMemo(...)`, including the generic form
+        // `useCallback<T>(...)`, whose callee is still an identifier) and the
+        // namespaced form (`React.useMemo(...)`).
+        function isHookCallee(callee, hookName) {
+            if (callee.type === utils_1.AST_NODE_TYPES.Identifier) {
+                return callee.name === hookName;
+            }
+            return (callee.type === utils_1.AST_NODE_TYPES.MemberExpression &&
+                !callee.computed &&
+                callee.property.type === utils_1.AST_NODE_TYPES.Identifier &&
+                callee.property.name === hookName);
+        }
         function isInsideUseCallback(node) {
             let current = node.parent;
             while (current) {
-                if (current.type === utils_1.AST_NODE_TYPES.CallExpression) {
-                    const { callee } = current;
-                    const isDirectUseCallback = callee.type === utils_1.AST_NODE_TYPES.Identifier &&
-                        callee.name === 'useCallback';
-                    const isMemberUseCallback = callee.type === utils_1.AST_NODE_TYPES.MemberExpression &&
-                        !callee.computed &&
-                        callee.property.type === utils_1.AST_NODE_TYPES.Identifier &&
-                        callee.property.name === 'useCallback';
-                    if (isDirectUseCallback || isMemberUseCallback) {
-                        return true;
-                    }
+                if (current.type === utils_1.AST_NODE_TYPES.CallExpression &&
+                    isHookCallee(current.callee, 'useCallback')) {
+                    return true;
+                }
+                current = current.parent;
+            }
+            return false;
+        }
+        // A `useMemo` factory re-runs only when its dependencies change, so every
+        // value it produces — including JSX and the inline handlers nested in that
+        // JSX — is exactly as referentially stable as the memo itself. Demanding an
+        // extra `useCallback` there buys nothing.
+        //
+        // `useCallback` deliberately gets no such treatment: it memoizes a function
+        // that runs on every invocation, so the JSX it returns (and the inline
+        // functions in it) is rebuilt each call and stays worth reporting.
+        function isInsideUseMemoFactory(node) {
+            let child = node;
+            let current = node.parent;
+            while (current) {
+                if (current.type === utils_1.AST_NODE_TYPES.CallExpression &&
+                    isHookCallee(current.callee, 'useMemo') &&
+                    // Only the factory argument is memoized; the dependency array is not.
+                    current.arguments[0] === child) {
+                    return true;
                 }
+                child = current;
                 current = current.parent;
             }
             return false;
@@ -167,6 +194,10 @@ exports.default = (0, createRule_1.createRule)({
                 node.value.type !== utils_1.AST_NODE_TYPES.JSXExpressionContainer) {
                 return;
             }
+            // Props of JSX built inside a useMemo factory inherit the memo's stability
+            if (isInsideUseMemoFactory(node)) {
+                return;
+            }
             const { expression } = node.value;
             // Skip if the prop is already wrapped in useCallback or useMemo
             if (expression.type === utils_1.AST_NODE_TYPES.CallExpression &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blumintinc/eslint-plugin-blumint",
-  "version": "1.20.37",
+  "version": "1.20.39",
   "description": "Custom eslint rules for use within BluMint",
   "author": {
     "name": "Brodie McGuire",

package/release-manifest.json

@@ -1,4 +1,23 @@
 [
+  {
+    "version": "1.20.39",
+    "date": "2026-07-30T21:18:09.189Z",
+    "rules": []
+  },
+  {
+    "version": "1.20.38",
+    "date": "2026-07-30T20:52:16.921Z",
+    "rules": [
+      {
+        "name": "enforce-callback-memo",
+        "changeType": "fix",
+        "issues": [
+          1465
+        ],
+        "summary": "allow an inline callback inside a useMemo factory (closes #1465)"
+      }
+    ]
+  },
   {
     "version": "1.20.37",
     "date": "2026-07-30T18:07:02.175Z",