Skip to content

Fix Windows alert settings and isolate notification mutes - #69

Merged
mylee04 merged 2 commits into
mainfrom
fix/issues-54-56-notification-controls
Nov 11, 2026
Merged

mylee04 merged 2 commits into
mainfrom
fix/issues-54-56-notification-controls

Conversation

@mylee04

@mylee04 mylee04 commented Nov 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add cn alerts support for Claude notification types on native Windows, with immediate updates to enabled global Claude hooks.
  • Keep Bash notification mutes scoped to the selected tool so cn on codex no longer unmutes Claude when both tools were muted.

Fixes #54. Addresses the cross-tool mute behavior relevant to #56. On a clean npm install, cn on codex already configured Codex alone. The reproducible case involved an existing Claude hook and a shared mute.

Verification

  • Local test suite passed (28/28).
  • The generated Windows PowerShell wrapper passed the new alert configuration regression on macOS PowerShell. Native Windows CI covers the installed wrapper on a Windows runner.
  • The Bash regression checks that enabling Codex preserves Claude's mute and that disabling Codex preserves Claude notification delivery.

@mylee04
mylee04 merged commit fd867b9 into main Nov 11, 2026
4 checks passed
@mylee04 mylee04 mentioned this pull request Nov 11, 2026
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 is not found

1 participant