Chore/lint cleanup

eslint.config.mjs

@@ -7,7 +7,10 @@ export default [
   },
   ...tsEslint.configs.recommended,
   {
-    plugins: { "@next/next": nextPlugin },
+    plugins: { 
+      "@next/next": nextPlugin,
+      "react-hooks": (await import("eslint-plugin-react-hooks")).default
+    },
     rules: {
       ...nextPlugin.configs["core-web-vitals"].rules,
       "@typescript-eslint/no-unused-vars": [

package.json

@@ -7,6 +7,7 @@
     "build": "next build",
     "start": "next start",
     "lint": "eslint",
+    "typecheck": "tsc --noEmit",
     "test": "vitest --run",
     "test:watch": "vitest",
     "test:ui": "vitest --ui",

src/hooks/useVirtualList.ts

@@ -173,7 +173,6 @@
 
     return { offsets: _offsets, heights: _heights, totalHeight: cumulative };
     // measureVersion dependency triggers recomputation after DOM measurement.
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [items.length, estimatedItemHeight, measureVersion]);
 
   // -----------------------------------------------------------------------
@@ -311,7 +310,7 @@
       const prev = measuredHeights.current.get(index);
       if (prev !== measured) {
         measuredHeights.current.set(index, measured);
         setMeasureVersion((v) => v + 1);
       }
     },
     []

tests/hooks/useVirtualList.test.ts

@@ -15,32 +15,7 @@
 }
 
 /** Stub the container ref with controllable scrollTop / clientHeight. */
-function mockContainerRef(
-  hook: ReturnType<typeof renderHook<ReturnType<typeof useVirtualList>, unknown>>,
-  scrollTop = 0,
-  clientHeight = 400
-) {
-  const div = document.createElement("div");
-
-  Object.defineProperties(div, {
-    scrollTop: { value: scrollTop, writable: true },
-    clientHeight: { value: clientHeight, configurable: true },
-    scrollTo: {
-      value: vi.fn(({ top }: ScrollToOptions) => {
-        Object.defineProperty(div, "scrollTop", {
-          value: top,
-          writable: true,
-          configurable: true,
-        });
-      }),
-      configurable: true,
-    },
-  });
 
-  // Assign the mock div to the containerRef
-  (hook.result.current.containerRef as { current: HTMLDivElement | null }).current = div;
-  return div;
-}
 
 // ---------------------------------------------------------------------------
 // Tests
@@ -219,7 +194,7 @@
     // The key should have been accessed
     const stored = sessionStorage.getItem(`vlist-scroll-${key}`);
     // It may be "0" on initial render
     expect(stored).not.toBeNull();
   });
 
   // -----------------------------------------------------------------------