fix: match Codex plugin hooks to supported event fields - #61
Conversation
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
|
Warning Review limit reachedNext included review available in 47 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (5)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex 0.151 reports warnings when a stop hook declares
additionalContextLimit: only startup hooks can emit that context. Keep the limit on SessionStart and SubagentStart, remove it from Stop and SubagentStop, and bump the Codex plugin to 0.3.6 so installed caches refresh.Validated with the repository package checks, including all four event forwarders and suppression when managed project hooks own the lifecycle. The warning was observed through the installed Codex host's
hooks/listAPI. This changes plugin packaging only; CLI 0.15.6 remains the runtime release.High-level PR Summary
This PR fixes a Codex 0.151 compatibility issue by removing the
additionalContextLimitfield from stop hooks (Stop and SubagentStop events) since only startup hooks can emit additional context. The field is correctly retained on SessionStart and SubagentStart hooks. The plugin version is bumped from 0.3.5 to 0.3.6 to refresh installed caches, and validation tests are updated to match the new hook configuration.⏱️ Estimated Review Time: 5-15 minutes
💡 Review Order Suggestion
plugins/tree-ring-memory/.codex-plugin/plugin.jsonplugins/tree-ring-memory/packaging/codex-skills-only/.codex-plugin/plugin.jsonplugins/tree-ring-memory/README.mdplugins/tree-ring-memory/hooks/codex-hooks.jsonscripts/validate-plugin-packages.py