Skip to content

claude-notify: add PermissionRequest hook alongside Stop hook#9

Merged
congmnguyen merged 1 commit into
mainfrom
claude-notify-both-hooks
Apr 20, 2026
Merged

claude-notify: add PermissionRequest hook alongside Stop hook#9
congmnguyen merged 1 commit into
mainfrom
claude-notify-both-hooks

Conversation

@congmnguyen
Copy link
Copy Markdown
Owner

Summary

  • Add PermissionRequest hook back alongside the existing Stop hook
  • Both hooks call ~/bin/claude-notify with different messages ("Done!" vs "Needs your input!")
  • Both already skip notification when Windows Terminal is the active window via the foreground-window check in the script
  • Update docs to reflect that both hooks are active

Test plan

  • Restart Claude Code after settings change
  • Finish a task while in another window — "Done!" balloon should appear
  • Trigger a permission prompt while in another window — "Needs your input!" balloon should appear
  • Both should be silent when Windows Terminal is the active window

Notify on both Stop (task done) and PermissionRequest (needs approval).
Both hooks skip the balloon when Windows Terminal is the foreground window.
@congmnguyen congmnguyen merged commit 67de95b into main Apr 20, 2026
1 check failed
@congmnguyen congmnguyen deleted the claude-notify-both-hooks branch May 13, 2026 14:55
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