fix: resolve all compiler warnings in test-runner and wormkit

- Remove unused DETACHED_PROCESS constant and unused `output` variable
- Remove unused DDGame import in fire_surrender
- Rename LPSECURITY_ATTRIBUTES to LpSecurityAttributes (camel case)
- Remove unused BOOL type alias in file_isolation
- Prefix unused `esp` with underscore in debug_watchpoint

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

crates/openwa-test-runner/src/main.rs

@@ -15,7 +15,6 @@ use std::thread;
 use std::time::{Duration, Instant};
 
 const CREATE_NO_WINDOW: u32 = 0x0800_0000;
-const DETACHED_PROCESS: u32 = 0x0000_0008;
 
 // ─── Configuration ──────────────────────────────────────────────────────────
 
@@ -133,8 +132,6 @@ fn discover_tests(filter: Option<&str>) -> Vec<TestCase> {
             continue; // Skip replays without expected logs
         }
 
-        let output = path.with_extension("log");
-
         // WA.exe runs in the game directory, so replay/log paths must be absolute.
         // Strip \\?\ UNC prefix that canonicalize adds on Windows — WA.exe can't handle it.
         let replay_abs = strip_unc(fs::canonicalize(&path).unwrap_or(path.clone()));

crates/openwa-wormkit/src/debug_watchpoint.rs

@@ -128,7 +128,7 @@ unsafe extern "system" fn veh_handler(info: *mut EXCEPTION_POINTERS) -> i32 {
                     // Walk EBP chain for stack trace
                     let mut trace = heapless::String::<512>::new();
                     let mut ebp = ctx.Ebp;
-                    let esp = ctx.Esp;
+                    let _esp = ctx.Esp;
                     for depth in 0..12 {
                         // Validate EBP: must be in plausible stack range, aligned,
                         // and above ESP (stack grows down)

crates/openwa-wormkit/src/replacements/file_isolation.rs

@@ -17,14 +17,13 @@ use std::ffi::CStr;
 
 type HANDLE = *mut c_void;
 type DWORD = u32;
-type BOOL = i32;
-type LPSECURITY_ATTRIBUTES = *mut c_void;
+type LpSecurityAttributes = *mut c_void;
 
 type CreateFileAFn = unsafe extern "system" fn(
     lpFileName: *const u8,
     dwDesiredAccess: DWORD,
     dwShareMode: DWORD,
-    lpSecurityAttributes: LPSECURITY_ATTRIBUTES,
+    lpSecurityAttributes: LpSecurityAttributes,
     dwCreationDisposition: DWORD,
     dwFlagsAndAttributes: DWORD,
     hTemplateFile: HANDLE,
@@ -56,7 +55,7 @@ unsafe extern "system" fn hook_create_file_a(
     lp_file_name: *const u8,
     desired_access: DWORD,
     share_mode: DWORD,
-    security_attributes: LPSECURITY_ATTRIBUTES,
+    security_attributes: LpSecurityAttributes,
     creation_disposition: DWORD,
     flags_and_attributes: DWORD,
     template_file: HANDLE,

crates/openwa-wormkit/src/replacements/weapon.rs

@@ -485,7 +485,6 @@ unsafe fn fire_skip_go(worm: *const CTaskWorm, entry: *const WeaponEntry) {
 /// WormEntry.turn_action_counter_Maybe by 14 (0x0E).
 unsafe fn fire_surrender(worm: *mut CTaskWorm) {
     use openwa_core::engine::ddgame::TeamArenaRef;
-    use openwa_core::engine::DDGame;
 
     fire_send_team_message(worm, 0x29);