Skip to content

feat(hooks): auto-gate Codex review before commit#428

Merged
BaseInfinity merged 1 commit into
mainfrom
feat/codex-gate-hook
Jun 23, 2026
Merged

feat(hooks): auto-gate Codex review before commit#428
BaseInfinity merged 1 commit into
mainfrom
feat/codex-gate-hook

Conversation

@BaseInfinity

Copy link
Copy Markdown
Owner

Summary

  • New codex-gate-check.sh hook — PreToolUse on Bash, blocks git commit without .reviews/handoff.json status REVIEWED or CERTIFIED
  • CODEX_GATE_SKIP=1 env var for emergency bypass with logged justification
  • Wired into project settings, CLI template, and installer (10 managed files → 11)

Test plan

  • 6 new TDD tests: blocks without artifact, allows CERTIFIED, allows REVIEWED, silent on non-commit, blocks PENDING status, respects skip override
  • 169/169 hook tests passing
  • 97/97 CLI tests passing
  • 17/17 cowork drift tests passing
  • CI green

Closes #420

@BaseInfinity BaseInfinity enabled auto-merge (squash) June 23, 2026 23:14
@BaseInfinity BaseInfinity merged commit 54f5407 into main Jun 23, 2026
4 checks passed
…del review (#420)

PreToolUse hook on Bash tool that checks for .reviews/handoff.json
with status REVIEWED or CERTIFIED before allowing git commit.
CODEX_GATE_SKIP=1 env var for emergency bypass.

Wired into project settings.json, CLI template, and installer.
6 new TDD tests (169 total hook tests passing).
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.

feat: auto-gate Codex cross-model review before commit on Class C+ changes

1 participant