@mhalder/qdrant-mcp-server 3.3.3 → 3.3.4

package/src/embeddings/factory.test.ts

@@ -1,9 +1,9 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
-import { EmbeddingProviderFactory, type FactoryConfig } from "./factory.js";
-import { OpenAIEmbeddings } from "./openai.js";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { CohereEmbeddings } from "./cohere.js";
-import { VoyageEmbeddings } from "./voyage.js";
+import { EmbeddingProviderFactory } from "./factory.js";
 import { OllamaEmbeddings } from "./ollama.js";
+import { OpenAIEmbeddings } from "./openai.js";
+import { VoyageEmbeddings } from "./voyage.js";
 
 vi.mock("../logger.js", () => ({
   default: {
@@ -36,7 +36,7 @@ describe("EmbeddingProviderFactory", () => {
         expect(() =>
           EmbeddingProviderFactory.create({
             provider: "unknown" as any,
-          }),
+          })
         ).toThrow("Unknown embedding provider: unknown");
       });
 
@@ -44,7 +44,7 @@ describe("EmbeddingProviderFactory", () => {
         expect(() =>
           EmbeddingProviderFactory.create({
             provider: "invalid" as any,
-          }),
+          })
         ).toThrow("openai, cohere, voyage, ollama");
       });
     });
@@ -54,7 +54,7 @@ describe("EmbeddingProviderFactory", () => {
         expect(() =>
           EmbeddingProviderFactory.create({
             provider: "openai",
-          }),
+          })
         ).toThrow("API key is required for OpenAI provider");
       });
 
@@ -110,7 +110,7 @@ describe("EmbeddingProviderFactory", () => {
         expect(() =>
           EmbeddingProviderFactory.create({
             provider: "cohere",
-          }),
+          })
         ).toThrow("API key is required for Cohere provider");
       });
 
@@ -151,7 +151,7 @@ describe("EmbeddingProviderFactory", () => {
         expect(() =>
           EmbeddingProviderFactory.create({
             provider: "voyage",
-          }),
+          })
         ).toThrow("API key is required for Voyage AI provider");
       });
 
@@ -357,7 +357,7 @@ describe("EmbeddingProviderFactory", () => {
         process.env.EMBEDDING_DIMENSIONS = "not-a-number";
 
         expect(() => EmbeddingProviderFactory.createFromEnv()).toThrow(
-          'Invalid EMBEDDING_DIMENSIONS: must be a positive integer, got "not-a-number"',
+          'Invalid EMBEDDING_DIMENSIONS: must be a positive integer, got "not-a-number"'
         );
       });
 
@@ -367,7 +367,7 @@ describe("EmbeddingProviderFactory", () => {
         process.env.EMBEDDING_DIMENSIONS = "-100";
 
         expect(() => EmbeddingProviderFactory.createFromEnv()).toThrow(
-          'Invalid EMBEDDING_DIMENSIONS: must be a positive integer, got "-100"',
+          'Invalid EMBEDDING_DIMENSIONS: must be a positive integer, got "-100"'
         );
       });
 
@@ -377,7 +377,7 @@ describe("EmbeddingProviderFactory", () => {
         process.env.EMBEDDING_DIMENSIONS = "0";
 
         expect(() => EmbeddingProviderFactory.createFromEnv()).toThrow(
-          'Invalid EMBEDDING_DIMENSIONS: must be a positive integer, got "0"',
+          'Invalid EMBEDDING_DIMENSIONS: must be a positive integer, got "0"'
         );
       });
 
@@ -387,7 +387,7 @@ describe("EmbeddingProviderFactory", () => {
         process.env.EMBEDDING_MAX_REQUESTS_PER_MINUTE = "invalid";
 
         expect(() => EmbeddingProviderFactory.createFromEnv()).toThrow(
-          'Invalid EMBEDDING_MAX_REQUESTS_PER_MINUTE: must be a positive integer, got "invalid"',
+          'Invalid EMBEDDING_MAX_REQUESTS_PER_MINUTE: must be a positive integer, got "invalid"'
         );
       });
 
@@ -397,7 +397,7 @@ describe("EmbeddingProviderFactory", () => {
         process.env.EMBEDDING_MAX_REQUESTS_PER_MINUTE = "-50";
 
         expect(() => EmbeddingProviderFactory.createFromEnv()).toThrow(
-          'Invalid EMBEDDING_MAX_REQUESTS_PER_MINUTE: must be a positive integer, got "-50"',
+          'Invalid EMBEDDING_MAX_REQUESTS_PER_MINUTE: must be a positive integer, got "-50"'
         );
       });
 
@@ -407,7 +407,7 @@ describe("EmbeddingProviderFactory", () => {
         process.env.EMBEDDING_RETRY_ATTEMPTS = "abc";
 
         expect(() => EmbeddingProviderFactory.createFromEnv()).toThrow(
-          'Invalid EMBEDDING_RETRY_ATTEMPTS: must be a non-negative integer, got "abc"',
+          'Invalid EMBEDDING_RETRY_ATTEMPTS: must be a non-negative integer, got "abc"'
         );
       });
 
@@ -417,7 +417,7 @@ describe("EmbeddingProviderFactory", () => {
         process.env.EMBEDDING_RETRY_ATTEMPTS = "-5";
 
         expect(() => EmbeddingProviderFactory.createFromEnv()).toThrow(
-          'Invalid EMBEDDING_RETRY_ATTEMPTS: must be a non-negative integer, got "-5"',
+          'Invalid EMBEDDING_RETRY_ATTEMPTS: must be a non-negative integer, got "-5"'
         );
       });
 
@@ -427,7 +427,7 @@ describe("EmbeddingProviderFactory", () => {
         process.env.EMBEDDING_RETRY_DELAY = "xyz";
 
         expect(() => EmbeddingProviderFactory.createFromEnv()).toThrow(
-          'Invalid EMBEDDING_RETRY_DELAY: must be a non-negative integer, got "xyz"',
+          'Invalid EMBEDDING_RETRY_DELAY: must be a non-negative integer, got "xyz"'
         );
       });
 
@@ -437,7 +437,7 @@ describe("EmbeddingProviderFactory", () => {
         process.env.EMBEDDING_RETRY_DELAY = "-1000";
 
         expect(() => EmbeddingProviderFactory.createFromEnv()).toThrow(
-          'Invalid EMBEDDING_RETRY_DELAY: must be a non-negative integer, got "-1000"',
+          'Invalid EMBEDDING_RETRY_DELAY: must be a non-negative integer, got "-1000"'
         );
       });
 

package/src/embeddings/factory.ts

@@ -1,9 +1,9 @@
 import logger from "../logger.js";
-import { EmbeddingProvider, ProviderConfig } from "./base.js";
-import { OpenAIEmbeddings } from "./openai.js";
+import type { EmbeddingProvider, ProviderConfig } from "./base.js";
 import { CohereEmbeddings } from "./cohere.js";
-import { VoyageEmbeddings } from "./voyage.js";
 import { OllamaEmbeddings } from "./ollama.js";
+import { OpenAIEmbeddings } from "./openai.js";
+import { VoyageEmbeddings } from "./voyage.js";
 
 export type EmbeddingProviderType = "openai" | "cohere" | "voyage" | "ollama";
 
@@ -13,8 +13,7 @@ export interface FactoryConfig extends ProviderConfig {
 
 export class EmbeddingProviderFactory {
   static create(config: FactoryConfig): EmbeddingProvider {
-    const { provider, model, dimensions, rateLimitConfig, apiKey, baseUrl } =
-      config;
+    const { provider, model, dimensions, rateLimitConfig, apiKey, baseUrl } = config;
 
     logger.info({ provider, model }, "Creating embedding provider");
 
@@ -27,7 +26,7 @@ export class EmbeddingProviderFactory {
           apiKey,
           model || "text-embedding-3-small",
           dimensions,
-          rateLimitConfig,
+          rateLimitConfig
         );
 
       case "cohere":
@@ -38,7 +37,7 @@ export class EmbeddingProviderFactory {
           apiKey,
           model || "embed-english-v3.0",
           dimensions,
-          rateLimitConfig,
+          rateLimitConfig
         );
 
       case "voyage":
@@ -50,7 +49,7 @@ export class EmbeddingProviderFactory {
           model || "voyage-2",
           dimensions,
           rateLimitConfig,
-          baseUrl || "https://api.voyageai.com/v1",
+          baseUrl || "https://api.voyageai.com/v1"
         );
 
       case "ollama":
@@ -58,12 +57,12 @@ export class EmbeddingProviderFactory {
           model || "nomic-embed-text",
           dimensions,
           rateLimitConfig,
-          baseUrl || "http://localhost:11434",
+          baseUrl || "http://localhost:11434"
         );
 
       default:
         throw new Error(
-          `Unknown embedding provider: ${provider}. Supported providers: openai, cohere, voyage, ollama`,
+          `Unknown embedding provider: ${provider}. Supported providers: openai, cohere, voyage, ollama`
         );
     }
   }
@@ -97,9 +96,9 @@ export class EmbeddingProviderFactory {
       : undefined;
 
     // Validate dimensions
-    if (dimensions !== undefined && (isNaN(dimensions) || dimensions <= 0)) {
+    if (dimensions !== undefined && (Number.isNaN(dimensions) || dimensions <= 0)) {
       throw new Error(
-        `Invalid EMBEDDING_DIMENSIONS: must be a positive integer, got "${process.env.EMBEDDING_DIMENSIONS}"`,
+        `Invalid EMBEDDING_DIMENSIONS: must be a positive integer, got "${process.env.EMBEDDING_DIMENSIONS}"`
       );
     }
 
@@ -113,10 +112,10 @@ export class EmbeddingProviderFactory {
     // Validate maxRequestsPerMinute
     if (
       maxRequestsPerMinute !== undefined &&
-      (isNaN(maxRequestsPerMinute) || maxRequestsPerMinute <= 0)
+      (Number.isNaN(maxRequestsPerMinute) || maxRequestsPerMinute <= 0)
     ) {
       throw new Error(
-        `Invalid EMBEDDING_MAX_REQUESTS_PER_MINUTE: must be a positive integer, got "${process.env.EMBEDDING_MAX_REQUESTS_PER_MINUTE}"`,
+        `Invalid EMBEDDING_MAX_REQUESTS_PER_MINUTE: must be a positive integer, got "${process.env.EMBEDDING_MAX_REQUESTS_PER_MINUTE}"`
       );
     }
 
@@ -125,12 +124,9 @@ export class EmbeddingProviderFactory {
       : undefined;
 
     // Validate retryAttempts
-    if (
-      retryAttempts !== undefined &&
-      (isNaN(retryAttempts) || retryAttempts < 0)
-    ) {
+    if (retryAttempts !== undefined && (Number.isNaN(retryAttempts) || retryAttempts < 0)) {
       throw new Error(
-        `Invalid EMBEDDING_RETRY_ATTEMPTS: must be a non-negative integer, got "${process.env.EMBEDDING_RETRY_ATTEMPTS}"`,
+        `Invalid EMBEDDING_RETRY_ATTEMPTS: must be a non-negative integer, got "${process.env.EMBEDDING_RETRY_ATTEMPTS}"`
       );
     }
 
@@ -139,12 +135,9 @@ export class EmbeddingProviderFactory {
       : undefined;
 
     // Validate retryDelayMs
-    if (
-      retryDelayMs !== undefined &&
-      (isNaN(retryDelayMs) || retryDelayMs < 0)
-    ) {
+    if (retryDelayMs !== undefined && (Number.isNaN(retryDelayMs) || retryDelayMs < 0)) {
       throw new Error(
-        `Invalid EMBEDDING_RETRY_DELAY: must be a non-negative integer, got "${process.env.EMBEDDING_RETRY_DELAY}"`,
+        `Invalid EMBEDDING_RETRY_DELAY: must be a non-negative integer, got "${process.env.EMBEDDING_RETRY_DELAY}"`
       );
     }
 
@@ -154,7 +147,7 @@ export class EmbeddingProviderFactory {
       retryDelayMs,
     };
 
-    return this.create({
+    return EmbeddingProviderFactory.create({
       provider,
       model,
       dimensions,

package/src/embeddings/ollama.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
+import { beforeEach, describe, expect, it, vi } from "vitest";
 import { OllamaEmbeddings } from "./ollama.js";
 
 // Mock fetch globally
@@ -54,7 +54,7 @@ describe("OllamaEmbeddings", () => {
         "nomic-embed-text",
         undefined,
         undefined,
-        "http://custom:11434",
+        "http://custom:11434"
       );
       expect(customEmbeddings).toBeDefined();
     });
@@ -89,19 +89,16 @@ describe("OllamaEmbeddings", () => {
         embedding: mockEmbedding,
         dimensions: 768,
       });
-      expect(mockFetch).toHaveBeenCalledWith(
-        "http://localhost:11434/api/embeddings",
-        {
-          method: "POST",
-          headers: {
-            "Content-Type": "application/json",
-          },
-          body: JSON.stringify({
-            model: "nomic-embed-text",
-            prompt: "test text",
-          }),
+      expect(mockFetch).toHaveBeenCalledWith("http://localhost:11434/api/embeddings", {
+        method: "POST",
+        headers: {
+          "Content-Type": "application/json",
         },
-      );
+        body: JSON.stringify({
+          model: "nomic-embed-text",
+          prompt: "test text",
+        }),
+      });
     });
 
     it("should handle long text", async () => {
@@ -124,7 +121,7 @@ describe("OllamaEmbeddings", () => {
         "nomic-embed-text",
         undefined,
         undefined,
-        "http://custom:11434",
+        "http://custom:11434"
       );
 
       const mockEmbedding = Array(768).fill(0.1);
@@ -139,7 +136,7 @@ describe("OllamaEmbeddings", () => {
 
       expect(mockFetch).toHaveBeenCalledWith(
         "http://custom:11434/api/embeddings",
-        expect.any(Object),
+        expect.any(Object)
       );
     });
 
@@ -150,7 +147,7 @@ describe("OllamaEmbeddings", () => {
       });
 
       await expect(embeddings.embed("test")).rejects.toThrow(
-        "No embedding returned from Ollama API",
+        "No embedding returned from Ollama API"
       );
     });
 
@@ -181,7 +178,7 @@ describe("OllamaEmbeddings", () => {
       await expect(embeddings.embed(longText)).rejects.toThrow(
         expect.objectContaining({
           message: expect.stringContaining("Text preview:"),
-        }),
+        })
       );
     });
 
@@ -189,7 +186,7 @@ describe("OllamaEmbeddings", () => {
       mockFetch.mockRejectedValue("Connection refused");
 
       await expect(embeddings.embed("test")).rejects.toThrow(
-        "Failed to call Ollama API at http://localhost:11434 with model nomic-embed-text",
+        "Failed to call Ollama API at http://localhost:11434 with model nomic-embed-text"
       );
     });
 
@@ -198,27 +195,21 @@ describe("OllamaEmbeddings", () => {
         message: "Custom error message",
       });
 
-      await expect(embeddings.embed("test")).rejects.toThrow(
-        "Custom error message",
-      );
+      await expect(embeddings.embed("test")).rejects.toThrow("Custom error message");
     });
 
     it("should handle non-Error objects in catch block", async () => {
       mockFetch.mockRejectedValue({ code: "ERR_UNKNOWN", details: "info" });
 
       await expect(embeddings.embed("test")).rejects.toThrow(
-        "Failed to call Ollama API at http://localhost:11434 with model nomic-embed-text",
+        "Failed to call Ollama API at http://localhost:11434 with model nomic-embed-text"
       );
     });
   });
 
   describe("embedBatch", () => {
     it("should generate embeddings for multiple texts in parallel", async () => {
-      const mockEmbeddings = [
-        Array(768).fill(0.1),
-        Array(768).fill(0.2),
-        Array(768).fill(0.3),
-      ];
+      const mockEmbeddings = [Array(768).fill(0.1), Array(768).fill(0.2), Array(768).fill(0.3)];
 
       // Mock sequential calls for each text
       mockEmbeddings.forEach((embedding) => {
@@ -290,9 +281,7 @@ describe("OllamaEmbeddings", () => {
         })
         .mockRejectedValueOnce(new Error("Batch API Error"));
 
-      await expect(embeddings.embedBatch(["text1", "text2"])).rejects.toThrow(
-        "Batch API Error",
-      );
+      await expect(embeddings.embedBatch(["text1", "text2"])).rejects.toThrow("Batch API Error");
     });
 
     it("should handle partial failures in batch", async () => {
@@ -378,14 +367,10 @@ describe("OllamaEmbeddings", () => {
     });
 
     it("should use exponential backoff with faster default delay", async () => {
-      const rateLimitEmbeddings = new OllamaEmbeddings(
-        "nomic-embed-text",
-        undefined,
-        {
-          retryAttempts: 3,
-          retryDelayMs: 100,
-        },
-      );
+      const rateLimitEmbeddings = new OllamaEmbeddings("nomic-embed-text", undefined, {
+        retryAttempts: 3,
+        retryDelayMs: 100,
+      });
 
       const mockEmbedding = Array(768).fill(0.5);
 
@@ -414,14 +399,10 @@ describe("OllamaEmbeddings", () => {
     });
 
     it("should throw error after max retries exceeded", async () => {
-      const rateLimitEmbeddings = new OllamaEmbeddings(
-        "nomic-embed-text",
-        undefined,
-        {
-          retryAttempts: 2,
-          retryDelayMs: 100,
-        },
-      );
+      const rateLimitEmbeddings = new OllamaEmbeddings("nomic-embed-text", undefined, {
+        retryAttempts: 2,
+        retryDelayMs: 100,
+      });
 
       mockFetch.mockResolvedValue({
         ok: false,
@@ -430,7 +411,7 @@ describe("OllamaEmbeddings", () => {
       });
 
       await expect(rateLimitEmbeddings.embed("test text")).rejects.toThrow(
-        "Ollama API rate limit exceeded after 2 retry attempts",
+        "Ollama API rate limit exceeded after 2 retry attempts"
       );
 
       expect(mockFetch).toHaveBeenCalledTimes(3);
@@ -473,15 +454,11 @@ describe("OllamaEmbeddings", () => {
     });
 
     it("should accept custom rate limit configuration", () => {
-      const customEmbeddings = new OllamaEmbeddings(
-        "nomic-embed-text",
-        undefined,
-        {
-          maxRequestsPerMinute: 2000,
-          retryAttempts: 5,
-          retryDelayMs: 1000,
-        },
-      );
+      const customEmbeddings = new OllamaEmbeddings("nomic-embed-text", undefined, {
+        maxRequestsPerMinute: 2000,
+        retryAttempts: 5,
+        retryDelayMs: 1000,
+      });
 
       expect(customEmbeddings).toBeDefined();
     });
@@ -503,9 +480,7 @@ describe("OllamaEmbeddings", () => {
     it("should handle string primitive errors", async () => {
       mockFetch.mockRejectedValue("Network unreachable");
 
-      await expect(embeddings.embed("test")).rejects.toThrow(
-        "Network unreachable",
-      );
+      await expect(embeddings.embed("test")).rejects.toThrow("Network unreachable");
     });
 
     it("should handle error objects with non-string message property", async () => {
@@ -523,9 +498,7 @@ describe("OllamaEmbeddings", () => {
       const testError = new Error("Connection timeout");
       mockFetch.mockRejectedValue(testError);
 
-      await expect(embeddings.embed("test")).rejects.toThrow(
-        "Connection timeout",
-      );
+      await expect(embeddings.embed("test")).rejects.toThrow("Connection timeout");
     });
 
     it("should handle Error instance from network error with enhanced message", async () => {
@@ -534,7 +507,7 @@ describe("OllamaEmbeddings", () => {
       mockFetch.mockRejectedValue(networkError);
 
       await expect(embeddings.embed("test")).rejects.toThrow(
-        "Failed to call Ollama API at http://localhost:11434 with model nomic-embed-text: ECONNREFUSED. Text preview:",
+        "Failed to call Ollama API at http://localhost:11434 with model nomic-embed-text: ECONNREFUSED. Text preview:"
       );
     });
 
@@ -547,9 +520,7 @@ describe("OllamaEmbeddings", () => {
       };
       mockFetch.mockRejectedValue(customError);
 
-      await expect(embeddings.embed("test")).rejects.toThrow(
-        "Custom API failure",
-      );
+      await expect(embeddings.embed("test")).rejects.toThrow("Custom API failure");
     });
   });
 });

package/src/embeddings/ollama.ts

@@ -1,6 +1,6 @@
 import Bottleneck from "bottleneck";
 import logger from "../logger.js";
-import { EmbeddingProvider, EmbeddingResult, RateLimitConfig } from "./base.js";
+import type { EmbeddingProvider, EmbeddingResult, RateLimitConfig } from "./base.js";
 
 interface OllamaError {
   status?: number;
@@ -24,7 +24,7 @@ export class OllamaEmbeddings implements EmbeddingProvider {
     model: string = "nomic-embed-text",
     dimensions?: number,
     rateLimitConfig?: RateLimitConfig,
-    baseUrl: string = "http://localhost:11434",
+    baseUrl: string = "http://localhost:11434"
   ) {
     this.model = model;
     this.baseUrl = baseUrl;
@@ -53,22 +53,15 @@ export class OllamaEmbeddings implements EmbeddingProvider {
   }
 
   private isOllamaError(e: unknown): e is OllamaError {
-    return (
-      typeof e === "object" && e !== null && ("status" in e || "message" in e)
-    );
+    return typeof e === "object" && e !== null && ("status" in e || "message" in e);
   }
 
-  private async retryWithBackoff<T>(
-    fn: () => Promise<T>,
-    attempt: number = 0,
-  ): Promise<T> {
+  private async retryWithBackoff<T>(fn: () => Promise<T>, attempt: number = 0): Promise<T> {
     try {
       return await fn();
     } catch (error: unknown) {
       // Type guard for OllamaError
-      const apiError = this.isOllamaError(error)
-        ? error
-        : { status: 0, message: String(error) };
+      const apiError = this.isOllamaError(error) ? error : { status: 0, message: String(error) };
 
       const isRateLimitError =
         apiError.status === 429 ||
@@ -76,7 +69,7 @@ export class OllamaEmbeddings implements EmbeddingProvider {
           apiError.message.toLowerCase().includes("rate limit"));
 
       if (isRateLimitError && attempt < this.retryAttempts) {
-        const delayMs = this.retryDelayMs * Math.pow(2, attempt);
+        const delayMs = this.retryDelayMs * 2 ** attempt;
         const waitTimeSeconds = (delayMs / 1000).toFixed(1);
         this.log.warn(
           {
@@ -84,7 +77,7 @@ export class OllamaEmbeddings implements EmbeddingProvider {
             attempt: attempt + 1,
             maxAttempts: this.retryAttempts,
           },
-          "Rate limit reached, retrying",
+          "Rate limit reached, retrying"
         );
 
         await new Promise((resolve) => setTimeout(resolve, delayMs));
@@ -93,7 +86,7 @@ export class OllamaEmbeddings implements EmbeddingProvider {
 
       if (isRateLimitError) {
         throw new Error(
-          `Ollama API rate limit exceeded after ${this.retryAttempts} retry attempts. Please try again later or reduce request frequency.`,
+          `Ollama API rate limit exceeded after ${this.retryAttempts} retry attempts. Please try again later or reduce request frequency.`
         );
       }
 
@@ -116,8 +109,7 @@ export class OllamaEmbeddings implements EmbeddingProvider {
 
       if (!response.ok) {
         const errorBody = await response.text();
-        const textPreview =
-          text.length > 100 ? text.substring(0, 100) + "..." : text;
+        const textPreview = text.length > 100 ? `${text.substring(0, 100)}...` : text;
         const error: OllamaError = {
           status: response.status,
           message: `Ollama API error (${response.status}) for model "${this.model}": ${errorBody}. Text preview: "${textPreview}"`,
@@ -134,10 +126,9 @@ export class OllamaEmbeddings implements EmbeddingProvider {
 
       // For Error instances (like network errors), enhance the message
       if (error instanceof Error) {
-        const textPreview =
-          text.length > 100 ? text.substring(0, 100) + "..." : text;
+        const textPreview = text.length > 100 ? `${text.substring(0, 100)}...` : text;
         throw new Error(
-          `Failed to call Ollama API at ${this.baseUrl} with model ${this.model}: ${error.message}. Text preview: "${textPreview}"`,
+          `Failed to call Ollama API at ${this.baseUrl} with model ${this.model}: ${error.message}. Text preview: "${textPreview}"`
         );
       }
 
@@ -148,12 +139,11 @@ export class OllamaEmbeddings implements EmbeddingProvider {
       }
 
       // For other types, create a descriptive error message
-      const textPreview =
-        text.length > 100 ? text.substring(0, 100) + "..." : text;
+      const textPreview = text.length > 100 ? `${text.substring(0, 100)}...` : text;
       const errorMessage = JSON.stringify(error);
 
       throw new Error(
-        `Failed to call Ollama API at ${this.baseUrl} with model ${this.model}: ${errorMessage}. Text preview: "${textPreview}"`,
+        `Failed to call Ollama API at ${this.baseUrl} with model ${this.model}: ${errorMessage}. Text preview: "${textPreview}"`
       );
     }
   }
@@ -171,7 +161,7 @@ export class OllamaEmbeddings implements EmbeddingProvider {
           embedding: response.embedding,
           dimensions: this.dimensions,
         };
-      }),
+      })
     );
   }
 
@@ -185,9 +175,7 @@ export class OllamaEmbeddings implements EmbeddingProvider {
     for (let i = 0; i < texts.length; i += CHUNK_SIZE) {
       const chunk = texts.slice(i, i + CHUNK_SIZE);
       // The Bottleneck limiter will handle rate limiting and concurrency (maxConcurrent: 10)
-      const chunkResults = await Promise.all(
-        chunk.map((text) => this.embed(text)),
-      );
+      const chunkResults = await Promise.all(chunk.map((text) => this.embed(text)));
       results.push(...chunkResults);
     }