Skip to content

Fix github_copilot; fix local MCP servers without oauth - #647

Closed
ekr1 wants to merge 4 commits into
cecli-dev:mainfrom
ekr1:main
Closed

ekr1 wants to merge 4 commits into
cecli-dev:mainfrom
ekr1:main

Conversation

@ekr1

@ekr1 ekr1 commented Aug 17, 2026

Copy link
Copy Markdown

Fix 1: github_copilot/... models work again.

After v1.1.0 the LiteLLM usage was restructured, which broke models of the format github_copilot/... for me. The error was:

agent> Hello
Client error '401 Unauthorized' for url 'https://api.anthropic.com/v1/messages'
For more information check: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/401
The API provider is not able to authenticate you. Check your API key.

https://api.anthropic.com/v1/messages

As you can see, it routed models like github_copilot/claude-sonnet-5 directly to Anthropic's API, instead of Github Copilot's.

Also, there was this warning when selecting such a model:

agent> /agent-model github_copilot/claude-sonnet-5

Warning for github_copilot/claude-sonnet-5: Unknown which environment variables are required.

Fix 2: local MCP servers with old-style password auth (instead of oauth) could not be used anymore; cecli gave some oauth error. This PR makes it so in this case cecli does not enter the oauth path anymore.

@dwash96

dwash96 commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

The copilot fix already exists in v1.2.2 with:
7924004

and the MCP fix is basically what was done for:
#633

Can you check out the v1.2.2rc2 pre-release version to confirm that these points are addressed?

@dwash96

dwash96 commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

I'll actually just push v1.2.2 as the official shortly and we can track if this PR still needs to exists

@ekr1

ekr1 commented Aug 18, 2026

Copy link
Copy Markdown
Author

Can confirm this PR is obsolete (current upstream v1.2.3.dev3).

@ekr1 ekr1 closed this Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants