From 0900230311ad5d2ffca03cf6e607803008e87dce Mon Sep 17 00:00:00 2001
From: Drew Newberry <385+drew@users.noreply.github.com>
Date: Thu, 27 Aug 2026 01:37:27 +0000
Subject: [PATCH] fix(server): gate in-tree tracing on Windows

Signed-off-by: Drew Newberry <385+drew@users.noreply.github.com>
---
 crates/openshell-server/src/tracing_setup.rs | 24 ++++++++++----------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/crates/openshell-server/src/tracing_setup.rs b/crates/openshell-server/src/tracing_setup.rs
index c5b8d9f4a..968a1c6a0 100644
--- a/crates/openshell-server/src/tracing_setup.rs
+++ b/crates/openshell-server/src/tracing_setup.rs
@@ -38,16 +38,16 @@ impl TracingHandle {
 
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 enum InProcessDriverTracing {
-    #[cfg(feature = "in-tree-compute-drivers")]
+    #[cfg(all(not(target_os = "windows"), feature = "in-tree-compute-drivers"))]
     Docker,
-    #[cfg(feature = "in-tree-compute-drivers")]
+    #[cfg(all(not(target_os = "windows"), feature = "in-tree-compute-drivers"))]
     Kubernetes,
-    #[cfg(feature = "in-tree-compute-drivers")]
+    #[cfg(all(not(target_os = "windows"), feature = "in-tree-compute-drivers"))]
     Podman,
 }
 
 impl InProcessDriverTracing {
-    #[cfg(feature = "in-tree-compute-drivers")]
+    #[cfg(all(not(target_os = "windows"), feature = "in-tree-compute-drivers"))]
     fn target_prefix(self) -> &'static str {
         match self {
             Self::Docker => openshell_driver_docker::otel_tracing::IN_PROCESS_TARGET_PREFIX,
@@ -58,7 +58,7 @@ impl InProcessDriverTracing {
 }
 
 fn in_process_driver_tracing(driver: &ConfiguredComputeDriver) -> Option<InProcessDriverTracing> {
-    #[cfg(feature = "in-tree-compute-drivers")]
+    #[cfg(all(not(target_os = "windows"), feature = "in-tree-compute-drivers"))]
     match driver {
         ConfiguredComputeDriver::Registered(registration) if registration.name == "docker" => {
             Some(InProcessDriverTracing::Docker)
@@ -71,7 +71,7 @@ fn in_process_driver_tracing(driver: &ConfiguredComputeDriver) -> Option<InProce
         }
         _ => None,
     }
-    #[cfg(not(feature = "in-tree-compute-drivers"))]
+    #[cfg(not(all(not(target_os = "windows"), feature = "in-tree-compute-drivers")))]
     {
         let _ = driver;
         None
@@ -79,18 +79,18 @@ fn in_process_driver_tracing(driver: &ConfiguredComputeDriver) -> Option<InProce
 }
 
 fn in_process_driver_target_prefix(driver: Option<InProcessDriverTracing>) -> Option<&'static str> {
-    #[cfg(feature = "in-tree-compute-drivers")]
+    #[cfg(all(not(target_os = "windows"), feature = "in-tree-compute-drivers"))]
     {
         driver.map(InProcessDriverTracing::target_prefix)
     }
-    #[cfg(not(feature = "in-tree-compute-drivers"))]
+    #[cfg(not(all(not(target_os = "windows"), feature = "in-tree-compute-drivers")))]
     {
         let _ = driver;
         None
     }
 }
 
-#[cfg(feature = "in-tree-compute-drivers")]
+#[cfg(all(not(target_os = "windows"), feature = "in-tree-compute-drivers"))]
 fn in_process_driver_provider(
     driver: Option<InProcessDriverTracing>,
     endpoint: Option<&str>,
@@ -109,7 +109,7 @@ fn in_process_driver_provider(
     }
 }
 
-#[cfg(not(feature = "in-tree-compute-drivers"))]
+#[cfg(not(all(not(target_os = "windows"), feature = "in-tree-compute-drivers")))]
 fn in_process_driver_provider(
     _driver: Option<InProcessDriverTracing>,
     _endpoint: Option<&str>,
@@ -117,7 +117,7 @@ fn in_process_driver_provider(
     (None, None)
 }
 
-#[cfg(feature = "in-tree-compute-drivers")]
+#[cfg(all(not(target_os = "windows"), feature = "in-tree-compute-drivers"))]
 fn in_process_driver_layer<S>(
     provider: &Option<SdkTracerProvider>,
     driver: Option<InProcessDriverTracing>,
@@ -139,7 +139,7 @@ where
     })
 }
 
-#[cfg(not(feature = "in-tree-compute-drivers"))]
+#[cfg(not(all(not(target_os = "windows"), feature = "in-tree-compute-drivers")))]
 fn in_process_driver_layer<S>(
     _provider: &Option<SdkTracerProvider>,
     _driver: Option<InProcessDriverTracing>,