François Wouts

Support require() imports

3

Preview.js currently crashes if you use require() for imports because Vite doesn't support them. This might be fixable with a Vite plugin, see https://github.com/vitejs/vite/issues/3409 for further reference.

https://discord.com/channels/969778805591908372/969778805591908375/1048554250864709652 Example addons to test with: mobile and a11y storybook

https://discord.com/channels/969778805591908372/969778805591908375/1046766196856467548

https://twitter.com/soviut/status/1585100610315358208

### Describe the bug ``` java.lang.IllegalStateException: You must not register toolwindow programmatically so early. Rework code or use ToolWindowManager.invokeLater at com.intellij.openapi.wm.impl.ToolWindowManagerImpl.getToolWindowPaneIfInitialized(ToolWindowManagerImpl.kt:490) at com.intellij.openapi.wm.impl.ToolWindowManagerImpl.registerToolWindow(ToolWindowManagerImpl.kt:928) at com.jetbrains.rdserver.toolWindow.BackendServerToolWindowManager.registerToolWindow(BackendServerToolWindowManager.kt:89) at com.previewjs.intellij.plugin.services.ProjectService.getOrCreateConsole(ProjectService.kt:135) at com.previewjs.intellij.plugin.services.ProjectService.printToConsole$lambda$1(ProjectService.kt:123) at...

https://discord.com/channels/969778805591908372/970086836384989254/980747277893259274

This doesn't make sense in normal codebases where we try to detect whether we're in a Node or browser environment. It can be added back with a simple Vite config...