bazel-contrib / bazel-contrib/vscode-bazel
Revisit debug-adapter/client.ts#setBreakPointsRequest
Open
Starlark Debuggger
type: cleanup
- Dominant language
- TypeScript
- Stars
- 296
- Forks
- 108
- Avg merge
- 3d 22h
- Merged PRs (30d)
- 5
Description
The code currently includes a workaround that sets the breakpoint twice because of external references. The linked issue is resolved, so this workaround might be safe to be removed, but needs some investigation/validation before it is removed.
Contributor guide
Research direction
Start in debug-adapter/client.ts at setBreakPointsRequest and trace the workaround that sets breakpoints twice. Investigate the resolved external issue and validate breakpoint behavior with and without the workaround. Done means the workaround is removed only if breakpoint setting remains correct.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript, vscode
- Domain
- developer-experience, tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100