microsoft / microsoft/debug-adapter-protocol
Support paginating 'loadedSources'
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 1.8k
- Forks
- 173
- Avg merge
- 7d 7h
- Merged PRs (30d)
- 2
Description
I've been looking at implementing loadedSources in lldb-dap but I'd like to be careful about returning to much data in a single request.
At the moment, the loadedSources request does not support pagination.
The modules request supports a simple pagination scheme where the request can include the starting offset and the number of items to include.
I wonder if it would be useful for paginating the loadedSources request to ensure we don't overload the client with data.
Contributor guide
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
Review the existing pagination scheme for the modules request and compare it with the loadedSources request in the protocol. Determine the expected offset and item-count behavior, client impact, and what protocol changes would define the work as complete.
Written by the indexing model from the issue text.
Assessment
- Domain
- devtools
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100