@ololoepepe/eslint-config-typescript 0.0.11 → 0.1.0

package/CHANGELOG.md

@@ -0,0 +1,67 @@
+# Changelog
+
+All notable changes to this project are documented in this file. The format is
+based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and this
+project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## [0.1.0] - 2026-07-01
+
+A major modernization of the config, brought to parity with the sibling
+`@ololoepepe/eslint-config` while staying standalone and TypeScript-specific.
+**This release contains breaking changes.**
+
+### Changed
+
+- **BREAKING:** Upgraded to **ESLint 10** (`>= 10.4`) and migrated to the **flat
+  config** format. The legacy `.eslintrc` format is no longer supported.
+- **BREAKING:** Each entry point now exports a **flat-config array** to be spread
+  into a consumer's `eslint.config.js`, instead of a legacy config object
+  (`env` / `parser` / `parserOptions` / `plugins: []` / `rules: {}`).
+- **BREAKING:** Replaced `eslint-plugin-import` with **`eslint-plugin-import-x`**
+  (all `import/*` rules are now `import-x/*`).
+- **BREAKING:** Core formatting rules were moved to **`@stylistic/eslint-plugin`**
+  (`@stylistic/*`), mirroring their removal from ESLint core in v10. Options are
+  preserved 1:1.
+- Adopted the **`typescript-eslint@8`** meta-package; typed linting is enabled via
+  `parserOptions.projectService`. The `@typescript-eslint/*` rule prefixes are
+  unchanged.
+- Bumped **`eslint-plugin-perfectionist`** to `^5.9` — retained as this package's
+  distinctive type/interface/enum-sorting feature.
+- Bumped `eslint-plugin-sort-keys-shorthand` to `^5`; it is now the **sole owner**
+  of object-literal key ordering (shorthand-first).
+
+### Added
+
+- **Layered entry points:**
+  - `@ololoepepe/eslint-config-typescript` — env-neutral TypeScript base.
+  - `@ololoepepe/eslint-config-typescript/node` — base + Node globals +
+    `eslint-plugin-n`.
+  - `@ololoepepe/eslint-config-typescript/react` — base + browser globals +
+    `@eslint-react`, `eslint-plugin-react-hooks`, and JSX `@stylistic`, scoped to
+    `.tsx` / `.jsx`.
+- Correctness plugins on the base: **`unicorn`**, **`regexp`**, **`promise`**, and
+  **`@eslint-community/eslint-comments`**, each with an explicitly configured rule
+  map.
+- **`typescript`** as a peer dependency (`>=5 <6.1`) and dev dependency.
+- A **fixture-driven test suite** (positive + negative fixtures across the base,
+  `/node`, and `/react` layers), alongside the retained self-lint test.
+- The base config now registers TypeScript file extensions (`.ts` / `.cts` /
+  `.mts`), so `eslint .` lints TypeScript out of the box.
+
+### Removed
+
+- **BREAKING:** Removed `eslint-plugin-import`, `eslint-find-rules`, and the
+  `find-rules` script.
+- Removed core rules that no longer exist in ESLint 10, replacing them with their
+  modern equivalents or `@typescript-eslint/*` typed analogs (single owner per
+  concern, no double-reporting).
+
+### Fixed
+
+- Resolved rule conflicts between overlapping plugins:
+  - Object-literal key ordering: `perfectionist/sort-objects` disabled in favor of
+    `sort-keys-shorthand` (shorthand-first).
+  - Import-statement ordering: `perfectionist/sort-imports` disabled in favor of
+    `import-x/order` (which carries the project path groups).
+  - Disabled `n/no-unsupported-features/es-syntax`, since downlevel syntax for
+    TypeScript is owned by the compiler (`tsconfig` `target`).

package/package.json

@@ -1,16 +1,20 @@
 {
   "name": "@ololoepepe/eslint-config-typescript",
-  "version": "0.0.11",
+  "version": "0.1.0",
   "description": "ESLint config for TypeScript",
   "main": "src/index.js",
+  "exports": {
+    ".": "./src/index.js",
+    "./node": "./src/node.js",
+    "./react": "./src/react.js"
+  },
   "imports": {
     "#src/*.js": "./src/*.js"
   },
   "type": "module",
   "scripts": {
-    "find-rules": "eslint-find-rules --unused ./src/index.js",
     "lint": "eslint ./src",
-    "test": "mocha ./test/*"
+    "test": "mocha './test/**/*.test.js'"
   },
   "publishConfig": {
     "registry": "https://registry.npmjs.com"
@@ -18,7 +22,8 @@
   "files": [
     "src",
     "package.json",
-    "README.md"
+    "README.md",
+    "CHANGELOG.md"
   ],
   "repository": {
     "type": "git",
@@ -31,16 +36,31 @@
   },
   "homepage": "https://gitea.void-walkers.com/libs/eslint-config-typescript#README",
   "dependencies": {
-    "@typescript-eslint/eslint-plugin": "^8.50.1",
-    "@typescript-eslint/parser": "^8.50.1",
-    "eslint": "^8.57.1",
-    "eslint-import-resolver-typescript": "^4.4.4",
-    "eslint-plugin-import": "^2.32.0",
-    "eslint-plugin-perfectionist": "^5.1.0",
-    "eslint-plugin-sort-keys-shorthand": "^2.3.0"
+    "@eslint-community/eslint-plugin-eslint-comments": "^4.7.2",
+    "@eslint-react/eslint-plugin": "^5.10.0",
+    "@stylistic/eslint-plugin": "^5.0.0",
+    "eslint": "^10.6.0",
+    "eslint-import-resolver-typescript": "^4.4.0",
+    "eslint-plugin-import-x": "^4.17.0",
+    "eslint-plugin-n": "^18.2.1",
+    "eslint-plugin-perfectionist": "^5.9.0",
+    "eslint-plugin-promise": "^7.3.0",
+    "eslint-plugin-react-hooks": "^7.1.1",
+    "eslint-plugin-regexp": "^3.1.1",
+    "eslint-plugin-sort-keys-shorthand": "^5.0.0",
+    "eslint-plugin-unicorn": "^69.0.0",
+    "globals": "^17.0.0",
+    "ts-declaration-location": "^1.0.7",
+    "typescript-eslint": "^8.62.0"
   },
   "devDependencies": {
-    "eslint-find-rules": "^5.0.0",
-    "mocha": "^11.7.5"
+    "@types/node": "^22.20.0",
+    "@types/react": "^19.2.0",
+    "mocha": "^11.7.5",
+    "react": "^19.2.0",
+    "typescript": "^5.9.0"
+  },
+  "peerDependencies": {
+    "typescript": ">=5 <6.1"
   }
 }

package/src/index.js

@@ -1,316 +1,88 @@
-import pluginImport from '#src/plugins/import.js';
+/* eslint-disable @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access --
+   The ESLint plugins and parser registered in this wiring file are untyped
+   third-party modules, so the typed-linting rules treat them as `any`. Suppress
+   the unsafe-access reports for the whole file. */
+import eslintComments from '@eslint-community/eslint-plugin-eslint-comments';
+import stylistic from '@stylistic/eslint-plugin';
+import importX from 'eslint-plugin-import-x';
+import perfectionist from 'eslint-plugin-perfectionist';
+import promise from 'eslint-plugin-promise';
+import regexp from 'eslint-plugin-regexp';
+import sortKeysShorthand from 'eslint-plugin-sort-keys-shorthand';
+import unicorn from 'eslint-plugin-unicorn';
+import globals from 'globals';
+import tseslint from 'typescript-eslint';
+
+import pluginEslintComments from '#src/plugins/eslint-comments.js';
+import pluginImportX from '#src/plugins/import-x.js';
 import pluginPerfectionist from '#src/plugins/perfectionist.js';
+import pluginPromise from '#src/plugins/promise.js';
+import pluginRegexp from '#src/plugins/regexp.js';
 import pluginSortKeysShorthand from '#src/plugins/sort-keys-shorthand.js';
+import pluginStylistic from '#src/plugins/stylistic.js';
 import pluginTypescript from '#src/plugins/typescript/index.js';
+import pluginUnicorn from '#src/plugins/unicorn.js';
+import rulesCore from '#src/rules/core.js';
 import rulesNo from '#src/rules/no.js';
 
-export const env = {
-  es2021: true,
-  node: true
-};
-
-export const parser = '@typescript-eslint/parser';
-
-export const parserOptions = {
-  ecmaFeatures: {
-    jsx: true
-  },
-  ecmaVersion: 'latest',
-  project: './tsconfig.json',
-  requireConfigFile: false,
-  sourceType: 'module'
-};
-
-export const plugins = [
-  '@typescript-eslint',
-  'import',
-  'perfectionist',
-  'sort-keys-shorthand'
-];
-
-// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-export const rules = {
-  'accessor-pairs': ['warn', {
-    enforceForClassMembers: true,
-    getWithoutSet: false,
-    setWithoutGet: true
-  }],
-  'array-bracket-newline': ['error', 'consistent'],
-  'array-bracket-spacing': ['error', 'never', {
-    arraysInArrays: false,
-    objectsInArrays: false,
-    singleValue: false
-  }],
-  'array-callback-return': ['error', {
-    allowImplicit: false,
-    checkForEach: false
-  }],
-  'array-element-newline': ['error', 'consistent'],
-  'arrow-body-style': ['error', 'as-needed'],
-  'arrow-parens': ['error', 'as-needed', {
-    requireForBlockBody: false
-  }],
-  'arrow-spacing': ['error', {
-    after: true,
-    before: true
-  }],
-  'block-scoped-var': 'error',
-  'block-spacing': ['error', 'always'],
-  'brace-style': ['error', '1tbs', {
-    allowSingleLine: false
-  }],
-  camelcase: ['error', {
-    ignoreDestructuring: false,
-    ignoreGlobals: true,
-    ignoreImports: false,
-    properties: 'never'
-  }],
-  'capitalized-comments': ['error', 'always', {
-    ignoreConsecutiveComments: true,
-    ignoreInlineComments: false,
-    ignorePattern: 'webpackChunkName'
-  }],
-  'class-methods-use-this': 'off',
-  'comma-dangle': ['error', 'never'],
-  'comma-spacing': ['error', {
-    after: true,
-    before: false
-  }],
-  'comma-style': ['error', 'last'],
-  complexity: ['warn', 20],
-  'computed-property-spacing': ['error', 'never', {
-    enforceForClassMembers: true
-  }],
-  'consistent-return': 'off',
-  'consistent-this': ['error', 'self'],
-  'constructor-super': 'error',
-  curly: ['error', 'all'],
-  'default-case': 'off',
-  'default-case-last': 'error',
-  'default-param-last': 'error',
-  'dot-location': ['error', 'property'],
-  'dot-notation': 'off', // NOTE: See TypeScript rule extending this one
-  'eol-last': ['error', 'always'],
-  eqeqeq: ['error', 'always'],
-  'for-direction': 'error',
-  'func-call-spacing': ['error', 'never'],
-  'func-name-matching': ['warn', 'always'],
-  'func-names': ['error', 'never', {
-    generators: 'never'
-  }],
-  'func-style': ['error', 'declaration', {
-    allowArrowFunctions: true
-  }],
-  'function-call-argument-newline': ['error', 'consistent'],
-  'function-paren-newline': ['error', 'multiline-arguments'],
-  'generator-star-spacing': ['error', {
-    after: true,
-    before: true
-  }],
-  'getter-return': ['error', {
-    allowImplicit: false
-  }],
-  'grouped-accessor-pairs': ['error', 'getBeforeSet'],
-  'guard-for-in': 'error',
-  'id-denylist': 'off',
-  'id-length': 'off',
-  'id-match': 'off',
-  'implicit-arrow-linebreak': ['error', 'beside'],
-  indent: ['error', 2],
-  'init-declarations': 'off', // NOTE: See TypeScript rule extending this one
-  'jsx-quotes': ['error', 'prefer-double'],
-  'key-spacing': ['error', {
-    afterColon: true,
-    beforeColon: false
-  }],
-  'keyword-spacing': ['error', {
-    after: true,
-    before: true
-  }],
-  'line-comment-position': 'off',
-  'linebreak-style': ['error', 'unix'],
-  'lines-around-comment': 'off',
-  'lines-between-class-members': ['error', 'always', {
-    exceptAfterSingleLine: true
-  }],
-  'logical-assignment-operators': ['error', 'always', {
-    enforceForIfStatements: false
-  }],
-  'max-classes-per-file': 'off',
-  'max-depth': ['warn', 5],
-  'max-len': ['error', {
-    code: 120,
-    comments: 120,
-    ignoreComments: false,
-    ignoreRegExpLiterals: false,
-    ignoreStrings: false,
-    ignoreTemplateLiterals: false,
-    ignoreTrailingComments: false,
-    ignoreUrls: false,
-    tabWidth: 2
-  }],
-  'max-lines': ['warn', {
-    max: 500,
-    skipBlankLines: true,
-    skipComments: true
-  }],
-  'max-lines-per-function': ['warn', {
-    IIFEs: true,
-    max: 100,
-    skipBlankLines: true,
-    skipComments: true
-  }],
-  'max-nested-callbacks': ['warn', 5],
-  'max-params': 'off', // NOTE: See TypeScript rule extending this one
-  'max-statements': 'off',
-  'max-statements-per-line': ['error', {
-    max: 1
-  }],
-  'multiline-comment-style': 'off',
-  'multiline-ternary': 'off',
-  'new-cap': ['error', {
-    capIsNew: true,
-    newIsCap: true,
-    properties: true
-  }],
-  'new-parens': ['error', 'always'],
-  'newline-per-chained-call': ['error', {
-    ignoreChainWithDepth: 3
-  }],
-  'nonblock-statement-body-position': 'off',
-  'object-curly-newline': 'off',
-  'object-curly-spacing': ['error', 'never', {
-    arraysInObjects: false,
-    objectsInObjects: false
-  }],
-  'object-property-newline': 'off', // Reconsider
-  'object-shorthand': ['error', 'always', {
-    avoidExplicitReturnArrows: false,
-    avoidQuotes: true,
-    ignoreConstructors: false
-  }],
-  'one-var': ['error', 'never'],
-  'one-var-declaration-per-line': 'off',
-  'operator-assignment': ['error', 'always'],
-  'operator-linebreak': ['error', 'after'],
-  'padded-blocks': ['error', 'never', {
-    allowSingleLineBlocks: false
-  }],
-  'padding-line-between-statements': ['error', {
-    blankLine: 'always',
-    next: '*',
-    prev: 'multiline-block-like'
-  }],
-  'prefer-arrow-callback': ['error', {
-    allowNamedFunctions: false,
-    allowUnboundThis: true
-  }],
-  'prefer-const': ['error', {
-    destructuring: 'all',
-    ignoreReadBeforeAssign: true
-  }],
-  'prefer-destructuring': 'off', // NOTE: See TypeScript rule extending this one
-  'prefer-exponentiation-operator': 'error',
-  'prefer-named-capture-group': 'warn',
-  'prefer-numeric-literals': 'error',
-  'prefer-object-has-own': 'off',
-  'prefer-object-spread': 'error',
-  'prefer-promise-reject-errors': 'off', // NOTE: See TypeScript rule extending this one
-  'prefer-regex-literals': ['error', {
-    disallowRedundantWrapping: true
-  }],
-  'prefer-rest-params': 'error',
-  'prefer-spread': 'error',
-  'prefer-template': 'error',
-  'quote-props': ['error', 'as-needed', {
-    keywords: false,
-    numbers: true,
-    unnecessary: true
-  }],
-  quotes: ['error', 'single', {
-    allowTemplateLiterals: false,
-    avoidEscape: true
-  }],
-  radix: ['error', 'as-needed'],
-  'require-atomic-updates': 'error',
-  'require-await': 'off', // NOTE: See TypeScript rule extending this one
-  'require-unicode-regexp': 'error',
-  'require-yield': 'off',
-  'rest-spread-spacing': ['error', 'never'],
-  semi: ['error', 'always', {
-    omitLastInOneLineBlock: false
-  }],
-  'semi-spacing': ['error', {
-    after: true,
-    before: false
-  }],
-  'semi-style': ['error', 'last'],
-  'sort-imports': 'off',
-  'sort-keys': 'off',
-  'sort-vars': ['error', {
-    ignoreCase: true
-  }],
-  'space-before-blocks': ['error', 'always'],
-  'space-before-function-paren': ['error', {
-    anonymous: 'always',
-    asyncArrow: 'always',
-    named: 'never'
-  }],
-  'space-in-parens': ['error', 'never'],
-  'space-infix-ops': ['error', {
-    int32Hint: false
-  }],
-  'space-unary-ops': ['error', {
-    nonwords: false,
-    words: true
-  }],
-  'spaced-comment': ['error', 'always', {
-    block: {
-      balanced: true,
-      exceptions: ['-', '+', '*'],
-      markers: ['!', '*']
+export default [{
+  files: [
+    '**/*.ts',
+    '**/*.tsx',
+    '**/*.cts',
+    '**/*.mts',
+    '**/*.js',
+    '**/*.jsx',
+    '**/*.cjs',
+    '**/*.mjs'
+  ],
+  languageOptions: {
+    ecmaVersion: 'latest',
+    globals: {
+      ...globals.es2021
     },
-    line: {
-      exceptions: ['-', '+', '*'],
-      markers: ['!', '/', '=>']
-    }
-  }],
-  strict: ['error', 'never'],
-  'switch-colon-spacing': ['error', {
-    after: true,
-    before: false
-  }],
-  'symbol-description': 'error',
-  'template-curly-spacing': ['error', 'never'],
-  'template-tag-spacing': ['error', 'never'],
-  'unicode-bom': ['error', 'never'],
-  'use-isnan': ['error', {
-    enforceForIndexOf: true,
-    enforceForSwitchCase: true
-  }],
-  'valid-typeof': ['error', {
-    requireStringLiterals: false
-  }],
-  'vars-on-top': 'error',
-  'wrap-iife': ['error', 'inside', {
-    functionPrototypeMethods: true
-  }],
-  'wrap-regex': 'off',
-  'yield-star-spacing': ['error', 'both'],
-  yoda: ['error', 'never'],
-  ...rulesNo,
-  /* eslint-disable @typescript-eslint/no-unsafe-member-access */
-  ...pluginTypescript.rules,
-  ...pluginPerfectionist.rules,
-  ...pluginImport.rules,
-  ...pluginSortKeysShorthand.rules
-  /* eslint-enable @typescript-eslint/no-unsafe-member-access */
-};
-
-export const settings = {
-  'import/resolver': {
-    typescript: {
-      project: './tsconfig.json'
+    parser: tseslint.parser,
+    parserOptions: {
+      ecmaFeatures: {
+        jsx: true
+      },
+      projectService: true
+    },
+    sourceType: 'module'
+  },
+  linterOptions: {
+    reportUnusedDisableDirectives: 'error'
+  },
+  name: '@ololoepepe/eslint-config-typescript',
+  plugins: {
+    perfectionist,
+    promise,
+    regexp,
+    unicorn,
+    '@eslint-community/eslint-comments': eslintComments,
+    '@stylistic': stylistic,
+    '@typescript-eslint': tseslint.plugin,
+    'import-x': importX,
+    'sort-keys-shorthand': sortKeysShorthand
+  },
+  rules: {
+    ...rulesCore,
+    ...rulesNo,
+    ...pluginEslintComments.rules,
+    ...pluginStylistic.rules,
+    ...pluginTypescript.rules,
+    ...pluginImportX.rules,
+    ...pluginPerfectionist.rules,
+    ...pluginPromise.rules,
+    ...pluginRegexp.rules,
+    ...pluginSortKeysShorthand.rules,
+    ...pluginUnicorn.rules
+  },
+  settings: {
+    'import-x/resolver': {
+      typescript: {
+        project: './tsconfig.json'
+      }
     }
   }
-};
+}];

package/src/node.js

@@ -0,0 +1,27 @@
+/* eslint-disable @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access --
+   The ESLint plugin and globals registered in this wiring file are untyped
+   third-party modules, so the typed-linting rules treat them as `any`. Suppress
+   the unsafe-access reports for the whole file. */
+import n from 'eslint-plugin-n';
+import globals from 'globals';
+
+import base from '#src/index.js';
+import pluginN from '#src/plugins/n.js';
+
+export default [
+  ...base,
+  {
+    languageOptions: {
+      globals: {
+        ...globals.node
+      }
+    },
+    name: '@ololoepepe/eslint-config-typescript/node',
+    plugins: {
+      n
+    },
+    rules: {
+      ...pluginN.rules
+    }
+  }
+];

package/src/plugins/eslint-comments.js

@@ -0,0 +1,16 @@
+export default {
+  rules: {
+    '@eslint-community/eslint-comments/disable-enable-pair': ['error', {
+      allowWholeFile: true
+    }],
+    '@eslint-community/eslint-comments/no-aggregating-enable': 'error',
+    '@eslint-community/eslint-comments/no-duplicate-disable': 'error',
+    '@eslint-community/eslint-comments/no-restricted-disable': 'off',
+    '@eslint-community/eslint-comments/no-unlimited-disable': 'error',
+    // NOTE: `no-unused-disable` is deprecated; `linterOptions.reportUnusedDisableDirectives`
+    // (set to 'error' in src/index.js) is the owner for unused disable directives.
+    '@eslint-community/eslint-comments/no-unused-enable': 'error',
+    '@eslint-community/eslint-comments/no-use': 'off',
+    '@eslint-community/eslint-comments/require-description': 'error'
+  }
+};