Make SyncStatus.toString() more concise

packages/powersync_core/lib/src/sync/sync_status.dart

@@ -199,7 +199,42 @@ final class SyncStatus {
 
   @override
   String toString() {
-    return "SyncStatus<connected: $connected connecting: $connecting downloading: $downloading (progress: $downloadProgress) uploading: $uploading lastSyncedAt: $lastSyncedAt, hasSynced: $hasSynced, error: $anyError>";
+    final buffer = StringBuffer('SyncStatus<');
+    var first = true;
+    void addProperty(String name, Object? value) {
+      if (!first) {
+        buffer.write(' ');
+      }
+
+      buffer.write('$name: $value');
+      first = false;
+    }
+
+    if (connected) {
+      addProperty('connected', connected);
+    } else if (connecting) {
+      addProperty('connected', 'connecting');
+    } else {
+      addProperty('connected', 'offline (not connecting)');
+    }
+
+    addProperty('downloading', '$downloading (progress: $downloadProgress)');
+    addProperty('uploading', uploading);
+    addProperty('lastSyncedAt', lastSyncedAt);
+    addProperty('hasSynced', hasSynced);
+
+    if (downloadError case final downloadError?) {
+      addProperty('downloadError', downloadError);
+    }
+    if (uploadError case final uploadError?) {
+      addProperty('uploadError', uploadError);
+    }
+    if (downloadError == null && uploadError == null) {
+      addProperty('error', null);
+    }
+
+    buffer.write('>');
+    return buffer.toString();
   }
 
   static const _listEquality = ListEquality<Object?>();

packages/powersync_core/test/sync/sync_status_test.dart

@@ -0,0 +1,41 @@
+import 'package:powersync_core/powersync_core.dart';
+import 'package:test/test.dart';
+
+void main() {
+  group('SyncStatus.toString', () {
+    test('default', () {
+      expect(SyncStatus().toString(),
+          'SyncStatus<connected: offline (not connecting) downloading: false (progress: null) uploading: false lastSyncedAt: null hasSynced: null error: null>');
+    });
+
+    test('connection status', () {
+      expect(SyncStatus(connected: true).toString(),
+          contains('SyncStatus<connected: true'));
+      expect(SyncStatus(connected: false, connecting: true).toString(),
+          contains('SyncStatus<connected: connecting'));
+      expect(SyncStatus().toString(),
+          contains('SyncStatus<connected: offline (not connecting)'));
+    });
+
+    group('errors', () {
+      test('upload error', () {
+        expect(SyncStatus(uploadError: 'test').toString(),
+            contains('uploadError: test'));
+      });
+
+      test('download error', () {
+        expect(SyncStatus(downloadError: 'test').toString(),
+            contains('downloadError: test'));
+      });
+
+      test('both upload and download error', () {
+        expect(SyncStatus(uploadError: 'a', downloadError: 'b').toString(),
+            contains('downloadError: b uploadError: a'));
+      });
+
+      test('no error', () {
+        expect(SyncStatus().toString(), contains('error: null'));
+      });
+    });
+  });
+}