fix: replace deprecated gpt-5-mini with gpt-5.4-mini in daily-model-resolution workflow#41625
Merged
Merged
Conversation
…t-5.4-mini The codex engine was mapping gpt-5-mini to gpt-5-codex-mini-alpha-2025-11-07 which no longer exists (404 Not Found), causing all retry attempts to fail. Replace gpt-5-mini with gpt-5.4-mini (confirmed working in other codex workflows like smoke-call-workflow.md) in both the main engine config and the run-analyzer sub-agent definition. Also update the guard test to expect the new model name. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix daily sub-agent model resolution audit failure
fix: replace deprecated gpt-5-mini with gpt-5.4-mini in daily-model-resolution workflow
Jun 26, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Daily Sub-Agent Model Resolution Audit workflow to avoid a deprecated gpt-5-mini codex resolution that was producing HTTP 404s, by switching to a codex-compatible gpt-5.4-mini model and aligning compiled artifacts/tests accordingly.
Changes:
- Switch the workflow’s codex engine model and
run-analyzersub-agent model fromgpt-5-minitogpt-5.4-mini. - Recompile the generated
.lock.ymlworkflow output to reflect the updated model. - Update the Go guard test to assert
gpt-5.4-minifor therun-analyzersub-agent.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/daily-model-resolution.md | Updates codex engine + run-analyzer sub-agent to use gpt-5.4-mini. |
| .github/workflows/daily-model-resolution.lock.yml | Recompiled generated workflow reflecting the new model in metadata/env. |
| pkg/workflow/prompts_test.go | Updates the workflow-content assertion to expect gpt-5.4-mini. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 3/3 changed files
- Comments generated: 1
- Review effort level: Low
Comment on lines
459
to
+466
| workflow := string(content) | ||
| agentStart := strings.Index(workflow, "## agent: `run-analyzer`") | ||
| if agentStart == -1 { | ||
| t.Fatal("Expected daily-model-resolution workflow to define the run-analyzer sub-agent") | ||
| } | ||
| agentBlock := workflow[agentStart:] | ||
| if !strings.Contains(agentBlock, "\nmodel: gpt-5-mini\n") { | ||
| t.Fatal("Expected daily-model-resolution run-analyzer sub-agent to use explicit codex-compatible model gpt-5-mini") | ||
| if !strings.Contains(agentBlock, "\nmodel: gpt-5.4-mini\n") { | ||
| t.Fatal("Expected daily-model-resolution run-analyzer sub-agent to use explicit codex-compatible model gpt-5.4-mini") |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The codex engine internally resolves
gpt-5-minitogpt-5-codex-mini-alpha-2025-11-07, which returns HTTP 404. This caused the Daily Sub-Agent Model Resolution Audit to exhaust all 4 retry attempts and fail on every scheduled run.Changes
daily-model-resolution.md— replacegpt-5-miniwithgpt-5.4-miniin both the top-level engine config and the inlinerun-analyzersub-agent definitiondaily-model-resolution.lock.yml— recompiled from updated sourcepkg/workflow/prompts_test.go— updateTestDailyModelResolutionUsesCodexCompatibleSubAgentModelguard to assertgpt-5.4-minigpt-5.4-miniis already in active use with the codex engine (e.g.smoke-call-workflow.md).