@blumintinc/eslint-plugin-blumint 1.19.27 → 1.19.28

package/lib/index.js

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

package/lib/rules/memo-nested-react-components.js

@@ -339,6 +339,35 @@ const returnExpressionIsJsx = (expression) => {
     return (unwrapped.type === utils_1.AST_NODE_TYPES.JSXElement ||
         unwrapped.type === utils_1.AST_NODE_TYPES.JSXFragment);
 };
+/**
+ * True when the expression is a `memo(...)` / `forwardRef(...)` call. Returned
+ * from a useMemo/useDeepCompareMemo callback, this is the complete, sanctioned
+ * stabilization pattern: the memo hook stabilizes the component *identity*
+ * across re-renders (a new identity only on a deps change), so the inner
+ * component never remounts on an ordinary re-render—the exact harm this rule
+ * exists to prevent does not occur.
+ */
+const returnExpressionIsMemoOrForwardRefCall = (expression, reactImports) => {
+    if (!expression) {
+        return false;
+    }
+    const unwrapped = unwrapExpression(expression);
+    return (unwrapped.type === utils_1.AST_NODE_TYPES.CallExpression &&
+        (isMemoCall(unwrapped, reactImports) ||
+            isForwardRefCall(unwrapped, reactImports)));
+};
+/**
+ * True when a useMemo/useDeepCompareMemo callback directly returns a
+ * memo()/forwardRef()-wrapped component. The memo hook stabilizes the returned
+ * component's identity, so such a binding does not remount on re-render and
+ * must not be flagged.
+ */
+const memoCallbackReturnsMemoizedComponent = (callback, reactImports) => {
+    if (!isFunctionExpression(callback)) {
+        return false;
+    }
+    return collectDirectReturnExpressions(callback).some((expression) => returnExpressionIsMemoOrForwardRefCall(expression, reactImports));
+};
 /**
  * Direct return-statement arguments of a function, traversing control flow
  * (if/switch/try/loops) but NOT descending into nested function definitions—
@@ -486,6 +515,12 @@ See: https://react.dev/learn/your-first-component#nesting-and-organizing-compone
                         // Returns JSX directly, so it's an element, not a component
                         return;
                     }
+                    // A memo()/forwardRef()-wrapped return is identity-stabilized by the
+                    // memo hook, so it does not remount on re-render and must not be
+                    // flagged (see #1336). Bare inner components stay flagged.
+                    if (memoCallbackReturnsMemoizedComponent(callback, reactImports)) {
+                        return;
+                    }
                 }
                 const variableName = getVariableName(node);
                 // A non-PascalCase binding (e.g. renderHit) is a render callback used

package/package.json

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

package/release-manifest.json

@@ -1,4 +1,18 @@
 [
+  {
+    "version": "1.19.28",
+    "date": "2026-07-23T17:25:39.744Z",
+    "rules": [
+      {
+        "name": "memo-nested-react-components",
+        "changeType": "fix",
+        "issues": [
+          1336
+        ],
+        "summary": "exempt memo/forwardRef components returned from useMemo/useDeepCompareMemo callbacks (closes #1336)"
+      }
+    ]
+  },
   {
     "version": "1.19.27",
     "date": "2026-07-23T04:26:05.419Z",