Take CodeRabbit's two nits: drop the unused kept set, handle spawn's error event

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PiGqmKAaxnRnDXwND8rq2y

Author: wormeyman

crates/fmw-noise/src/fixtures.rs

@@ -6298,13 +6298,6 @@ fn the_oriented_tile_test_through_the_apply_stage() {
     assert!(totals[1].wrong < totals[0].wrong);
     assert!(totals[1].surplus < totals[0].surplus);
     assert!(totals[1].missing < totals[0].missing);
-    // Every leftover is a cell the game's own collision test KEPT.
-    let kept: BTreeSet<(u64, u64)> = would_collide_calls(&calls)
-        .iter()
-        .flat_map(|(_, _, region)| region.iter())
-        .filter(|c| !c.result)
-        .map(|c| (c.x.to_bits(), c.y.to_bits()))
-        .collect();
     assert_eq!(leftovers.len(), 16);
     let tested: BTreeMap<(u64, u64), (u8, bool)> = would_collide_calls(&calls)
         .iter()
@@ -6339,7 +6332,6 @@ fn the_oriented_tile_test_through_the_apply_stage() {
             (1630.0, 1602.5, "north-to-west")
         ]
     );
-    let _ = kept;
 }
 
 /// **The game tests every cell with its RAW queued orientation, and the port's

scripts/probes/vulcanus-cliff-wouldcollide/capture.ts

@@ -203,6 +203,12 @@ async function captureRegion(region: Region): Promise<{ calls: Call[]; cliffs: n
         child.kill("SIGKILL");
         reject(new Error("lldb replay timed out after 900s"));
       }, 900_000);
+      // A missing lldb emits `error` and never `exit`; without this the
+      // promise would sit until the timer fired and blame a timeout.
+      child.on("error", (err) => {
+        clearTimeout(timer);
+        reject(new Error(`failed to start lldb: ${err.message}`));
+      });
       child.on("exit", (code) => {
         clearTimeout(timer);
         if (!out.includes("probe: wrote")) {