Skip to content

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

Open
JianJroh wants to merge 1 commit intovitejs:mainfrom
JianJroh:fix/messages-list-cursor-df0020
Open

fix(core): pass null to avoid serialization error#325
JianJroh wants to merge 1 commit intovitejs: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

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.

1 participant