@mastra/qdrant 0.2.13-alpha.0 → 0.2.13

package/.turbo/turbo-build.log

@@ -1,23 +1,23 @@
 
-> @mastra/qdrant@0.2.13-alpha.0 build /home/runner/work/mastra/mastra/stores/qdrant
+> @mastra/qdrant@0.2.13-alpha.1 build /home/runner/work/mastra/mastra/stores/qdrant
 > tsup src/index.ts --format esm,cjs --experimental-dts --clean --treeshake=smallest --splitting
 
 CLI Building entry: src/index.ts
 CLI Using tsconfig: tsconfig.json
 CLI tsup v8.4.0
 TSC Build start
-TSC ⚡️ Build success in 6421ms
+TSC ⚡️ Build success in 8886ms
 DTS Build start
 CLI Target: es2022
 Analysis will use the bundled TypeScript version 5.8.3
 Writing package typings: /home/runner/work/mastra/mastra/stores/qdrant/dist/_tsup-dts-rollup.d.ts
 Analysis will use the bundled TypeScript version 5.8.3
 Writing package typings: /home/runner/work/mastra/mastra/stores/qdrant/dist/_tsup-dts-rollup.d.cts
-DTS ⚡️ Build success in 8435ms
+DTS ⚡️ Build success in 9642ms
 CLI Cleaning output folder
 ESM Build start
 CJS Build start
-CJS dist/index.cjs 16.64 KB
-CJS ⚡️ Build success in 655ms
-ESM dist/index.js 16.58 KB
-ESM ⚡️ Build success in 656ms
+ESM dist/index.js 16.97 KB
+ESM ⚡️ Build success in 811ms
+CJS dist/index.cjs 17.03 KB
+CJS ⚡️ Build success in 820ms

package/CHANGELOG.md

@@ -1,5 +1,33 @@
 # @mastra/qdrant
 
+## 0.2.13
+
+### Patch Changes
+
+- 9cd1a46: [MASTRA-3338] update naming scheme for embedding index based on vector store rules and added duplicate index checks
+- Updated dependencies [e450778]
+- Updated dependencies [8902157]
+- Updated dependencies [ca0dc88]
+- Updated dependencies [526c570]
+- Updated dependencies [d7a6a33]
+- Updated dependencies [9cd1a46]
+- Updated dependencies [b5d2de0]
+- Updated dependencies [644f8ad]
+- Updated dependencies [70dbf51]
+  - @mastra/core@0.9.3
+
+## 0.2.13-alpha.1
+
+### Patch Changes
+
+- 9cd1a46: [MASTRA-3338] update naming scheme for embedding index based on vector store rules and added duplicate index checks
+- Updated dependencies [e450778]
+- Updated dependencies [8902157]
+- Updated dependencies [ca0dc88]
+- Updated dependencies [9cd1a46]
+- Updated dependencies [70dbf51]
+  - @mastra/core@0.9.3-alpha.1
+
 ## 0.2.13-alpha.0
 
 ### Patch Changes

package/dist/index.cjs

@@ -279,14 +279,23 @@ var QdrantVector = class extends vector.MastraVector {
     if (!Number.isInteger(dimension) || dimension <= 0) {
       throw new Error("Dimension must be a positive integer");
     }
-    await this.client.createCollection(indexName, {
-      vectors: {
-        // @ts-expect-error
-        size: dimension,
-        // @ts-expect-error
-        distance: DISTANCE_MAPPING[metric]
+    if (!DISTANCE_MAPPING[metric]) {
+      throw new Error(`Invalid metric: "${metric}". Must be one of: cosine, euclidean, dotproduct`);
+    }
+    try {
+      await this.client.createCollection(indexName, {
+        vectors: {
+          size: dimension,
+          distance: DISTANCE_MAPPING[metric]
+        }
+      });
+    } catch (error) {
+      const message = error?.message || error?.toString();
+      if (error?.status === 409 || typeof message === "string" && message.toLowerCase().includes("exists")) {
+        await this.validateExistingIndex(indexName, dimension, metric);
+        return;
       }
-    });
+    }
   }
   transformFilter(filter) {
     const translator = new QdrantFilterTranslator();

package/dist/index.js

@@ -277,14 +277,23 @@ var QdrantVector = class extends MastraVector {
     if (!Number.isInteger(dimension) || dimension <= 0) {
       throw new Error("Dimension must be a positive integer");
     }
-    await this.client.createCollection(indexName, {
-      vectors: {
-        // @ts-expect-error
-        size: dimension,
-        // @ts-expect-error
-        distance: DISTANCE_MAPPING[metric]
+    if (!DISTANCE_MAPPING[metric]) {
+      throw new Error(`Invalid metric: "${metric}". Must be one of: cosine, euclidean, dotproduct`);
+    }
+    try {
+      await this.client.createCollection(indexName, {
+        vectors: {
+          size: dimension,
+          distance: DISTANCE_MAPPING[metric]
+        }
+      });
+    } catch (error) {
+      const message = error?.message || error?.toString();
+      if (error?.status === 409 || typeof message === "string" && message.toLowerCase().includes("exists")) {
+        await this.validateExistingIndex(indexName, dimension, metric);
+        return;
       }
-    });
+    }
   }
   transformFilter(filter) {
     const translator = new QdrantFilterTranslator();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mastra/qdrant",
-  "version": "0.2.13-alpha.0",
+  "version": "0.2.13",
   "description": "Qdrant vector store provider for Mastra",
   "type": "module",
   "main": "dist/index.js",
@@ -21,7 +21,7 @@
   "license": "MIT",
   "dependencies": {
     "@qdrant/js-client-rest": "^1.13.0",
-    "@mastra/core": "^0.9.3-alpha.0"
+    "@mastra/core": "^0.9.3"
   },
   "devDependencies": {
     "@microsoft/api-extractor": "^7.52.5",
@@ -30,7 +30,7 @@
     "tsup": "^8.4.0",
     "typescript": "^5.8.2",
     "vitest": "^3.1.2",
-    "@internal/lint": "0.0.3"
+    "@internal/lint": "0.0.4"
   },
   "scripts": {
     "build": "tsup src/index.ts --format esm,cjs --experimental-dts --clean --treeshake=smallest --splitting",

package/src/vector/index.test.ts

@@ -184,8 +184,8 @@ describe('QdrantVector', () => {
       // expect(results[0]?.vector).toEqual(newVector);
     });
 
-    it('should throw exception when no updates are given', () => {
-      expect(qdrant.updateIndexById(testCollectionName, 'id', {})).rejects.toThrow('No updates provided');
+    it('should throw exception when no updates are given', async () => {
+      await expect(qdrant.updateIndexById(testCollectionName, 'id', {})).rejects.toThrow('No updates provided');
     });
 
     it('should throw error for non-existent index', async () => {
@@ -755,6 +755,15 @@ describe('QdrantVector', () => {
     });
   });
   describe('Error Handling', () => {
+    const testIndexName = 'test_index_error';
+    beforeAll(async () => {
+      await qdrant.createIndex({ indexName: testIndexName, dimension: 3 });
+    });
+
+    afterAll(async () => {
+      await qdrant.deleteIndex(testIndexName);
+    });
+
     it('should handle non-existent index query gracefully', async () => {
       const nonExistentIndex = 'non-existent-index';
       await expect(qdrant.query({ indexName: nonExistentIndex, queryVector: [1, 0, 0] })).rejects.toThrow();
@@ -764,6 +773,65 @@ describe('QdrantVector', () => {
       const wrongDimVector = [[1, 0]]; // 2D vector for 3D index
       await expect(qdrant.upsert({ indexName: testCollectionName, vectors: wrongDimVector })).rejects.toThrow();
     }, 50000);
+
+    it('should handle mismatched metadata and vectors length', async () => {
+      const vectors = [[1, 2, 3]];
+      const metadata = [{}, {}];
+      await expect(qdrant.upsert({ indexName: testCollectionName, vectors, metadata })).rejects.toThrow();
+    });
+
+    it('should handle duplicate index creation gracefully', async () => {
+      const duplicateIndexName = `duplicate_test`;
+      const dimension = 768;
+      const infoSpy = vi.spyOn(qdrant['logger'], 'info');
+      const warnSpy = vi.spyOn(qdrant['logger'], 'warn');
+      try {
+        // Create index first time
+        await qdrant.createIndex({
+          indexName: duplicateIndexName,
+          dimension,
+          metric: 'cosine',
+        });
+
+        // Try to create with same dimensions - should not throw
+        await expect(
+          qdrant.createIndex({
+            indexName: duplicateIndexName,
+            dimension,
+            metric: 'cosine',
+          }),
+        ).resolves.not.toThrow();
+
+        expect(infoSpy).toHaveBeenCalledWith(expect.stringContaining('already exists with'));
+
+        // Try to create with same dimensions and different metric - should not throw
+        await expect(
+          qdrant.createIndex({
+            indexName: duplicateIndexName,
+            dimension,
+            metric: 'euclidean',
+          }),
+        ).resolves.not.toThrow();
+
+        expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining('Attempted to create index with metric'));
+
+        // Try to create with different dimensions - should throw
+        await expect(
+          qdrant.createIndex({
+            indexName: duplicateIndexName,
+            dimension: dimension + 1,
+            metric: 'cosine',
+          }),
+        ).rejects.toThrow(
+          `Index "${duplicateIndexName}" already exists with ${dimension} dimensions, but ${dimension + 1} dimensions were requested`,
+        );
+      } finally {
+        infoSpy.mockRestore();
+        warnSpy.mockRestore();
+        // Cleanup
+        await qdrant.deleteIndex(duplicateIndexName);
+      }
+    });
   });
 
   describe('Empty/Undefined Filters', () => {

package/src/vector/index.ts

@@ -75,14 +75,25 @@ export class QdrantVector extends MastraVector {
     if (!Number.isInteger(dimension) || dimension <= 0) {
       throw new Error('Dimension must be a positive integer');
     }
-    await this.client.createCollection(indexName, {
-      vectors: {
-        // @ts-expect-error
-        size: dimension,
-        // @ts-expect-error
-        distance: DISTANCE_MAPPING[metric],
-      },
-    });
+    if (!DISTANCE_MAPPING[metric]) {
+      throw new Error(`Invalid metric: "${metric}". Must be one of: cosine, euclidean, dotproduct`);
+    }
+    try {
+      await this.client.createCollection(indexName, {
+        vectors: {
+          size: dimension,
+          distance: DISTANCE_MAPPING[metric],
+        },
+      });
+    } catch (error: any) {
+      const message = error?.message || error?.toString();
+      // Qdrant typically returns 409 for existing collection
+      if (error?.status === 409 || (typeof message === 'string' && message.toLowerCase().includes('exists'))) {
+        // Fetch collection info and check dimension
+        await this.validateExistingIndex(indexName, dimension, metric);
+        return;
+      }
+    }
   }
 
   transformFilter(filter?: VectorFilter) {