Skip to content

feat(codex): add PreToolUse hook support - #2198

Closed
yangchuansheng wants to merge 1 commit into
rtk-ai:developfrom
yangchuansheng:develop
Closed

yangchuansheng wants to merge 1 commit into
rtk-ai:developfrom
yangchuansheng:develop

Conversation

@yangchuansheng

Copy link
Copy Markdown

Summary

  • add Codex CLI PreToolUse hook processing via rtk hook codex
  • patch Codex hooks.json during rtk init --codex --auto-patch
  • update Codex hook documentation and support matrix

Verification

  • cargo fmt --check
  • env -u CFLAGS cargo test hook_cmd -- --nocapture
  • env -u CFLAGS cargo test codex -- --nocapture
  • env -u CFLAGS cargo check

@CLAassistant

CLAassistant commented Jun 1, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@yangchuansheng

Copy link
Copy Markdown
Author

Any progress?

@KNaiFen

KNaiFen commented Jun 4, 2026

Copy link
Copy Markdown

we need this

@GengDaPeng

Copy link
Copy Markdown

+1

1 similar comment
@alirizagurtas

Copy link
Copy Markdown

+1

@KuSh

KuSh commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Closing in favour of #3552, which covers the same Codex PreToolUse integration, is the version a collaborator has reviewed, and uses the transparent updatedInput path that Codex now applies end to end. Thanks for the work here. Please take anything your branch handles that #3552 misses to that thread so the effort stays in one place.

@KuSh KuSh closed this Sep 6, 2026
@KuSh KuSh added the duplicate This issue or pull request already exists label Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

duplicate This issue or pull request already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants