Repository navigation
fix(opencode): use public Copilot host for github.com - #51895
dajiaohuang wants to merge 1 commit into
Conversation
|
The following comment was made by an LLM, it may be inaccurate: |
|
github.com OAuth accounts are sent to the enterprise-remapped Copilot host instead of Head Fork leaf kvnloo#193 recorded fail→pass on That locks the public host for github.com accounts. It does not cover enterprise hosts. |
|
@kvnloo Thanks for the review and the fork-based verification! Acknowledged the issue: github.com OAuth accounts were being incorrectly routed to the enterprise Copilot host. I'll keep the existing enterprise-host behavior as-is (head unchanged) and focus the fix strictly on the public host routing for github.com accounts. Happy to discuss the fork/branch handling approach — if there's a preferred way to structure this (e.g., separate commits, test coverage for enterprise hosts), let me know. |
Issue for this PR
Closes #45302
Type of change
What does this PR do?
GitHub.com OAuth accounts can store
enterpriseUrl: "github.com". The host builder treated that as a GitHub Enterprise domain and generatedcopilot-api.github.com. This routesgithub.comaccounts to the public Copilot API and keeps the existing enterprise host behavior.How did you verify your code works?
bun test test/plugin/github-copilot-models.test.tsfrompackages/opencodebun run typecheckfrompackages/opencodeScreenshots / recordings
Not applicable.
Checklist