Skip to content

Route Codex Aliases to GPT-5.5#4219

Merged
Trecek merged 7 commits into
developfrom
codex-model-alias-gpt55-4218
Jul 9, 2026
Merged

Route Codex Aliases to GPT-5.5#4219
Trecek merged 7 commits into
developfrom
codex-model-alias-gpt55-4218

Conversation

@Trecek

@Trecek Trecek commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Codex model aliases now route sonnet, opus, and haiku to gpt-5.5, preventing
default 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.4 and
gpt-5.4-mini as 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.md

Generated via AutoSkillit

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit PR Review — Verdict: changes_requested

Comment thread src/autoskillit/core/types/_type_backend.py Outdated
Comment thread src/autoskillit/cli/doctor/_doctor_runtime.py Outdated
Comment thread tests/execution/test_model_alias_registry.py Outdated
Comment thread tests/execution/backends/test_model_translation.py Outdated

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit PR Review — Verdict: approved_with_comments

Comment thread src/autoskillit/core/types/_type_backend.py
Comment thread src/autoskillit/core/types/_type_backend.py Outdated
Comment thread tests/cli/test_doctor_runtime.py
Comment thread tests/execution/backends/test_model_translation.py
Comment thread src/autoskillit/core/types/_type_backend.py
Comment thread src/autoskillit/core/types/_type_backend.py Outdated
Comment thread src/autoskillit/core/types/_type_backend.py
Comment thread src/autoskillit/core/types/_type_backend.py Outdated

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit PR Review — Verdict: approved_with_comments

Comment thread src/autoskillit/cli/doctor/_doctor_runtime.py

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit review: warning-only findings detected. See inline comments — no blocking changes required.

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

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.

Comment thread src/autoskillit/core/types/_type_backend.py Outdated
Comment thread src/autoskillit/core/types/_type_backend.py Outdated

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit PR Review — Verdict: approved_with_comments

Comment thread src/autoskillit/core/types/_type_backend.py

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

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

Comment thread src/autoskillit/cli/doctor/_doctor_runtime.py

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

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.

@Trecek Trecek added this pull request to the merge queue Jul 9, 2026
Merged via the queue into develop with commit e0460e7 Jul 9, 2026
3 checks passed
@Trecek Trecek deleted the codex-model-alias-gpt55-4218 branch July 9, 2026 19:04
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.

1 participant