-
Notifications
You must be signed in to change notification settings - Fork 1.8k
TUI power inhibitor ignores prevent_sleep_while_streaming=false #452
Copy link
Copy link
Closed
Labels
autonomous: clearHands-off: unambiguous bug, obvious fix, no decisions. Don't even look - an agent can fully solve.Hands-off: unambiguous bug, obvious fix, no decisions. Don't even look - an agent can fully solve.bugSomething isn't workingSomething isn't workingpriority: mediumP2 - normal priorityP2 - normal prioritytriage: fixed-pending-releaseFixed in code/committed; will close automatically on next releaseFixed in code/committed; will close automatically on next release
Description
Metadata
Metadata
Assignees
Labels
autonomous: clearHands-off: unambiguous bug, obvious fix, no decisions. Don't even look - an agent can fully solve.Hands-off: unambiguous bug, obvious fix, no decisions. Don't even look - an agent can fully solve.bugSomething isn't workingSomething isn't workingpriority: mediumP2 - normal priorityP2 - normal prioritytriage: fixed-pending-releaseFixed in code/committed; will close automatically on next releaseFixed in code/committed; will close automatically on next release
Summary
[power].prevent_sleep_while_streaming = falsedisables the shared daemon power inhibitor, but the TUI client can still acquire a power inhibitor while a turn is processing.On Linux this can still hit the
systemd-inhibit/ polkit /pkttyagentpath even though the config says sleep prevention is disabled.Repro
Set:
Start a Linux TUI/remote-client session.
Run an agent turn.
Observe the power-inhibit path can still be used from the TUI sleep guard.
Expected
When
prevent_sleep_while_streaming = false, neither the daemon nor the TUI should acquire a power inhibitor.Reference fix
I could not create a PR from this account, but pushed a minimal reference fix here:
The main change gates
sync_sleep_guard()onconfig().power.prevent_sleep_while_streaming, matching the existing daemon-side config gate.