chore: address util review feedback

Author: QDyanbing

.dumirc.ts

@@ -16,6 +16,6 @@ export default defineConfig({
   base: basePath,
   publicPath,
   alias: {
-    'rc-util/es': path.resolve(__dirname, 'src'),
+    '@rc-component/util/es': path.resolve(__dirname, 'src'),
   },
 });

.github/workflows/codeql.yml

@@ -29,15 +29,15 @@ jobs:
           persist-credentials: false
 
       - name: Initialize CodeQL
-        uses: github/codeql-action/init@8aad20d150bbac5944a9f9d289da16a4b0d87c1e
+        uses: github/codeql-action/init@411bbbe57033eedfc1a82d68c01345aa96c737d7
         with:
           languages: ${{ matrix.language }}
           queries: +security-and-quality
 
       - name: Autobuild
-        uses: github/codeql-action/autobuild@8aad20d150bbac5944a9f9d289da16a4b0d87c1e
+        uses: github/codeql-action/autobuild@411bbbe57033eedfc1a82d68c01345aa96c737d7
 
       - name: Perform CodeQL Analysis
-        uses: github/codeql-action/analyze@8aad20d150bbac5944a9f9d289da16a4b0d87c1e
+        uses: github/codeql-action/analyze@411bbbe57033eedfc1a82d68c01345aa96c737d7
         with:
           category: '/language:${{ matrix.language }}'

README.md

@@ -30,7 +30,7 @@
 - React 18+ compatible helpers used by rc-component packages.
 - Ref composition, React 19 ref compatibility checks, and portal helpers.
 - DOM helpers for focus management, dynamic styles, visibility, shadow roots, and scrollbar measurement.
-- Hooks for controlled state, layout effects, stable callbacks, memo comparison, ids, and mobile detection.
+- Hooks for controlled state, layout effects, stable callbacks, memo comparison, and ids.
 - Small data utilities for object paths, shallow props merging, omission, equality checks, and child flattening.
 
 ## Install

docs/examples/dynaymicCSS.tsx

@@ -1,5 +1,5 @@
-import type { Prepend } from 'rc-util/es/Dom/dynamicCSS';
-import { removeCSS, updateCSS } from 'rc-util/es/Dom/dynamicCSS';
+import type { Prepend } from '@rc-component/util/es/Dom/dynamicCSS';
+import { removeCSS, updateCSS } from '@rc-component/util/es/Dom/dynamicCSS';
 import React from 'react';
 
 function injectStyle(id: number, prepend?: Prepend, priority?: number) {

docs/examples/focus.tsx

@@ -1,5 +1,5 @@
+import { useLockFocus } from '@rc-component/util/es/Dom/focus';
 import React, { useRef } from 'react';
-import { useLockFocus } from '../../src/Dom/focus';
 
 export default function FocusDemo() {
   const containerRef = useRef<HTMLDivElement>(null);

docs/examples/getScrollBarSize.tsx

@@ -1,6 +1,6 @@
 import getScrollBarSize, {
   getTargetScrollBarSize,
-} from 'rc-util/es/getScrollBarSize';
+} from '@rc-component/util/es/getScrollBarSize';
 import React from 'react';
 
 const cssText = `

docs/examples/portal.tsx

@@ -1,5 +1,5 @@
 import React from 'react';
-import PortalWrapper from 'rc-util/es/PortalWrapper';
+import PortalWrapper from '@rc-component/util/es/PortalWrapper';
 
 const Demo: React.FC = () => {
   const divRef = React.useRef<HTMLDivElement>(null);

docs/examples/styleChecker.tsx

@@ -1,5 +1,5 @@
 import React from 'react';
-import { isStyleSupport } from 'rc-util/es/Dom/styleChecker';
+import { isStyleSupport } from '@rc-component/util/es/Dom/styleChecker';
 
 export default () => {
   const supportFlex = isStyleSupport('flex');

docs/examples/toArray.tsx

@@ -1,5 +1,5 @@
 import React from 'react';
-import toArray, { type Option } from 'rc-util/es/Children/toArray';
+import toArray, { type Option } from '@rc-component/util/es/Children/toArray';
 
 const initialState = {
   'no-children': false,
@@ -8,45 +8,56 @@ const initialState = {
   'nested-children': false,
   'array-of-children': false,
   'is-falsy': false,
-  'fragment': false,
-} as const
+  fragment: false,
+} as const;
 
-type Action = keyof typeof initialState
+type Action = keyof typeof initialState;
 
-const Wrapper = ({ children, ...options }: { children?: React.ReactNode } & Option) => {
+const Wrapper = ({
+  children,
+  ...options
+}: { children?: React.ReactNode } & Option) => {
   window.console.log(toArray(children, options));
 
   return children as any;
 };
 
-const DemoBox = React.memo(({ children, name }: { children?: any, name: Action }) => {
-  const [keepEmpty, setkeepEmpty] = React.useState(false);
-  const [show, setShow] = React.useState(false)
-  return (
-    <React.Fragment key={name}>
-      <label style={{ display: 'inline-block', width: 180 }}>
-        <input type="checkbox" checked={show} onChange={() => setShow(prev => !prev)} />
-        {name}
-      </label>
-      <label>
-        <input
-          type="checkbox"
-          checked={keepEmpty}
-          onChange={() => setkeepEmpty(prev => !prev)}
-          disabled={!show}
-        />
-        keepEmpty={keepEmpty ? 'true' : 'false'}
-      </label>
-      <br />
-      {show && React.cloneElement<any>(children, { keepEmpty })}
-      <br />
-    </React.Fragment>
-  )
-})
+const DemoBox = React.memo(
+  ({ children, name }: { children?: any; name: Action }) => {
+    const [keepEmpty, setkeepEmpty] = React.useState(false);
+    const [show, setShow] = React.useState(false);
+    return (
+      <React.Fragment key={name}>
+        <label style={{ display: 'inline-block', width: 180 }}>
+          <input
+            type="checkbox"
+            checked={show}
+            onChange={() => setShow(prev => !prev)}
+          />
+          {name}
+        </label>
+        <label>
+          <input
+            type="checkbox"
+            checked={keepEmpty}
+            onChange={() => setkeepEmpty(prev => !prev)}
+            disabled={!show}
+          />
+          keepEmpty={keepEmpty ? 'true' : 'false'}
+        </label>
+        <br />
+        {show && React.cloneElement<any>(children, { keepEmpty })}
+        <br />
+      </React.Fragment>
+    );
+  },
+);
 
 const App = () => (
   <>
-    <p onClick={window.console.clear}>Press F12 to open the console (Click Clear Console)</p>
+    <p onClick={window.console.clear}>
+      Press F12 to open the console (Click Clear Console)
+    </p>
 
     <DemoBox name="no-children">
       <Wrapper />
@@ -59,42 +70,42 @@ const App = () => (
     </DemoBox>
 
     <DemoBox name="multiple-children">
-      <Wrapper >
+      <Wrapper>
         <p>1</p>
         <span>2</span>
       </Wrapper>
     </DemoBox>
 
     <DemoBox name="nested-children">
-      <Wrapper >
+      <Wrapper>
         <ul>
-          {Array.from({ length: 5 }, (_, i) => <li key={i}>{i}</li>)}
+          {Array.from({ length: 5 }, (_, i) => (
+            <li key={i}>{i}</li>
+          ))}
         </ul>
       </Wrapper>
     </DemoBox>
 
     <DemoBox name="array-of-children">
-      <Wrapper >
-        {
+      <Wrapper>
+        {[
           [
             [
-              [
-                <span key="1">111</span>,
-                <span key="2">222</span>,
-                <span key="3">333</span>
-              ],
-              <span key="4">444</span>,
-              <span key="5">555</span>
+              <span key="1">111</span>,
+              <span key="2">222</span>,
+              <span key="3">333</span>,
             ],
-            <span key="6">666</span>,
-            <span key="7">777</span>
-          ]
-        }
+            <span key="4">444</span>,
+            <span key="5">555</span>,
+          ],
+          <span key="6">666</span>,
+          <span key="7">777</span>,
+        ]}
       </Wrapper>
     </DemoBox>
 
     <DemoBox name="is-falsy">
-      <Wrapper >
+      <Wrapper>
         {null}
         {undefined}
         {false}
@@ -106,18 +117,16 @@ const App = () => (
     </DemoBox>
 
     <DemoBox name="fragment">
-      <Wrapper >
+      <Wrapper>
         <React.Fragment />
-        <React.Fragment >A</React.Fragment>
-        <React.Fragment >
+        <React.Fragment>A</React.Fragment>
+        <React.Fragment>
           <React.Fragment />
-          <React.Fragment >B</React.Fragment>
+          <React.Fragment>B</React.Fragment>
         </React.Fragment>
       </Wrapper>
     </DemoBox>
   </>
 );
 
-
-
-export default App;
+export default App;

tsconfig.json

@@ -11,8 +11,7 @@
       "@/*": ["src/*"],
       "@@/*": [".dumi/tmp/*"],
       "@rc-component/util": ["src/index.ts"],
-      "rc-util": ["src/index.ts"],
-      "rc-util/es/*": ["src/*"]
+      "@rc-component/util/es/*": ["src/*"]
     }
   },
   "include": [".dumirc.ts", ".fatherrc.ts", "docs", "src", "tests"]