Skip to content

Fix Claude alert changes not applying immediately - #58

Merged
mylee04 merged 1 commit into
mainfrom
agent/fix-claude-permission-alerts
Aug 4, 2026
Merged

Fix Claude alert changes not applying immediately#58
mylee04 merged 1 commit into
mainfrom
agent/fix-claude-permission-alerts

Conversation

@mylee04

@mylee04 mylee04 commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • apply cn alerts add, remove, and reset changes immediately when Claude hooks are already enabled
  • repair stale Claude notification matchers when an already-enabled alert type is added again
  • add regression coverage and update the README behavior notes

Root cause

Alert commands updated the persisted alert-type list but left the installed Claude Notification matcher unchanged until the user ran cn on again. The README examples did not make that extra step clear, so permission prompt alerts appeared enabled while the active hook still matched only idle_prompt.

Validation

  • ./scripts/run_tests.sh — 28/28 passed
  • Bash syntax checks passed through the test runner
  • make lint could not run because shellcheck is not installed locally

Fixes #57

@mylee04
mylee04 merged commit dde2594 into main Aug 4, 2026
2 checks passed
@mylee04
mylee04 deleted the agent/fix-claude-permission-alerts branch August 4, 2026 02:58
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.

cn alerts for claude code permission prompt not working

1 participant