Skip to content

feat(devtools): add dataInspector option + lazy-load integrations - #1061

Merged
antfu merged 1 commit into
mainfrom
perf/lazy-panel
Aug 10, 2026
Merged

feat(devtools): add dataInspector option + lazy-load integrations#1061
antfu merged 1 commit into
mainfrom
perf/lazy-panel

Conversation

@danielroe

Copy link
Copy Markdown
Member

🔗 Linked issue

📚 Description

this adds the ability to gate off the data inspector, which pulls in @devframes/plugin-data-inspector (1610 KB) + jora (1078 KB) + cac, which is a lot of code to have vite transform on every load

additionally, this avoids loading code from @devframes/plugin-code-server (260 KB) if not enabled (which it isn't by default)

Note

this is purely about runtime performance - it doesn't affect install size, of course

Base automatically changed from perf/diff-dev to main August 10, 2026 00:30
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying nuxt-devtools with  Cloudflare Pages  Cloudflare Pages

Latest commit: 885bc47
Status:⚡️  Build in progress...

View logs

@antfu
antfu merged commit eeb29a9 into main Aug 10, 2026
3 of 6 checks passed
@antfu
antfu deleted the perf/lazy-panel branch August 10, 2026 00:30
@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b319605f-d66a-4180-ae77-ea7df9a512a4

📥 Commits

Reviewing files that changed from the base of the PR and between 5ba4c3e and 885bc47.

📒 Files selected for processing (4)
  • packages/devtools-kit/src/_types/options.ts
  • packages/devtools/src/constant.ts
  • packages/devtools/src/module-main.ts
  • packages/devtools/src/server-rpc/index.ts

📝 Walkthrough

Walkthrough

The change adds an optional dataInspector module setting and enables it by default. Module initialization now conditionally sets up Data Inspector and Code Server integrations. It emits a deprecation warning when the legacy vscode option is used without Code Server. The getServerData RPC handler dynamically imports the Data Inspector integration before invoking it.

Estimated code review effort: 2 (Simple) | ~10 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch perf/lazy-panel

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants