Merge pull request #179 from Maxteabag/worktree-fix-d1-query-endpoint

Fix D1 adapter hitting wrong endpoint (/execute -> /query)

Author: Maxteabag

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

@@ -125,8 +125,8 @@ def get_databases(self, conn: D1Connection) -> list[str]:
         return [db["name"] for db in databases if "name" in db]
 
     def _execute(self, conn: D1Connection, query: str) -> dict[str, Any]:
-        """Internal method to run a command on the D1 execute endpoint."""
-        api_url = f"{self._api_base_url()}/client/v4/accounts/{conn.account_id}/d1/database/{conn.database_id}/execute"
+        """Internal method to run a command on the D1 query endpoint."""
+        api_url = f"{self._api_base_url()}/client/v4/accounts/{conn.account_id}/d1/database/{conn.database_id}/query"
         response = conn.session.post(api_url, json={"sql": query})
         response.raise_for_status()
         # The result is a list containing a single result object

tests/fixtures/d1.py

@@ -63,7 +63,7 @@ def create_database(self, name: str) -> None:
     def execute(self, db_name: str, sql: str) -> dict | None:
         payload = self._request(
             "POST",
-            f"/client/v4/accounts/{self._account_id}/d1/database/{db_name}/execute",
+            f"/client/v4/accounts/{self._account_id}/d1/database/{db_name}/query",
             {"sql": sql},
         )
         if not payload.get("success", False):

tests/fixtures/d1/index.js

@@ -14,7 +14,7 @@ export default {
     }
 
     const execMatch = path.match(
-      /^\/client\/v4\/accounts\/([^/]+)\/d1\/database\/([^/]+)\/execute$/
+      /^\/client\/v4\/accounts\/([^/]+)\/d1\/database\/([^/]+)\/query$/
     );
     if (execMatch && request.method === "POST") {
       let body;