Merge pull request #64 from runbasehq/dev

fix: modify behaviour on empty mcp server auth token

Author: jeresrc

packages/mcp-check/CHANGELOG.md

@@ -1,5 +1,11 @@
 # mcp-testing-library
 
+## 0.4.6
+
+### Patch Changes
+
+- Fix empty authorization token config option
+
 ## 0.4.5
 
 ### Patch Changes

packages/mcp-check/package.json

@@ -1,7 +1,7 @@
 {
   "name": "mcp-check",
   "module": "dist/src/index.js",
-  "version": "0.4.5",
+  "version": "0.4.6",
   "type": "module",
   "main": "dist/src/index.js",
   "types": "dist/src/index.d.ts",

packages/mcp-check/src/index.ts

@@ -84,15 +84,15 @@ export class Scorer {
 
 export class McpServer {
   public url: string;
-  public authorizationToken: string;
+  public authorizationToken?: string;
   public name: string;
   public type: string;
 
-  constructor({ url, authorizationToken, name, type }: { url: string; authorizationToken: string; name: string; type: string }) {
+  constructor({ url, authorizationToken, name, type }: { url: string; authorizationToken?: string; name: string; type: string }) {
     this.url = url;
-    this.authorizationToken = authorizationToken;
     this.name = name;
     this.type = type;
+    this.authorizationToken = authorizationToken;
   }
 }
 

packages/mcp-check/src/providers/anthropic.ts

@@ -127,9 +127,9 @@ export class AnthropicProvider extends Provider {
       mcp_servers: [
         {
           url: this.mcpServer.url,
-          authorization_token: this.mcpServer.authorizationToken,
           name: this.mcpServer.name,
           type: "url",
+          ...(this.mcpServer.authorizationToken && { authorization_token: this.mcpServer.authorizationToken }),
         },
       ],
       betas: ["mcp-client-2025-04-04"],

packages/mcp-check/src/providers/openai.ts

@@ -119,9 +119,7 @@ export class OpenAIProvider extends Provider {
           require_approval: "never",
           server_label: this.mcpServer.name,
           server_url: this.mcpServer.url,
-          headers: {
-            Authorization: this.mcpServer.authorizationToken || "",
-          },
+          ...(this.mcpServer.authorizationToken && { headers: { Authorization: this.mcpServer.authorizationToken } }),
         },
       ],
       input: this.promptText,