@codfish/eslint-config 0.0.0-PR-132--78c4c74 → 0.0.0-semantically-released

package/dist/tests/integration/eslint-execution.spec.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=eslint-execution.spec.d.ts.map

package/dist/tests/integration/eslint-execution.spec.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"eslint-execution.spec.d.ts","sourceRoot":"","sources":["../../../tests/integration/eslint-execution.spec.js"],"names":[],"mappings":""}

package/dist/tests/integration/rule-application.spec.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=rule-application.spec.d.ts.map

package/dist/tests/integration/rule-application.spec.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"rule-application.spec.d.ts","sourceRoot":"","sources":["../../../tests/integration/rule-application.spec.js"],"names":[],"mappings":""}

package/dist/tests/scenarios/edge-cases.spec.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=edge-cases.spec.d.ts.map

package/dist/tests/scenarios/edge-cases.spec.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"edge-cases.spec.d.ts","sourceRoot":"","sources":["../../../tests/scenarios/edge-cases.spec.js"],"names":[],"mappings":""}

package/dist/tests/unit/config-loading.spec.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=config-loading.spec.d.ts.map

package/dist/tests/unit/config-loading.spec.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"config-loading.spec.d.ts","sourceRoot":"","sources":["../../../tests/unit/config-loading.spec.js"],"names":[],"mappings":""}

package/dist/tests/unit/dynamic-detection.spec.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=dynamic-detection.spec.d.ts.map

package/dist/tests/unit/dynamic-detection.spec.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"dynamic-detection.spec.d.ts","sourceRoot":"","sources":["../../../tests/unit/dynamic-detection.spec.js"],"names":[],"mappings":""}

package/dist/tests/unit/utils.spec.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=utils.spec.d.ts.map

package/dist/tests/unit/utils.spec.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"utils.spec.d.ts","sourceRoot":"","sources":["../../../tests/unit/utils.spec.js"],"names":[],"mappings":""}

package/index.js

@@ -4,7 +4,7 @@ import markdown from '@eslint/markdown';
 import html from '@html-eslint/eslint-plugin';
 import htmlParser from '@html-eslint/parser';
 import { defineConfig } from 'eslint/config';
-import prettierPlugin from 'eslint-plugin-prettier';
+import prettierConfig from 'eslint-plugin-prettier/recommended';
 import simpleImportSortPlugin from 'eslint-plugin-simple-import-sort';
 import ymlPlugin from 'eslint-plugin-yml';
 import globals from 'globals';
@@ -294,7 +294,7 @@ export default defineConfig([
       // Allow GitHub-style alerts and checkbox syntax (task lists)
       'markdown/no-missing-label-refs': [
         'error',
-        { allowLabels: ['!NOTE', '!WARNING', '!IMPORTANT', '!TIP', '', 'x', 'X'] },
+        { allowLabels: ['!NOTE', '!WARNING', '!IMPORTANT', '!TIP', '!CAUTION', '', 'x', 'X'] },
       ],
     },
   },
@@ -313,9 +313,7 @@ export default defineConfig([
   {
     files: ['**/*.{js,ts,jsx,tsx,md,yml,yaml,html,json,jsonc,json5}'],
 
-    plugins: {
-      prettier: prettierPlugin,
-    },
+    extends: [prettierConfig],
 
     rules: {
       // Reset prettier rule passing in custom prettier config.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codfish/eslint-config",
-  "version": "0.0.0-PR-132--78c4c74",
+  "version": "0.0.0-semantically-released",
   "description": "Modern ESLint configuration with TypeScript, React, and testing framework support.",
   "type": "module",
   "repository": {
@@ -45,7 +45,7 @@
   "scripts": {
     "build": "tsc --emitDeclarationOnly",
     "dev": "tsc --watch",
-    "type-check": "tsc --noEmit",
+    "typecheck": "tsc --noEmit",
     "lint": "eslint",
     "format": "eslint --fix",
     "test": "vitest",
@@ -82,6 +82,7 @@
     "typescript-eslint": "^8.44.0"
   },
   "devDependencies": {
+    "@vitest/coverage-v8": "^4.0.18",
     "doctoc": "^2.2.1",
     "eslint": "^9.35.0",
     "husky": "^9.1.7",

package/rules/react.js

@@ -16,50 +16,69 @@ import { ifAnyDep } from '../utils.js';
  * - https://github.com/facebook/react/tree/main/packages/eslint-plugin-react-hooks
  * - https://github.com/jsx-eslint/eslint-plugin-jsx-a11y
  */
-export default defineConfig({
-  name: 'codfish/react',
+export default defineConfig([
+  {
+    name: 'codfish/react',
 
-  files: ['**/*.jsx', '**/*.tsx', '**/*.mjsx', '**/*.mtsx'],
+    files: ['**/*.jsx', '**/*.tsx', '**/*.mjsx', '**/*.mtsx'],
 
-  extends: [
-    react.configs.flat.recommended,
-    reactHooks.configs.flat['recommended-latest'],
-    reactRefresh.configs.recommended,
-    jsxA11y.flatConfigs.recommended,
-    ifAnyDep('@tanstack/react-query', tanstackQuery.configs['flat/recommended']),
-    ifAnyDep('next', nextPlugin.configs.recommended),
-    ifAnyDep('next', nextPlugin.configs['core-web-vitals']),
-  ].filter(Boolean),
+    extends: [
+      react.configs.flat.recommended,
+      reactHooks.configs.flat['recommended-latest'],
+      reactRefresh.configs.recommended,
+      jsxA11y.flatConfigs.recommended,
+      ifAnyDep('@tanstack/react-query', tanstackQuery.configs['flat/recommended']),
+      ifAnyDep('next', nextPlugin.configs.recommended),
+      ifAnyDep('next', nextPlugin.configs['core-web-vitals']),
+    ].filter(Boolean),
 
-  languageOptions: {
-    globals: {
-      ...globals.serviceworker,
-      ...globals.browser,
+    languageOptions: {
+      globals: {
+        ...globals.serviceworker,
+        ...globals.browser,
+      },
     },
-  },
 
-  settings: {
-    react: {
-      version: 'detect',
+    settings: {
+      react: {
+        version: 'detect',
+      },
     },
-  },
 
-  rules: {
-    // React 17+ uses automatic JSX runtime, no need to import React
-    'react/react-in-jsx-scope': 'off',
-    'react/jsx-uses-react': 'off',
+    rules: {
+      // React 17+ uses automatic JSX runtime, no need to import React
+      'react/react-in-jsx-scope': 'off',
+      'react/jsx-uses-react': 'off',
 
-    // TypeScript provides type checking, no need for PropTypes
-    'react/prop-types': 'off',
+      // TypeScript provides type checking, no need for PropTypes
+      'react/prop-types': 'off',
 
-    // this rule is too noisey and doesn"t account for certain valid use cases
-    'react-hooks/exhaustive-deps': 'off',
+      // this rule is too noisey and doesn't account for certain valid use cases
+      'react-hooks/exhaustive-deps': 'off',
 
-    // sometimes autoFocus is needed for better UX. @todo - revisit later?
-    'jsx-a11y/no-autofocus': 'off',
+      // sometimes autoFocus is needed for better UX. @todo - revisit later?
+      'jsx-a11y/no-autofocus': 'off',
 
-    'react-refresh/only-export-components': ['warn', { allowConstantExport: true }],
+      'react-refresh/only-export-components': ['warn', { allowConstantExport: true }],
 
-    ...ifAnyDep('@tanstack/react-query', { '@tanstack/query/exhaustive-deps': 'error' }, {}),
+      ...ifAnyDep('@tanstack/react-query', { '@tanstack/query/exhaustive-deps': 'error' }, {}),
+
+      ...ifAnyDep(
+        'next',
+        {
+          // annoying cause this is common in Next.js, i.e. generateMetadata()
+          'react-refresh/only-export-components': ['off'],
+        },
+        {},
+      ),
+    },
+  },
+
+  {
+    name: 'codfish/react-next-config',
+    files: ['next.config.js', 'next.config.ts'],
+    rules: {
+      'require-await': 'off',
+    },
   },
-});
+]);

package/rules/tests.js

@@ -30,8 +30,12 @@ export default defineConfig({
   extends: [
     ifAnyDep('jest', jest.configs['flat/recommended'], false),
     ifAnyDep('vitest', vitest.configs.recommended, false),
-    ifAnyDep('react-testing-library', testingLibrary.configs['flat/react'], false),
-    ifAnyDep('vue-testing-library', testingLibrary.configs['flat/vue'], false),
+    ifAnyDep('@testing-library/dom', testingLibrary.configs['flat/dom'], false),
+    ifAnyDep('@testing-library/react', testingLibrary.configs['flat/react'], false),
+    ifAnyDep('@testing-library/vue', testingLibrary.configs['flat/vue'], false),
+    ifAnyDep('@testing-library/svelte', testingLibrary.configs['flat/svelte'], false),
+    ifAnyDep('@testing-library/angular', testingLibrary.configs['flat/angular'], false),
+    ifAnyDep('@testing-library/marko', testingLibrary.configs['flat/marko'], false),
   ].filter(Boolean),
 
   languageOptions: {