feat: adopt stylistic eslint and more

Author: jasonkuhrt

eslint.config.js

@@ -1,13 +1,55 @@
-import configPrisma from 'eslint-config-prisma'
+import eslint from '@eslint/js'
+import stylistic from '@stylistic/eslint-plugin'
+import onlyWarn from 'eslint-plugin-only-warn'
+import tsdoc from 'eslint-plugin-tsdoc'
 import tsEslint from 'typescript-eslint'
 
-export default tsEslint.config({
-  extends: [...configPrisma],
-  ignores: ['**/build/**/*', 'eslint.config.js'],
-  languageOptions: {
-    parserOptions: {
-      project: true,
-      tsconfigRootDir: import.meta.dirname,
+export default tsEslint.config(
+  eslint.configs.recommended,
+  stylistic.configs.recommended,
+  tsEslint.configs.recommendedTypeChecked,
+  tsEslint.configs.strictTypeChecked,
+  tsEslint.configs.eslintRecommended,
+  tsEslint.configs.stylisticTypeChecked,
+  {
+    ignores: ['build', 'eslint.config.js'],
+  },
+  {
+    languageOptions: {
+      parserOptions: {
+        projectService: true,
+        tsconfigRootDir: import.meta.dirname,
+      },
+    },
+    plugins: {
+      // https://github.com/microsoft/tsdoc/tree/master/eslint-plugin
+      tsdoc: tsdoc,
+      // This makes it so the IDE reports lint rejections as warnings only. This is
+      // better than errors because most lint rejections are not runtime errors. This
+      // allows IDE errors to be exclusive for e.g. static type errors which often are
+      // reflective of real runtime errors.
+      // https://github.com/bfanger/eslint-plugin-only-warn
+      onlyWarn,
+      '@stylistic': stylistic,
+    },
+    rules: {
+      '@stylistic/quotes': ['warn', 'backtick'],
+      '@typescript-eslint/consistent-type-imports': 'warn',
+      'tsdoc/syntax': 'warn',
+      // TypeScript makes these safe & effective
+      'no-case-declarations': 'off',
+      // Same approach used by TypeScript noUnusedLocals
+      '@typescript-eslint/no-unused-vars': ['warn', { varsIgnorePattern: '^_', argsIgnorePattern: '^_' }],
+      // Needed when working with .mts/.cts where a lone e.g. <T> is not allowed
+      '@typescript-eslint/no-unnecessary-type-constraint': 'off',
+      // Useful for organizing Types
+      '@typescript-eslint/no-namespace': 'off',
+      // Turn training wheels off. When we want these we want these.
+      '@typescript-eslint/no-non-null-assertion': 'off',
+      '@typescript-eslint/no-unsafe-argument': 'off',
+      '@typescript-eslint/no-unsafe-return': 'off',
+      '@typescript-eslint/no-explicit-any': 'off',
+      '@typescript-eslint/ban-ts-comment': ['warn', { 'ts-expect-error': false }],
     },
   },
-})
+)

package.json

@@ -45,6 +45,7 @@
     "check:publint": "publint run --strict"
   },
   "devDependencies": {
+    "@stylistic/eslint-plugin": "^4.2.0",
     "@tsconfig/node22": "^22.0.0",
     "@tsconfig/recommended": "1.0.8",
     "@tsconfig/strictest": "2.0.5",
@@ -53,10 +54,7 @@
     "@typescript-eslint/parser": "8.27.0",
     "dripip": "0.10.0",
     "eslint": "9.22.0",
-    "eslint-config-prisma": "0.6.0",
-    "eslint-plugin-deprecation": "3.0.0",
     "eslint-plugin-only-warn": "1.1.0",
-    "eslint-plugin-prefer-arrow": "1.2.3",
     "eslint-plugin-tsdoc": "0.4.0",
     "execa": "9.5.2",
     "fast-glob": "3.3.3",
@@ -67,6 +65,7 @@
     "publint": "^0.3.9",
     "tsx": "4.19.3",
     "typescript": "5.8.2",
+    "typescript-eslint": "^8.27.0",
     "vitest": "3.0.9"
   }
 }