From 283a1484202601689f905e4b02fc483f347f1fdf Mon Sep 17 00:00:00 2001
From: Naseq <nasim.naseq@gmail.com>
Date: Sun, 8 Nov 2026 09:07:05 +0430
Subject: [PATCH] ui-egui: cfg opening_generation off wasm32; lint the wasm set
 with -D warnings (#1954)

---
 crates/ui-egui/src/app.rs | 1 +
 crates/ui-egui/src/lib.rs | 3 ++-
 xtask/src/main.rs         | 8 +++++---
 3 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/crates/ui-egui/src/app.rs b/crates/ui-egui/src/app.rs
index 178ab24ba..5c1314657 100644
--- a/crates/ui-egui/src/app.rs
+++ b/crates/ui-egui/src/app.rs
@@ -72,6 +72,7 @@ impl SlideApp {
             synthetic_injected_time: None,
             ribbon_scroll: HashMap::new(),
             show_instance: 0,
+            #[cfg(not(target_arch = "wasm32"))]
             opening_generation: 0,
             canvas_rect: None,
             eyedropper: None,
diff --git a/crates/ui-egui/src/lib.rs b/crates/ui-egui/src/lib.rs
index 08fcc2391..0d2f1787b 100644
--- a/crates/ui-egui/src/lib.rs
+++ b/crates/ui-egui/src/lib.rs
@@ -276,7 +276,8 @@ pub struct SlideApp {
     ribbon_scroll: HashMap<String, u32>,
     /// Monotonic identity for a new slide show instance (`start_show`).
     show_instance: u64,
-    /// Monotonic identity for a new background-open attempt.
+    /// Monotonic identity for a new background-open attempt (the web build has no threads).
+    #[cfg(not(target_arch = "wasm32"))]
     opening_generation: u64,
     /// Canvas screen rect and points-per-slide-point scale (for control & tests).
     pub canvas_rect: Option<egui::Rect>,
diff --git a/xtask/src/main.rs b/xtask/src/main.rs
index 632769fb3..15a0d604e 100644
--- a/xtask/src/main.rs
+++ b/xtask/src/main.rs
@@ -147,13 +147,15 @@ fn cmd_wasm() -> Result<(), String> {
     if set.is_empty() {
         return Err("no wasm-safe crates found in the workspace metadata".into());
     }
+    // Clippy with `-D warnings`, as the host step: a plain `check` let wasm-only dead code through (#1954).
     let mut c = cargo();
-    c.args(["check", "--target", "wasm32-unknown-unknown", "--keep-going"]);
+    c.args(["clippy", "--target", "wasm32-unknown-unknown", "--keep-going"]);
     for pkg in &set {
         c.args(["-p", pkg]);
     }
-    run(c, &format!("cargo check --target wasm32-unknown-unknown --keep-going -p {}", set.join(" -p ")))?;
-    println!("\nwasm32-unknown-unknown check: ok ({} crates: {})", set.len(), set.join(", "));
+    c.args(["--", "-D", "warnings"]);
+    run(c, &format!("cargo clippy --target wasm32-unknown-unknown --keep-going -p {} -- -D warnings", set.join(" -p ")))?;
+    println!("\nwasm32-unknown-unknown clippy: ok ({} crates: {})", set.len(), set.join(", "));
     Ok(())
 }