fix: Refine eslint import plugin rules to accommodate .jsx and .cjs

This change adds ".jsx" and ".cjs" to the list of recognized file extensions in the eslint import resolver. This allows better support and error handling for these newly accepted file types. Furthermore, the rule "import/no-commonjs" has been fine-tuned to allow primitive modules, but discourage non-primitive 'require' statements, providing a clearer distinction between anti-patterns and necessary exceptions. "import/no-unused-modules" rule has been edited to prevent it from considering ".cjs" files as unused modules.

Signed-off-by: prisis <[email protected]>

Author: prisis

packages/eslint-config/src/config/plugins/import.ts

@@ -117,7 +117,7 @@ const config: Linter.Config = createConfigs([
 
                 // disallow require()
                 // https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-commonjs.md
-                "import/no-commonjs": packageIsTypeModule ? "error" : "off",
+                "import/no-commonjs": packageIsTypeModule ? ["error", { allowPrimitiveModules: true }] : "off",
 
                 // Forbid cyclical dependencies between modules
                 // https://medium.com/@steven-lemon182/are-typescript-barrel-files-an-anti-pattern-72a713004250
@@ -254,7 +254,7 @@ const config: Linter.Config = createConfigs([
                 "import/no-unused-modules": [
                     packageIsTypeModule ? "error" : "off",
                     {
-                        ignoreExports: [],
+                        ignoreExports: ["**/*.cjs"],
                         missingExports: true,
                         unusedExports: true,
                     },
@@ -305,7 +305,7 @@ const config: Linter.Config = createConfigs([
                 "import/ignore": ["\\.(coffee|scss|css|less|hbs|svg|json)$"],
                 "import/resolver": {
                     node: {
-                        extensions: [".mjs", ".js", ".json"],
+                        extensions: [".mjs", ".js", ".json", ".cjs", ".jsx"],
                     },
                     ...(hasTypescript
                         ? {