From d41add32c48f28dd498271b1552daceba8c85600 Mon Sep 17 00:00:00 2001
From: Valentin Popov <valentin@popov.link>
Date: Wed, 24 Jun 2026 01:05:31 +0400
Subject: feat: create Vulkan swapchain probe

---
 apps/fparkan-vulkan-smoke/src/main.rs | 43 +++++++++++++++++++++++++----------
 1 file changed, 31 insertions(+), 12 deletions(-)

(limited to 'apps/fparkan-vulkan-smoke/src')

diff --git a/apps/fparkan-vulkan-smoke/src/main.rs b/apps/fparkan-vulkan-smoke/src/main.rs
index 9425c31..f742502 100644
--- a/apps/fparkan-vulkan-smoke/src/main.rs
+++ b/apps/fparkan-vulkan-smoke/src/main.rs
@@ -15,8 +15,9 @@ use fparkan_platform::{NativeWindowHandles, WindowPort};
 use fparkan_platform_winit::{probe_smoke_window, WinitWindowPlan};
 use fparkan_render_vulkan::{
     create_vulkan_instance_probe, create_vulkan_logical_device_probe, create_vulkan_surface_probe,
-    probe_vulkan_loader, triangle_shader_manifest, validate_shader_manifest, VulkanInstanceConfig,
-    VulkanInstanceProbe, VulkanLogicalDeviceProbe,
+    create_vulkan_swapchain_probe, probe_vulkan_loader, triangle_shader_manifest,
+    validate_shader_manifest, VulkanInstanceConfig, VulkanInstanceProbe, VulkanLogicalDeviceProbe,
+    VulkanSwapchainProbe,
 };
 use std::path::PathBuf;
 use std::process::Command;
@@ -424,7 +425,14 @@ impl VulkanBootstrapProbe {
                 self.window_height.unwrap_or(1).max(1),
             ),
         ) {
-            Ok(device) => self.record_logical_device_probe(&device),
+            Ok(device) => match create_vulkan_swapchain_probe(instance, surface, &device) {
+                Ok(swapchain) => self.record_swapchain_probe(&device, &swapchain),
+                Err(err) => {
+                    self.record_logical_device_probe(&device);
+                    self.swapchain_status = VulkanSwapchainStatus::Failed;
+                    self.swapchain_error = Some(err.to_string());
+                }
+            },
             Err(err) => {
                 self.device_status = VulkanDeviceStatus::Failed;
                 self.device_error = Some(err.to_string());
@@ -450,11 +458,22 @@ impl VulkanBootstrapProbe {
                 .try_into()
                 .unwrap_or(u32::MAX),
         );
-        self.swapchain_status = VulkanSwapchainStatus::Planned;
         self.swapchain_width = Some(device.runtime.swapchain.extent.0);
         self.swapchain_height = Some(device.runtime.swapchain.extent.1);
         self.swapchain_image_count = Some(device.runtime.swapchain.image_count);
     }
+
+    fn record_swapchain_probe(
+        &mut self,
+        device: &VulkanLogicalDeviceProbe,
+        swapchain: &VulkanSwapchainProbe,
+    ) {
+        self.record_logical_device_probe(device);
+        self.swapchain_status = VulkanSwapchainStatus::Created;
+        self.swapchain_width = Some(swapchain.report.plan.extent.0);
+        self.swapchain_height = Some(swapchain.report.plan.extent.1);
+        self.swapchain_image_count = Some(swapchain.report.image_count);
+    }
 }
 
 #[derive(Clone, Copy, Debug, Eq, PartialEq)]
@@ -564,7 +583,7 @@ impl VulkanLogicalDeviceStatus {
 #[derive(Clone, Copy, Debug, Eq, PartialEq)]
 enum VulkanSwapchainStatus {
     Skipped,
-    Planned,
+    Created,
     Failed,
 }
 
@@ -572,7 +591,7 @@ impl VulkanSwapchainStatus {
     const fn as_str(self) -> &'static str {
         match self {
             Self::Skipped => "skipped",
-            Self::Planned => "planned",
+            Self::Created => "created",
             Self::Failed => "failed",
         }
     }
@@ -691,9 +710,9 @@ fn validate_smoke_options(
                     "passed native smoke report requires created Vulkan logical device".to_string(),
                 );
             }
-            if bootstrap.swapchain_status != VulkanSwapchainStatus::Planned {
+            if bootstrap.swapchain_status != VulkanSwapchainStatus::Created {
                 return Err(
-                    "passed native smoke report requires planned Vulkan swapchain".to_string(),
+                    "passed native smoke report requires created Vulkan swapchain".to_string(),
                 );
             }
         }
@@ -953,7 +972,7 @@ mod tests {
             logical_device_present_queue_family: Some(0),
             logical_device_enabled_extension_count: Some(1),
             logical_device_error: None,
-            swapchain_status: VulkanSwapchainStatus::Planned,
+            swapchain_status: VulkanSwapchainStatus::Created,
             swapchain_width: Some(1280),
             swapchain_height: Some(720),
             swapchain_image_count: Some(3),
@@ -1340,7 +1359,7 @@ mod tests {
     }
 
     #[test]
-    fn rejects_passed_without_planned_swapchain() {
+    fn rejects_passed_without_created_swapchain() {
         let options = SmokeOptions::parse(&strings(&[
             "--platform",
             "linux",
@@ -1365,11 +1384,11 @@ mod tests {
                 &options,
                 &VulkanBootstrapProbe {
                     swapchain_status: VulkanSwapchainStatus::Failed,
-                    swapchain_error: Some("Vulkan swapchain has no surface format".to_string()),
+                    swapchain_error: Some("Vulkan swapchain creation failed".to_string()),
                     ..probe_fixture()
                 },
             ),
-            Err("passed native smoke report requires planned Vulkan swapchain".to_string())
+            Err("passed native smoke report requires created Vulkan swapchain".to_string())
         );
     }
 
-- 
cgit v1.2.3