Skip to content

Fix Codex workflow by removing invalid codex-args#128

Merged
Cahllagerfeld merged 1 commit intomainfrom
bug/fix-codex-gha
Feb 4, 2026
Merged

Fix Codex workflow by removing invalid codex-args#128
Cahllagerfeld merged 1 commit intomainfrom
bug/fix-codex-gha

Conversation

@strickvl
Copy link
Copy Markdown
Contributor

@strickvl strickvl commented Feb 4, 2026

Summary

Removes the invalid codex-args line from the Codex GitHub Actions workflow. The --allowed-tools flag does not exist in codex exec, causing the workflow to fail with:

error: unexpected argument '--allowed-tools' found

Changes

  • Removed codex-args: --max-turns 5 --allowed-tools "..." from .github/workflows/codex-comment.yml

Security

The workflow retains adequate security via:

  • permissions: contents: read (cannot modify repo)
  • sandbox: workspace-write (filesystem restrictions)
  • safety-strategy: drop-sudo (no sudo access)

Note

This also removes --max-turns 5 which was bundled in the same argument. If turn limiting is needed, it would need to be configured via a different mechanism (e.g., config.toml).

Test Plan

  • Verify workflow YAML is valid
  • Trigger /codex comment on a test PR to confirm the workflow runs

The --allowed-tools flag does not exist in `codex exec`. This was
causing the workflow to fail with:

  error: unexpected argument '--allowed-tools' found

The workflow still has adequate security via:
- permissions: contents: read (cannot modify repo)
- sandbox: workspace-write (filesystem restrictions)
- safety-strategy: drop-sudo (no sudo access)

Note: This also removes --max-turns 5 which was in the same arg.
@strickvl strickvl added bug Something isn't working gtm-team Issues that are being handled by the gtm-team. labels Feb 4, 2026
@Cahllagerfeld Cahllagerfeld merged commit 16ca5bc into main Feb 4, 2026
3 checks passed
@Cahllagerfeld Cahllagerfeld deleted the bug/fix-codex-gha branch February 4, 2026 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working gtm-team Issues that are being handled by the gtm-team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants