-
Notifications
You must be signed in to change notification settings - Fork 688
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🐛 BUG: 3.15.0 breaks Remix HMR with "TypeError: error loading dynamically imported module" #4318
Comments
Hey! 👋 Thanks again for raising this issue. The problematic change in this case appears to be bumping to Essentially, the ready message is logged before Before (working)
After (broken)
We could fix this by delaying logs until the runtime is ready (12011ab). This isn't a very nice solution, however, we're currently working on a re-architecture of For now, I'd recommend you stay on your pinned /cc @RamIdeas for another example of how the proxy server will be useful in local mode 🙂 |
Hi @mrbbot which version of wrangler would you recommend sticking to? 3.14? or 3.9? I moved from 3.9 to 3.15 and currently experiencing this |
@pmbanugo 3.10.1 |
Hey everyone! 👋 This change has now landed in |
Confirmed to be fixed using the beta version! Thanks! |
@mrbbot I can confirm it works for me as well. Thanks! Any idea when this will get to stable release? |
@pmbanugo Assuming everything goes smoothly, we're hoping to release this early next week 👍 |
Hey! 👋 Due to a security hotfix, wrangler |
Thank you ❤️ |
Because the HMR bug is solved in wrangler v3.19.0 (cloudflare/workers-sdk#4318 (comment)), we can now pin the wrangler version to "^3.20.0".
@mrbbot Please take a look at this remix-run/remix#8288 (comment), there is still an issue with Remix Cloudflare Workers template. Thank you. |
I'm getting similar error with a proxy requires setting vite hmr export default defineConfig(({ mode }) => ({
plugins: [
remixCloudflareDevProxy(),
remix({
presets: []
}),
tsconfigPaths()
],
server: {
hmr: {
protocol: 'ws',
clientPort: 3000
}
}
})); I've actually had been able to narrow it down a bit with
yarn wrangler --version
⛅️ wrangler 3.34.2 |
Closing as this seems to be fixed in later versions of wrangler (at least, running the repro with a new version of wrangler seems fine). Please let us know if anything like this pops up again :)) |
Which Cloudflare product(s) does this pertain to?
Workers Runtime, Wrangler core
What version(s) of the tool(s) are you using?
3.15.0
What version of Node are you using?
v18.18.2
What operating system are you using?
Mac
Describe the Bug
3.15.0 has introduced an issue that breaks Remix hot reloading with the browser error "TypeError: error loading dynamically imported module".
While 3.14.0 had a different issue which was fixed in 3.15.0, @izznatsir patched 3.14.0 with the fix, and it doesn't have the "TypeError: error loading dynamically imported module" issue. So it seems another change in 3.15.0 is causing this.
This makes wrangler unusable for Remix dev, so many of us have pegged the wrangler version at 3.10.1 until it's fixed.
The repro repo contains a stock
npm create cloudflare@latest my-remix-app -- --framework=remix
with wrangler updated. Simplenpm run dev
and edit a file to trigger the issue.Please provide a link to a minimal reproduction
https://github.com/KrisBraun/wrangler-3.15.0-remix-issue
Please provide any relevant error logs
No response
The text was updated successfully, but these errors were encountered: