Skip to content

fix(bundle): fix circular deps#1061

Open
makhnatkin wants to merge 9 commits intomainfrom
fix/circular-deps-bundle
Open

fix(bundle): fix circular deps#1061
makhnatkin wants to merge 9 commits intomainfrom
fix/circular-deps-bundle

Conversation

@makhnatkin
Copy link
Copy Markdown
Collaborator

Category C — bundle/MarkdownEditorView.tsx declared and exported cnEditorComponent.
bundle/HorizontalDrag.tsx imported cnEditorComponent from MarkdownEditorView; but
MarkdownEditorView imported the HorizontalDrag component — closing the cycle.

Fix: extract cnEditorComponent to bundle/editor-classname.ts (which imports nothing from
within bundle/). Both MarkdownEditorView.tsx and HorizontalDrag.tsx import from editor-classname.ts.

After this PR: 0 circular dependencies. Threshold set to 0.

@gravity-ui
Copy link
Copy Markdown

gravity-ui bot commented Apr 13, 2026

Storybook Deployed

@gravity-ui
Copy link
Copy Markdown

gravity-ui bot commented Apr 13, 2026

🎭 Playwright Report

@makhnatkin makhnatkin force-pushed the fix/circular-deps-gpt branch from c90e94c to 7507d11 Compare April 16, 2026 18:49
@makhnatkin makhnatkin changed the title fix(bundle): extract cnEditorComponent to break MarkdownEditorView↔HorizontalDrag cycle fix(bundle): fix circular deps Apr 16, 2026
@makhnatkin makhnatkin force-pushed the fix/circular-deps-bundle branch from 926823a to ffb6e62 Compare April 16, 2026 18:50
@makhnatkin makhnatkin force-pushed the fix/circular-deps-gpt branch from 7507d11 to fee5850 Compare April 16, 2026 19:09
@makhnatkin makhnatkin force-pushed the fix/circular-deps-bundle branch from ffb6e62 to 3fcbbb7 Compare April 16, 2026 19:10
@makhnatkin makhnatkin force-pushed the fix/circular-deps-gpt branch from fee5850 to f980d0d Compare April 16, 2026 20:14
@makhnatkin makhnatkin force-pushed the fix/circular-deps-bundle branch from 3fcbbb7 to 3bf05da Compare April 16, 2026 20:14
@makhnatkin makhnatkin marked this pull request as ready for review April 17, 2026 07:12
@makhnatkin makhnatkin requested a review from d3m1d0v as a code owner April 17, 2026 07:12

import './MarkdownEditorView.scss';
import '../styles/styles.scss';
import './MarkdownEditorView.scss'; // eslint-disable-line import/order
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is very likely that this is an important order of importing styles and should not be changed unless necessary.

Base automatically changed from fix/circular-deps-gpt to main April 17, 2026 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants