🐛 fix: • Improve viewer robustness and icon discovery

  - Keep existing integrated-viewer app IDs stable to preserve cookies
  - Add safer migration for viewer storage and legacy profile cleanup
  - Improve favicon selection with manifest, SVG, and larger icon priority
  - Increase Google favicon fallback size
  - Fix auto-hidden headerbar reveal threshold blocking page clicks
  - Enable additional WebKit media settings for embedded playback
  - Prefer external browsers for DRM/media webapps such as YouTube Music
  - Stabilize env-var tests in the launcher shim
  - Bump app version to 4.0.1 and add AppStream release notes

Author: talesam

Cargo.lock

@@ -8,7 +8,7 @@ members = [
 ]
 
 [workspace.package]
-version = "4.0.0"
+version = "4.0.1"
 edition = "2021"
 license = "GPL-3.0-or-later"
 repository = "https://github.com/biglinux/biglinux-webapps"

Cargo.toml

@@ -462,6 +462,11 @@
     <control>touch</control>
   </supports>
   <releases>
+    <release version="4.0.1" date="2026-05-16">
+      <description>
+        <p>Fixes Wayland desktop IDs and icon mapping for browser-backed webapps, preserves integrated-viewer profiles across ID migrations, improves favicon quality selection, and refines auto-hidden headerbar and media playback behavior.</p>
+      </description>
+    </release>
     <release version="4.0.0" date="2026-04-23">
       <description>
         <p>First stable release of the Rust rewrite. Replaces the previous Python/GTK3 stack with a native Rust/GTK4/libadwaita build.</p>
@@ -643,4 +648,4 @@
       </description>
     </release>
   </releases>
-</component>
+</component>

biglinux-webapps/usr/share/metainfo/br.com.biglinux.webapps.metainfo.xml

@@ -1,11 +1,11 @@
 use crate::models::{AppMode, WebApp};
 
-use super::paths::desktop_file_id;
+use super::paths::viewer_app_id;
 use super::sanitize::{sanitize_desktop_field, sanitize_desktop_value};
 use super::wm_class::derive_wm_class;
 
 pub fn generate_desktop_entry(webapp: &WebApp) -> String {
-    let app_id = desktop_file_id(&webapp.app_url);
+    let app_id = viewer_app_id(webapp);
     let exec = build_exec_command(webapp, &app_id);
     let wm_class = sanitize_desktop_field(&derive_wm_class(webapp));
 

crates/webapps-core/src/desktop/builder.rs

@@ -8,7 +8,7 @@ pub use builder::generate_desktop_entry;
 pub use icon::{persist_icon, webapp_icons_dir};
 pub use paths::{
     desktop_file_id, desktop_file_path, install_desktop_entry, legacy_host_desktop_file_id,
-    remove_desktop_entry, remove_desktop_file, viewer_desktop_filename,
+    remove_desktop_entry, remove_desktop_file, viewer_app_id, viewer_desktop_filename,
 };
 pub use wm_class::{canonical_browser_desktop_filename, chromium_browser_app_id};
 
@@ -71,6 +71,13 @@ mod tests {
         );
     }
 
+    #[test]
+    fn viewer_app_id_uses_persisted_desktop_filename() {
+        let mut w = app("https://cloud.talesam.org/apps/notes", AppMode::App);
+        w.app_file = "biglinux-webapp-cloudtalesamorg.desktop".to_string();
+        assert_eq!(viewer_app_id(&w), "cloudtalesamorg");
+    }
+
     #[test]
     fn desktop_file_id_invalid_url_falls_back() {
         assert_eq!(desktop_file_id("not-a-url"), "webapp");

crates/webapps-core/src/desktop/mod.rs

@@ -45,6 +45,20 @@ pub fn viewer_desktop_filename(url: &str) -> String {
     format!("biglinux-webapp-{}.desktop", desktop_file_id(url))
 }
 
+pub fn viewer_app_id(webapp: &WebApp) -> String {
+    webapp
+        .desktop_file_name()
+        .and_then(|file_name| {
+            file_name
+                .as_str()
+                .strip_prefix("biglinux-webapp-")
+                .and_then(|name| name.strip_suffix(".desktop"))
+                .map(sanitize_id_part)
+        })
+        .filter(|id| !id.is_empty())
+        .unwrap_or_else(|| desktop_file_id(&webapp.app_url))
+}
+
 fn sanitize_id_part(value: &str) -> String {
     let mut result = String::with_capacity(value.len());
     let mut previous_was_separator = false;

crates/webapps-core/src/desktop/paths.rs

@@ -1,18 +1,15 @@
 use crate::models::{AppMode, WebApp};
 
-use super::paths::desktop_file_id;
+use super::paths::viewer_app_id;
 
 pub(super) fn derive_wm_class(webapp: &WebApp) -> String {
     match webapp.app_mode {
         AppMode::App => {
             // Must match the GTK application_id set by the viewer
-            // (`br.com.biglinux.webapp.{desktop_file_id}`), so Wayland compositors
+            // (`br.com.biglinux.webapp.{viewer_app_id}`), so Wayland compositors
             // can associate the window with this desktop entry — otherwise the
             // taskbar falls back to displaying the raw app_id and a generic icon.
-            format!(
-                "br.com.biglinux.webapp.{}",
-                desktop_file_id(&webapp.app_url)
-            )
+            format!("br.com.biglinux.webapp.{}", viewer_app_id(webapp))
         }
         // Chromium-family browsers (Brave/Chrome/Edge/Vivaldi/Chromium) ignore
         // `--class` when picking the xdg-shell `app_id` on Wayland — they

crates/webapps-core/src/desktop/wm_class.rs

@@ -131,6 +131,9 @@ fn release_lock(fd: i32, path: &str) {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use std::sync::Mutex;
+
+    static ENV_GUARD: Mutex<()> = Mutex::new(());
 
     #[test]
     fn original_desktop_name_strips_big_webapp_suffix() {
@@ -156,20 +159,28 @@ mod tests {
 
     #[test]
     fn swap_settle_falls_back_to_default_when_env_missing() {
-        // Use a serial guard via a unique env name so other tests don't interfere.
+        let _guard = ENV_GUARD
+            .lock()
+            .unwrap_or_else(|poisoned| poisoned.into_inner());
         std::env::remove_var(ENV_SWAP_SETTLE_MS);
         assert_eq!(swap_settle(), Duration::from_millis(DEFAULT_SWAP_SETTLE_MS));
     }
 
     #[test]
     fn swap_settle_honours_env_override() {
+        let _guard = ENV_GUARD
+            .lock()
+            .unwrap_or_else(|poisoned| poisoned.into_inner());
         std::env::set_var(ENV_SWAP_SETTLE_MS, "1234");
         assert_eq!(swap_settle(), Duration::from_millis(1234));
         std::env::remove_var(ENV_SWAP_SETTLE_MS);
     }
 
     #[test]
     fn swap_settle_ignores_non_numeric_env() {
+        let _guard = ENV_GUARD
+            .lock()
+            .unwrap_or_else(|poisoned| poisoned.into_inner());
         std::env::set_var(ENV_SWAP_SETTLE_MS, "not-a-number");
         assert_eq!(swap_settle(), Duration::from_millis(DEFAULT_SWAP_SETTLE_MS));
         std::env::remove_var(ENV_SWAP_SETTLE_MS);