docs(ipc): idle inhibit persists across restarts, plus the two undocumented verbs - #138
Merged
purian23 merged 1 commit intoAug 23, 2026
Conversation
Persistence: the inhibitor now survives a shell restart (AvengeMedia/DankMaterialShell#3123). Before that change it lived only in memory, so `dms restart` — or systemd bringing the service back under Restart=on-failure — silently turned it off. Worth stating explicitly, because "Keep Awake" that quietly stops keeping the machine awake is the kind of thing you only notice when the screen locks. While in this section, two verbs the shell implements were missing from the table. Both checked against a running shell rather than read off the source: dms ipc call inhibit status -> "Idle inhibit is enabled" dms ipc call inhibit reason "" -> "Current reason: Keep system awake" `reason` is documented as taking a required argument because the CLI rejects the bare form ("Too few arguments provided") even though the QML handler treats an empty string as a read. Edited docs/ only, not versioned_docs/: the persistence lands in a release that has not been cut yet.
Merged
12 tasks
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
2 participants
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.
Companion to AvengeMedia/DankMaterialShell#3123, which makes the idle inhibitor survive a shell restart.
Persistence. Before that change the inhibitor lived only in memory, so
dms restart— or systemd bringing the service back underRestart=on-failure— silently turned it off. Worth stating in the docs, because "Keep Awake" that quietly stops keeping the machine awake is the kind of thing you only notice once the screen has locked.Two missing verbs. While in this section:
inhibitimplements five functions and the table listed three. Both additions were checked against a running shell rather than read off the source:reasonis documented as taking a required argument because the CLI rejects the bare form (Too few arguments provided) even though the QML handler treats an empty string as a read — so the empty string is the documented way to query it.Edited
docs/only, notversioned_docs/: the persistence change lands in a release that has not been cut yet. Happy to split the two verbs into their own PR if you would rather keep this one to the persistence note.