Implement session state tracking for "Run and Debug" button when Inte…

[Prashant Kumar Rai (8prashant)]

Fixes #13001

When C_Cpp.intelliSenseEngine is set to "disabled", the language server client is not started. Previously, cpptools.buildAndDebug.isSourceFile and cpptools.buildAndDebug.isFolderOpen context keys (which control the visibility of the "Run / Debug C/C++ File" and "Add Debug Configuration" editor action buttons) were only updated within the language server client (client.ts). Consequently, disabling IntelliSense would cause the Run/Debug shortcut buttons to never appear in the editor title bar.

This PR updates the Debugger extension initialization (Debugger/extension.ts) to track active text editor changes and keep buildAndDebugIsSourceFile and buildAndDebugIsFolderOpen up to date regardless of whether IntelliSense is enabled or disabled.

Changes

1. Added updateBuildAndDebugSessionState and registered an onDidChangeActiveTextEditor listener in DebuggerExtension.initialize.
2. Added unit tests in Extension/test/scenarios/SingleRootProject/tests/buildAndDebug.test.ts verifying session state updates for source files, header files, non-C/C++ files, and undefined editors.

Verification

• Compiled with TypeScript (yarn compile) with 0 errors.
• Ran tests via yarn test --scenario=SingleRootProject and verified all BuildAndDebug SessionState Tests pass.

[Copilot]

Pull request overview

Ensures Run/Debug editor actions remain available when IntelliSense is disabled.

Changes:

• Tracks active editor changes in the debugger extension.
• Updates build/debug session context keys.
• Adds source, header, non-C/C++, and undefined-editor tests.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
Extension/src/Debugger/extension.ts	Adds independent session-state tracking.
Extension/test/scenarios/SingleRootProject/tests/buildAndDebug.test.ts	Tests source-file state updates.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Sean McManus (sean-mcmanus)]

Found one session-state lifecycle gap that should be addressed before merging.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

🟡 Changes recommended

Active-document language-mode changes can leave the Run/Debug context stale.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 1
• Review effort level: Balanced

[Copilot]

🟢 Approval recommended

The implementation matches existing classification behavior and covers the reported regression with focused tests.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[Prashant Kumar Rai (8prashant)]

Hi Sean McManus (@sean-mcmanus),
I have addressed all the review feedback
Could you please re-review the updated changes?
Thanks.

[Sean McManus (sean-mcmanus)]

The correctness fixes now look sound. One focused test-coverage gap remains for the new event-driven paths.

[Sean McManus (sean-mcmanus)]

Prashant Kumar Rai (@8prashant) FYI, the full Run/Debug feature doesn't work since it relies on compiler info that is returned from the cpptools process, but if you have your tasks.json/launch.json already generated (with the IntelliSense engine enabled for example), then it works.

[Prashant Kumar Rai (8prashant)]

Hi Sean McManus (@sean-mcmanus), thanks for reviewing the PR and for adding the regression coverage. I have confirmed that the requested changes are included, and the latest checks are clean.

The PR is approved, but I do not have the merge permission for the upstream repository, so the Merge button is not available to me. Could you please merge PR #14719 when you have a chance?

Thanks!