@statezero/core 0.2.39 → 0.2.40

package/dist/syncEngine/stores/operationEventHandlers.js

@@ -6,7 +6,7 @@ import { getFingerprint } from './utils.js';
 import { QuerySet } from '../../flavours/django/querySet.js';
 import { isEqual, isNil } from 'lodash-es';
 import hash from 'object-hash';
-import { filter } from '../../filtering/localFiltering.js';
+import { filter, applyOrderBy } from '../../filtering/localFiltering.js';
 /**
  * Evaluates and routes a CREATE operation to querysets, tracking membership state.
  *
@@ -34,7 +34,7 @@ function routeCreateOperation(operation, applyAction) {
         }
         // Evaluate if instances match this queryset's filter
         const ast = store.queryset.build();
-        const matchingInstances = filter(instances, ast, modelClass, false);
+        const matchingInstances = filter(instances, ast, modelClass, true);
         if (matchingInstances.length === 0) {
             // No instances match - mark DEFINITELY_NO (no need to sync this queryset)
             operationRegistry.setQuerysetState(operation.operationId, semanticKey, OperationMembership.DEFINITELY_NO);
@@ -49,8 +49,34 @@ function routeCreateOperation(operation, applyAction) {
                 const hasExplicitOrdering = store.queryset._orderBy && store.queryset._orderBy.length > 0;
                 const hasImplicitOrdering = (modelClass.schema?.default_ordering?.length || 0) > 0;
                 if (hasExplicitOrdering || hasImplicitOrdering) {
-                    // With ordering, new item could displace existing items - can't know without server
-                    operationRegistry.setQuerysetState(operation.operationId, semanticKey, OperationMembership.MAYBE);
+                    // Use local sorting to determine if new items would be in top N
+                    const orderBy = hasExplicitOrdering
+                        ? store.queryset._orderBy
+                        : modelClass.schema.default_ordering;
+                    // Get current items from model store
+                    const modelStore = modelStoreRegistry.getStore(modelClass);
+                    const renderedItems = modelStore.render(store.groundTruthPks, true, false) || [];
+                    const currentItems = renderedItems.filter(Boolean);
+                    // If some ground truth items couldn't be rendered, we can't trust local sort
+                    if (currentItems.length < store.groundTruthPks.length) {
+                        operationRegistry.setQuerysetState(operation.operationId, semanticKey, OperationMembership.MAYBE);
+                        return;
+                    }
+                    const allItems = [...currentItems, ...matchingInstances];
+                    // Sort and take top N
+                    const sorted = applyOrderBy(allItems, orderBy, modelClass);
+                    const topN = sorted.slice(0, limit);
+                    const topNPks = new Set(topN.map(item => item[modelClass.primaryKeyField]));
+                    // Check if any new items made it into the top N
+                    const newItemPks = matchingInstances.map(item => item[modelClass.primaryKeyField]);
+                    const anyInTopN = newItemPks.some(pk => topNPks.has(pk));
+                    if (anyInTopN) {
+                        applyAction(store);
+                        operationRegistry.setQuerysetState(operation.operationId, semanticKey, OperationMembership.DEFINITELY_YES);
+                    }
+                    else {
+                        operationRegistry.setQuerysetState(operation.operationId, semanticKey, OperationMembership.DEFINITELY_NO);
+                    }
                 }
                 else {
                     // No ordering - new items go at end, won't be in first N

package/dist/syncEngine/sync.d.ts

@@ -36,6 +36,11 @@ export class SyncManager {
     unfollowModel(registry: any, modelClass: any): void;
     manageRegistry(registry: any): void;
     removeRegistry(registry: any): void;
+    /**
+     * Sync querysets that were marked MAYBE for a local operation.
+     * Called when a backend event arrives for an operation we initiated locally.
+     */
+    syncMaybeQuerysets(operationId: any): void;
     handleEvent: (event: any) => void;
     processBatch(reason?: string): void;
     isQuerysetFollowed(queryset: any): boolean;

package/dist/syncEngine/sync.js

@@ -1,5 +1,5 @@
 import { getAllEventReceivers } from "../core/eventReceivers.js";
-import { operationRegistry } from "./stores/operation.js";
+import { operationRegistry, OperationMembership } from "./stores/operation.js";
 import { initializeAllEventReceivers } from "../config.js";
 import { getEventReceiver } from "../core/eventReceivers.js";
 import { querysetStoreRegistry, QuerysetStoreRegistry, } from "./registries/querysetStoreRegistry.js";
@@ -55,6 +55,9 @@ export class SyncManager {
                 this.processMetrics(payload);
             }
             if (isLocalOperation) {
+                // Check if any querysets were marked MAYBE for this operation
+                // These need to be synced since we couldn't determine membership client-side
+                this.syncMaybeQuerysets(payload.operation_id);
                 return;
             }
             // Add to batch for queryset/model processing
@@ -237,6 +240,33 @@ export class SyncManager {
     removeRegistry(registry) {
         this.registries.delete(registry.constructor);
     }
+    /**
+     * Sync querysets that were marked MAYBE for a local operation.
+     * Called when a backend event arrives for an operation we initiated locally.
+     */
+    syncMaybeQuerysets(operationId) {
+        const states = operationRegistry.getQuerysetStates(operationId);
+        if (!states)
+            return;
+        const registry = this.registries.get(QuerysetStoreRegistry);
+        if (!registry)
+            return;
+        const storesToSync = [];
+        for (const [semanticKey, membership] of states.entries()) {
+            if (membership === OperationMembership.MAYBE) {
+                const store = registry._stores.get(semanticKey);
+                if (store) {
+                    storesToSync.push(store);
+                }
+            }
+        }
+        if (storesToSync.length === 0)
+            return;
+        console.log(`[SyncManager] Syncing ${storesToSync.length} MAYBE querysets for local operation ${operationId}`);
+        const syncOperationId = `maybe-sync-${uuidv7()}`;
+        const dbSyncedKeys = new Set([...this.followedQuerysets].map(qs => qs.semanticKey));
+        Promise.all(storesToSync.map(store => registry.groupSync(store.queryset, syncOperationId, dbSyncedKeys)));
+    }
     processBatch(reason = "unknown") {
         if (this.eventBatch.size === 0)
             return;

package/package.json

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