8 Commits

Author SHA1 Message Date
Abdelkader Boudih
6c67bf6d6d Merge branch 'main' into main 2025-03-31 18:32:18 +00:00
Abdelkader Boudih
da9dd09765 refactor: Update default ports for MCPI client and MPCP server
Changes the default ports used by the MCP Inspector client UI and the MCP Proxy server to avoid conflicts with common development ports and provide a memorable mnemonic based on T9 mapping.
2025-03-31 00:31:28 +00:00
cliffhall
054741be03 Run prettier. 2025-03-29 12:36:03 -04:00
cliffhall
1b754f52ca This fixes #214 where when the inspector is started, it can report that the inspector is up, when in fact it isn't because the address is already in use. It catches this condition, as well as the condition where the proxy server port is in use, reports it, and exits.
* In bin/cli.js
  - in the delay function, have the setTimeout return a true value.
  - try the server's spawnPromise call and within the try block, use Promise.race to get the return value of the first promise to resolve. If the server is up, it will not resolve and the 2 second delay will resolve with true, telling us the server is ok. Any error will have been reported by the server startup process, so we will not pile on with more output in the catch block.
  - If the server started ok, then we will await the spawnPromise call for starting the client. If the client fails to start it will report and exit, otherwise we are done and both servers are running and have reported as much. If an error is caught and it isn't SIGINT or if process.env.DEBUG is true then the error will be thrown before exiting.
* In client/bin/cli.js
  - add a "listening" handler to the server, logging that the MCP inspector is up at the given port
  - Add an "error" handler to the server that reports  that the client port is in use if the error message includes "EADDRINUSE", otherwise throw the error so the entire contents can be seen.
* In server/src/index.ts
  - add a "listening" handler to the server, logging that the Proxy server is up at the given port
  - Add an "error" handler to the server that reports  that the server port is in use if the error message includes "EADDRINUSE", otherwise throw the error so the entire contents can be seen.
* In package.json
  - in preview script
    - add --port 5173 to start the client on the proper port. This was useful in getting an instance of the client running before trying the start script. otherwise it starts on 4173
2025-03-28 11:59:21 -04:00
Jerome
4201b31a24 Fix OAuth callback route in production builds
Add SPA routing configuration to serve-handler to ensure /oauth/callback
routes are correctly handled in production builds.
2025-02-21 14:47:31 +00:00
Ashwin Bhat
f684d2e891 improve inspector startup message 2024-11-19 16:58:33 -08:00
Justin Spahr-Summers
3a9b08bd37 Pick non-conflicting default client port 2024-11-11 11:11:27 +00:00
Justin Spahr-Summers
196f2f801d Use serve to create npm package for the client 2024-11-11 11:04:42 +00:00