docs: describe current behavior, not change history, in comments

Author: zfarrell

Cargo.toml

@@ -13,11 +13,10 @@ path = "src/main.rs"
 
 [dependencies]
 # Hotdata Rust SDK. The CLI's sync wrapper (src/sdk.rs) drives this async SDK
-# behind a shared multi-thread tokio runtime. Pinned to the rev that adds
-# `content_length` to the streaming `upload_stream` (sized body, not chunked, so
-# the server can fast-fail an oversized upload) — merged via hotdata-dev/sdk-rust#35
-# (which subsumes the earlier #32 CLI-consumption surface: attribution
-# client_id, async submit_query, streaming upload_stream).
+# behind a shared multi-thread tokio runtime. The `arrow` feature backs result
+# decode; the pinned rev is the first to carry `content_length` on the streaming
+# `upload_stream` (sized body, not chunked, so the server can fast-fail an
+# oversized upload — see src/sdk.rs::Api::upload_stream).
 hotdata = { git = "https://github.com/hotdata-dev/sdk-rust", rev = "1687ba2ffb8f816540d00d7c1fdfe165a8fbd150", features = ["arrow"] }
 # Shared multi-thread runtime for the sync wrapper; block_on is called
 # concurrently from rayon worker threads (see src/indexes.rs). `sync` backs the

src/raw_http.rs

@@ -10,18 +10,13 @@
 //!   `sandbox_session.rs`);
 //! * `skill.rs`'s arbitrary-URL markdown fetch.
 //!
-//! (The streaming `/files` upload moved onto the SDK seam's
-//! [`Api::upload_stream`](crate::sdk::Api::upload_stream), which owns its own
-//! no-timeout client.)
-//!
 //! This module owns the timeout-bounded blocking client builder and a thin
-//! bearer/header request builder. It does NOT carry the old `ApiClient`'s
-//! 401-retry loop: token freshness is now the `CliTokenProvider`'s job
-//! (proactive refresh at the 30s leeway).
+//! bearer/header request builder. Token freshness is the `CliTokenProvider`'s
+//! job (proactive refresh at the 30s leeway), so these helpers carry no
+//! 401-retry loop.
 
-// Consumers (jwt.rs token mints, session mints, the streaming upload,
-// skill.rs) are migrated to this helper incrementally; the allow keeps the
-// build warning-free until those call sites land.
+// Not every helper here is wired to a call site yet; the allow keeps the build
+// warning-free.
 #![allow(dead_code)]
 
 use std::time::Duration;