Update sqlite3 to version 2.4.4 (#47)

* Bump sqlite3 version

* Update to single database type

* Update test dart sdk version

* Revert to using CommonDatabase type

* Use CommonDatabase type

* Revert some changes
Update to version 0.7.0

* Fix import

* Change open DB to be synchronous

* Update changelog

* Fix changelog message

* Fix linting errors

Author: mugikhan

.github/workflows/test.yaml

@@ -32,19 +32,19 @@ jobs:
         include:
           - sqlite_version: "3440200"
             sqlite_url: "https://www.sqlite.org/2023/sqlite-autoconf-3440200.tar.gz"
-            dart_sdk: 3.3.3
+            dart_sdk: 3.4.0
           - sqlite_version: "3430200"
             sqlite_url: "https://www.sqlite.org/2023/sqlite-autoconf-3430200.tar.gz"
-            dart_sdk: 3.3.3
+            dart_sdk: 3.4.0
           - sqlite_version: "3420000"
             sqlite_url: "https://www.sqlite.org/2023/sqlite-autoconf-3420000.tar.gz"
-            dart_sdk: 3.3.3
+            dart_sdk: 3.4.0
           - sqlite_version: "3410100"
             sqlite_url: "https://www.sqlite.org/2023/sqlite-autoconf-3410100.tar.gz"
-            dart_sdk: 3.3.3
+            dart_sdk: 3.4.0
           - sqlite_version: "3380000"
             sqlite_url: "https://www.sqlite.org/2022/sqlite-autoconf-3380000.tar.gz"
-            dart_sdk: 3.3.3
+            dart_sdk: 3.4.0
     steps:
       - uses: actions/checkout@v3
       - uses: dart-lang/setup-dart@v1

CHANGELOG.md

@@ -1,3 +1,9 @@
+## 0.7.0
+
+- BREAKING CHANGE: Update all Database types to use a `CommonDatabase` interface.
+- Update `openDB` and `open` methods to be synchronous.
+- Fix `ArgumentError (Invalid argument(s): argument value for 'return_value' is null)` in sqlite3 when closing the database connection by upgrading to version 2.4.4.
+
 ## 0.6.1
 
 - Fix errors when closing a `SqliteDatabase`.

example/custom_functions_example.dart

@@ -1,4 +1,3 @@
-import 'dart:async';
 import 'dart:io';
 import 'dart:isolate';
 
@@ -11,8 +10,8 @@ class TestOpenFactory extends DefaultSqliteOpenFactory {
   TestOpenFactory({required super.path, super.sqliteOptions});
 
   @override
-  FutureOr<CommonDatabase> open(SqliteOpenOptions options) async {
-    final db = await super.open(options);
+  CommonDatabase open(SqliteOpenOptions options) {
+    final db = super.open(options);
 
     db.createFunction(
       functionName: 'sleep',

example/linux_cli_example.dart

@@ -1,4 +1,3 @@
-import 'dart:async';
 import 'dart:ffi';
 
 import 'package:sqlite3/common.dart';
@@ -17,7 +16,7 @@ class TestOpenFactory extends DefaultSqliteOpenFactory {
       this.sqlitePath = defaultSqlitePath});
 
   @override
-  FutureOr<CommonDatabase> open(SqliteOpenOptions options) async {
+  CommonDatabase open(SqliteOpenOptions options) {
     // For details, see:
     // https://pub.dev/packages/sqlite3#manually-providing-sqlite3-libraries
     sqlite_open.open.overrideFor(sqlite_open.OperatingSystem.linux, () {

lib/src/common/abstract_open_factory.dart

@@ -15,7 +15,7 @@ abstract class SqliteOpenFactory<Database extends sqlite.CommonDatabase> {
   String get path;
 
   /// Opens a direct connection to the SQLite database
-  FutureOr<Database> open(SqliteOpenOptions options);
+  Database open(SqliteOpenOptions options);
 
   /// Opens an asynchronous [SqliteConnection]
   FutureOr<SqliteConnection> openConnection(SqliteOpenOptions options);
@@ -77,14 +77,14 @@ abstract class AbstractDefaultSqliteOpenFactory<
   @protected
 
   /// Opens a direct connection to a SQLite database connection
-  FutureOr<Database> openDB(SqliteOpenOptions options);
+  Database openDB(SqliteOpenOptions options);
 
   @override
 
   /// Opens a direct connection to a SQLite database connection
   /// and executes setup pragma statements to initialize the DB
-  FutureOr<Database> open(SqliteOpenOptions options) async {
-    var db = await openDB(options);
+  Database open(SqliteOpenOptions options) {
+    var db = openDB(options);
 
     // Pragma statements don't have the same BUSY_TIMEOUT behavior as normal statements.
     // We add a manual retry loop for those.

lib/src/native/database/native_sqlite_connection_impl.dart

@@ -264,7 +264,7 @@ void _sqliteConnectionIsolate(_SqliteConnectionParams params) async {
     await client.post(const InitDb());
   }
 
-  final db = await params.openFactory.open(SqliteOpenOptions(
+  final db = params.openFactory.open(SqliteOpenOptions(
       primaryConnection: params.primary, readOnly: params.readOnly));
 
   runZonedGuarded(() async {

pubspec.yaml

@@ -1,12 +1,12 @@
 name: sqlite_async
 description: High-performance asynchronous interface for SQLite on Dart and Flutter.
-version: 0.6.1
+version: 0.7.0
 repository: https://github.com/powersync-ja/sqlite_async.dart
 environment:
   sdk: ">=3.2.0 <4.0.0"
 
 dependencies:
-  sqlite3: "^2.3.0"
+  sqlite3: "^2.4.4"
   async: ^2.10.0
   collection: ^1.17.0
   meta: ^1.10.0

test/utils/native_test_utils.dart

@@ -21,11 +21,11 @@ class TestSqliteOpenFactory extends TestDefaultSqliteOpenFactory {
       initStatements});
 
   @override
-  FutureOr<CommonDatabase> open(SqliteOpenOptions options) async {
+  CommonDatabase open(SqliteOpenOptions options) {
     sqlite_open.open.overrideFor(sqlite_open.OperatingSystem.linux, () {
       return DynamicLibrary.open(sqlitePath);
     });
-    final db = await super.open(options);
+    final db = super.open(options);
 
     db.createFunction(
       functionName: 'test_sleep',