Changelog and cleanup entries.

Author: Chriztiaan

PowerSync/PowerSync.Common/CHANGELOG.md

@@ -1,5 +1,8 @@
 # PowerSync.Common Changelog
 
+## 0.0.5-alpha.1
+- Using the latest (0.4.9) version of the core extension, it introduces support for the Rust Sync implementation and also makes it the default - users can still opt out and use the legacy C# sync implementation as option when calling `connect()`.
+
 ## 0.0.4-alpha.1
 - Fixed MAUI issues related to extension loading when installing package outside of the monorepo. 
 

PowerSync/PowerSync.Common/Client/Sync/Stream/StreamingSyncImplementation.cs

@@ -81,7 +81,7 @@ public class BaseConnectionOptions(Dictionary<string, object>? parameters = null
     public Dictionary<string, object>? Params { get; set; } = parameters;
 
     /// <summary>
-    /// Wehether to use the RUST or C# sync client implementation.
+    /// Whether to use the RUST or C# sync client implementation.
     /// </summary>
     public SyncClientImplementation? ClientImplementation { get; set; } = clientImplementation;
 }
@@ -561,7 +561,6 @@ void HandleInstruction(Instruction instruction)
 
     protected async Task<StreamingSyncIterationResult> LegacyStreamingSyncIteration(CancellationToken signal, RequiredPowerSyncConnectionOptions resolvedOptions)
     {
-        // There's also a warning explaining that the legacy client will be removed in the future
         logger.LogWarning("The legacy sync client implementation is deprecated and will be removed in a future release.");
         logger.LogDebug("Streaming sync iteration started");
         Options.Adapter.StartSession();
@@ -600,7 +599,7 @@ protected async Task<StreamingSyncIterationResult> LegacyStreamingSyncIteration(
                 Buckets = req,
                 IncludeChecksum = true,
                 RawData = true,
-                Parameters = resolvedOptions.Params, // Replace with actual params
+                Parameters = resolvedOptions.Params,
                 ClientId = clientId
             }
         };
@@ -664,7 +663,6 @@ protected async Task<StreamingSyncIterationResult> LegacyStreamingSyncIteration(
                 if (!result.CheckpointValid)
                 {
                     // This means checksums failed. Start again with a new checkpoint.
-                    // TODO: better back-off
                     await Task.Delay(50);
                     return new StreamingSyncIterationResult { LegacyRetry = true };
                 }
@@ -802,7 +800,6 @@ protected async Task<StreamingSyncIterationResult> LegacyStreamingSyncIteration(
                     if (!result.CheckpointValid)
                     {
                         // This means checksums failed. Start again with a new checkpoint.
-                        // TODO: better back-off
                         await Task.Delay(50);
                         return new StreamingSyncIterationResult { LegacyRetry = false };
                     }

PowerSync/PowerSync.Maui/CHANGELOG.md

@@ -1,5 +1,9 @@
 # PowerSync.Maui Changelog
 
+## 0.0.3-alpha.1
+- Upstream PowerSync.Common version bump
+- Using the latest (0.4.9) version of the core extension, it introduces support for the Rust Sync implementation and also makes it the default - users can still opt out and use the legacy C# sync implementation as option when calling `connect()`.
+
 ## 0.0.2-alpha.1
 - Fixed issues related to extension loading when installing package outside of the monorepo.
 

README.md

@@ -106,13 +106,13 @@ dotnet test -v n --framework net8.0 --filter "test-file-pattern"
 
 ### Integration Tests
 Integration tests in `PowerSync.Common.IntegrationTests` are intended to run against the [self-host nodejs demo](https://github.com/powersync-ja/self-host-demo/tree/main/demos/nodejs). 
-The integration tests are disabled by default, define the following environment variable to let them run.
+The integration tests are disabled by default, define the following environment variable when running the tests.
 
 ```bash
 RUN_INTEGRATION_TESTS=true dotnet test -v n --framework net8.0
 ```
 
-Run integration tests exclusively.
+Only run integration tests, without any unit tests.
 
 ```bash
 RUN_INTEGRATION_TESTS=true dotnet test -v n --framework net8.0 --filter "Category=Integration"

Tests/PowerSync/PowerSync.Common.IntegrationTests/PowerSync.Common.IntegrationTests.csproj

@@ -1,9 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-        <!-- <TargetFrameworks>netstandard2.0;net6.0;net8.0;net9.0</TargetFrameworks> -->
     <TargetFrameworks>net8.0</TargetFrameworks>
-
     <ImplicitUsings>enable</ImplicitUsings>
     <Nullable>enable</Nullable>
     <IsPackable>false</IsPackable>

Tests/PowerSync/PowerSync.Common.IntegrationTests/SyncIntegrationTests.cs

@@ -248,7 +248,7 @@ public IntegrationFactAttribute()
         // Set default timeout if not already set
         if (Timeout == 0)
         {
-            Timeout = 5000; // 30 seconds default for all integration tests
+            Timeout = 5000; // 5 seconds default for all integration tests
         }
     }
 }

Tests/PowerSync/PowerSync.Common.Tests/Client/Sync/CRUDTests.cs

@@ -127,8 +127,6 @@ public async Task UpdateTest()
             { "description", "test2" }
         });
 
-        Console.WriteLine(JsonConvert.SerializeObject(tx.Crud.First()));
-        Console.WriteLine(JsonConvert.SerializeObject(expectedCrudEntry));
         Assert.True(tx.Crud.First().Equals(expectedCrudEntry));
     }
 

Tools/Setup/Setup.cs

@@ -9,7 +9,6 @@ public class PowerSyncSetup
 {
     private const string VERSION = "0.4.9";
 
-    // https://github.com/powersync-ja/powersync-sqlite-core/releases/download/v0.4.9/libpowersync_aarch64.android.a
     private const string GITHUB_BASE_URL = $"https://github.com/powersync-ja/powersync-sqlite-core/releases/download/v{VERSION}";
     private const string MAVEN_BASE_URL = $"https://repo1.maven.org/maven2/com/powersync/powersync-sqlite-core/{VERSION}";