@syncular/client 0.13.0 → 0.14.0

package/dist/apply.d.ts

@@ -61,7 +61,7 @@ export interface SqliteSegmentDescriptor {
  * the in-file metadata against the descriptor, validate the data table's
  * column names/order against the generated schema, run the §5.6
  * first-page clear when fresh, then copy every row with a single
- * `INSERT OR REPLACE … SELECT` — `_syncular_version` lands in
+ * one primary-key upsert — `_syncular_version` lands in
  * `_sync_version` exactly like a rows segment's per-row `serverVersion`.
  */
 export declare function applySqliteSegment(db: ClientDatabase, schema: CompiledClientSchema, table: CompiledClientTable, bytes: Uint8Array, descriptor: SqliteSegmentDescriptor, options: {

package/dist/apply.js

@@ -12,7 +12,12 @@ function upsertSql(table) {
         quoteIdent(SYNC_VERSION_COLUMN),
     ];
     const placeholders = names.map(() => '?').join(', ');
-    return `INSERT OR REPLACE INTO ${quoteIdent(table.name)} (${names.join(', ')}) VALUES (${placeholders})`;
+    const updates = names
+        .filter((name) => name !== quoteIdent(table.primaryKey))
+        .map((name) => `${name}=excluded.${name}`)
+        .join(', ');
+    return `INSERT INTO ${quoteIdent(table.name)} (${names.join(', ')}) VALUES (${placeholders})
+    ON CONFLICT (${quoteIdent(table.primaryKey)}) DO UPDATE SET ${updates}`;
 }
 export function upsertLocalRow(db, table, values, syncVersion) {
     db.exec(upsertSql(table), [...values.map(toSqlValue), syncVersion]);
@@ -182,7 +187,7 @@ function imageInvalid(detail) {
  * the in-file metadata against the descriptor, validate the data table's
  * column names/order against the generated schema, run the §5.6
  * first-page clear when fresh, then copy every row with a single
- * `INSERT OR REPLACE … SELECT` — `_syncular_version` lands in
+ * one primary-key upsert — `_syncular_version` lands in
  * `_sync_version` exactly like a rows segment's per-row `serverVersion`.
  */
 export function applySqliteSegment(db, schema, table, bytes, descriptor, options) {
@@ -239,16 +244,23 @@ export function applySqliteSegment(db, schema, table, bytes, descriptor, options
             expected.some((name, index) => actual[index] !== name)) {
             throw new ClientSyncError('sync.schema_mismatch', `sqlite segment for ${JSON.stringify(table.name)} does not match the generated schema: columns [${actual.join(', ')}] (§5.3)`);
         }
-        // 3. One transaction: fresh-bootstrap clear, then replace-or-upsert.
+        // 3. One transaction: fresh-bootstrap clear, then primary-key upsert.
         const names = table.columns.map((column) => quoteIdent(column.name));
+        const insertNames = [...names, quoteIdent(SYNC_VERSION_COLUMN)];
+        const updates = insertNames
+            .filter((name) => name !== quoteIdent(table.primaryKey))
+            .map((name) => `${name}=excluded.${name}`)
+            .join(', ');
         return (options.transaction ?? ((fn) => db.transaction(fn)))(() => {
             if (options.clearFirst) {
                 deleteScopedRows(db, table, options.effective);
             }
-            db.exec(`INSERT OR REPLACE INTO ${quoteIdent(table.name)}
-           (${[...names, quoteIdent(SYNC_VERSION_COLUMN)].join(', ')})
+            db.exec(`INSERT INTO ${quoteIdent(table.name)}
+           (${insertNames.join(', ')})
          SELECT ${[...names, quoteIdent('_syncular_version')].join(', ')}
-         FROM ${IMAGE_ALIAS}.${quoteIdent(table.name)}`);
+         FROM ${IMAGE_ALIAS}.${quoteIdent(table.name)}
+         WHERE true
+         ON CONFLICT (${quoteIdent(table.primaryKey)}) DO UPDATE SET ${updates}`);
             const counted = db.query(`SELECT count(*) AS n FROM ${IMAGE_ALIAS}.${quoteIdent(table.name)}`)[0];
             const applied = Number(counted?.n ?? 0);
             if (applied !== descriptor.rowCount) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@syncular/client",
-  "version": "0.13.0",
+  "version": "0.14.0",
   "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.13.0"
+    "@syncular/core": "0.14.0"
   },
   "peerDependencies": {
     "better-sqlite3": ">=11"
@@ -92,7 +92,7 @@
     }
   },
   "devDependencies": {
-    "@syncular/server": "0.13.0",
+    "@syncular/server": "0.14.0",
     "@types/better-sqlite3": "^7.6.13",
     "better-sqlite3": "^12.11.1"
   }

package/src/apply.ts

@@ -31,7 +31,12 @@ function upsertSql(table: CompiledClientTable): string {
     quoteIdent(SYNC_VERSION_COLUMN),
   ];
   const placeholders = names.map(() => '?').join(', ');
-  return `INSERT OR REPLACE INTO ${quoteIdent(table.name)} (${names.join(', ')}) VALUES (${placeholders})`;
+  const updates = names
+    .filter((name) => name !== quoteIdent(table.primaryKey))
+    .map((name) => `${name}=excluded.${name}`)
+    .join(', ');
+  return `INSERT INTO ${quoteIdent(table.name)} (${names.join(', ')}) VALUES (${placeholders})
+    ON CONFLICT (${quoteIdent(table.primaryKey)}) DO UPDATE SET ${updates}`;
 }
 
 export function upsertLocalRow(
@@ -285,7 +290,7 @@ function imageInvalid(detail: string): never {
  * the in-file metadata against the descriptor, validate the data table's
  * column names/order against the generated schema, run the §5.6
  * first-page clear when fresh, then copy every row with a single
- * `INSERT OR REPLACE … SELECT` — `_syncular_version` lands in
+ * one primary-key upsert — `_syncular_version` lands in
  * `_sync_version` exactly like a rows segment's per-row `serverVersion`.
  */
 export function applySqliteSegment(
@@ -365,17 +370,24 @@ export function applySqliteSegment(
       );
     }
 
-    // 3. One transaction: fresh-bootstrap clear, then replace-or-upsert.
+    // 3. One transaction: fresh-bootstrap clear, then primary-key upsert.
     const names = table.columns.map((column) => quoteIdent(column.name));
+    const insertNames = [...names, quoteIdent(SYNC_VERSION_COLUMN)];
+    const updates = insertNames
+      .filter((name) => name !== quoteIdent(table.primaryKey))
+      .map((name) => `${name}=excluded.${name}`)
+      .join(', ');
     return (options.transaction ?? ((fn) => db.transaction(fn)))(() => {
       if (options.clearFirst) {
         deleteScopedRows(db, table, options.effective);
       }
       db.exec(
-        `INSERT OR REPLACE INTO ${quoteIdent(table.name)}
-           (${[...names, quoteIdent(SYNC_VERSION_COLUMN)].join(', ')})
+        `INSERT INTO ${quoteIdent(table.name)}
+           (${insertNames.join(', ')})
          SELECT ${[...names, quoteIdent('_syncular_version')].join(', ')}
-         FROM ${IMAGE_ALIAS}.${quoteIdent(table.name)}`,
+         FROM ${IMAGE_ALIAS}.${quoteIdent(table.name)}
+         WHERE true
+         ON CONFLICT (${quoteIdent(table.primaryKey)}) DO UPDATE SET ${updates}`,
       );
       const counted = db.query(
         `SELECT count(*) AS n FROM ${IMAGE_ALIAS}.${quoteIdent(table.name)}`,