Skip to content

fix(ci): make codex review post reliably and skip Dependabot PRs#329

Merged
simonholmes001 merged 1 commit intomainfrom
feature/codex-pr-review-enforcement
Mar 30, 2026
Merged

fix(ci): make codex review post reliably and skip Dependabot PRs#329
simonholmes001 merged 1 commit intomainfrom
feature/codex-pr-review-enforcement

Conversation

@simonholmes001
Copy link
Copy Markdown
Owner

Summary

  • fix Codex review script to use max_completion_tokens with GPT-5 models
  • skip Codex review execution for Dependabot PRs (pass check without posting review)
  • stop running ruleset-enforcement workflow on PR events; run on push to main and manual dispatch only

Why

PRs #310-#327 show the review workflow triggering but failing before posting any review. This change unblocks normal PR reviews and removes noisy/failing checks on Dependabot updates.

Validation

  • node --check .github/scripts/codex-pr-review.mjs
  • local pre-commit hooks passed (CLI + backend tests)

@simonholmes001 simonholmes001 merged commit 37ac935 into main Mar 30, 2026
8 of 10 checks passed
@simonholmes001 simonholmes001 deleted the feature/codex-pr-review-enforcement branch March 30, 2026 19:17
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