feat(eslint-config): drop eslint-plugin-prettier in favor of prettier directly

Author: rozsival

.changeset/early-llamas-yawn.md

@@ -0,0 +1,5 @@
+---
+'@apitree.cz/eslint-config': patch
+---
+
+Remove `eslint-plugin-prettier` in favor of `prettier` directly (only `eslint-config-prettier` overrides are applied).

lint-staged.config.js

@@ -1,6 +1,6 @@
 export default {
-  './*.{cjs,js}': 'eslint --cache --fix',
-  './*.{json,md,yml,yaml}': 'prettier --write',
+  './*.js': 'eslint --cache --fix',
+  './*.{js,json,md,yml,yaml}': 'prettier --write',
   './.changeset/**/*.{json,md}': 'prettier --write',
   './.github/**/*.{yml,yaml}': 'prettier --write',
 };

package.json

@@ -16,12 +16,12 @@
   "scripts": {
     "build": "turbo run build --no-daemon",
     "cleanup": "turbo run cleanup --no-daemon",
-    "postcleanup": "del .eslintcache tsconfig.tsbuildinfo node_modules",
+    "postcleanup": "del .eslintcache .turbo tsconfig.tsbuildinfo node_modules",
     "fix": "turbo run fix --no-daemon",
     "postfix": "run-p postts postformat:fix postlint:fix",
     "format": "turbo run format --no-daemon --parallel",
     "format:fix": "turbo run format:fix --no-daemon --parallel",
-    "postformat": "prettier --check \"./*.{json,md,yaml}\" \".github/**/*.yaml\"",
+    "postformat": "prettier --check \"./*.{js,json,md,yaml}\" \".changeset/**/*.md\" \".github/**/*.yaml\"",
     "postformat:fix": "pnpm run postformat --write",
     "postinstall": "is-ci || manypkg check",
     "lint": "turbo run lint --no-daemon",

packages/cli/lint-staged.config.js

@@ -1,4 +1,5 @@
 export default {
-  './**/.{cjs,js,ts}': 'eslint --cache --fix',
-  './**/*.{json,md}': 'prettier --write',
+  './**/*.{js,ts}': 'eslint --cache --fix',
+  './**/*.{js,json,md,ts}': 'prettier --write',
+  './**/*.ts': [() => 'tsc --build tsconfig.json'],
 };

packages/cli/package.json

@@ -25,7 +25,7 @@
     "docs:format": "prettier --write \"./docs/**/*\"",
     "docs:generate": "typedoc",
     "fix": "run-p ts format:fix lint:fix",
-    "format": "prettier --check \"./**/*.{json,md}\"",
+    "format": "prettier --check \"./**/*.{js,json,md,ts}\"",
     "format:fix": "pnpm run format --write",
     "lint": "eslint --cache .",
     "lint:fix": "pnpm run lint --fix",

packages/eslint-config/lint-staged.config.js

@@ -1,5 +1,5 @@
 export default {
-  './**/*.js': 'eslint --cache --fix',
-  './**/*.{json,md}': 'prettier --write',
-  './**/*.ts': [() => 'tsc --build tsconfig.json', 'eslint --cache --fix'],
+  './**/*.{js,ts}': 'eslint --cache --fix',
+  './**/*.{js,json,md,ts}': 'prettier --write',
+  './**/*.ts': [() => 'tsc --build tsconfig.json'],
 };

packages/eslint-config/package.json

@@ -43,7 +43,7 @@
     "build": "tsc --build tsconfig.build.json",
     "cleanup": "del .eslintcache .turbo tsconfig.tsbuildinfo tsconfig.build.tsbuildinfo dist node_modules",
     "fix": "run-p ts format:fix lint:fix",
-    "format": "prettier --check \"./**/*.{json,md}\"",
+    "format": "prettier --check \"./**/*.{js,json,md,ts}\"",
     "format:fix": "pnpm run format --write",
     "lint": "eslint --cache .",
     "lint:fix": "pnpm run lint --fix",
@@ -65,7 +65,6 @@
     "eslint-plugin-jest-dom": "^5.5.0",
     "eslint-plugin-jsx-a11y": "^6.10.2",
     "eslint-plugin-mdx": "^3.1.5",
-    "eslint-plugin-prettier": "^5.2.3",
     "eslint-plugin-react": "^7.37.4",
     "eslint-plugin-react-hooks": "^5.1.0",
     "eslint-plugin-storybook": "^0.11.2",
@@ -80,7 +79,6 @@
   "devDependencies": {
     "@types/eslint__eslintrc": "^2.1.2",
     "eslint": "^9.19.0",
-    "prettier": "^3.4.2",
     "react": "^19.0.0",
     "typescript": "^5.7.3"
   },

packages/eslint-config/src/base/config.ts

@@ -1,8 +1,7 @@
 import eslint from '@eslint/js';
 import vitest from '@vitest/eslint-plugin';
-import type { Linter } from 'eslint';
-import imports from 'eslint-plugin-import';
-import prettier from 'eslint-plugin-prettier/recommended';
+import prettier from 'eslint-config-prettier';
+import * as imports from 'eslint-plugin-import';
 import turbo from 'eslint-plugin-turbo';
 import unicorn from 'eslint-plugin-unicorn';
 import unusedImports from 'eslint-plugin-unused-imports';
@@ -35,9 +34,8 @@ export const config: ConfigArray = tsEslint.config(
     ],
   },
   eslint.configs.recommended,
-  prettier,
   {
-    extends: [imports.flatConfigs.recommended, imports.flatConfigs.typescript] as Linter.Config[],
+    extends: [imports.flatConfigs.recommended, imports.flatConfigs.typescript],
     plugins: {
       '@typescript-eslint': tsEslintPlugin,
       'unused-imports': unusedImports,
@@ -100,4 +98,5 @@ export const config: ConfigArray = tsEslint.config(
     files: ['**/*.d.ts'],
     rules: rules.dts,
   },
+  prettier,
 );

packages/eslint-config/src/base/rules/common.ts

@@ -1,7 +1,5 @@
 import type { Linter } from 'eslint';
 
-const lineLength = 120;
-
 export const common: Linter.RulesRecord = {
   'array-callback-return': [
     'error',
@@ -10,28 +8,13 @@ export const common: Linter.RulesRecord = {
     },
   ], // Allow implicit return in array methods
   'arrow-body-style': ['error', 'as-needed'], // Allow omitting braces when possible
-  'arrow-parens': ['error', 'always'], // Require parens around arrow function arguments
-  'comma-dangle': 'off', // Allow dangling commas
   'consistent-return': 'off', // Allow implicit return in arrow functions
   curly: ['error', 'all'], // Require braces around all blocks
   'default-case': 'off', // Allow switch statements without default
   'function-paren-newline': 'off', // Allow parens on same line as function name
   'guard-for-in': 'off', // Allow for-in without if statement
   'implicit-arrow-linebreak': 'off', // Allow implicit return in arrow functions
   'max-classes-per-file': ['error', { ignoreExpressions: true, max: 3 }], // Allow multiple classes per file
-  'max-len': [
-    // Enforce max line length
-    'error',
-    lineLength, // Characters per line
-    2, // Tab width
-    {
-      ignoreUrls: true, // Let URLs be as long as they need to be
-      ignoreComments: true, // Allow comments to be as long as they need to be
-      ignoreRegExpLiterals: true, // Let regex literals be as long as they need to be
-      ignoreStrings: true, // Let strings be as long as they need to be
-      ignoreTemplateLiterals: true, // Let template literals be as long as they need to be
-    },
-  ],
   'no-confusing-arrow': 'off', // Do not guard against confusing arrow functions as we enforce parens
   'no-console': 'warn', // Warn against console.log
   'no-debugger': 'warn', // Warn against debugger
@@ -67,5 +50,4 @@ export const common: Linter.RulesRecord = {
   'no-shadow': 'off', // Allow variables shadowing
   'no-use-before-define': 'off', // Guarded by @typescript-eslint/no-use-before-define
   'no-warning-comments': ['warn', { terms: ['fixme', 'todo'] }], // Warn against fixme and todo comments
-  'object-curly-newline': ['error', { consistent: true }], // Enforce consistent line breaks inside braces
 };

packages/eslint-config/src/base/rules/typescript.ts

@@ -9,7 +9,7 @@ export const typescript: Linter.RulesRecord = {
   '@typescript-eslint/no-unsafe-declaration-merging': 'error', // Guard declaration merging against unsafe usage
   '@typescript-eslint/no-unused-vars': 'off', // Guarded by eslint-plugin-unused-imports
   '@typescript-eslint/no-use-before-define': 'error', // Disallow usage of variables before their declaration
-  // Enforce template literal expressions to be of string type
+  // Enforce template literal expressions to be of string type or safely cast to string (no complex types)
   '@typescript-eslint/restrict-template-expressions': [
     'error',
     { allowBoolean: true, allowNumber: true, allowNullish: true },