fix(embeddings): address code review feedback - remove eager export, fix OLLAMA_HOST, add embeddinggemma

Author: rothnic

src/embeddings/index.ts

@@ -34,6 +34,8 @@ export function getConfiguredDimensions(config: Partial<EmbeddingConfig> = {}):
     const ollamaDimensions: Record<string, number> = {
       'nomic-embed-text': 768,
       'nomic-embed-text:latest': 768,
+      embeddinggemma: 768,
+      'embeddinggemma:latest': 768,
       'mxbai-embed-large': 1024,
       'mxbai-embed-large:latest': 1024,
       'all-minilm': 384,
@@ -73,9 +75,11 @@ export async function getEmbeddingProvider(
 
   if (mergedConfig.provider === 'ollama') {
     const { OllamaEmbeddingProvider } = await import('./ollama.js');
+    const endpoint =
+      mergedConfig.apiEndpoint || process.env.OLLAMA_HOST || 'http://localhost:11434';
     const provider = new OllamaEmbeddingProvider(
       mergedConfig.model || 'nomic-embed-text',
-      mergedConfig.apiEndpoint || 'http://localhost:11434'
+      endpoint
     );
     await provider.initialize();
     cachedProvider = provider;
@@ -93,6 +97,6 @@ export async function getEmbeddingProvider(
   return provider;
 }
 
-// Re-export TransformersEmbeddingProvider and MODEL_CONFIGS for consumers who need them
-// These will trigger transformers loading, but only when explicitly imported
-export { TransformersEmbeddingProvider, MODEL_CONFIGS } from './transformers.js';
+// Note: transformers provider is lazy-loaded in getEmbeddingProvider to avoid
+// eager heavy dependency loading. Consumers should import from './transformers'
+// directly if they need access to provider implementation or MODEL_CONFIGS.

src/embeddings/ollama.ts

@@ -8,6 +8,8 @@ interface OllamaEmbeddingResponse {
 const MODEL_CONTEXT_WINDOWS: Record<string, number> = {
   'nomic-embed-text': 2048,
   'nomic-embed-text:latest': 2048,
+  embeddinggemma: 2048,
+  'embeddinggemma:latest': 2048,
   'mxbai-embed-large': 512,
   'mxbai-embed-large:latest': 512,
   'all-minilm': 512,
@@ -37,6 +39,8 @@ export class OllamaEmbeddingProvider implements EmbeddingProvider {
     const modelDimensions: Record<string, number> = {
       'nomic-embed-text': 768,
       'nomic-embed-text:latest': 768,
+      embeddinggemma: 768,
+      'embeddinggemma:latest': 768,
       'mxbai-embed-large': 1024,
       'mxbai-embed-large:latest': 1024,
       'all-minilm': 384,