fix: rm useEffectDev (#9271)

Author: SevereCloud

packages/vkui/src/components/ActionSheet/ActionSheetDropdownMenu.tsx

@@ -3,7 +3,6 @@
 import * as React from 'react';
 import { classNames } from '@vkontakte/vkjs';
 import { useAdaptivityWithJSMediaQueries } from '../../hooks/useAdaptivityWithJSMediaQueries';
-import { useEffectDev } from '../../hooks/useEffectDev';
 import { usePlatform } from '../../hooks/usePlatform';
 import { useDOM } from '../../lib/dom';
 import { isRefObject } from '../../lib/isRefObject';
@@ -39,10 +38,12 @@ export const ActionSheetDropdownMenu = ({
   const { sizeY } = useAdaptivityWithJSMediaQueries();
   const elementRef = React.useRef<HTMLDivElement | null>(null);
 
-  useEffectDev(() => {
-    const toggleEl = getEl(toggleRef);
-    if (!toggleEl) {
-      warn(`Свойство "toggleRef" не передано`, 'error');
+  React.useEffect(() => {
+    if (process.env.NODE_ENV === 'development') {
+      const toggleEl = getEl(toggleRef);
+      if (!toggleEl) {
+        warn(`Свойство "toggleRef" не передано`, 'error');
+      }
     }
   }, [toggleRef]);
 

packages/vkui/src/components/OnboardingTooltip/OnboardingTooltip.test.tsx

@@ -130,16 +130,40 @@ describe(OnboardingTooltip, () => {
     setNodeEnv('development');
     const warn = vi.spyOn(console, 'warn').mockImplementation(noop);
 
-    const component = render(<OnboardingTooltip onClose={noop} title="title" />);
+    const component = render(
+      <OnboardingTooltipContainer>
+        <OnboardingTooltip onClose={noop} title="title">
+          <div />
+        </OnboardingTooltip>
+      </OnboardingTooltipContainer>,
+    );
     expect(warn).not.toHaveBeenCalled();
 
-    component.rerender(<OnboardingTooltip onClose={noop} aria-label="title" />);
+    component.rerender(
+      <OnboardingTooltipContainer>
+        <OnboardingTooltip onClose={noop} aria-label="title">
+          <div />
+        </OnboardingTooltip>
+      </OnboardingTooltipContainer>,
+    );
     expect(warn).not.toHaveBeenCalled();
 
-    component.rerender(<OnboardingTooltip onClose={noop} aria-labelledby="labelId" />);
+    component.rerender(
+      <OnboardingTooltipContainer>
+        <OnboardingTooltip onClose={noop} aria-labelledby="labelId">
+          <div />
+        </OnboardingTooltip>
+      </OnboardingTooltipContainer>,
+    );
     expect(warn).not.toHaveBeenCalled();
 
-    component.rerender(<OnboardingTooltip onClose={noop} />);
+    component.rerender(
+      <OnboardingTooltipContainer>
+        <OnboardingTooltip onClose={noop}>
+          <div />
+        </OnboardingTooltip>
+      </OnboardingTooltipContainer>,
+    );
 
     expect(warn.mock.calls[0][0]).toBe(
       '%c[VKUI/OnboardingTooltip] Если "title" не используется, то необходимо задать либо "aria-label", либо "aria-labelledby" (см. правило axe aria-dialog-name)',

packages/vkui/src/hooks/useEffectDev.ts

@@ -46,9 +46,7 @@ vi.mock('../lib/warnOnce', () => ({
   warnOnce: () => () => console.error('custom-error'),
 }));
 
-vi.mock('./useEffectDev', () => ({
-  useEffectDev: React.useEffect,
-}));
+vi.stubEnv('NODE_ENV', 'development');
 
 type ConsoleErrorArgs = Parameters<typeof console.error>;
 

packages/vkui/src/hooks/usePatchChildren.test.tsx

@@ -7,7 +7,6 @@ import {
 } from '../lib/utils';
 import { warnOnce } from '../lib/warnOnce';
 import type { HasRootRef } from '../types';
-import { useEffectDev } from './useEffectDev';
 import { useExternRef } from './useExternRef';
 
 const warn = warnOnce('usePatchChildren');
@@ -77,14 +76,16 @@ export const usePatchChildren = <ElementType extends HTMLElement = HTMLElement>(
 
   const patchedChildren = isValidElementResult ? React.cloneElement(children, props) : children;
 
-  useEffectDev(() => {
-    if (!childRef.current && !shouldUseRef) {
-      warn(
-        'Кажется, в children передан компонент, который не поддерживает свойство getRootRef. Мы не можем получить ссылку на корневой dom-элемент этого компонента',
-        'error',
-      );
+  React.useEffect(() => {
+    if (process.env.NODE_ENV === 'development') {
+      if (!childRef.current && !shouldUseRef) {
+        warn(
+          'Кажется, в children передан компонент, который не поддерживает свойство getRootRef. Мы не можем получить ссылку на корневой dom-элемент этого компонента',
+          'error',
+        );
+      }
     }
-  }, [isValidElementResult ? children.type : null, shouldUseRef, childRef]);
+  }, [shouldUseRef, childRef]);
 
   return [childRef, patchedChildren];
 };