chore: address review comments

Author: afc163

eslint.config.mjs

@@ -1,12 +1,16 @@
 import js from '@eslint/js';
 import { defineConfig } from 'eslint/config';
+import { dirname } from 'node:path';
+import { fileURLToPath } from 'node:url';
 import prettier from 'eslint-config-prettier';
 import jest from 'eslint-plugin-jest';
 import react from 'eslint-plugin-react';
 import reactHooks from 'eslint-plugin-react-hooks';
 import globals from 'globals';
 import tseslint from 'typescript-eslint';
 
+const tsconfigRootDir = dirname(fileURLToPath(import.meta.url));
+
 export default defineConfig([
   {
     plugins: {
@@ -15,7 +19,7 @@ export default defineConfig([
   },
   {
     linterOptions: {
-      reportUnusedDisableDirectives: 'off',
+      reportUnusedDisableDirectives: 'warn',
     },
   },
   {
@@ -26,10 +30,10 @@ export default defineConfig([
       'lib/',
       'dist/',
       'docs-dist/',
+      '.docs-dist/',
       '.dumi/',
       '.doc/',
       '.vercel/',
-      'src/index.d.ts',
     ],
   },
   {
@@ -88,7 +92,7 @@ export default defineConfig([
     languageOptions: {
       parserOptions: {
         projectService: true,
-        tsconfigRootDir: import.meta.dirname,
+        tsconfigRootDir,
       },
     },
   },

examples/fill-width.tsx

@@ -65,11 +65,13 @@ const Demo = () => {
   const measureRef = React.useRef<HTMLDivElement>(null);
 
   useLayoutEffect(() => {
-    setInputWidth(measureRef.current.offsetWidth);
+    if (measureRef.current) {
+      setInputWidth(measureRef.current.offsetWidth);
+    }
   }, [inputValue]);
 
   React.useEffect(() => {
-    inputRef.current.focus();
+    inputRef.current?.focus();
   }, []);
 
   return (

package.json

@@ -63,9 +63,7 @@
     "dumi": "^2.4.38",
     "eslint": "^9.39.4",
     "eslint-config-prettier": "^10.1.8",
-    "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-jest": "^29.15.4",
-    "eslint-plugin-jsx-a11y": "^6.10.2",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^7.1.1",
     "father": "^4.6.24",

tests/wrapper.ts

@@ -124,7 +124,24 @@ class NodeCollection {
   }
 }
 
-export function mount(element: React.ReactElement): any {
+interface MountWrapper extends ReturnType<typeof render> {
+  find: (selector: string) => NodeCollection;
+  findItems: () => NodeCollection;
+  findRest: () => NodeCollection;
+  findPrefix: () => NodeCollection;
+  findSuffix: () => NodeCollection;
+  triggerResize: (clientWidth: number) => Promise<MountWrapper>;
+  triggerItemResize: (
+    index: number,
+    offsetWidth: number,
+  ) => Promise<MountWrapper>;
+  initSize: (width: number, itemWidth: number) => Promise<MountWrapper>;
+  setProps: (props: Record<string, any>) => MountWrapper;
+  update: () => MountWrapper;
+  render: () => ChildNode | null;
+}
+
+export function mount(element: React.ReactElement): MountWrapper {
   let mergedElement = element;
   const result = render(mergedElement);
 
@@ -154,7 +171,7 @@ export function mount(element: React.ReactElement): any {
     await flushResize();
   };
 
-  const wrapper = {
+  const wrapper: MountWrapper = {
     ...result,
     find(selector: string) {
       if (selector === 'ResizeObserver') {