@mhalder/qdrant-mcp-server 3.3.2 → 3.3.4

package/src/qdrant/client.test.ts

@@ -39,9 +39,7 @@ describe("QdrantManager", () => {
     // Reset mocks and restore default implementations
     mockClient.createCollection.mockReset().mockResolvedValue({});
     mockClient.getCollection.mockReset().mockResolvedValue({});
-    mockClient.getCollections
-      .mockReset()
-      .mockResolvedValue({ collections: [] });
+    mockClient.getCollections.mockReset().mockResolvedValue({ collections: [] });
     mockClient.deleteCollection.mockReset().mockResolvedValue({});
     mockClient.upsert.mockReset().mockResolvedValue({});
     mockClient.search.mockReset().mockResolvedValue([]);
@@ -76,50 +74,41 @@ describe("QdrantManager", () => {
     it("should create a collection with default distance metric", async () => {
       await manager.createCollection("test-collection", 1536);
 
-      expect(mockClient.createCollection).toHaveBeenCalledWith(
-        "test-collection",
-        {
-          vectors: {
-            size: 1536,
-            distance: "Cosine",
-          },
+      expect(mockClient.createCollection).toHaveBeenCalledWith("test-collection", {
+        vectors: {
+          size: 1536,
+          distance: "Cosine",
         },
-      );
+      });
     });
 
     it("should create a collection with custom distance metric", async () => {
       await manager.createCollection("test-collection", 1536, "Euclid");
 
-      expect(mockClient.createCollection).toHaveBeenCalledWith(
-        "test-collection",
-        {
-          vectors: {
-            size: 1536,
-            distance: "Euclid",
-          },
+      expect(mockClient.createCollection).toHaveBeenCalledWith("test-collection", {
+        vectors: {
+          size: 1536,
+          distance: "Euclid",
         },
-      );
+      });
     });
 
     it("should create a hybrid collection with sparse vectors enabled", async () => {
       await manager.createCollection("test-collection", 1536, "Cosine", true);
 
-      expect(mockClient.createCollection).toHaveBeenCalledWith(
-        "test-collection",
-        {
-          vectors: {
-            dense: {
-              size: 1536,
-              distance: "Cosine",
-            },
+      expect(mockClient.createCollection).toHaveBeenCalledWith("test-collection", {
+        vectors: {
+          dense: {
+            size: 1536,
+            distance: "Cosine",
           },
-          sparse_vectors: {
-            text: {
-              modifier: "idf",
-            },
+        },
+        sparse_vectors: {
+          text: {
+            modifier: "idf",
           },
         },
-      );
+      });
     });
   });
 
@@ -145,20 +134,12 @@ describe("QdrantManager", () => {
   describe("listCollections", () => {
     it("should return list of collection names", async () => {
       mockClient.getCollections.mockResolvedValue({
-        collections: [
-          { name: "collection1" },
-          { name: "collection2" },
-          { name: "collection3" },
-        ],
+        collections: [{ name: "collection1" }, { name: "collection2" }, { name: "collection3" }],
       });
 
       const collections = await manager.listCollections();
 
-      expect(collections).toEqual([
-        "collection1",
-        "collection2",
-        "collection3",
-      ]);
+      expect(collections).toEqual(["collection1", "collection2", "collection3"]);
     });
 
     it("should return empty array when no collections exist", async () => {
@@ -253,9 +234,7 @@ describe("QdrantManager", () => {
     it("should delete a collection", async () => {
       await manager.deleteCollection("test-collection");
 
-      expect(mockClient.deleteCollection).toHaveBeenCalledWith(
-        "test-collection",
-      );
+      expect(mockClient.deleteCollection).toHaveBeenCalledWith("test-collection");
     });
   });
 
@@ -304,7 +283,7 @@ describe("QdrantManager", () => {
       const normalizedId = calls[0][1].points[0].id;
       // Check that it's a valid UUID format
       expect(normalizedId).toMatch(
-        /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i,
+        /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i
       );
       // Ensure it's not the original ID
       expect(normalizedId).not.toBe("my-custom-id");
@@ -312,9 +291,7 @@ describe("QdrantManager", () => {
 
     it("should preserve UUID format IDs without modification", async () => {
       const uuidId = "123e4567-e89b-12d3-a456-426614174000";
-      const points = [
-        { id: uuidId, vector: [0.1, 0.2, 0.3], payload: { text: "test" } },
-      ];
+      const points = [{ id: uuidId, vector: [0.1, 0.2, 0.3], payload: { text: "test" } }];
 
       await manager.addPoints("test-collection", points);
 
@@ -342,10 +319,8 @@ describe("QdrantManager", () => {
         },
       });
 
-      await expect(
-        manager.addPoints("test-collection", points),
-      ).rejects.toThrow(
-        'Failed to add points to collection "test-collection": Vector dimension mismatch',
+      await expect(manager.addPoints("test-collection", points)).rejects.toThrow(
+        'Failed to add points to collection "test-collection": Vector dimension mismatch'
       );
     });
 
@@ -354,10 +329,8 @@ describe("QdrantManager", () => {
 
       mockClient.upsert.mockRejectedValue(new Error("Network error"));
 
-      await expect(
-        manager.addPoints("test-collection", points),
-      ).rejects.toThrow(
-        'Failed to add points to collection "test-collection": Network error',
+      await expect(manager.addPoints("test-collection", points)).rejects.toThrow(
+        'Failed to add points to collection "test-collection": Network error'
       );
     });
 
@@ -366,10 +339,8 @@ describe("QdrantManager", () => {
 
       mockClient.upsert.mockRejectedValue("Unknown error");
 
-      await expect(
-        manager.addPoints("test-collection", points),
-      ).rejects.toThrow(
-        'Failed to add points to collection "test-collection": Unknown error',
+      await expect(manager.addPoints("test-collection", points)).rejects.toThrow(
+        'Failed to add points to collection "test-collection": Unknown error'
       );
     });
   });
@@ -397,11 +368,7 @@ describe("QdrantManager", () => {
         { id: 2, score: 0.85, payload: { text: "result2" } },
       ]);
 
-      const results = await manager.search(
-        "test-collection",
-        [0.1, 0.2, 0.3],
-        5,
-      );
+      const results = await manager.search("test-collection", [0.1, 0.2, 0.3], 5);
 
       expect(results).toEqual([
         { id: 1, score: 0.95, payload: { text: "result1" } },
@@ -500,9 +467,7 @@ describe("QdrantManager", () => {
     });
 
     it("should handle null payload in results", async () => {
-      mockClient.search.mockResolvedValue([
-        { id: 1, score: 0.95, payload: null },
-      ]);
+      mockClient.search.mockResolvedValue([{ id: 1, score: 0.95, payload: null }]);
 
       const results = await manager.search("test-collection", [0.1, 0.2, 0.3]);
 
@@ -531,19 +496,11 @@ describe("QdrantManager", () => {
         },
       });
 
-      mockClient.search.mockResolvedValue([
-        { id: 1, score: 0.95, payload: { text: "result1" } },
-      ]);
+      mockClient.search.mockResolvedValue([{ id: 1, score: 0.95, payload: { text: "result1" } }]);
 
-      const results = await manager.search(
-        "hybrid-collection",
-        [0.1, 0.2, 0.3],
-        5,
-      );
+      const results = await manager.search("hybrid-collection", [0.1, 0.2, 0.3], 5);
 
-      expect(results).toEqual([
-        { id: 1, score: 0.95, payload: { text: "result1" } },
-      ]);
+      expect(results).toEqual([{ id: 1, score: 0.95, payload: { text: "result1" } }]);
       expect(mockClient.search).toHaveBeenCalledWith("hybrid-collection", {
         vector: { name: "dense", vector: [0.1, 0.2, 0.3] },
         limit: 5,
@@ -566,19 +523,11 @@ describe("QdrantManager", () => {
         },
       });
 
-      mockClient.search.mockResolvedValue([
-        { id: 1, score: 0.95, payload: { text: "result1" } },
-      ]);
+      mockClient.search.mockResolvedValue([{ id: 1, score: 0.95, payload: { text: "result1" } }]);
 
-      const results = await manager.search(
-        "standard-collection",
-        [0.1, 0.2, 0.3],
-        5,
-      );
+      const results = await manager.search("standard-collection", [0.1, 0.2, 0.3], 5);
 
-      expect(results).toEqual([
-        { id: 1, score: 0.95, payload: { text: "result1" } },
-      ]);
+      expect(results).toEqual([{ id: 1, score: 0.95, payload: { text: "result1" } }]);
       expect(mockClient.search).toHaveBeenCalledWith("standard-collection", {
         vector: [0.1, 0.2, 0.3],
         limit: 5,
@@ -589,9 +538,7 @@ describe("QdrantManager", () => {
 
   describe("getPoint", () => {
     it("should retrieve a point by id", async () => {
-      mockClient.retrieve.mockResolvedValue([
-        { id: 1, payload: { text: "test" } },
-      ]);
+      mockClient.retrieve.mockResolvedValue([{ id: 1, payload: { text: "test" } }]);
 
       const point = await manager.getPoint("test-collection", 1);
 
@@ -691,11 +638,7 @@ describe("QdrantManager", () => {
         ],
       });
 
-      const results = await manager.hybridSearch(
-        "test-collection",
-        denseVector,
-        sparseVector,
-      );
+      const results = await manager.hybridSearch("test-collection", denseVector, sparseVector);
 
       expect(results).toEqual([
         { id: 1, score: 0.95, payload: { text: "result1" } },
@@ -731,12 +674,7 @@ describe("QdrantManager", () => {
 
       mockClient.query.mockResolvedValue({ points: [] });
 
-      await manager.hybridSearch(
-        "test-collection",
-        denseVector,
-        sparseVector,
-        10,
-      );
+      await manager.hybridSearch("test-collection", denseVector, sparseVector, 10);
 
       expect(mockClient.query).toHaveBeenCalledWith(
         "test-collection",
@@ -745,7 +683,7 @@ describe("QdrantManager", () => {
             expect.objectContaining({ limit: 40 }), // 10 * 4
           ]),
           limit: 10,
-        }),
+        })
       );
     });
 
@@ -756,13 +694,7 @@ describe("QdrantManager", () => {
 
       mockClient.query.mockResolvedValue({ points: [] });
 
-      await manager.hybridSearch(
-        "test-collection",
-        denseVector,
-        sparseVector,
-        5,
-        filter,
-      );
+      await manager.hybridSearch("test-collection", denseVector, sparseVector, 5, filter);
 
       expect(mockClient.query).toHaveBeenCalledWith("test-collection", {
         prefetch: [
@@ -804,13 +736,7 @@ describe("QdrantManager", () => {
 
       mockClient.query.mockResolvedValue({ points: [] });
 
-      await manager.hybridSearch(
-        "test-collection",
-        denseVector,
-        sparseVector,
-        5,
-        filter,
-      );
+      await manager.hybridSearch("test-collection", denseVector, sparseVector, 5, filter);
 
       const call = mockClient.query.mock.calls[0][1];
       expect(call.prefetch[0].filter).toEqual(filter);
@@ -826,13 +752,7 @@ describe("QdrantManager", () => {
 
       mockClient.query.mockResolvedValue({ points: [] });
 
-      await manager.hybridSearch(
-        "test-collection",
-        denseVector,
-        sparseVector,
-        5,
-        filter,
-      );
+      await manager.hybridSearch("test-collection", denseVector, sparseVector, 5, filter);
 
       const call = mockClient.query.mock.calls[0][1];
       expect(call.prefetch[0].filter).toEqual(filter);
@@ -848,13 +768,7 @@ describe("QdrantManager", () => {
 
       mockClient.query.mockResolvedValue({ points: [] });
 
-      await manager.hybridSearch(
-        "test-collection",
-        denseVector,
-        sparseVector,
-        5,
-        filter,
-      );
+      await manager.hybridSearch("test-collection", denseVector, sparseVector, 5, filter);
 
       const call = mockClient.query.mock.calls[0][1];
       expect(call.prefetch[0].filter).toEqual(filter);
@@ -867,13 +781,7 @@ describe("QdrantManager", () => {
 
       mockClient.query.mockResolvedValue({ points: [] });
 
-      await manager.hybridSearch(
-        "test-collection",
-        denseVector,
-        sparseVector,
-        5,
-        {},
-      );
+      await manager.hybridSearch("test-collection", denseVector, sparseVector, 5, {});
 
       const call = mockClient.query.mock.calls[0][1];
       expect(call.prefetch[0].filter).toBeUndefined();
@@ -888,11 +796,7 @@ describe("QdrantManager", () => {
         points: [{ id: 1, score: 0.95, payload: null }],
       });
 
-      const results = await manager.hybridSearch(
-        "test-collection",
-        denseVector,
-        sparseVector,
-      );
+      const results = await manager.hybridSearch("test-collection", denseVector, sparseVector);
 
       expect(results).toEqual([{ id: 1, score: 0.95, payload: undefined }]);
     });
@@ -910,9 +814,9 @@ describe("QdrantManager", () => {
       });
 
       await expect(
-        manager.hybridSearch("test-collection", denseVector, sparseVector),
+        manager.hybridSearch("test-collection", denseVector, sparseVector)
       ).rejects.toThrow(
-        'Hybrid search failed on collection "test-collection": Named vector not found',
+        'Hybrid search failed on collection "test-collection": Named vector not found'
       );
     });
 
@@ -923,10 +827,8 @@ describe("QdrantManager", () => {
       mockClient.query.mockRejectedValue(new Error("Network timeout"));
 
       await expect(
-        manager.hybridSearch("test-collection", denseVector, sparseVector),
-      ).rejects.toThrow(
-        'Hybrid search failed on collection "test-collection": Network timeout',
-      );
+        manager.hybridSearch("test-collection", denseVector, sparseVector)
+      ).rejects.toThrow('Hybrid search failed on collection "test-collection": Network timeout');
     });
 
     it("should throw error with String(error) fallback", async () => {
@@ -936,10 +838,8 @@ describe("QdrantManager", () => {
       mockClient.query.mockRejectedValue("Unknown error");
 
       await expect(
-        manager.hybridSearch("test-collection", denseVector, sparseVector),
-      ).rejects.toThrow(
-        'Hybrid search failed on collection "test-collection": Unknown error',
-      );
+        manager.hybridSearch("test-collection", denseVector, sparseVector)
+      ).rejects.toThrow('Hybrid search failed on collection "test-collection": Unknown error');
     });
   });
 
@@ -1030,7 +930,7 @@ describe("QdrantManager", () => {
       const normalizedId = calls[0][1].points[0].id;
       // Check that it's a valid UUID format
       expect(normalizedId).toMatch(
-        /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i,
+        /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i
       );
       expect(normalizedId).not.toBe("my-doc-id");
     });
@@ -1080,10 +980,8 @@ describe("QdrantManager", () => {
         },
       });
 
-      await expect(
-        manager.addPointsWithSparse("test-collection", points),
-      ).rejects.toThrow(
-        'Failed to add points with sparse vectors to collection "test-collection": Sparse vector not configured',
+      await expect(manager.addPointsWithSparse("test-collection", points)).rejects.toThrow(
+        'Failed to add points with sparse vectors to collection "test-collection": Sparse vector not configured'
       );
     });
 
@@ -1098,10 +996,8 @@ describe("QdrantManager", () => {
 
       mockClient.upsert.mockRejectedValue(new Error("Connection refused"));
 
-      await expect(
-        manager.addPointsWithSparse("test-collection", points),
-      ).rejects.toThrow(
-        'Failed to add points with sparse vectors to collection "test-collection": Connection refused',
+      await expect(manager.addPointsWithSparse("test-collection", points)).rejects.toThrow(
+        'Failed to add points with sparse vectors to collection "test-collection": Connection refused'
       );
     });
 
@@ -1116,10 +1012,8 @@ describe("QdrantManager", () => {
 
       mockClient.upsert.mockRejectedValue("Unexpected error");
 
-      await expect(
-        manager.addPointsWithSparse("test-collection", points),
-      ).rejects.toThrow(
-        'Failed to add points with sparse vectors to collection "test-collection": Unexpected error',
+      await expect(manager.addPointsWithSparse("test-collection", points)).rejects.toThrow(
+        'Failed to add points with sparse vectors to collection "test-collection": Unexpected error'
       );
     });
   });

package/src/qdrant/client.ts

@@ -39,8 +39,7 @@ export class QdrantManager {
     }
 
     // Check if already a valid UUID (8-4-4-4-12 format)
-    const uuidRegex =
-      /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
+    const uuidRegex = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
     if (uuidRegex.test(id)) {
       return id;
     }
@@ -54,12 +53,9 @@ export class QdrantManager {
     name: string,
     vectorSize: number,
     distance: "Cosine" | "Euclid" | "Dot" = "Cosine",
-    enableSparse: boolean = false,
+    enableSparse: boolean = false
   ): Promise<void> {
-    this.log.debug(
-      { collection: name, vectorSize, distance, enableSparse },
-      "createCollection",
-    );
+    this.log.debug({ collection: name, vectorSize, distance, enableSparse }, "createCollection");
     const config: any = {};
 
     // When hybrid search is enabled, use named vectors
@@ -147,12 +143,9 @@ export class QdrantManager {
       id: string | number;
       vector: number[];
       payload?: Record<string, any>;
-    }>,
+    }>
   ): Promise<void> {
-    this.log.debug(
-      { collection: collectionName, count: points.length },
-      "addPoints",
-    );
+    this.log.debug({ collection: collectionName, count: points.length }, "addPoints");
     try {
       // Normalize all IDs to ensure string IDs are in UUID format
       const normalizedPoints = points.map((point) => ({
@@ -165,11 +158,8 @@ export class QdrantManager {
         points: normalizedPoints,
       });
     } catch (error: any) {
-      const errorMessage =
-        error?.data?.status?.error || error?.message || String(error);
-      throw new Error(
-        `Failed to add points to collection "${collectionName}": ${errorMessage}`,
-      );
+      const errorMessage = error?.data?.status?.error || error?.message || String(error);
+      throw new Error(`Failed to add points to collection "${collectionName}": ${errorMessage}`);
     }
   }
 
@@ -177,7 +167,7 @@ export class QdrantManager {
     collectionName: string,
     vector: number[],
     limit: number = 5,
-    filter?: Record<string, any>,
+    filter?: Record<string, any>
   ): Promise<SearchResult[]> {
     this.log.debug({ collection: collectionName, limit }, "search");
     // Convert simple key-value filter to Qdrant filter format
@@ -218,7 +208,7 @@ export class QdrantManager {
 
   async getPoint(
     collectionName: string,
-    id: string | number,
+    id: string | number
   ): Promise<{ id: string | number; payload?: Record<string, any> } | null> {
     try {
       const normalizedId = this.normalizeId(id);
@@ -239,14 +229,8 @@ export class QdrantManager {
     }
   }
 
-  async deletePoints(
-    collectionName: string,
-    ids: (string | number)[],
-  ): Promise<void> {
-    this.log.debug(
-      { collection: collectionName, count: ids.length },
-      "deletePoints",
-    );
+  async deletePoints(collectionName: string, ids: (string | number)[]): Promise<void> {
+    this.log.debug({ collection: collectionName, count: ids.length }, "deletePoints");
     // Normalize IDs to ensure string IDs are in UUID format
     const normalizedIds = ids.map((id) => this.normalizeId(id));
 
@@ -260,10 +244,7 @@ export class QdrantManager {
    * Deletes points matching a filter condition.
    * Useful for deleting all chunks associated with a specific file path.
    */
-  async deletePointsByFilter(
-    collectionName: string,
-    filter: Record<string, any>,
-  ): Promise<void> {
+  async deletePointsByFilter(collectionName: string, filter: Record<string, any>): Promise<void> {
     this.log.debug({ collection: collectionName }, "deletePointsByFilter");
     await this.client.delete(collectionName, {
       wait: true,
@@ -281,7 +262,7 @@ export class QdrantManager {
     sparseVector: SparseVector,
     limit: number = 5,
     filter?: Record<string, any>,
-    _semanticWeight: number = 0.7,
+    _semanticWeight: number = 0.7
   ): Promise<SearchResult[]> {
     this.log.debug({ collection: collectionName, limit }, "hybridSearch");
     // Convert simple key-value filter to Qdrant filter format
@@ -332,11 +313,8 @@ export class QdrantManager {
         payload: result.payload || undefined,
       }));
     } catch (error: any) {
-      const errorMessage =
-        error?.data?.status?.error || error?.message || String(error);
-      throw new Error(
-        `Hybrid search failed on collection "${collectionName}": ${errorMessage}`,
-      );
+      const errorMessage = error?.data?.status?.error || error?.message || String(error);
+      throw new Error(`Hybrid search failed on collection "${collectionName}": ${errorMessage}`);
     }
   }
 
@@ -350,12 +328,9 @@ export class QdrantManager {
       vector: number[];
       sparseVector: SparseVector;
       payload?: Record<string, any>;
-    }>,
+    }>
   ): Promise<void> {
-    this.log.debug(
-      { collection: collectionName, count: points.length },
-      "addPointsWithSparse",
-    );
+    this.log.debug({ collection: collectionName, count: points.length }, "addPointsWithSparse");
     try {
       // Normalize all IDs to ensure string IDs are in UUID format
       const normalizedPoints = points.map((point) => ({
@@ -372,10 +347,9 @@ export class QdrantManager {
         points: normalizedPoints,
       });
     } catch (error: any) {
-      const errorMessage =
-        error?.data?.status?.error || error?.message || String(error);
+      const errorMessage = error?.data?.status?.error || error?.message || String(error);
       throw new Error(
-        `Failed to add points with sparse vectors to collection "${collectionName}": ${errorMessage}`,
+        `Failed to add points with sparse vectors to collection "${collectionName}": ${errorMessage}`
       );
     }
   }

package/src/resources/index.ts

@@ -2,19 +2,13 @@
  * Resource registration module
  */
 
-import {
-  McpServer,
-  ResourceTemplate,
-} from "@modelcontextprotocol/sdk/server/mcp.js";
+import { type McpServer, ResourceTemplate } from "@modelcontextprotocol/sdk/server/mcp.js";
 import type { QdrantManager } from "../qdrant/client.js";
 
 /**
  * Register all MCP resources on the server
  */
-export function registerAllResources(
-  server: McpServer,
-  qdrant: QdrantManager,
-): void {
+export function registerAllResources(server: McpServer, qdrant: QdrantManager): void {
   // Static resource: list all collections
   server.registerResource(
     "collections",
@@ -35,7 +29,7 @@ export function registerAllResources(
           },
         ],
       };
-    },
+    }
   );
 
   // Dynamic resource: individual collection info
@@ -74,6 +68,6 @@ export function registerAllResources(
           },
         ],
       };
-    },
+    }
   );
 }