Skip to content

ci: pin codex-action to v1.11 - #1667

Merged
tiann merged 1 commit into
tiann:mainfrom
junmo-kim:fix/codex-action-pin
Aug 22, 2026
Merged

ci: pin codex-action to v1.11#1667
tiann merged 1 commit into
tiann:mainfrom
junmo-kim:fix/codex-action-pin

Conversation

@junmo-kim

@junmo-kim junmo-kim commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

All three Codex workflows (codex-pr-review, codex-mention-response, issue-auto-response) have been failing since codex-action v1.12 landed (~2026-08-20 16:00 UTC): its new protected-run validation rejects the deliberate model_providers.codex-action-responses-proxy.* stream-retry overrides in codex-args under drop-sudo.

This pins all three workflows to v1.12 and moves the retry tuning into a trusted codex-home config.toml seeded by an earlier step — the action preserves existing config content, so the settings survive intact and the validation is satisfied. Pinning also stops the workflows from floating, which is what turned an upstream release into a repo-wide outage.

@junmo-kim
junmo-kim force-pushed the fix/codex-action-pin branch from a34005b to 428e546 Compare August 21, 2026 16:00
…-home

The floating @v1 tag picked up v1.12, whose protected-run validation
rejects model_providers.* overrides passed through codex-args under
drop-sudo, breaking all three Codex workflows. Pin the action to v1.12
and seed a trusted codex-home config.toml with the same settings in an
earlier step instead - the action preserves existing config content.
@junmo-kim
junmo-kim force-pushed the fix/codex-action-pin branch from 428e546 to bec4607 Compare August 21, 2026 16:01
@junmo-kim

Copy link
Copy Markdown
Contributor Author

@tiann This PR fixes the repo-wide pr-review failure across all three Codex workflows. Since v1.12, the action rejects the deliberate stream-retry overrides in codex-args under drop-sudo, and because these run via pull_request_target, every open PR shows the same error until this merges.

@tiann
tiann merged commit e5ffb24 into tiann:main Aug 22, 2026
2 of 3 checks passed
@junmo-kim
junmo-kim deleted the fix/codex-action-pin branch August 22, 2026 04:55
junmo-kim added a commit to junmo-kim/hapi that referenced this pull request Aug 22, 2026
tiann#1667 pinned @v1.12 and moved the flaky-relay stream retry tuning from
codex-args into a seeded codex-home config.toml, but the action appends
its own [model_providers.codex-action-responses-proxy] table instead of
merging, so every Codex workflow fails with a TOML duplicate key error.
The overrides cannot be expressed on v1.12 at all.

Pin all three workflows to v1.11, the last release that accepts these
deliberate codex-args overrides under drop-sudo, restoring the original
behavior.
tiann pushed a commit that referenced this pull request Aug 23, 2026
#1667 pinned @v1.12 and moved the flaky-relay stream retry tuning from
codex-args into a seeded codex-home config.toml, but the action appends
its own [model_providers.codex-action-responses-proxy] table instead of
merging, so every Codex workflow fails with a TOML duplicate key error.
The overrides cannot be expressed on v1.12 at all.

Pin all three workflows to v1.11, the last release that accepts these
deliberate codex-args overrides under drop-sudo, restoring the original
behavior.
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