@mhalder/qdrant-mcp-server 3.3.3 → 3.3.4

package/src/git/extractor.integration.test.ts

@@ -3,12 +3,12 @@
  * These tests run against real git repositories (not mocked)
  */
 
-import { describe, it, expect, beforeAll } from "vitest";
-import { GitExtractor } from "./extractor.js";
-import { DEFAULT_GIT_CONFIG } from "./config.js";
-import type { GitConfig } from "./types.js";
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";
+import { beforeAll, describe, expect, it } from "vitest";
+import { DEFAULT_GIT_CONFIG } from "./config.js";
+import { GitExtractor } from "./extractor.js";
+import type { GitConfig } from "./types.js";
 
 const execFileAsync = promisify(execFile);
 
@@ -93,11 +93,9 @@ describe("GitExtractor Integration Tests", () => {
 
     it("should return correct commit count matching git rev-list", async () => {
       // Get expected count from git directly
-      const { stdout } = await execFileAsync(
-        "git",
-        ["rev-list", "--count", "-n", "50", "HEAD"],
-        { cwd: repoPath },
-      );
+      const { stdout } = await execFileAsync("git", ["rev-list", "--count", "-n", "50", "HEAD"], {
+        cwd: repoPath,
+      });
       const expectedCount = Math.min(parseInt(stdout.trim(), 10), 50);
 
       // Get commits via extractor
@@ -112,7 +110,7 @@ describe("GitExtractor Integration Tests", () => {
       const { stdout: logOutput } = await execFileAsync(
         "git",
         ["log", "--oneline", "--shortstat", "-n", "10", "HEAD"],
-        { cwd: repoPath },
+        { cwd: repoPath }
       );
 
       // If there are commits with files changed, verify our extractor gets them
@@ -199,11 +197,9 @@ describe("GitExtractor Integration Tests", () => {
       const count = await extractor.getCommitCount();
 
       // Verify against git rev-list
-      const { stdout } = await execFileAsync(
-        "git",
-        ["rev-list", "--count", "HEAD"],
-        { cwd: repoPath },
-      );
+      const { stdout } = await execFileAsync("git", ["rev-list", "--count", "HEAD"], {
+        cwd: repoPath,
+      });
       expect(count).toBe(parseInt(stdout.trim(), 10));
     });
 
@@ -218,7 +214,7 @@ describe("GitExtractor Integration Tests", () => {
         const { stdout } = await execFileAsync(
           "git",
           ["rev-list", "--count", `${sinceHash}..HEAD`],
-          { cwd: repoPath },
+          { cwd: repoPath }
         );
         expect(count).toBe(parseInt(stdout.trim(), 10));
       }

package/src/git/extractor.test.ts

@@ -1,6 +1,6 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import { GitExtractor, normalizeRemoteUrl } from "./extractor.js";
+import { beforeEach, describe, expect, it, vi } from "vitest";
 import { DEFAULT_GIT_CONFIG, GIT_LOG_COMMIT_DELIMITER } from "./config.js";
+import { GitExtractor, normalizeRemoteUrl } from "./extractor.js";
 import type { GitConfig } from "./types.js";
 
 // Mock child_process with promisify-compatible execFile
@@ -35,7 +35,7 @@ describe("GitExtractor", () => {
       expect(mockExecFile).toHaveBeenCalledWith(
         "git",
         ["rev-parse", "--git-dir"],
-        expect.objectContaining({ cwd: "/test/repo" }),
+        expect.objectContaining({ cwd: "/test/repo" })
       );
     });
 
@@ -61,7 +61,7 @@ describe("GitExtractor", () => {
       expect(mockExecFile).toHaveBeenCalledWith(
         "git",
         ["rev-parse", "HEAD"],
-        expect.objectContaining({ cwd: "/test/repo" }),
+        expect.objectContaining({ cwd: "/test/repo" })
       );
     });
   });
@@ -76,7 +76,7 @@ describe("GitExtractor", () => {
       expect(mockExecFile).toHaveBeenCalledWith(
         "git",
         ["rev-list", "--count", "HEAD"],
-        expect.objectContaining({ cwd: "/test/repo" }),
+        expect.objectContaining({ cwd: "/test/repo" })
       );
     });
 
@@ -89,7 +89,7 @@ describe("GitExtractor", () => {
       expect(mockExecFile).toHaveBeenCalledWith(
         "git",
         ["rev-list", "--count", "abc123..HEAD"],
-        expect.objectContaining({ cwd: "/test/repo" }),
+        expect.objectContaining({ cwd: "/test/repo" })
       );
     });
   });
@@ -202,7 +202,7 @@ describe("GitExtractor", () => {
       expect(mockExecFile).toHaveBeenCalledWith(
         "git",
         expect.arrayContaining(["-n100"]),
-        expect.any(Object),
+        expect.any(Object)
       );
     });
 
@@ -214,7 +214,7 @@ describe("GitExtractor", () => {
       expect(mockExecFile).toHaveBeenCalledWith(
         "git",
         expect.arrayContaining(["abc123..HEAD"]),
-        expect.any(Object),
+        expect.any(Object)
       );
     });
 
@@ -226,7 +226,7 @@ describe("GitExtractor", () => {
       expect(mockExecFile).toHaveBeenCalledWith(
         "git",
         expect.arrayContaining(["--since=2024-01-01"]),
-        expect.any(Object),
+        expect.any(Object)
       );
     });
 
@@ -287,7 +287,7 @@ diff --git a/file.ts b/file.ts
       expect(mockExecFile).toHaveBeenCalledWith(
         "git",
         ["show", "--no-color", "-p", "abc123"],
-        expect.objectContaining({ cwd: "/test/repo" }),
+        expect.objectContaining({ cwd: "/test/repo" })
       );
     });
 
@@ -326,14 +326,12 @@ diff --git a/file.ts b/file.ts
       expect(mockExecFile).toHaveBeenCalledWith(
         "git",
         ["remote", "get-url", "origin"],
-        expect.objectContaining({ cwd: "/test/repo" }),
+        expect.objectContaining({ cwd: "/test/repo" })
       );
     });
 
     it("should return empty string when no remote configured", async () => {
-      mockExecFile.mockRejectedValue(
-        new Error("fatal: No such remote 'origin'"),
-      );
+      mockExecFile.mockRejectedValue(new Error("fatal: No such remote 'origin'"));
 
       const result = await extractor.getRemoteUrl();
 
@@ -355,22 +353,16 @@ diff --git a/file.ts b/file.ts
 
 describe("normalizeRemoteUrl", () => {
   it("should normalize SSH URL", () => {
-    expect(normalizeRemoteUrl("git@github.com:user/repo.git")).toBe(
-      "user/repo",
-    );
+    expect(normalizeRemoteUrl("git@github.com:user/repo.git")).toBe("user/repo");
   });
 
   it("should normalize HTTPS URL", () => {
-    expect(normalizeRemoteUrl("https://github.com/user/repo.git")).toBe(
-      "user/repo",
-    );
+    expect(normalizeRemoteUrl("https://github.com/user/repo.git")).toBe("user/repo");
   });
 
   it("should handle URL without .git suffix", () => {
     expect(normalizeRemoteUrl("git@github.com:user/repo")).toBe("user/repo");
-    expect(normalizeRemoteUrl("https://github.com/user/repo")).toBe(
-      "user/repo",
-    );
+    expect(normalizeRemoteUrl("https://github.com/user/repo")).toBe("user/repo");
   });
 
   it("should return empty string for empty input", () => {
@@ -378,26 +370,20 @@ describe("normalizeRemoteUrl", () => {
   });
 
   it("should handle GitLab SSH URL", () => {
-    expect(normalizeRemoteUrl("git@gitlab.com:group/project.git")).toBe(
-      "group/project",
-    );
+    expect(normalizeRemoteUrl("git@gitlab.com:group/project.git")).toBe("group/project");
   });
 
   it("should handle Bitbucket SSH URL", () => {
-    expect(normalizeRemoteUrl("git@bitbucket.org:team/repo.git")).toBe(
-      "team/repo",
-    );
+    expect(normalizeRemoteUrl("git@bitbucket.org:team/repo.git")).toBe("team/repo");
   });
 
   it("should handle HTTP URL (not HTTPS)", () => {
-    expect(normalizeRemoteUrl("http://github.com/user/repo.git")).toBe(
-      "user/repo",
-    );
+    expect(normalizeRemoteUrl("http://github.com/user/repo.git")).toBe("user/repo");
   });
 
   it("should handle nested paths", () => {
-    expect(
-      normalizeRemoteUrl("https://github.com/org/group/subgroup/repo.git"),
-    ).toBe("org/group/subgroup/repo");
+    expect(normalizeRemoteUrl("https://github.com/org/group/subgroup/repo.git")).toBe(
+      "org/group/subgroup/repo"
+    );
   });
 });

package/src/git/extractor.ts

@@ -5,11 +5,7 @@
 
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";
-import {
-  GIT_LOG_COMMIT_DELIMITER,
-  GIT_LOG_FORMAT,
-  GIT_MAX_BUFFER,
-} from "./config.js";
+import { GIT_LOG_COMMIT_DELIMITER, GIT_LOG_FORMAT, GIT_MAX_BUFFER } from "./config.js";
 import type { GitConfig, GitExtractOptions, RawCommit } from "./types.js";
 
 const execFileAsync = promisify(execFile);
@@ -34,7 +30,7 @@ export function normalizeRemoteUrl(url: string): string {
 export class GitExtractor {
   constructor(
     private repoPath: string,
-    private config: GitConfig,
+    private config: GitConfig
   ) {}
 
   /**
@@ -70,15 +66,11 @@ export class GitExtractor {
    */
   async getRemoteUrl(): Promise<string> {
     try {
-      const { stdout } = await execFileAsync(
-        "git",
-        ["remote", "get-url", "origin"],
-        {
-          cwd: this.repoPath,
-          maxBuffer: GIT_MAX_BUFFER,
-          timeout: this.config.gitTimeout,
-        },
-      );
+      const { stdout } = await execFileAsync("git", ["remote", "get-url", "origin"], {
+        cwd: this.repoPath,
+        maxBuffer: GIT_MAX_BUFFER,
+        timeout: this.config.gitTimeout,
+      });
       return stdout.trim();
     } catch {
       return ""; // No remote configured
@@ -144,15 +136,11 @@ export class GitExtractor {
    */
   async getCommitDiff(commitHash: string): Promise<string> {
     try {
-      const { stdout } = await execFileAsync(
-        "git",
-        ["show", "--no-color", "-p", commitHash],
-        {
-          cwd: this.repoPath,
-          maxBuffer: GIT_MAX_BUFFER,
-          timeout: this.config.gitTimeout,
-        },
-      );
+      const { stdout } = await execFileAsync("git", ["show", "--no-color", "-p", commitHash], {
+        cwd: this.repoPath,
+        maxBuffer: GIT_MAX_BUFFER,
+        timeout: this.config.gitTimeout,
+      });
 
       // Truncate diff if it exceeds maxDiffSize
       if (stdout.length > this.config.maxDiffSize) {
@@ -204,15 +192,7 @@ export class GitExtractor {
 
     if (parts.length < 6) return null;
 
-    const [
-      hash,
-      shortHash,
-      author,
-      authorEmail,
-      dateStr,
-      subject,
-      ...bodyParts
-    ] = parts;
+    const [hash, shortHash, author, authorEmail, dateStr, subject, ...bodyParts] = parts;
 
     // Parse files and stats from numstat output
     const { files, insertions, deletions } = this.parseNumstat(lines.slice(1));
@@ -266,9 +246,7 @@ export class GitExtractor {
         if (filename.includes(" => ")) {
           const renameParts = filename.match(/(.+)\{(.+) => (.+)\}(.+)?/);
           if (renameParts) {
-            files.push(
-              `${renameParts[1]}${renameParts[3]}${renameParts[4] || ""}`,
-            );
+            files.push(`${renameParts[1]}${renameParts[3]}${renameParts[4] || ""}`);
           } else {
             const simpleRename = filename.split(" => ");
             files.push(simpleRename[1] || filename);

package/src/git/index.ts

@@ -2,6 +2,14 @@
  * Git history indexing module - barrel export
  */
 
+export { CommitChunker } from "./chunker.js";
+
+// Config
+export { COMMIT_TYPE_PATTERNS, DEFAULT_GIT_CONFIG } from "./config.js";
+export { GitExtractor } from "./extractor.js";
+// Main classes
+export { GitHistoryIndexer } from "./indexer.js";
+export { GitSynchronizer } from "./sync/synchronizer.js";
 // Types
 export type {
   CommitChunk,
@@ -9,10 +17,10 @@ export type {
   GitChangeStats,
   GitConfig,
   GitExtractOptions,
+  GitIndexingStatus,
   GitIndexOptions,
   GitIndexStats,
   GitIndexStatus,
-  GitIndexingStatus,
   GitProgressCallback,
   GitProgressUpdate,
   GitSearchOptions,
@@ -20,12 +28,3 @@ export type {
   GitSnapshot,
   RawCommit,
 } from "./types.js";
-
-// Config
-export { DEFAULT_GIT_CONFIG, COMMIT_TYPE_PATTERNS } from "./config.js";
-
-// Main classes
-export { GitHistoryIndexer } from "./indexer.js";
-export { GitExtractor } from "./extractor.js";
-export { CommitChunker } from "./chunker.js";
-export { GitSynchronizer } from "./sync/synchronizer.js";

package/src/git/indexer.test.ts

@@ -1,6 +1,6 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import { GitHistoryIndexer } from "./indexer.js";
+import { beforeEach, describe, expect, it, vi } from "vitest";
 import { DEFAULT_GIT_CONFIG } from "./config.js";
+import { GitHistoryIndexer } from "./indexer.js";
 import type { GitConfig } from "./types.js";
 
 // Create mock instances
@@ -100,9 +100,7 @@ describe("GitHistoryIndexer", () => {
     // Reset mock instances
     mockExtractorInstance.validateRepository.mockResolvedValue(true);
     mockExtractorInstance.getLatestCommitHash.mockResolvedValue("abc123def456");
-    mockExtractorInstance.getRemoteUrl.mockResolvedValue(
-      "git@github.com:test/repo.git",
-    );
+    mockExtractorInstance.getRemoteUrl.mockResolvedValue("git@github.com:test/repo.git");
     mockExtractorInstance.getCommits.mockResolvedValue([]);
     mockExtractorInstance.getCommitDiff.mockResolvedValue("");
 
@@ -138,9 +136,7 @@ describe("GitHistoryIndexer", () => {
       collectionExists: vi.fn().mockResolvedValue(false),
       createCollection: vi.fn().mockResolvedValue(undefined),
       deleteCollection: vi.fn().mockResolvedValue(undefined),
-      getCollectionInfo: vi
-        .fn()
-        .mockResolvedValue({ pointsCount: 0, hybridEnabled: false }),
+      getCollectionInfo: vi.fn().mockResolvedValue({ pointsCount: 0, hybridEnabled: false }),
       addPoints: vi.fn().mockResolvedValue(undefined),
       addPointsWithSparse: vi.fn().mockResolvedValue(undefined),
       search: vi.fn().mockResolvedValue([]),
@@ -151,9 +147,7 @@ describe("GitHistoryIndexer", () => {
     mockEmbeddings = {
       getDimensions: vi.fn().mockReturnValue(768),
       embed: vi.fn().mockResolvedValue({ embedding: Array(768).fill(0.5) }),
-      embedBatch: vi
-        .fn()
-        .mockResolvedValue([{ embedding: Array(768).fill(0.5) }]),
+      embedBatch: vi.fn().mockResolvedValue([{ embedding: Array(768).fill(0.5) }]),
     };
 
     indexer = new GitHistoryIndexer(mockQdrant, mockEmbeddings, config);
@@ -196,9 +190,7 @@ describe("GitHistoryIndexer", () => {
       const stats = await indexer.indexHistory("/not/a/repo");
 
       expect(stats.status).toBe("failed");
-      expect(
-        stats.errors?.some((e) => e.includes("Not a valid git repository")),
-      ).toBe(true);
+      expect(stats.errors?.some((e) => e.includes("Not a valid git repository"))).toBe(true);
     });
 
     it("should handle empty repository", async () => {
@@ -253,7 +245,7 @@ describe("GitHistoryIndexer", () => {
       expect(progressCallback).toHaveBeenCalledWith(
         expect.objectContaining({
           phase: expect.stringMatching(/extracting|chunking|embedding|storing/),
-        }),
+        })
       );
     });
   });
@@ -291,9 +283,9 @@ describe("GitHistoryIndexer", () => {
     it("should throw error when history not indexed", async () => {
       mockQdrant.collectionExists.mockResolvedValue(false);
 
-      await expect(
-        indexer.searchHistory("/test/repo", "query"),
-      ).rejects.toThrow("Git history not indexed");
+      await expect(indexer.searchHistory("/test/repo", "query")).rejects.toThrow(
+        "Git history not indexed"
+      );
     });
 
     it("should apply commit type filter", async () => {
@@ -314,7 +306,7 @@ describe("GitHistoryIndexer", () => {
               match: { any: ["fix", "feat"] },
             }),
           ]),
-        }),
+        })
       );
     });
 
@@ -341,7 +333,7 @@ describe("GitHistoryIndexer", () => {
               range: { lte: "2024-12-31" },
             }),
           ]),
-        }),
+        })
       );
     });
 
@@ -448,7 +440,7 @@ describe("GitHistoryIndexer", () => {
       mockSynchronizerInstance.initialize.mockResolvedValue(false);
 
       await expect(indexer.indexNewCommits("/test/repo")).rejects.toThrow(
-        "No previous snapshot found",
+        "No previous snapshot found"
       );
     });
 
@@ -484,7 +476,7 @@ describe("GitHistoryIndexer", () => {
     it("should ignore snapshot deletion errors", async () => {
       mockQdrant.collectionExists.mockResolvedValue(true);
       mockSynchronizerInstance.deleteSnapshot.mockRejectedValue(
-        new Error("Snapshot deletion failed"),
+        new Error("Snapshot deletion failed")
       );
 
       await expect(indexer.clearIndex("/test/repo")).resolves.not.toThrow();
@@ -568,7 +560,7 @@ describe("GitHistoryIndexer", () => {
               ]),
             }),
           ]),
-        }),
+        })
       );
     });
   });
@@ -593,9 +585,7 @@ describe("GitHistoryIndexer", () => {
       mockExtractorInstance.validateRepository.mockResolvedValue(true);
       mockExtractorInstance.getLatestCommitHash.mockResolvedValue("abc123");
       mockExtractorInstance.getCommits.mockResolvedValue(mockCommits);
-      mockExtractorInstance.getCommitDiff.mockRejectedValue(
-        new Error("Diff extraction failed"),
-      );
+      mockExtractorInstance.getCommitDiff.mockRejectedValue(new Error("Diff extraction failed"));
 
       const stats = await indexer.indexHistory("/test/repo");
 
@@ -623,9 +613,7 @@ describe("GitHistoryIndexer", () => {
       mockExtractorInstance.getLatestCommitHash.mockResolvedValue("abc123");
       mockExtractorInstance.getCommits.mockResolvedValue(mockCommits);
       mockExtractorInstance.getCommitDiff.mockResolvedValue("");
-      mockEmbeddings.embedBatch.mockRejectedValue(
-        new Error("Embedding API error"),
-      );
+      mockEmbeddings.embedBatch.mockRejectedValue(new Error("Embedding API error"));
 
       const stats = await indexer.indexHistory("/test/repo");
 
@@ -653,9 +641,7 @@ describe("GitHistoryIndexer", () => {
       mockExtractorInstance.getLatestCommitHash.mockResolvedValue("abc123");
       mockExtractorInstance.getCommits.mockResolvedValue(mockCommits);
       mockExtractorInstance.getCommitDiff.mockResolvedValue("");
-      mockSynchronizerInstance.updateSnapshot.mockRejectedValue(
-        new Error("Snapshot write failed"),
-      );
+      mockSynchronizerInstance.updateSnapshot.mockRejectedValue(new Error("Snapshot write failed"));
 
       const stats = await indexer.indexHistory("/test/repo");
 
@@ -684,11 +670,7 @@ describe("GitHistoryIndexer", () => {
         ...DEFAULT_GIT_CONFIG,
         enableHybridSearch: true,
       };
-      const hybridIndexer = new GitHistoryIndexer(
-        mockQdrant,
-        mockEmbeddings,
-        hybridConfig,
-      );
+      const hybridIndexer = new GitHistoryIndexer(mockQdrant, mockEmbeddings, hybridConfig);
 
       const mockCommits = [
         {
@@ -720,7 +702,7 @@ describe("GitHistoryIndexer", () => {
         expect.any(String),
         768,
         "Cosine",
-        true,
+        true
       );
       expect(mockQdrant.addPointsWithSparse).toHaveBeenCalled();
     });
@@ -758,7 +740,7 @@ describe("GitHistoryIndexer", () => {
       mockQdrant.collectionExists.mockResolvedValue(false);
 
       await expect(indexer.indexNewCommits("/test/repo")).rejects.toThrow(
-        "Git history not indexed",
+        "Git history not indexed"
       );
     });
 
@@ -768,7 +750,7 @@ describe("GitHistoryIndexer", () => {
       mockSynchronizerInstance.getLastCommitHash.mockReturnValue(null);
 
       await expect(indexer.indexNewCommits("/test/repo")).rejects.toThrow(
-        "Invalid snapshot: no last commit hash",
+        "Invalid snapshot: no last commit hash"
       );
     });
 
@@ -805,7 +787,7 @@ describe("GitHistoryIndexer", () => {
       expect(progressCallback).toHaveBeenCalledWith(
         expect.objectContaining({
           phase: expect.stringMatching(/extracting|chunking|embedding|storing/),
-        }),
+        })
       );
     });
 
@@ -814,11 +796,7 @@ describe("GitHistoryIndexer", () => {
         ...DEFAULT_GIT_CONFIG,
         enableHybridSearch: true,
       };
-      const hybridIndexer = new GitHistoryIndexer(
-        mockQdrant,
-        mockEmbeddings,
-        hybridConfig,
-      );
+      const hybridIndexer = new GitHistoryIndexer(mockQdrant, mockEmbeddings, hybridConfig);
 
       mockQdrant.collectionExists.mockResolvedValue(true);
 
@@ -900,9 +878,7 @@ describe("GitHistoryIndexer", () => {
       mockSynchronizerInstance.initialize.mockResolvedValue(true);
       mockSynchronizerInstance.getCommitsIndexed.mockReturnValue(100);
       mockSynchronizerInstance.getLastCommitHash.mockReturnValue("latest123");
-      mockSynchronizerInstance.getLastIndexedAt.mockReturnValue(
-        new Date("2024-01-15T10:00:00Z"),
-      );
+      mockSynchronizerInstance.getLastIndexedAt.mockReturnValue(new Date("2024-01-15T10:00:00Z"));
 
       const status = await indexer.getIndexStatus("/test/repo");
 
@@ -945,9 +921,9 @@ describe("GitHistoryIndexer", () => {
         indexer.searchHistory("/test/repo", "query", {
           dateFrom: "2024-12-31",
           dateTo: "2024-01-01",
-        }),
+        })
       ).rejects.toThrow(
-        "Invalid date range: dateFrom (2024-12-31) must be before dateTo (2024-01-01)",
+        "Invalid date range: dateFrom (2024-12-31) must be before dateTo (2024-01-01)"
       );
     });
 
@@ -1076,9 +1052,7 @@ describe("GitHistoryIndexer", () => {
       mockChunkerInstance.generateChunkId.mockReturnValue("chunk-1");
 
       // All retries fail
-      mockEmbeddings.embedBatch.mockRejectedValue(
-        new Error("Persistent error"),
-      );
+      mockEmbeddings.embedBatch.mockRejectedValue(new Error("Persistent error"));
 
       mockQdrant.collectionExists.mockResolvedValue(false);
       mockQdrant.getCollectionInfo.mockResolvedValue({ hybridEnabled: false });
@@ -1087,9 +1061,7 @@ describe("GitHistoryIndexer", () => {
 
       expect(stats.status).toBe("partial");
       expect(stats.errors).toBeDefined();
-      expect(stats.errors?.some((e) => e.includes("after 3 attempts"))).toBe(
-        true,
-      );
+      expect(stats.errors?.some((e) => e.includes("after 3 attempts"))).toBe(true);
     });
   });
 });