Skip to content

src: do not enable the inspector during snapshot building - #67154

Open
joyeecheung wants to merge 1 commit into
nodejs:mainfrom
joyeecheung:fix-snapshot-inspector
Open

joyeecheung wants to merge 1 commit into
nodejs:mainfrom
joyeecheung:fix-snapshot-inspector

Conversation

@joyeecheung

@joyeecheung joyeecheung commented Oct 31, 2026 •

Copy link
Copy Markdown
Member

V8 no longer allows attaching a debuger or setting a debug context ID during snapshot building. Avoid enabling the inspector in this case, and reject the inspector options when the user-land --build-snapshot is used.

Refs: https://chromium-review.googlesource.com/c/v8/v8/+/8679045

V8 no longer allows attaching a debuger or setting a debug context ID
during snapshot building. Avoid enabling the inspector in this case,
and reject the inspector options when the user-land --build-snapshot
is used.

Refs: Refs: https://chromium-review.googlesource.com/c/v8/v8/+/8679045
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/config
  • @nodejs/inspector
  • @nodejs/startup

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. config Issues and PRs related to Node.js configuration and feature settings. inspector Issues and PRs related to the V8 inspector protocol. needs-ci PRs that need a full CI run. labels Oct 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. config Issues and PRs related to Node.js configuration and feature settings. inspector Issues and PRs related to the V8 inspector protocol. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants