eslint-plugin-code-style 1.6.1 → 1.6.4

package/CHANGELOG.md

@@ -7,6 +7,30 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ---
 
+## [1.6.4] - 2026-02-01
+
+### Enhanced
+
+- **`function-object-destructure`** - Add more module paths: apis, configs, utilities, routes
+
+---
+
+## [1.6.3] - 2026-02-01
+
+### Fixed
+
+- **`component-props-destructure`** - Preserve TypeScript type annotation when auto-fixing
+
+---
+
+## [1.6.2] - 2026-02-01
+
+### Enhanced
+
+- **`function-object-destructure`** - Expand to check more module paths (services, constants, config, api, utils, helpers, lib) for dot notation enforcement
+
+---
+
 ## [1.6.1] - 2026-02-01
 
 ### Enhanced
@@ -987,6 +1011,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ---
 
+[1.6.4]: https://github.com/Mohamed-Elhawary/eslint-plugin-code-style/compare/v1.6.3...v1.6.4
+[1.6.3]: https://github.com/Mohamed-Elhawary/eslint-plugin-code-style/compare/v1.6.2...v1.6.3
+[1.6.2]: https://github.com/Mohamed-Elhawary/eslint-plugin-code-style/compare/v1.6.1...v1.6.2
 [1.6.1]: https://github.com/Mohamed-Elhawary/eslint-plugin-code-style/compare/v1.6.0...v1.6.1
 [1.6.0]: https://github.com/Mohamed-Elhawary/eslint-plugin-code-style/compare/v1.5.2...v1.6.0
 [1.5.2]: https://github.com/Mohamed-Elhawary/eslint-plugin-code-style/compare/v1.5.1...v1.5.2

package/index.js

@@ -12414,15 +12414,32 @@ const functionObjectDestructure = {
     create(context) {
         const sourceCode = context.sourceCode || context.getSourceCode();
 
-        // Track imports from data-related paths (should use dot notation, not destructure)
-        const dataImports = new Set();
-
-        const isDataImportPath = (importPath) => {
-            // Match paths like @/data, ./data, ../data, or any path containing /data/
-            return importPath === "@/data"
-                || importPath.endsWith("/data")
-                || importPath.includes("/data/")
-                || /^\.\.?\/data$/.test(importPath);
+        // Track imports from module paths that should use dot notation, not destructure
+        // This improves searchability: api.loginHandler is easier to find than loginHandler
+        const moduleImports = new Set();
+
+        // Folders that contain modules which should be accessed via dot notation
+        const modulePathPatterns = [
+            "api",
+            "apis",
+            "config",
+            "configs",
+            "constants",
+            "data",
+            "helpers",
+            "lib",
+            "routes",
+            "services",
+            "utils",
+            "utilities",
+        ];
+
+        const isModuleImportPath = (importPath) => {
+            // Match paths like @/services, @/constants, ./data, ../config, etc.
+            return modulePathPatterns.some((pattern) => importPath === `@/${pattern}`
+                    || importPath.endsWith(`/${pattern}`)
+                    || importPath.includes(`/${pattern}/`)
+                    || new RegExp(`^\\.?\\.?/${pattern}$`).test(importPath));
         };
 
         const checkImportHandler = (node) => {
@@ -12430,13 +12447,13 @@ const functionObjectDestructure = {
 
             const importPath = node.source.value;
 
-            if (isDataImportPath(importPath)) {
-                // Track all imported specifiers from data paths
+            if (isModuleImportPath(importPath)) {
+                // Track all imported specifiers from module paths
                 node.specifiers.forEach((spec) => {
                     if (spec.type === "ImportSpecifier" && spec.local && spec.local.name) {
-                        dataImports.add(spec.local.name);
+                        moduleImports.add(spec.local.name);
                     } else if (spec.type === "ImportDefaultSpecifier" && spec.local && spec.local.name) {
-                        dataImports.add(spec.local.name);
+                        moduleImports.add(spec.local.name);
                     }
                 });
             }
@@ -12467,7 +12484,7 @@ const functionObjectDestructure = {
                         }
                     }
 
-                    if (sourceVarName && dataImports.has(sourceVarName)) {
+                    if (sourceVarName && moduleImports.has(sourceVarName)) {
                         const destructuredProps = decl.id.properties
                             .filter((p) => p.type === "Property" && p.key && p.key.name)
                             .map((p) => p.key.name);
@@ -12475,7 +12492,7 @@ const functionObjectDestructure = {
                         const sourceText = sourceCode.getText(decl.init);
 
                         context.report({
-                            message: `Do not destructure data imports. Use dot notation for searchability: "${sourceText}.${destructuredProps[0]}" instead of destructuring`,
+                            message: `Do not destructure module imports. Use dot notation for searchability: "${sourceText}.${destructuredProps[0]}" instead of destructuring`,
                             node: decl.id,
                         });
                     }
@@ -13301,8 +13318,19 @@ const componentPropsDestructure = {
                         ? (fixer) => {
                             const fixes = [];
 
+                            // Build destructured pattern, preserving type annotation if present
+                            const destructuredPattern = `{ ${accessedProps.join(", ")} }`;
+                            let replacement = destructuredPattern;
+
+                            // Preserve TypeScript type annotation if present
+                            if (firstParam.typeAnnotation) {
+                                const typeText = sourceCode.getText(firstParam.typeAnnotation);
+
+                                replacement = `${destructuredPattern}${typeText}`;
+                            }
+
                             // Replace param with destructured pattern
-                            fixes.push(fixer.replaceText(firstParam, `{ ${accessedProps.join(", ")} }`));
+                            fixes.push(fixer.replaceText(firstParam, replacement));
 
                             // Replace all props.x with just x
                             accesses.forEach((access) => {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "eslint-plugin-code-style",
-    "version": "1.6.1",
+    "version": "1.6.4",
     "description": "A custom ESLint plugin for enforcing consistent code formatting and style rules in React/JSX projects",
     "main": "index.js",
     "types": "index.d.ts",