fix: show Databases folder for MotherDuck

- Rename database field to default_database (stored in options)
- This keeps tcp_endpoint.database empty so tree shows Databases folder
- Add get_databases() to list all MotherDuck databases
- Add get_tables/get_views() with database filtering

Author: Maxteabag

sqlit/domains/connections/providers/motherduck/adapter.py

@@ -4,6 +4,7 @@
 
 from typing import TYPE_CHECKING, Any
 
+from sqlit.domains.connections.providers.adapters.base import TableInfo
 from sqlit.domains.connections.providers.duckdb.adapter import DuckDBAdapter
 
 if TYPE_CHECKING:
@@ -36,10 +37,8 @@ def connect(self, config: ConnectionConfig) -> Any:
             package_name=self.install_package,
         )
 
-        # Get database from options or tcp_endpoint
-        database = config.get_option("database", "")
-        if not database and config.tcp_endpoint:
-            database = config.tcp_endpoint.database
+        # Get default database from options
+        database = config.get_option("default_database", "")
 
         # Get token from tcp_endpoint.password (stored in keyring)
         token = ""
@@ -56,6 +55,51 @@ def connect(self, config: ConnectionConfig) -> Any:
         duckdb_any: Any = duckdb
         return duckdb_any.connect(conn_str)
 
+    def get_databases(self, conn: Any) -> list[str]:
+        """List all MotherDuck databases."""
+        result = conn.execute("SELECT database_name FROM duckdb_databases() WHERE NOT internal")
+        return [row[0] for row in result.fetchall()]
+
+    def get_tables(self, conn: Any, database: str | None = None) -> list[TableInfo]:
+        """Get tables from a specific MotherDuck database."""
+        if database:
+            result = conn.execute(
+                "SELECT table_schema, table_name FROM information_schema.tables "
+                "WHERE table_catalog = ? "
+                "AND table_type = 'BASE TABLE' "
+                "AND table_schema NOT IN ('pg_catalog', 'information_schema') "
+                "ORDER BY table_schema, table_name",
+                (database,),
+            )
+        else:
+            result = conn.execute(
+                "SELECT table_schema, table_name FROM information_schema.tables "
+                "WHERE table_type = 'BASE TABLE' "
+                "AND table_schema NOT IN ('pg_catalog', 'information_schema') "
+                "ORDER BY table_schema, table_name"
+            )
+        return [(row[0], row[1]) for row in result.fetchall()]
+
+    def get_views(self, conn: Any, database: str | None = None) -> list[TableInfo]:
+        """Get views from a specific MotherDuck database."""
+        if database:
+            result = conn.execute(
+                "SELECT table_schema, table_name FROM information_schema.tables "
+                "WHERE table_catalog = ? "
+                "AND table_type = 'VIEW' "
+                "AND table_schema NOT IN ('pg_catalog', 'information_schema') "
+                "ORDER BY table_schema, table_name",
+                (database,),
+            )
+        else:
+            result = conn.execute(
+                "SELECT table_schema, table_name FROM information_schema.tables "
+                "WHERE table_type = 'VIEW' "
+                "AND table_schema NOT IN ('pg_catalog', 'information_schema') "
+                "ORDER BY table_schema, table_name"
+            )
+        return [(row[0], row[1]) for row in result.fetchall()]
+
     def build_select_query(
         self, table: str, limit: int, database: str | None = None, schema: str | None = None
     ) -> str:

sqlit/domains/connections/providers/motherduck/schema.py

@@ -11,8 +11,8 @@
     display_name="MotherDuck",
     fields=(
         SchemaField(
-            name="database",
-            label="Database",
+            name="default_database",
+            label="Default Database",
             placeholder="my_database",
             required=True,
         ),
@@ -24,6 +24,6 @@
         ),
     ),
     supports_ssh=False,
-    is_file_based=False,  # Not file-based, uses database + token
+    is_file_based=False,
     requires_auth=True,
 )

tests/unit/test_motherduck_adapter.py

@@ -36,13 +36,17 @@ def test_motherduck_schema_uses_password_field():
     from sqlit.domains.connections.providers.motherduck.schema import SCHEMA
 
     field_names = [f.name for f in SCHEMA.fields]
-    assert "database" in field_names
+    assert "default_database" in field_names
     assert "password" in field_names  # Uses standard password field for token
 
     # Password field should be labeled as "Access Token"
     password_field = next(f for f in SCHEMA.fields if f.name == "password")
     assert password_field.label == "Access Token"
 
+    # Database field should be labeled as "Default Database"
+    db_field = next(f for f in SCHEMA.fields if f.name == "default_database")
+    assert db_field.label == "Default Database"
+
 
 def test_motherduck_supports_multiple_databases():
     """Test MotherDuck reports support for multiple databases."""