Skip to content

Add opusplan builtin alias to Claude model routing#34263

Merged
pelikhan merged 3 commits into
mainfrom
copilot/add-opus-and-opusplan-aliases
May 23, 2026
Merged

Add opusplan builtin alias to Claude model routing#34263
pelikhan merged 3 commits into
mainfrom
copilot/add-opus-and-opusplan-aliases

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 23, 2026

ANTHROPIC_MODEL=opusplan was not recognized by AWF, so Claude workflows silently fell through the alias chain and often resolved to Sonnet via any. This change makes opusplan a first-class builtin alias and maps it to Opus with high reasoning effort.

  • Alias table update
    • Added opusplan to builtin model aliases in pkg/workflow/data/model_aliases.json.
    • Mapping is parameterized (not a raw provider pattern) so it composes with existing alias resolution:
"opusplan": ["opus?effort=high"]
  • Behavioral coverage in tests

    • Updated pkg/workflow/model_aliases_test.go to:
      • assert opusplan exists in the builtin alias set,
      • assert it resolves to ["opus?effort=high"].
  • Reference docs alignment

    • Updated docs/src/content/docs/reference/model-tables.md to document opusplan and clarify that some meta-aliases include baked-in model parameters (e.g., effort=high).

pr-sous-chef: updated branch to refresh mergeability. Run: https://github.com/github/gh-aw/actions/runs/26339837877

Generated by 👨‍🍳 PR Sous Chef · ● 3.5M ·

Copilot AI and others added 2 commits May 23, 2026 17:32
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add support for opus and opusplan model aliases in claude engine Add opusplan builtin alias to Claude model routing May 23, 2026
Copilot AI requested a review from pelikhan May 23, 2026 17:33
@pelikhan pelikhan marked this pull request as ready for review May 23, 2026 17:36
Copilot AI review requested due to automatic review settings May 23, 2026 17:36
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new built-in model alias (opusplan) to prevent ANTHROPIC_MODEL=opusplan from falling through to broader aliases (e.g., any) and unintentionally resolving to lower-tier models, by explicitly routing it to Opus with effort=high.

Changes:

  • Added opusplan to the embedded builtin alias map, expanding to opus?effort=high.
  • Extended unit tests to assert opusplan is present and maps exactly to ["opus?effort=high"].
  • Updated reference documentation to include opusplan and note that some meta-aliases include baked-in model parameters.
Show a summary per file
File Description
pkg/workflow/data/model_aliases.json Adds the new builtin alias opusplan expanding to opus?effort=high.
pkg/workflow/model_aliases_test.go Verifies opusplan exists in builtin aliases and resolves to the expected parameterized target.
docs/src/content/docs/reference/model-tables.md Documents opusplan and clarifies that meta-aliases may embed model parameters.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 3/3 changed files
  • Comments generated: 0

@github-actions
Copy link
Copy Markdown
Contributor

@copilot refresh the branch view and rerun checks after the update.

Generated by 👨‍🍳 PR Sous Chef · ● 3.5M ·

@pelikhan pelikhan merged commit fdbf473 into main May 23, 2026
4 checks passed
@pelikhan pelikhan deleted the copilot/add-opus-and-opusplan-aliases branch May 23, 2026 18:36
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.

Add support for opus and opusplan model aliases in the claude engine

3 participants