@syncular/client 0.15.12 → 0.15.13

package/dist/client.js

@@ -181,9 +181,7 @@ export class SyncClient {
         this.#detectAndResetSchema();
         // A registration remains app intent only while its table exists in the
         // running schema. Removed-table registrations would poison every pull.
-        this.#db.transaction(() => {
-            pruneUnknownSubscriptions(this.#db, new Set(this.#schema.tables.keys()));
-        });
+        const subscriptions = pruneUnknownSubscriptions(this.#db, loadSubscriptions(this.#db), new Set(this.#schema.tables.keys()));
         this.#started = true;
         // A persisted active subscription needs one catch-up round on every open:
         // realtime only covers changes after the socket connects, and an
@@ -193,7 +191,7 @@ export class SyncClient {
         // an application-issued sync() call.
         const startupWork = this.#schemaFloor === undefined &&
             (listOutbox(this.#db).length > 0 ||
-                loadSubscriptions(this.#db).some((sub) => sub.status === 'active'));
+                subscriptions.some((sub) => sub.status === 'active'));
         if (startupWork) {
             this.#needsPull = true;
             this.#config.onSyncNeeded?.('startup');

package/dist/state.d.ts

@@ -42,9 +42,10 @@ export declare function resetSubscriptionsForBump(db: ClientDatabase): void;
  * Remove registrations whose table no longer exists in the running schema.
  * Keeping one would make every subsequent pull fail with
  * `sync.unknown_table`. Window bookkeeping belongs to the registration and
- * is removed with it.
+ * is removed with it. The caller supplies its startup subscription snapshot
+ * so pruning and startup-work detection stay a single read.
  */
-export declare function pruneUnknownSubscriptions(db: ClientDatabase, tableNames: ReadonlySet<string>): void;
+export declare function pruneUnknownSubscriptions(db: ClientDatabase, subscriptions: readonly SubscriptionRecord[], tableNames: ReadonlySet<string>): SubscriptionRecord[];
 export declare function getMeta(db: ClientDatabase, key: string): string | undefined;
 export declare function setMeta(db: ClientDatabase, key: string, value: string): void;
 /** Read the durable client-local observer revision (SPEC §7.5). */

package/dist/state.js

@@ -69,20 +69,30 @@ export function resetSubscriptionsForBump(db) {
  * Remove registrations whose table no longer exists in the running schema.
  * Keeping one would make every subsequent pull fail with
  * `sync.unknown_table`. Window bookkeeping belongs to the registration and
- * is removed with it.
+ * is removed with it. The caller supplies its startup subscription snapshot
+ * so pruning and startup-work detection stay a single read.
  */
-export function pruneUnknownSubscriptions(db, tableNames) {
-    const staleIds = db
-        .query('SELECT id, tbl FROM _syncular_subscriptions')
-        .filter((row) => !tableNames.has(String(row.tbl)))
-        .map((row) => String(row.id));
-    for (const id of staleIds) {
-        db.exec('DELETE FROM _syncular_windows WHERE sub_id = ?', [id]);
-        db.exec('DELETE FROM _syncular_window_pending_evict WHERE sub_id = ?', [
-            id,
-        ]);
-        db.exec('DELETE FROM _syncular_subscriptions WHERE id = ?', [id]);
-    }
+export function pruneUnknownSubscriptions(db, subscriptions, tableNames) {
+    const retained = subscriptions.filter((record) => tableNames.has(record.table));
+    const staleIds = subscriptions
+        .filter((record) => !tableNames.has(record.table))
+        .map((record) => record.id);
+    // A redundant subscription scan plus an empty write transaction cut the
+    // fresh-client bootstrap lane by more than half on bun:sqlite. The supplied
+    // snapshot and this branch are synchronous, so no application work can
+    // interleave before a real pruning transaction begins.
+    if (staleIds.length === 0)
+        return retained;
+    db.transaction(() => {
+        for (const id of staleIds) {
+            db.exec('DELETE FROM _syncular_windows WHERE sub_id = ?', [id]);
+            db.exec('DELETE FROM _syncular_window_pending_evict WHERE sub_id = ?', [
+                id,
+            ]);
+            db.exec('DELETE FROM _syncular_subscriptions WHERE id = ?', [id]);
+        }
+    });
+    return retained;
 }
 export function getMeta(db, key) {
     const row = db.query('SELECT value FROM _syncular_meta WHERE key = ?', [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@syncular/client",
-  "version": "0.15.12",
+  "version": "0.15.13",
   "description": "Syncular TypeScript client core — offline-first sync over SQLite (WASM/OPFS, Bun, Node)",
   "license": "Apache-2.0",
   "author": "Benjamin Kniffler",
@@ -81,7 +81,7 @@
   },
   "dependencies": {
     "@sqlite.org/sqlite-wasm": "^3.53.0-build1",
-    "@syncular/core": "0.15.12"
+    "@syncular/core": "0.15.13"
   },
   "peerDependencies": {
     "better-sqlite3": ">=11"
@@ -92,7 +92,7 @@
     }
   },
   "devDependencies": {
-    "@syncular/server": "0.15.12",
+    "@syncular/server": "0.15.13",
     "@types/better-sqlite3": "^7.6.13",
     "better-sqlite3": "^12.11.1"
   }

package/src/client.ts

@@ -597,9 +597,11 @@ export class SyncClient {
     this.#detectAndResetSchema();
     // A registration remains app intent only while its table exists in the
     // running schema. Removed-table registrations would poison every pull.
-    this.#db.transaction(() => {
-      pruneUnknownSubscriptions(this.#db, new Set(this.#schema.tables.keys()));
-    });
+    const subscriptions = pruneUnknownSubscriptions(
+      this.#db,
+      loadSubscriptions(this.#db),
+      new Set(this.#schema.tables.keys()),
+    );
     this.#started = true;
     // A persisted active subscription needs one catch-up round on every open:
     // realtime only covers changes after the socket connects, and an
@@ -610,7 +612,7 @@ export class SyncClient {
     const startupWork =
       this.#schemaFloor === undefined &&
       (listOutbox(this.#db).length > 0 ||
-        loadSubscriptions(this.#db).some((sub) => sub.status === 'active'));
+        subscriptions.some((sub) => sub.status === 'active'));
     if (startupWork) {
       this.#needsPull = true;
       this.#config.onSyncNeeded?.('startup');

package/src/state.ts

@@ -119,23 +119,35 @@ export function resetSubscriptionsForBump(db: ClientDatabase): void {
  * Remove registrations whose table no longer exists in the running schema.
  * Keeping one would make every subsequent pull fail with
  * `sync.unknown_table`. Window bookkeeping belongs to the registration and
- * is removed with it.
+ * is removed with it. The caller supplies its startup subscription snapshot
+ * so pruning and startup-work detection stay a single read.
  */
 export function pruneUnknownSubscriptions(
   db: ClientDatabase,
+  subscriptions: readonly SubscriptionRecord[],
   tableNames: ReadonlySet<string>,
-): void {
-  const staleIds = db
-    .query('SELECT id, tbl FROM _syncular_subscriptions')
-    .filter((row) => !tableNames.has(String(row.tbl)))
-    .map((row) => String(row.id));
-  for (const id of staleIds) {
-    db.exec('DELETE FROM _syncular_windows WHERE sub_id = ?', [id]);
-    db.exec('DELETE FROM _syncular_window_pending_evict WHERE sub_id = ?', [
-      id,
-    ]);
-    db.exec('DELETE FROM _syncular_subscriptions WHERE id = ?', [id]);
-  }
+): SubscriptionRecord[] {
+  const retained = subscriptions.filter((record) =>
+    tableNames.has(record.table),
+  );
+  const staleIds = subscriptions
+    .filter((record) => !tableNames.has(record.table))
+    .map((record) => record.id);
+  // A redundant subscription scan plus an empty write transaction cut the
+  // fresh-client bootstrap lane by more than half on bun:sqlite. The supplied
+  // snapshot and this branch are synchronous, so no application work can
+  // interleave before a real pruning transaction begins.
+  if (staleIds.length === 0) return retained;
+  db.transaction(() => {
+    for (const id of staleIds) {
+      db.exec('DELETE FROM _syncular_windows WHERE sub_id = ?', [id]);
+      db.exec('DELETE FROM _syncular_window_pending_evict WHERE sub_id = ?', [
+        id,
+      ]);
+      db.exec('DELETE FROM _syncular_subscriptions WHERE id = ?', [id]);
+    }
+  });
+  return retained;
 }
 
 export function getMeta(db: ClientDatabase, key: string): string | undefined {