refactor(sdk): drop stale dead_code allows

Author: zfarrell

src/jwt.rs

@@ -378,9 +378,6 @@ pub fn ensure_access_token(
 /// blocking CLI function on every request so session.json, the 30s leeway
 /// table, no-clobber for Flag/Env, and clear-on-dead-refresh stay owned by
 /// the CLI — the SDK never re-implements JWT exchange.
-// Wired into command modules via `sdk::Api::new` incrementally; unused until
-// the first module migrates off the legacy ApiClient.
-#[allow(dead_code)]
 #[derive(Debug, Clone)]
 pub enum AuthMode {
     /// `HOTDATA_DATABASE_TOKEN` env var (a `databases run` child).
@@ -407,13 +404,11 @@ pub enum AuthMode {
 /// own `TokenManager` is bypassed for the user-JWT path and the CLI keeps
 /// full ownership of auth. The blocking functions run inside
 /// `spawn_blocking` so they don't stall the wrapper's async runtime.
-#[allow(dead_code)]
 #[derive(Debug, Clone)]
 pub struct CliTokenProvider {
     mode: AuthMode,
 }
 
-#[allow(dead_code)]
 impl CliTokenProvider {
     pub fn new(mode: AuthMode) -> Self {
         Self { mode }

src/sdk.rs

@@ -24,12 +24,6 @@
 //! the SDK passes through unchanged; the CLI keeps full ownership of
 //! session.json and the refresh table.
 
-// The wrapper is wired into command modules incrementally (one commit per
-// module). Until every call site is migrated, parts of this seam are unused;
-// the allow keeps the build warning-free through the transition and is removed
-// once api.rs is retired.
-#![allow(dead_code)]
-
 use std::sync::Arc;
 use std::sync::OnceLock;
 
@@ -470,11 +464,6 @@ impl Api {
         }
     }
 
-    /// Override the database id for a single query without touching config.
-    pub fn with_database(mut self, database_id: &str) -> Self {
-        self.database_id = Some(database_id.to_string());
-        self
-    }
 
     pub fn workspace_id(&self) -> Option<&str> {
         self.workspace_id.as_deref()
@@ -1139,7 +1128,7 @@ mod tests {
             .with_body(r#"{"ok":true}"#)
             .create();
 
-        let api = Api::test_new(&server.url(), "test-jwt", Some("ws-1")).with_database("db-1");
+        let api = Api::test_new_scoped(&server.url(), "test-jwt", Some("ws-1"), None, Some("db-1"));
         let (status, _body) = api
             .post_raw("/query", &serde_json::json!({"sql": "select 1"}))
             .expect("post_raw should succeed");