microsoft / microsoft/vscode-debugadapter-node
`DebugSession` client path conversion should work in web extensions
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 306
- Forks
- 90
- Avg merge
- 8h 36m
- Merged PRs (30d)
- 3
Description
Writing cross-platform and web cabable extensions, one is unable to call DebugSession::setDebuggerPathFormat and rely on the conversion methods for client and debugger path conversion.
Expected behavior
DebugSession::convertClientPathToDebugger and DebugSession::convertDebuggerPathToClient should work in desktop and web extensions.
Current behavior
Calling DebugSession::convertClientPathToDebugger or DebugSession::convertDebuggerPathToClient throws an exception process is not defined when DebugSession::path2uri and DebugSession::uri2path are called due to a dependency on process.platform from node.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in adapter/src/debugSession.ts with setDebuggerPathFormat, convertClientPathToDebugger, convertDebuggerPathToClient, path2uri, and uri2path. Inspect the node dependency on process.platform and verify that both conversion methods work in desktop and web extensions without a process-is-not-defined exception.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- node.js, typescript
- Domain
- devtools
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 48/100