@statezero/core 0.1.86 → 0.1.88

package/dist/flavours/django/tempPk.js

@@ -22,8 +22,9 @@ export function createTempPk(uuid) {
  * Register a real PK for a temporary PK
  */
 export function setRealPk(uuid, realPk) {
-    const key = `TempPK_${uuid}`;
-    tempPkMap.set(key, realPk);
+    const key = `"TempPK_${uuid}"`;
+    const value = typeof realPk === 'string' ? `"${realPk}"` : String(realPk);
+    tempPkMap.set(key, value);
 }
 /**
  * Replace all temporary PKs in an object (or string) with their real values

package/dist/syncEngine/stores/operationEventHandlers.js

@@ -37,6 +37,7 @@ function relatedQuerysets(queryset) {
 }
 /**
  * Process an operation in the model store
+ * Only processes if the model store doesn't already have the operation.
  *
  * @param {Operation} operation - The operation to process
  * @param {string} actionType - The action to perform ('add', 'update', 'confirm', 'reject')
@@ -46,6 +47,10 @@ function processModelStore(operation, actionType) {
     const modelStore = modelStoreRegistry.getStore(ModelClass);
     if (!modelStore)
         return;
+    // Skip if the model store already has this operation
+    if (modelStore.operationsMap.has(operation.operationId)) {
+        return;
+    }
     switch (actionType) {
         case 'add':
             modelStore.addOperation(operation);
@@ -113,7 +118,12 @@ function processQuerysetStores(operation, actionType) {
             querysetStoreMap = relatedQuerysets(queryset);
             break;
     }
-    Array.from(querysetStoreMap.values()).forEach(applyAction);
+    // Filter to only stores that DON'T already have this operation
+    const storesToRoute = Array.from(querysetStoreMap.values()).filter(store => {
+        return !store.operationsMap.has(operation.operationId);
+    });
+    // Route only to stores that don't have the operation yet
+    storesToRoute.forEach(applyAction);
 }
 /**
  * Process an operation in the metric stores based on operation type

package/dist/syncEngine/sync.js

@@ -16,16 +16,16 @@ export class EventPayload {
         this.operation_id = data.operation_id;
         this.pk_field_name = data.pk_field_name;
         this.configKey = data.configKey;
-        const pkType = getModelClass(data.model, data.configKey)?.schema
-            ?.properties?.[data.pk_field_name]?.type;
-        this.instances =
-            data.instances?.map((inst) => {
-                const pk = inst?.[data.pk_field_name];
-                if (pk != null && pkType === "integer" && typeof pk === "string") {
-                    inst[data.pk_field_name] = parseInt(pk, 10);
-                }
-                return inst;
-            }) || [];
+        // Parse PK fields to numbers in instances
+        this.instances = data.instances?.map(instance => {
+            if (instance && this.pk_field_name && instance[this.pk_field_name] != null) {
+                return {
+                    ...instance,
+                    [this.pk_field_name]: Number(instance[this.pk_field_name])
+                };
+            }
+            return instance;
+        }) || data.instances;
         this._cachedInstances = null;
     }
     get modelClass() {
@@ -55,7 +55,19 @@ export class SyncManager {
                 this.processMetrics(payload);
             }
             if (isLocalOperation) {
-                return;
+                // Check if any stores for this model class DON'T have the operation yet
+                const modelStore = modelStoreRegistry.getStore(payload.modelClass);
+                const querysetStores = querysetStoreRegistry.getAllStoresForModel(payload.modelClass);
+                // If model store and all queryset stores already have the operation, skip
+                // Note: getStore() always returns a store (creates if needed)
+                // and operationsMap is always initialized in constructors
+                const modelStoreHasIt = modelStore.operationsMap.has(payload.operation_id);
+                const allQuerysetStoresHaveIt = querysetStores.length === 0 ||
+                    querysetStores.every(store => store.operationsMap.has(payload.operation_id));
+                if (modelStoreHasIt && allQuerysetStoresHaveIt) {
+                    return; // All stores already have it, no need to process
+                }
+                // Otherwise, continue to process for stores that don't have it yet
             }
             // Add to batch for queryset/model processing
             const key = `${event.model}::${event.configKey}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@statezero/core",
-  "version": "0.1.86",
+  "version": "0.1.88",
   "type": "module",
   "module": "ESNext",
   "description": "The type-safe frontend client for StateZero - connect directly to your backend models with zero boilerplate",