Add memory inspector configurations#149
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
|
Coverage Impact ⬆️ Merging this pull request will increase total coverage on Modified Files with Diff Coverage (1)
🛟 Help
|
There was a problem hiding this comment.
Pull request overview
This PR aims to improve the default user experience when viewing embedded targets in the VS Code Memory Inspector by setting preferred default configuration values during debug task updates.
Changes:
- Add logic in debug task update flow to set default
memory-inspectorsettings when they are not already configured. - Add/adjust unit tests covering the new default-setting behavior and ensuring existing settings are not overwritten.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/debug/debug-launch-provider.ts | Sets Memory Inspector configuration defaults (and keeps existing debug-default behavior) during handleUpdateDebugTasks(). |
| src/debug/debug-launch-provider.test.ts | Adds test coverage for Memory Inspector defaults and updates existing expectations to avoid false failures. |
mguzmanm
force-pushed
the
memory_inspector_configurations
branch
from
cc45ee0 to
aadef7d
Compare
Fixes
Changes
Screenshots
Checklist