feature(eslint): enable linting with type information

.changeset/fifty-aliens-rhyme.md

@@ -0,0 +1,10 @@
+---
+'@hono/eslint-config': major
+---
+
+Includes `typescript-eslint` presets for typed linting
+
+- [`strict-type-checked`](https://typescript-eslint.io/users/configs#strict-type-checked)
+- [`stylistic-type-checked`](https://typescript-eslint.io/users/configs#stylistic-type-checked)
+
+See [Linting with Type Information](https://typescript-eslint.io/getting-started/typed-linting) for more information

eslint.config.mjs

@@ -1,6 +1,62 @@
 import baseConfig from '@hono/eslint-config'
 import { defineConfig, globalIgnores } from 'eslint/config'
 
-export default defineConfig(globalIgnores(['.yarn', '**/dist']), {
+export default defineConfig(globalIgnores(['.yarn', '**/coverage', '**/dist']), {
   extends: baseConfig,
+
+  languageOptions: {
+    parserOptions: {
+      projectService: true,
+      tsconfigRootDir: import.meta.dirname,
+    },
+  },
+
+  linterOptions: {
+    reportUnusedDisableDirectives: 'error',
+    reportUnusedInlineConfigs: 'error',
+  },
+
+  rules: {
+    '@typescript-eslint/array-type': 'off',
+    '@typescript-eslint/await-thenable': 'off',
+    '@typescript-eslint/consistent-indexed-object-style': 'off',
+    '@typescript-eslint/consistent-type-definitions': 'off',
+    '@typescript-eslint/dot-notation': 'off',
+    '@typescript-eslint/no-base-to-string': 'off',
+    '@typescript-eslint/no-confusing-void-expression': 'off',
+    '@typescript-eslint/no-deprecated': 'off',
+    '@typescript-eslint/no-duplicate-type-constituents': 'off',
+    '@typescript-eslint/no-dynamic-delete': 'off',
+    '@typescript-eslint/no-floating-promises': 'off',
+    '@typescript-eslint/no-invalid-void-type': 'off',
+    '@typescript-eslint/no-misused-promises': 'off',
+    '@typescript-eslint/no-non-null-assertion': 'off',
+    '@typescript-eslint/no-redundant-type-constituents': 'off',
+    '@typescript-eslint/no-unnecessary-boolean-literal-compare': 'off',
+    '@typescript-eslint/no-unnecessary-condition': 'off',
+    '@typescript-eslint/no-unnecessary-template-expression': 'off',
+    '@typescript-eslint/no-unnecessary-type-arguments': 'off',
+    '@typescript-eslint/no-unnecessary-type-assertion': 'off',
+    '@typescript-eslint/no-unnecessary-type-parameters': 'off',
+    '@typescript-eslint/no-unsafe-argument': 'off',
+    '@typescript-eslint/no-unsafe-assignment': 'off',
+    '@typescript-eslint/no-unsafe-call': 'off',
+    '@typescript-eslint/no-unsafe-enum-comparison': 'off',
+    '@typescript-eslint/no-unsafe-member-access': 'off',
+    '@typescript-eslint/no-unsafe-return': 'off',
+    '@typescript-eslint/no-useless-constructor': 'off',
+    '@typescript-eslint/non-nullable-type-assertion-style': 'off',
+    '@typescript-eslint/only-throw-error': 'off',
+    '@typescript-eslint/prefer-function-type': 'off',
+    '@typescript-eslint/prefer-includes': 'off',
+    '@typescript-eslint/prefer-nullish-coalescing': 'off',
+    '@typescript-eslint/prefer-optional-chain': 'off',
+    '@typescript-eslint/prefer-regexp-exec': 'off',
+    '@typescript-eslint/prefer-return-this-type': 'off',
+    '@typescript-eslint/require-await': 'off',
+    '@typescript-eslint/restrict-plus-operands': 'off',
+    '@typescript-eslint/restrict-template-expressions': 'off',
+    '@typescript-eslint/unbound-method': 'off',
+    '@typescript-eslint/unified-signatures': 'off',
+  },
 })

package.json

@@ -32,15 +32,13 @@
     "@cloudflare/workers-types": "^4.20230307.0",
     "@hono/eslint-config": "workspace:*",
     "@ryoppippi/unplugin-typia": "^1.2.0",
+    "@types/bun": "^1.0.0",
     "@types/node": "^20.17.28",
     "@types/ws": "^8.18.0",
-    "@typescript-eslint/eslint-plugin": "^8.7.0",
-    "@typescript-eslint/parser": "^8.7.0",
     "@vitest/coverage-istanbul": "^3.0.8",
-    "eslint": "^9.17.0",
+    "eslint": "^9.23.0",
     "hono": "^4.7.5",
-    "npm-run-all2": "^6.2.2",
-    "prettier": "^2.7.1",
+    "prettier": "^3.5.3",
     "tsup": "^8.4.0",
     "typescript": "^5.8.2",
     "vitest": "^3.0.8"

packages/ajv-validator/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/ajv-validator",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/arktype-validator/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/arktype-validator",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/auth-js/tsconfig.spec.json

@@ -5,7 +5,7 @@
     "noEmit": true,
     "jsx": "react"
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/bun-transpiler/package.json

@@ -44,7 +44,6 @@
   },
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.17.4",
-    "@types/bun": "^1.0.0",
     "publint": "^0.3.9",
     "tsup": "^8.4.0",
     "typescript": "^5.8.2",

packages/bun-transpiler/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/bun-transpiler",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/casbin/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/casbin",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts", "vitest.setup.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/class-validator/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/class-validator",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/clerk-auth/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/clerk-auth",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/cloudflare-access/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/cloudflare-access",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/conform-validator/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/conform-validator",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/effect-validator/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/effect-validator",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/esbuild-transpiler/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/esbuild-transpiler",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/eslint-config/index.js

@@ -7,7 +7,8 @@ import tseslint from 'typescript-eslint'
 export default [
   js.configs.recommended,
   nodePlugin.configs['flat/recommended'],
-  ...tseslint.configs.recommended,
+  ...tseslint.configs.strictTypeChecked,
+  ...tseslint.configs.stylisticTypeChecked,
   {
     plugins: {
       '@typescript-eslint': tseslint.plugin,

packages/eslint-config/package.json

@@ -28,9 +28,5 @@
     "eslint-plugin-import-x": "^4.1.1",
     "eslint-plugin-n": "^17.10.2",
     "typescript-eslint": "^8.27.0"
-  },
-  "devDependencies": {
-    "eslint": "^9.23.0",
-    "typescript": "^5.3.3"
   }
 }

packages/event-emitter/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/event-emitter",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/firebase-auth/package.json

@@ -52,7 +52,6 @@
     "@arethetypeswrong/cli": "^0.17.4",
     "firebase-tools": "^13.29.1",
     "miniflare": "^3.20240208.0",
-    "prettier": "^3.2.5",
     "publint": "^0.3.9",
     "tsup": "^8.4.0",
     "typescript": "^5.8.2",

packages/firebase-auth/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/firebase-auth",
     "noEmit": true
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/graphql-server/tsconfig.bun.json

@@ -0,0 +1,14 @@
+{
+  "extends": "../../tsconfig.base.json",
+  "compilerOptions": {
+    "outDir": "../../dist/out-tsc/packages/graphql-server",
+    "types": ["bun"]
+  },
+  "exclude": ["src/**/*.test.ts"],
+  "include": ["bun_test", "vitest.config.ts"],
+  "references": [
+    {
+      "path": "./tsconfig.build.json"
+    }
+  ]
+}

packages/graphql-server/tsconfig.json

@@ -6,6 +6,9 @@
     {
       "path": "./tsconfig.build.json"
     },
+    {
+      "path": "./tsconfig.bun.json"
+    },
     {
       "path": "./tsconfig.spec.json"
     }

packages/graphql-server/tsconfig.spec.json

@@ -5,7 +5,7 @@
     "types": ["vitest/globals"]
   },
   "exclude": ["bun_test"],
-
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/hello/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/hello",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/medley-router/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/medley-router",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/node-ws/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/node-ws",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/oauth-providers/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/oauth-providers",
     "types": ["vitest/globals"]
   },
-  "include": ["mocks.ts", "**/*.test.ts"],
+  "include": ["mocks.ts", "**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/oidc-auth/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/oidc-auth",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/otel/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/oidc-auth",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/prometheus/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/prometheus",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/react-renderer/tsconfig.spec.json

@@ -8,7 +8,7 @@
     "outDir": "../../dist/out-tsc/packages/react-renderer",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.tsx"],
+  "include": ["**/*.test.tsx", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/sentry/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/sentry",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/standard-validator/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/standard-validator",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/swagger-editor/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/swagger-editor",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"

packages/swagger-ui/src/option-renderer.test.ts

@@ -1,5 +1,3 @@
-/*eslint quotes: ["off", "single"]*/
-
 import type { DistSwaggerUIOptions } from './swagger/renderer'
 import { renderSwaggerUIOptions } from './swagger/renderer'
 

packages/swagger-ui/tsconfig.spec.json

@@ -4,7 +4,7 @@
     "outDir": "../../dist/out-tsc/packages/swagger-ui",
     "types": ["vitest/globals"]
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "vitest.config.ts"],
   "references": [
     {
       "path": "./tsconfig.build.json"