@workglow/indexeddb 0.3.38 → 0.3.40

package/dist/storage/node.js

@@ -311,11 +311,7 @@ async function dropIndexedDbTable(tableName) {
   return deleteIndexedDbTable(tableName);
 }
 // src/storage/IndexedDbKvStorage.ts
-import {
-  DefaultKeyValueKey,
-  DefaultKeyValueSchema,
-  KvViaTabularStorage
-} from "@workglow/storage";
+import { DefaultKeyValueKey, DefaultKeyValueSchema, KvViaTabularStorage } from "@workglow/storage";
 import { createServiceToken as createServiceToken2 } from "@workglow/util";
 
 // src/storage/IndexedDbTabularStorage.ts
@@ -324,9 +320,12 @@ import {
   HybridSubscriptionManager,
   isSearchCondition,
   isSearchInCondition,
+  matchesEqualityCriterion,
+  matchesInequalityCriterion,
   normalizeCriterion,
   pickCoveringIndex,
-  safeEmit
+  safeEmit,
+  StorageUnsupportedError
 } from "@workglow/storage";
 import { createServiceToken, deepEqual as deepEqual2, makeFingerprint, uuid4 } from "@workglow/util";
 
@@ -904,7 +903,7 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
             recordToStore = { ...recordToStore, [keyName]: request.result };
           }
         }
-        this.events.emit("put", recordToStore);
+        safeEmit(this.events, "put", recordToStore);
         resolve(recordToStore);
       };
       transaction.oncomplete = () => {
@@ -1003,11 +1002,11 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
       request.onerror = () => reject(request.error);
       request.onsuccess = () => {
         if (!request.result) {
-          this.events.emit("get", key, undefined);
+          safeEmit(this.events, "get", key, undefined);
           resolve(undefined);
           return;
         }
-        this.events.emit("get", key, request.result);
+        safeEmit(this.events, "get", key, request.result);
         resolve(request.result);
       };
     });
@@ -1063,7 +1062,7 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
       const request = store.delete(this.getIndexedKey(key));
       request.onerror = () => reject(request.error);
       request.onsuccess = () => {
-        this.events.emit("delete", key);
+        safeEmit(this.events, "delete", key);
         resolve();
       };
       transaction.oncomplete = () => {
@@ -1079,7 +1078,7 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
       const request = store.clear();
       request.onerror = () => reject(request.error);
       request.onsuccess = () => {
-        this.events.emit("clearall");
+        safeEmit(this.events, "clearall");
         resolve();
       };
       transaction.oncomplete = () => {
@@ -1101,7 +1100,8 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
     if (this.cursorSafeIndexes)
       return this.cursorSafeIndexes;
     const required = new Set(this.schema.required ?? []);
-    this.cursorSafeIndexes = [...this.indexes, ...this.uniqueIndexes].filter((columns) => columns.every((column) => required.has(String(column))));
+    const properties = this.schema.properties;
+    this.cursorSafeIndexes = [...this.indexes, ...this.uniqueIndexes].filter((columns) => columns.every((column) => required.has(String(column)) && !schemaAdmitsNull(properties[String(column)])));
     return this.cursorSafeIndexes;
   }
   createIndexedRange(store, criteria) {
@@ -1220,12 +1220,16 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
         continue;
       }
       const { operator, value } = normalized;
-      if (operator !== "=" && (recordValue === null || recordValue === undefined)) {
+      if (operator !== "=" && operator !== "!=" && (recordValue === null || recordValue === undefined)) {
         return false;
       }
       switch (operator) {
         case "=":
-          if (recordValue !== value)
+          if (!matchesEqualityCriterion(recordValue, value))
+            return false;
+          break;
+        case "!=":
+          if (!matchesInequalityCriterion(recordValue, value))
             return false;
           break;
         case "<":
@@ -1261,7 +1265,7 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
         const transaction = db.transaction(this.table, "readwrite");
         const store = transaction.objectStore(this.table);
         transaction.oncomplete = () => {
-          this.events.emit("delete", this.deleteIdentity(criteria));
+          safeEmit(this.events, "delete", this.deleteIdentity(criteria));
           this.hybridManager?.notifyLocalChange();
           resolve();
         };
@@ -1347,8 +1351,14 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
     if (isSearchInCondition(criterion))
       return;
     if (isSearchCondition(criterion)) {
-      return criterion.operator === "=" ? criterion.value : undefined;
+      if (criterion.operator !== "=")
+        return;
+      if (criterion.value === null)
+        return;
+      return criterion.value;
     }
+    if (criterion === null)
+      return;
     return criterion;
   }
   compareByOrder(a, b, options) {
@@ -1450,7 +1460,7 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
             finalResults = finalResults.slice(0, options.limit);
           }
           const result = finalResults.length > 0 ? finalResults : undefined;
-          this.events.emit("query", criteria, result);
+          safeEmit(this.events, "query", criteria, result);
           resolve(result);
           return;
         }
@@ -1462,7 +1472,7 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
             results.push(record);
             if (indexedQuery.appliedLimit && results.length === options?.limit) {
               const result = results.length > 0 ? results : undefined;
-              this.events.emit("query", criteria, result);
+              safeEmit(this.events, "query", criteria, result);
               resolve(result);
               return;
             }
@@ -1491,6 +1501,13 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
       selectColumns: options.select.map(String),
       primaryKeyColumns: this.primaryKeyColumns().map(String)
     });
+    for (const col of picked.keyPath) {
+      const criterion = criteria[col];
+      const isNullEquality = isSearchCondition(criterion) ? criterion.operator === "=" && criterion.value === null : criterion === null;
+      if (isNullEquality) {
+        throw new StorageUnsupportedError(`A null equality criterion on indexed column "${col}" (IndexedDB omits ` + `null-valued records from an index, so an index scan can never reach ` + `them — use query() instead)`, "IndexedDbTabularStorage");
+      }
+    }
     const db = await this.getDb();
     return new Promise((resolve, reject) => {
       const tx = db.transaction(this.table, "readonly");
@@ -1506,8 +1523,12 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
         if (isSearchCondition(c)) {
           if (c.operator !== "=")
             break;
+          if (c.value === null)
+            break;
           prefix.push(c.value);
         } else {
+          if (c === null)
+            break;
           prefix.push(c);
         }
       }
@@ -1613,12 +1634,29 @@ class IndexedDbTabularStorage extends BaseTabularStorage {
     this.db?.close();
   }
 }
+function schemaAdmitsNull(typeDef) {
+  if (typeDef === undefined)
+    return false;
+  if (typeof typeDef === "boolean")
+    return typeDef;
+  if (typeDef.type === "null")
+    return true;
+  if (Array.isArray(typeDef.type))
+    return typeDef.type.includes("null");
+  const branches = typeDef.anyOf ?? typeDef.oneOf;
+  if (Array.isArray(branches)) {
+    return branches.some((branch) => typeof branch !== "boolean" && branch.type === "null");
+  }
+  return false;
+}
 function compareWithOperator(a, op, b) {
   const av = a;
   const bv = b;
   switch (op) {
     case "=":
-      return av === bv;
+      return matchesEqualityCriterion(av, bv);
+    case "!=":
+      return matchesInequalityCriterion(av, bv);
     case "<":
       return av !== null && av !== undefined && av < bv;
     case "<=":
@@ -1645,10 +1683,10 @@ class IndexedDbKvStorage extends KvViaTabularStorage {
 // src/storage/IndexedDbVectorStorage.ts
 import {
   assertVectorShape,
+  emitSimilaritySearch,
   getMetadataProperty,
   getVectorProperty,
   matchesFilter,
-  safeEmit as safeEmit2,
   validateVectorEntities
 } from "@workglow/storage";
 import { createServiceToken as createServiceToken3 } from "@workglow/util";
@@ -1702,9 +1740,7 @@ class IndexedDbVectorStorage extends IndexedDbTabularStorage {
       });
     }
     results.sort((a, b) => b.score - a.score);
-    const topResults = results.slice(0, topK);
-    safeEmit2(this.events, "similaritySearch", query, topResults);
-    return topResults;
+    return emitSimilaritySearch(this.events, query, results.slice(0, topK));
   }
 }
 export {
@@ -1724,4 +1760,4 @@ export {
   IDB_KV_REPOSITORY
 };
 
-//# debugId=DA162F737BD6A01964756E2164756E21
+//# debugId=CD6EC5A9A562618C64756E2164756E21