feat: remove compatibility under React v18

src/hooks/useEffectCleanupRegister.ts

@@ -1,13 +1,9 @@
 import { warning } from 'rc-util/lib/warning';
-import * as React from 'react';
-
-const fullClone = {
-  ...React,
-};
-const { useInsertionEffect } = fullClone;
+import { useEffect } from 'react';
+import type { DependencyList } from 'react';
 
 // DO NOT register functions in useEffect cleanup function, or functions that registered will never be called.
-const useCleanupRegister = (deps?: React.DependencyList) => {
+const useEffectCleanupRegister = (deps?: DependencyList) => {
   const effectCleanups: (() => void)[] = [];
   let cleanupFlag = false;
   function register(fn: () => void) {
@@ -23,7 +19,7 @@ const useCleanupRegister = (deps?: React.DependencyList) => {
     effectCleanups.push(fn);
   }
 
-  React.useEffect(() => {
+  useEffect(() => {
     // Compatible with strict mode
     cleanupFlag = false;
     return () => {
@@ -37,14 +33,4 @@ const useCleanupRegister = (deps?: React.DependencyList) => {
   return register;
 };
 
-const useRun = () => {
-  return function (fn: () => void) {
-    fn();
-  };
-};
-
-// Only enable register in React 18
-const useEffectCleanupRegister =
-  typeof useInsertionEffect !== 'undefined' ? useCleanupRegister : useRun;
-
 export default useEffectCleanupRegister;

src/hooks/useGlobalCache.tsx

@@ -1,9 +1,9 @@
 import * as React from 'react';
 import { pathKey, type KeyType } from '../Cache';
 import StyleContext from '../StyleContext';
-import useCompatibleInsertionEffect from './useCompatibleInsertionEffect';
 import useEffectCleanupRegister from './useEffectCleanupRegister';
 import useHMR from './useHMR';
+import { useInsertionEffect } from 'react';
 
 export type ExtractStyle<CacheValue> = (
   cache: CacheValue,
@@ -74,20 +74,13 @@ export default function useGlobalCache<CacheType>(
   const cacheContent = cacheEntity![1];
 
   // Remove if no need anymore
-  useCompatibleInsertionEffect(
+  useInsertionEffect(
     () => {
       onCacheEffect?.(cacheContent);
-    },
-    (polyfill) => {
       // It's bad to call build again in effect.
       // But we have to do this since StrictMode will call effect twice
       // which will clear cache on the first time.
-      buildCache(([times, cache]) => {
-        if (polyfill && times === 0) {
-          onCacheEffect?.(cacheContent);
-        }
-        return [times + 1, cache];
-      });
+      buildCache(([times, cache]) => [times + 1, cache]);
 
       return () => {
         globalCache.opUpdate(fullPathStr, (prevCache) => {
@@ -100,7 +93,7 @@ export default function useGlobalCache<CacheType>(
               // With polyfill, registered callback will always be called synchronously
               // But without polyfill, it will be called in effect clean up,
               // And by that time this cache is cleaned up.
-              if (polyfill || !globalCache.opGet(fullPathStr)) {
+              if (!globalCache.opGet(fullPathStr)) {
                 onCacheRemove?.(cache, false);
               }
             });