feat(devtools): restore Vue DevTools through vite-plugin-vue-devtools v9 - #1099
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Deploying nuxt-devtools with
|
| Latest commit: |
9817231
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4f5af24b.nuxt-devtools.pages.dev |
| Branch Preview URL: | https://moody-hoops-find.nuxt-devtools.pages.dev |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe change adds the optional Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to No actionable regression from this change remains identified; the Vue DevTools integration is ready to merge after normal checks. Security Architecture ReviewSecurity architecture risk: 🟡 Moderate · up to The integration is limited to development and can be disabled, but it adds a default-on path to inspect application state. Its authorization behavior and the exception for its beta dependency warrant review before rollout. Retained concerns
Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
Hardening Proposals
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 5 files. (2 skipped: 2 unsupported.)
✨ 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 |
What
vuejs/devtools v9.0.0-beta.0 is now rebuilt on Vite DevTools and Devframe, so the Vue DevTools integration removed in #1056 can come back:
vite-plugin-vue-devtoolsis now a dependency of@nuxt/devtools, and the DevTools dock shows both a Nuxt and a Vue DevTools entry.How
vite-plugin-vue-devtools@^9.0.0-beta.0added to theprodcatalog and topackages/devtools.integrations/vue-devtools.ts:{ server: false }would strip thedevtoolsproperty), wrapping its dock registration with the existingskipInSSRguard so Nuxt's SSR Vite instance doesn't register a duplicate dock;virtual:vue-devtools-client(same shape as the pre-feat!: remove @vue/devtools integration #1056 integration).vueDevToolsmodule option is restored (defaulttrue) to allow opting out.vite-plugin-vue-devtools/clientadded tooptimizeDeps.include.Verified against
playgrounds/empty(Nuxt 5 nightly / Vite 8.3): the dock registers thenuxtgroup and thevue-devtoolsentry side by side, and the Vue DevTools panel renders the live component tree and state of the Nuxt app inside its iframe.This PR was created with the help of an agent.