Yehowshua/ZIm
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

debugger: Fix errors in JavaScript DAP schema (#33884)

Browse source 
`program` isn't required, and in fact our built-in `JavaScript debug
terminal` configuration doesn't have it.

Also add `node-terminal` to the list of allowed types.

Co-authored-by: Michael <michael@zed.dev>

Release Notes:

- N/A
This commit is contained in:
Cole Miller 2025-07-04 13:33:10 -04:00 • committed by GitHub
parent 8fecacfbaa
commit 543a7b123a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 1 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

6
crates/dap_adapters/src/javascript.rs
View file

@ -245,7 +245,7 @@ impl DebugAdapter for JsDebugAdapter {
"properties": { "properties": {
"type": { "type": {
"type": "string", "type": "string",
"enum": ["pwa-node", "node", "chrome", "pwa-chrome", "msedge", "pwa-msedge"], "enum": ["pwa-node", "node", "chrome", "pwa-chrome", "msedge", "pwa-msedge", "node-terminal"],
"description": "The type of debug session", "description": "The type of debug session",
"default": "pwa-node" "default": "pwa-node"
}, },
@ -379,10 +379,6 @@ impl DebugAdapter for JsDebugAdapter {
} }
} }
}, },
"oneOf": [
{ "required": ["program"] },
{ "required": ["url"] }
]
} }
] ]
}, },