Skip to content

Remove model alias resolution for inline agent model references#2206

Merged
dgageot merged 1 commit intodocker:mainfrom
dgageot:avoid-model-lookup
Mar 21, 2026
Merged

Remove model alias resolution for inline agent model references#2206
dgageot merged 1 commit intodocker:mainfrom
dgageot:avoid-model-lookup

Conversation

@dgageot
Copy link
Copy Markdown
Member

@dgageot dgageot commented Mar 21, 2026

This avoids seeing e.g. claude-haiku-4-5-20251001 when using inline mode anthropic/claude-haiku-4-5.

Assisted-By: docker-agent

@dgageot dgageot requested a review from a team as a code owner March 21, 2026 17:16
Copy link
Copy Markdown

@docker-agent docker-agent bot 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

No bugs detected in this change. The code removal is clean and complete:

  • Removes inline model alias resolution logic for agent model references
  • Updates all test expectations to match the new behavior
  • No dangling references or incomplete removals

The change achieves its stated goal of avoiding resolved model names (e.g., claude-haiku-4-5-20251001) when using inline mode (anthropic/claude-haiku-4-5).

@dgageot dgageot merged commit 4b3a638 into docker:main Mar 21, 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.

2 participants