Skip to content

fix: change monaco-editor imports to type-only imports#6578

Open
Federicorao wants to merge 1 commit into
keephq:mainfrom
Federicorao:codex/issue-6545-1781270507
Open

fix: change monaco-editor imports to type-only imports#6578
Federicorao wants to merge 1 commit into
keephq:mainfrom
Federicorao:codex/issue-6545-1781270507

Conversation

@Federicorao

Copy link
Copy Markdown

Fixes #6545.

Change monaco-editor imports to type-only imports in MonacoCELEditor validation and completions files to prevent SSR crash. The error 'window is not defined' occurs because monaco-editor module accesses window during initialization. Two files in the MonacoCELEditor directory import from monaco-editor at module level but only use types. Changing these to type-only imports prevents the module from being loaded on the server side, fixing the SSR issue.

I ran the available lightweight check for the frontend change.

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. Bug Something isn't working labels Jun 12, 2026
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Federicorao Federicorao changed the title Change monaco-editor imports to type-only imports in MonacoCELEditor validation and completions files to prevent SSR cra fix: change monaco-editor imports to type-only imports Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛 Bug]: Window is not defined - in Frontend logs

2 participants