added --comments arg

Author: kavehtehrani

src/cli.rs

@@ -88,6 +88,10 @@ pub struct Cli {
     /// Automatically start a test when the app launches
     #[arg(long, default_value_t = true, action = clap::ArgAction::Set)]
     pub test_on_launch: bool,
+
+    /// Attach custom comments to this run (saved/exported and shown in TUI status)
+    #[arg(long)]
+    pub comments: Option<String>,
 }
 
 pub async fn run(args: Cli) -> Result<()> {
@@ -122,6 +126,7 @@ pub fn build_config(args: &Cli) -> RunConfig {
     RunConfig {
         base_url: args.base_url.clone(),
         meas_id: gen_meas_id(),
+        comments: args.comments.clone(),
         download_bytes_per_req: args.download_bytes_per_req,
         upload_bytes_per_req: args.upload_bytes_per_req,
         concurrency: args.concurrency,
@@ -271,6 +276,11 @@ async fn run_text(args: Cli) -> Result<()> {
     if let Some(server) = enriched.server.as_deref() {
         println!("Server: {server}");
     }
+    if let Some(comments) = enriched.comments.as_deref() {
+        if !comments.trim().is_empty() {
+            println!("Comments: {}", comments);
+        }
+    }
 
     // Compute and display throughput metrics (mean, median, p25, p75)
     let dl_values: Vec<f64> = dl_points.iter().map(|(_, y)| *y).collect();

src/engine/mod.rs

@@ -166,6 +166,7 @@ impl TestEngine {
                 .unwrap_or_else(|_| "now".into()),
             base_url: self.cfg.base_url.clone(),
             meas_id: self.cfg.meas_id.clone(),
+            comments: self.cfg.comments.clone(),
             meta,
             server,
             idle_latency,

src/model.rs

@@ -5,6 +5,8 @@ use std::time::Duration;
 pub struct RunConfig {
     pub base_url: String,
     pub meas_id: String,
+    #[serde(default)]
+    pub comments: Option<String>,
     pub download_bytes_per_req: u64,
     pub upload_bytes_per_req: u64,
     pub concurrency: usize,
@@ -99,6 +101,8 @@ pub struct RunResult {
     pub timestamp_utc: String,
     pub base_url: String,
     pub meas_id: String,
+    #[serde(default)]
+    pub comments: Option<String>,
     pub meta: Option<serde_json::Value>,
     #[serde(default)]
     pub server: Option<String>,

src/storage.rs

@@ -58,12 +58,13 @@ pub fn export_csv(path: &Path, result: &RunResult) -> Result<()> {
         std::fs::create_dir_all(parent).context("create export directory")?;
     }
     let mut out = String::new();
-    out.push_str("timestamp_utc,base_url,meas_id,server,download_mbps,upload_mbps,idle_mean_ms,idle_median_ms,idle_p25_ms,idle_p75_ms,idle_loss,dl_loaded_mean_ms,dl_loaded_median_ms,dl_loaded_p25_ms,dl_loaded_p75_ms,dl_loaded_loss,ul_loaded_mean_ms,ul_loaded_median_ms,ul_loaded_p25_ms,ul_loaded_p75_ms,ul_loaded_loss,ip,colo,asn,as_org,interface_name,network_name,is_wireless,interface_mac,link_speed_mbps\n");
+    out.push_str("timestamp_utc,base_url,meas_id,comments,server,download_mbps,upload_mbps,idle_mean_ms,idle_median_ms,idle_p25_ms,idle_p75_ms,idle_loss,dl_loaded_mean_ms,dl_loaded_median_ms,dl_loaded_p25_ms,dl_loaded_p75_ms,dl_loaded_loss,ul_loaded_mean_ms,ul_loaded_median_ms,ul_loaded_p25_ms,ul_loaded_p75_ms,ul_loaded_loss,ip,colo,asn,as_org,interface_name,network_name,is_wireless,interface_mac,link_speed_mbps\n");
     out.push_str(&format!(
-        "{},{},{},{},{:.3},{:.3},{:.3},{:.3},{:.3},{:.3},{:.6},{:.3},{:.3},{:.3},{:.3},{:.6},{:.3},{:.3},{:.3},{:.3},{:.6},{},{},{},{},{},{},{},{},{}\n",
+        "{},{},{},{},{},{:.3},{:.3},{:.3},{:.3},{:.3},{:.3},{:.6},{:.3},{:.3},{:.3},{:.3},{:.6},{:.3},{:.3},{:.3},{:.3},{:.6},{},{},{},{},{},{},{},{},{}\n",
         csv_escape(&result.timestamp_utc),
         csv_escape(&result.base_url),
         csv_escape(&result.meas_id),
+        csv_escape(result.comments.as_deref().unwrap_or("")),
         csv_escape(result.server.as_deref().unwrap_or("")),
         result.download.mbps,
         result.upload.mbps,

src/tui/mod.rs

@@ -28,6 +28,7 @@ struct UiState {
     paused: bool,
     phase: Phase,
     info: String,
+    comments: Option<String>,
 
     dl_series: Vec<u64>,
     ul_series: Vec<u64>,
@@ -92,6 +93,7 @@ impl Default for UiState {
             paused: false,
             phase: Phase::IdleLatency,
             info: String::new(),
+            comments: None,
             dl_series: Vec::new(),
             ul_series: Vec::new(),
             idle_lat_series: Vec::new(),
@@ -143,6 +145,52 @@ impl Default for UiState {
     }
 }
 
+fn push_wrapped_status_kv(
+    out: &mut Vec<Line<'static>>,
+    label: &str,
+    value: &str,
+    status_area_width: u16,
+) {
+    let value = value.trim();
+    if value.is_empty() {
+        return;
+    }
+
+    // Account for borders (2 chars on each side)
+    let usable_width = status_area_width.saturating_sub(4).max(1);
+    let label_text = format!("{label}:");
+    let label_width = label_text.chars().count() as u16;
+
+    let value_chars: Vec<char> = value.chars().collect();
+    let mut remaining = value_chars.as_slice();
+    let mut first = true;
+
+    while !remaining.is_empty() {
+        let line_width = if first {
+            usable_width.saturating_sub(label_width + 1).max(1)
+        } else {
+            usable_width.saturating_sub(2).max(1)
+        };
+
+        let chars_to_take = (remaining.len() as u16).min(line_width) as usize;
+        let (line_chars, rest) = remaining.split_at(chars_to_take);
+        let line_text: String = line_chars.iter().collect();
+
+        if first {
+            out.push(Line::from(vec![
+                Span::styled(label_text.clone(), Style::default().fg(Color::Gray)),
+                Span::raw(" "),
+                Span::raw(line_text),
+            ]));
+            first = false;
+        } else {
+            out.push(Line::from(vec![Span::raw("  "), Span::raw(line_text)]));
+        }
+
+        remaining = rest;
+    }
+}
+
 impl UiState {
     fn push_series(series: &mut Vec<u64>, v: u64) {
         const MAX: usize = 120;
@@ -250,6 +298,7 @@ pub async fn run(args: Cli) -> Result<()> {
     let mut state = UiState {
         phase: Phase::IdleLatency,
         auto_save: args.auto_save,
+        comments: args.comments.clone(),
         ..Default::default()
     };
     state.initial_history_load_size = initial_load;
@@ -1280,12 +1329,17 @@ fn draw_dashboard(area: Rect, f: &mut ratatui::Frame, state: &UiState) {
         ),
     ])];
 
+    // Custom comments (wrapping to fit status area)
+    if let Some(comments) = state.comments.as_deref() {
+        push_wrapped_status_kv(&mut status_lines, "Comments", comments, main[3].width);
+    }
+
     // Info line - split into two lines if it contains a saved path, with wrapping
     if state.info.starts_with("Saved:") || state.info.starts_with("Saved (verifying):") {
         // Split into label and path
         if let Some(colon_pos) = state.info.find(':') {
             let (label, path) = state.info.split_at(colon_pos + 1);
-            let label_text = label.trim();
+            let label_text = label.trim().to_string();
             let path_str = path.trim();
 
             // Wrap the path to fit within available width
@@ -1313,7 +1367,7 @@ fn draw_dashboard(area: Rect, f: &mut ratatui::Frame, state: &UiState) {
                 if is_first_path_line {
                     // First line - include label and first part of path
                     status_lines.push(Line::from(vec![
-                        Span::styled(label_text, Style::default().fg(Color::Gray)),
+                        Span::styled(label_text.clone(), Style::default().fg(Color::Gray)),
                         Span::raw(" "),
                         Span::raw(line_text),
                     ]));
@@ -1328,13 +1382,13 @@ fn draw_dashboard(area: Rect, f: &mut ratatui::Frame, state: &UiState) {
         } else {
             status_lines.push(Line::from(vec![
                 Span::styled("Info: ", Style::default().fg(Color::Gray)),
-                Span::raw(&state.info),
+                Span::raw(state.info.clone()),
             ]));
         }
     } else {
         status_lines.push(Line::from(vec![
             Span::styled("Info: ", Style::default().fg(Color::Gray)),
-            Span::raw(&state.info),
+            Span::raw(state.info.clone()),
         ]));
     }