fix!: re-exec config preservation, protocol state names, test coverage

.gitignore

@@ -2,3 +2,5 @@
 /.venv
 __pycache__/
 .pytest_cache/
+*.profraw
+lcov.info

Cargo.toml

@@ -26,6 +26,7 @@ serde = { version = "1", features = ["derive"] }
 [dev-dependencies]
 tempfile = "3"
 filetime = "0.2"
+libc = "0.2"
 
 [profile.release]
 strip = true

README.md

@@ -9,7 +9,7 @@ Fork. Watch. Control. From anywhere.
 *Named for the Norse guardian who watches over Bifrost — Heimdall sees all, hears all, and nothing escapes on his watch.*
 
 [![CI](https://github.com/nazq/heimdall/actions/workflows/ci.yml/badge.svg)](https://github.com/nazq/heimdall/actions/workflows/ci.yml)
-[![codecov](https://codecov.io/gh/nazq/heimdall/graph/badge.svg)](https://codecov.io/gh/nazq/heimdall)
+[![codecov](https://codecov.io/gh/nazq/heimdall/graph/badge.svg?token=opgTAX2W8k)](https://codecov.io/gh/nazq/heimdall)
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
 [![Rust](https://img.shields.io/badge/rust-2024_edition-orange.svg)](https://www.rust-lang.org/)
 

src/attach.rs

@@ -35,7 +35,7 @@ const STATUS_POLL_INTERVAL: std::time::Duration =
 /// (Ctrl-\ or session exit), the supervisor keeps running in the background.
 pub fn launch_and_attach(params: SessionParams) -> anyhow::Result<()> {
     let exe = std::env::current_exe()?;
-    let child_args = params.to_detach_args();
+    let child_args = params.to_detach_args()?;
 
     // Spawn supervisor in background with setsid() for terminal independence.
     let _supervisor = unsafe {

src/classify/claude.rs

@@ -190,7 +190,10 @@ impl StateClassifier for ClaudeClassifier {
         self.state_entered_at = now_ms;
         self.pending_state = None;
     }
+}
 
+#[cfg(test)]
+impl ClaudeClassifier {
     fn state_name(&self, state: u8) -> &'static str {
         match state {
             0x00 => "idle",

src/classify/mod.rs

@@ -54,9 +54,6 @@ pub trait StateClassifier: Send {
 
     /// Force state to Dead (on child exit).
     fn set_dead(&mut self, now_ms: u64);
-
-    /// Human-readable name for a state byte.
-    fn state_name(&self, state: u8) -> &'static str;
 }
 
 /// Create a classifier from config.
@@ -96,7 +93,6 @@ mod tests {
             debounce_ms: 200,
         });
         assert_eq!(c.state(), ProcessState::Idle);
-        assert_eq!(c.state_name(0x01), "thinking"); // only claude knows "thinking"
     }
 
     #[test]
@@ -105,15 +101,12 @@ mod tests {
             idle_threshold_ms: 3000,
         });
         assert_eq!(c.state(), ProcessState::Idle);
-        assert_eq!(c.state_name(0x04), "active"); // simple knows "active"
-        assert_eq!(c.state_name(0x01), "unknown"); // simple doesn't know "thinking"
     }
 
     #[test]
     fn from_config_none() {
         let c = from_config(&ClassifierConfig::None);
         assert_eq!(c.state(), ProcessState::Idle);
-        assert_eq!(c.state_name(0x01), "idle"); // none reports everything as idle
     }
 
     /// Issue #6: classifier orthogonality — simple uses Active, claude never does.

src/classify/none.rs

@@ -20,7 +20,10 @@ impl StateClassifier for NoneClassifier {
     }
 
     fn set_dead(&mut self, _now_ms: u64) {}
+}
 
+#[cfg(test)]
+impl NoneClassifier {
     fn state_name(&self, state: u8) -> &'static str {
         match state {
             0xFF => "dead",

src/classify/simple.rs

@@ -61,7 +61,10 @@ impl StateClassifier for SimpleClassifier {
         self.current_state = ProcessState::Dead;
         self.state_entered_at = now_ms;
     }
+}
 
+#[cfg(test)]
+impl SimpleClassifier {
     fn state_name(&self, state: u8) -> &'static str {
         match state {
             0x00 => "idle",