@syncular/server 0.13.0 → 0.15.0

package/dist/relational-rows.js

@@ -223,13 +223,9 @@ export function upsertSql(table, dialect) {
     const names = tableColumnNames(table);
     const quoted = names.map((name) => quoteIdent(name)).join(', ');
     const values = placeholderList(names.length, dialect);
-    if (dialect === 'sqlite') {
-        // INSERT OR REPLACE keys on the (_sync_partition, _sync_row_id) PK.
-        return `INSERT OR REPLACE INTO ${quoteIdent(table.name)} (${quoted}) VALUES (${values})`;
-    }
     const updates = names
         .slice(2) // partition + row id are the conflict key
-        .map((name) => `${quoteIdent(name)}=EXCLUDED.${quoteIdent(name)}`)
+        .map((name) => `${quoteIdent(name)}=excluded.${quoteIdent(name)}`)
         .join(', ');
     return `INSERT INTO ${quoteIdent(table.name)} (${quoted}) VALUES (${values})
      ON CONFLICT (${quoteIdent(SYNC_PARTITION_COLUMN)}, ${quoteIdent(SYNC_ROW_ID_COLUMN)}) DO UPDATE SET ${updates}`;

package/dist/sqlite-dialect.d.ts

@@ -2,8 +2,8 @@
  * Shared SQLite dialect for the two SQLite-family storages: `bun:sqlite`
  * (synchronous, `SqliteServerStorage`) and Cloudflare D1 (async,
  * `D1ServerStorage`). D1 *is* SQLite — same DDL, same statement grammar,
- * same `?` positional placeholders, same `INSERT OR REPLACE` / `INSERT OR
- * IGNORE` upsert idioms — so the schema and the value (de)serialization are
+ * same `?` positional placeholders, same `INSERT ... ON CONFLICT` / `INSERT
+ * OR IGNORE` upsert idioms — so the schema and the value (de)serialization are
  * genuinely common ground and live here.
  *
  * What is NOT shared: statement *execution*. `bun:sqlite` is sync

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@syncular/server",
-  "version": "0.13.0",
+  "version": "0.15.0",
   "description": "Syncular server: handleSyncRequest + storage/auth interfaces for the sync protocol",
   "license": "Apache-2.0",
   "author": "Benjamin Kniffler",
@@ -53,7 +53,7 @@
     "!dist/**/*.test.d.ts"
   ],
   "dependencies": {
-    "@syncular/core": "0.13.0"
+    "@syncular/core": "0.15.0"
   },
   "devDependencies": {
     "@electric-sql/pglite": "^0.5.4"

package/src/relational-rows.ts

@@ -261,13 +261,9 @@ export function upsertSql(
   const names = tableColumnNames(table);
   const quoted = names.map((name) => quoteIdent(name)).join(', ');
   const values = placeholderList(names.length, dialect);
-  if (dialect === 'sqlite') {
-    // INSERT OR REPLACE keys on the (_sync_partition, _sync_row_id) PK.
-    return `INSERT OR REPLACE INTO ${quoteIdent(table.name)} (${quoted}) VALUES (${values})`;
-  }
   const updates = names
     .slice(2) // partition + row id are the conflict key
-    .map((name) => `${quoteIdent(name)}=EXCLUDED.${quoteIdent(name)}`)
+    .map((name) => `${quoteIdent(name)}=excluded.${quoteIdent(name)}`)
     .join(', ');
   return `INSERT INTO ${quoteIdent(table.name)} (${quoted}) VALUES (${values})
      ON CONFLICT (${quoteIdent(SYNC_PARTITION_COLUMN)}, ${quoteIdent(SYNC_ROW_ID_COLUMN)}) DO UPDATE SET ${updates}`;

package/src/sqlite-dialect.ts

@@ -2,8 +2,8 @@
  * Shared SQLite dialect for the two SQLite-family storages: `bun:sqlite`
  * (synchronous, `SqliteServerStorage`) and Cloudflare D1 (async,
  * `D1ServerStorage`). D1 *is* SQLite — same DDL, same statement grammar,
- * same `?` positional placeholders, same `INSERT OR REPLACE` / `INSERT OR
- * IGNORE` upsert idioms — so the schema and the value (de)serialization are
+ * same `?` positional placeholders, same `INSERT ... ON CONFLICT` / `INSERT
+ * OR IGNORE` upsert idioms — so the schema and the value (de)serialization are
  * genuinely common ground and live here.
  *
  * What is NOT shared: statement *execution*. `bun:sqlite` is sync