-
Notifications
You must be signed in to change notification settings - Fork 5.4k
core: add support for user defined event hooks #2904
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
recheck |
443a1e9
to
cc17700
Compare
Rebased on top of latest |
@bolinfest do you see any chance of this landing? Would you mind PTAL? |
cc17700
to
6013ea6
Compare
Rebased on latest |
Please review this. Hooks are a big deal! Would be great if we could get them in Codex. |
Ping @bolinfest, could you PTAL? Seems like many are interested in this feature. |
6013ea6
to
462599b
Compare
I've been using this PR and it's useful, so thanks. However it does not allow blocking on these actions or controlling flow like claude code hooks do. |
It would be awesome to be able to “facilitate” good behavior 😁 |
462599b
to
46743ea
Compare
Excited for this! |
46743ea
to
01b505f
Compare
Can't wait for this :) |
01b505f
to
f1c4f3d
Compare
f1c4f3d
to
78dc4fc
Compare
Rebased! |
This is the point, we need this. |
Agreed, this is very much needed. Currently I'm asking the model to run |
This PR adds support for user defined event hooks (similar feature to what already exists in Claude Code).
It's an attempted fix for: #2109
See the second commit for usage and example configuration!