bluera-knowledge 0.13.3 → 0.14.1

package/src/services/search.service.test.ts

@@ -1,2263 +0,0 @@
-import { describe, it, expect, beforeAll, afterAll, vi, beforeEach } from 'vitest';
-import { SearchService } from './search.service.js';
-import { LanceStore } from '../db/lance.js';
-import { EmbeddingEngine } from '../db/embeddings.js';
-import { createStoreId, createDocumentId } from '../types/brands.js';
-import { rm, mkdtemp } from 'node:fs/promises';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import type { SearchResult } from '../types/search.js';
-
-describe('SearchService', () => {
-  let searchService: SearchService;
-  let lanceStore: LanceStore;
-  let embeddingEngine: EmbeddingEngine;
-  let tempDir: string;
-  const storeId = createStoreId('test-store');
-
-  beforeAll(async () => {
-    tempDir = await mkdtemp(join(tmpdir(), 'search-test-'));
-    lanceStore = new LanceStore(tempDir);
-    embeddingEngine = new EmbeddingEngine();
-
-    await embeddingEngine.initialize();
-    await lanceStore.initialize(storeId);
-
-    // Add test documents
-    const texts = [
-      'TypeScript is a typed superset of JavaScript',
-      'Python is great for machine learning',
-      'React is a JavaScript library for building user interfaces',
-    ];
-
-    for (let i = 0; i < texts.length; i++) {
-      const text = texts[i]!;
-      const vector = await embeddingEngine.embed(text);
-      await lanceStore.addDocuments(storeId, [
-        {
-          id: createDocumentId(`doc-${i}`),
-          content: text,
-          vector,
-          metadata: {
-            type: 'file',
-            storeId,
-            indexedAt: new Date(),
-          },
-        },
-      ]);
-    }
-
-    searchService = new SearchService(lanceStore, embeddingEngine);
-  }, 120000);
-
-  afterAll(async () => {
-    await rm(tempDir, { recursive: true, force: true });
-  });
-
-  it('searches with vector mode', async () => {
-    const results = await searchService.search({
-      query: 'JavaScript programming',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBeGreaterThan(0);
-    expect(results.mode).toBe('vector');
-  });
-
-  it('returns results with scores', async () => {
-    const results = await searchService.search({
-      query: 'machine learning Python',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-    });
-
-    expect(results.results[0]?.score).toBeGreaterThan(0);
-    expect(results.results[0]?.content).toContain('Python');
-  });
-
-  it('performs hybrid search combining vector and FTS', async () => {
-    // First create FTS index
-    await lanceStore.createFtsIndex(storeId);
-
-    const results = await searchService.search({
-      query: 'JavaScript programming',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.mode).toBe('hybrid');
-    expect(results.results.length).toBeGreaterThan(0);
-    // Hybrid should have RRF scores that differ from pure vector scores
-    expect(results.results[0]?.score).toBeGreaterThan(0);
-  });
-});
-
-describe('SearchService - RRF Ranking Algorithm', () => {
-  let mockLanceStore: LanceStore;
-  let mockEmbeddingEngine: EmbeddingEngine;
-  let searchService: SearchService;
-  const storeId = createStoreId('test-store');
-
-  beforeEach(() => {
-    mockLanceStore = {
-      search: vi.fn(),
-      fullTextSearch: vi.fn(),
-    } as unknown as LanceStore;
-
-    mockEmbeddingEngine = {
-      embed: vi.fn().mockResolvedValue([0.1, 0.2, 0.3]),
-    } as unknown as EmbeddingEngine;
-
-    searchService = new SearchService(mockLanceStore, mockEmbeddingEngine);
-  });
-
-  it('combines vector and FTS results with RRF', async () => {
-    const vectorResults = [
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'result 1',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.8,
-        content: 'result 2',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ];
-
-    const ftsResults = [
-      {
-        id: createDocumentId('doc2'),
-        score: 0.95,
-        content: 'result 2',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc3'),
-        score: 0.85,
-        content: 'result 3',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ];
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue(vectorResults);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue(ftsResults);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBeGreaterThan(0);
-    // doc2 should rank higher as it appears in both results
-    expect(results.results.some((r) => r.id === createDocumentId('doc2'))).toBe(true);
-  });
-
-  it('uses web RRF preset for web content (url metadata)', async () => {
-    // Web content has url in metadata - should use web preset (k=30)
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'result 1',
-        metadata: {
-          type: 'web' as const,
-          storeId,
-          indexedAt: new Date(),
-          url: 'https://example.com/docs',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.95,
-        content: 'result 1',
-        metadata: {
-          type: 'web' as const,
-          storeId,
-          indexedAt: new Date(),
-          url: 'https://example.com/docs',
-        },
-      },
-    ]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBe(1);
-    expect(results.results[0]?.score).toBeGreaterThan(0);
-  });
-
-  it('uses code RRF preset for file content (path metadata)', async () => {
-    // File content has path, no url - should use code preset (k=20)
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'function test() {}',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date(), path: '/src/test.ts' },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([
-      {
-        id: createDocumentId('doc2'),
-        score: 0.95,
-        content: 'class Example {}',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/example.ts',
-        },
-      },
-    ]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBeGreaterThan(0);
-  });
-
-  it('handles documents appearing only in vector results', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'vector only',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBe(1);
-    expect(results.results[0]?.id).toBe(createDocumentId('doc1'));
-  });
-
-  it('handles documents appearing only in FTS results', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'fts only',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBe(1);
-    expect(results.results[0]?.id).toBe(createDocumentId('doc1'));
-  });
-
-  it('normalizes scores to 0-1 range', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'result 1',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.5,
-        content: 'result 2',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.95,
-        content: 'result 1',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    // Scores should be normalized
-    results.results.forEach((r) => {
-      expect(r.score).toBeGreaterThanOrEqual(0);
-      expect(r.score).toBeLessThanOrEqual(1);
-    });
-  });
-
-  it('handles identical scores correctly', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'result 1',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.9,
-        content: 'result 2',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBe(2);
-    // Should handle identical scores without errors
-    expect(results.results[0]?.score).toBeDefined();
-  });
-});
-
-describe('SearchService - Query Intent Classification', () => {
-  let mockLanceStore: LanceStore;
-  let mockEmbeddingEngine: EmbeddingEngine;
-  let searchService: SearchService;
-  const storeId = createStoreId('test-store');
-
-  beforeEach(() => {
-    mockLanceStore = {
-      search: vi.fn(),
-      fullTextSearch: vi.fn(),
-    } as unknown as LanceStore;
-
-    mockEmbeddingEngine = {
-      embed: vi.fn().mockResolvedValue([0.1, 0.2, 0.3]),
-    } as unknown as EmbeddingEngine;
-
-    searchService = new SearchService(mockLanceStore, mockEmbeddingEngine);
-  });
-
-  it('classifies "how to" queries correctly', async () => {
-    const howToQueries = [
-      'how to use React hooks',
-      'how do I create a component',
-      'show me how to configure',
-      "what's the best way to implement",
-    ];
-
-    for (const query of howToQueries) {
-      vi.mocked(mockLanceStore.search).mockResolvedValue([
-        {
-          id: createDocumentId('doc1'),
-          score: 0.9,
-          content: 'example code',
-          metadata: {
-            type: 'file' as const,
-            storeId,
-            indexedAt: new Date(),
-            fileType: 'example',
-          },
-        },
-      ]);
-      vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-      const results = await searchService.search({
-        query,
-        stores: [storeId],
-        mode: 'hybrid',
-        limit: 10,
-      });
-
-      // Should boost example files for how-to queries
-      expect(results.results.length).toBeGreaterThan(0);
-    }
-  });
-
-  it('classifies "implementation" queries correctly', async () => {
-    const implQueries = [
-      'how is React implemented',
-      'source code for useState',
-      'implementation details of router',
-      'under the hood of Vue',
-    ];
-
-    for (const query of implQueries) {
-      vi.mocked(mockLanceStore.search).mockResolvedValue([
-        {
-          id: createDocumentId('doc1'),
-          score: 0.9,
-          content: 'internal implementation',
-          metadata: {
-            type: 'file' as const,
-            storeId,
-            indexedAt: new Date(),
-            fileType: 'source-internal',
-          },
-        },
-      ]);
-      vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-      const results = await searchService.search({
-        query,
-        stores: [storeId],
-        mode: 'hybrid',
-        limit: 10,
-      });
-
-      expect(results.results.length).toBeGreaterThan(0);
-    }
-  });
-
-  it('classifies "conceptual" queries correctly', async () => {
-    const conceptQueries = [
-      'what is a React hook',
-      'explain virtual DOM',
-      'what does reconciliation mean',
-      'how does routing work',
-    ];
-
-    for (const query of conceptQueries) {
-      vi.mocked(mockLanceStore.search).mockResolvedValue([
-        {
-          id: createDocumentId('doc1'),
-          score: 0.9,
-          content: 'concept explanation',
-          metadata: {
-            type: 'file' as const,
-            storeId,
-            indexedAt: new Date(),
-            fileType: 'documentation',
-          },
-        },
-      ]);
-      vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-      const results = await searchService.search({
-        query,
-        stores: [storeId],
-        mode: 'hybrid',
-        limit: 10,
-      });
-
-      expect(results.results.length).toBeGreaterThan(0);
-    }
-  });
-
-  it('classifies "comparison" queries correctly', async () => {
-    const comparisonQueries = [
-      'React vs Vue',
-      'difference between hooks and classes',
-      'should I use Redux or Context',
-      'when to use useMemo',
-    ];
-
-    for (const query of comparisonQueries) {
-      vi.mocked(mockLanceStore.search).mockResolvedValue([
-        {
-          id: createDocumentId('doc1'),
-          score: 0.9,
-          content: 'comparison guide',
-          metadata: {
-            type: 'file' as const,
-            storeId,
-            indexedAt: new Date(),
-            fileType: 'documentation-primary',
-          },
-        },
-      ]);
-      vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-      const results = await searchService.search({
-        query,
-        stores: [storeId],
-        mode: 'hybrid',
-        limit: 10,
-      });
-
-      expect(results.results.length).toBeGreaterThan(0);
-    }
-  });
-
-  it('classifies "debugging" queries correctly', async () => {
-    const debugQueries = [
-      'error: cannot read property',
-      'React component not rendering',
-      'why is my hook not working',
-      'how to fix TypeScript error',
-    ];
-
-    for (const query of debugQueries) {
-      vi.mocked(mockLanceStore.search).mockResolvedValue([
-        {
-          id: createDocumentId('doc1'),
-          score: 0.9,
-          content: 'troubleshooting guide',
-          metadata: {
-            type: 'file' as const,
-            storeId,
-            indexedAt: new Date(),
-            fileType: 'test',
-          },
-        },
-      ]);
-      vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-      const results = await searchService.search({
-        query,
-        stores: [storeId],
-        mode: 'hybrid',
-        limit: 10,
-      });
-
-      expect(results.results.length).toBeGreaterThan(0);
-    }
-  });
-
-  it('defaults to "how-to" for ambiguous queries', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'generic content',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          fileType: 'documentation',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'React components',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBeGreaterThan(0);
-  });
-});
-
-describe('SearchService - Framework Context Boosting', () => {
-  let mockLanceStore: LanceStore;
-  let mockEmbeddingEngine: EmbeddingEngine;
-  let searchService: SearchService;
-  const storeId = createStoreId('test-store');
-
-  beforeEach(() => {
-    mockLanceStore = {
-      search: vi.fn(),
-      fullTextSearch: vi.fn(),
-    } as unknown as LanceStore;
-
-    mockEmbeddingEngine = {
-      embed: vi.fn().mockResolvedValue([0.1, 0.2, 0.3]),
-    } as unknown as EmbeddingEngine;
-
-    searchService = new SearchService(mockLanceStore, mockEmbeddingEngine);
-  });
-
-  it('boosts React-related results when query mentions React', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('react-doc'),
-        score: 0.8,
-        content: 'React component documentation',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/docs/react/components.md',
-        },
-      },
-      {
-        id: createDocumentId('vue-doc'),
-        score: 0.8,
-        content: 'Vue component documentation',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/docs/vue/components.md',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'React hooks usage',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    // React doc should rank higher
-    expect(results.results[0]?.id).toBe(createDocumentId('react-doc'));
-  });
-
-  it('recognizes Express framework patterns', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('express-doc'),
-        score: 0.8,
-        content: 'Express middleware guide',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/node_modules/express/README.md',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'Express routing',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBeGreaterThan(0);
-  });
-
-  it('recognizes Vue framework patterns', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('vue-doc'),
-        score: 0.8,
-        content: 'Vue3 composition API',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/docs/vue3/api.md',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'Vue computed properties',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBeGreaterThan(0);
-  });
-
-  it('recognizes TypeScript framework patterns', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('ts-doc'),
-        score: 0.8,
-        content: 'TypeScript generics guide',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/docs/typescript/generics.md',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'TypeScript type inference',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBeGreaterThan(0);
-  });
-
-  it('handles queries without framework mentions', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.8,
-        content: 'generic programming guide',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/docs/general.md',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'programming patterns',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBeGreaterThan(0);
-  });
-});
-
-describe('SearchService - Deduplication Logic', () => {
-  let mockLanceStore: LanceStore;
-  let mockEmbeddingEngine: EmbeddingEngine;
-  let searchService: SearchService;
-  const storeId = createStoreId('test-store');
-
-  beforeEach(() => {
-    mockLanceStore = {
-      search: vi.fn(),
-      fullTextSearch: vi.fn(),
-    } as unknown as LanceStore;
-
-    mockEmbeddingEngine = {
-      embed: vi.fn().mockResolvedValue([0.1, 0.2, 0.3]),
-    } as unknown as EmbeddingEngine;
-
-    searchService = new SearchService(mockLanceStore, mockEmbeddingEngine);
-  });
-
-  it('deduplicates chunks from same source file', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('chunk1'),
-        score: 0.9,
-        content: 'first chunk with query term',
-        metadata: {
-          type: 'chunk' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/file.ts',
-          chunkIndex: 0,
-          totalChunks: 2,
-        },
-      },
-      {
-        id: createDocumentId('chunk2'),
-        score: 0.85,
-        content: 'second chunk with query term',
-        metadata: {
-          type: 'chunk' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/file.ts',
-          chunkIndex: 1,
-          totalChunks: 2,
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'query term',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    // Should only return one result per source file
-    expect(results.results.length).toBe(1);
-  });
-
-  it('keeps chunk with more query term matches', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('chunk1'),
-        score: 0.9,
-        content: 'test content',
-        metadata: {
-          type: 'chunk' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/file.ts',
-          chunkIndex: 0,
-          totalChunks: 2,
-        },
-      },
-      {
-        id: createDocumentId('chunk2'),
-        score: 0.85,
-        content: 'test content with React hooks and components',
-        metadata: {
-          type: 'chunk' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/file.ts',
-          chunkIndex: 1,
-          totalChunks: 2,
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'React hooks components',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    // Should keep chunk2 as it has more query term matches
-    expect(results.results[0]?.id).toBe(createDocumentId('chunk2'));
-  });
-
-  it('keeps different source files separate', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('file1'),
-        score: 0.9,
-        content: 'content from file 1',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/file1.ts',
-        },
-      },
-      {
-        id: createDocumentId('file2'),
-        score: 0.85,
-        content: 'content from file 2',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/file2.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    // Should keep both results as they're from different files
-    expect(results.results.length).toBe(2);
-  });
-});
-
-describe('SearchService - Progressive Context Enhancement', () => {
-  let mockLanceStore: LanceStore;
-  let mockEmbeddingEngine: EmbeddingEngine;
-  let searchService: SearchService;
-  const storeId = createStoreId('test-store');
-
-  beforeEach(() => {
-    mockLanceStore = {
-      search: vi.fn(),
-      fullTextSearch: vi.fn(),
-    } as unknown as LanceStore;
-
-    mockEmbeddingEngine = {
-      embed: vi.fn().mockResolvedValue([0.1, 0.2, 0.3]),
-    } as unknown as EmbeddingEngine;
-
-    searchService = new SearchService(mockLanceStore, mockEmbeddingEngine);
-  });
-
-  it('provides minimal detail level by default', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'function example() { return 42; }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/example.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'example function',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-    });
-
-    // Should have summary but not context or full
-    expect(results.results[0]?.summary).toBeDefined();
-    expect(results.results[0]?.context).toBeUndefined();
-    expect(results.results[0]?.full).toBeUndefined();
-  });
-
-  it('provides contextual detail when requested', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'import React from "react";\n\nfunction Component() { return <div>Test</div>; }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/Component.tsx',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'Component',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'contextual',
-    });
-
-    // Should have summary and context but not full
-    expect(results.results[0]?.summary).toBeDefined();
-    expect(results.results[0]?.context).toBeDefined();
-    expect(results.results[0]?.context?.keyImports).toBeDefined();
-    expect(results.results[0]?.full).toBeUndefined();
-  });
-
-  it('provides full detail when requested', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: '/** Documentation */\nfunction example() { return 42; }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/example.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'example',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'full',
-    });
-
-    // Should have summary, context, and full
-    expect(results.results[0]?.summary).toBeDefined();
-    expect(results.results[0]?.context).toBeDefined();
-    expect(results.results[0]?.full).toBeDefined();
-    expect(results.results[0]?.full?.completeCode).toBeDefined();
-  });
-
-  it('extracts function names correctly', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'export async function fetchData() { return data; }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/api.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'fetchData',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-    });
-
-    expect(results.results[0]?.summary?.name).toBe('fetchData');
-  });
-
-  it('extracts class names correctly', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'export class MyComponent { constructor() {} }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/Component.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'MyComponent',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-    });
-
-    expect(results.results[0]?.summary?.name).toBe('MyComponent');
-  });
-});
-
-describe('SearchService - Edge Cases', () => {
-  let mockLanceStore: LanceStore;
-  let mockEmbeddingEngine: EmbeddingEngine;
-  let searchService: SearchService;
-  const storeId = createStoreId('test-store');
-
-  beforeEach(() => {
-    mockLanceStore = {
-      search: vi.fn(),
-      fullTextSearch: vi.fn(),
-    } as unknown as LanceStore;
-
-    mockEmbeddingEngine = {
-      embed: vi.fn().mockResolvedValue([0.1, 0.2, 0.3]),
-    } as unknown as EmbeddingEngine;
-
-    searchService = new SearchService(mockLanceStore, mockEmbeddingEngine);
-  });
-
-  it('handles empty query gracefully', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: '',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBe(0);
-  });
-
-  it('handles no results gracefully', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'nonexistent term',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBe(0);
-    expect(results.totalResults).toBe(0);
-  });
-
-  it('respects limit parameter', async () => {
-    const manyResults = Array.from({ length: 20 }, (_, i) => ({
-      id: createDocumentId(`doc${i}`),
-      score: 0.9 - i * 0.01,
-      content: `result ${i}`,
-      metadata: {
-        type: 'file' as const,
-        storeId,
-        indexedAt: new Date(),
-        path: `/src/file${i}.ts`,
-      },
-    }));
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue(manyResults);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 5,
-    });
-
-    expect(results.results.length).toBeLessThanOrEqual(5);
-  });
-
-  it('handles multiple stores correctly', async () => {
-    const store1 = createStoreId('store1');
-    const store2 = createStoreId('store2');
-
-    vi.mocked(mockLanceStore.search).mockImplementation(async (storeId) => {
-      if (storeId === store1) {
-        return [
-          {
-            id: createDocumentId('doc1'),
-            score: 0.9,
-            content: 'from store 1',
-            metadata: { type: 'file' as const, storeId: store1, indexedAt: new Date() },
-          },
-        ];
-      }
-      return [
-        {
-          id: createDocumentId('doc2'),
-          score: 0.8,
-          content: 'from store 2',
-          metadata: { type: 'file' as const, storeId: store2, indexedAt: new Date() },
-        },
-      ];
-    });
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test',
-      stores: [store1, store2],
-      mode: 'vector',
-      limit: 10,
-    });
-
-    expect(results.results.length).toBe(2);
-  });
-
-  it('returns timing information', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-    });
-
-    expect(results.timeMs).toBeGreaterThanOrEqual(0);
-  });
-
-  it('handles threshold parameter for vector search', async () => {
-    // Setup: multiple results with varying scores
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.95,
-        content: 'high score',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.5,
-        content: 'medium score',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc3'),
-        score: 0.3,
-        content: 'low score',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-
-    // Search with high threshold
-    const results = await searchService.search({
-      query: 'test',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      threshold: 0.8,
-    });
-
-    // Verify lanceStore.search was called (without checking threshold param - it's unused in LanceStore)
-    expect(vi.mocked(mockLanceStore.search)).toHaveBeenCalledWith(
-      storeId,
-      expect.anything(),
-      expect.anything()
-    );
-
-    // Verify threshold filtering works: only high-score result should pass
-    // After normalization, the top result has score 1.0, and threshold 0.8 filters out lower ones
-    // The normalized scores are: doc1=1.0, doc2=0.31, doc3=0.0 (relative to max 0.95)
-    expect(results.results.length).toBe(1);
-    expect(results.results[0]?.id).toBe('doc1');
-  });
-});
-
-describe('SearchService - Path Keyword Boosting', () => {
-  let mockLanceStore: LanceStore;
-  let mockEmbeddingEngine: EmbeddingEngine;
-  let searchService: SearchService;
-  const storeId = createStoreId('test-store');
-
-  beforeEach(() => {
-    mockLanceStore = {
-      search: vi.fn(),
-      fullTextSearch: vi.fn(),
-    } as unknown as LanceStore;
-
-    mockEmbeddingEngine = {
-      embed: vi.fn().mockResolvedValue([0.1, 0.2, 0.3]),
-    } as unknown as EmbeddingEngine;
-
-    searchService = new SearchService(mockLanceStore, mockEmbeddingEngine);
-  });
-
-  it('boosts results when path contains query keywords', async () => {
-    // Two files with same base score, but one has "dispatcher" in the path
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('generic-file'),
-        score: 0.85,
-        content: 'handles async operations',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/utils/helpers.py',
-        },
-      },
-      {
-        id: createDocumentId('dispatcher-file'),
-        score: 0.85,
-        content: 'handles async operations',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/async_dispatcher.py',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'dispatcher',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    // dispatcher-file should rank higher due to path keyword match
-    expect(results.results[0]?.id).toBe(createDocumentId('dispatcher-file'));
-  });
-
-  it('boosts results with multiple path keyword matches', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('single-match'),
-        score: 0.85,
-        content: 'crawler implementation',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/crawler.py',
-        },
-      },
-      {
-        id: createDocumentId('double-match'),
-        score: 0.85,
-        content: 'crawler implementation',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/deep_crawling/crawler.py',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'deep crawler',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    // double-match should rank higher (both "deep" and "crawler" in path)
-    expect(results.results[0]?.id).toBe(createDocumentId('double-match'));
-  });
-
-  it('ignores stop words when matching path keywords', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.85,
-        content: 'configuration guide',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/config.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'how to configure',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    // Should match "config" from "configure", not boost from "how" or "to"
-    expect(results.results.length).toBeGreaterThan(0);
-  });
-
-  it('does not boost when path has no matching keywords', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('unrelated-path'),
-        score: 0.9,
-        content: 'dispatcher implementation details',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/utils/helpers.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'dispatcher',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    // Should still return result (content matches), just no path boost
-    expect(results.results.length).toBe(1);
-  });
-});
-
-describe('SearchService - Code Graph Integration', () => {
-  let mockLanceStore: LanceStore;
-  let mockEmbeddingEngine: EmbeddingEngine;
-  let mockCodeGraphService: { loadGraph: ReturnType<typeof vi.fn> };
-  let searchService: SearchService;
-  const storeId = createStoreId('test-store');
-
-  beforeEach(() => {
-    mockLanceStore = {
-      search: vi.fn(),
-      fullTextSearch: vi.fn(),
-    } as unknown as LanceStore;
-
-    mockEmbeddingEngine = {
-      embed: vi.fn().mockResolvedValue([0.1, 0.2, 0.3]),
-    } as unknown as EmbeddingEngine;
-
-    mockCodeGraphService = {
-      loadGraph: vi.fn(),
-    };
-
-    // Create SearchService with mock codeGraphService
-    searchService = new SearchService(
-      mockLanceStore,
-      mockEmbeddingEngine,
-      mockCodeGraphService as unknown as import('./code-graph.service.js').CodeGraphService
-    );
-  });
-
-  it('includes usage stats from code graph when detail is contextual', async () => {
-    // Create a mock CodeGraph with getCalledByCount and getCallsCount methods
-    const mockGraph = {
-      getCalledByCount: vi.fn().mockReturnValue(3),
-      getCallsCount: vi.fn().mockReturnValue(5),
-      getIncomingEdges: vi.fn().mockReturnValue([]),
-      getEdges: vi.fn().mockReturnValue([]),
-    };
-
-    mockCodeGraphService.loadGraph.mockResolvedValue(mockGraph);
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'export function myFunction() { return 42; }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/utils.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'myFunction',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'contextual',
-    });
-
-    expect(results.results.length).toBe(1);
-    expect(results.results[0]?.context).toBeDefined();
-    expect(results.results[0]?.context?.usage).toBeDefined();
-    expect(results.results[0]?.context?.usage?.calledBy).toBe(3);
-    expect(results.results[0]?.context?.usage?.calls).toBe(5);
-    expect(mockGraph.getCalledByCount).toHaveBeenCalledWith('/src/utils.ts:myFunction');
-    expect(mockGraph.getCallsCount).toHaveBeenCalledWith('/src/utils.ts:myFunction');
-  });
-
-  it('returns zero usage stats when symbol is anonymous', async () => {
-    const mockGraph = {
-      getCalledByCount: vi.fn(),
-      getCallsCount: vi.fn(),
-      getIncomingEdges: vi.fn().mockReturnValue([]),
-      getEdges: vi.fn().mockReturnValue([]),
-    };
-
-    mockCodeGraphService.loadGraph.mockResolvedValue(mockGraph);
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        // Content that matches no symbol patterns (no function/class/const keywords followed by identifiers)
-        content: 'This is just plain documentation text with no code symbols at all.',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/readme.md',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'documentation text',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'contextual',
-    });
-
-    expect(results.results.length).toBe(1);
-    expect(results.results[0]?.context?.usage?.calledBy).toBe(0);
-    expect(results.results[0]?.context?.usage?.calls).toBe(0);
-    // Graph methods should NOT be called for anonymous symbols
-    expect(mockGraph.getCalledByCount).not.toHaveBeenCalled();
-    expect(mockGraph.getCallsCount).not.toHaveBeenCalled();
-  });
-
-  it('returns zero usage stats when symbol is empty', async () => {
-    const mockGraph = {
-      getCalledByCount: vi.fn(),
-      getCallsCount: vi.fn(),
-      getIncomingEdges: vi.fn().mockReturnValue([]),
-      getEdges: vi.fn().mockReturnValue([]),
-    };
-
-    mockCodeGraphService.loadGraph.mockResolvedValue(mockGraph);
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: '   \n\n   ', // whitespace only content
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/empty.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'empty content',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'contextual',
-    });
-
-    // Graph methods should NOT be called for symbols that can't be extracted
-    expect(mockGraph.getCalledByCount).not.toHaveBeenCalled();
-  });
-
-  it('includes related code from graph when detail is full', async () => {
-    const mockGraph = {
-      getCalledByCount: vi.fn().mockReturnValue(2),
-      getCallsCount: vi.fn().mockReturnValue(1),
-      getIncomingEdges: vi.fn().mockReturnValue([
-        {
-          from: '/src/caller.ts:callerFunction',
-          to: '/src/utils.ts:myFunction',
-          type: 'calls',
-          confidence: 0.8,
-        },
-        {
-          from: '/src/main.ts:init',
-          to: '/src/utils.ts:myFunction',
-          type: 'calls',
-          confidence: 0.9,
-        },
-      ]),
-      getEdges: vi.fn().mockReturnValue([
-        {
-          from: '/src/utils.ts:myFunction',
-          to: '/src/helper.ts:helperFn',
-          type: 'calls',
-          confidence: 0.8,
-        },
-      ]),
-    };
-
-    mockCodeGraphService.loadGraph.mockResolvedValue(mockGraph);
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content:
-          '/** My function does stuff */\nexport function myFunction() { return helperFn(); }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/utils.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'myFunction',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'full',
-    });
-
-    expect(results.results.length).toBe(1);
-    expect(results.results[0]?.full).toBeDefined();
-    expect(results.results[0]?.full?.relatedCode).toBeDefined();
-    expect(results.results[0]?.full?.relatedCode?.length).toBe(3);
-
-    // Check incoming (callers)
-    const callers = results.results[0]?.full?.relatedCode?.filter(
-      (r) => r.relationship === 'calls this'
-    );
-    expect(callers?.length).toBe(2);
-    expect(
-      callers?.some((c) => c.file === '/src/caller.ts' && c.summary === 'callerFunction()')
-    ).toBe(true);
-    expect(callers?.some((c) => c.file === '/src/main.ts' && c.summary === 'init()')).toBe(true);
-
-    // Check outgoing (callees)
-    const callees = results.results[0]?.full?.relatedCode?.filter(
-      (r) => r.relationship === 'called by this'
-    );
-    expect(callees?.length).toBe(1);
-    expect(callees?.[0]?.file).toBe('/src/helper.ts');
-    expect(callees?.[0]?.summary).toBe('helperFn()');
-  });
-
-  it('returns empty related code for anonymous symbols', async () => {
-    const mockGraph = {
-      getCalledByCount: vi.fn(),
-      getCallsCount: vi.fn(),
-      getIncomingEdges: vi.fn(),
-      getEdges: vi.fn(),
-    };
-
-    mockCodeGraphService.loadGraph.mockResolvedValue(mockGraph);
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'Just plain text without code symbols',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/notes.txt',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'notes',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'full',
-    });
-
-    expect(results.results.length).toBe(1);
-    expect(results.results[0]?.full?.relatedCode).toEqual([]);
-    // Graph methods should NOT be called for anonymous symbols
-    expect(mockGraph.getIncomingEdges).not.toHaveBeenCalled();
-    expect(mockGraph.getEdges).not.toHaveBeenCalled();
-  });
-
-  it('handles edges with non-calls type gracefully', async () => {
-    const mockGraph = {
-      getCalledByCount: vi.fn().mockReturnValue(0),
-      getCallsCount: vi.fn().mockReturnValue(0),
-      getIncomingEdges: vi.fn().mockReturnValue([
-        {
-          from: '/src/index.ts',
-          to: '/src/utils.ts:myFunction',
-          type: 'imports',
-          confidence: 1.0,
-        },
-      ]),
-      getEdges: vi.fn().mockReturnValue([
-        {
-          from: '/src/utils.ts:myFunction',
-          to: '/src/types.ts:MyInterface',
-          type: 'implements',
-          confidence: 1.0,
-        },
-      ]),
-    };
-
-    mockCodeGraphService.loadGraph.mockResolvedValue(mockGraph);
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'export function myFunction() { return 42; }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/utils.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'myFunction',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'full',
-    });
-
-    expect(results.results.length).toBe(1);
-    // No related code should be returned because edges are not 'calls' type
-    expect(results.results[0]?.full?.relatedCode).toEqual([]);
-  });
-
-  it('parses node IDs without colons correctly', async () => {
-    const mockGraph = {
-      getCalledByCount: vi.fn().mockReturnValue(1),
-      getCallsCount: vi.fn().mockReturnValue(0),
-      getIncomingEdges: vi.fn().mockReturnValue([
-        // Edge with nodeId that has no colon (edge case)
-        { from: 'simpleNodeId', to: '/src/utils.ts:myFunction', type: 'calls', confidence: 0.8 },
-      ]),
-      getEdges: vi.fn().mockReturnValue([]),
-    };
-
-    mockCodeGraphService.loadGraph.mockResolvedValue(mockGraph);
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'export function myFunction() { return 42; }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/utils.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'myFunction',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'full',
-    });
-
-    expect(results.results.length).toBe(1);
-    const callers = results.results[0]?.full?.relatedCode?.filter(
-      (r) => r.relationship === 'calls this'
-    );
-    expect(callers?.length).toBe(1);
-    // When nodeId has no colon, file should be the whole nodeId and symbol should be empty -> 'unknown'
-    expect(callers?.[0]?.file).toBe('simpleNodeId');
-    expect(callers?.[0]?.summary).toBe('unknown');
-  });
-
-  it('handles null graph gracefully', async () => {
-    mockCodeGraphService.loadGraph.mockResolvedValue(null);
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'export function myFunction() { return 42; }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/utils.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'myFunction',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'full',
-    });
-
-    expect(results.results.length).toBe(1);
-    expect(results.results[0]?.context?.usage?.calledBy).toBe(0);
-    expect(results.results[0]?.context?.usage?.calls).toBe(0);
-    expect(results.results[0]?.full?.relatedCode).toEqual([]);
-  });
-
-  it('limits related code to 10 items', async () => {
-    // Create 15 incoming edges
-    const manyIncomingEdges = Array.from({ length: 15 }, (_, i) => ({
-      from: `/src/file${i}.ts:func${i}`,
-      to: '/src/utils.ts:myFunction',
-      type: 'calls' as const,
-      confidence: 0.8,
-    }));
-
-    const mockGraph = {
-      getCalledByCount: vi.fn().mockReturnValue(15),
-      getCallsCount: vi.fn().mockReturnValue(0),
-      getIncomingEdges: vi.fn().mockReturnValue(manyIncomingEdges),
-      getEdges: vi.fn().mockReturnValue([]),
-    };
-
-    mockCodeGraphService.loadGraph.mockResolvedValue(mockGraph);
-
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'export function myFunction() { return 42; }',
-        metadata: {
-          type: 'file' as const,
-          storeId,
-          indexedAt: new Date(),
-          path: '/src/utils.ts',
-        },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'myFunction',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      detail: 'full',
-    });
-
-    expect(results.results.length).toBe(1);
-    // Should be limited to 10 related items
-    expect(results.results[0]?.full?.relatedCode?.length).toBe(10);
-  });
-});
-
-describe('SearchService - Threshold Filtering', () => {
-  let mockLanceStore: LanceStore;
-  let mockEmbeddingEngine: EmbeddingEngine;
-  let searchService: SearchService;
-  const storeId = createStoreId('test-store');
-
-  beforeEach(() => {
-    mockLanceStore = {
-      search: vi.fn(),
-      fullTextSearch: vi.fn(),
-    } as unknown as LanceStore;
-
-    mockEmbeddingEngine = {
-      embed: vi.fn().mockResolvedValue([0.1, 0.2, 0.3]),
-    } as unknown as EmbeddingEngine;
-
-    searchService = new SearchService(mockLanceStore, mockEmbeddingEngine);
-  });
-
-  it('applies threshold to normalized scores, not raw scores', async () => {
-    // Setup: 3 results with different raw scores
-    // In hybrid mode with RRF, ranks matter more than raw scores
-    // doc1 appears in both vector and FTS -> highest RRF score -> normalized to 1.0
-    // doc2 appears only in vector -> middle RRF score -> normalized to ~0.5
-    // doc3 appears only in vector, lowest rank -> lowest RRF score -> normalized to 0.0
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'result 1',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.7,
-        content: 'result 2',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc3'),
-        score: 0.5,
-        content: 'result 3',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    // Add doc1 and doc2 to FTS results so they both have good RRF scores
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'result 1',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.7,
-        content: 'result 2',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-
-    // With threshold 0.4, doc1 (1.0) and doc2 (~0.47) should pass
-    // doc3 (0.0) should be filtered out
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-      threshold: 0.4,
-    });
-
-    // Should return 2 results: scores >= 0.4 (normalized)
-    expect(results.results.length).toBe(2);
-    expect(results.results[0]?.id).toBe(createDocumentId('doc1'));
-    expect(results.results[1]?.id).toBe(createDocumentId('doc2'));
-
-    // Verify normalized scores
-    expect(results.results[0]?.score).toBe(1.0);
-    expect(results.results[1]?.score).toBeGreaterThanOrEqual(0.4);
-
-    // Verify doc3 was filtered out (its normalized score is 0.0)
-    expect(results.results.find((r) => r.id === createDocumentId('doc3'))).toBeUndefined();
-  });
-
-  it('returns all results when threshold is 0', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'result 1',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.1,
-        content: 'result 2',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-      threshold: 0,
-    });
-
-    // All results should be returned (scores >= 0)
-    expect(results.results.length).toBe(2);
-  });
-
-  it('returns no results when threshold is higher than all scores', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'result 1',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.8,
-        content: 'result 2',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    // Threshold > 1.0 means no results pass
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-      threshold: 1.1,
-    });
-
-    expect(results.results.length).toBe(0);
-  });
-
-  it('applies threshold in vector mode after score calculation', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'result 1',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.3,
-        content: 'result 2',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'vector',
-      limit: 10,
-      threshold: 0.5,
-    });
-
-    // Only doc1 should pass (normalized score 1.0 >= 0.5)
-    // doc2 has normalized score 0.0 which is < 0.5
-    expect(results.results.length).toBe(1);
-    expect(results.results[0]?.id).toBe(createDocumentId('doc1'));
-  });
-
-  it('maintains correct result count metadata after threshold filtering', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.9,
-        content: 'result 1',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.5,
-        content: 'result 2',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc3'),
-        score: 0.1,
-        content: 'result 3',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-      threshold: 0.5,
-    });
-
-    // Check response metadata
-    expect(results.totalResults).toBe(results.results.length);
-    expect(results.query).toBe('test query');
-  });
-});
-
-describe('SearchService - Raw Score and Confidence', () => {
-  let mockLanceStore: LanceStore;
-  let mockEmbeddingEngine: EmbeddingEngine;
-  let searchService: SearchService;
-  const storeId = createStoreId('test-store');
-
-  beforeEach(() => {
-    mockLanceStore = {
-      search: vi.fn(),
-      fullTextSearch: vi.fn(),
-    } as unknown as LanceStore;
-
-    mockEmbeddingEngine = {
-      embed: vi.fn().mockResolvedValue([0.1, 0.2, 0.3]),
-    } as unknown as EmbeddingEngine;
-
-    searchService = new SearchService(mockLanceStore, mockEmbeddingEngine);
-  });
-
-  it('exposes rawVectorScore in rankingMetadata for hybrid search', async () => {
-    // Mock results with known raw vector scores
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.85, // Raw cosine similarity
-        content: 'vector result',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-      {
-        id: createDocumentId('doc2'),
-        score: 0.65,
-        content: 'another vector result',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    // Verify results have rawVectorScore in rankingMetadata
-    expect(results.results.length).toBeGreaterThan(0);
-    const firstResult = results.results[0];
-    expect(firstResult?.rankingMetadata).toBeDefined();
-    expect(firstResult?.rankingMetadata?.rawVectorScore).toBeDefined();
-    expect(firstResult?.rankingMetadata?.rawVectorScore).toBe(0.85);
-  });
-
-  it('returns confidence level based on maxRawScore', async () => {
-    // Mock results with high raw vector score (>= 0.5)
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.6, // High confidence threshold
-        content: 'high score result',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.confidence).toBe('high');
-    expect(results.maxRawScore).toBe(0.6);
-  });
-
-  it('returns medium confidence for scores between 0.3 and 0.5', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.4, // Medium confidence
-        content: 'medium score result',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.confidence).toBe('medium');
-    expect(results.maxRawScore).toBe(0.4);
-  });
-
-  it('returns low confidence for scores below 0.3', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.2, // Low confidence
-        content: 'low score result',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'test query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-    });
-
-    expect(results.confidence).toBe('low');
-    expect(results.maxRawScore).toBe(0.2);
-  });
-
-  it('filters results with minRelevance based on raw score', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.25, // Below minRelevance threshold
-        content: 'low relevance result',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'irrelevant query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-      minRelevance: 0.4, // Filter out results below this raw score
-    });
-
-    // Should return empty since max raw score (0.25) < minRelevance (0.4)
-    expect(results.results.length).toBe(0);
-    expect(results.confidence).toBe('low');
-  });
-
-  it('returns results when maxRawScore meets minRelevance', async () => {
-    vi.mocked(mockLanceStore.search).mockResolvedValue([
-      {
-        id: createDocumentId('doc1'),
-        score: 0.5, // Above minRelevance threshold
-        content: 'relevant result',
-        metadata: { type: 'file' as const, storeId, indexedAt: new Date() },
-      },
-    ]);
-    vi.mocked(mockLanceStore.fullTextSearch).mockResolvedValue([]);
-
-    const results = await searchService.search({
-      query: 'relevant query',
-      stores: [storeId],
-      mode: 'hybrid',
-      limit: 10,
-      minRelevance: 0.4,
-    });
-
-    // Should return results since max raw score (0.5) >= minRelevance (0.4)
-    expect(results.results.length).toBe(1);
-    expect(results.confidence).toBe('high');
-  });
-});
-
-describe('SearchService Environment Variables', () => {
-  afterEach(() => {
-    vi.unstubAllEnvs();
-  });
-
-  it('uses default values when env vars are not set', async () => {
-    // Clear env vars to test defaults
-    vi.stubEnv('SEARCH_CONFIDENCE_HIGH', undefined as unknown as string);
-    vi.stubEnv('SEARCH_CONFIDENCE_MEDIUM', undefined as unknown as string);
-    vi.stubEnv('SEARCH_TEST_FILE_BOOST', undefined as unknown as string);
-
-    const tempDir = await mkdtemp(join(tmpdir(), 'search-env-test-'));
-    const lanceStore = new LanceStore(tempDir);
-    const embeddingEngine = new EmbeddingEngine();
-    await embeddingEngine.initialize();
-
-    const storeId = createStoreId('env-test-store');
-    await lanceStore.initialize(storeId);
-
-    const text = 'test document';
-    const vector = await embeddingEngine.embed(text);
-    await lanceStore.addDocuments(storeId, [
-      {
-        id: createDocumentId('doc-1'),
-        content: text,
-        vector,
-        metadata: { type: 'file', storeId, indexedAt: new Date() },
-      },
-    ]);
-
-    const searchService = new SearchService(lanceStore, embeddingEngine);
-
-    // Should work with defaults (0.5 for high, 0.3 for medium)
-    const results = await searchService.search({ query: 'test', stores: [storeId] });
-    expect(results.results.length).toBeGreaterThan(0);
-    expect(results.confidence).toBeDefined();
-
-    await rm(tempDir, { recursive: true });
-  });
-
-  it('uses default test file boost when env var is not set', async () => {
-    vi.stubEnv('SEARCH_CONFIDENCE_HIGH', '0.5');
-    vi.stubEnv('SEARCH_CONFIDENCE_MEDIUM', '0.3');
-    vi.stubEnv('SEARCH_TEST_FILE_BOOST', undefined as unknown as string);
-
-    const tempDir = await mkdtemp(join(tmpdir(), 'search-env-test-'));
-    const lanceStore = new LanceStore(tempDir);
-    const embeddingEngine = new EmbeddingEngine();
-    await embeddingEngine.initialize();
-
-    const storeId = createStoreId('env-test-store');
-    await lanceStore.initialize(storeId);
-
-    // Add a test file document with fileType: 'test'
-    const text = 'test document content';
-    const vector = await embeddingEngine.embed(text);
-    await lanceStore.addDocuments(storeId, [
-      {
-        id: createDocumentId('test-file'),
-        content: text,
-        vector,
-        metadata: {
-          type: 'file',
-          storeId,
-          indexedAt: new Date(),
-          filePath: 'tests/example.test.ts',
-          fileType: 'test', // Uses default SEARCH_TEST_FILE_BOOST (0.5)
-        },
-      },
-    ]);
-
-    const searchService = new SearchService(lanceStore, embeddingEngine);
-
-    // Should work with default test file boost
-    const results = await searchService.search({ query: 'test', stores: [storeId] });
-    expect(results.results.length).toBeGreaterThan(0);
-
-    await rm(tempDir, { recursive: true });
-  });
-});