ix: add no_input to BaseArgs test fixtures and simplify eval SSE parsing

Author: viadezo1er

src/args.rs

@@ -28,6 +28,10 @@ pub struct BaseArgs {
     #[arg(long, env = "BRAINTRUST_NO_COLOR", global = true, value_parser = clap::builder::BoolishValueParser::new(), default_value_t = false)]
     pub no_color: bool,
 
+    /// Disable all interactive prompts
+    #[arg(long, env = "BRAINTRUST_NO_INPUT", global = true, value_parser = clap::builder::BoolishValueParser::new(), default_value_t = false)]
+    pub no_input: bool,
+
     /// Use a saved login profile (or via BRAINTRUST_PROFILE)
     #[arg(long, env = "BRAINTRUST_PROFILE", global = true)]
     pub profile: Option<String>,

src/auth.rs

@@ -2708,6 +2708,7 @@ mod tests {
             verbose: false,
             quiet: false,
             no_color: false,
+            no_input: false,
             profile: None,
             project: None,
             org_name: None,

src/functions/push.rs

@@ -3436,6 +3436,7 @@ mod tests {
             verbose: false,
             quiet: false,
             no_color: false,
+            no_input: false,
             profile: None,
             org_name: None,
             project: None,

src/main.rs

@@ -404,6 +404,7 @@ mod tests {
             verbose: false,
             quiet: false,
             no_color: false,
+            no_input: false,
             profile: None,
             org_name: None,
             project: None,

src/switch.rs

@@ -256,6 +256,7 @@ mod tests {
             verbose: false,
             quiet: false,
             no_color: false,
+            no_input: false,
             profile: None,
             org_name: org.map(String::from),
             project: project.map(String::from),

src/traces.rs

@@ -6061,6 +6061,7 @@ mod tests {
             verbose: false,
             quiet: false,
             no_color: false,
+            no_input: false,
             profile: None,
             org_name: None,
             project: None,

tests/eval_dev_server.rs

@@ -167,10 +167,10 @@ fn parse_sse_events(body: &str) -> Vec<SseEvent> {
     let mut current_data = Vec::<String>::new();
 
     for line in body.lines() {
-        if line.starts_with("event: ") {
-            current_event = line["event: ".len()..].to_string();
-        } else if line.starts_with("data: ") {
-            current_data.push(line["data: ".len()..].to_string());
+        if let Some(stripped) = line.strip_prefix("event: ") {
+            current_event = stripped.to_string();
+        } else if let Some(stripped) = line.strip_prefix("data: ") {
+            current_data.push(stripped.to_string());
         } else if line.is_empty() && !current_event.is_empty() {
             events.push(SseEvent {
                 event: std::mem::take(&mut current_event),
@@ -562,10 +562,10 @@ fn streaming_eval_post(
             Ok(l) => l,
             Err(_) => break,
         };
-        if line.starts_with("event: ") {
-            current_event = line["event: ".len()..].to_string();
-        } else if line.starts_with("data: ") {
-            current_data.push(line["data: ".len()..].to_string());
+        if let Some(stripped) = line.strip_prefix("event: ") {
+            current_event = stripped.to_string();
+        } else if let Some(stripped) = line.strip_prefix("data: ") {
+            current_data.push(stripped.to_string());
         } else if line.is_empty() && !current_event.is_empty() {
             let event = SseEvent {
                 event: std::mem::take(&mut current_event),