@wkovacs64/eslint-config 7.2.1 → 7.2.3

package/index.js

@@ -2,6 +2,7 @@
 //
 // Forked from https://github.com/epicweb-dev/config
 //
+import resolveFrom from 'resolve-from';
 import globals from 'globals';
 import eslint from '@eslint/js';
 import tseslint from 'typescript-eslint';
@@ -13,18 +14,16 @@ const ERROR = 'error';
 const WARN = 'warn';
 const OFF = 'off';
 
-const has = (pkg) =>
-  import(pkg).then(
-    () => true,
-    () => false,
-  );
+function has(pkgName) {
+  return Boolean(resolveFrom.silent(process.cwd(), pkgName));
+}
 
-const hasTypeScript = await has('typescript');
-const hasReact = await has('react');
-const hasTestingLibrary = await has('@testing-library/dom');
-const hasJestDom = await has('@testing-library/jest-dom');
-const hasVitest = await has('vitest');
-const hasPlaywright = await has('playwright');
+const hasTypeScript = has('typescript');
+const hasReact = has('react');
+const hasTestingLibrary = has('@testing-library/dom');
+const hasJestDom = has('@testing-library/jest-dom');
+const hasVitest = has('vitest');
+const hasPlaywright = has('@playwright/test');
 
 const vitestFiles = ['**/__tests__/**/*', '**/*.test.*'];
 const testFiles = ['**/tests/**', '**/#tests/**', ...vitestFiles];
@@ -160,108 +159,106 @@ export const config = [
         // TODO: figure out how to switch to type-checked configs
         ...tseslint.configs.recommended,
         ...tseslint.configs.stylistic,
-      ]
-    : []),
-  hasTypeScript
-    ? {
-        files: ['**/*.ts?(x)'],
-        languageOptions: {
-          parserOptions: {
-            parser: tseslint.parser,
-            projectService: true,
-          },
-        },
-        plugins: {
-          '@typescript-eslint': tseslint.plugin,
-        },
-        rules: {
-          '@typescript-eslint/ban-ts-comment': OFF,
-          '@typescript-eslint/consistent-type-assertions': [
-            ERROR,
-            {
-              assertionStyle: 'as',
-              objectLiteralTypeAssertions: 'allow-as-parameter',
-            },
-          ],
-          '@typescript-eslint/consistent-type-definitions': OFF,
-          // Note: use this rule _OR_ verbatimModuleSyntax in tsconfig.json - not both
-          // '@typescript-eslint/consistent-type-imports': [
-          //   ERROR,
-          //   {
-          //     prefer: 'type-imports',
-          //     disallowTypeAnnotations: true,
-          //     fixStyle: 'inline-type-imports',
-          //   },
-          // ],
-          '@typescript-eslint/explicit-module-boundary-types': OFF,
-          '@typescript-eslint/naming-convention': [
-            ERROR,
-            {
-              selector: 'typeLike',
-              format: ['PascalCase'],
-              custom: { regex: '^I[A-Z]', match: false },
-            },
-          ],
-          '@typescript-eslint/no-confusing-void-expression': [
-            ERROR,
-            {
-              ignoreArrowShorthand: true,
-            },
-          ],
-          '@typescript-eslint/no-explicit-any': OFF,
-          '@typescript-eslint/no-floating-promises': [
-            ERROR,
-            {
-              ignoreIIFE: true,
-            },
-          ],
-          '@typescript-eslint/no-import-type-side-effects': ERROR,
-          'no-invalid-this': OFF,
-          '@typescript-eslint/no-invalid-this': ERROR,
-          'no-redeclare': OFF,
-          '@typescript-eslint/no-non-null-assertion': OFF,
-          '@typescript-eslint/no-redeclare': ERROR,
-          'no-shadow': OFF,
-          '@typescript-eslint/no-shadow': ERROR,
-          '@typescript-eslint/no-unnecessary-type-constraint': 'warn',
-          '@typescript-eslint/no-unused-vars': [
-            ERROR,
-            {
-              vars: 'all',
-              args: 'after-used',
-              argsIgnorePattern: '^_',
-              ignoreRestSiblings: true,
-            },
-          ],
-          'no-use-before-define': OFF,
-          '@typescript-eslint/no-use-before-define': [
-            ERROR,
-            {
-              functions: false,
-              classes: true,
-              variables: true,
-            },
-          ],
-          '@typescript-eslint/prefer-nullish-coalescing': OFF,
-          '@typescript-eslint/restrict-template-expressions': [
-            ERROR,
-            {
-              allowBoolean: true,
-              allowNullish: true,
+        {
+          files: ['**/*.ts?(x)'],
+          languageOptions: {
+            parserOptions: {
+              parser: tseslint.parser,
+              projectService: true,
             },
-          ],
-          '@typescript-eslint/require-await': OFF,
-          '@typescript-eslint/unified-signatures': 'warn',
-        },
-        settings: {
-          'import/resolver': {
-            typescript: {
-              alwaysTryTypes: true,
+          },
+          plugins: {
+            '@typescript-eslint': tseslint.plugin,
+          },
+          rules: {
+            '@typescript-eslint/ban-ts-comment': OFF,
+            '@typescript-eslint/consistent-type-assertions': [
+              ERROR,
+              {
+                assertionStyle: 'as',
+                objectLiteralTypeAssertions: 'allow-as-parameter',
+              },
+            ],
+            '@typescript-eslint/consistent-type-definitions': OFF,
+            // Note: use this rule _OR_ verbatimModuleSyntax in tsconfig.json - not both
+            // '@typescript-eslint/consistent-type-imports': [
+            //   ERROR,
+            //   {
+            //     prefer: 'type-imports',
+            //     disallowTypeAnnotations: true,
+            //     fixStyle: 'inline-type-imports',
+            //   },
+            // ],
+            '@typescript-eslint/explicit-module-boundary-types': OFF,
+            '@typescript-eslint/naming-convention': [
+              ERROR,
+              {
+                selector: 'typeLike',
+                format: ['PascalCase'],
+                custom: { regex: '^I[A-Z]', match: false },
+              },
+            ],
+            '@typescript-eslint/no-confusing-void-expression': [
+              ERROR,
+              {
+                ignoreArrowShorthand: true,
+              },
+            ],
+            '@typescript-eslint/no-explicit-any': OFF,
+            '@typescript-eslint/no-floating-promises': [
+              ERROR,
+              {
+                ignoreIIFE: true,
+              },
+            ],
+            '@typescript-eslint/no-import-type-side-effects': ERROR,
+            'no-invalid-this': OFF,
+            '@typescript-eslint/no-invalid-this': ERROR,
+            'no-redeclare': OFF,
+            '@typescript-eslint/no-non-null-assertion': OFF,
+            '@typescript-eslint/no-redeclare': ERROR,
+            'no-shadow': OFF,
+            '@typescript-eslint/no-shadow': ERROR,
+            '@typescript-eslint/no-unnecessary-type-constraint': 'warn',
+            '@typescript-eslint/no-unused-vars': [
+              ERROR,
+              {
+                vars: 'all',
+                args: 'after-used',
+                argsIgnorePattern: '^_',
+                ignoreRestSiblings: true,
+              },
+            ],
+            'no-use-before-define': OFF,
+            '@typescript-eslint/no-use-before-define': [
+              ERROR,
+              {
+                functions: false,
+                classes: true,
+                variables: true,
+              },
+            ],
+            '@typescript-eslint/prefer-nullish-coalescing': OFF,
+            '@typescript-eslint/restrict-template-expressions': [
+              ERROR,
+              {
+                allowBoolean: true,
+                allowNullish: true,
+              },
+            ],
+            '@typescript-eslint/require-await': OFF,
+            '@typescript-eslint/unified-signatures': 'warn',
+          },
+          settings: {
+            'import/resolver': {
+              typescript: {
+                alwaysTryTypes: true,
+              },
             },
           },
         },
-      }
-    : null,
+      ]
+    : []),
 
   // This assumes test files are those which are in the test directory or have
   // *.test.* in the filename. If a file doesn't match this assumption, then it

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wkovacs64/eslint-config",
-  "version": "7.2.1",
+  "version": "7.2.3",
   "description": "@wKovacs64 ESLint config",
   "keywords": [
     "eslint",
@@ -60,18 +60,19 @@
     "eslint-plugin-testing-library": "^6.2.2",
     "eslint-plugin-vitest": "^0.5.4",
     "globals": "^15.3.0",
+    "resolve-from": "5.0.0",
     "typescript-eslint": "^8.0.0-alpha.20"
   },
   "devDependencies": {
     "@changesets/changelog-github": "0.5.0",
     "@changesets/cli": "2.27.5",
     "@types/eslint__js": "8.42.3",
-    "@types/node": "20.14.1",
+    "@types/node": "20.14.2",
     "@types/react": "18.3.3",
     "@types/react-dom": "18.3.0",
     "@wkovacs64/prettier-config": "4.1.0",
     "eslint": "8.57.0",
-    "prettier": "3.3.0",
+    "prettier": "3.3.1",
     "typescript": "5.4.5"
   }
 }