Remove desktop notifications - #7
Merged
Merged
Conversation
Taken out rather than left switched off. The status-bar pills say how many agents want you, the window badge says which window, the picker's agent box says what it wants, and M-a goes there — a system alert on top of that interrupts whatever you are doing to tell you something four other surfaces already show. Gone with it: the @kaku-tab-agent-notify option, the osascript and notify-send calls and their runtime/os-exec imports, and the argument-passing care those needed because the notification body was an agent's own output and a quote in a reply could otherwise have closed the AppleScript literal. settled() keeps its remaining job — the counter redraw and the window rollup, on a real state change only — and no longer needs the record passed to it, which also retires the one assignment that existed solely to give the notifier a message to print. Nothing else changes: state, message, rollup and counter all still update on a transition, verified against a live server. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Taken out rather than left switched off.
The status-bar pills say how many agents want you, the window badge says which
window, the picker's agent box says what it wants, and
M-agoes there. Asystem alert on top of that interrupts whatever you are doing to tell you
something four other surfaces already show.
Removed:
@kaku-tab-agent-notifyoptionnotify(), theosascript/notify-sendcalls, and theruntimeandos/execimports they neededagent's own output and a quote in a reply could otherwise have closed the
AppleScript string literal
llm.txt,docs/agents.mdanddocs/configuration.mdsettled()keeps its remaining job — redraw the counter, refresh the windowrollup, on a real state change only — and no longer takes the record.
Verified against a live tmux server after removal: a
busy→permtransitionstill updates the pane state, the message, the window rollup and the status
counter, and spawns no notifier process.
🤖 Generated with Claude Code