chore: strict unused check (#1163)

* chore: strict unused check

* prefer dummy expect in tests

Author: dai-shi

eslint.config.mjs

@@ -72,7 +72,7 @@ export default tseslint.config(
       ],
       '@typescript-eslint/no-explicit-any': 'off',
       '@typescript-eslint/no-unused-vars': [
-        'warn',
+        'error',
         { argsIgnorePattern: '^_', varsIgnorePattern: '^_' },
       ],
       'react-hooks/react-compiler': 'error',
@@ -95,7 +95,6 @@ export default tseslint.config(
     files: ['tests/**/*.{ts,tsx}'],
     rules: {
       'import/extensions': ['error', 'never'],
-      '@typescript-eslint/no-unused-vars': 'off',
       'vitest/consistent-test-it': [
         'error',
         { fn: 'it', withinDescribe: 'it' },

tests/proxyMap.bench.ts

@@ -63,8 +63,8 @@ TEST_SIZES.forEach((size) => {
     const testData = generateTestData(size)
 
     bench('proxyMap', () => {
-      const map = proxyMap<number, number>(testData)
-      testData.forEach(([key, value]) => {})
+      const _map = proxyMap<number, number>(testData)
+      testData.forEach(([_key, _value]) => {})
     })
   })
 
@@ -120,7 +120,7 @@ TEST_SIZES.forEach((size) => {
     bench('proxyMap', () => {
       const map = proxyMap<number, number>(testData)
       const snap = snapshot(map)
-      testData.forEach(([key, value]) => {
+      testData.forEach(([key, _value]) => {
         snap.get(key)
       })
     })
@@ -132,9 +132,9 @@ TEST_SIZES.forEach((size) => {
 
     bench('proxyMap', () => {
       const map = proxyMap<number, number>(testData)
-      const snap1 = snapshot(map)
+      const _snap1 = snapshot(map)
       map.set(oneData[0], oneData[1])
-      const snap2 = snapshot(map)
+      const _snap2 = snapshot(map)
     })
   })
 

tests/proxyMap.test.tsx

@@ -625,6 +625,7 @@ describe('proxyMap', () => {
       const state = proxyMap()
       const TestComponent = () => {
         const snap = useSnapshot(state)
+        expect(snap).toBeDefined()
 
         return (
           <>

tests/snapshot.test.ts

@@ -194,6 +194,7 @@ describe('snapshot', () => {
       }
 
       const user = new User()
+      expect(user).toBeDefined()
 
       expectTypeOf<A>().toEqualTypeOf<B>()
     })