fix(vscode): runtime dep resolution, debug shutdown, multi-root workflow (#9330) [hotfix/v5.961]

[lambrianmsft]

Cherry-pick of #9330 (fix(vscode): Fix runtime dependency resolution, debug shutdown, and multi-root workflow creation) into hotfix/v5.961.

Source merge commit: f3634ee60d135f21276a11890bcf5a78ea5959be

Commit Type

• feature - New functionality
• fix - Bug fix
• refactor - Code restructuring without behavior change
• perf - Performance improvement
• docs - Documentation update
• test - Test-related changes
• chore - Maintenance/tooling

Risk Level

• Low - Minor changes, limited scope
• Medium - Moderate changes, some user impact
• High - Major changes, significant user/system impact

What & Why

Fixes VS Code extension regressions related to runtime dependency resolution, debug session ordering, and multi-root workspace workflow creation. This is Part 1 of 2 — the core bug fixes (Part 2 adds webview validation features + E2E tests in a follow-up PR).

Key fixes:

• Design-time API shutdown: Make stopDesignTimeApi awaitable on Windows (taskkill wait) to prevent file-lock races between debug sessions
• Binary path resolution: Permission repair, devcontainer sync checks, Windows .exe fallback, and GitHub lookup fallback telemetry
• NodeJS/DotNet/FuncCoreTools validation: Fix async dependency existence checks by properly awaiting binariesExist
• LSP installation: Retry on EBUSY/EPERM locked-file errors during extraction
• Multi-root workflow creation: Add project dropdown for workspace folder selection; fix Error: {} when creating workflows
• execTaskkill type safety: Widen to accept number | string | undefined matching childFuncPid type

Impact of Change

• Users: More reliable debug sessions (no stale file locks), proper binary detection across platforms, workflow creation works in multi-root workspaces
• Developers: resolveNodeJsCommand now exported for direct testing; sync binary existence helpers available
• System: No new dependencies; improved error handling and telemetry

Test Plan

• Unit tests added/updated
• E2E tests added/updated
• Manual testing completed
• Tested in: VS Code on Windows with multi-root workspaces, verified via 1244 extension unit tests + 220 React unit tests (all passing)

Contributors

@lambrianmsft
@copilot

Screenshots/Videos

N/A — backend/infrastructure fixes

[github-actions]

🤖 AI PR Validation Report

PR Review Results

Thank you for your submission! Here's detailed feedback on your PR title and body compliance:

✅ PR Title

• Current: fix(vscode): runtime dep resolution, debug shutdown, multi-root workflow (#9330) [hotfix/v5.961]
• Issue: None. The title is specific, uses a clear prefix, and describes the scope well.
• Recommendation: No change needed.

---

✅ Commit Type

• Properly selected (fix).
• Only one option is checked, which is correct.

---

⚠️ Risk Level

• The PR body marks Medium, but the diff includes broad, cross-cutting changes across runtime dependency resolution, Node/Func/.NET validation flows, design-time shutdown behavior, workflow creation, and several new tests. Based on scope and blast radius, I would advise High risk.
• Recommendation: Update the PR body to mark High risk to better reflect the impact.

---

✅ What & Why

• Current: A clear description of the cherry-pick scope and the major functional fixes is provided.
• Issue: None.
• Recommendation: No change needed.

---

✅ Impact of Change

• The section is present and describes user, developer, and system impact appropriately.
• Recommendation: You may optionally mention the multi-root and Windows-specific behavior more explicitly, but this is not required.

---

✅ Test Plan

• Unit tests are added/updated in the diff, so the test plan passes.
• No E2E tests are required for pass given the rubric.

---

✅ Contributors

• Contributors are listed.
• Recommendation: No change needed.

---

✅ Screenshots/Videos

• Marked N/A, which is appropriate for backend/infrastructure fixes.
• Recommendation: No change needed.

---

Summary Table

Section	Status	Recommendation
Title	✅
Commit Type	✅
Risk Level	⚠️	Change to High
What & Why	✅
Impact of Change	✅
Test Plan	✅
Contributors	✅
Screenshots/Videos	✅

---

Passes overall, but the advised risk is higher than the submitted estimate. Please update the Risk Level to High to align the PR metadata with the diff scope.

---

Last updated: Thu, 02 Jul 2026 00:56:48 GMT