Skip to content

Fix too many notifications for Bash commands#39

Merged
kosinal merged 1 commit intomasterfrom
feature/37-fix-bash-notifications
Mar 4, 2026
Merged

Fix too many notifications for Bash commands#39
kosinal merged 1 commit intomasterfrom
feature/37-fix-bash-notifications

Conversation

@kosinal
Copy link
Copy Markdown
Owner

@kosinal kosinal commented Mar 4, 2026

Summary

  • Remove Bash from INTERACTIVE_TOOLS set so Bash PreToolUse events set status to "running" instead of "waiting"
  • This prevents spurious "Waiting for input" desktop notifications for every auto-approved Bash command

Fixes #37

Test plan

  • npm test — all 90 tests pass
  • Manual: start dashboard, run Claude Code session with Bash commands → status shows "Running" not "Waiting for input", no spurious notifications

🤖 Generated with Claude Code

Remove Bash from INTERACTIVE_TOOLS so PreToolUse events for Bash
commands set status to "running" instead of "waiting", preventing
spurious "Waiting for input" notifications for auto-approved commands.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kosinal kosinal merged commit 40353f6 into master Mar 4, 2026
6 checks passed
@kosinal kosinal deleted the feature/37-fix-bash-notifications branch March 4, 2026 10:47
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.

Too many notification for Bash scripts.

1 participant