@orion-js/mongodb 4.2.1 → 4.2.3

package/dist/index.d.cts

@@ -317,11 +317,14 @@ interface DeleteUnusedIndexesResult {
 /**
  * Registry that tracks all collections created via createCollection().
  * Maps connection name to a map of collection name to collection instance.
+ * When the same collection is registered multiple times, indexes are merged.
  */
 declare const collectionsRegistry: Map<string, Map<string, Collection<ModelClassBase>>>;
 /**
  * Registers a collection in the registry.
  * Called automatically when a collection is created via createCollection().
+ * If the same collection name is registered multiple times, indexes are merged
+ * to support multiple createCollection() calls for the same collection.
  * @param connectionName - The name of the MongoDB connection
  * @param collection - The collection instance to register
  */
@@ -337,6 +340,10 @@ declare function getRegisteredCollections(connectionName?: string): Array<Collec
  * Iterates over all collections registered via createCollection() and
  * removes indexes that exist in MongoDB but are not defined in the collection configuration.
  *
+ * **Important**: This function waits for all pending index creation to complete before
+ * deleting any indexes, to avoid race conditions where an index being created might
+ * be incorrectly identified as unused.
+ *
  * @param connectionName - The name of the MongoDB connection (defaults to 'main')
  * @returns Array of results for each collection that had indexes deleted
  *

package/dist/index.d.ts

@@ -317,11 +317,14 @@ interface DeleteUnusedIndexesResult {
 /**
  * Registry that tracks all collections created via createCollection().
  * Maps connection name to a map of collection name to collection instance.
+ * When the same collection is registered multiple times, indexes are merged.
  */
 declare const collectionsRegistry: Map<string, Map<string, Collection<ModelClassBase>>>;
 /**
  * Registers a collection in the registry.
  * Called automatically when a collection is created via createCollection().
+ * If the same collection name is registered multiple times, indexes are merged
+ * to support multiple createCollection() calls for the same collection.
  * @param connectionName - The name of the MongoDB connection
  * @param collection - The collection instance to register
  */
@@ -337,6 +340,10 @@ declare function getRegisteredCollections(connectionName?: string): Array<Collec
  * Iterates over all collections registered via createCollection() and
  * removes indexes that exist in MongoDB but are not defined in the collection configuration.
  *
+ * **Important**: This function waits for all pending index creation to complete before
+ * deleting any indexes, to avoid race conditions where an index being created might
+ * be incorrectly identified as unused.
+ *
  * @param connectionName - The name of the MongoDB connection (defaults to 'main')
  * @returns Array of results for each collection that had indexes deleted
  *

package/dist/index.js

@@ -1283,11 +1283,38 @@ async function loadIndexes(collection) {
 // src/createCollection/collectionsRegistry.ts
 import { logger as logger4 } from "@orion-js/logger";
 var collectionsRegistry = /* @__PURE__ */ new Map();
+function indexesAreEqual(indexA, indexB) {
+  const nameA = getIndexName(indexA);
+  const nameB = getIndexName(indexB);
+  if (nameA && nameB) {
+    return nameA === nameB;
+  }
+  return keysMatch(
+    indexA.keys,
+    indexB.keys
+  );
+}
+function mergeIndexes(existingIndexes, newIndexes) {
+  const merged = [...existingIndexes];
+  for (const newIndex of newIndexes) {
+    const isDuplicate = existingIndexes.some((existing) => indexesAreEqual(existing, newIndex));
+    if (!isDuplicate) {
+      merged.push(newIndex);
+    }
+  }
+  return merged;
+}
 function registerCollection(connectionName, collection) {
   if (!collectionsRegistry.has(connectionName)) {
     collectionsRegistry.set(connectionName, /* @__PURE__ */ new Map());
   }
-  collectionsRegistry.get(connectionName).set(collection.name, collection);
+  const connectionMap = collectionsRegistry.get(connectionName);
+  const existingCollection = connectionMap.get(collection.name);
+  if (existingCollection) {
+    existingCollection.indexes = mergeIndexes(existingCollection.indexes, collection.indexes);
+  } else {
+    connectionMap.set(collection.name, collection);
+  }
 }
 function getRegisteredCollections(connectionName = "main") {
   const connectionCollections = collectionsRegistry.get(connectionName);
@@ -1302,6 +1329,10 @@ async function deleteAllUnusedIndexes(connectionName = "main") {
     logger4.warn(`No collections registered for connection "${connectionName}"`);
     return [];
   }
+  if (createIndexesPromises.length > 0) {
+    logger4.info("Waiting for pending index creation to complete before deleting unused indexes...");
+    await Promise.all(createIndexesPromises);
+  }
   logger4.info(
     `Deleting unused indexes from ${collections.length} collections on connection "${connectionName}"`
   );