feat(desktop): add mako notifications (task 0025) #17
Reference in New Issue
Block a user
Delete Branch "task-0025-mako-notifications"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Task:
.claude/tasks/0025-mako-notifications.mdSummary
Adds a mako notification daemon module to the desktop group, enabled by the aggregator.
modules/desktop/mako.nixenablesservices.makothrough home-manager, added tomodules/desktop/desktop.nix.default-timeout=5000); mako's own default of 0 would leave them on screen forever.[mode=dnd]withinvisible=true) hides toasts while still recording them to history, so notifications missed during do-not-disturb stay retrievable.Super+Nrecalls the last notification viamakoctl restore, keyboard-only like the rest of the session.The bar-side do-not-disturb toggle (
custom/dnd, callingmakoctl mode -t dnd) and MPRIS media controls already live inwaybar.nixfrom task 0024, so this task adds only the daemon. The mode namedndmatches on both sides. mako renders toasts as a layer-shell overlay with no tiled window, so no Waybarwindow-rewriteicon mapping applies.Deviation: a
Super+Shift+Ndismiss-all binding was drafted alongside the recall bind but dropped as unrequested scope — the acceptance criteria call for history recall, whichrestorealone serves.Review
Risk
Overall: Low
mkDefaultenable line, no changes to existing callers.Standards & Spec findings
No unaddressed findings. The review raised three, all fixed in this diff before commit: a semicolon in a comment (recast into two sentences), a comment narrating the bar's use of the mode name (reworded to state the local reason), and the unrequested
dismiss --allkeybind (dropped).— Claude