Update sqlite3_web to 0.8.0

Author: simolus3

.github/workflows/release.yml

@@ -25,7 +25,7 @@ jobs:
       - name: Compile web worker
         run: |
           dart pub get
-          dart compile js -O4 -o assets/db_worker.js packages/sqlite_async/lib/src/web/worker/worker.dart
+          dart compile js -O4 -Dsqlite3.dartbigints=false -o assets/db_worker.js packages/sqlite_async/lib/src/web/worker/worker.dart
 
       - name: Set tag name
         id: tag

.github/workflows/test.yaml

@@ -92,7 +92,7 @@ jobs:
       - name: Setup
         run: |
           dart run tool/sqlite3_wasm_download.dart
-          dart compile js -O4 --no-minify -o assets/db_worker.js packages/sqlite_async/lib/src/web/worker/worker.dart
+          dart compile js -O4 --no-minify -Dsqlite3.dartbigints=false -o assets/db_worker.js packages/sqlite_async/lib/src/web/worker/worker.dart
 
       - name: Test sqlite_async
         working-directory: packages/sqlite_async

packages/sqlite_async/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.14.2
+
+- Support versions `0.8.x` of `package:sqlite3_web`.
+
 ## 0.14.1
 
 - Fix update notifications sometimes firing before a write has completed.

packages/sqlite_async/lib/src/utils/profiler.dart

@@ -55,7 +55,11 @@ extension TimeSync on TimelineTask? {
       if (parameters != null)
         'parameters': [
           for (final parameter in parameters)
-            if (parameter is List) '<blob>' else parameter
+            switch (parameter) {
+              List() => '<blob>',
+              BigInt() => parameter.toString(),
+              _ => parameter,
+            }
         ],
     }
   );

packages/sqlite_async/lib/src/web/web_sqlite_open_factory.dart

@@ -45,9 +45,9 @@ base class WebSqliteOpenFactory extends InternalOpenFactory {
   /// when provided with the same [options] again.
   Future<WebSqlite> openWebSqlite(WebSqliteOptions options) async {
     return WebSqlite.open(
-      wasmModule: Uri.parse(sqliteOptions.webSqliteOptions.wasmUri),
+      wasmModule: sqliteOptions.webSqliteOptions.wasmUri,
       workers: WorkerConnector.defaultWorkers(
-          Uri.parse(sqliteOptions.webSqliteOptions.workerUri)),
+          sqliteOptions.webSqliteOptions.workerUri),
       controller: AsyncSqliteController(),
       handleCustomRequest: handleCustomRequest,
     );

packages/sqlite_async/lib/src/web/worker/worker.dart

@@ -1,8 +1,22 @@
 /// This is an example of a database worker script
-/// Custom database logic can be achieved by implementing this template
-/// This file needs to be compiled to JavaScript with the command:
-///   dart compile js -O4 lib/src/web/worker/db_worker.dart -o build/db_worker.js
-/// The output should then be included in each project's `web` directory
+/// Custom database logic can be achieved by implementing this template.
+///
+/// This file needs to be compiled to JavaScript via `dart compile js`:
+///
+/// ```
+/// dart compile js \
+///   lib/src/web/worker/db_worker.dart \
+///   -o build/db_worker.js \
+///   -O4
+///   -Dsqlite3.dartbigints=false
+/// ```
+///
+/// The output should then be included in each project's `web` directory.
+///
+/// Disabling `sqlite3.dartbigints` as a compile-time option reduces the size of
+/// the worker by disabling support for [BigInt] values which are implemented in
+/// Dart. The worker still supports native JavaScript bigint values, so this
+/// doesn't affect functionality.
 library;
 
 import 'package:sqlite3_web/sqlite3_web.dart';

packages/sqlite_async/pubspec.yaml

@@ -1,6 +1,6 @@
 name: sqlite_async
 description: High-performance asynchronous interface for SQLite on Dart and Flutter.
-version: 0.14.1
+version: 0.14.2
 resolution: workspace
 repository: https://github.com/powersync-ja/sqlite_async.dart
 environment:
@@ -14,7 +14,7 @@ topics:
 
 dependencies:
   sqlite3: ^3.2.0
-  sqlite3_web: ^0.7.0
+  sqlite3_web: ^0.8.0
   sqlite3_connection_pool: ^0.2.4
   async: ^2.10.0
   collection: ^1.17.0

packages/sqlite_async/test/web/web_test.dart

@@ -0,0 +1,60 @@
+@TestOn('browser')
+library;
+
+import 'dart:async';
+import 'dart:math';
+import 'dart:typed_data';
+import 'package:test/test.dart';
+
+import '../utils/test_utils_impl.dart';
+
+const _isDart2Wasm = bool.fromEnvironment('dart.tool.dart2wasm');
+final testUtils = TestUtils();
+
+void main() {
+  group('Web database tests', () {
+    late String path;
+
+    setUp(() async {
+      path = testUtils.dbPath();
+      await testUtils.cleanDb(path: path);
+    });
+
+    tearDown(() async {
+      await testUtils.cleanDb(path: path);
+    });
+
+    test('can bind all types', () async {
+      final db = await testUtils.setupDatabase(path: path);
+      addTearDown(db.close);
+
+      Future<void> assertTypeOf(Object? value, String typeName) async {
+        final row = await db.get('SELECT typeof(?)', [value]);
+        expect(row.values[0], typeName);
+      }
+
+      await assertTypeOf(null, 'null');
+      await assertTypeOf('hello web', 'text');
+      await assertTypeOf(Uint8List(12), 'blob');
+      await assertTypeOf(2, 'integer');
+      await assertTypeOf(BigInt.two, 'integer');
+      await assertTypeOf(BigInt.one << 62, 'integer');
+      await assertTypeOf(pi, 'real');
+    });
+
+    test('large integers are read as bigints on JS', () async {
+      final db = await testUtils.setupDatabase(path: path);
+      addTearDown(db.close);
+
+      final value = (await db.get('SELECT 1 << 62')).values[0];
+      if (_isDart2Wasm) {
+        // We have 64bit ints on dart2wasm, which should be used here.
+        expect(value, isA<int>());
+        expect(value, 1 << 62);
+      } else {
+        expect(value, isA<BigInt>());
+        expect(value, BigInt.parse('4611686018427387904'));
+      }
+    });
+  });
+}