Skip to content

Conversation

@lishaduck
Copy link
Contributor

@lishaduck lishaduck commented Oct 22, 2025

Warning

Breaking Change

Drop support for Vite <6

Overview

Drop vite-node in favor of Vite-native runnerImport, which is based on ModuleRunner.

Manual Testing

Good question... not entirely sure what fails the treeshake in the first place, but all the tests pass so I assume it loads the entrypoints at least. 🤷🏻‍♂️

Related Issue

Stacked on #1945.
Supersedes/closes #1884.

@netlify
Copy link

netlify bot commented Oct 22, 2025

Deploy Preview for creative-fairy-df92c4 ready!

Name Link
🔨 Latest commit f137b01
🔍 Latest deploy log https://app.netlify.com/projects/creative-fairy-df92c4/deploys/68f85a0207997100088da048
😎 Deploy Preview https://deploy-preview-1946--creative-fairy-df92c4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@lishaduck lishaduck changed the base branch from main to major October 26, 2025 01:38
@aklinker1
Copy link
Member

If tests pass, that should be good enough.

},
"peerDependencies": {
"vite": "^5.4.19 || ^6.3.4 || ^7.0.0"
"vite": "^6.3.4 || ^7.0.0"
Copy link
Member

Choose a reason for hiding this comment

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

Perfect, was gonna call this out.

@aklinker1 aklinker1 changed the title refactor: move to module-runner refactor!: move to module-runner Oct 27, 2025
@lishaduck
Copy link
Contributor Author

lishaduck commented Oct 29, 2025

This PR is an absolute mess, but it works. Cleanup later if I have time, I need to hit the hay

I say than and then I whip out grbi. Life it better now, I can sleep in good conscience.
Apparently the issue was that #1945 updated the locked vite version but didn't sync it right, which I tried to fix in here but downgraded instead of upgrading. Fixed that 😅 And guess what? Wouldn't have been (as much of) an issue if Vitepress used a peer... lol

Life would be soooo much easier if Vitepress would just release Vite>5.
@lishaduck lishaduck force-pushed the module-runner branch 2 times, most recently from a12076a to 42a8693 Compare October 29, 2025 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants