Skip to content

fix(hook): minor cursor fixes and code cleanups#2742

Open
KuSh wants to merge 1 commit into
rtk-ai:developfrom
KuSh:2609-follow-up
Open

fix(hook): minor cursor fixes and code cleanups#2742
KuSh wants to merge 1 commit into
rtk-ai:developfrom
KuSh:2609-follow-up

Conversation

@KuSh

@KuSh KuSh commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

Building on #2609:

  • Fix the documentation, as Cursor now uses the native Rust binary hook
  • Update rtk-rewrite.sh so it fails on any exit code other than 0 or 3
  • Test that "continue": true is also present for "permission": "ask" and sync shell rewrite hook

Test plan

  • cargo fmt --all && cargo clippy --all-targets && cargo test
  • Manual testing: rtk <command> output inspected

- Fix documentation, Cursor now uses native rust binary hook
- Fix rtk-rewrite.sh shell script to fail on any exit code except 0 and 3
- Test that "continue": true is also present for "permission": "ask" and sync shell rewrite hook
@KuSh KuSh force-pushed the 2609-follow-up branch from c6f66a1 to d9c53c9 Compare June 29, 2026 22:20
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.

1 participant