fix(devtools): pre-bundle additional injected client deps in dev#991
fix(devtools): pre-bundle additional injected client deps in dev#991
Conversation
Extend the dev optimizeDeps.include list so Vite doesn't trigger a page reload when discovering deps imported by the runtime client plugins: `@vitejs/devtools/client/inject`, `@vitejs/devtools-kit/client`, `error-stack-parser-es`, and `vite-plugin-vue-tracer/client/overlay`. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Deploying nuxt-devtools with
|
| Latest commit: |
4d08ebc
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f8946834.nuxt-devtools.pages.dev |
| Branch Preview URL: | https://antfu-prebundle-client-deps.nuxt-devtools.pages.dev |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe module-main.ts file has been updated to include additional packages in Vite's dependency pre-bundling configuration during dev mode. The change expands the Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.Comment |
Summary
Extends the existing dev
optimizeDeps.includeblock inmodule-main.tsso Vite stops triggering a page reload when it discovers the deps imported by the injected runtime client plugins. Adds@vitejs/devtools/client/inject,@vitejs/devtools-kit/client,error-stack-parser-es, andvite-plugin-vue-tracer/client/overlayto the list that already had@vue/devtools-kitand@vue/devtools-core(originally added under #980/#981). All four are already direct deps of@nuxt/devtools.Test plan
pnpm -C playgrounds/empty exec nuxt dev— confirm the "Pre-bundle them in your nuxt.config.ts" warning listing those packages no longer appearsplaygrounds/tab-piniaandplaygrounds/tab-seo🤖 Generated with Claude Code