* Detect a compositor session lock through one helper omarchy-restart-shell decided whether the session was locked by looking for "LOCK" anywhere in the hyprctl monitors payload. That works, but not for the reason the code reads like: Hyprland reports no lock state of its own, and the string comes from solitaryBlockedBy, the list of reasons a monitor cannot hand a client the whole screen. An active ext-session-lock is one of those reasons. A substring match over the whole payload also answers yes to a workspace or a monitor description that merely spells LOCK, and locking a desktop nobody asked to lock is the worst way to be wrong. Match the reason list itself, and put it behind a helper now that a second caller needs the same answer. That second caller needs a third answer too, because the reason list is not always readable. Hyprland stops at the first reason on a monitor with no workspace yet — one just coming back — and returns before it ever looks at the lock, so a missing LOCK there means nothing was asked rather than nothing was found. Neither that nor an unreachable compositor is an unlocked session, and locks strand precisely while outputs are coming and going, so both exit 2. Callers that only branch on success are unaffected. The test fixture claimed the string came from a workspace name, so it was encoding the wrong model of the compositor. It now returns what Hyprland actually returns. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * Retake a session lock stranded by a dead shell ext-session-lock keeps the session locked when its client goes away — that is the point of the protocol, so a crashing lock screen cannot expose the desktop. The cost is that a shell which dies while locked leaves the compositor locked with nothing left to authenticate against: Hyprland's failsafe, which takes a TTY or another machine to clear. Nothing carried the lock across a restart. Quickshell relaunches itself after a crash and omarchy-restart-shell can be run by hand, but both bring back a shell holding no lock, so the failsafe stayed up. A fresh shell never holds a lock, so a session already locked as the lock service starts can only be that orphan: take it back and let the user type their way out. Asking once is not enough. These deaths happen while outputs are going away, and the replacement shell comes up inside that same window, where there is nothing to read a lock off. So the question is asked until the answer means something: on a short timer while the session settles, and again when a screen comes back, since a display asleep for hours outlasts any timer worth running and returns through a state the compositor cannot answer for either. Once an answer does arrive the search ends, so the timer stops and later screen changes cost nothing. Three ways this could lock a desktop nobody asked to lock, all closed. A lock this shell took itself is not an orphan, including one taken while the question was in flight — omarchy-restart-shell re-locks a fresh shell, and the answer cannot tell whose lock it found. Recovery runs once and clears the flag, so nothing lingers to fire after an unlock. And PAM landing late reopens the question rather than answering it: clearing the failsafe from a TTY is the documented way out, so a yes from before there was anything to do about it may be stale by the time it can be acted on. The check has to live here rather than in the launcher. Quickshell's crash handler re-execs in place, keeping the same pid, so a supervising process never sees the restarts that recovery matters most for. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * Relaunch the shell when it dies without a signal Quickshell restarts itself after a crash, but only from its signal handlers: SIGSEGV, SIGABRT, SIGFPE, SIGILL, SIGBUS, SIGTRAP. Qt does not always leave that way. When the Wayland connection fails, QWaylandDisplay::checkWaylandError calls _exit() directly, which raises no signal at all — so the crash handler never runs, no report lands in ~/.cache/quickshell/crashes, and the desktop is left with no bar and no explanation. That is how #6684 ends: the lock path meets a screen with no valid Wayland output, declines to create a lock surface for it, and the connection dies with EINVAL. Supervise the launcher so those deaths come back. A clean exit is deliberate — omarchy-restart-shell stops the shell over IPC and starts its own replacement — and a signal to the supervisor means the session is going away, so neither relaunches. Neither does a shell that outlived its compositor, though that takes more than one unanswered query to conclude: the shell dies while outputs are being reconfigured, which is also when a busy compositor can miss one without being gone. A shell that cannot stay up gives up after five tries in a minute rather than spinning. Signals need care now that a launcher stands between the session and the shell. Bash defers a trap until a foreground command returns, so the shell runs as a job and the supervisor waits on it. Stopping the launcher used to stop the shell with it, back when this script exec'd Quickshell, so the signal is passed on rather than leaving a desktop nobody is watching. One arriving during the backoff sleep only reaches the trap afterwards, so the flag is read again at the top of the loop: a shutdown racing a crash would otherwise get one more Quickshell on its way out. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
268 lines
9.6 KiB
Bash
Executable File
268 lines
9.6 KiB
Bash
Executable File
#!/bin/bash
|
|
|
|
set -euo pipefail
|
|
|
|
source "$(cd -- "$(dirname -- "${BASH_SOURCE[0]}")" && pwd)/base-test.sh"
|
|
|
|
test_tmp=$(mktemp -d)
|
|
restart_pid_one=""
|
|
restart_pid_two=""
|
|
|
|
cleanup() {
|
|
[[ -n $restart_pid_one ]] && kill "$restart_pid_one" 2>/dev/null || true
|
|
[[ -n $restart_pid_two ]] && kill "$restart_pid_two" 2>/dev/null || true
|
|
rm -rf "$test_tmp"
|
|
}
|
|
trap cleanup EXIT
|
|
|
|
wrapper_root="$test_tmp/wrapper-root"
|
|
wrapper_bin="$test_tmp/wrapper-bin"
|
|
mkdir -p "$wrapper_root/shell" "$wrapper_bin"
|
|
touch "$wrapper_root/shell/shell.qml"
|
|
|
|
cat >"$wrapper_bin/qs" <<'SH'
|
|
#!/bin/bash
|
|
|
|
[[ -n ${OMARCHY_TEST_QS_ARGS:-} ]] && printf '%s\n' "$*" >"$OMARCHY_TEST_QS_ARGS"
|
|
|
|
if [[ ${OMARCHY_TEST_QS_HANG:-0} == 1 ]]; then
|
|
sleep 5
|
|
elif [[ ${OMARCHY_TEST_QS_STARTING:-0} == 1 ]]; then
|
|
printf 'Not ready to accept queries yet.\n'
|
|
else
|
|
printf 'ok\n'
|
|
fi
|
|
SH
|
|
chmod +x "$wrapper_bin/qs"
|
|
|
|
wrapper_error=$(PATH="$wrapper_bin:$PATH" \
|
|
OMARCHY_PATH="$wrapper_root" \
|
|
OMARCHY_SHELL_IPC_TIMEOUT=0.1s \
|
|
OMARCHY_TEST_QS_HANG=1 \
|
|
"$ROOT/bin/omarchy-shell" shell ping 2>&1) && fail "hung shell IPC returns a failure"
|
|
[[ $wrapper_error == "omarchy-shell is not responding" ]] || fail "hung shell IPC reports that the shell is unresponsive" "$wrapper_error"
|
|
pass "shell IPC calls time out when Quickshell is unresponsive"
|
|
|
|
# A starting shell answers on stdout and exits 0, so a ping reads it as up.
|
|
wrapper_error=$(PATH="$wrapper_bin:$PATH" \
|
|
OMARCHY_PATH="$wrapper_root" \
|
|
OMARCHY_TEST_QS_STARTING=1 \
|
|
"$ROOT/bin/omarchy-shell" shell ping 2>&1) && fail "a starting shell answers IPC calls with a failure"
|
|
[[ $wrapper_error == "omarchy-shell is not ready" ]] || fail "a starting shell reports that it is not ready" "$wrapper_error"
|
|
pass "shell IPC calls fail while Quickshell is still starting"
|
|
|
|
PATH="$wrapper_bin:$PATH" \
|
|
OMARCHY_PATH="$wrapper_root" \
|
|
OMARCHY_TEST_QS_STARTING=1 \
|
|
"$ROOT/bin/omarchy-shell" -q shell ping >/dev/null 2>&1 ||
|
|
fail "quiet best-effort IPC calls tolerate a starting shell"
|
|
pass "quiet best-effort IPC calls tolerate a starting shell"
|
|
|
|
wrapper_args="$test_tmp/wrapper-args"
|
|
PATH="$wrapper_bin:$PATH" \
|
|
OMARCHY_PATH="$wrapper_root" \
|
|
OMARCHY_TEST_QS_ARGS="$wrapper_args" \
|
|
"$ROOT/bin/omarchy-shell" shell ping >/dev/null
|
|
|
|
grep -F -- 'ipc -n -p' "$wrapper_args" >/dev/null || fail "shell IPC targets the newest live Quickshell instance"
|
|
pass "shell IPC targets the newest live Quickshell instance"
|
|
|
|
restart_root="$test_tmp/restart-root"
|
|
restart_bin="$restart_root/bin"
|
|
restart_state="$test_tmp/restart-pids"
|
|
restart_log="$test_tmp/restart.log"
|
|
restart_env_log="$test_tmp/restart-env.log"
|
|
dispatch_log="$test_tmp/dispatch.log"
|
|
ipc_log="$test_tmp/ipc.log"
|
|
runtime_dir="$test_tmp/runtime"
|
|
mkdir -p "$restart_root/shell" "$restart_bin" "$runtime_dir"
|
|
touch "$restart_root/shell/shell.qml"
|
|
ln -s "$ROOT/bin/omarchy-shell" "$restart_bin/omarchy-shell"
|
|
ln -s "$ROOT/bin/omarchy-launch-shell" "$restart_bin/omarchy-launch-shell"
|
|
ln -s "$ROOT/bin/omarchy-cmd-missing" "$restart_bin/omarchy-cmd-missing"
|
|
ln -s "$ROOT/bin/omarchy-hyprland-session-locked" "$restart_bin/omarchy-hyprland-session-locked"
|
|
|
|
cat >"$restart_bin/qs" <<'SH'
|
|
#!/bin/bash
|
|
|
|
printf '%s\n' "$*" >>"$OMARCHY_TEST_IPC_LOG"
|
|
|
|
case "$*" in
|
|
*'shell ping')
|
|
[[ $* == *"-p $OMARCHY_TEST_SESSION_PATH/shell"* ]] &&
|
|
grep -Fx '303' "$OMARCHY_TEST_QS_STATE" >/dev/null &&
|
|
printf 'ok\n'
|
|
;;
|
|
*'lock lock')
|
|
touch "$OMARCHY_TEST_QS_STATE.locked"
|
|
printf 'ok\n'
|
|
;;
|
|
*'lock status')
|
|
if [[ -f $OMARCHY_TEST_QS_STATE.locked ]]; then
|
|
printf '{"secure": true, "requested": true}\n'
|
|
else
|
|
printf '{"secure": false, "requested": false}\n'
|
|
fi
|
|
;;
|
|
esac
|
|
SH
|
|
|
|
cat >"$restart_bin/quickshell" <<'SH'
|
|
#!/bin/bash
|
|
|
|
printf '%s\n' "$*" >>"$OMARCHY_TEST_QS_LOG"
|
|
|
|
case " $* " in
|
|
*' kill -p '*)
|
|
pid=$(head -n 1 "$OMARCHY_TEST_QS_STATE")
|
|
[[ $pid =~ ^[0-9]+$ ]] || exit 1
|
|
kill "$pid" 2>/dev/null
|
|
while kill -0 "$pid" 2>/dev/null; do sleep 0.01; done
|
|
awk 'NR > 1' "$OMARCHY_TEST_QS_STATE" >"$OMARCHY_TEST_QS_STATE.next"
|
|
mv "$OMARCHY_TEST_QS_STATE.next" "$OMARCHY_TEST_QS_STATE"
|
|
;;
|
|
*' -n -p '*)
|
|
printf '%s\n' "${OMARCHY_TEST_TRANSIENT_ENV-unset}" >"$OMARCHY_TEST_QS_ENV_LOG"
|
|
printf '303\n' >"$OMARCHY_TEST_QS_STATE"
|
|
;;
|
|
esac
|
|
SH
|
|
|
|
cat >"$restart_bin/hyprctl" <<'SH'
|
|
#!/bin/bash
|
|
|
|
if [[ ${1:-} == "-j" && ${2:-} == "monitors" ]]; then
|
|
# Hyprland reports an active session lock as a reason the monitor cannot hand
|
|
# a client the whole screen, not as a workspace.
|
|
if [[ ${OMARCHY_TEST_SESSION_LOCKED:-0} == 1 ]]; then
|
|
printf '[{"name":"eDP-1","solitaryBlockedBy":["WINDOWED","LOCK","CANDIDATE"]}]\n'
|
|
else
|
|
printf '[{"name":"eDP-1","solitaryBlockedBy":["WINDOWED","CANDIDATE"]}]\n'
|
|
fi
|
|
elif [[ ${1:-} == "dispatch" && ${2:-} == hl.dsp.exec_cmd* ]]; then
|
|
printf '%s\n' "${2:-}" >>"$OMARCHY_TEST_DISPATCH_LOG"
|
|
OMARCHY_PATH="$OMARCHY_TEST_SESSION_PATH" \
|
|
env -u OMARCHY_TEST_TRANSIENT_ENV omarchy-launch-shell
|
|
printf 'ok\n'
|
|
elif [[ ${1:-} == "dispatch" ]]; then
|
|
exit 1
|
|
fi
|
|
SH
|
|
|
|
# Keep the test hermetic where journald has no usable stream socket.
|
|
cat >"$restart_bin/systemd-cat" <<'SH'
|
|
#!/bin/bash
|
|
|
|
while (( $# > 0 )); do
|
|
[[ $1 == "--" ]] && { shift; break; }
|
|
shift
|
|
done
|
|
exec "$@"
|
|
SH
|
|
|
|
cat >"$restart_bin/systemctl" <<'SH'
|
|
#!/bin/bash
|
|
|
|
if [[ ${1:-} == "--user" && ${2:-} == "show-environment" ]]; then
|
|
printf 'OMARCHY_PATH=%s\n' "$OMARCHY_TEST_SESSION_PATH"
|
|
elif [[ ${1:-} == "--user" && ${2:-} == "try-restart" ]]; then
|
|
exit 0
|
|
else
|
|
exit 1
|
|
fi
|
|
SH
|
|
|
|
chmod +x "$restart_bin/qs" "$restart_bin/quickshell" "$restart_bin/hyprctl" "$restart_bin/systemd-cat" "$restart_bin/systemctl"
|
|
|
|
sleep 30 &
|
|
restart_pid_one=$!
|
|
sleep 30 &
|
|
restart_pid_two=$!
|
|
printf '%s\n%s\n' "$restart_pid_one" "$restart_pid_two" >"$restart_state"
|
|
|
|
caller_root="$test_tmp/caller-root"
|
|
mkdir -p "$caller_root/shell"
|
|
touch "$caller_root/shell/shell.qml"
|
|
|
|
PATH="$restart_bin:$PATH" \
|
|
OMARCHY_PATH="$caller_root" \
|
|
XDG_RUNTIME_DIR="$runtime_dir" \
|
|
OMARCHY_TEST_QS_STATE="$restart_state" \
|
|
OMARCHY_TEST_QS_LOG="$restart_log" \
|
|
OMARCHY_TEST_QS_ENV_LOG="$restart_env_log" \
|
|
OMARCHY_TEST_DISPATCH_LOG="$dispatch_log" \
|
|
OMARCHY_TEST_IPC_LOG="$ipc_log" \
|
|
OMARCHY_TEST_SESSION_PATH="$restart_root" \
|
|
OMARCHY_TEST_TRANSIENT_ENV=leaked \
|
|
timeout 5 "$ROOT/bin/omarchy-restart-shell"
|
|
|
|
if kill -0 "$restart_pid_one" 2>/dev/null; then
|
|
fail "restart stops the first matching shell instance"
|
|
fi
|
|
if kill -0 "$restart_pid_two" 2>/dev/null; then
|
|
fail "restart stops duplicate matching shell instances"
|
|
fi
|
|
wait "$restart_pid_one" 2>/dev/null || true
|
|
wait "$restart_pid_two" 2>/dev/null || true
|
|
restart_pid_one=""
|
|
restart_pid_two=""
|
|
[[ $(<"$restart_state") == 303 ]] || fail "restart leaves exactly one fresh shell instance"
|
|
[[ $(grep -c '^-n -p ' "$restart_log") == 1 ]] || fail "restart launches one fresh shell process"
|
|
grep -F "kill -p $restart_root/shell --any-display" "$restart_log" >/dev/null || fail "restart stops the shell from the session checkout"
|
|
[[ $(<"$restart_env_log") == "unset" ]] || fail "restart uses the Hyprland session environment for the fresh shell"
|
|
grep -F 'hl.dsp.exec_cmd("omarchy-launch-shell")' "$dispatch_log" >/dev/null || fail "restart launches the fresh shell through Hyprland"
|
|
grep -F "ipc -n -p $restart_root/shell call -- shell ping" "$ipc_log" >/dev/null || fail "restart checks readiness in the session checkout"
|
|
pass "restart replaces duplicate shell instances from the session checkout"
|
|
|
|
: >"$restart_log"
|
|
printf '303\n' >"$restart_state"
|
|
touch "$restart_state.locked"
|
|
|
|
locked_error=$(PATH="$restart_bin:$PATH" \
|
|
OMARCHY_PATH="$restart_root" \
|
|
XDG_RUNTIME_DIR="$runtime_dir" \
|
|
OMARCHY_TEST_SESSION_LOCKED=1 \
|
|
OMARCHY_TEST_QS_STATE="$restart_state" \
|
|
OMARCHY_TEST_QS_LOG="$restart_log" \
|
|
OMARCHY_TEST_DISPATCH_LOG="$dispatch_log" \
|
|
OMARCHY_TEST_IPC_LOG="$ipc_log" \
|
|
OMARCHY_TEST_SESSION_PATH="$restart_root" \
|
|
"$ROOT/bin/omarchy-restart-shell" 2>&1) && fail "restart refuses while the shell lock is active"
|
|
|
|
[[ $locked_error == "Refusing to restart Omarchy shell while the session is locked." ]] || fail "locked restart explains why it was refused" "$locked_error"
|
|
[[ $(<"$restart_state") == 303 ]] || fail "locked restart preserves the running shell"
|
|
[[ ! -s $restart_log ]] || fail "locked restart does not stop or launch Quickshell"
|
|
pass "restart preserves the shell while its lock is active"
|
|
|
|
# A LOCK session without an active locker — dead shell or a crash-handler
|
|
# relaunch holding no lock — is the failsafe: restart must proceed,
|
|
# re-acquire the session lock, and wait for it to report secure.
|
|
sleep 30 &
|
|
restart_pid_one=$!
|
|
printf '%s\n' "$restart_pid_one" >"$restart_state"
|
|
rm -f "$restart_state.locked"
|
|
: >"$restart_log"
|
|
: >"$ipc_log"
|
|
|
|
PATH="$restart_bin:$PATH" \
|
|
OMARCHY_PATH="$restart_root" \
|
|
XDG_RUNTIME_DIR="$runtime_dir" \
|
|
OMARCHY_TEST_SESSION_LOCKED=1 \
|
|
OMARCHY_TEST_QS_STATE="$restart_state" \
|
|
OMARCHY_TEST_QS_LOG="$restart_log" \
|
|
OMARCHY_TEST_QS_ENV_LOG="$restart_env_log" \
|
|
OMARCHY_TEST_DISPATCH_LOG="$dispatch_log" \
|
|
OMARCHY_TEST_IPC_LOG="$ipc_log" \
|
|
OMARCHY_TEST_SESSION_PATH="$restart_root" \
|
|
timeout 5 "$ROOT/bin/omarchy-restart-shell" || fail "locked restart recovers when the lock client is dead"
|
|
|
|
if kill -0 "$restart_pid_one" 2>/dev/null; then
|
|
fail "dead-lock recovery stops the stale shell instance"
|
|
fi
|
|
wait "$restart_pid_one" 2>/dev/null || true
|
|
restart_pid_one=""
|
|
[[ $(<"$restart_state") == 303 ]] || fail "dead-lock recovery leaves one fresh shell instance"
|
|
grep -F "ipc -n -p $restart_root/shell call -- lock lock" "$ipc_log" >/dev/null || fail "dead-lock recovery re-acquires the session lock"
|
|
grep -F "ipc -n -p $restart_root/shell call -- lock status" "$ipc_log" >/dev/null || fail "dead-lock recovery waits for the lock to become secure"
|
|
pass "restart recovers a locked session whose lock client died"
|