Skip to content

fix: update github-copilot model from gpt-4o to gpt-4.1#2961

Merged
dgageot merged 1 commit into
docker:mainfrom
dgageot:board/65c4c750d56792bd
Jun 2, 2026
Merged

fix: update github-copilot model from gpt-4o to gpt-4.1#2961
dgageot merged 1 commit into
docker:mainfrom
dgageot:board/65c4c750d56792bd

Conversation

@dgageot
Copy link
Copy Markdown
Member

@dgageot dgageot commented Jun 2, 2026

The GitHub Copilot provider removed gpt-4o from models.dev, causing TestParseExamples in pkg/config to fail. This test validates every example config's model against the live models.dev catalog. CI fetches fresh data and fails; locally it passed due to a stale 24h cache that still contained gpt-4o.

Updated examples/github-copilot.yaml to use gpt-4.1 (a currently-available Copilot model) for both the agent's model reference and the models entry. Verified go test ./pkg/config/... passes against freshly-fetched models.dev data.

@dgageot dgageot requested a review from a team as a code owner June 2, 2026 07:13
Copy link
Copy Markdown

@docker-agent docker-agent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

This is a clean, minimal config update. The PR replaces the removed gpt-4o model with gpt-4.1 consistently across all three references in examples/github-copilot.yaml: the agent's model field, the models map key, and the models map model value. No bugs, logic errors, or consistency issues were found in the changed lines.

@dgageot dgageot merged commit 107e190 into docker:main Jun 2, 2026
8 checks passed
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.

3 participants