fix(bundle): fix circular deps (#1061)

Author: makhnatkin

demo/tests/visual-tests/YfmExtensions.visual.test.tsx

@@ -23,12 +23,10 @@ test.describe('Extensions, YFM', () => {
         await mount(<YFMStories.YfmTabs />);
         await expectScreenshot();
     });
-    test('YFM HTML', async ({mount, expectScreenshot, page, wait}) => {
+    test('YFM HTML', async ({mount, expectScreenshot, wait}) => {
         await mount(<YFMStories.YfmHtmlBlock />);
 
-        // TODO: @makhnatkin Improve iframe height stabilization wait
-        await page.waitForTimeout(200);
-        await wait.markupRendered();
+        await wait.markupRendered(1000, 3);
         await expectScreenshot();
     });
     test('YFM File', async ({mount, expectScreenshot}) => {

packages/editor/package.json

@@ -26,7 +26,7 @@
     "test:cov": "jest --coverage",
     "test:watch": "jest --watchAll",
     "test:esbuild": "node tests/esbuild-test/esbuild-tester.js",
-    "test:circular-deps": "node scripts/check-circular-deps.js 1",
+    "test:circular-deps": "node scripts/check-circular-deps.js 0",
     "prepack": "cp ../../README.md ./README.md",
     "postpack": "rm -f ./README.md",
     "prepublishOnly": "pnpm run lint && pnpm run clean && pnpm run build"

packages/editor/src/bundle/HorizontalDrag.tsx

@@ -3,7 +3,7 @@ import {useCallback, useEffect, useRef, useState} from 'react';
 import {clamp} from '../lodash';
 
 import type {EditorInt} from './Editor';
-import {cnEditorComponent} from './MarkdownEditorView';
+import {cnEditorComponent} from './editor-classname';
 
 export const IN_RESIZE_CLASSNAME = 'in-resize';
 

packages/editor/src/bundle/MarkdownEditorView.tsx

@@ -12,7 +12,7 @@ import {type QAProps, useToaster} from '@gravity-ui/uikit';
 import {ErrorBoundary} from 'react-error-boundary';
 import {useEnsuredForwardedRef, useKey, useUpdate} from 'react-use';
 
-import {type ClassNameProps, cn} from '../classname';
+import type {ClassNameProps} from '../classname';
 import {i18n} from '../i18n/bundle';
 import {globalLogger} from '../logger';
 import type {ToolbarsPreset} from '../modules/toolbars/types';
@@ -25,6 +25,7 @@ import {MarkupEditorView} from './MarkupEditorView';
 import {SplitModeView} from './SplitModeView';
 import {WysiwygEditorView} from './WysiwygEditorView';
 import {useMarkdownEditorContext} from './context';
+import {cnEditorComponent} from './editor-classname';
 import {EditorSettings, type EditorSettingsProps, type SettingItems} from './settings';
 import {stickyCn} from './sticky';
 import type {MToolbarData, MToolbarItemData, WToolbarData, WToolbarItemData} from './toolbar/types';
@@ -34,7 +35,6 @@ import type {MarkdownEditorMode} from './types';
 import '../styles/styles.scss';
 import './MarkdownEditorView.scss'; // eslint-disable-line import/order
 
-export const cnEditorComponent = cn('editor-component');
 const b = cnEditorComponent;
 
 interface EditorWrapperProps extends QAProps, ToolbarConfigs, Omit<ViewProps, 'editor'> {

packages/editor/src/bundle/editor-classname.ts

@@ -0,0 +1,3 @@
+import {cn} from '../classname';
+
+export const cnEditorComponent = cn('editor-component');