From a4a8a59a5789a5ff06962968e05958c04d40e15a Mon Sep 17 00:00:00 2001 From: cliffhall Date: Thu, 11 Sep 2025 17:08:42 -0400 Subject: [PATCH] * In src/everything/package.json - bump TS SDK to 1.18.0 * In src/everything/stdio.ts - remove logging related imports - remove custom log-level handling, now handled automatically by the SDK * In src/everything/everything.ts - remove console.log of sessionId --- package-lock.json | 8 ++++---- src/everything/everything.ts | 1 - src/everything/package.json | 2 +- src/everything/stdio.ts | 38 +----------------------------------- 4 files changed, 6 insertions(+), 43 deletions(-) diff --git a/package-lock.json b/package-lock.json index fe72fc4328..34ecd5d41b 100644 --- a/package-lock.json +++ b/package-lock.json @@ -5818,7 +5818,7 @@ "version": "0.6.2", "license": "MIT", "dependencies": { - "@modelcontextprotocol/sdk": "^1.17.5", + "@modelcontextprotocol/sdk": "^1.18.0", "express": "^4.21.1", "zod": "^3.23.8", "zod-to-json-schema": "^3.23.5" @@ -5833,9 +5833,9 @@ } }, "src/everything/node_modules/@modelcontextprotocol/sdk": { - "version": "1.17.5", - "resolved": "https://registry.npmjs.org/@modelcontextprotocol/sdk/-/sdk-1.17.5.tgz", - "integrity": "sha512-QakrKIGniGuRVfWBdMsDea/dx1PNE739QJ7gCM41s9q+qaCYTHCdsIBXQVVXry3mfWAiaM9kT22Hyz53Uw8mfg==", + "version": "1.18.0", + "resolved": "https://registry.npmjs.org/@modelcontextprotocol/sdk/-/sdk-1.18.0.tgz", + "integrity": "sha512-JvKyB6YwS3quM+88JPR0axeRgvdDu3Pv6mdZUy+w4qVkCzGgumb9bXG/TmtDRQv+671yaofVfXSQmFLlWU5qPQ==", "license": "MIT", "dependencies": { "ajv": "^6.12.6", diff --git a/src/everything/everything.ts b/src/everything/everything.ts index 3635174fde..e8b2f6724e 100644 --- a/src/everything/everything.ts +++ b/src/everything/everything.ts @@ -196,7 +196,6 @@ export const createServer = () => { }, 10000); } - console.log(sessionId) const maybeAppendSessionId = sessionId ? ` - SessionId ${sessionId}`: ""; const messages: { level: LoggingLevel; data: string }[] = [ { level: "debug", data: `Debug-level message${maybeAppendSessionId}` }, diff --git a/src/everything/package.json b/src/everything/package.json index 8e417a28a4..c0a240de49 100644 --- a/src/everything/package.json +++ b/src/everything/package.json @@ -22,7 +22,7 @@ "start:streamableHttp": "node dist/streamableHttp.js" }, "dependencies": { - "@modelcontextprotocol/sdk": "^1.17.5", + "@modelcontextprotocol/sdk": "^1.18.0", "express": "^4.21.1", "zod": "^3.23.8", "zod-to-json-schema": "^3.23.5" diff --git a/src/everything/stdio.ts b/src/everything/stdio.ts index 5e01fd7fe9..e443a983b6 100644 --- a/src/everything/stdio.ts +++ b/src/everything/stdio.ts @@ -2,48 +2,12 @@ import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"; import { createServer } from "./everything.js"; -import { - LoggingLevel, - LoggingLevelSchema, - LoggingMessageNotification, - SetLevelRequestSchema -} from "@modelcontextprotocol/sdk/types.js"; console.error('Starting default (STDIO) server...'); async function main() { const transport = new StdioServerTransport(); - const {server, cleanup, startNotificationIntervals } = createServer(); - - // Currently, for STDIO servers, automatic log-level support is not available, as levels are tracked by sessionId. - // The listener will be set, so if the STDIO server advertises support for logging, and the client sends a setLevel - // request, it will be handled and thus not throw a "Method not found" error. However, the STDIO server will need to - // implement its own listener and level handling for now. This will be remediated in a future SDK version. - - let logLevel: LoggingLevel = "debug"; - server.setRequestHandler(SetLevelRequestSchema, async (request) => { - const { level } = request.params; - logLevel = level; - return {}; - }); - - server.sendLoggingMessage = async (params: LoggingMessageNotification["params"], _: string|undefined): Promise => { - const LOG_LEVEL_SEVERITY = new Map( - LoggingLevelSchema.options.map((level, index) => [level, index]) - ); - - const isMessageIgnored = (level: LoggingLevel): boolean => { - const currentLevel = logLevel; - return (currentLevel) - ? LOG_LEVEL_SEVERITY.get(level)! < LOG_LEVEL_SEVERITY.get(currentLevel)! - : false; - }; - - if (!isMessageIgnored(params.level)) { - return server.notification({method: "notifications/message", params}) - } - - } + const {server, cleanup, startNotificationIntervals} = createServer(); await server.connect(transport); startNotificationIntervals();