Repository navigation
docs(model-routing): explain Copilot CLI vs SDK mode, define allowed-models, add compiling examples - #67755
Conversation
…ted examples Co-authored-by: SivaKesava1 <11771739+SivaKesava1@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
Two statements overstate CLI recovery behavior and smoke workflow validation coverage.
2 open findings
What changed in this PR
Updates the model-routing reference to match current routing behavior and observability.
Changes:
- Clarifies
allowed-modelsand model policy semantics. - Documents CLI versus SDK cross-family sub-agent behavior with examples.
- Refreshes AWF versions, smoke references, and steering notices.
| File | Description |
|---|---|
docs/src/content/docs/reference/model-routing.md |
Updates model-routing guidance, examples, limitations, and observability. |
🧠 Review effort: Balanced
|
✅ Test Quality Sentinel completed test quality analysis. No test files were added or modified in this PR. Test Quality Sentinel skipped. PR #67755 only modifies docs/src/content/docs/reference/model-routing.md (a documentation file); no Go or JavaScript test files are in scope.
|
|
@copilot address the following outstanding work in one pass:
Push the necessary fixes, reply to each listed review thread and resolve it when addressed. Ignore feedback already answered or resolved. Use the pr-finisher skill and stop when only human review or CI remains; do not trigger CI. Sous-chef head: dc125f9
|
…routing-guide Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
…ted workflows Co-authored-by: SivaKesava1 <11771739+SivaKesava1@users.noreply.github.com>
|
@copilot /pr-finisher |
The two linked review threads were updated in |
Co-authored-by: SivaKesava1 <11771739+SivaKesava1@users.noreply.github.com>
|
🎉 This pull request is included in a new release. Release: |

The model routing guide had fallen behind
main. It never said that cross-family sub-agents work only in Copilot SDK mode. It calledallowed-modelsthe request policy. It still listed threat detection as a limitation (fixed in #66670) and stated an old default AWF version. Onlydocs/src/content/docs/reference/model-routing.mdchanges.CLI mode vs SDK mode (new section)
engine.copilot-sdk: true) supports sub-agents from a different model family in the same run, because it builds one provider per wire API (Fix Copilot SDK model routing and cross-family sub-agents #67509).allowed-modelsrouting.candidateModels.models.allowed/models.blocked, linked to the glossary's Model Policy entry. Both the candidates and the sub-agent models must satisfy it.Examples and reference workflows
## agent:) setsmodel: claude-haiku-4.5.smoke-copilot-routed.mdroutes over three models and has no sub-agents.smoke-copilot-sdk-routed.mdandsmoke-pi-routed.mdroute over a single model only so their sub-agent checks stay deterministic. This is not a recommendation.Other fixes
steering_notices(Emit AWF agentTimeout for literal timeouts and show delivered steering notices in audit/logs #67597).sandbox.agent.version.sandbox.md.Compile results (required by the issue)
gh-aw was built from this branch, which is based on
b04b77e(not the7f713c7named in the issue) and has no code changes. Both examples were compiled as scratch workflows:In both, the compiled
candidateModelsmatchesallowed-models.allowedModelswas not checked.The docs build (
astro build) reports "All internal links are valid." The automated code review did not run (its model was unavailable), so the wording has not been reviewed yet. No checks were found failing onmainbecause of this PR.