From 661d477e7806d322d3499c19537541c5dd4e7d82 Mon Sep 17 00:00:00 2001 From: "Danilo M." Date: Tue, 15 Sep 2026 12:29:42 +0200 Subject: fix(status): address final review findings F1: after a click the shared Switch writes checked directly, dropping StatusRow's declarative binding, so an external mode change from waybar or statusctl no longer moved the switch. Resync via onValueChanged. F2: the spec claimed hyprctl clients counts idle inhibitors, which is false for a layer-surface inhibitor; replace with the behavioral hypridle marker check. F3: README said the check covers both watch states; it exercises the activated report and the absent-file down report. F4: give breakProc an onExited check so a failed breaktimer.sh verb is visible instead of silent. F5: drop StatusTile's unused required st property and its injection. F6: comment the startup-order limit on dndBeforePresentation. --- docs/superpowers/specs/2026-09-15-status-registry-design.md | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'docs/superpowers/specs') diff --git a/docs/superpowers/specs/2026-09-15-status-registry-design.md b/docs/superpowers/specs/2026-09-15-status-registry-design.md index 34e5945..db1e79c 100644 --- a/docs/superpowers/specs/2026-09-15-status-registry-design.md +++ b/docs/superpowers/specs/2026-09-15-status-registry-design.md @@ -222,8 +222,10 @@ That covers the parse, the write, the atomic rename and the watch together, and it fails if any of them break. The effects need the shell and are confirmed by hand, once, because each is -observable: `hyprctl clients` counts inhibitors, `breaktimer.sh status` reports -`paused`, and the notification daemon's own behaviour under DND is the second -spec's verification. +observable: a throwaway `hypridle -c` instance with a short timeout writes a +marker on `on-timeout`, and the marker must not appear while presentation is on +and must appear within a few seconds of it being turned off; `breaktimer.sh +status` reports `paused`; and the notification daemon's own behaviour under DND +is the second spec's verification. Per AGENTS.md, anything visual is confirmed by the user rather than screenshotted. -- cgit v1.2.3