@brave/brave-search-mcp-server 1.3.5 → 1.3.6

package/dist/BraveAPI/index.js

@@ -1,6 +1,5 @@
 import config from '../config.js';
 import { stringify } from '../utils.js';
-import ClientLogger from '../ClientLogger.js';
 const typeToPathMap = {
     images: '/res/v1/images/search',
     localPois: '/res/v1/local/pois',
@@ -23,7 +22,6 @@ requestHeaders = {}) {
     // Determine URL, and setup parameters
     const url = new URL(`https://api.search.brave.com${typeToPathMap[endpoint]}`);
     const queryParams = new URLSearchParams();
-    await ClientLogger.log('info', `Preparing to issue request to ${url.toString()}`);
     // TODO (Sampson): Move param-construction/validation to modules
     for (const [key, value] of Object.entries(parameters)) {
         // The 'ids' parameter is expected to appear multiple times for multiple IDs
@@ -65,12 +63,10 @@ requestHeaders = {}) {
             queryParams.set(key === 'query' ? 'q' : key, value.toString());
         }
     }
-    await ClientLogger.log('debug', `Using parameters: ${queryParams.toString()}`);
     // Issue Request
     const urlWithParams = url.toString() + '?' + queryParams.toString();
     const headers = { ...defaultRequestHeaders, ...requestHeaders };
     const response = await fetch(urlWithParams, { headers });
-    await ClientLogger.log('debug', `Received response from ${urlWithParams}`);
     // Handle Error
     if (!response.ok) {
         let errorMessage = `${response.status} ${response.statusText}`;
@@ -81,13 +77,11 @@ requestHeaders = {}) {
         catch (error) {
             errorMessage += `\n${await response.text()}`;
         }
-        await ClientLogger.log('error', errorMessage);
         // TODO (Sampson): Setup proper error handling, updating state, etc.
         throw new Error(errorMessage);
     }
     // Return Response
     const responseBody = await response.json();
-    await ClientLogger.log('debug', `Returning response: ${stringify(responseBody, true)}`);
     return responseBody;
 }
 export default {

package/dist/index.js

@@ -7,18 +7,13 @@ async function main() {
         console.error('Invalid configuration');
         process.exit(1);
     }
-    // default to http server
-    if (!options.transport || options.transport === 'http') {
-        httpServer.start();
-        return;
-    }
     // stdio requires explicit request
     if (options.transport === 'stdio') {
         await stdioServer.start();
         return;
     }
-    console.error('Invalid transport');
-    process.exit(1);
+    // default to http server
+    httpServer.start();
 }
 main().catch((error) => {
     console.error(error);

package/dist/protocols/http.js

@@ -1,9 +1,9 @@
 import { randomUUID } from 'node:crypto';
 import express from 'express';
 import config from '../config.js';
-import { mcpServer } from '../server.js';
+import createMcpServer from '../server.js';
 import { StreamableHTTPServerTransport } from '@modelcontextprotocol/sdk/server/streamableHttp.js';
-import { isInitializeRequest, ListToolsRequestSchema } from '@modelcontextprotocol/sdk/types.js';
+import { ListToolsRequestSchema } from '@modelcontextprotocol/sdk/types.js';
 const yieldGenericServerError = (res) => {
     res.status(500).json({
         id: null,
@@ -12,34 +12,34 @@ const yieldGenericServerError = (res) => {
     });
 };
 const transports = new Map();
-export const getTransport = async (request) => {
+const isListToolsRequest = (value) => ListToolsRequestSchema.safeParse(value).success;
+const getTransport = async (request) => {
     // Check for an existing session
     const sessionId = request.headers['mcp-session-id'];
     if (sessionId && transports.has(sessionId)) {
         return transports.get(sessionId);
     }
-    // Is the client attempting to initialize a new session?
-    if (isInitializeRequest(request.body)) {
-        const transport = new StreamableHTTPServerTransport({
-            sessionIdGenerator: () => randomUUID(),
-            onsessioninitialized: (sessionId) => {
-                transports.set(sessionId, transport);
-            },
-        });
-        await mcpServer.connect(transport);
-        return transport;
-    }
     // We have a special case where we'll permit ListToolsRequest w/o a session ID
-    if (request.body.method === ListToolsRequestSchema.shape.method.value) {
+    if (!sessionId && isListToolsRequest(request.body)) {
         const transport = new StreamableHTTPServerTransport({
             sessionIdGenerator: undefined,
         });
+        const mcpServer = createMcpServer();
         await mcpServer.connect(transport);
         return transport;
     }
-    throw new Error('Invalid request: must be an initialization request, include a valid session ID, or be a ListTools method request');
+    // Otherwise, start a new transport/session
+    const transport = new StreamableHTTPServerTransport({
+        sessionIdGenerator: () => randomUUID(),
+        onsessioninitialized: (sessionId) => {
+            transports.set(sessionId, transport);
+        },
+    });
+    const mcpServer = createMcpServer();
+    await mcpServer.connect(transport);
+    return transport;
 };
-export const createApp = () => {
+const createApp = () => {
     const app = express();
     app.use(express.json());
     app.all('/mcp', async (req, res) => {
@@ -59,14 +59,14 @@ export const createApp = () => {
     });
     return app;
 };
-export const start = () => {
+const start = () => {
     if (!config.ready) {
         console.error('Invalid configuration');
         process.exit(1);
     }
     const app = createApp();
     app.listen(config.port, config.host, () => {
-        console.error(`Server is running on http://${config.host}:${config.port}/mcp`);
+        console.log(`Server is running on http://${config.host}:${config.port}/mcp`);
     });
 };
 export default { start, createApp };

package/dist/protocols/stdio.js

@@ -1,11 +1,11 @@
-import { mcpServer } from '../server.js';
+import newMcpServer from '../server.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
-export const createTransport = () => {
+const createTransport = () => {
     return new StdioServerTransport();
 };
-export const start = async () => {
+const start = async () => {
     const transport = createTransport();
+    const mcpServer = newMcpServer();
     await mcpServer.connect(transport);
-    console.error('Stdio server started');
 };
 export default { start, createTransport };

package/dist/server.js

@@ -1,20 +1,31 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import tools from './tools/index.js';
-import ClientLogger, { maybeRegisterCustomSetLevelRequestHandler } from './ClientLogger.js';
-export const mcpServer = new McpServer({
-    version: '0.1.0',
-    name: 'brave-search-mcp-server',
-    title: 'Brave Search MCP Server',
-}, {
-    capabilities: {
-        logging: {},
-        tools: { listChanged: false },
-    },
-    instructions: `Use this server to search the Web for various types of data via the Brave Search API.`,
-});
-ClientLogger.setServer(mcpServer);
-// https://github.com/modelcontextprotocol/typescript-sdk/issues/871
-maybeRegisterCustomSetLevelRequestHandler(mcpServer.server);
-for (const tool of Object.values(tools)) {
-    mcpServer.tool(tool.name, tool.description, tool.inputSchema, tool.annotations, tool.execute);
+import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+import { SetLevelRequestSchema } from '@modelcontextprotocol/sdk/types.js';
+import pkg from '../package.json' with { type: 'json' };
+export default function createMcpServer() {
+    const mcpServer = new McpServer({
+        version: pkg.version,
+        name: 'brave-search-mcp-server',
+        title: 'Brave Search MCP Server',
+    }, {
+        capabilities: {
+            logging: {},
+            tools: { listChanged: false },
+        },
+        instructions: `Use this server to search the Web for various types of data via the Brave Search API.`,
+    });
+    try {
+        mcpServer.server.assertCanSetRequestHandler(SetLevelRequestSchema.shape.method.value);
+        mcpServer.server.setRequestHandler(SetLevelRequestSchema, () => ({}));
+    }
+    catch (error) {
+        /**
+         * An error here signifies native-handling of the SetLevel request.
+         * See https://github.com/modelcontextprotocol/typescript-sdk/issues/871.
+         */
+    }
+    for (const tool of Object.values(tools)) {
+        mcpServer.tool(tool.name, tool.description, tool.inputSchema, tool.annotations, tool.execute);
+    }
+    return mcpServer;
 }

package/dist/tools/images/index.js

@@ -1,7 +1,6 @@
 import params from './params.js';
 import API from '../../BraveAPI/index.js';
 import { stringify } from '../../utils.js';
-import ClientLogger from '../../ClientLogger.js';
 export const name = 'brave_image_search';
 export const annotations = {
     title: 'Brave Image Search',
@@ -28,7 +27,6 @@ export const execute = async (params) => {
     return { content, isError: false };
 };
 async function fetchImage(url) {
-    await ClientLogger.log('info', `Fetching image data from ${url}`);
     try {
         const response = await fetch(url);
         const buffer = await response.arrayBuffer();
@@ -38,7 +36,6 @@ async function fetchImage(url) {
         };
     }
     catch (error) {
-        await ClientLogger.log('error', `Error fetching image data from ${url}: ${error}`);
         return null;
     }
 }

package/dist/tools/summarizer/index.js

@@ -1,6 +1,5 @@
 import { summarizerQueryParams } from './params.js';
 import API from '../../BraveAPI/index.js';
-import ClientLogger from '../../ClientLogger.js';
 export const name = 'brave_summarizer';
 export const annotations = {
     title: 'Brave Summarizer',
@@ -70,7 +69,6 @@ const pollForSummary = async (params, pollInterval = 50, attempts = 20) => {
             }
         }
         catch (error) {
-            await ClientLogger.log('error', `Error polling for summarizer results: ${error}`);
             await new Promise((resolve) => setTimeout(resolve, pollInterval));
         }
         attempts--;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brave/brave-search-mcp-server",
-  "version": "1.3.5",
+  "version": "1.3.6",
   "description": "MCP server for Brave Search. Uses the Brave Search API to return results from the web, including ranked links, images, and videos, as well as AI summaries of pages, rich results, and more.",
   "keywords": [
     "api",
@@ -33,7 +33,7 @@
     "inspector:stdio": "npx @modelcontextprotocol/inspector --transport stdio"
   },
   "dependencies": {
-    "@modelcontextprotocol/sdk": "1.17.2",
+    "@modelcontextprotocol/sdk": "1.17.3",
     "commander": "14.0.0",
     "dotenv": "^17.2.1",
     "express": "5.1.0",

package/dist/ClientLogger.js

@@ -1,63 +0,0 @@
-import { LoggingLevelSchema, SetLevelRequestSchema } from '@modelcontextprotocol/sdk/types.js';
-import config from './config.js';
-let mcpServer;
-const setServer = (server) => {
-    mcpServer = server;
-};
-let customLogger;
-let currentLevel = LoggingLevelSchema.options.indexOf(config.loggingLevel);
-/**
- * Attempt to register a custom set level request handler.
- * If the method is already handled by the SDK/Server, we'll ignore the error.
- * @see https://github.com/modelcontextprotocol/typescript-sdk/issues/871
- * @param server {Server} The server to register the handler on.
- */
-export const maybeRegisterCustomSetLevelRequestHandler = (server) => {
-    try {
-        server.assertCanSetRequestHandler(SetLevelRequestSchema.shape.method.value);
-        server.setRequestHandler(SetLevelRequestSchema, async (request) => {
-            await log('info', `Setting logging level to ${request.params.level}`);
-            currentLevel = LoggingLevelSchema.options.indexOf(request.params.level);
-            return {};
-        });
-    }
-    catch (error) {
-        console.error(`Failed to register custom SetLevelRequest handler. The SDK may now provide its own handler. See https://github.com/modelcontextprotocol/typescript-sdk/issues/871 for more information.`, error);
-    }
-};
-const log = async (level, message) => {
-    // If a custom logger exists, call it, and let it handle it's own log-level filtering
-    if (customLogger) {
-        await customLogger(level, message);
-        return;
-    }
-    // If the log-level is less than the current log-level, skip it
-    if (LoggingLevelSchema.options.indexOf(level) < currentLevel) {
-        return;
-    }
-    // Fall back to default logger if no custom logger is set
-    const time = new Date().toISOString();
-    if (!mcpServer?.isConnected()) {
-        console.error(`${time} [${level}] ${message}`);
-        return;
-    }
-    try {
-        await mcpServer.server.sendLoggingMessage({ level, data: { message, time } });
-    }
-    catch (error) {
-        console.error(`Error sending logging message: ${error}`);
-    }
-};
-const setLogger = (logger) => {
-    customLogger = logger;
-};
-const setLoggingLevel = (desiredLevel) => {
-    const desiredLevelIndex = LoggingLevelSchema.options.indexOf(desiredLevel);
-    if (desiredLevelIndex === -1) {
-        console.error(`Invalid logging level: ${desiredLevel}. Must be one of: ${LoggingLevelSchema.options.join(', ')}`);
-        return;
-    }
-    currentLevel = desiredLevelIndex;
-};
-const getLoggingLevel = () => LoggingLevelSchema.options[currentLevel];
-export default { log, setServer, setLogger, setLoggingLevel, getLoggingLevel };