Skip to content

Cursor pre tool use#128

Open
nickdavies wants to merge 2 commits intoldayton:mainfrom
nickdavies:cursor_pre_tool_use
Open

Cursor pre tool use#128
nickdavies wants to merge 2 commits intoldayton:mainfrom
nickdavies:cursor_pre_tool_use

Conversation

@nickdavies
Copy link
Copy Markdown
Contributor

This replaces #127. I found out that there is a bug in cursor that makes Dippy not work with the setup I recommended in the earlier docs. This makes dippy support both hooks for cursor but only one of them works today.

Dippy supports Cursor but the README had no mention of it. This adds
a Cursor configure section alongside Claude Code, covering the
hooks.json config, auto-detection behaviour, the --cursor flag, log
location, and the known single-hook-per-array bug.

https://claude.ai/code/session_01B2Dz1qooEahbdHDbLrkqm3
@nickdavies nickdavies force-pushed the cursor_pre_tool_use branch from 96cfd03 to 69c8c7e Compare March 26, 2026 19:04
The `beforeShellExecution` hook is a more natural place for Dippy but
it's currently broken because Cursor won't honor "allow" responses it
always runs it's own approval prompt after the hook unless the hook
denies the request.

I updated the docs too to call out the issue but this appears to now
work the same as claude
@nickdavies nickdavies force-pushed the cursor_pre_tool_use branch from 69c8c7e to 8a520f2 Compare March 26, 2026 19:06
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.

2 participants