@librechat/agents 3.1.75-dev.1 → 3.1.76

package/src/tools/search/tavily.test.ts

@@ -0,0 +1,965 @@
+import axios from 'axios';
+import type * as t from './types';
+import { TavilyScraper, createTavilyScraper } from './tavily-scraper';
+import { createSearchAPI } from './search';
+import { createSearchTool } from './tool';
+
+jest.mock('axios');
+const mockedAxios = axios as jest.Mocked<typeof axios>;
+
+const mockLogger = {
+  error: jest.fn(),
+  warn: jest.fn(),
+  info: jest.fn(),
+  debug: jest.fn(),
+} as unknown as t.Logger;
+
+describe('Tavily search API', () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it('throws when Tavily API key is missing', () => {
+    expect(() =>
+      createSearchAPI({
+        searchProvider: 'tavily',
+        tavilyApiKey: '',
+      })
+    ).toThrow('TAVILY_API_KEY is required for Tavily API');
+  });
+
+  it('returns an error for empty Tavily search queries', async () => {
+    const searchAPI = createSearchAPI({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+    });
+
+    const result = await searchAPI.getSources({ query: '   ' });
+
+    expect(result).toEqual({
+      success: false,
+      error: 'Query cannot be empty',
+    });
+    expect(mockedAxios.post).not.toHaveBeenCalled();
+  });
+
+  it('returns an error when the Tavily search request fails', async () => {
+    mockedAxios.post.mockRejectedValueOnce(new Error('Network error'));
+
+    const searchAPI = createSearchAPI({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+    });
+
+    const result = await searchAPI.getSources({ query: 'example query' });
+
+    expect(result.success).toBe(false);
+    expect(result.error).toBe('Tavily API request failed: Network error');
+  });
+
+  it('passes string-mode options and maps Tavily response fields', async () => {
+    mockedAxios.post.mockResolvedValueOnce({
+      data: {
+        answer: 'A concise answer.',
+        images: [
+          {
+            url: 'https://example.com/image.png',
+            description: 'Example image',
+          },
+          {
+            description: 'Skipped image',
+          },
+          'https://example.com/second.png',
+        ],
+        results: [
+          {
+            title: 'Example',
+            url: 'https://example.com',
+            content: 'Example summary',
+            published_date: '2026-01-02',
+          },
+        ],
+      },
+    });
+
+    const searchAPI = createSearchAPI({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+      tavilySearchOptions: {
+        includeAnswer: 'advanced',
+        includeRawContent: 'markdown',
+        includeImages: true,
+        includeImageDescriptions: true,
+        safeSearch: true,
+        timeRange: 'd',
+      },
+    });
+
+    const result = await searchAPI.getSources({
+      query: 'example query',
+      country: 'US',
+    });
+    const [, payload] = mockedAxios.post.mock.calls[0];
+
+    expect(payload).toMatchObject({
+      query: 'example query',
+      country: 'united states',
+      safe_search: true,
+      include_answer: 'advanced',
+      include_raw_content: 'markdown',
+      include_images: true,
+      include_image_descriptions: true,
+      time_range: 'day',
+    });
+    expect(result.success).toBe(true);
+    expect(result.data?.answerBox?.snippet).toBe('A concise answer.');
+    expect(result.data?.organic?.[0]).toMatchObject({
+      title: 'Example',
+      link: 'https://example.com',
+      snippet: 'Example summary',
+      date: '2026-01-02',
+    });
+    expect(result.data?.images?.[0]).toMatchObject({
+      imageUrl: 'https://example.com/image.png',
+      title: 'Example image',
+      position: 1,
+    });
+    expect(result.data?.images?.[1]).toMatchObject({
+      imageUrl: 'https://example.com/second.png',
+      position: 2,
+    });
+  });
+
+  it('omits country for Tavily news searches', async () => {
+    mockedAxios.post.mockResolvedValueOnce({
+      data: {
+        results: [],
+      },
+    });
+
+    const searchAPI = createSearchAPI({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+    });
+
+    await searchAPI.getSources({
+      query: 'example query',
+      country: 'US',
+      news: true,
+    });
+    const [, payload] = mockedAxios.post.mock.calls[0];
+
+    expect(payload).toMatchObject({
+      query: 'example query',
+      topic: 'news',
+    });
+    expect(payload).not.toHaveProperty('country');
+    expect(payload).not.toHaveProperty('safe_search');
+  });
+
+  it('prioritizes request-level news mode over configured Tavily topic', async () => {
+    mockedAxios.post.mockResolvedValueOnce({
+      data: {
+        results: [
+          {
+            title: 'Market news',
+            url: 'https://example.com/news',
+            content: 'A news result',
+            published_date: '2026-01-03',
+          },
+        ],
+      },
+    });
+
+    const searchAPI = createSearchAPI({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+      tavilySearchOptions: {
+        topic: 'finance',
+      },
+    });
+
+    const result = await searchAPI.getSources({
+      query: 'example query',
+      type: 'news',
+    });
+    const [, payload] = mockedAxios.post.mock.calls[0];
+
+    expect(payload).toMatchObject({
+      query: 'example query',
+      topic: 'news',
+    });
+    expect(result.data?.news?.[0]).toMatchObject({
+      title: 'Market news',
+      link: 'https://example.com/news',
+    });
+  });
+
+  it('maps ISO country codes to Tavily country enum values', async () => {
+    mockedAxios.post.mockResolvedValueOnce({
+      data: {
+        results: [],
+      },
+    });
+
+    const searchAPI = createSearchAPI({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+    });
+
+    await searchAPI.getSources({
+      query: 'example query',
+      country: 'CZ',
+    });
+    const [, payload] = mockedAxios.post.mock.calls[0];
+
+    expect(payload).toMatchObject({
+      query: 'example query',
+      country: 'czech republic',
+    });
+    expect(payload).not.toHaveProperty('safe_search');
+  });
+
+  it('omits ISO country mappings that Tavily does not support', async () => {
+    mockedAxios.post.mockResolvedValueOnce({
+      data: {
+        results: [],
+      },
+    });
+
+    const searchAPI = createSearchAPI({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+    });
+
+    await searchAPI.getSources({
+      query: 'example query',
+      country: 'CD',
+    });
+    const [, payload] = mockedAxios.post.mock.calls[0];
+
+    expect(payload).toMatchObject({
+      query: 'example query',
+    });
+    expect(payload).not.toHaveProperty('country');
+    expect(payload).not.toHaveProperty('safe_search');
+  });
+
+  it('omits safe search for unsupported Tavily search depths', async () => {
+    mockedAxios.post.mockResolvedValueOnce({
+      data: {
+        results: [],
+      },
+    });
+
+    const searchAPI = createSearchAPI({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+      tavilySearchOptions: {
+        searchDepth: 'fast',
+        safeSearch: true,
+      },
+    });
+
+    await searchAPI.getSources({
+      query: 'example query',
+    });
+    const [, payload] = mockedAxios.post.mock.calls[0];
+
+    expect(payload).toMatchObject({
+      query: 'example query',
+      search_depth: 'fast',
+    });
+    expect(payload).not.toHaveProperty('safe_search');
+  });
+
+  it('only sends chunks per source for advanced Tavily searches', async () => {
+    mockedAxios.post.mockResolvedValue({
+      data: {
+        results: [],
+      },
+    });
+
+    const basicSearchAPI = createSearchAPI({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+      tavilySearchOptions: {
+        chunksPerSource: 2,
+      },
+    });
+
+    await basicSearchAPI.getSources({
+      query: 'example query',
+    });
+    const [, basicPayload] = mockedAxios.post.mock.calls[0];
+
+    expect(basicPayload).toMatchObject({
+      query: 'example query',
+      search_depth: 'basic',
+    });
+    expect(basicPayload).not.toHaveProperty('chunks_per_source');
+
+    const advancedSearchAPI = createSearchAPI({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+      tavilySearchOptions: {
+        searchDepth: 'advanced',
+        chunksPerSource: 2,
+      },
+    });
+
+    await advancedSearchAPI.getSources({
+      query: 'example query',
+    });
+    const [, advancedPayload] = mockedAxios.post.mock.calls[1];
+
+    expect(advancedPayload).toMatchObject({
+      query: 'example query',
+      search_depth: 'advanced',
+      chunks_per_source: 2,
+    });
+  });
+
+  it('uses explicit tool safe search config and skips Tavily video subqueries', async () => {
+    mockedAxios.post.mockResolvedValue({
+      data: {
+        results: [],
+      },
+    });
+
+    const searchTool = createSearchTool({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+      scraperProvider: 'tavily',
+      rerankerType: 'none',
+      logger: mockLogger,
+      safeSearch: 2,
+    });
+
+    await searchTool.invoke({
+      query: 'example query',
+      videos: true,
+    });
+    const [, payload] = mockedAxios.post.mock.calls[0];
+
+    expect(mockedAxios.post).toHaveBeenCalledTimes(1);
+    expect(payload).toMatchObject({
+      query: 'example query',
+      safe_search: true,
+    });
+  });
+
+  it('lets explicit tool safe search override Tavily option safe search', async () => {
+    mockedAxios.post.mockResolvedValue({
+      data: {
+        results: [],
+      },
+    });
+
+    const searchTool = createSearchTool({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'test-key',
+      scraperProvider: 'tavily',
+      rerankerType: 'none',
+      logger: mockLogger,
+      safeSearch: 0,
+      tavilySearchOptions: {
+        safeSearch: true,
+      },
+    });
+
+    await searchTool.invoke({
+      query: 'example query',
+    });
+    const [, payload] = mockedAxios.post.mock.calls[0];
+
+    expect(payload).toMatchObject({
+      query: 'example query',
+      safe_search: false,
+    });
+  });
+
+  it('preserves Tavily scraper connection overrides in the search tool', async () => {
+    mockedAxios.post
+      .mockResolvedValueOnce({
+        data: {
+          results: [
+            {
+              title: 'Example',
+              url: 'https://example.com',
+              content: 'Example summary',
+            },
+          ],
+        },
+      })
+      .mockResolvedValueOnce({
+        data: {
+          results: [
+            {
+              url: 'https://example.com',
+              raw_content: 'Extracted content',
+              images: [],
+            },
+          ],
+          failed_results: [],
+        },
+      });
+
+    const searchTool = createSearchTool({
+      searchProvider: 'tavily',
+      tavilyApiKey: 'search-key',
+      scraperProvider: 'tavily',
+      tavilyScraperOptions: {
+        apiKey: 'scraper-key',
+        apiUrl: 'https://proxy.example.com/extract',
+      },
+      rerankerType: 'none',
+      logger: mockLogger,
+    });
+
+    await searchTool.invoke({
+      query: 'example query',
+    });
+    const [, , searchConfig] = mockedAxios.post.mock.calls[0];
+    const [extractUrl, , extractConfig] = mockedAxios.post.mock.calls[1];
+
+    expect(mockedAxios.post).toHaveBeenCalledTimes(2);
+    expect(searchConfig).toMatchObject({
+      headers: {
+        Authorization: 'Bearer search-key',
+      },
+    });
+    expect(extractUrl).toBe('https://proxy.example.com/extract');
+    expect(extractConfig).toMatchObject({
+      headers: {
+        Authorization: 'Bearer scraper-key',
+      },
+    });
+  });
+});
+
+describe('TavilyScraper', () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  describe('constructor', () => {
+    it('warns when TAVILY_API_KEY is not set', () => {
+      const logger = { ...mockLogger, warn: jest.fn() } as unknown as t.Logger;
+      new TavilyScraper({ apiKey: '', logger });
+      expect(logger.warn).toHaveBeenCalledWith(
+        'TAVILY_API_KEY is not set. Scraping will not work.'
+      );
+    });
+
+    it('uses TAVILY_EXTRACT_URL env var for apiUrl', async () => {
+      const original = process.env.TAVILY_EXTRACT_URL;
+      try {
+        process.env.TAVILY_EXTRACT_URL =
+          'https://custom-proxy.example.com/extract';
+        mockedAxios.post.mockResolvedValueOnce({
+          data: { results: [], failed_results: [] },
+        });
+        const scraper = new TavilyScraper({
+          apiKey: 'test-key',
+          logger: mockLogger,
+        });
+
+        await scraper.scrapeUrl('https://example.com');
+
+        expect(mockedAxios.post.mock.calls[0][0]).toBe(
+          'https://custom-proxy.example.com/extract'
+        );
+      } finally {
+        if (original !== undefined) {
+          process.env.TAVILY_EXTRACT_URL = original;
+        } else {
+          delete process.env.TAVILY_EXTRACT_URL;
+        }
+      }
+    });
+
+    it('defaults to https://api.tavily.com/extract', async () => {
+      const original = process.env.TAVILY_EXTRACT_URL;
+      try {
+        delete process.env.TAVILY_EXTRACT_URL;
+        mockedAxios.post.mockResolvedValueOnce({
+          data: { results: [], failed_results: [] },
+        });
+        const scraper = new TavilyScraper({
+          apiKey: 'test-key',
+          logger: mockLogger,
+        });
+
+        await scraper.scrapeUrl('https://example.com');
+
+        expect(mockedAxios.post.mock.calls[0][0]).toBe(
+          'https://api.tavily.com/extract'
+        );
+      } finally {
+        if (original !== undefined) {
+          process.env.TAVILY_EXTRACT_URL = original;
+        }
+      }
+    });
+
+    it('defaults timeout to 15000ms', async () => {
+      mockedAxios.post.mockResolvedValueOnce({
+        data: { results: [], failed_results: [] },
+      });
+      const scraper = new TavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+
+      await scraper.scrapeUrl('https://example.com');
+
+      expect(mockedAxios.post.mock.calls[0][2]).toMatchObject({
+        timeout: 15000,
+      });
+    });
+
+    it('defaults advanced extraction timeout to 30000ms', async () => {
+      mockedAxios.post.mockResolvedValueOnce({
+        data: { results: [], failed_results: [] },
+      });
+      const scraper = new TavilyScraper({
+        apiKey: 'test-key',
+        extractDepth: 'advanced',
+        logger: mockLogger,
+      });
+
+      await scraper.scrapeUrl('https://example.com');
+
+      expect(mockedAxios.post.mock.calls[0][2]).toMatchObject({
+        timeout: 30000,
+      });
+    });
+  });
+
+  describe('scrapeUrl', () => {
+    it('returns error when API key is not set', async () => {
+      const scraper = createTavilyScraper({ apiKey: '', logger: mockLogger });
+      const [url, response] = await scraper.scrapeUrl('https://example.com');
+      expect(url).toBe('https://example.com');
+      expect(response.success).toBe(false);
+      expect(response.error).toBe('TAVILY_API_KEY is not set');
+    });
+
+    it('returns scraped content on success', async () => {
+      mockedAxios.post.mockResolvedValueOnce({
+        data: {
+          results: [
+            {
+              url: 'https://example.com',
+              raw_content: '# Hello World\nSome content here.',
+              images: ['https://example.com/img.png'],
+              favicon: 'https://example.com/favicon.ico',
+            },
+          ],
+          failed_results: [],
+        },
+      });
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const [url, response] = await scraper.scrapeUrl('https://example.com');
+
+      expect(url).toBe('https://example.com');
+      expect(response.success).toBe(true);
+      expect(response.data?.rawContent).toBe(
+        '# Hello World\nSome content here.'
+      );
+      expect(response.data?.images).toEqual(['https://example.com/img.png']);
+      expect(response.data?.favicon).toBe('https://example.com/favicon.ico');
+    });
+
+    it('applies per-call extract options to the Tavily payload', async () => {
+      mockedAxios.post.mockResolvedValueOnce({
+        data: {
+          results: [
+            {
+              url: 'https://example.com',
+              raw_content: 'Content',
+              images: [],
+            },
+          ],
+          failed_results: [],
+        },
+      });
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      await scraper.scrapeUrl('https://example.com', {
+        includeFavicon: true,
+        format: 'text',
+        timeout: 2000,
+      });
+      const [, payload, config] = mockedAxios.post.mock.calls[0];
+
+      expect(payload).toMatchObject({
+        urls: ['https://example.com'],
+        include_favicon: true,
+        format: 'text',
+        timeout: 2,
+      });
+      expect(payload).not.toHaveProperty('chunks_per_source');
+      expect(config).toMatchObject({ timeout: 2000 });
+    });
+
+    it('omits extract timeout from the payload when using Tavily defaults', async () => {
+      mockedAxios.post.mockResolvedValueOnce({
+        data: {
+          results: [
+            {
+              url: 'https://example.com',
+              raw_content: 'Content',
+              images: [],
+            },
+          ],
+          failed_results: [],
+        },
+      });
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+        extractDepth: 'advanced',
+      });
+      await scraper.scrapeUrl('https://example.com');
+      const [, payload, config] = mockedAxios.post.mock.calls[0];
+
+      expect(payload).toMatchObject({
+        urls: ['https://example.com'],
+        extract_depth: 'advanced',
+      });
+      expect(payload).not.toHaveProperty('timeout');
+      expect(config).toMatchObject({ timeout: 30000 });
+    });
+
+    it('uses the advanced default client timeout for per-call extract depth', async () => {
+      mockedAxios.post.mockResolvedValueOnce({
+        data: {
+          results: [
+            {
+              url: 'https://example.com',
+              raw_content: 'Content',
+              images: [],
+            },
+          ],
+          failed_results: [],
+        },
+      });
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      await scraper.scrapeUrl('https://example.com', {
+        extractDepth: 'advanced',
+      });
+      const [, payload, config] = mockedAxios.post.mock.calls[0];
+
+      expect(payload).toMatchObject({
+        urls: ['https://example.com'],
+        extract_depth: 'advanced',
+      });
+      expect(payload).not.toHaveProperty('timeout');
+      expect(config).toMatchObject({ timeout: 30000 });
+    });
+
+    it('handles API failure gracefully', async () => {
+      mockedAxios.post.mockRejectedValueOnce(new Error('Network error'));
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const [url, response] = await scraper.scrapeUrl('https://example.com');
+
+      expect(url).toBe('https://example.com');
+      expect(response.success).toBe(false);
+      expect(response.error).toContain('Tavily Extract API request failed');
+      expect(response.error).toContain('Network error');
+    });
+
+    it('reads error from failed_results when results is empty', async () => {
+      mockedAxios.post.mockResolvedValueOnce({
+        data: {
+          results: [],
+          failed_results: [
+            {
+              url: 'https://example.com',
+              error: 'Page is behind a paywall',
+            },
+          ],
+        },
+      });
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const [url, response] = await scraper.scrapeUrl('https://example.com');
+
+      expect(url).toBe('https://example.com');
+      expect(response.success).toBe(false);
+      expect(response.error).toBe('Page is behind a paywall');
+    });
+
+    it('matches normalized URLs from Tavily Extract responses', async () => {
+      mockedAxios.post.mockResolvedValueOnce({
+        data: {
+          results: [
+            {
+              url: 'https://example.com/article',
+              raw_content: 'Content',
+              images: [],
+            },
+          ],
+          failed_results: [],
+        },
+      });
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const [url, response] = await scraper.scrapeUrl(
+        'https://example.com/article/'
+      );
+
+      expect(url).toBe('https://example.com/article/');
+      expect(response.success).toBe(true);
+      expect(response.data?.rawContent).toBe('Content');
+    });
+
+    it('returns descriptive error when URL not in results or failed_results', async () => {
+      mockedAxios.post.mockResolvedValueOnce({
+        data: { results: [], failed_results: [] },
+      });
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const [, response] = await scraper.scrapeUrl('https://missing.com');
+
+      expect(response.success).toBe(false);
+      expect(response.error).toBe('URL not found in Tavily Extract response');
+    });
+  });
+
+  describe('scrapeUrls (batch)', () => {
+    it('batches multiple URLs into a single API call', async () => {
+      const urls = [
+        'https://example.com/1',
+        'https://example.com/2',
+        'https://example.com/3',
+      ];
+
+      mockedAxios.post.mockResolvedValueOnce({
+        data: {
+          results: urls.map((url) => ({
+            url,
+            raw_content: `Content for ${url}`,
+            images: [],
+          })),
+          failed_results: [],
+        },
+      });
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const results = await scraper.scrapeUrls(urls);
+
+      expect(mockedAxios.post).toHaveBeenCalledTimes(1);
+      expect(results).toHaveLength(3);
+
+      for (let i = 0; i < results.length; i++) {
+        const [url, response] = results[i];
+        expect(url).toBe(urls[i]);
+        expect(response.success).toBe(true);
+        expect(response.data?.rawContent).toBe(`Content for ${urls[i]}`);
+      }
+    });
+
+    it('handles mixed success and failure results', async () => {
+      mockedAxios.post.mockResolvedValueOnce({
+        data: {
+          results: [
+            {
+              url: 'https://example.com/ok',
+              raw_content: 'Good content',
+              images: [],
+            },
+          ],
+          failed_results: [
+            {
+              url: 'https://example.com/fail',
+              error: 'Access denied',
+            },
+          ],
+        },
+      });
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const results = await scraper.scrapeUrls([
+        'https://example.com/ok',
+        'https://example.com/fail',
+      ]);
+
+      expect(results).toHaveLength(2);
+      expect(results[0][1].success).toBe(true);
+      expect(results[1][1].success).toBe(false);
+      expect(results[1][1].error).toBe('Access denied');
+    });
+
+    it('splits large batches into chunks of 20', async () => {
+      const urls = Array.from(
+        { length: 25 },
+        (_, i) => `https://example.com/${i}`
+      );
+
+      mockedAxios.post
+        .mockResolvedValueOnce({
+          data: {
+            results: urls.slice(0, 20).map((url) => ({
+              url,
+              raw_content: 'content',
+              images: [],
+            })),
+            failed_results: [],
+          },
+        })
+        .mockResolvedValueOnce({
+          data: {
+            results: urls.slice(20).map((url) => ({
+              url,
+              raw_content: 'content',
+              images: [],
+            })),
+            failed_results: [],
+          },
+        });
+
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const results = await scraper.scrapeUrls(urls);
+
+      expect(mockedAxios.post).toHaveBeenCalledTimes(2);
+      expect(results).toHaveLength(25);
+    });
+
+    it('returns errors for all URLs when API key is missing', async () => {
+      const scraper = createTavilyScraper({ apiKey: '', logger: mockLogger });
+      const results = await scraper.scrapeUrls([
+        'https://a.com',
+        'https://b.com',
+      ]);
+
+      expect(results).toHaveLength(2);
+      for (const [, response] of results) {
+        expect(response.success).toBe(false);
+        expect(response.error).toBe('TAVILY_API_KEY is not set');
+      }
+    });
+  });
+
+  describe('extractContent', () => {
+    it('returns content and image references', () => {
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const [content, references] = scraper.extractContent({
+        success: true,
+        data: {
+          rawContent: 'Hello world',
+          images: ['https://img.example.com/1.png'],
+        },
+      });
+
+      expect(content).toBe('Hello world');
+      expect(references).toBeDefined();
+      expect(references?.images).toHaveLength(1);
+      expect(references?.images[0].originalUrl).toBe(
+        'https://img.example.com/1.png'
+      );
+    });
+
+    it('returns empty content for failed response', () => {
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const [content, references] = scraper.extractContent({
+        success: false,
+        error: 'Failed',
+      });
+
+      expect(content).toBe('');
+      expect(references).toBeUndefined();
+    });
+
+    it('returns undefined references when no images', () => {
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const [content, references] = scraper.extractContent({
+        success: true,
+        data: { rawContent: 'No images here', images: [] },
+      });
+
+      expect(content).toBe('No images here');
+      expect(references).toBeUndefined();
+    });
+  });
+
+  describe('extractMetadata', () => {
+    it('returns images_count for successful response', () => {
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const metadata = scraper.extractMetadata({
+        success: true,
+        data: {
+          rawContent: 'content',
+          images: ['a', 'b', 'c'],
+          favicon: 'https://example.com/favicon.ico',
+        },
+      });
+
+      expect(metadata).toEqual({
+        favicon: 'https://example.com/favicon.ico',
+        images_count: 3,
+      });
+    });
+
+    it('returns empty object for failed response', () => {
+      const scraper = createTavilyScraper({
+        apiKey: 'test-key',
+        logger: mockLogger,
+      });
+      const metadata = scraper.extractMetadata({
+        success: false,
+        error: 'Failed',
+      });
+
+      expect(metadata).toEqual({});
+    });
+  });
+});