Fix tables list: honor --table filter and fall through on --limit/--cursor

When an active database is set, --table now passes through to the API
instead of being silently dropped. --limit and --cursor fall through to
tables::list (which supports pagination) rather than the active-db path.

Author: eddietejeda

src/commands/databases.rs

@@ -766,14 +766,19 @@ fn download_to_temp(
     Ok(temp)
 }
 
-fn collect_tables(api: &Api, connection_id: &str, schema: Option<&str>) -> Vec<InfoTable> {
+fn collect_tables(
+    api: &Api,
+    connection_id: &str,
+    schema: Option<&str>,
+    table: Option<&str>,
+) -> Vec<InfoTable> {
     let mut out = Vec::new();
     let mut cursor: Option<String> = None;
     loop {
         let resp = crate::client::sdk::block(api.client().information_schema().get(
             Some(connection_id),
             schema,
-            None,
+            table,
             None,
             None,
             cursor.as_deref(),
@@ -1333,7 +1338,13 @@ pub fn delete(workspace_id: &str, id_or_name: &str) {
     println!("{}", "Database deleted.".green());
 }
 
-pub fn tables_list(workspace_id: &str, database: Option<&str>, schema: Option<&str>, format: &str) {
+pub fn tables_list(
+    workspace_id: &str,
+    database: Option<&str>,
+    schema: Option<&str>,
+    table: Option<&str>,
+    format: &str,
+) {
     let database = resolve_current_database(database, workspace_id);
     let api = Api::new(Some(workspace_id));
     let db = resolve_database(&api, &database);
@@ -1342,7 +1353,7 @@ pub fn tables_list(workspace_id: &str, database: Option<&str>, schema: Option<&s
         .as_deref()
         .or(db.name.as_deref())
         .unwrap_or("default");
-    let tables = collect_tables(&api, &db.default_connection_id, schema);
+    let tables = collect_tables(&api, &db.default_connection_id, schema, table);
 
     let rows = table_rows(catalog, tables);
 
@@ -1473,7 +1484,7 @@ pub fn tables_load(
         // The table wasn't declared at create time. Collect existing tables so
         // they are re-declared in the replacement database, then delete and
         // recreate with all tables (including the new one) declared.
-        let existing = collect_tables(&api, &db.default_connection_id, None);
+        let existing = collect_tables(&api, &db.default_connection_id, None, None);
         let mut all_tables: Vec<String> = existing
             .iter()
             .map(|t| format!("{}.{}", t.schema, t.table))
@@ -2098,7 +2109,7 @@ mod tests {
             .create();
 
         let api = Api::test_new(&server.url(), "k", Some("ws"));
-        let tables = collect_tables(&api, "conn1", None);
+        let tables = collect_tables(&api, "conn1", None, None);
         page0.assert();
         page1.assert();
         assert_eq!(tables.len(), 2);

src/main.rs

@@ -327,6 +327,7 @@ fn main() {
                                 &workspace_id,
                                 db_flag.as_deref().or(database.as_deref()),
                                 schema.as_deref(),
+                                None,
                                 &output,
                             ),
                             Some(DatabaseTablesCommands::Load {
@@ -363,6 +364,7 @@ fn main() {
                                         &workspace_id,
                                         Some(db.as_str()),
                                         None,
+                                        None,
                                         "table",
                                     )
                                 } else {
@@ -408,11 +410,16 @@ fn main() {
                     output,
                 } => {
                     let workspace_id = resolve_workspace(workspace_id);
-                    if crate::config::load_current_database("default", &workspace_id).is_some() {
+                    let use_db_path = limit.is_none()
+                        && cursor.is_none()
+                        && crate::config::load_current_database("default", &workspace_id)
+                            .is_some();
+                    if use_db_path {
                         databases::tables_list(
                             &workspace_id,
                             None,
                             schema.as_deref(),
+                            table.as_deref(),
                             &output,
                         )
                     } else {