microsoft / microsoft/vscode-js-debug
Support evaluateName for all variable references
@connor4312 is already working on this.
Since Jan 24, 2024.
- Dominant language
- TypeScript
- Stars
- 2k
- Forks
- 373
- Avg merge
- 1d 9m
- Merged PRs (30d)
- 6
Description
Sometimes, there can be anonymous variable references, like this:
It would be nice if these variable references also have an evaluateName that can act as a js variable name for evaluate calls.
This evaluateName should be safe to use in expression, e.g. myFunction(${myVar.evaluateName}).
This means, before evaluating expressions, these evaluateNames have to be pre-processed.
If you use replaceAll, make sure the evaluateName is long enough (256 bits should be fine) to avoid any accidental replacements.
To be consistent, the evaluateName only works for the given frame id, so you cannot mix variables from different frame ids in one expression.
CDP has these things that might be helpful:
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.
Assessment
This issue has not been assessed yet.