Tags: aaddrick/claude-desktop-debian
Tags
fix: gate quick window patch to KDE sessions only (#393) (#406) * fix: gate quick window patch to KDE sessions only (#393) PR #390 fixed a quick-window regression on KDE but regressed GNOME/Ubuntu — @Andrej730 confirmed removing patch_quick_window restores quick entry on Ubuntu 24.04. Without a reproduction environment for GNOME yet, the safe minimum-viable fix is to gate the patch behind a runtime XDG_CURRENT_DESKTOP check: apply on KDE (where the fix is validated), fall back to upstream behavior everywhere else (which Ubuntu users confirmed works). Both halves of the patch are gated: - blur() before hide(): wrapped in a ternary so non-KDE sessions get the original unconditional hide() - focusFn()||show() replacement: wrapped so non-KDE sessions keep the original focus check instead of the visibility check Adds an idempotency pre-check in the node block (XDG_CURRENT_DESKTOP substring near the anchor) so re-runs skip cleanly. Part 1's existing grep idempotency still works because `Q.blur(),Q.hide()` appears inside the ternary literally. This is a temporary gate. VMs are being spun up to bisect which half actually regresses GNOME; once isolated, only that half needs the gate. Refs #393, #370, #404 Co-Authored-By: Claude <claude@anthropic.com> * style: split de_check assignment to fit under 80 chars Matches the concatenation style already used for the node block's deCheck, bringing the bash literal under the style guide's line limit. No functional change — the expanded string is identical. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude <claude@anthropic.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
fix: kill cowork daemon on app quit (#391) * fix: kill cowork daemon on app quit The upstream cowork-vm-shutdown quit handler uses the Swift VM addon which isn't available on Linux, so it's never registered. Our forked cowork-vm-service daemon was invisible to the quit system, surviving app exit and leaving QEMU/virtiofsd processes running. Register a Linux-specific quit handler via the upstream registerQuitHandler infrastructure. The handler sends SIGTERM to the daemon (which already handles it gracefully), verifies the PID via /proc/cmdline to prevent killing the wrong process on PID reuse, and polls for exit up to 10 seconds. The daemon PID is captured at fork time on a global, avoiding any need for pgrep/execSync at quit time. The handler is registered unconditionally for Linux so it works regardless of how the daemon was launched. Fixes #369 Co-Authored-By: Claude <claude@anthropic.com> * style: simplify quit handler patch comments and scope Add block scope for consistency with Patches 8-9, trim header comment, remove hardcoded minified name from implementation comment, simplify insertIdx calculation to match Patch 4 pattern. Co-Authored-By: Claude <claude@anthropic.com> --------- Co-authored-by: Claude <claude@anthropic.com>
PreviousNext