Add GitHub Enterprise support for GitHub Copilot provider#1491
Add GitHub Enterprise support for GitHub Copilot provider#1491Proxicon wants to merge 3 commits into
Conversation
6af73f9 to
eb8ad92
Compare
…uth. When GHE host environment variables are set, provider add / TUI / device-code login use the configured enterprise GitHub host and API base for OAuth and Copilot token exchange. Without those variables, the provider keeps the existing github.com defaults, so standard GitHub Copilot users should remain on the current flow. Validated: - GHE device-code login and live Copilot chat path - default github.com endpoint behavior via focused regression tests - provider CLI/TUI configuration paths - Copilot request header contract and token exchange behavior Not included: - No change to the default GitHub Copilot host when GHE env vars/options are absent - No persisted enterprise secrets; environment-token mode reads COPILOT_GITHUB_TOKEN / GH_TOKEN / GITHUB_TOKEN at runtime
eb8ad92 to
ecd3522
Compare
|
thanks @Proxicon ! |
|
My pleasure @Aaronontheweb , ill work on the failed windows test runs/28184934691/job/83490200686 in GatewayRoutingContractTests |
Or actually does not look like the GHE provider work touched the contract relating to GatewayRoutingContractTest, StackGatewayContractTests, SpawnAgentStreamingTests Perhaps a timing on the 3 seconds waits for sub agents that could be increased? |
Ah perfect! no problem thanks for keeping me posted - just glad were getting some GHE playtime with Netclaw :-) |
Related to #1456
When GHE host environment variables are set, provider add / TUI / device-code login use the configured enterprise GitHub host and API base for OAuth and Copilot token exchange. Without those variables, the provider keeps the existing github.com defaults, so standard GitHub Copilot users remain on the current flow.
Validated
dotnet build Netclaw.slnxdotnet slopwatch analyze(reports existing unrelated timing-test warnings)./scripts/Add-FileHeaders.ps1 -VerifyNot included
COPILOT_GITHUB_TOKEN/GH_TOKEN/GITHUB_TOKENat runtimeTests:
With my GHE tenant env set TUI behavior:
Requests device login flow for the tenant:

Browser login:

Status netclaw status with daemon running:
Chat test: