fix: use sonnet-level model as copilot default, not opus - #48605
Conversation
…e-sonnet-5) Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Aligns Copilot model examples with the existing claude-sonnet-5 runtime fallback.
Changes:
- Updates Go and documentation examples.
- Updates the assign-to-agent test workflow defaults and instructions.
Show a summary per file
| File | Description |
|---|---|
pkg/workflow/assign_to_agent.go |
Updates the model example comment. |
pkg/cli/workflows/test-assign-to-agent-with-model.md |
Uses Sonnet 5 in the test workflow. |
docs/src/content/docs/reference/safe-outputs.md |
Updates the assign-to-agent example. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 3/3 changed files
- Comments generated: 1
- Review effort level: Medium
| assign-to-agent: | ||
| name: "copilot" # default agent (default: "copilot") | ||
| model: "claude-opus-4.6" # default AI model (default: "auto") | ||
| model: "claude-sonnet-5" # default AI model (default: "auto") |
|
✅ PR Code Quality Reviewer completed the code quality review. |
|
✅ Design Decision Gate 🏗️ completed the design decision gate check. No ADR enforcement needed: PR does not have the 'implementation' label and has ≤100 new lines of code in business logic directories (4 additions detected, threshold is 100). |
|
🧠 Matt Pocock Skills Reviewer has completed the skills-based review. ✅ |
|
✅ Test Quality Sentinel completed test quality analysis. No test files were added or modified in this PR. Test Quality Sentinel skipped. |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
APPROVE — cosmetic-only doc/comment update
All three hunks simply replace the example model string claude-opus-4.6 with claude-sonnet-5 to match the actual runtime default (CopilotBYOKDefaultModel). No logic, config parsing, or test assertions changed.
Review notes
assign_to_agent.go: comment-only change, no behavior impact.safe-outputs.md: doc example updated to match default.test-assign-to-agent-with-model.md: test workflow default and prose updated consistently.- A prior reviewer comment already flags a related leftover
claude-opus-4.6reference incopilot-cloud-agent.mdx, which is outside this diff's changed files -- not duplicating it here.
No correctness, security, performance, or maintainability concerns found in the changed lines.
🔎 Code quality review by PR Code Quality Reviewer · sonnet50 · 10.6 AIC · ⌖ 4.28 AIC · ⊞ 7.4K
Comment /review to run again
There was a problem hiding this comment.
The changes are correct and well-scoped. All three claude-opus-4.6 references updated to claude-sonnet-5, aligning docs, code comment, and test workflow with the runtime default.
Note: docs/src/content/docs/reference/copilot-cloud-agent.mdx:53 still has claude-opus-4.6 (already flagged in an existing review comment).
🧵 Reviewed using Impeccable skills by Impeccable Skills Reviewer · sonnet46 · 16.1 AIC · ⌖ 5.18 AIC · ⊞ 5K
There was a problem hiding this comment.
Skills-Based Review 🧠
Applied /diagnosing-bugs — the fix correctly aligns docs/comments/test workflow to the runtime default (CopilotBYOKDefaultModel = claude-sonnet-5).
📋 Key Themes & Highlights
One gap to close
- Incomplete sweep:
docs/src/content/docs/reference/copilot-cloud-agent.mdx:53still referencesclaude-opus-4.6— already flagged in an existing review comment and should be updated in this PR for a complete fix.
Positive Highlights
- ✅ PR description clearly explains intent and each changed file
- ✅ All three changed callsites are consistent with each other
- ✅ Small, surgical diff — easy to verify correctness
🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · sonnet46 · 16.3 AIC · ⌖ 4.59 AIC · ⊞ 6.7K
Comment /matt to run again
|
🎉 This pull request is included in a new release. Release: |
Opus was referenced as the default/example model for Copilot in several places. The runtime default (
CopilotBYOKDefaultModel) was alreadyclaude-sonnet-5; this aligns the docs, code comment, and test workflow to match.Changes
pkg/workflow/assign_to_agent.go— update example inDefaultModelfield comment:claude-opus-4.6→claude-sonnet-5docs/src/content/docs/reference/safe-outputs.md— updateassign-to-agentYAML example:claude-opus-4.6→claude-sonnet-5pkg/cli/workflows/test-assign-to-agent-with-model.md— update workflow default model (frontmatter +workflow_dispatchinput default):claude-opus-4.6→claude-sonnet-5