feat: switch to systemd-journal-logger, add debug logging (v0.6.0)

Replace env_logger with systemd-journal-logger for consistent logging across moonset/moonlock/moongreet. Add MOONSET_DEBUG env var and debug statements across all modules. Also includes shared blur cache for multi-monitor and detached moonlock spawn for lock action.
perf: replace CPU blur with GPU blur via GskBlurNode (v0.5.0)
2026-03-28 22:58:25 +01:00 · 2026-03-28 22:35:18 +01:00 · 2026-03-28 21:50:03 +01:00 · 2026-03-28 21:46:08 +01:00 · 2026-03-28 21:39:34 +01:00 · 2026-03-28 21:22:48 +01:00
13 changed files with 287 additions and 375 deletions
@@ -3,11 +3,42 @@
 All notable changes to this project will be documented in this file.
 Format based on [Keep a Changelog](https://keepachangelog.com/).

+## [0.6.0] - 2026-03-28
+
+### Added
+
+- Systemd journal logging (`journalctl -t moonset`) replacing env_logger stderr output
+- `MOONSET_DEBUG` env var to enable debug-level journal output
+- Debug logging across all modules (config resolution, wallpaper source, avatar loading, power actions, locale detection, blur cache)
+- Shared blur cache for multi-monitor — GPU blur computed once, reused by all windows
+
+### Changed
+
+- Lock action spawns moonlock as detached process instead of blocking via run_command — moonset can quit immediately while moonlock runs independently
+
+## [0.4.1] - 2026-03-28
+
+### Added
+
+- Fade-in/fade-out animation (250ms ease-in) for panel and wallpaper windows via CSS opacity transition
+
+### Fixed
+
+- Fix pixel format mismatch in blur path — `texture.download()` yields BGRA but was passed to `RgbaImage` without channel swap, now explicitly converts B↔R
+- Logout action now calls `app.quit()` to dismiss the menu immediately (previously only Lock did)
+- Log TOML parse errors to stderr instead of silently falling back to defaults
+
+### Changed
+
+- Replace magic string `"lock"` comparison with `quit_after` field on `ActionDef` for type-safe action dispatch
+- Remove `compressed="true"` from JPEG wallpaper in GResource — JPEG is already compressed, zlib overhead hurts startup for negligible size savings
+
 ## [0.4.0] - 2026-03-28

 ### Added

 - Optional background blur via `background_blur` config option (Gaussian blur, `image` crate)
+- Disk cache for blurred wallpaper (`~/.cache/moonset/`) — avoids re-blurring on subsequent starts

 ## [0.1.1] - 2026-03-28

@@ -35,7 +35,7 @@ LD_PRELOAD=/usr/lib/libgtk4-layer-shell.so ./target/release/moonset

 ## Architektur

- `main.rs` — Entry Point, GTK App, Layer Shell Setup, Multi-Monitor, zentrale `GRESOURCE_PREFIX`-Konstante
+- `main.rs` — Entry Point, GTK App, Layer Shell Setup, Multi-Monitor, systemd-Journal-Logging, Debug-Level per `MOONSET_DEBUG` Env-Var, zentrale `GRESOURCE_PREFIX`-Konstante
 - `power.rs` — 5 Power-Action-Wrapper mit absoluten Pfaden und 30s Timeout (lock, logout, hibernate, reboot, shutdown)
 - `i18n.rs` — Locale-Erkennung und String-Tabellen (DE/EN)
 - `config.rs` — TOML-Config + Wallpaper-Fallback
@@ -56,3 +56,4 @@ Kurzfassung der wichtigsten Entscheidungen:
 - **Absolute Pfade für Binaries**: `/usr/bin/systemctl` etc. statt relativer Pfade (Security)
 - **GResource-Bundle**: CSS, Wallpaper (komprimiert) und Default-Avatar sind in die Binary kompiliert
 - **Async Power Actions**: `glib::spawn_future_local` + `gio::spawn_blocking` mit 30s Timeout
+- **Journal-Logging**: `systemd-journal-logger` statt File-Logging — `journalctl -t moonset`, Debug-Level per `MOONSET_DEBUG` Env-Var
@@ -2,71 +2,6 @@
 # It is not intended for manual editing.
 version = 4

-[[package]]
-name = "adler2"
-version = "2.0.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "320119579fcad9c21884f5c4861d16174d0e06250625266f50fe6898340abefa"
-
-[[package]]
-name = "aho-corasick"
-version = "1.1.4"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ddd31a130427c27518df266943a5308ed92d4b226cc639f5a8f1002816174301"
-dependencies = [
- "memchr",
-]
-
-[[package]]
-name = "anstream"
-version = "1.0.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "824a212faf96e9acacdbd09febd34438f8f711fb84e09a8916013cd7815ca28d"
-dependencies = [
- "anstyle",
- "anstyle-parse",
- "anstyle-query",
- "anstyle-wincon",
- "colorchoice",
- "is_terminal_polyfill",
- "utf8parse",
-]
-
-[[package]]
-name = "anstyle"
-version = "1.0.14"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "940b3a0ca603d1eade50a4846a2afffd5ef57a9feac2c0e2ec2e14f9ead76000"
-
-[[package]]
-name = "anstyle-parse"
-version = "1.0.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "52ce7f38b242319f7cabaa6813055467063ecdc9d355bbb4ce0c68908cd8130e"
-dependencies = [
- "utf8parse",
-]
-
-[[package]]
-name = "anstyle-query"
-version = "1.1.5"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "40c48f72fd53cd289104fc64099abca73db4166ad86ea0b4341abe65af83dadc"
-dependencies = [
- "windows-sys",
-]
-
-[[package]]
-name = "anstyle-wincon"
-version = "3.0.11"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "291e6a250ff86cd4a820112fb8898808a366d8f9f58ce16d1f538353ad55747d"
-dependencies = [
- "anstyle",
- "once_cell_polyfill",
- "windows-sys",
-]
-
 [[package]]
 name = "anyhow"
 version = "1.0.102"
@@ -85,18 +20,6 @@ version = "2.11.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "843867be96c8daad0d758b57df9392b6d8d271134fce549de6ce169ff98a92af"

-[[package]]
-name = "bytemuck"
-version = "1.25.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c8efb64bd706a16a1bdde310ae86b351e4d21550d98d056f22f8a7f7a2183fec"
-
-[[package]]
-name = "byteorder-lite"
-version = "0.1.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "8f1fe948ff07f4bd06c30984e69f5b4899c516a3ef74f34df92a2df2ab535495"
-
 [[package]]
 name = "cairo-rs"
 version = "0.22.0"
@@ -142,21 +65,6 @@ version = "0.2.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "613afe47fcd5fac7ccf1db93babcb082c5994d996f20b8b159f2ad1658eb5724"

-[[package]]
-name = "colorchoice"
-version = "1.0.5"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1d07550c9036bf2ae0c684c4297d503f838287c83c53686d05370d0e139ae570"
-
-[[package]]
-name = "crc32fast"
-version = "1.5.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9481c1c90cbf2ac953f07c8d4a58aa3945c425b7185c9154d67a65e4230da511"
-dependencies = [
- "cfg-if",
-]
-
 [[package]]
 name = "dirs"
 version = "6.0.0"
@@ -178,29 +86,6 @@ dependencies = [
 "windows-sys",
 ]

-[[package]]
-name = "env_filter"
-version = "1.0.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "32e90c2accc4b07a8456ea0debdc2e7587bdd890680d71173a15d4ae604f6eef"
-dependencies = [
- "log",
- "regex",
-]
-
-[[package]]
-name = "env_logger"
-version = "0.11.10"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "0621c04f2196ac3f488dd583365b9c09be011a4ab8b9f37248ffcc8f6198b56a"
-dependencies = [
- "anstream",
- "anstyle",
- "env_filter",
- "jiff",
- "log",
-]
-
 [[package]]
 name = "equivalent"
 version = "1.0.2"
@@ -223,15 +108,6 @@ version = "2.3.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "37909eebbb50d72f9059c3b6d82c0463f2ff062c9e95845c43a6c9c0355411be"

-[[package]]
-name = "fdeflate"
-version = "0.3.7"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1e6853b52649d4ac5c0bd02320cddc5ba956bdb407c4b75a2c6b75bf51500f8c"
-dependencies = [
- "simd-adler32",
-]
-
 [[package]]
 name = "field-offset"
 version = "0.3.6"
@@ -242,16 +118,6 @@ dependencies = [
 "rustc_version",
 ]

-[[package]]
-name = "flate2"
-version = "1.1.9"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "843fba2746e448b37e26a819579957415c8cef339bf08564fe8b7ddbd959573c"
-dependencies = [
- "crc32fast",
- "miniz_oxide",
-]
-
 [[package]]
 name = "foldhash"
 version = "0.1.5"
@@ -676,21 +542,6 @@ version = "2.3.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "3d3067d79b975e8844ca9eb072e16b31c3c1c36928edf9c6789548c524d0d954"

-[[package]]
-name = "image"
-version = "0.25.10"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "85ab80394333c02fe689eaf900ab500fbd0c2213da414687ebf995a65d5a6104"
-dependencies = [
- "bytemuck",
- "byteorder-lite",
- "moxcms",
- "num-traits",
- "png",
- "zune-core",
- "zune-jpeg",
-]
-
 [[package]]
 name = "indexmap"
 version = "2.13.0"
@@ -703,42 +554,12 @@ dependencies = [
 "serde_core",
 ]

-[[package]]
-name = "is_terminal_polyfill"
-version = "1.70.2"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a6cb138bb79a146c1bd460005623e142ef0181e3d0219cb493e02f7d08a35695"
-
 [[package]]
 name = "itoa"
 version = "1.0.18"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "8f42a60cbdf9a97f5d2305f08a87dc4e09308d1276d28c869c684d7777685682"

-[[package]]
-name = "jiff"
-version = "0.2.23"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1a3546dc96b6d42c5f24902af9e2538e82e39ad350b0c766eb3fbf2d8f3d8359"
-dependencies = [
- "jiff-static",
- "log",
- "portable-atomic",
- "portable-atomic-util",
- "serde_core",
-]
-
-[[package]]
-name = "jiff-static"
-version = "0.2.23"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "2a8c8b344124222efd714b73bb41f8b5120b27a7cc1c75593a6ff768d9d05aa4"
-dependencies = [
- "proc-macro2",
- "quote",
- "syn",
-]
-
 [[package]]
 name = "khronos_api"
 version = "3.1.0"
@@ -793,46 +614,26 @@ dependencies = [
 "autocfg",
 ]

-[[package]]
-name = "miniz_oxide"
-version = "0.8.9"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1fa76a2c86f704bdb222d66965fb3d63269ce38518b83cb0575fca855ebb6316"
-dependencies = [
- "adler2",
- "simd-adler32",
-]
-
 [[package]]
 name = "moonset"
-version = "0.1.1"
+version = "0.5.0"
 dependencies = [
 "dirs",
- "env_logger",
 "gdk-pixbuf",
 "gdk4",
 "glib",
 "glib-build-tools",
+ "graphene-rs",
 "gtk4",
 "gtk4-layer-shell",
- "image",
 "log",
 "nix",
 "serde",
+ "systemd-journal-logger",
 "tempfile",
 "toml 0.8.23",
 ]

-[[package]]
-name = "moxcms"
-version = "0.8.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "bb85c154ba489f01b25c0d36ae69a87e4a1c73a72631fc6c0eb6dde34a73e44b"
-dependencies = [
- "num-traits",
- "pxfm",
-]
-
 [[package]]
 name = "nix"
 version = "0.29.0"
@@ -845,27 +646,12 @@ dependencies = [
 "libc",
 ]

-[[package]]
-name = "num-traits"
-version = "0.2.19"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "071dfc062690e90b734c0b2273ce72ad0ffa95f0c74596bc250dcfd960262841"
-dependencies = [
- "autocfg",
-]
-
 [[package]]
 name = "once_cell"
 version = "1.21.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "9f7c3e4beb33f85d45ae3e3a1792185706c8e16d043238c593331cc7cd313b50"

-[[package]]
-name = "once_cell_polyfill"
-version = "1.70.2"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "384b8ab6d37215f3c5301a95a4accb5d64aa607f1fcb26a11b5303878451b4fe"
-
 [[package]]
 name = "option-ext"
 version = "0.2.0"
@@ -908,34 +694,6 @@ version = "0.3.32"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "7edddbd0b52d732b21ad9a5fab5c704c14cd949e5e9a1ec5929a24fded1b904c"

-[[package]]
-name = "png"
-version = "0.18.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "60769b8b31b2a9f263dae2776c37b1b28ae246943cf719eb6946a1db05128a61"
-dependencies = [
- "bitflags",
- "crc32fast",
- "fdeflate",
- "flate2",
- "miniz_oxide",
-]
-
-[[package]]
-name = "portable-atomic"
-version = "1.13.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c33a9471896f1c69cecef8d20cbe2f7accd12527ce60845ff44c153bb2a21b49"
-
-[[package]]
-name = "portable-atomic-util"
-version = "0.2.6"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "091397be61a01d4be58e7841595bd4bfedb15f1cd54977d79b8271e94ed799a3"
-dependencies = [
- "portable-atomic",
-]
-
 [[package]]
 name = "prettyplease"
 version = "0.2.37"
@@ -964,12 +722,6 @@ dependencies = [
 "unicode-ident",
 ]

-[[package]]
-name = "pxfm"
-version = "0.1.28"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "b5a041e753da8b807c9255f28de81879c78c876392ff2469cde94799b2896b9d"
-
 [[package]]
 name = "quote"
 version = "1.0.45"
@@ -996,35 +748,6 @@ dependencies = [
 "thiserror",
 ]

-[[package]]
-name = "regex"
-version = "1.12.3"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e10754a14b9137dd7b1e3e5b0493cc9171fdd105e0ab477f51b72e7f3ac0e276"
-dependencies = [
- "aho-corasick",
- "memchr",
- "regex-automata",
- "regex-syntax",
-]
-
-[[package]]
-name = "regex-automata"
-version = "0.4.14"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6e1dd4122fc1595e8162618945476892eefca7b88c52820e74af6262213cae8f"
-dependencies = [
- "aho-corasick",
- "memchr",
- "regex-syntax",
-]
-
-[[package]]
-name = "regex-syntax"
-version = "0.8.10"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "dc897dd8d9e8bd1ed8cdad82b5966c3e0ecae09fb1907d58efaa013543185d0a"
-
 [[package]]
 name = "rustc_version"
 version = "0.4.1"
@@ -1114,12 +837,6 @@ dependencies = [
 "serde_core",
 ]

-[[package]]
-name = "simd-adler32"
-version = "0.3.9"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "703d5c7ef118737c72f1af64ad2f6f8c5e1921f818cdcb97b8fe6fc69bf66214"
-
 [[package]]
 name = "slab"
 version = "0.4.12"
@@ -1156,6 +873,16 @@ dependencies = [
 "version-compare",
 ]

+[[package]]
+name = "systemd-journal-logger"
+version = "2.2.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "7266304d24ca5a4b230545fc558c80e18bd3e1d2eb1be149b6bcd04398d3e79c"
+dependencies = [
+ "log",
+ "rustix",
+]
+
 [[package]]
 name = "target-lexicon"
 version = "0.13.3"
@@ -1308,12 +1035,6 @@ version = "0.2.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "ebc1c04c71510c7f702b52b7c350734c9ff1295c464a03335b00bb84fc54f853"

-[[package]]
-name = "utf8parse"
-version = "0.2.2"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "06abde3611657adf66d383f00b093d7faecc7fa57071cce2578660c9f1010821"
-
 [[package]]
 name = "version-compare"
 version = "0.2.1"
@@ -1510,18 +1231,3 @@ name = "zmij"
 version = "1.0.21"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "b8848ee67ecc8aedbaf3e4122217aff892639231befc6a1b58d29fff4c2cabaa"
-
-[[package]]
-name = "zune-core"
-version = "0.5.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "cb8a0807f7c01457d0379ba880ba6322660448ddebc890ce29bb64da71fb40f9"
-
-[[package]]
-name = "zune-jpeg"
-version = "0.5.15"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "27bc9d5b815bc103f142aa054f561d9187d191692ec7c2d1e2b4737f8dbd7296"
-dependencies = [
- "zune-core",
-]
@@ -1,6 +1,6 @@
 [package]
 name = "moonset"
-version = "0.4.0"
+version = "0.6.0"
 edition = "2024"
 description = "Wayland session power menu with GTK4 and Layer Shell"
 license = "MIT"
@@ -15,9 +15,9 @@ toml = "0.8"
 dirs = "6"
 serde = { version = "1", features = ["derive"] }
 nix = { version = "0.29", features = ["user"] }
-image = { version = "0.25", default-features = false, features = ["jpeg", "png"] }
+graphene-rs = { version = "0.22", package = "graphene-rs" }
 log = "0.4"
-env_logger = "0.11"
+systemd-journal-logger = "2.2"

 [dev-dependencies]
 tempfile = "3"
@@ -2,12 +2,26 @@

 Architectural and design decisions for Moonset, in reverse chronological order.

-## 2026-03-28 – Optional background blur via `image` crate
+## 2026-03-28 – Switch from env_logger to systemd-journal-logger
+
+- **Who**: Ragnar, Dom
+- **Why**: moonlock and moongreet already use systemd-journal-logger. moonset used env_logger which writes to stderr — not useful for a GUI app launched via keybind. Journal integration enables `journalctl -t moonset` and consistent troubleshooting across all three Moon projects.
+- **Tradeoffs**: Requires systemd at runtime. Graceful fallback to eprintln if journal logger fails. Acceptable since Moonarch targets systemd-based Arch Linux.
+- **How**: Replace `env_logger` dep with `systemd-journal-logger`, add `setup_logging()` with `MOONSET_DEBUG` env var for debug-level output. Same pattern as moonlock/moongreet.
+
+## 2026-03-28 – Replace action name dispatch with `quit_after` field

 - **Who**: Hekate, Dom
- **Why**: Blurred wallpaper as background is a common UX pattern for overlay menus
- **Tradeoffs**: Adds `image` crate dependency (~15 transitive crates); CPU-side Gaussian blur at load time adds startup latency proportional to image size and sigma. Acceptable because blur runs once and the texture is shared across monitors.
- **How**: `load_background_texture(bg_path, blur_radius)` loads texture, optionally applies `imageops::blur()`, returns `gdk::Texture`. Config option `background_blur: Option<f32>` in TOML.
+- **Why**: Post-action behavior (quit the app or not) was controlled by comparing `action_name == "lock"` — a magic string duplicated from the action definition. Renaming an action would silently break the dispatch.
+- **Tradeoffs**: Adds a field to `ActionDef` that most actions set to `false`. Acceptable because it makes the contract explicit and testable.
+- **How**: `ActionDef.quit_after: bool` — `true` for lock and logout, `false` for hibernate/reboot/shutdown.
+
+## 2026-03-28 – GPU blur via GskBlurNode replaces CPU blur
+
+- **Who**: Ragnar, Dom
+- **Why**: CPU-side Gaussian blur (`image` crate) blocked startup and added caching complexity. moonlock already migrated to GPU blur.
+- **Tradeoffs**: GPU blur quality is slightly different (box-blur approximation vs true Gaussian), acceptable for wallpaper backgrounds. Removes `image` crate dependency entirely. No disk cache needed.
+- **How**: `Snapshot::push_blur()` + `GskRenderer::render_texture()` on `connect_realize`. Blur happens once on the GPU when the widget gets its renderer. Symmetric with moonlock and moongreet.

 ## 2026-03-28 – Use absolute paths for system binaries

@@ -2,7 +2,7 @@
 <gresources>
  <gresource prefix="/dev/moonarch/moonset">
    <file>style.css</file>
-    <file compressed="true">wallpaper.jpg</file>
+    <file>wallpaper.jpg</file>
    <file>default-avatar.svg</file>
  </gresource>
 </gresources>
@@ -6,11 +6,23 @@ window.panel {
    background-color: @theme_bg_color;
    background-size: cover;
    background-position: center;
+    opacity: 0;
+    transition: opacity 250ms ease-in;
+}
+
+window.panel.visible {
+    opacity: 1;
 }

 /* Wallpaper-only window for secondary monitors */
 window.wallpaper {
    background-color: @theme_bg_color;
+    opacity: 0;
+    transition: opacity 250ms ease-in;
+}
+
+window.wallpaper.visible {
+    opacity: 1;
 }

 /* Round avatar image */
@@ -31,12 +31,18 @@ pub fn load_config(config_paths: Option<&[PathBuf]>) -> Config {
    let mut merged = Config::default();
    for path in paths {
        if let Ok(content) = fs::read_to_string(path) {
-            if let Ok(parsed) = toml::from_str::<Config>(&content) {
-                if parsed.background_path.is_some() {
-                    merged.background_path = parsed.background_path;
+            match toml::from_str::<Config>(&content) {
+                Ok(parsed) => {
+                    log::debug!("Config loaded: {}", path.display());
+                    if parsed.background_path.is_some() {
+                        merged.background_path = parsed.background_path;
+                    }
+                    if parsed.background_blur.is_some() {
+                        merged.background_blur = parsed.background_blur;
+                    }
                }
-                if parsed.background_blur.is_some() {
-                    merged.background_blur = parsed.background_blur;
+                Err(e) => {
+                    log::warn!("Failed to parse {}: {e}", path.display());
                }
            }
        }
@@ -58,17 +64,20 @@ pub fn resolve_background_path_with(config: &Config, moonarch_wallpaper: &Path)
    if let Some(ref bg) = config.background_path {
        let path = PathBuf::from(bg);
        if path.is_file() {
+            log::debug!("Wallpaper source: config ({})", path.display());
            return path;
        }
    }

    // Moonarch ecosystem default
    if moonarch_wallpaper.is_file() {
+        log::debug!("Wallpaper source: moonarch default ({})", moonarch_wallpaper.display());
        return moonarch_wallpaper.to_path_buf();
    }

    // GResource fallback path (loaded from compiled resources at runtime)
    let prefix = crate::GRESOURCE_PREFIX;
+    log::debug!("Wallpaper source: GResource fallback");
    PathBuf::from(format!("{prefix}/wallpaper.jpg"))
 }

@@ -185,4 +194,35 @@ mod tests {
        let result = resolve_background_path_with(&config, Path::new("/nonexistent"));
        assert!(result.to_str().unwrap().contains("wallpaper.jpg"));
    }
+
+    #[test]
+    fn load_config_ignores_invalid_toml_syntax() {
+        let dir = tempfile::tempdir().unwrap();
+        let conf = dir.path().join("bad.toml");
+        fs::write(&conf, "this is not valid [[[ toml").unwrap();
+        let paths = vec![conf];
+        let config = load_config(Some(&paths));
+        assert!(config.background_path.is_none());
+        assert!(config.background_blur.is_none());
+    }
+
+    #[test]
+    fn load_config_ignores_wrong_field_types() {
+        let dir = tempfile::tempdir().unwrap();
+        let conf = dir.path().join("wrong_type.toml");
+        fs::write(&conf, "background_blur = \"not_a_number\"\n").unwrap();
+        let paths = vec![conf];
+        let config = load_config(Some(&paths));
+        assert!(config.background_blur.is_none());
+    }
+
+    #[test]
+    fn load_config_accepts_negative_blur() {
+        let dir = tempfile::tempdir().unwrap();
+        let conf = dir.path().join("negative.toml");
+        fs::write(&conf, "background_blur = -5.0\n").unwrap();
+        let paths = vec![conf];
+        let config = load_config(Some(&paths));
+        assert_eq!(config.background_blur, Some(-5.0));
+    }
 }
@@ -112,15 +112,20 @@ fn read_lang_from_conf(path: &Path) -> Option<String> {

 /// Determine the system language from LANG env var or /etc/locale.conf.
 pub fn detect_locale() -> String {
-    let lang = env::var("LANG")
-        .ok()
-        .filter(|s| !s.is_empty())
-        .or_else(|| read_lang_from_conf(Path::new(DEFAULT_LOCALE_CONF)));
+    let (raw, source) = if let Some(val) = env::var("LANG").ok().filter(|s| !s.is_empty()) {
+        (Some(val), "LANG env")
+    } else if let Some(val) = read_lang_from_conf(Path::new(DEFAULT_LOCALE_CONF)) {
+        (Some(val), "locale.conf")
+    } else {
+        (None, "default")
+    };

-    match lang {
+    let result = match raw {
        Some(l) => parse_lang_prefix(&l),
        None => "en".to_string(),
-    }
+    };
+    log::debug!("Detected locale: {result} (source: {source})");
+    result
 }

 /// Return the string table for the given locale, defaulting to English.
@@ -53,13 +53,16 @@ fn activate(app: &gtk::Application) {

    load_css(&display);

-    // Resolve wallpaper once, decode texture once, share across all windows
+    // Resolve wallpaper once, decode texture once, share across all windows.
+    // Blur is applied on the GPU via GskBlurNode at first widget realization,
+    // then cached and reused by all subsequent windows.
    let config = config::load_config(None);
    let bg_path = config::resolve_background_path(&config);
-    let texture = panel::load_background_texture(&bg_path, config.background_blur);
+    let texture = panel::load_background_texture(&bg_path);
+    let blur_cache = panel::new_blur_cache();

    // Panel on focused output (no set_monitor → compositor picks focused)
-    let panel = panel::create_panel_window(&texture, app);
+    let panel = panel::create_panel_window(&texture, config.background_blur, &blur_cache, app);
    setup_layer_shell(&panel, true, gtk4_layer_shell::Layer::Overlay);
    panel.present();

@@ -67,7 +70,7 @@ fn activate(app: &gtk::Application) {
    let monitors = display.monitors();
    for i in 0..monitors.n_items() {
        if let Some(monitor) = monitors.item(i).and_then(|obj| obj.downcast::<gdk::Monitor>().ok()) {
-            let wallpaper = panel::create_wallpaper_window(&texture, app);
+            let wallpaper = panel::create_wallpaper_window(&texture, config.background_blur, &blur_cache, app);
            setup_layer_shell(&wallpaper, false, gtk4_layer_shell::Layer::Top);
            wallpaper.set_monitor(Some(&monitor));
            wallpaper.present();
@@ -75,10 +78,27 @@ fn activate(app: &gtk::Application) {
    }
 }

+fn setup_logging() {
+    match systemd_journal_logger::JournalLog::new() {
+        Ok(logger) => {
+            if let Err(e) = logger.install() {
+                eprintln!("Failed to install journal logger: {e}");
+            }
+        }
+        Err(e) => {
+            eprintln!("Failed to create journal logger: {e}");
+        }
+    }
+    let level = if std::env::var("MOONSET_DEBUG").is_ok() {
+        log::LevelFilter::Debug
+    } else {
+        log::LevelFilter::Info
+    };
+    log::set_max_level(level);
+}
+
 fn main() {
-    env_logger::Builder::from_default_env()
-        .filter_level(log::LevelFilter::Info)
-        .init();
+    setup_logging();
    log::info!("Moonset starting");

    // Register compiled GResources
@@ -6,10 +6,10 @@ use gdk_pixbuf::Pixbuf;
 use glib::clone;
 use gtk4::prelude::*;
 use gtk4::{self as gtk, gio};
-use image::imageops;
 use std::cell::RefCell;
 use std::path::Path;
 use std::rc::Rc;
+use std::time::Duration;

 use crate::i18n::{load_strings, Strings};
 use crate::power::{self, PowerError};
@@ -27,6 +27,7 @@ pub struct ActionDef {
    pub label_attr: fn(&Strings) -> &'static str,
    pub error_attr: fn(&Strings) -> &'static str,
    pub confirm_attr: Option<fn(&Strings) -> &'static str>,
+    pub quit_after: bool,
 }

 /// All 5 power action definitions.
@@ -40,6 +41,7 @@ pub fn action_definitions() -> Vec<ActionDef> {
            label_attr: |s| s.lock_label,
            error_attr: |s| s.lock_failed,
            confirm_attr: None,
+            quit_after: true,
        },
        ActionDef {
            name: "logout",
@@ -49,6 +51,7 @@ pub fn action_definitions() -> Vec<ActionDef> {
            label_attr: |s| s.logout_label,
            error_attr: |s| s.logout_failed,
            confirm_attr: Some(|s| s.logout_confirm),
+            quit_after: true,
        },
        ActionDef {
            name: "hibernate",
@@ -58,6 +61,7 @@ pub fn action_definitions() -> Vec<ActionDef> {
            label_attr: |s| s.hibernate_label,
            error_attr: |s| s.hibernate_failed,
            confirm_attr: Some(|s| s.hibernate_confirm),
+            quit_after: false,
        },
        ActionDef {
            name: "reboot",
@@ -67,6 +71,7 @@ pub fn action_definitions() -> Vec<ActionDef> {
            label_attr: |s| s.reboot_label,
            error_attr: |s| s.reboot_failed,
            confirm_attr: Some(|s| s.reboot_confirm),
+            quit_after: false,
        },
        ActionDef {
            name: "shutdown",
@@ -76,16 +81,18 @@ pub fn action_definitions() -> Vec<ActionDef> {
            label_attr: |s| s.shutdown_label,
            error_attr: |s| s.shutdown_failed,
            confirm_attr: Some(|s| s.shutdown_confirm),
+            quit_after: false,
        },
    ]
 }

 /// Load the wallpaper as a texture once, for sharing across all windows.
-/// When `blur_radius` is `Some(sigma)` with sigma > 0, a Gaussian blur is applied.
-pub fn load_background_texture(bg_path: &Path, blur_radius: Option<f32>) -> gdk::Texture {
+/// Blur is applied on the GPU via GskBlurNode at widget realization time.
+pub fn load_background_texture(bg_path: &Path) -> gdk::Texture {
    let fallback = format!("{}/wallpaper.jpg", crate::GRESOURCE_PREFIX);

-    let texture = if bg_path.starts_with(crate::GRESOURCE_PREFIX) {
+    log::debug!("Background: {}", bg_path.display());
+    if bg_path.starts_with(crate::GRESOURCE_PREFIX) {
        let resource_path = bg_path.to_str().unwrap_or(&fallback);
        gdk::Texture::from_resource(resource_path)
    } else {
@@ -93,45 +100,54 @@ pub fn load_background_texture(bg_path: &Path, blur_radius: Option<f32>) -> gdk:
        gdk::Texture::from_file(&file).unwrap_or_else(|_| {
            gdk::Texture::from_resource(&fallback)
        })
-    };
-
-    match blur_radius {
-        Some(sigma) if sigma > 0.0 => apply_blur(&texture, sigma),
-        _ => texture,
    }
 }

-/// Apply Gaussian blur to a texture and return a blurred texture.
-fn apply_blur(texture: &gdk::Texture, sigma: f32) -> gdk::Texture {
-    let width = texture.width() as u32;
-    let height = texture.height() as u32;
-    let stride = width as usize * 4;
-    let mut pixel_data = vec![0u8; stride * height as usize];
-    texture.download(&mut pixel_data, stride);
+// -- GPU blur via GskBlurNode -------------------------------------------------

-    let img = image::RgbaImage::from_raw(width, height, pixel_data)
-        .expect("pixel buffer size matches texture dimensions");
-    let blurred = imageops::blur(&image::DynamicImage::ImageRgba8(img), sigma);
-
-    let bytes = glib::Bytes::from(blurred.as_raw());
-    let mem_texture = gdk::MemoryTexture::new(
-        width as i32,
-        height as i32,
-        gdk::MemoryFormat::B8g8r8a8Premultiplied,
-        &bytes,
-        stride,
+/// Render a blurred texture using the GPU via GskBlurNode.
+fn render_blurred_texture(
+    widget: &impl IsA<gtk::Widget>,
+    texture: &gdk::Texture,
+    sigma: f32,
+) -> Option<gdk::Texture> {
+    let native = widget.native()?;
+    let renderer = native.renderer()?;
+    let snapshot = gtk::Snapshot::new();
+    let bounds = graphene_rs::Rect::new(
+        0.0, 0.0, texture.width() as f32, texture.height() as f32,
    );
-    mem_texture.upcast()
+    snapshot.push_blur(sigma as f64);
+    snapshot.append_texture(texture, &bounds);
+    snapshot.pop();
+    let node = snapshot.to_node()?;
+    Some(renderer.render_texture(&node, None))
+}
+
+/// Fade out all windows and quit the app after the CSS transition completes.
+fn fade_out_and_quit(app: &gtk::Application) {
+    for window in app.windows() {
+        window.remove_css_class("visible");
+    }
+    let app = app.clone();
+    glib::timeout_add_local_once(Duration::from_millis(250), move || {
+        app.quit();
+    });
+}
+
+/// Create a new shared blur cache for GPU-blurred wallpaper textures.
+pub fn new_blur_cache() -> BlurCache {
+    Rc::new(RefCell::new(None))
 }

 /// Create a wallpaper-only window for secondary monitors.
-pub fn create_wallpaper_window(texture: &gdk::Texture, app: &gtk::Application) -> gtk::ApplicationWindow {
+pub fn create_wallpaper_window(texture: &gdk::Texture, blur_radius: Option<f32>, blur_cache: &BlurCache, app: &gtk::Application) -> gtk::ApplicationWindow {
    let window = gtk::ApplicationWindow::builder()
        .application(app)
        .build();
    window.add_css_class("wallpaper");

-    let background = create_background_picture(texture);
+    let background = create_background_picture(texture, blur_radius, blur_cache);
    window.set_child(Some(&background));

    // Fade-in on map
@@ -149,7 +165,7 @@ pub fn create_wallpaper_window(texture: &gdk::Texture, app: &gtk::Application) -
 }

 /// Create the main panel window with action buttons and confirm flow.
-pub fn create_panel_window(texture: &gdk::Texture, app: &gtk::Application) -> gtk::ApplicationWindow {
+pub fn create_panel_window(texture: &gdk::Texture, blur_radius: Option<f32>, blur_cache: &BlurCache, app: &gtk::Application) -> gtk::ApplicationWindow {
    let window = gtk::ApplicationWindow::builder()
        .application(app)
        .build();
@@ -162,6 +178,7 @@ pub fn create_panel_window(texture: &gdk::Texture, app: &gtk::Application) -> gt
        home: dirs::home_dir().unwrap_or_default(),
        uid: u32::MAX,
    });
+    log::debug!("User: {} ({})", user.display_name, user.username);

    // State for confirm box
    let confirm_box: Rc<RefCell<Option<gtk::Box>>> = Rc::new(RefCell::new(None));
@@ -171,7 +188,7 @@ pub fn create_panel_window(texture: &gdk::Texture, app: &gtk::Application) -> gt
    window.set_child(Some(&overlay));

    // Background wallpaper
-    let background = create_background_picture(texture);
+    let background = create_background_picture(texture, blur_radius, blur_cache);
    overlay.set_child(Some(&background));

    // Click on background dismisses the menu
@@ -180,7 +197,7 @@ pub fn create_panel_window(texture: &gdk::Texture, app: &gtk::Application) -> gt
        #[weak]
        app,
        move |_, _, _, _| {
-            app.quit();
+            fade_out_and_quit(&app);
        }
    ));
    background.add_controller(click_controller);
@@ -250,7 +267,7 @@ pub fn create_panel_window(texture: &gdk::Texture, app: &gtk::Application) -> gt
        glib::Propagation::Proceed,
        move |_, keyval, _, _| {
            if keyval == gdk::Key::Escape {
-                app.quit();
+                fade_out_and_quit(&app);
                glib::Propagation::Stop
            } else {
                glib::Propagation::Proceed
@@ -275,12 +292,40 @@ pub fn create_panel_window(texture: &gdk::Texture, app: &gtk::Application) -> gt
    window
 }

-/// Create a Picture widget for the wallpaper background from a shared texture.
-fn create_background_picture(texture: &gdk::Texture) -> gtk::Picture {
+/// Shared cache for the GPU-blurred wallpaper texture.
+/// Computed once on first window realize, reused by all subsequent windows.
+type BlurCache = Rc<RefCell<Option<gdk::Texture>>>;
+
+/// Create a Picture widget for the wallpaper background, optionally with GPU blur.
+/// When a blur_cache is provided, the blurred texture is computed once and shared.
+fn create_background_picture(
+    texture: &gdk::Texture,
+    blur_radius: Option<f32>,
+    blur_cache: &BlurCache,
+) -> gtk::Picture {
    let background = gtk::Picture::for_paintable(texture);
    background.set_content_fit(gtk::ContentFit::Cover);
    background.set_hexpand(true);
    background.set_vexpand(true);
+
+    if let Some(sigma) = blur_radius.filter(|s| *s > 0.0) {
+        let texture = texture.clone();
+        let cache = blur_cache.clone();
+        background.connect_realize(move |picture| {
+            // Use cached blur if available, otherwise compute and cache
+            if let Some(ref cached) = *cache.borrow() {
+                log::debug!("Blur cache hit");
+                picture.set_paintable(Some(cached));
+                return;
+            }
+            log::debug!("Blur cache miss, rendering GPU blur");
+            if let Some(blurred) = render_blurred_texture(picture, &texture, sigma) {
+                picture.set_paintable(Some(&blurred));
+                *cache.borrow_mut() = Some(blurred);
+            }
+        });
+    }
+
    background
 }

@@ -450,9 +495,11 @@ fn execute_action(
    error_label: &gtk::Label,
 ) {
    dismiss_confirm(confirm_area, confirm_box);
+    log::debug!("Executing power action: {}", action_def.name);

    let action_fn = action_def.action_fn;
    let action_name = action_def.name;
+    let quit_after = action_def.quit_after;
    let error_message = (action_def.error_attr)(strings).to_string();

    // Use glib::spawn_future_local + gio::spawn_blocking to avoid Send issues
@@ -468,9 +515,8 @@ fn execute_action(

            match result {
                Ok(Ok(())) => {
-                    // Lock action: quit after successful execution
-                    if action_name == "lock" {
-                        app.quit();
+                    if quit_after {
+                        fade_out_and_quit(&app);
                    }
                }
                Ok(Err(e)) => {
@@ -494,6 +540,7 @@ fn load_avatar_async(image: &gtk::Image, window: &gtk::ApplicationWindow, user:

    match avatar_path {
        Some(path) => {
+            log::debug!("Avatar source: file {}", path.display());
            // File-based avatar: load and scale in background thread
            glib::spawn_future_local(clone!(
                #[weak]
@@ -501,7 +548,7 @@ fn load_avatar_async(image: &gtk::Image, window: &gtk::ApplicationWindow, user:
                async move {
                    let result = gio::spawn_blocking(move || {
                        Pixbuf::from_file_at_scale(
-                            path.to_str().unwrap_or(""),
+                            &path,
                            AVATAR_SIZE,
                            AVATAR_SIZE,
                            true,
@@ -519,6 +566,7 @@ fn load_avatar_async(image: &gtk::Image, window: &gtk::ApplicationWindow, user:
            ));
        }
        None => {
+            log::debug!("Avatar source: default SVG");
            // Default SVG avatar: needs widget color, keep synchronous
            set_default_avatar(image, window);
        }
@@ -627,4 +675,20 @@ mod tests {
        let confirm_fn = defs[1].confirm_attr.unwrap();
        assert_eq!(confirm_fn(strings), "Wirklich abmelden?");
    }
+
+    #[test]
+    fn lock_and_logout_quit_after() {
+        let defs = action_definitions();
+        assert!(defs[0].quit_after, "lock should quit after");
+        assert!(defs[1].quit_after, "logout should quit after");
+    }
+
+    #[test]
+    fn destructive_actions_do_not_quit_after() {
+        let defs = action_definitions();
+        for def in &defs[2..] {
+            assert!(!def.quit_after, "{} should not quit after", def.name);
+        }
+    }
+
 }
@@ -31,6 +31,7 @@ impl std::error::Error for PowerError {}

 /// Run a command with timeout and return a PowerError on failure.
 fn run_command(action: &'static str, program: &str, args: &[&str]) -> Result<(), PowerError> {
+    log::debug!("Power action: {action} ({program} {args:?})");
    let mut child = Command::new(program)
        .args(args)
        .stdout(Stdio::piped())
@@ -45,6 +46,9 @@ fn run_command(action: &'static str, program: &str, args: &[&str]) -> Result<(),
    loop {
        match child.try_wait() {
            Ok(Some(status)) => {
+                if status.success() {
+                    log::debug!("Power action {action} completed");
+                }
                if !status.success() {
                    let mut stderr_buf = String::new();
                    if let Some(mut stderr) = child.stderr.take() {
@@ -76,8 +80,21 @@ fn run_command(action: &'static str, program: &str, args: &[&str]) -> Result<(),
 }

 /// Lock the current session by launching moonlock.
+/// Spawns moonlock as a detached process and returns immediately —
+/// moonlock runs independently until the user unlocks.
 pub fn lock() -> Result<(), PowerError> {
-    run_command("lock", "/usr/bin/moonlock", &[])
+    log::debug!("Power action: lock (spawning moonlock)");
+    Command::new("/usr/bin/moonlock")
+        .stdin(Stdio::null())
+        .stdout(Stdio::null())
+        .stderr(Stdio::null())
+        .spawn()
+        .map_err(|e| PowerError::CommandFailed {
+            action: "lock",
+            message: e.to_string(),
+        })?;
+    // Child handle is dropped here — moonlock continues running independently.
+    Ok(())
 }

 /// Quit the Niri compositor (logout).
@@ -55,6 +55,7 @@ pub fn get_avatar_path_with(
    // ~/.face takes priority
    let face = home.join(".face");
    if face.exists() {
+        log::debug!("Avatar: using ~/.face");
        return Some(face);
    }

@@ -63,6 +64,7 @@ pub fn get_avatar_path_with(
        if accountsservice_dir.exists() {
            let icon = accountsservice_dir.join(name);
            if icon.exists() {
+                log::debug!("Avatar: using AccountsService icon");
                return Some(icon);
            }
        }
Author	SHA1	Message	Date
nevaforget	71670eb263	feat: switch to systemd-journal-logger, add debug logging (v0.6.0) Replace env_logger with systemd-journal-logger for consistent logging across moonset/moonlock/moongreet. Add MOONSET_DEBUG env var and debug statements across all modules. Also includes shared blur cache for multi-monitor and detached moonlock spawn for lock action.	2026-03-28 22:58:25 +01:00
nevaforget	14affb1533	perf: replace CPU blur with GPU blur via GskBlurNode (v0.5.0) Replace image crate + disk cache blur with GPU-side GskBlurNode, symmetric with moonlock and moongreet. Removes ~15 transitive dependencies and ~160 lines of caching code. Blur now happens once on the GPU at widget realization — zero startup latency, no cache management needed.	2026-03-28 22:35:18 +01:00
nevaforget	4d8e306b74	feat: add fade-out animation on dismiss for smooth visual exit Without this, app.quit() destroys windows instantly, creating a jarring pop-out. Now all windows fade out over 250ms (matching the fade-in) before the app exits. Uses the same CSS opacity transition — just removes the "visible" class and defers quit via glib timeout.	2026-03-28 21:50:03 +01:00
nevaforget	2e88a9b6c4	feat: activate fade-in animation for panel and wallpaper windows The Rust code already adds a "visible" CSS class on map, but the stylesheet had no corresponding opacity transition. Add 250ms ease-in fade via GPU-accelerated CSS opacity to eliminate the visual pop-in.	2026-03-28 21:46:08 +01:00
nevaforget	412ed159a4	fix: address audit findings — blur channel mismatch, logout quit, config error logging - Fix BGRA→RGBA channel swap in apply_blur so image::RgbaImage semantics match the actual pixel data from GDK texture download - Logout now calls app.quit() like lock does, via new quit_after field on ActionDef (replaces fragile magic string comparison) - Log TOML parse errors to stderr instead of silently ignoring - Remove pointless zlib compression of JPEG wallpaper in GResource - Add tests for quit_after behavior and config error handling	2026-03-28 21:39:34 +01:00
nevaforget	478caed8e0	perf: cache blurred wallpaper to disk to avoid re-blur on startup First launch with blur blurs and saves to ~/.cache/moonset/. Subsequent starts load the cached PNG directly (~9x faster). Cache invalidates when wallpaper path, size, mtime, or sigma changes.	2026-03-28 21:22:48 +01:00