Skip to content

fix(core): pass null to avoid serialization error#325

Merged
antfu merged 1 commit into
vitejs:mainfrom
JianJroh:fix/messages-list-cursor-df0020
May 11, 2026
Merged

fix(core): pass null to avoid serialization error#325
antfu merged 1 commit into
vitejs:mainfrom
JianJroh:fix/messages-list-cursor-df0020

Conversation

@JianJroh
Copy link
Copy Markdown
Contributor

@JianJroh JianJroh commented May 9, 2026

Summary

Caused by #301. First call to devtoolskit:internal:messages:list passed undefined as the cursor, which the strict JSON wire format rejects. Pass null instead.

Reproduction

  1. pnpm play
  2. Open browser console — see [DF0020] ... value at "[0]" is a undefined on load.

Additional context

if (original === undefined) {
if (Array.isArray(holder))
throw nonJsonAt(fnName, 'undefined', holder, key)
return val
}

@netlify
Copy link
Copy Markdown

netlify Bot commented May 9, 2026

Deploy Preview for devfra canceled.

Name Link
🔨 Latest commit dc4f925
🔍 Latest deploy log https://app.netlify.com/projects/devfra/deploys/69fed336a2369e0008bc7db6

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 9, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@vitejs/devtools@325
npm i https://pkg.pr.new/@vitejs/devtools-kit@325
npm i https://pkg.pr.new/@vitejs/devtools-rolldown@325
npm i https://pkg.pr.new/@vitejs/devtools-self-inspect@325

commit: dc4f925

@antfu antfu merged commit e491710 into vitejs:main May 11, 2026
10 checks passed
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