@youdotcom-oss/mcp 1.5.0 → 1.6.0

package/src/shared/tests/shared.utils.spec.ts

@@ -1,6 +1,6 @@
-import { describe, expect, test } from 'bun:test';
-import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-import { useGetClientVersion } from '../use-client-version.ts';
+import { describe, expect, test } from 'bun:test'
+import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js'
+import { useGetClientVersion } from '../use-client-version.ts'
 
 describe('useGetClientVersion', () => {
   test('returns formatted string with all fields present', () => {
@@ -13,15 +13,15 @@ describe('useGetClientVersion', () => {
           websiteUrl: 'https://example.com',
         }),
       },
-    } as unknown as McpServer;
+    } as unknown as McpServer
 
-    const getUserAgent = useGetClientVersion(mockMcp);
-    const result = getUserAgent();
+    const getUserAgent = useGetClientVersion(mockMcp)
+    const result = getUserAgent()
 
     expect(result).toMatch(
       /^MCP\/[\d.]+(-[\w.]+)? \(You\.com; test-client; 1\.0\.0; Test Client; https:\/\/example\.com\)$/,
-    );
-  });
+    )
+  })
 
   test('returns formatted string with name and version only', () => {
     const mockMcp = {
@@ -31,39 +31,39 @@ describe('useGetClientVersion', () => {
           version: '1.0.0',
         }),
       },
-    } as unknown as McpServer;
+    } as unknown as McpServer
 
-    const getUserAgent = useGetClientVersion(mockMcp);
-    const result = getUserAgent();
+    const getUserAgent = useGetClientVersion(mockMcp)
+    const result = getUserAgent()
 
-    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; test-client; 1\.0\.0\)$/);
-  });
+    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; test-client; 1\.0\.0\)$/)
+  })
 
   test('returns UNKNOWN when no client version available', () => {
     const mockMcp = {
       server: {
         getClientVersion: () => null,
       },
-    } as unknown as McpServer;
+    } as unknown as McpServer
 
-    const getUserAgent = useGetClientVersion(mockMcp);
-    const result = getUserAgent();
+    const getUserAgent = useGetClientVersion(mockMcp)
+    const result = getUserAgent()
 
-    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; UNKNOWN\)$/);
-  });
+    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; UNKNOWN\)$/)
+  })
 
   test('returns UNKNOWN when getClientVersion returns undefined', () => {
     const mockMcp = {
       server: {
         getClientVersion: () => undefined,
       },
-    } as unknown as McpServer;
+    } as unknown as McpServer
 
-    const getUserAgent = useGetClientVersion(mockMcp);
-    const result = getUserAgent();
+    const getUserAgent = useGetClientVersion(mockMcp)
+    const result = getUserAgent()
 
-    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; UNKNOWN\)$/);
-  });
+    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; UNKNOWN\)$/)
+  })
 
   test('filters out empty strings from fields', () => {
     const mockMcp = {
@@ -75,14 +75,14 @@ describe('useGetClientVersion', () => {
           websiteUrl: 'https://example.com',
         }),
       },
-    } as unknown as McpServer;
+    } as unknown as McpServer
 
-    const getUserAgent = useGetClientVersion(mockMcp);
-    const result = getUserAgent();
+    const getUserAgent = useGetClientVersion(mockMcp)
+    const result = getUserAgent()
 
-    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; test-client; 1\.0\.0; https:\/\/example\.com\)$/);
-    expect(result).not.toContain(';;'); // No double semicolons
-  });
+    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; test-client; 1\.0\.0; https:\/\/example\.com\)$/)
+    expect(result).not.toContain(';;') // No double semicolons
+  })
 
   test('filters out null values from fields', () => {
     const mockMcp = {
@@ -94,13 +94,13 @@ describe('useGetClientVersion', () => {
           websiteUrl: 'https://example.com',
         }),
       },
-    } as unknown as McpServer;
+    } as unknown as McpServer
 
-    const getUserAgent = useGetClientVersion(mockMcp);
-    const result = getUserAgent();
+    const getUserAgent = useGetClientVersion(mockMcp)
+    const result = getUserAgent()
 
-    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; test-client; 1\.0\.0; https:\/\/example\.com\)$/);
-  });
+    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; test-client; 1\.0\.0; https:\/\/example\.com\)$/)
+  })
 
   test('handles partial fields - name, version, and title only', () => {
     const mockMcp = {
@@ -111,13 +111,13 @@ describe('useGetClientVersion', () => {
           title: 'Claude Desktop App',
         }),
       },
-    } as unknown as McpServer;
+    } as unknown as McpServer
 
-    const getUserAgent = useGetClientVersion(mockMcp);
-    const result = getUserAgent();
+    const getUserAgent = useGetClientVersion(mockMcp)
+    const result = getUserAgent()
 
-    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; Claude Desktop; 0\.7\.6; Claude Desktop App\)$/);
-  });
+    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; Claude Desktop; 0\.7\.6; Claude Desktop App\)$/)
+  })
 
   test('handles Claude Desktop client info format', () => {
     const mockMcp = {
@@ -127,13 +127,13 @@ describe('useGetClientVersion', () => {
           version: '0.7.6',
         }),
       },
-    } as unknown as McpServer;
+    } as unknown as McpServer
 
-    const getUserAgent = useGetClientVersion(mockMcp);
-    const result = getUserAgent();
+    const getUserAgent = useGetClientVersion(mockMcp)
+    const result = getUserAgent()
 
-    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; Claude Desktop; 0\.7\.6\)$/);
-  });
+    expect(result).toMatch(/^MCP\/[\d.]+(-[\w.]+)? \(You\.com; Claude Desktop; 0\.7\.6\)$/)
+  })
 
   test('returns a function that can be called multiple times', () => {
     const mockMcp = {
@@ -143,18 +143,18 @@ describe('useGetClientVersion', () => {
           version: '1.0.0',
         }),
       },
-    } as unknown as McpServer;
+    } as unknown as McpServer
 
-    const getUserAgent = useGetClientVersion(mockMcp);
+    const getUserAgent = useGetClientVersion(mockMcp)
 
     // Call multiple times to ensure consistent results
-    const result1 = getUserAgent();
-    const result2 = getUserAgent();
-    const result3 = getUserAgent();
-
-    const pattern = /^MCP\/[\d.]+(-[\w.]+)? \(You\.com; test-client; 1\.0\.0\)$/;
-    expect(result1).toMatch(pattern);
-    expect(result2).toMatch(pattern);
-    expect(result3).toMatch(pattern);
-  });
-});
+    const result1 = getUserAgent()
+    const result2 = getUserAgent()
+    const result3 = getUserAgent()
+
+    const pattern = /^MCP\/[\d.]+(-[\w.]+)? \(You\.com; test-client; 1\.0\.0\)$/
+    expect(result1).toMatch(pattern)
+    expect(result2).toMatch(pattern)
+    expect(result3).toMatch(pattern)
+  })
+})

package/src/shared/use-client-version.ts

@@ -1,21 +1,21 @@
-import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-import packageJson from '../../package.json' with { type: 'json' };
+import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js'
+import packageJson from '../../package.json' with { type: 'json' }
 
 /**
  * Creates User-Agent string for API requests
  * Used by search and express agent API calls
  */
-const setUserAgent = (client: string) => `MCP/${packageJson.version} (You.com; ${client})`;
+const setUserAgent = (client: string) => `MCP/${packageJson.version} (You.com; ${client})`
 
 /**
  * Get's function that returns a formatted client version information into a string
  * Used by stdio.ts and http.ts for logging/debugging
  */
 export const useGetClientVersion = (mcp: McpServer) => () => {
-  const clientVersion = mcp.server.getClientVersion();
+  const clientVersion = mcp.server.getClientVersion()
   if (clientVersion) {
-    const { name, version, title, websiteUrl } = clientVersion;
-    return setUserAgent([name, version, title, websiteUrl].filter(Boolean).join('; '));
+    const { name, version, title, websiteUrl } = clientVersion
+    return setUserAgent([name, version, title, websiteUrl].filter(Boolean).join('; '))
   }
-  return setUserAgent('UNKNOWN');
-};
+  return setUserAgent('UNKNOWN')
+}

package/src/stdio.ts

@@ -1,24 +1,24 @@
 #!/usr/bin/env node
-import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
-import { registerContentsTool } from './contents/register-contents-tool.ts';
-import { registerExpressTool } from './express/register-express-tool.ts';
-import { getMCpServer } from './get-mcp-server.ts';
-import { registerSearchTool } from './search/register-search-tool.ts';
-import { useGetClientVersion } from './shared/use-client-version.ts';
+import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js'
+import { registerContentsTool } from './contents/register-contents-tool.ts'
+import { registerExpressTool } from './express/register-express-tool.ts'
+import { getMCpServer } from './get-mcp-server.ts'
+import { registerSearchTool } from './search/register-search-tool.ts'
+import { useGetClientVersion } from './shared/use-client-version.ts'
 
-const YDC_API_KEY = process.env.YDC_API_KEY;
+const YDC_API_KEY = process.env.YDC_API_KEY
 
 try {
-  const mcp = getMCpServer();
-  const getUserAgent = useGetClientVersion(mcp);
+  const mcp = getMCpServer()
+  const getUserAgent = useGetClientVersion(mcp)
 
-  registerSearchTool({ mcp, YDC_API_KEY, getUserAgent });
-  registerExpressTool({ mcp, YDC_API_KEY, getUserAgent });
-  registerContentsTool({ mcp, YDC_API_KEY, getUserAgent });
+  registerSearchTool({ mcp, YDC_API_KEY, getUserAgent })
+  registerExpressTool({ mcp, YDC_API_KEY, getUserAgent })
+  registerContentsTool({ mcp, YDC_API_KEY, getUserAgent })
 
-  const transport = new StdioServerTransport();
-  await mcp.connect(transport);
+  const transport = new StdioServerTransport()
+  await mcp.connect(transport)
 } catch (error) {
-  process.stderr.write(`Failed to start server: ${error}\n`);
-  process.exit(1);
+  process.stderr.write(`Failed to start server: ${error}\n`)
+  process.exit(1)
 }

package/src/tests/http.spec.ts

@@ -1,56 +1,56 @@
-import { afterAll, beforeAll, describe, expect, setDefaultTimeout, test } from 'bun:test';
-import httpApp from '../http.ts';
+import { afterAll, beforeAll, describe, expect, setDefaultTimeout, test } from 'bun:test'
+import httpApp from '../http.ts'
 
 // Increase default timeout for hooks to prevent intermittent failures
-setDefaultTimeout(15_000);
+setDefaultTimeout(15_000)
 
-let server: ReturnType<typeof Bun.serve>;
-let baseUrl: string;
-const testApiKey = process.env.YDC_API_KEY;
+let server: ReturnType<typeof Bun.serve>
+let baseUrl: string
+const testApiKey = process.env.YDC_API_KEY
 
 beforeAll(async () => {
   // Start HTTP server on random port
-  const port = Math.floor(Math.random() * 10000) + 20000;
-  baseUrl = `http://localhost:${port}`;
+  const port = Math.floor(Math.random() * 10000) + 20000
+  baseUrl = `http://localhost:${port}`
 
   // Start actual HTTP server using Bun
   server = Bun.serve({
     port,
     fetch: httpApp.fetch.bind(httpApp),
-  });
+  })
 
   // Wait a bit for server to start
-  await new Promise((resolve) => setTimeout(resolve, 500));
-});
+  await new Promise((resolve) => setTimeout(resolve, 500))
+})
 
 afterAll(async () => {
   if (server) {
-    server.stop();
+    server.stop()
     // Wait a bit for server to fully stop
-    await new Promise((resolve) => setTimeout(resolve, 100));
+    await new Promise((resolve) => setTimeout(resolve, 100))
   }
-});
+})
 
 describe('HTTP Server Endpoints', () => {
   test('health endpoint returns service status', async () => {
-    const response = await fetch(`${baseUrl}/mcp-health`);
+    const response = await fetch(`${baseUrl}/mcp-health`)
 
-    expect(response.status).toBe(200);
-    expect(response.headers.get('content-type')).toContain('application/json');
+    expect(response.status).toBe(200)
+    expect(response.headers.get('content-type')).toContain('application/json')
 
     const data = (await response.json()) as {
-      status: string;
-      timestamp: string;
-      version: string;
-      service: string;
-    };
-    expect(data).toHaveProperty('status', 'healthy');
-    expect(data).toHaveProperty('timestamp');
-    expect(data).toHaveProperty('version');
-    expect(data).toHaveProperty('service', 'youdotcom-mcp-server');
-    expect(typeof data.timestamp).toBe('string');
-    expect(typeof data.version).toBe('string');
-  });
+      status: string
+      timestamp: string
+      version: string
+      service: string
+    }
+    expect(data).toHaveProperty('status', 'healthy')
+    expect(data).toHaveProperty('timestamp')
+    expect(data).toHaveProperty('version')
+    expect(data).toHaveProperty('service', 'youdotcom-mcp-server')
+    expect(typeof data.timestamp).toBe('string')
+    expect(typeof data.version).toBe('string')
+  })
 
   test('mcp endpoint requires authorization header', async () => {
     const response = await fetch(`${baseUrl}/mcp`, {
@@ -59,14 +59,14 @@ describe('HTTP Server Endpoints', () => {
         'Content-Type': 'application/json',
       },
       body: JSON.stringify({}),
-    });
+    })
 
-    expect(response.status).toBe(401);
-    expect(response.headers.get('content-type')).toContain('text/plain');
+    expect(response.status).toBe(401)
+    expect(response.headers.get('content-type')).toContain('text/plain')
 
-    const text = await response.text();
-    expect(text).toBe('Unauthorized: Authorization header required');
-  });
+    const text = await response.text()
+    expect(text).toBe('Unauthorized: Authorization header required')
+  })
 
   test('mcp endpoint requires Bearer token format', async () => {
     const response = await fetch(`${baseUrl}/mcp`, {
@@ -76,14 +76,14 @@ describe('HTTP Server Endpoints', () => {
         Authorization: 'InvalidFormat token123',
       },
       body: JSON.stringify({}),
-    });
+    })
 
-    expect(response.status).toBe(401);
-    expect(response.headers.get('content-type')).toContain('text/plain');
+    expect(response.status).toBe(401)
+    expect(response.headers.get('content-type')).toContain('text/plain')
 
-    const text = await response.text();
-    expect(text).toBe('Unauthorized: Bearer token required');
-  });
+    const text = await response.text()
+    expect(text).toBe('Unauthorized: Bearer token required')
+  })
 
   test('mcp endpoint accepts valid Bearer token', async () => {
     const response = await fetch(`${baseUrl}/mcp`, {
@@ -106,19 +106,19 @@ describe('HTTP Server Endpoints', () => {
           },
         },
       }),
-    });
+    })
 
-    expect(response.status).toBe(200);
-    expect(response.headers.get('content-type')).toContain('text/event-stream');
+    expect(response.status).toBe(200)
+    expect(response.headers.get('content-type')).toContain('text/event-stream')
 
     // StreamableHTTPTransport uses SSE format, so response will be streaming
-    const text = await response.text();
-    expect(text).toContain('data:');
-    expect(text).toContain('jsonrpc');
-    expect(text).toContain('result');
-    expect(text).toContain('protocolVersion');
-    expect(text).toContain('capabilities');
-  });
+    const text = await response.text()
+    expect(text).toContain('data:')
+    expect(text).toContain('jsonrpc')
+    expect(text).toContain('result')
+    expect(text).toContain('protocolVersion')
+    expect(text).toContain('capabilities')
+  })
 
   test('mcp endpoint with trailing slash works identically', async () => {
     const response = await fetch(`${baseUrl}/mcp/`, {
@@ -141,18 +141,18 @@ describe('HTTP Server Endpoints', () => {
           },
         },
       }),
-    });
+    })
 
-    expect(response.status).toBe(200);
-    expect(response.headers.get('content-type')).toContain('text/event-stream');
+    expect(response.status).toBe(200)
+    expect(response.headers.get('content-type')).toContain('text/event-stream')
 
-    const text = await response.text();
-    expect(text).toContain('data:');
-    expect(text).toContain('jsonrpc');
-    expect(text).toContain('result');
-    expect(text).toContain('protocolVersion');
-    expect(text).toContain('capabilities');
-  });
+    const text = await response.text()
+    expect(text).toContain('data:')
+    expect(text).toContain('jsonrpc')
+    expect(text).toContain('result')
+    expect(text).toContain('protocolVersion')
+    expect(text).toContain('capabilities')
+  })
 
   test('mcp endpoint with trailing slash requires authorization', async () => {
     const response = await fetch(`${baseUrl}/mcp/`, {
@@ -161,15 +161,15 @@ describe('HTTP Server Endpoints', () => {
         'Content-Type': 'application/json',
       },
       body: JSON.stringify({}),
-    });
+    })
 
-    expect(response.status).toBe(401);
-    expect(response.headers.get('content-type')).toContain('text/plain');
+    expect(response.status).toBe(401)
+    expect(response.headers.get('content-type')).toContain('text/plain')
 
-    const text = await response.text();
-    expect(text).toBe('Unauthorized: Authorization header required');
-  });
-});
+    const text = await response.text()
+    expect(text).toBe('Unauthorized: Authorization header required')
+  })
+})
 
 describe('HTTP MCP Endpoint Basic Functionality', () => {
   test('mcp endpoint responds to valid Bearer token', async () => {
@@ -186,15 +186,15 @@ describe('HTTP MCP Endpoint Basic Functionality', () => {
         method: 'ping',
         id: 1,
       }),
-    });
+    })
 
     // Should get a response (not 401/403), even if the method isn't supported
-    expect(response.status).not.toBe(401);
-    expect(response.status).not.toBe(403);
+    expect(response.status).not.toBe(401)
+    expect(response.status).not.toBe(403)
 
     // Should be SSE response for StreamableHTTPTransport
-    expect(response.headers.get('content-type')).toContain('text/event-stream');
-  });
+    expect(response.headers.get('content-type')).toContain('text/event-stream')
+  })
 
   test('mcp endpoint processes JSON-RPC requests', async () => {
     // Test basic JSON-RPC structure handling
@@ -210,17 +210,17 @@ describe('HTTP MCP Endpoint Basic Functionality', () => {
         method: 'unknown-method',
         id: 123,
       }),
-    });
+    })
 
-    expect(response.status).toBe(200);
-    expect(response.headers.get('content-type')).toContain('text/event-stream');
+    expect(response.status).toBe(200)
+    expect(response.headers.get('content-type')).toContain('text/event-stream')
 
     // StreamableHTTPTransport uses SSE format
-    const text = await response.text();
-    expect(text).toContain('data:');
-    expect(text).toContain('jsonrpc');
-    expect(text).toContain('123');
-  });
+    const text = await response.text()
+    expect(text).toContain('data:')
+    expect(text).toContain('jsonrpc')
+    expect(text).toContain('123')
+  })
 
   test('mcp endpoint extracts Bearer token correctly', async () => {
     // Test that different tokens are processed
@@ -236,7 +236,7 @@ describe('HTTP MCP Endpoint Basic Functionality', () => {
         method: 'test',
         id: 1,
       }),
-    });
+    })
 
     const response2 = await fetch(`${baseUrl}/mcp`, {
       method: 'POST',
@@ -250,12 +250,12 @@ describe('HTTP MCP Endpoint Basic Functionality', () => {
         method: 'test',
         id: 2,
       }),
-    });
+    })
 
     // Both should be processed (not authentication errors)
-    expect(response1.status).not.toBe(401);
-    expect(response2.status).not.toBe(401);
-  });
+    expect(response1.status).not.toBe(401)
+    expect(response2.status).not.toBe(401)
+  })
 
   test('mcp endpoint uses StreamableHTTPTransport', async () => {
     // Test that the transport is properly handling requests
@@ -271,17 +271,17 @@ describe('HTTP MCP Endpoint Basic Functionality', () => {
         method: 'test',
         id: 42,
       }),
-    });
+    })
 
-    expect(response.status).toBe(200);
-    expect(response.headers.get('content-type')).toContain('text/event-stream');
+    expect(response.status).toBe(200)
+    expect(response.headers.get('content-type')).toContain('text/event-stream')
 
     // StreamableHTTPTransport uses SSE format
-    const text = await response.text();
-    expect(text).toContain('data:');
-    expect(text).toContain('jsonrpc');
-    expect(text).toContain('42');
-  });
+    const text = await response.text()
+    expect(text).toContain('data:')
+    expect(text).toContain('jsonrpc')
+    expect(text).toContain('42')
+  })
 
   test(
     'mcp server handles search tool request for latest tech news',
@@ -305,18 +305,18 @@ describe('HTTP MCP Endpoint Basic Functionality', () => {
             },
           },
         }),
-      });
+      })
 
-      expect(response.status).toBe(200);
-      expect(response.headers.get('content-type')).toContain('text/event-stream');
+      expect(response.status).toBe(200)
+      expect(response.headers.get('content-type')).toContain('text/event-stream')
 
-      const text = await response.text();
-      expect(text).toContain('data:');
-      expect(text).toContain('jsonrpc');
-      expect(text).toContain('result');
-      expect(text).toContain('latest tech news');
-      expect(text).toContain('Search Results for');
+      const text = await response.text()
+      expect(text).toContain('data:')
+      expect(text).toContain('jsonrpc')
+      expect(text).toContain('result')
+      expect(text).toContain('latest tech news')
+      expect(text).toContain('Search Results for')
     },
     { retry: 2 },
-  );
-});
+  )
+})