@statezero/core 0.1.62 → 0.1.64

package/dist/flavours/django/errors.js

@@ -23,7 +23,7 @@ export class StateZeroError extends Error {
      */
     constructor(message, code, detail, status) {
         super(message);
-        this.name = this.constructor.name;
+        this.name = "StateZeroError";
         this.code = code;
         this.detail = detail;
         this.status = status;
@@ -66,6 +66,7 @@ export class ValidationError extends StateZeroError {
      */
     constructor(detail, status = 400) {
         super("Validation error", "validation_error", detail, status);
+        this.name = "ValidationError";
     }
 }
 /**
@@ -80,6 +81,7 @@ export class DoesNotExist extends StateZeroError {
      */
     constructor(detail = "Does not exist", status = 404) {
         super("DoesNotExist", "does_not_exist", detail, status);
+        this.name = "DoesNotExist";
     }
 }
 /**
@@ -94,6 +96,7 @@ export class PermissionDenied extends StateZeroError {
      */
     constructor(detail = "Permission denied", status = 403) {
         super("Permission denied", "permission_denied", detail, status);
+        this.name = "PermissionDenied";
     }
 }
 /**
@@ -108,6 +111,7 @@ export class MultipleObjectsReturned extends StateZeroError {
      */
     constructor(detail = "Multiple objects returned", status = 500) {
         super("Multiple objects returned", "multiple_objects_returned", detail, status);
+        this.name = "MultipleObjectsReturned";
     }
 }
 /**
@@ -122,6 +126,7 @@ export class ASTValidationError extends StateZeroError {
      */
     constructor(detail, status = 400) {
         super("Query syntax error", "ast_validation_error", detail, status);
+        this.name = "ASTValidationError";
     }
 }
 /**
@@ -136,6 +141,7 @@ export class ConfigError extends StateZeroError {
      */
     constructor(detail, status = 500) {
         super("Configuration error", "config_error", detail, status);
+        this.name = "ConfigError";
     }
 }
 /**

package/dist/flavours/django/makeApiCall.js

@@ -110,19 +110,6 @@ export async function makeApiCall(querySet, operationType, args = {}, operationI
             throw new Error(`API call failed: ${error.message}`);
         }
     };
-    /*
-      We use p-queue for write operations to ensure that the user does not
-      try and perform a write operation using a temporary primary key that
-      exists on the local optimistic instance until the backend has replied with the
-      new primary key. This would result in a 400 Bad Request error from the backend.
-      This is effective, but does mean that write operations are queued which can lead to delays.
-      In general, this is not a problem because the UI will optimistically update the state so the
-      user does not perceive the delay, but it is something to be aware of.
-  
-      In the future, we will want to implement an approach where we specifically detect
-      if a write operation is using a temporary pk and then await for the temporary
-      pk to be replaced before proceeding with the write operation. But for now,
-      this is a simple and effective solution.
-    */
+    // Queue write operations, execute read operations immediately
     return isWriteOperation ? apiCallQueue.add(apiCall) : apiCall();
 }

package/dist/syncEngine/stores/operationEventHandlers.js

@@ -35,38 +35,6 @@ function relatedQuerysets(queryset) {
     });
     return result;
 }
-/**
- * Returns querysets that contain any of the specified instances
- * @param {Operation} operation - The operation containing instances to check
- * @returns {Map<QuerySet, Store>}
- */
-function querysetsContainingInstances(operation) {
-    const ModelClass = operation.queryset.ModelClass;
-    const pkField = ModelClass.primaryKeyField;
-    const instancePks = new Set(operation.instances
-        .filter(instance => instance && typeof instance === 'object' && pkField in instance)
-        .map(instance => instance[pkField]));
-    if (instancePks.size === 0) {
-        return new Map();
-    }
-    const result = new Map();
-    Array.from(querysetStoreRegistry._stores.entries()).forEach(([queryset, store]) => {
-        if (store.modelClass !== ModelClass)
-            return;
-        try {
-            // Check if this queryset contains any of the instances
-            const renderedPks = new Set(store.render(false)); // Get without optimistic updates
-            const hasIntersection = [...instancePks].some(pk => renderedPks.has(pk));
-            if (hasIntersection) {
-                result.set(store.queryset, store);
-            }
-        }
-        catch (e) {
-            console.warn('Error checking queryset for instances', e);
-        }
-    });
-    return result;
-}
 /**
  * Process an operation in the model store
  *
@@ -156,14 +124,9 @@ function processQuerysetStores(operation, actionType) {
 function processMetricStores(operation, actionType) {
     const queryset = operation.queryset;
     const ModelClass = queryset.ModelClass;
-    // For metrics, we use related querysets as the base
+    // For metrics, we can use a similar strategy but might be more conservative
+    // and always use related querysets since metrics are aggregations
     const allQuerysets = Array.from(relatedQuerysets(queryset).keys());
-    // For update/delete operations, also include querysets that contain the instances
-    if (operation.type === Type.UPDATE || operation.type === Type.UPDATE_INSTANCE ||
-        operation.type === Type.DELETE || operation.type === Type.DELETE_INSTANCE) {
-        const containingQuerysets = Array.from(querysetsContainingInstances(operation).keys());
-        allQuerysets.push(...containingQuerysets);
-    }
     let allMetricStores = new Set();
     allQuerysets.forEach(qs => {
         const stores = metricRegistry.getAllStoresForQueryset(qs);

package/dist/syncEngine/sync.js

@@ -2,11 +2,12 @@ import { getAllEventReceivers } from "../core/eventReceivers";
 import { operationRegistry } from "./stores/operation";
 import { initializeAllEventReceivers } from "../config";
 import { getEventReceiver } from "../core/eventReceivers";
-import { querysetStoreRegistry } from "./registries/querysetStoreRegistry";
-import { modelStoreRegistry } from "./registries/modelStoreRegistry";
-import { metricRegistry } from "./registries/metricRegistry";
+import { querysetStoreRegistry, QuerysetStoreRegistry } from "./registries/querysetStoreRegistry";
+import { modelStoreRegistry, ModelStoreRegistry } from "./registries/modelStoreRegistry";
+import { metricRegistry, MetricRegistry } from "./registries/metricRegistry";
 import { getModelClass, getConfig } from "../config";
 import { isNil } from "lodash-es";
+import { QuerysetStore } from "./stores/querysetStore";
 export class EventPayload {
     constructor(data) {
         this.event = data.event;
@@ -38,17 +39,17 @@ export class SyncManager {
         this.handleEvent = (event) => {
             let payload = new EventPayload(event);
             let isLocalOperation = operationRegistry.has(payload.operation_id);
-            if (this.registries.has("MetricRegistry")) {
+            if (this.registries.has(MetricRegistry)) {
                 this.processMetrics(payload);
             }
             if (isLocalOperation) {
                 // This is a local operation, so don't resync querysets or models
                 return;
             }
-            if (this.registries.has("QuerysetStoreRegistry")) {
+            if (this.registries.has(QuerysetStoreRegistry)) {
                 this.processQuerysets(payload);
             }
-            if (this.registries.has("ModelStoreRegistry")) {
+            if (this.registries.has(ModelStoreRegistry)) {
                 this.processModels(payload);
             }
         };
@@ -102,7 +103,7 @@ export class SyncManager {
     syncStaleQuerysets() {
         let syncedCount = 0;
         // Sync all followed querysets - keep it simple
-        const querysetRegistry = this.registries.get("QuerysetStoreRegistry");
+        const querysetRegistry = this.registries.get(QuerysetStoreRegistry);
         if (querysetRegistry) {
             for (const [semanticKey, store] of querysetRegistry._stores.entries()) {
                 // Only sync if this store is actually being followed
@@ -153,11 +154,11 @@ export class SyncManager {
         }
     }
     manageRegistry(registry) {
-        this.registries.set(registry.constructor.name, registry);
+        this.registries.set(registry.constructor, registry);
         registry.setSyncManager(this);
     }
     removeRegistry(registry) {
-        this.registries.delete(registry.constructor.name);
+        this.registries.delete(registry.constructor);
     }
     isQuerysetFollowed(queryset) {
         const activeSemanticKeys = new Set([...this.followedQuerysets].map((qs) => qs.semanticKey));
@@ -171,7 +172,7 @@ export class SyncManager {
         return false;
     }
     processQuerysets(event) {
-        const registry = this.registries.get("QuerysetStoreRegistry");
+        const registry = this.registries.get(QuerysetStoreRegistry);
         for (const [semanticKey, store] of registry._stores.entries()) {
             if (store.modelClass.modelName === event.model &&
                 store.modelClass.configKey === event.configKey) {
@@ -195,7 +196,7 @@ export class SyncManager {
         }
     }
     processMetrics(event) {
-        const registry = this.registries.get("MetricRegistry");
+        const registry = this.registries.get(MetricRegistry);
         for (const [key, entry] of registry._stores.entries()) {
             // Check if the store has a queryset with the model class AND correct backend
             if (entry.queryset &&

package/package.json

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