@geckou/eslint-config 0.2.0 → 0.3.0

package/README.md

@@ -68,6 +68,9 @@ export default [
 
 ## 方針
 
+- **ESLint 本体の推奨（`eslint:recommended`）を含む。** typescript-eslint の `recommended` は
+  これを含まないため、各プリセットで明示的に足している（`no-useless-escape` / `no-empty` /
+  `no-fallthrough` など）。`no-undef` は TypeScript が見るので `.ts` / `.tsx` / `.vue` では無効
 - **フォーマット系のルールは持たない。** Prettier に委譲する（`eslint-config-prettier` で無効化済み）。
   フォーマットは [`@geckou/prettier-config`](https://www.npmjs.com/package/@geckou/prettier-config) を使う
 - 共通の値（無視するパス・共有ルール）は `rules.js` に置き、各プリセットから参照する。

package/expo.js

@@ -10,6 +10,7 @@
 import path from 'node:path'
 import { fileURLToPath } from 'node:url'
 
+import js from '@eslint/js'
 import { FlatCompat } from '@eslint/eslintrc'
 import typescriptEslint from '@typescript-eslint/eslint-plugin'
 import tsParser from '@typescript-eslint/parser'
@@ -22,6 +23,7 @@ const compat = new FlatCompat({
 
 export default [
   { ignores: [...commonIgnores, '.expo/**'] },
+  js.configs.recommended,
   ...compat.extends('expo', 'prettier'),
   {
     plugins: {

package/index.js

@@ -8,6 +8,7 @@
 // Next.js / Expo のアプリは ./next / ./expo を使う。プリセットは互いに
 // 独立していて、重ねて使わない（同じプラグインを別々の実体で登録すると
 // ESLint が Cannot redefine plugin で落ちるため）。
+import js from '@eslint/js'
 import prettier from 'eslint-config-prettier'
 import tseslint from 'typescript-eslint'
 
@@ -15,6 +16,9 @@ import { commonIgnores, sharedRules, typescriptRules } from './rules.js'
 
 export default tseslint.config(
   { ignores: commonIgnores },
+  // ESLint 本体の推奨（no-useless-escape / no-empty など）。typescript-eslint の
+  // recommended はこれを含まないので、明示的に足す
+  js.configs.recommended,
   ...tseslint.configs.recommended,
   prettier,
   {

package/next.js

@@ -11,6 +11,7 @@
 import path from 'node:path'
 import { fileURLToPath } from 'node:url'
 
+import js from '@eslint/js'
 import { FlatCompat } from '@eslint/eslintrc'
 
 import { commonIgnores, sharedRules } from './rules.js'
@@ -21,6 +22,7 @@ const compat = new FlatCompat({
 
 export default [
   { ignores: [...commonIgnores, '.next/**', 'next-env.d.ts'] },
+  js.configs.recommended,
   ...compat.extends('next/core-web-vitals', 'next/typescript', 'prettier'),
   {
     rules: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geckou/eslint-config",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Shared ESLint flat config for geckou projects (base / next / expo / vue / react)",
   "type": "module",
   "main": "./index.js",
@@ -38,6 +38,7 @@
   },
   "dependencies": {
     "@eslint/eslintrc": "^3.0.0",
+    "@eslint/js": "^9.0.0",
     "@typescript-eslint/eslint-plugin": "^8.0.0",
     "@typescript-eslint/parser": "^8.0.0",
     "eslint-config-expo": "~8.0.0",

package/react.js

@@ -9,6 +9,7 @@
 // Hooks のルール（依存配列の漏れ・条件付き呼び出し）はテストで拾えない
 // ため、このプリセットの主目的はそこにある。
 import reactHooks from 'eslint-plugin-react-hooks'
+import js from '@eslint/js'
 import prettier from 'eslint-config-prettier'
 import tseslint from 'typescript-eslint'
 
@@ -16,6 +17,7 @@ import { commonIgnores, sharedRules } from './rules.js'
 
 export default [
   { ignores: commonIgnores },
+  js.configs.recommended,
   ...tseslint.configs.recommended,
   prettier,
   {

package/vue.js

@@ -8,6 +8,7 @@
 // .vue / .ts の両方を検査する。React と Vue が同居するリポジトリでは
 // ./react を配列で後ろに並べる（プラグインの実体は各プリセットで
 // 別々に登録されるため、files で対象を分けている限り衝突しない）。
+import js from '@eslint/js'
 import pluginVue from 'eslint-plugin-vue'
 import prettier from 'eslint-config-prettier'
 import tseslint from 'typescript-eslint'
@@ -17,6 +18,7 @@ import { commonIgnores, sharedRules, vueRules } from './rules.js'
 
 export default [
   { ignores: commonIgnores },
+  js.configs.recommended,
   ...tseslint.configs.recommended,
   ...pluginVue.configs['flat/recommended'],
   prettier,
@@ -32,6 +34,15 @@ export default [
       },
     },
   },
+  {
+    // .vue の script は TypeScript なので、未定義の識別子は vue-tsc が見る。
+    // typescript-eslint は .ts にしか no-undef の無効化を当てないため、
+    // ここで .vue にも当てる（当てないとブラウザのグローバルが全部 error になる）
+    files: ['**/*.vue'],
+    rules: {
+      'no-undef': 'off',
+    },
+  },
   {
     rules: {
       ...sharedRules,