Route Codex Aliases to GPT-5.5#4219
Conversation
Trecek
left a comment
There was a problem hiding this comment.
AutoSkillit PR Review — Verdict: changes_requested
Trecek
left a comment
There was a problem hiding this comment.
AutoSkillit PR Review — Verdict: approved_with_comments
Trecek
left a comment
There was a problem hiding this comment.
AutoSkillit PR Review — Verdict: approved_with_comments
Trecek
left a comment
There was a problem hiding this comment.
AutoSkillit review: warning-only findings detected. See inline comments — no blocking changes required.
Trecek
left a comment
There was a problem hiding this comment.
AutoSkillit PR Review — Verdict: approved_with_comments
Warning-level findings detected: 1 mutation-after-import concern, 1 lossy return type concern. No blocking issues. See inline comments.
Trecek
left a comment
There was a problem hiding this comment.
AutoSkillit PR Review — Verdict: approved_with_comments
Trecek
left a comment
There was a problem hiding this comment.
AutoSkillit PR Review — Verdict: approved_with_comments
AutoSkillit review: warning-only findings detected. See inline comments — no blocking changes required.
⚠️ Outside Diff Range
These findings target lines not in the diff and could not be posted as inline comments:
tests/execution/test_model_alias_registry.py
- L24 [warning/duplication]: Hardcoded model ID strings {'sonnet': 'gpt-5.5', 'opus': 'gpt-5.5', 'haiku': 'gpt-5.5'} in monkeypatch duplicate the sam
- L48 [warning/duplication]: Same hardcoded model ID strings as L24. Centralizing the alias monkeypatch into a fixture/helper would reduce duplicatio
Trecek
left a comment
There was a problem hiding this comment.
AutoSkillit review passed. No blocking issues found — all new findings match prior resolved threads (within ±5 line tolerance) on the same files and are suppressed by the prior-resolution rule.
Summary
Codex model aliases now route
sonnet,opus, andhaikutogpt-5.5, preventingdefault Codex fleet sessions from launching on forbidden 5.4-era model IDs. The local
class distinction is preserved through
model_reasoning_effort.The doctor allowlist and model-alias regression tests now reject
gpt-5.4andgpt-5.4-minias Codex alias targets.Closes #4218
Implementation Plan
Plan file:
/home/talon/projects/generic_automation_mcp/.autoskillit/temp/prepare-pr/plan_codex_model_alias_gpt55_4218.mdGenerated via AutoSkillit