Skip to content

fix: support renamed Codex hooks feature#225

Open
Inference1 wants to merge 1 commit into
PolyArch:mainfrom
Inference1:fix/codex-hooks-feature-rename
Open

fix: support renamed Codex hooks feature#225
Inference1 wants to merge 1 commit into
PolyArch:mainfrom
Inference1:fix/codex-hooks-feature-rename

Conversation

@Inference1

@Inference1 Inference1 commented Jul 13, 2026

Copy link
Copy Markdown

Summary

  • detect the current hooks feature while retaining compatibility with legacy codex_hooks
  • use the detected feature name when enabling hooks and disabling hooks for nested Codex calls
  • update the Codex install guide and add regression coverage for current, legacy, dual-name, and unsupported feature sets
  • keep empty nested-hook argument arrays safe on Bash 3.2

Testing

  • bash tests/test-codex-hooks-feature.sh
  • bash tests/test-codex-hook-install.sh
  • bash tests/test-bitlesson-select-routing.sh
  • bash tests/test-disable-nested-codex-hooks.sh
  • bash tests/test-agent-teams.sh
  • bash -n on all changed shell scripts
  • isolated real-CLI dry run confirmed DRY-RUN enable hooks feature

Fixes #208

@Inference1

Copy link
Copy Markdown
Author

@tony-nowatzki Would you please review this pr?

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.

Codex 0.139.0 renamed codex_hooks to hooks, causing Codex install to fail

1 participant