Label selected connection deletion accurately - #932
Conversation
|
Copilot please run full PR checks (contributing md). fix minor issues on your own. report others here on the PR as comments. |
Co-authored-by: tnaum-ms <171359267+tnaum-ms@users.noreply.github.com>
Fixed the CI Prettier failure in 6d295ee. Full Jest (238 suites/3,687 tests), build, package, and CodeQL pass. Local lint remains blocked by 40 unrelated unsafe-type errors in |
Tomasz Naumowicz (tnaum-ms)
left a comment
There was a problem hiding this comment.
👍 Approved.
✅ Code Quality Checks
This comment is updated automatically on each push. |
📦 Build Size Report
Download artifact · updated automatically on each push. |
What changed
removeConnectionhandler.Why
The existing deletion handler already processes all selected connections, but its context-menu title remained singular during multi-selection. Separate static command aliases make the operation's scope explicit without duplicating deletion behavior.
Scope
This PR changes only command contributions, registration, and manifest coverage. It does not change deletion, confirmation, resource-conflict, telemetry, or partial-failure behavior.
Validation
npm run buildnpx jest --no-coverage src/commands/removeConnection/removeConnection.packageJson.test.ts src/commands/localQuickStart/contributions.test.ts(2 suites, 31 tests)Split from #930 so dedicated index copy/paste remains focused.