feat: improve telemetry (#79)

* feat(cli): print first-run telemetry notice

* refactor(cli): switch telemetry mutex to parking_lot

* refactor(cli): batch telemetry events via background flusher

Author: junhsss

Cargo.lock

@@ -45,6 +45,7 @@ getrandom = "0.4"
 libc = "0.2"
 tempfile = "3"
 jiff = "0.2"
+parking_lot = "0.12"
 
 [dev-dependencies]
 wiremock = "0.6"

Cargo.toml

@@ -77,6 +77,8 @@ For generated flags and argument schemas, use [../cli-reference.md](../cli-refer
 - Browser session state: `~/.config/steel/browser-session-state.json`
 - Profile metadata: `~/.config/steel/profiles/<name>.json`
 
+On the first run where telemetry is enabled, the CLI prints a one-time notice to stderr describing what is collected and how to opt out. The notice is suppressed in JSON/non-TTY output.
+
 Telemetry can be disabled persistently in `config.json` with:
 
 ```json

docs/references/steel-cli.md

@@ -402,13 +402,14 @@ mod tests {
     }
 
     async fn capture_events(server: &MockServer) -> Vec<serde_json::Value> {
-        server
-            .received_requests()
-            .await
-            .unwrap_or_default()
-            .into_iter()
-            .map(|request| serde_json::from_slice(&request.body).unwrap())
-            .collect()
+        let mut out = Vec::new();
+        for request in server.received_requests().await.unwrap_or_default() {
+            let payload: serde_json::Value = serde_json::from_slice(&request.body).unwrap();
+            if let Some(batch) = payload.get("batch").and_then(|v| v.as_array()) {
+                out.extend(batch.iter().cloned());
+            }
+        }
+        out
     }
 
     #[tokio::test]
@@ -421,7 +422,7 @@ mod tests {
         crate::telemetry::set_test_override(temp.path(), &server.uri());
 
         Mock::given(method("POST"))
-            .and(path("/i/v0/e/"))
+            .and(path("/batch/"))
             .respond_with(ResponseTemplate::new(200))
             .mount(&server)
             .await;
@@ -467,7 +468,7 @@ mod tests {
         crate::telemetry::set_test_override(temp.path(), &server.uri());
 
         Mock::given(method("POST"))
-            .and(path("/i/v0/e/"))
+            .and(path("/batch/"))
             .respond_with(ResponseTemplate::new(200))
             .mount(&server)
             .await;
@@ -539,7 +540,7 @@ mod tests {
         crate::telemetry::set_test_override(temp.path(), &server.uri());
 
         Mock::given(method("POST"))
-            .and(path("/i/v0/e/"))
+            .and(path("/batch/"))
             .respond_with(ResponseTemplate::new(200))
             .mount(&server)
             .await;