fix(renderer): correctly handle lost/outdated surface errors

Previously, minimizing and then restoring the window would cause a `SurfaceError::Lost` panic. This was because the render loop attempted to rebuild render targets without first reconfiguring the underlying `wgpu::Surface`, which becomes invalid after being lost.

This commit fixes the issue by:
1. Introducing a new `WgpuApp::resize_surface()` method that properly reconfigures the surface with the device and then rebuilds all dependent render targets (pass targets, MSAA textures, etc.).
2. Updating the main render loop in `renderer.rs` to call this new method upon receiving a `SurfaceError::Lost` or `SurfaceError::Outdated`, ensuring a full recovery.
3. Refactoring the existing `resize_if_needed` logic to use the new centralized `resize_surface` method for consistency.

Author: shadow3aaa

tessera-ui/src/renderer.rs

@@ -723,10 +723,22 @@ Fps: {:.2}
         }
 
         debug!("Rendering draw commands...");
-        // Forward commands to WgpuApp::render which accepts the same iterator type
-        args.app.render(commands).unwrap_or_else(|e| {
-            error!("Render error: {e}");
-        });
+        if let Err(e) = args.app.render(commands) {
+            match e {
+                wgpu::SurfaceError::Outdated | wgpu::SurfaceError::Lost => {
+                    debug!("Surface outdated/lost, resizing...");
+                    args.app.resize_surface();
+                }
+                wgpu::SurfaceError::Timeout => warn!("Surface timeout. Frame will be dropped."),
+                wgpu::SurfaceError::OutOfMemory => {
+                    error!("Surface out of memory. Panicking.");
+                    panic!("Surface out of memory");
+                }
+                _ => {
+                    error!("Surface error: {e}. Attempting to continue.");
+                }
+            }
+        }
         let render_cost = render_timer.elapsed();
         debug!("Rendered to surface in {render_cost:?}");
         render_cost

tessera-ui/src/renderer/app.rs

@@ -365,48 +365,52 @@ impl WgpuApp {
         self.size
     }
 
-    pub(crate) fn resize_pass_targets_if_needed(&mut self) {
-        if self.size_changed {
-            self.pass_a.texture.destroy();
-            self.pass_b.texture.destroy();
-            self.compute_target_a.texture.destroy();
-            self.compute_target_b.texture.destroy();
-
-            self.pass_a = Self::create_pass_target(&self.gpu, &self.config, "A");
-            self.pass_b = Self::create_pass_target(&self.gpu, &self.config, "B");
-            self.compute_target_a = Self::create_compute_pass_target(
-                &self.gpu,
-                &self.config,
-                TextureFormat::Rgba8Unorm,
-                "Compute A",
-            );
-            self.compute_target_b = Self::create_compute_pass_target(
-                &self.gpu,
-                &self.config,
-                TextureFormat::Rgba8Unorm,
-                "Compute B",
-            );
+    pub(crate) fn resize_surface(&mut self) {
+        if self.size.width > 0 && self.size.height > 0 {
+            self.config.width = self.size.width;
+            self.config.height = self.size.height;
+            self.surface.configure(&self.gpu, &self.config);
+            self.rebuild_pass_targets();
+        }
+    }
 
-            if self.sample_count > 1 {
-                if let Some(t) = self.msaa_texture.take() {
-                    t.destroy();
-                }
-                let (msaa_texture, msaa_view) =
-                    Self::make_msaa_resources(&self.gpu, self.sample_count, &self.config);
-                self.msaa_texture = msaa_texture;
-                self.msaa_view = msaa_view;
+    pub(crate) fn rebuild_pass_targets(&mut self) {
+        self.pass_a.texture.destroy();
+        self.pass_b.texture.destroy();
+        self.compute_target_a.texture.destroy();
+        self.compute_target_b.texture.destroy();
+
+        self.pass_a = Self::create_pass_target(&self.gpu, &self.config, "A");
+        self.pass_b = Self::create_pass_target(&self.gpu, &self.config, "B");
+        self.compute_target_a = Self::create_compute_pass_target(
+            &self.gpu,
+            &self.config,
+            TextureFormat::Rgba8Unorm,
+            "Compute A",
+        );
+        self.compute_target_b = Self::create_compute_pass_target(
+            &self.gpu,
+            &self.config,
+            TextureFormat::Rgba8Unorm,
+            "Compute B",
+        );
+
+        if self.sample_count > 1 {
+            if let Some(t) = self.msaa_texture.take() {
+                t.destroy();
             }
+            let (msaa_texture, msaa_view) =
+                Self::make_msaa_resources(&self.gpu, self.sample_count, &self.config);
+            self.msaa_texture = msaa_texture;
+            self.msaa_view = msaa_view;
         }
     }
 
     /// Resize the surface if needed.
     pub(crate) fn resize_if_needed(&mut self) -> bool {
         let result = self.size_changed;
         if self.size_changed {
-            self.config.width = self.size.width;
-            self.config.height = self.size.height;
-            self.resize_pass_targets_if_needed();
-            self.surface.configure(&self.gpu, &self.config);
+            self.resize_surface();
             self.size_changed = false;
         }
         result