fix: lint

Author: fmenezes

src/index.ts

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-import logger, { LogId, LoggerBase, McpLogger, DiskLogger, ConsoleLogger } from "./common/logger.js";
+import logger, { LogId } from "./common/logger.js";
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { config } from "./common/config.js";
 import { Session } from "./common/session.js";

src/server.ts

@@ -189,12 +189,14 @@ export class Server {
     }
 
     private async validateConfig(): Promise<void> {
-        if (this.userConfig.transport !== "http" && this.userConfig.transport !== "stdio") {
-            throw new Error(`Invalid transport: ${this.userConfig.transport}`);
+        const transport = this.userConfig.transport as string;
+        if (transport !== "http" && transport !== "stdio") {
+            throw new Error(`Invalid transport: ${transport}`);
         }
 
-        if (this.userConfig.telemetry !== "enabled" && this.userConfig.telemetry !== "disabled") {
-            throw new Error(`Invalid telemetry: ${this.userConfig.telemetry}`);
+        const telemetry = this.userConfig.telemetry as string;
+        if (telemetry !== "enabled" && telemetry !== "disabled") {
+            throw new Error(`Invalid telemetry: ${telemetry}`);
         }
 
         if (this.userConfig.httpPort < 1 || this.userConfig.httpPort > 65535) {
@@ -210,7 +212,7 @@ export class Server {
             throw new Error("Duplicate loggers found in config");
         }
 
-        for (const loggerType of this.userConfig.loggers) {
+        for (const loggerType of this.userConfig.loggers as string[]) {
             if (loggerType !== "mcp" && loggerType !== "disk" && loggerType !== "stderr") {
                 throw new Error(`Invalid logger: ${loggerType}`);
             }

src/transports/streamableHttpTransport.ts

@@ -22,7 +22,7 @@ export function createHttpTransport(): StreamableHTTPServerTransport {
             logger.error(
                 LogId.streamableHttpTransportRequestFailure,
                 "streamableHttpTransport",
-                `Error handling request: ${error}`
+                `Error handling request: ${error instanceof Error ? error.message : String(error)}`
             );
             res.sendStatus(400);
         }
@@ -35,7 +35,7 @@ export function createHttpTransport(): StreamableHTTPServerTransport {
             logger.error(
                 LogId.streamableHttpTransportRequestFailure,
                 "streamableHttpTransport",
-                `Error handling request: ${error}`
+                `Error handling request: ${error instanceof Error ? error.message : String(error)}`
             );
             res.sendStatus(400);
         }
@@ -48,7 +48,7 @@ export function createHttpTransport(): StreamableHTTPServerTransport {
             logger.error(
                 LogId.streamableHttpTransportRequestFailure,
                 "streamableHttpTransport",
-                `Error handling request: ${error}`
+                `Error handling request: ${error instanceof Error ? error.message : String(error)}`
             );
             res.sendStatus(400);
         }
@@ -62,26 +62,19 @@ export function createHttpTransport(): StreamableHTTPServerTransport {
         );
     });
 
-    transport.onclose = async () => {
-        try {
-            logger.info(LogId.streamableHttpTransportCloseRequested, "streamableHttpTransport", `Closing server`);
-            await new Promise<void>((resolve, reject) => {
-                server.close((err?: Error) => {
-                    if (err) {
-                        reject(err);
-                    } else {
-                        resolve();
-                    }
-                });
-            });
+    transport.onclose = () => {
+        logger.info(LogId.streamableHttpTransportCloseRequested, "streamableHttpTransport", `Closing server`);
+        server.close((err?: Error) => {
+            if (err) {
+                logger.error(
+                    LogId.streamableHttpTransportCloseFailure,
+                    "streamableHttpTransport",
+                    `Error closing server: ${err.message}`
+                );
+                return;
+            }
             logger.info(LogId.streamableHttpTransportCloseSuccess, "streamableHttpTransport", `Server closed`);
-        } catch (error: unknown) {
-            logger.error(
-                LogId.streamableHttpTransportCloseFailure,
-                "streamableHttpTransport",
-                `Error closing server: ${error instanceof Error ? error.message : String(error)}`
-            );
-        }
+        });
     };
 
     return transport;