@blumintinc/eslint-plugin-blumint 1.7.2 → 1.7.3

package/lib/index.js

@@ -73,7 +73,7 @@ const prefer_fragment_component_1 = require("./rules/prefer-fragment-component")
 module.exports = {
     meta: {
         name: '@blumintinc/eslint-plugin-blumint',
-        version: '1.7.2',
+        version: '1.7.3',
     },
     parseOptions: {
         ecmaVersion: 2020,

package/lib/rules/enforce-centralized-mock-firestore.js

@@ -3,7 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.enforceCentralizedMockFirestore = void 0;
 const utils_1 = require("@typescript-eslint/utils");
 const createRule_1 = require("../utils/createRule");
-const MOCK_FIRESTORE_PATH = '../../../../../__mocks__/functions/src/config/mockFirestore';
+const MOCK_FIRESTORE_PATH = '../../../../../__test-utils__/mockFirestore';
 exports.enforceCentralizedMockFirestore = (0, createRule_1.createRule)({
     name: 'enforce-centralized-mock-firestore',
     meta: {

package/lib/rules/enforce-mock-firestore.js

@@ -18,8 +18,8 @@ exports.enforceFirestoreMock = (0, createRule_1.createRule)({
         },
         schema: [],
         messages: {
-            noManualFirestoreMock: 'Use mockFirestore from __mocks__/functions/src/config/mockFirestore instead of manually mocking Firestore. Replace `jest.mock("firebase-admin", () => ({ firestore: () => ({ /* mock */ }) }))` with `import { mockFirestore } from "__mocks__/functions/src/config/mockFirestore"; jest.mock("firebase-admin", () => mockFirestore)`.',
-            noMockFirebase: 'Use mockFirestore from __mocks__/functions/src/config/mockFirestore instead of mockFirebase. Replace `import { mockFirebase } from "firestore-jest-mock"` with `import { mockFirestore } from "__mocks__/functions/src/config/mockFirestore"`.',
+            noManualFirestoreMock: 'Use mockFirestore from __test-utils__/mockFirestore instead of manually mocking Firestore. Replace `jest.mock("firebase-admin", () => ({ firestore: () => ({ /* mock */ }) }))` with `import { mockFirestore } from "__test-utils__/mockFirestore"; jest.mock("firebase-admin", () => mockFirestore)`.',
+            noMockFirebase: 'Use mockFirestore from __test-utils__/mockFirestore instead of mockFirebase. Replace `import { mockFirebase } from "firestore-jest-mock"` with `import { mockFirestore } from "__test-utils__/mockFirestore"`.',
         },
     },
     defaultOptions: [],

package/lib/rules/enforce-render-hits-memoization.js

@@ -35,7 +35,27 @@ exports.enforceRenderHitsMemoization = (0, createRule_1.createRule)({
             const name = node.name;
             return /^[A-Z]/.test(name);
         };
+        const isMemoizedVariable = (node) => {
+            if (node.type !== utils_1.AST_NODE_TYPES.Identifier)
+                return false;
+            // Get the variable declaration for this identifier
+            const variable = context.getScope().variables.find(v => v.name === node.name);
+            if (!variable)
+                return false;
+            // Check if the variable is initialized with a memoized call
+            const def = variable.defs[0];
+            if (!def || !def.node)
+                return false;
+            if (def.node.type === utils_1.AST_NODE_TYPES.VariableDeclarator && def.node.init) {
+                return isMemoizedCall(def.node.init);
+            }
+            return false;
+        };
         const isInsideMemoizedCall = (node) => {
+            // Check if the node is a reference to a memoized variable
+            if (isMemoizedVariable(node))
+                return true;
+            // Check if the node is inside a memoization hook call
             let current = node;
             while (current?.parent) {
                 if (isMemoizedCall(current.parent))

package/lib/rules/enforce-verb-noun-naming.js

@@ -10,6 +10,7 @@ const compromise_1 = __importDefault(require("compromise"));
 const PREPOSITIONS = new Set(['to', 'from', 'with', 'by', 'at', 'of']);
 // Create a Set from the verbs list for O(1) lookup
 const VERBS_SET = new Set([
+    'clean',
     'abbreviate',
     'abort',
     'abstract',

package/lib/rules/no-firestore-jest-mock.js

@@ -32,7 +32,7 @@ exports.noFirestoreJestMock = (0, createRule_1.createRule)({
                         messageId: 'noFirestoreJestMock',
                         fix(fixer) {
                             // Replace with mockFirestore import
-                            return fixer.replaceText(node, "import { mockFirestore } from '../../../../../__mocks__/functions/src/config/mockFirestore';");
+                            return fixer.replaceText(node, "import { mockFirestore } from '../../../../../__test-utils__/mockFirestore';");
                         },
                     });
                 }

package/lib/rules/no-mock-firebase-admin.js

@@ -19,7 +19,7 @@ exports.noMockFirebaseAdmin = (0, createRule_1.createRule)({
         },
         schema: [],
         messages: {
-            noMockFirebaseAdmin: 'Do not mock firebaseAdmin directly. Use mockFirestore from __mocks__/functions/src/config/mockFirestore instead.',
+            noMockFirebaseAdmin: 'Do not mock firebaseAdmin directly. Use mockFirestore from __test-utils__/mockFirestore instead.',
         },
     },
     defaultOptions: [],

package/lib/rules/no-unused-props.js

@@ -36,7 +36,53 @@ exports.noUnusedProps = (0, createRule_1.createRule)({
                             });
                         }
                         else if (typeNode.type === utils_1.AST_NODE_TYPES.TSIntersectionType) {
-                            typeNode.types.forEach(extractProps);
+                            typeNode.types.forEach((type) => {
+                                if (type.type === utils_1.AST_NODE_TYPES.TSTypeReference) {
+                                    const typeName = type.typeName;
+                                    if (typeName.type === utils_1.AST_NODE_TYPES.Identifier) {
+                                        if (typeName.name === 'Pick' && type.typeParameters) {
+                                            // Handle Pick utility type in intersection
+                                            const [baseType, pickedProps] = type.typeParameters.params;
+                                            if (baseType.type === utils_1.AST_NODE_TYPES.TSTypeReference &&
+                                                baseType.typeName.type === utils_1.AST_NODE_TYPES.Identifier) {
+                                                // Extract the picked properties from the union type
+                                                if (pickedProps.type === utils_1.AST_NODE_TYPES.TSUnionType) {
+                                                    pickedProps.types.forEach((t) => {
+                                                        if (t.type === utils_1.AST_NODE_TYPES.TSLiteralType &&
+                                                            t.literal.type === utils_1.AST_NODE_TYPES.Literal &&
+                                                            typeof t.literal.value === 'string') {
+                                                            // Add each picked property as a regular prop
+                                                            props[t.literal.value] = t.literal;
+                                                        }
+                                                    });
+                                                }
+                                                else if (pickedProps.type === utils_1.AST_NODE_TYPES.TSLiteralType &&
+                                                    pickedProps.literal.type === utils_1.AST_NODE_TYPES.Literal &&
+                                                    typeof pickedProps.literal.value === 'string') {
+                                                    // Single property pick
+                                                    props[pickedProps.literal.value] = pickedProps.literal;
+                                                }
+                                            }
+                                        }
+                                        else {
+                                            // For referenced types in intersections, we need to find their type declaration
+                                            const scope = context.getScope();
+                                            const variable = scope.variables.find(v => v.name === typeName.name);
+                                            if (variable && variable.defs[0]?.node.type === utils_1.AST_NODE_TYPES.TSTypeAliasDeclaration) {
+                                                extractProps(variable.defs[0].node.typeAnnotation);
+                                            }
+                                            else {
+                                                // If we can't find the type declaration, it's likely an imported type
+                                                // Mark it as a forwarded prop
+                                                props[`...${typeName.name}`] = typeName;
+                                            }
+                                        }
+                                    }
+                                }
+                                else {
+                                    extractProps(type);
+                                }
+                            });
                         }
                         else if (typeNode.type === utils_1.AST_NODE_TYPES.TSTypeReference) {
                             if (typeNode.typeName.type === utils_1.AST_NODE_TYPES.Identifier) {

package/package.json

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