fix: ensure MCP_PROXY_PORT is included when auth is disabled

Address PR feedback by restructuring getClientUrl to always include
MCP_PROXY_PORT parameter when using non-default port, regardless of
auth status. Also improved code clarity by using params.size check.

Author: felixweinberger

client/bin/start.js

@@ -17,19 +17,20 @@ function getClientUrl(port, authDisabled, sessionToken, serverPort) {
   const host = process.env.HOST || "localhost";
   const baseUrl = `http://${host}:${port}`;
 
-  if (authDisabled) {
-    return baseUrl;
-  }
-
   const params = new URLSearchParams();
-  params.set("MCP_PROXY_AUTH_TOKEN", sessionToken);
 
-  // Add server port if it's not the default
+  // Always add the proxy port if it's not the default
   if (serverPort && serverPort !== DEFAULT_MCP_PROXY_LISTEN_PORT) {
     params.set("MCP_PROXY_PORT", serverPort);
   }
 
-  return `${baseUrl}/?${params.toString()}`;
+  // Only add auth token if auth is enabled
+  if (!authDisabled) {
+    params.set("MCP_PROXY_AUTH_TOKEN", sessionToken);
+  }
+
+  // Only append query string if there are parameters
+  return params.size > 0 ? `${baseUrl}/?${params.toString()}` : baseUrl;
 }
 
 async function startDevServer(serverOptions) {