Repository navigation
fix(vite): pre-bundle deps of layers installed in node_modules - #36208
Conversation
4ff88ea to
0594f96
Compare
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. WalkthroughVite dependency optimisation now scans application files from installed Nuxt layers and registered module files. Client Merge Risk: ⚪ Minimal · up to This PR expands client dependency pre-bundling to dependencies provided by installed layers without changing server-side runtime behavior; no actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Linked Issues checkExplanation The changes address issue
✨ Finishing Touches 💡 1📝 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. Comment |
0594f96 to
3e3f006
Compare

🔗 Linked issue
resolves #36135
📚 Description
this cleans up some dead code that wasn't running, and improves how we optimise layer dependencies.
optimizeDepswhich are not installed in the user directory, we make these resolvable under pnpm.