From 85f6c00e3d20cc82f873b3125aba67b9455350ba Mon Sep 17 00:00:00 2001
From: Steven Ontong <steven@journeyapps.com>
Date: Mon, 30 Oct 2023 09:08:08 +0200
Subject: [PATCH] cleanup

---
 .changeset/chilled-walls-promise.md                       | 3 +++
 apps/supabase-todolist                                    | 2 +-
 .../src/client/AbstractPowerSyncDatabase.ts               | 2 +-
 .../sync/stream/AbstractStreamingSyncImplementation.ts    | 3 ---
 packages/powersync-sdk-react-native/package.json          | 4 ++--
 .../src/sync/stream/ReactNativeRemote.ts                  | 1 -
 yarn.lock                                                 | 8 ++++----
 7 files changed, 11 insertions(+), 12 deletions(-)

diff --git a/.changeset/chilled-walls-promise.md b/.changeset/chilled-walls-promise.md
index 6af427c0..ca8247b0 100644
--- a/.changeset/chilled-walls-promise.md
+++ b/.changeset/chilled-walls-promise.md
@@ -4,3 +4,6 @@
 ---
 
 Updated logic to correspond with React Native Quick SQLite concurrent transactions. Added helper methods on transaction contexts.
+
+API changes include:
+- Removal of synchronous DB operations in transactions: `execute`, `commit`, `rollback` are now async functions. `executeAsync`, `commitAsync` and `rollbackAsync` have been removed. 
diff --git a/apps/supabase-todolist b/apps/supabase-todolist
index 2acf466d..c9552550 160000
--- a/apps/supabase-todolist
+++ b/apps/supabase-todolist
@@ -1 +1 @@
-Subproject commit 2acf466dc78d69f44692dfe49bd478fc5d08ccaa
+Subproject commit c9552550e7f7de168a0ad7bdfbfc58f7ca0c9c13
diff --git a/packages/powersync-sdk-common/src/client/AbstractPowerSyncDatabase.ts b/packages/powersync-sdk-common/src/client/AbstractPowerSyncDatabase.ts
index 9380e775..1b5fcd8e 100644
--- a/packages/powersync-sdk-common/src/client/AbstractPowerSyncDatabase.ts
+++ b/packages/powersync-sdk-common/src/client/AbstractPowerSyncDatabase.ts
@@ -387,7 +387,7 @@ export abstract class AbstractPowerSyncDatabase extends BaseObserver<PowerSyncDB
     );
   }
 
-  async *watch(sql: string, parameters: any[], options?: SQLWatchOptions): AsyncIterable<QueryResult> {
+  async *watch(sql: string, parameters?: any[], options?: SQLWatchOptions): AsyncIterable<QueryResult> {
     //Fetch initial data
     yield await this.execute(sql, parameters);
 
diff --git a/packages/powersync-sdk-common/src/client/sync/stream/AbstractStreamingSyncImplementation.ts b/packages/powersync-sdk-common/src/client/sync/stream/AbstractStreamingSyncImplementation.ts
index 30061be5..2dd7fa8c 100644
--- a/packages/powersync-sdk-common/src/client/sync/stream/AbstractStreamingSyncImplementation.ts
+++ b/packages/powersync-sdk-common/src/client/sync/stream/AbstractStreamingSyncImplementation.ts
@@ -269,9 +269,7 @@ export abstract class AbstractStreamingSyncImplementation extends BaseObserver<S
   }
 
   async *streamingSyncRequest(req: StreamingSyncRequest, signal: AbortSignal): AsyncGenerator<StreamingSyncLine> {
-    console.log('sending request');
     const body = await this.options.remote.postStreaming('/sync/stream', req, {}, signal);
-    console.log('[streamingSyncRequest]', body);
     const stream = ndjsonStream(body);
     const reader = stream.getReader();
 
@@ -279,7 +277,6 @@ export abstract class AbstractStreamingSyncImplementation extends BaseObserver<S
       while (true) {
         // Read from the stream
         const { done, value } = await reader.read();
-        console.log('value', value);
         // Exit if we're done
         if (done) return;
         // Else yield the chunk
diff --git a/packages/powersync-sdk-react-native/package.json b/packages/powersync-sdk-react-native/package.json
index 4fd4568b..f5c9dc82 100644
--- a/packages/powersync-sdk-react-native/package.json
+++ b/packages/powersync-sdk-react-native/package.json
@@ -24,7 +24,7 @@
   },
   "homepage": "https://docs.powersync.co/",
   "peerDependencies": {
-    "@journeyapps/react-native-quick-sqlite": "0.0.0-dev-20231026145554",
+    "@journeyapps/react-native-quick-sqlite": "0.0.0-dev-20231030064733",
     "base-64": "^1.0.0",
     "react": "*",
     "react-native-fetch-api": "^3.0.0",
@@ -40,7 +40,7 @@
     "async-lock": "^1.4.0"
   },
   "devDependencies": {
-    "@journeyapps/react-native-quick-sqlite": "0.0.0-dev-20231026145554",
+    "@journeyapps/react-native-quick-sqlite": "0.0.0-dev-20231030064733",
     "@types/async-lock": "^1.4.0",
     "react": "18.2.0",
     "typescript": "^4.1.3"
diff --git a/packages/powersync-sdk-react-native/src/sync/stream/ReactNativeRemote.ts b/packages/powersync-sdk-react-native/src/sync/stream/ReactNativeRemote.ts
index 7a342846..8af575d8 100644
--- a/packages/powersync-sdk-react-native/src/sync/stream/ReactNativeRemote.ts
+++ b/packages/powersync-sdk-react-native/src/sync/stream/ReactNativeRemote.ts
@@ -71,7 +71,6 @@ export class ReactNativeRemote extends AbstractRemote {
       throw error;
     }
 
-    console.log(res.body);
     return res.body;
   }
 }
diff --git a/yarn.lock b/yarn.lock
index 759e0839..3190fcbe 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -2134,10 +2134,10 @@
     "@types/yargs" "^17.0.8"
     chalk "^4.0.0"
 
-"@journeyapps/react-native-quick-sqlite@0.0.0-dev-20231026145554":
-  version "0.0.0-dev-20231026145554"
-  resolved "https://registry.npmjs.org/@journeyapps/react-native-quick-sqlite/-/react-native-quick-sqlite-0.0.0-dev-20231026145554.tgz#bb2169f270d58b03fd7a8e3a81a956563010e45a"
-  integrity sha512-0JCfOsgfynOKCPoP7YijpGamBBG0AQi/3Ai01WV6mbJylGGHCPeWo3ErM0x02eL5sIiy64ZIFUsTNhxt/Z5dIg==
+"@journeyapps/react-native-quick-sqlite@0.0.0-dev-20231030064733":
+  version "0.0.0-dev-20231030064733"
+  resolved "https://registry.npmjs.org/@journeyapps/react-native-quick-sqlite/-/react-native-quick-sqlite-0.0.0-dev-20231030064733.tgz#33008ee214a52726d4e4429af7e196bbea2e4293"
+  integrity sha512-IR7fWhMY/SRAdBINie67OncupRSluIAH2LyR3YEOMJGbXvVbPtDGieT1fXIiUZ2VFbhbIhXOTusgZ4RADmbFiQ==
   dependencies:
     lodash "^4.17.21"