@viclafouch/eslint-config-viclafouch 4.22.1-beta.1 → 4.22.1-beta.2

package/.claude/settings.local.json

@@ -0,0 +1,16 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(npm run lint)",
+      "Bash(npm outdated:*)",
+      "WebFetch(domain:github.com)",
+      "WebFetch(domain:typescript-eslint.io)",
+      "WebFetch(domain:www.npmjs.com)",
+      "Bash(npm install:*)",
+      "Bash(npm view:*)",
+      "Bash(node -e:*)",
+      "Bash(git add:*)",
+      "Bash(git commit -m \"$\\(cat <<''EOF''\nfeat: update dependencies and refactor next config to flat config\n\n- Update all dependencies to latest versions\n- Refactor next.mjs to use nextPlugin.configs.recommended directly\n- Remove @eslint/eslintrc FlatCompat dependency\n- Reorganize exports alphabetically in index.mjs\n- Add ignores for *.d.ts files in eslint.config.mjs\n- Add coding conventions section to CLAUDE.md\n\nCo-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>\nEOF\n\\)\")"
+    ]
+  }
+}

package/CLAUDE.md

@@ -0,0 +1,165 @@
+# CLAUDE.md
+
+This file documents the **@viclafouch/eslint-config-viclafouch** project to facilitate working with Claude Code.
+
+## What is this project?
+
+This is a reusable ESLint and Prettier configuration package published on npm. It provides a comprehensive, modular set of linting and formatting rules designed for modern JavaScript/TypeScript projects. The goal is to share consistent code quality standards across all of Victor de la Fouchardiere's projects.
+
+**Key features:**
+- JavaScript ES6+, TypeScript, JSX/TSX support
+- React.js and Next.js frameworks
+- Tailwind CSS v4 linting
+- Testing libraries support
+- Promise handling and best practices
+
+## File Structure
+
+### Entry Files
+
+| File | Description |
+|------|-------------|
+| `index.mjs` | Main entry point, exports all configuration modules |
+| `index.d.ts` | TypeScript type definitions for the package |
+| `eslint.config.mjs` | ESLint configuration for this package itself (uses baseConfig + prettierConfig) |
+
+### Configuration Modules (root)
+
+| File | Description |
+|------|-------------|
+| `base.mjs` | Core JavaScript/TypeScript linting rules (imports all `/rules/` files) |
+| `typescript.mjs` | TypeScript-specific rules with parser and project service |
+| `react.mjs` | React library rules and JSX accessibility (jsx-a11y) |
+| `next.mjs` | Next.js-specific rules (extends react + hooks + Next.js plugin) |
+| `hooks.mjs` | React Hooks rules (rules-of-hooks, exhaustive-deps, useState naming) |
+| `imports.mjs` | Import sorting with simple-import-sort in priority groups |
+| `prettier.mjs` | Prettier integration via eslint-plugin-prettier |
+| `better-tailwindcss.mjs` | Tailwind CSS v4 linting (function accepting `{ entryPoint }`) |
+
+### Utility Files for Other Projects
+
+| File | Description |
+|------|-------------|
+| `tsconfig.json` | Shareable TypeScript configuration, can be extended in other projects |
+| `reset.d.ts` | Re-exports improved typings from @total-typescript/ts-reset |
+
+## NPM Scripts
+
+```bash
+npm run lint          # Lint the entire project
+npm run lint:fix      # Lint and auto-fix issues
+npm run bump          # Increment minor version
+npm run publish       # Publish to npm
+npm run bump:beta     # Increment beta version
+npm run publish:beta  # Publish beta version
+```
+
+## Configuration Architecture
+
+### Flat Config Pattern (ESLint 9+)
+
+Each module exports an array of configurations that can be spread:
+
+```javascript
+// Minimal setup
+import { baseConfig } from '@viclafouch/eslint-config-viclafouch'
+export default [...baseConfig]
+
+// Full Next.js setup
+import { baseConfig, typescriptConfig, nextConfig, prettierConfig } from '@viclafouch/eslint-config-viclafouch'
+export default [...baseConfig, ...typescriptConfig, ...nextConfig, ...prettierConfig]
+```
+
+### Loading Hierarchy
+
+1. **baseConfig** → loads 7 rule modules + Unicorn plugin
+2. **typescriptConfig** → replaces some rules with TypeScript versions
+3. **reactConfig** → adds JSX/a11y rules
+4. **nextConfig** → extends react + hooks + Next.js plugin
+5. **importsConfig** → organizes import sorting
+6. **prettierConfig** → formatting (must be last)
+7. **betterTailwindcssConfig({ entryPoint })** → Tailwind CSS v4
+
+## Usage in Other Projects
+
+### Extending tsconfig.json
+
+```json
+{
+  "extends": "@viclafouch/eslint-config-viclafouch/tsconfig.json",
+  "compilerOptions": {
+    // Project-specific overrides
+  }
+}
+```
+
+### Using reset.d.ts
+
+The `reset.d.ts` file re-exports improved types from `@total-typescript/ts-reset`, providing:
+
+- Better inference for `.filter(Boolean)`
+- Stricter types for `JSON.parse`
+- Improved `Array.includes()`
+
+To use it, reference it in your project's tsconfig.json or include it in your types.
+
+## Integrated Plugins
+
+- `@eslint/js` - Core ESLint rules
+- `typescript-eslint` - TypeScript linting
+- `eslint-plugin-react` - React rules
+- `eslint-plugin-react-hooks` - Hooks enforcement
+- `eslint-plugin-jsx-a11y` - Accessibility
+- `eslint-plugin-import` + `simple-import-sort` - Import ordering
+- `eslint-plugin-promise` - Promise handling
+- `eslint-plugin-unicorn` - Best practices
+- `eslint-plugin-prettier` - Code formatting
+- `eslint-plugin-better-tailwindcss` - Tailwind v4
+- `@next/eslint-plugin-next` - Next.js specific
+
+## Peer Dependencies
+
+- ESLint >= 9
+- Prettier >= 3
+- TypeScript >= 5
+
+## Coding Conventions
+
+### Rule Severity
+
+**IMPORTANT**: This project follows a strict binary approach for rule severity:
+- Use `'error'` for rules that should be enforced
+- Use `'off'` for rules that should be disabled
+- **NEVER use `'warn'`** - warnings are not acceptable in this configuration
+
+This ensures that linting is either passing or failing, with no ambiguous middle ground.
+
+### Rule Comment Format
+
+**IMPORTANT**: Every rule must be documented with a comment following this exact format:
+
+```javascript
+// Description of what the rule does
+// https://eslint.org/docs/rules/rule-name
+'rule-name': 'error',
+```
+
+For TypeScript rules, use the appropriate documentation URL:
+
+```javascript
+// Description of what the rule does
+// https://typescript-eslint.io/rules/rule-name
+'@typescript-eslint/rule-name': 'error',
+```
+
+**Format requirements:**
+1. **Line 1**: A description of what the rule enforces/disallows (starts with a verb like "Enforce", "Disallow", "Require", etc.)
+2. **Line 2**: The URL to the rule's official documentation
+3. **Line 3+**: The rule configuration itself
+
+**URL patterns by plugin:**
+- ESLint core: `https://eslint.org/docs/rules/rule-name` or `https://eslint.org/docs/latest/rules/rule-name`
+- TypeScript ESLint: `https://typescript-eslint.io/rules/rule-name`
+- React: `https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/rule-name.md`
+- React Hooks: `https://react.dev/reference/rules/rules-of-hooks`
+

package/eslint.config.mjs

@@ -4,6 +4,9 @@ import { baseConfig, prettierConfig } from './index.mjs'
  * @type {import("eslint").Linter.Config}
  */
 export default [
+  {
+    ignores: ['**/*.d.ts']
+  },
   ...baseConfig,
   ...prettierConfig,
   {

package/index.mjs

@@ -1,8 +1,8 @@
 export { default as baseConfig } from './base.mjs'
-export { default as typescriptConfig } from './typescript.mjs'
-export { default as reactConfig } from './react.mjs'
-export { default as nextConfig } from './next.mjs'
-export { default as prettierConfig } from './prettier.mjs'
+export { default as betterTailwindcssConfig } from './better-tailwindcss.mjs'
 export { default as hooksConfig } from './hooks.mjs'
 export { default as importsConfig } from './imports.mjs'
-export { default as betterTailwindcssConfig } from './better-tailwindcss.mjs'
+export { default as nextConfig } from './next.mjs'
+export { default as prettierConfig } from './prettier.mjs'
+export { default as reactConfig } from './react.mjs'
+export { default as typescriptConfig } from './typescript.mjs'

package/next.mjs

@@ -1,26 +1,18 @@
-import { FlatCompat } from '@eslint/eslintrc'
+import nextPlugin from '@next/eslint-plugin-next'
 import hooksConfig from './hooks.mjs'
 import reactConfig from './react.mjs'
 
-const compat = new FlatCompat({
-  baseDirectory: import.meta.dirname
-})
-
 /**
  * @type {import("eslint").Linter.Config}
  */
 export default [
   ...reactConfig,
   ...hooksConfig,
-  ...compat.config({
-    extends: ['plugin:@next/next/recommended'],
-    rules: {
-      'react/no-unescaped-entities': 'off',
-      '@next/next/no-page-custom-font': 'off'
-    }
-  }),
+  nextPlugin.configs.recommended,
   {
     rules: {
+      'react/no-unescaped-entities': 'off',
+      '@next/next/no-page-custom-font': 'off',
       // No error for anchor inside a Next Link
       'jsx-a11y/anchor-is-valid': [
         'error',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@viclafouch/eslint-config-viclafouch",
-  "version": "4.22.1-beta.1",
+  "version": "4.22.1-beta.2",
   "description": "ESLint and Prettier Config from Victor de la Fouchardiere",
   "type": "module",
   "main": "./index.mjs",
@@ -38,31 +38,31 @@
     "typescript": ">= 5"
   },
   "dependencies": {
-    "@babel/core": "^7.26.9",
-    "@babel/eslint-parser": "^7.26.8",
-    "@eslint/js": "^9.21.0",
-    "@next/eslint-plugin-next": "^15.2.1",
+    "@babel/core": "^7.28.6",
+    "@babel/eslint-parser": "^7.28.6",
+    "@eslint/js": "^9.39.2",
+    "@next/eslint-plugin-next": "^16.1.6",
     "@total-typescript/ts-reset": "^0.6.1",
     "app-root-path": "^3.1.0",
     "babel-loader": "^10.0.0",
     "eslint": ">= 9",
-    "eslint-config-prettier": "^10.1.1",
-    "eslint-plugin-better-tailwindcss": "^3.8.0",
-    "eslint-plugin-import": "^2.31.0",
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-plugin-better-tailwindcss": "^4.0.2",
+    "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-jsx-a11y": "^6.10.2",
-    "eslint-plugin-prettier": "^5.2.3",
+    "eslint-plugin-prettier": "^5.5.5",
     "eslint-plugin-promise": "^7.2.1",
-    "eslint-plugin-react": "^7.37.4",
-    "eslint-plugin-react-hooks": "^5.2.0",
+    "eslint-plugin-react": "^7.37.5",
+    "eslint-plugin-react-hooks": "^7.0.1",
     "eslint-plugin-simple-import-sort": "^12.1.1",
-    "eslint-plugin-testing-library": "^7.1.1",
-    "eslint-plugin-unicorn": "^58.0.0",
-    "get-tsconfig": "^4.10.0",
-    "globals": "^16.0.0",
+    "eslint-plugin-testing-library": "^7.15.4",
+    "eslint-plugin-unicorn": "^62.0.0",
+    "get-tsconfig": "^4.13.0",
+    "globals": "^17.2.0",
     "prettier": ">= 3",
-    "prettier-plugin-curly": "^0.3.1",
+    "prettier-plugin-curly": "^0.4.1",
     "typescript": ">= 5",
-    "typescript-eslint": "^8.26.0"
+    "typescript-eslint": "^8.54.0"
   },
   "scripts": {
     "lint": "eslint .",

package/rules/best-practices.mjs

@@ -388,6 +388,130 @@ export default {
 
     // Enforce the use of u flag on RegExp
     // https://eslint.org/docs/rules/require-unicode-regexp
-    'require-unicode-regexp': 'off'
+    'require-unicode-regexp': 'off',
+
+    // Disallow creating a variable and immediately mutating it
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-immediate-mutation.md
+    'unicorn/no-immediate-mutation': 'error',
+
+    // Disallow useless arguments when constructing Set, Map, WeakSet, WeakMap
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-useless-collection-argument.md
+    'unicorn/no-useless-collection-argument': 'error',
+
+    // Prefer class fields over constructor assignments
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-class-fields.md
+    'unicorn/prefer-class-fields': 'error',
+
+    // Prefer Array#toReversed() over Array#reverse() to avoid mutation
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-array-reverse.md
+    'unicorn/no-array-reverse': 'error',
+
+    // Require using new when throwing an error
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/throw-new-error.md
+    'unicorn/throw-new-error': 'error',
+
+    // Prefer includes() over indexOf() when checking for existence
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-includes.md
+    'unicorn/prefer-includes': 'error',
+
+    // Prefer find() over filter()[0] when searching for a single element
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-array-find.md
+    'unicorn/prefer-array-find': 'error',
+
+    // Prefer startsWith() and endsWith() over regex or slice comparisons
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-string-starts-ends-with.md
+    'unicorn/prefer-string-starts-ends-with': 'error',
+
+    // Prefer .at() for accessing elements by negative index
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-at.md
+    'unicorn/prefer-at': 'error',
+
+    // Prefer Number static properties over global ones
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-number-properties.md
+    'unicorn/prefer-number-properties': 'error',
+
+    // Prefer for...of over Array#forEach
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-array-for-each.md
+    'unicorn/no-array-for-each': 'error',
+
+    // Prefer Array#flat() over legacy techniques to flatten arrays
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-array-flat.md
+    'unicorn/prefer-array-flat': 'error',
+
+    // Prefer flatMap() over map().flat()
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-array-flat-map.md
+    'unicorn/prefer-array-flat-map': 'error',
+
+    // Disallow useless undefined
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-useless-undefined.md
+    'unicorn/no-useless-undefined': 'error',
+
+    // Prefer String#replaceAll() over regex with global flag
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-string-replace-all.md
+    'unicorn/prefer-string-replace-all': 'error',
+
+    // Prefer String#trimStart() / String#trimEnd() over trimLeft() / trimRight()
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-string-trim-start-end.md
+    'unicorn/prefer-string-trim-start-end': 'error',
+
+    // Disallow if statements as the only statement in else blocks
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-lonely-if.md
+    'unicorn/no-lonely-if': 'error',
+
+    // Prefer RegExp#test() over String#match() for boolean checks
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-regexp-test.md
+    'unicorn/prefer-regexp-test': 'error',
+
+    // Prefer modern DOM APIs
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-modern-dom-apis.md
+    'unicorn/prefer-modern-dom-apis': 'error',
+
+    // Prefer [...iterable] over Array.from(iterable)
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-spread.md
+    'unicorn/prefer-spread': 'off',
+
+    // Prefer omitting catch binding when unused
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-optional-catch-binding.md
+    'unicorn/prefer-optional-catch-binding': 'off',
+
+    // Disallow negated conditions when alternative exists
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-negated-condition.md
+    'unicorn/no-negated-condition': 'off',
+
+    // Prefer TypeError for type-related errors
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-type-error.md
+    'unicorn/prefer-type-error': 'off',
+
+    // Prefer Date.now() over new Date().getTime()
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-date-now.md
+    'unicorn/prefer-date-now': 'error',
+
+    // Prefer === undefined over typeof === 'undefined'
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-typeof-undefined.md
+    'unicorn/no-typeof-undefined': 'error',
+
+    // Prefer Object.fromEntries() over reduce to create objects
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-object-from-entries.md
+    'unicorn/prefer-object-from-entries': 'error',
+
+    // Prefer Set#has() over Array#includes() for frequent checks
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-set-has.md
+    'unicorn/prefer-set-has': 'off',
+
+    // Prefer some() over find() !== undefined for boolean checks
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-array-some.md
+    'unicorn/prefer-array-some': 'error',
+
+    // Disallow new Array() and prefer Array.from({length: n})
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-new-array.md
+    'unicorn/no-new-array': 'error',
+
+    // Prefer default parameters over reassignment
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-default-parameters.md
+    'unicorn/prefer-default-parameters': 'error',
+
+    // Prefer negative index over length minus index
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-negative-index.md
+    'unicorn/prefer-negative-index': 'error'
   }
 }

package/rules/node.mjs

@@ -18,6 +18,7 @@ export default {
     'global-require': 'error',
 
     // Enforce usage of the `node:` prefix for builtin imports
+    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-node-protocol.md
     'unicorn/prefer-node-protocol': 'error'
   }
 }

package/rules/react-hooks.mjs

@@ -7,9 +7,7 @@ import globals from 'globals'
  */
 export default {
   name: 'react-hooks',
-  plugins: {
-    'react-hooks': pluginReactHooks
-  },
+  ...pluginReactHooks.configs.flat.recommended,
   languageOptions: {
     ...pluginReact.configs.flat.recommended.languageOptions,
     globals: {
@@ -17,13 +15,7 @@ export default {
     }
   },
   rules: {
-    // Enforce Rules of Hooks
-    // https://github.com/facebook/react/blob/c11015ff4f610ac2924d1fc6d569a17657a404fd/packages/eslint-plugin-react-hooks/src/RulesOfHooks.js
-    'react-hooks/rules-of-hooks': 'error',
-
-    // Verify the list of the dependencies for Hooks like useEffect and similar
-    // https://github.com/facebook/react/blob/1204c789776cb01fbaf3e9f032e7e2ba85a44137/packages/eslint-plugin-react-hooks/src/ExhaustiveDeps.js
-    'react-hooks/exhaustive-deps': 'error',
+    ...pluginReactHooks.configs.flat.recommended.rules,
 
     // Prefer to have a convention for naming states
     // E.g: [thing, setThing]

package/rules/typescript.mjs

@@ -174,7 +174,15 @@ export default tseslint.config(
 
       // Disallow non-null assertions after an optional chain expression
       // https://typescript-eslint.io/rules/no-non-null-asserted-optional-chain
-      '@typescript-eslint/no-non-null-asserted-optional-chain': 'error'
+      '@typescript-eslint/no-non-null-asserted-optional-chain': 'error',
+
+      // Disallow default values that will never be used (type guarantees non-undefined)
+      // https://typescript-eslint.io/rules/no-useless-default-assignment
+      '@typescript-eslint/no-useless-default-assignment': 'error',
+
+      // Disallow passing a value-returning function where void is expected
+      // https://typescript-eslint.io/rules/strict-void-return
+      '@typescript-eslint/strict-void-return': 'error'
 
       // Prefer using nullish coalescing (??) over logical (||) when possible.
       // '@typescript-eslint/prefer-nullish-coalescing': 'error'

package/tsconfig.json

@@ -15,7 +15,7 @@
     // Make indexing stricter, not accessing unknown properties
     "noUncheckedIndexedAccess": true,
     "noPropertyAccessFromIndexSignature": false,
-    "moduleResolution": "node",
+    "moduleResolution": "bundler",
     // No accidental global script
     "moduleDetection": "force",
     "resolveJsonModule": true,