refactor(fspy): own Windows shared-memory mapping

Co-authored-by: GPT-5.6 <gpt-5.6@openai.com>

Author: wan9chi

Cargo.lock

@@ -167,6 +167,7 @@ wax = "0.7.0"
 which = "8.0.0"
 widestring = "1.2.0"
 winapi = "0.3.9"
+windows-sys = "0.61"
 winsafe = { version = "0.0.27", features = ["kernel"] }
 xxhash-rust = { version = "0.8.15", features = ["const_xxh3"] }
 ntest = "0.9.5"

Cargo.toml

@@ -99,8 +99,13 @@ pub const fn access_mask_to_mode(desired_access: ACCESS_MASK) -> AccessMode {
     }
 }
 
+#[link(name = "kernel32")]
 unsafe extern "system" {
     fn LocalFree(hmem: HLOCAL) -> HLOCAL;
+}
+
+#[link(name = "pathcch")]
+unsafe extern "system" {
     fn PathAllocCombine(
         pszpathin: PCWSTR,
         pszmore: PCWSTR,

crates/fspy_preload_windows/src/windows/winapi_utils.rs

@@ -7,7 +7,7 @@ license.workspace = true
 publish = false
 rust-version.workspace = true
 
-[target.'cfg(not(target_os = "linux"))'.dependencies]
+[target.'cfg(not(any(target_os = "linux", target_os = "windows")))'.dependencies]
 shared_memory = { workspace = true, features = ["logging"] }
 
 [target.'cfg(target_os = "linux")'.dependencies]
@@ -18,6 +18,15 @@ tempfile = { workspace = true }
 tokio = { workspace = true, features = ["macros", "net", "rt", "sync", "time"] }
 uuid = { workspace = true, features = ["v4"] }
 
+[target.'cfg(target_os = "windows")'.dependencies]
+uuid = { workspace = true, features = ["v4"] }
+windows-sys = { workspace = true, features = [
+  "Win32_Foundation",
+  "Win32_Security",
+  "Win32_Storage_FileSystem",
+  "Win32_System_Memory",
+] }
+
 [dev-dependencies]
 ctor = { workspace = true }
 subprocess_test = { workspace = true }

crates/fspy_shm/Cargo.toml

@@ -1,24 +1,28 @@
 //! Platform shared-memory implementation for fspy channels.
 
-#[cfg(not(target_os = "linux"))]
+#[cfg(not(any(target_os = "linux", target_os = "windows")))]
 use std::io;
 
 #[cfg(target_os = "linux")]
 mod linux;
+#[cfg(target_os = "windows")]
+mod windows;
 
 #[cfg(target_os = "linux")]
 pub use linux::{CreatedShm, Shm, ShmBroker, ShmBrokerHandle, create, open};
-#[cfg(not(target_os = "linux"))]
+#[cfg(not(any(target_os = "linux", target_os = "windows")))]
 use shared_memory::{Shmem, ShmemConf};
+#[cfg(target_os = "windows")]
+pub use windows::{CreatedShm, Shm, create, open};
 
 /// An owned shared-memory mapping.
-#[cfg(not(target_os = "linux"))]
+#[cfg(not(any(target_os = "linux", target_os = "windows")))]
 pub struct Shm {
     inner: Shmem,
 }
 
 /// A newly created shared-memory mapping and its platform service.
-#[cfg(not(target_os = "linux"))]
+#[cfg(not(any(target_os = "linux", target_os = "windows")))]
 pub struct CreatedShm {
     /// The owned shared-memory mapping.
     pub shm: Shm,
@@ -29,11 +33,9 @@ pub struct CreatedShm {
 /// # Errors
 ///
 /// Returns an error if the platform cannot create or map the region.
-#[cfg(not(target_os = "linux"))]
+#[cfg(not(any(target_os = "linux", target_os = "windows")))]
 pub fn create(size: usize) -> io::Result<CreatedShm> {
     let conf = ShmemConf::new().size(size);
-    #[cfg(target_os = "windows")]
-    let conf = conf.allow_raw(true);
 
     let inner = conf.create().map_err(io::Error::other)?;
     Ok(CreatedShm { shm: Shm { inner } })
@@ -44,17 +46,15 @@ pub fn create(size: usize) -> io::Result<CreatedShm> {
 /// # Errors
 ///
 /// Returns an error if the mapping does not exist or cannot be mapped.
-#[cfg(not(target_os = "linux"))]
+#[cfg(not(any(target_os = "linux", target_os = "windows")))]
 pub fn open(id: &str, size: usize) -> io::Result<Shm> {
     let conf = ShmemConf::new().size(size).os_id(id);
-    #[cfg(target_os = "windows")]
-    let conf = conf.allow_raw(true);
 
     let inner = conf.open().map_err(io::Error::other)?;
     Ok(Shm { inner })
 }
 
-#[cfg(not(target_os = "linux"))]
+#[cfg(not(any(target_os = "linux", target_os = "windows")))]
 #[expect(clippy::len_without_is_empty, reason = "shared-memory mappings are always non-empty")]
 impl Shm {
     /// Returns this mapping's opaque platform identifier.