Skip to content

Conversation

@rwoll
Copy link
Member

@rwoll rwoll commented Aug 14, 2025

When testing Copilot Chat in automated scenarios with VSC as the Extension Host, Copilot Chat needs to use environment-variable based authentication as well as have some additional access to services that were previously blocked when running in ExtensionMode.Test.

  • Required moving TestAuthenticationService out of test and into platform code. Renamed to StaticGitHubAuthenticationService in an attempt to capture its impl. Though, it really is just for Test (be it stest or heavier weight e2e eval tests).
  • The token provider needs to be explicitly passed instead of defaulted due to import zone violations.
  • Added an isScenarioAutomation flag. Eventually, it would be nice to move onto EnvService, but EnvService is not yet initialized in some locations that value of the flag is needed.

@rwoll rwoll marked this pull request as ready for review August 14, 2025 21:03
@rwoll
Copy link
Member Author

rwoll commented Aug 14, 2025

/cc @connor4312 @Tyriar @bpasero

connor4312
connor4312 previously approved these changes Aug 14, 2025
@vs-code-engineering vs-code-engineering bot added this to the August 2025 milestone Aug 14, 2025
DonJayamanne
DonJayamanne previously approved these changes Aug 14, 2025
@rwoll rwoll added this pull request to the merge queue Aug 14, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 14, 2025
@connor4312 connor4312 added this pull request to the merge queue Aug 14, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Aug 14, 2025
@rwoll rwoll dismissed stale reviews from DonJayamanne and connor4312 via 0ef087d August 15, 2025 15:58
@rwoll rwoll enabled auto-merge August 15, 2025 16:17
@rwoll rwoll added this pull request to the merge queue Aug 15, 2025
Merged via the queue into microsoft:main with commit dc2b611 Aug 15, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants