fix: preserve child argv flags after separator

refs #383

1 files changed, 49 insertions, 0 deletions

diff --git a/src/session.rs b/src/session.rs
index a4daf5e4..67b77dc0 100644
--- a/src/session.rs
+++ b/src/session.rs
@@ -52,6 +52,10 @@ pub fn configure_from_args(args: &[String]) -> Result<Vec<String>, String> {
     let mut index = 1;
     while index < args.len() {
         let arg = &args[index];
+        if arg == "--" {
+            cleaned.extend_from_slice(&args[index..]);
+            break;
+        }
         if arg == "--session" {
             let Some(value) = args.get(index + 1) else {
                 return Err("missing value for --session".to_string());
@@ -569,6 +573,51 @@ mod tests {
     }
 
     #[test]
+    fn configure_from_args_preserves_child_session_option_after_separator() {
+        let _guard = env_lock().lock().unwrap();
+        std::env::remove_var(SESSION_ENV_VAR);
+        clear_explicit_session_for_test();
+        let args = vec![
+            "herdr".to_string(),
+            "agent".to_string(),
+            "start".to_string(),
+            "repro".to_string(),
+            "--".to_string(),
+            "/bin/echo".to_string(),
+            "--session".to_string(),
+            "child-session".to_string(),
+        ];
+
+        let cleaned = configure_from_args(&args).unwrap();
+
+        assert_eq!(cleaned, args);
+        assert!(std::env::var(SESSION_ENV_VAR).is_err());
+        assert!(!explicit_session_requested());
+    }
+
+    #[test]
+    fn configure_from_args_preserves_child_session_equals_option_after_separator() {
+        let _guard = env_lock().lock().unwrap();
+        std::env::remove_var(SESSION_ENV_VAR);
+        clear_explicit_session_for_test();
+        let args = vec![
+            "herdr".to_string(),
+            "agent".to_string(),
+            "start".to_string(),
+            "repro".to_string(),
+            "--".to_string(),
+            "/bin/echo".to_string(),
+            "--session=child-session".to_string(),
+        ];
+
+        let cleaned = configure_from_args(&args).unwrap();
+
+        assert_eq!(cleaned, args);
+        assert!(std::env::var(SESSION_ENV_VAR).is_err());
+        assert!(!explicit_session_requested());
+    }
+
+    #[test]
     fn configure_from_args_rewrites_session_attach_to_default_launch() {
         let _guard = env_lock().lock().unwrap();
         std::env::set_var(SESSION_ENV_VAR, "bad/name");