Commit Graph

26 Commits

Author SHA1 Message Date
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
Max Gerber
029823bb92 Merge branch 'main' into patch-1 2025-03-25 11:05:42 -04:00
Max Gerber
fb667fd4d0 fix: Prefer 127.0.0.1 over localhost 2025-03-14 16:16:31 -07:00
Ola Hungerford
60c4645eaf Fix formatting 2025-03-12 08:20:11 -07:00
Ryan Rozich
fe8b1ee88b remove comments 2025-03-11 22:46:38 -05:00
Ryan Rozich
04a90e8d89 Fix environment variable parsing to handle values with equals signs 2025-03-11 22:46:38 -05:00
Ashwin Bhat
052de8690d respond to PR feedback 2025-01-10 07:51:55 -08:00
Ashwin Bhat
a976aefb39 allow passing env vars to server from command line 2025-01-10 07:51:54 -08:00
Ashwin Bhat
295ccac27e fix arg passing 2024-11-27 19:17:47 -05:00
Ani Betts
98470a12f9 Make stdout/error echo for client and server 2024-11-27 15:57:02 +01:00
Ashwin Bhat
f876b1ec0d consolidate server URL configuration 2024-11-26 13:40:28 -05:00
Ashwin Bhat
a3d542c0a3 make server port configurable via URL query param 2024-11-26 13:12:45 -05:00
Ani Betts
1f28b4474c Don't eat the env, add PORT 2024-11-26 18:08:43 +01:00
Anaïs Betts
d69d67cb64 Fix server args 2024-11-26 17:33:22 +01:00
Anaïs Betts
7792070d81 Add debugging 2024-11-26 17:18:33 +01:00
Anaïs Betts
34a2843756 resolve usually better than join 2024-11-26 17:11:32 +01:00
Anaïs Betts
2a34770959 Don't 🔥 the hello message 2024-11-26 17:06:56 +01:00
Anaïs Betts
ca8db1f417 Handle spawning the client and server on Windows using correct paths 2024-11-26 17:02:39 +01:00
Ashwin Bhat
f684d2e891 improve inspector startup message 2024-11-19 16:58:33 -08:00
Ashwin Bhat
9a560e3f06 handle quotes better 2024-11-19 15:42:53 -08:00
Ashwin Bhat
7c04df5e2e fix arg passing in cli 2024-11-19 14:19:59 -08:00
Ashwin Bhat
83abe4e00f command 2024-11-19 13:08:42 -08:00
Ashwin Bhat
b127ba177a fix arg 2024-11-19 11:50:45 -08:00
Ashwin Bhat
c1a56810fb pass in args 2024-11-19 11:42:16 -08:00
Justin Spahr-Summers
bce3a7b8d6 Make root package installable/executable 2024-11-11 11:10:42 +00:00