Merge branch 'main' into fix/bug-2697

Author: dai-shi

.eslintrc.json

@@ -111,13 +111,14 @@
     {
       "files": ["tests/**/*.ts", "tests/**/*.tsx"],
       "rules": {
-        "import/extensions": ["error", "never"]
+        "import/extensions": ["error", "never"],
+        "@typescript-eslint/no-unused-vars": "off"
       }
     },
     {
       "files": ["./*.js"],
       "rules": {
-        "@typescript-eslint/no-var-requires": "off"
+        "@typescript-eslint/no-require-imports": "off"
       }
     }
   ]

package.json

@@ -74,11 +74,11 @@
     "build:react:utils": "rollup -c --config-react_utils --client-only",
     "postbuild": "pnpm patch-d-ts && pnpm copy && pnpm patch-ts3.8 && pnpm patch-old-ts && pnpm patch-esm-ts && pnpm patch-readme",
     "prettier": "prettier '*.{js,json,md}' '{src,tests,benchmarks,docs}/**/*.{ts,tsx,md,mdx}' --write",
-    "eslint": "eslint --fix --no-eslintrc --c .eslintrc.json '*.{js,json,ts}' '{src,tests,benchmarks}/**/*.{ts,tsx}'",
+    "eslint": "eslint --fix --no-eslintrc --c .eslintrc.json '*.{js,ts}' '{src,tests,benchmarks}/**/*.{ts,tsx}'",
     "test": "pnpm run '/^test:.*/'",
     "test:format": "prettier '*.{js,json,md}' '{src,tests,benchmarks,docs}/**/*.{ts,tsx,md,mdx}' --list-different",
     "test:types": "tsc --noEmit",
-    "test:lint": "eslint --no-eslintrc --c .eslintrc.json '*.{js,json,ts}' '{src,tests,benchmarks}/**/*.{ts,tsx}'",
+    "test:lint": "eslint --no-eslintrc --c .eslintrc.json '*.{js,ts}' '{src,tests,benchmarks}/**/*.{ts,tsx}'",
     "test:spec": "vitest run",
     "test-build:spec": "vitest run",
     "patch-d-ts": "node -e \"var {entries}=require('./rollup.config.js');require('shelljs').find('dist/**/*.d.ts').forEach(f=>{entries.forEach(({find,replacement})=>require('shelljs').sed('-i',new RegExp(' from \\''+find.source.slice(0,-1)+'\\';$'),' from \\''+replacement+'\\';',f));require('shelljs').sed('-i',/ from '(\\.[^']+)\\.ts';$/,' from \\'\\$1\\';',f)})\"",
@@ -137,8 +137,8 @@
     "@types/node": "^22.1.0",
     "@types/react": "^18.3.3",
     "@types/react-dom": "^18.3.0",
-    "@typescript-eslint/eslint-plugin": "^7.18.0",
-    "@typescript-eslint/parser": "^7.18.0",
+    "@typescript-eslint/eslint-plugin": "^8.0.1",
+    "@typescript-eslint/parser": "^8.0.1",
     "@vitest/coverage-v8": "^2.0.5",
     "@vitest/ui": "^2.0.5",
     "benny": "^3.7.1",

pnpm-lock.yaml

@@ -1,6 +1,6 @@
 import { expectType } from 'ts-expect'
 import type { TypeEqual } from 'ts-expect'
-import { it } from 'vitest'
+import { expect, it } from 'vitest'
 import { useAtom, useSetAtom } from 'jotai/react'
 import { atom } from 'jotai/vanilla'
 
@@ -29,7 +29,7 @@ it('useAtom should return the correct types', () => {
     })
     expectType<[null, (arg: number) => void]>(useAtom(writeonlyDerivedAtom))
   }
-  Component
+  expect(Component).toBeDefined()
 })
 
 it('useAtom should handle inference of atoms (#1831 #1387)', () => {
@@ -67,7 +67,7 @@ it('useAtom should handle inference of atoms (#1831 #1387)', () => {
       >
     >(true)
   }
-  Component
+  expect(Component).toBeDefined()
 })
 
 it('useAtom should handle inference of read-only atoms', () => {
@@ -84,7 +84,7 @@ it('useAtom should handle inference of read-only atoms', () => {
     expectType<[number, never]>(useField('age'))
     expectType<[boolean, never]>(useField('checked'))
   }
-  Component
+  expect(Component).toBeDefined()
 })
 
 it('useSetAtom should handle inference of atoms', () => {
@@ -119,7 +119,7 @@ it('useSetAtom should handle inference of atoms', () => {
       >
     >(true)
   }
-  Component
+  expect(Component).toBeDefined()
 })
 
 it('useAtom should handle primitive atom with one type argeument', () => {
@@ -134,5 +134,5 @@ it('useAtom should handle primitive atom with one type argeument', () => {
       >
     >(true)
   }
-  Component
+  expect(Component).toBeDefined()
 })

tests/react/types.test.tsx

@@ -1,4 +1,4 @@
-import { it } from 'vitest'
+import { expect, it } from 'vitest'
 import { useHydrateAtoms } from 'jotai/react/utils'
 import { atom } from 'jotai/vanilla'
 
@@ -31,5 +31,5 @@ it('useHydrateAtoms should not allow invalid atom types when array is passed', (
       [activeAtom, false],
     ])
   }
-  Component
+  expect(Component).toBeDefined()
 })