Skip to content

fix(markup): fix circular deps#1057

Merged
makhnatkin merged 5 commits intomainfrom
fix/circular-deps-markup
Apr 17, 2026
Merged

fix(markup): fix circular deps#1057
makhnatkin merged 5 commits intomainfrom
fix/circular-deps-markup

Conversation

@makhnatkin
Copy link
Copy Markdown
Collaborator

Two cycle categories:

Category Dfiles-upload-facet.ts passed FilesUploadPlugin.extension to Facet.define({enables}).
CodeMirror evaluates enables at module load time, not call time. FilesUploadPlugin imported
FileUploadHandlerFacet — both files deadlocked at startup.
Fix: remove enables, register FilesUploadPlugin.extension explicitly in create.ts.

Category Acommands/inline.ts imported DirectiveSyntaxFacet through the ../codemirror barrel.
Fix: import directly from ../codemirror/directive-facet.

@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-utils branch from 2bdc946 to 3ccab58 Compare April 16, 2026 18:46
@makhnatkin makhnatkin force-pushed the fix/circular-deps-markup branch from b88eb17 to 278df10 Compare April 16, 2026 18:47
@makhnatkin makhnatkin changed the title fix(markup): break circular deps in codemirror files-upload and commands/inline fix(markup): fix circular deps Apr 16, 2026
@makhnatkin makhnatkin force-pushed the fix/circular-deps-utils branch from 3ccab58 to d1c5ecb Compare April 16, 2026 19:09
@makhnatkin makhnatkin force-pushed the fix/circular-deps-markup branch from 278df10 to a3fc563 Compare April 16, 2026 19:09
Comment thread packages/editor/src/markup/codemirror/files-upload-facet.ts
@makhnatkin makhnatkin marked this pull request as ready for review April 16, 2026 20:07
@makhnatkin makhnatkin requested a review from d3m1d0v as a code owner April 16, 2026 20:07
@makhnatkin makhnatkin force-pushed the fix/circular-deps-utils branch from d1c5ecb to 895da92 Compare April 16, 2026 20:14
@makhnatkin makhnatkin force-pushed the fix/circular-deps-markup branch from 51bc15c to 60e2f22 Compare April 16, 2026 20:14
Base automatically changed from fix/circular-deps-utils to main April 17, 2026 07:55
@makhnatkin makhnatkin merged commit 445923b into main Apr 17, 2026
6 checks passed
@makhnatkin makhnatkin deleted the fix/circular-deps-markup branch April 17, 2026 08:04
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