@syncular/client 0.30.9 → 0.30.10

package/README.md

@@ -375,7 +375,9 @@ columns are present, and the durable local operation keeps the same presence
 set so conflict and rejection UI knows which columns the user touched. A
 full-row `mutate` upsert marks every column present. `mutate` also accepts
 `op: 'patch'` with the primary key and supplied columns. A batch can mix sparse
-patches, upserts and deletes atomically. Omitted encrypted columns require no
+patches, upserts and deletes atomically. Every sparse patch requires a row in
+the local replica at call time. An absent row rejects the entire batch with
+`sync.row_missing` before any local write or outbox insert. Omitted encrypted columns require no
 key and retain their ciphertext.
 
 Set `retainFailedCommits: true` to keep the complete failed aggregate visible
@@ -383,7 +385,9 @@ and durable after its outbox drains. Incoming server state advances a separate
 base. `outcome.retainedRows` exposes complete intended rows and the latest
 server rows and versions. Resolve explicitly with `resolved_keep_server`, or
 create a reviewed replacement and link it through `superseded`. Security purge
-and scope revocation remove whole retained aggregates.
+and scope revocation remove whole retained aggregates. If a sparse conflict
+loses its server base, the client keeps the sparse operation and conflict
+evidence without materializing a local row from the saved before-image.
 
 ## Application-authorized local security purge
 

package/dist/client.js

@@ -1827,6 +1827,10 @@ export class SyncClient {
                 const normalized = normalizeRecordKeys(table, mutation.values);
                 const scopeColumns = new Set(table.scopeColumnByVariable.values());
                 const pkColumn = table.columns[table.primaryKeyIndex];
+                const pkValue = requireRowId(normalized.get(pkColumn.name));
+                const local = this.#db.query(`SELECT * FROM ${quoteIdent(table.name)} WHERE ${quoteIdent(table.primaryKey)} = ?`, [pkValue])[0];
+                if (!local)
+                    throw new ClientSyncError('sync.row_missing', 'a sparse patch requires a local row');
                 for (const column of table.columns) {
                     if (!normalized.has(column.name))
                         continue;
@@ -1841,12 +1845,7 @@ export class SyncClient {
                     // by construction the row id being patched (§6.1), so its value is
                     // proven equal already.
                     if (scopeColumns.has(column.name) && column.name !== pkColumn.name) {
-                        const pkValue = normalized.get(pkColumn.name);
-                        const local = typeof pkValue === 'string' && pkValue.length > 0
-                            ? this.#db.query(`SELECT * FROM ${quoteIdent(table.name)} WHERE ${quoteIdent(table.primaryKey)} = ?`, [pkValue])[0]
-                            : undefined;
-                        if (local === undefined ||
-                            !Object.is(rowValueToJson(fromSqlValue(column, local[column.name] ?? null)), rowValueToJson(coerceSqlRepresentation(column, normalized.get(column.name))))) {
+                        if (!Object.is(rowValueToJson(fromSqlValue(column, local[column.name] ?? null)), rowValueToJson(coerceSqlRepresentation(column, normalized.get(column.name))))) {
                             throw invalidRequest(`table ${table.name}: patch cannot write scope column ${JSON.stringify(column.name)} (§3.4)`);
                         }
                         continue;
@@ -1868,10 +1867,7 @@ export class SyncClient {
                     json[column.name] = rowValueToJson(values[index] ?? null);
                 });
             }
-            const pkValue = json[table.primaryKey];
-            if (typeof pkValue !== 'string' || pkValue.length === 0) {
-                throw invalidRequest(`table ${table.name}: upsert requires a non-empty string primary key`);
-            }
+            const pkValue = requireRowId(json[table.primaryKey]);
             return {
                 table: mutation.table,
                 rowId: pkValue,
@@ -3826,3 +3822,8 @@ export class SyncClient {
         }
     }
 }
+function requireRowId(value) {
+    if (typeof value !== 'string' || !value.length)
+        throw invalidRequest('a mutation requires a non-empty string primary key');
+    return value;
+}

package/dist/failed-overlay.js

@@ -58,6 +58,11 @@ export function restoreFailedBases(db, schema, absentIntent = false) {
             deleteLocalRow(db, table, String(row.id), false);
         else {
             const values = JSON.parse(String(json));
+            if (absentIntent) {
+                const operation = JSON.parse(String(row.op));
+                if (!table.columns.every((column) => Object.hasOwn(operation.values ?? {}, column.name)))
+                    continue;
+            }
             upsertLocalRow(db, table, table.columns.map((column) => jsonToRowValue(values[column.name] ?? null)), Number(row.version ?? -1), false);
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@syncular/client",
-  "version": "0.30.9",
+  "version": "0.30.10",
   "description": "Syncular TypeScript client core — offline-first sync over SQLite (WASM/OPFS, Bun, Node)",
   "license": "Apache-2.0",
   "author": "Benjamin Kniffler",
@@ -89,10 +89,11 @@
   },
   "dependencies": {
     "@sqlite.org/sqlite-wasm": "^3.53.0-build1",
-    "@syncular/core": "0.30.9"
+    "@syncular/core": "0.30.10"
   },
   "devDependencies": {
-    "@syncular/server": "0.30.9",
-    "playwright": "1.63.0"
+    "@syncular/server": "0.30.10",
+    "playwright": "1.63.0",
+    "@electric-sql/pglite": "^0.5.4"
   }
 }

package/src/client.ts

@@ -2750,6 +2750,16 @@ export class SyncClient {
         const normalized = normalizeRecordKeys(table, mutation.values);
         const scopeColumns = new Set(table.scopeColumnByVariable.values());
         const pkColumn = table.columns[table.primaryKeyIndex] as RowColumn;
+        const pkValue = requireRowId(normalized.get(pkColumn.name));
+        const local = this.#db.query(
+          `SELECT * FROM ${quoteIdent(table.name)} WHERE ${quoteIdent(table.primaryKey)} = ?`,
+          [pkValue],
+        )[0];
+        if (!local)
+          throw new ClientSyncError(
+            'sync.row_missing',
+            'a sparse patch requires a local row',
+          );
         for (const column of table.columns) {
           if (!normalized.has(column.name)) continue;
           // §3.4 rule 5 / §6.2: scope columns are immutable on update. The
@@ -2763,16 +2773,7 @@ export class SyncClient {
           // by construction the row id being patched (§6.1), so its value is
           // proven equal already.
           if (scopeColumns.has(column.name) && column.name !== pkColumn.name) {
-            const pkValue = normalized.get(pkColumn.name);
-            const local =
-              typeof pkValue === 'string' && pkValue.length > 0
-                ? this.#db.query(
-                    `SELECT * FROM ${quoteIdent(table.name)} WHERE ${quoteIdent(table.primaryKey)} = ?`,
-                    [pkValue],
-                  )[0]
-                : undefined;
             if (
-              local === undefined ||
               !Object.is(
                 rowValueToJson(
                   fromSqlValue(column, local[column.name] ?? null),
@@ -2811,12 +2812,7 @@ export class SyncClient {
           json[column.name] = rowValueToJson(values[index] ?? null);
         });
       }
-      const pkValue = json[table.primaryKey];
-      if (typeof pkValue !== 'string' || pkValue.length === 0) {
-        throw invalidRequest(
-          `table ${table.name}: upsert requires a non-empty string primary key`,
-        );
-      }
+      const pkValue = requireRowId(json[table.primaryKey]);
       return {
         table: mutation.table,
         rowId: pkValue,
@@ -5239,3 +5235,9 @@ export class SyncClient {
     }
   }
 }
+
+function requireRowId(value: unknown): string {
+  if (typeof value !== 'string' || !value.length)
+    throw invalidRequest('a mutation requires a non-empty string primary key');
+  return value;
+}

package/src/failed-overlay.ts

@@ -101,6 +101,15 @@ export function restoreFailedBases(
     if (json === null) deleteLocalRow(db, table, String(row.id), false);
     else {
       const values: Record<string, JsonRowValue> = JSON.parse(String(json));
+      if (absentIntent) {
+        const operation: OutboxOperation = JSON.parse(String(row.op));
+        if (
+          !table.columns.every((column) =>
+            Object.hasOwn(operation.values ?? {}, column.name),
+          )
+        )
+          continue;
+      }
       upsertLocalRow(
         db,
         table,