Fix Future already completed when calling waitForFirstSync (#117)

* Ensure future is not completed

* Bump version

* Add changelog entry for issue 114.

* Iterate through stream instead of listening to it

* Fix changelog message

---------

Co-authored-by: Ralf Kistner <[email protected]>

Author: mugikhan

packages/powersync/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 1.5.2
+
+- Refactor `waitForFirstSync()` to iterate through the stream and remove the use of a `Future`.
+- Fix sync connection not immediately closed when calling `db.disconnect()` (#114).
+
 ## 1.5.1
 
 - Adds a hasSynced flag to check if initial data has been synced.

packages/powersync/lib/src/powersync_database.dart

@@ -197,14 +197,11 @@ class PowerSyncDatabase with SqliteQueries implements SqliteConnection {
     if (currentStatus.hasSynced ?? false) {
       return;
     }
-    final completer = Completer<void>();
-    statusStream.listen((result) {
+    await for (final result in statusStream) {
       if (result.hasSynced ?? false) {
-        completer.complete();
+        break;
       }
-    });
-
-    return completer.future;
+    }
   }
 
   @override

packages/powersync/pubspec.yaml

@@ -1,5 +1,5 @@
 name: powersync
-version: 1.5.1
+version: 1.5.2
 homepage: https://powersync.com
 repository: https://github.com/powersync-ja/powersync.dart
 description: PowerSync Flutter SDK - keep PostgreSQL databases in sync with on-device SQLite databases.