Commit Graph

540 Commits

Author SHA1 Message Date
Pulkit Sharma
834eb0c934 add Sidebar tests 2025-03-29 22:31:57 +05:30
cliffhall
054741be03 Run prettier. 2025-03-29 12:36:03 -04:00
Cliff Hall
eaa8055fd1 Merge branch 'main' into fix-214 2025-03-29 12:34:20 -04:00
Pulkit Sharma
9e1186f5ac Document configuration in Readme and add TSDoc for the same. 2025-03-29 09:40:13 +05:30
Pulkit Sharma
81c0ef29ab Merge branch 'main' into main 2025-03-29 09:11:01 +05:30
Cliff Hall
dbaa731097 Merge branch 'main' into fix-rules-of-hooks-warning 2025-03-28 17:49:40 -04:00
Cliff Hall
ed31f9893f Merge branch 'main' into perf_useTheme 2025-03-28 17:46:00 -04:00
Cliff Hall
240c67037c Merge branch 'main' into feat/json-view-component 2025-03-28 17:32:41 -04:00
Cliff Hall
02155570df Merge branch 'main' into cleanup-old-transports 2025-03-28 16:34:21 -04:00
cliffhall
7227909df3 Fix spurious linebreak 2025-03-28 16:03:39 -04:00
cliffhall
84335ae5f4 Remove else block wrapper for final component return to reduce indenting and hopefully make the PR diff easier to approve. 2025-03-28 15:57:11 -04:00
Pulkit Sharma
8486696240 Merge branch 'main' of github.com:pulkitsharma07/inspector 2025-03-29 01:23:29 +05:30
Pulkit Sharma
6a6b15ab45 don't disable rules-of-hooks 2025-03-29 01:22:35 +05:30
Cliff Hall
804a144ffb Merge branch 'main' into fix-rules-of-hooks-warning 2025-03-28 15:20:11 -04:00
Cliff Hall
1d4ca435b8 Merge branch 'main' into fix-214 2025-03-28 12:32:59 -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
Ola Hungerford
e47b1d8f4d Merge pull request #200 from morinokami/patch-1
Restructure link buttons in sidebar to respect theme
2025-03-28 06:50:47 -07:00
Shinya Fujino
d2e211a597 Update Sidebar component icons to use 'text-foreground' for better visibility 2025-03-28 22:13:31 +09:00
Shinya Fujino
827d867aae Merge branch 'main' into patch-1 2025-03-28 21:01:22 +09:00
Simon Knott
da0c855ef5 adapt title 2025-03-28 09:45:17 +01:00
cgoing
8180b0bd6f refactor: JsonEditor 2025-03-28 15:16:17 +09:00
yusheng chen
f09d2b6096 style: prettier format file client/src/lib/useTheme.ts 2025-03-28 07:05:22 +08:00
Cliff Hall
f846c154f5 Merge branch 'main' into perf_useTheme 2025-03-27 17:10:02 -04:00
Cliff Hall
9244ecf859 Merge branch 'main' into show-all-notifications 2025-03-27 16:42:44 -04:00
cliffhall
0891077402 This fixes the linter's warning that Hooks must be run in order each time.
* In App.tsx
  - move the conditional that returns suspense if the path is oauth/callback to the end of the component after all hooks, rendering either suspense or the normal component.
  - move handleApproveSampling and handleRejectSampling functions down below all the hooks for clarity. There are a lot of hooks so finding the end of them is a scroll, and these function constants aren't referenced until the rendering section below anyway.
2025-03-27 15:59:41 -04:00
Simon Knott
353d6b549b fix: clean up previous transport processes 2025-03-27 09:52:35 +01:00
cgoing
18ca6e28a7 Use <pre> tag inside JsonView component for consistency 2025-03-27 12:55:36 +09:00
cgoing
2d252a389c Remove escapeUnicode function from ToolsTab.tsx 2025-03-27 11:42:39 +09:00
cgoing
e6f5da8383 Improve JsonView component styling and change to use JsonView in PromptsTab 2025-03-27 10:49:37 +09:00
choi sung keun
196fd67ce9 Merge branch 'main' into feat/json-view-component 2025-03-27 09:40:29 +09:00
Cliff Hall
b97233f43a Merge branch 'main' into update-prism 2025-03-26 15:29:36 -04:00
Cliff Hall
00118f7cf9 Merge branch 'main' into show-all-notifications 2025-03-25 15:31:46 -04:00
choi sung keun
fcc06ab556 Merge branch 'main' into feat/json-view-component 2025-03-26 00:11:57 +09:00
cgoing
03c1ba3092 Change JsonView default initialExpandDepth from 2 to 3 2025-03-26 00:11:07 +09:00
cgoing
2588f3aeb3 Change tooltip title from Korean to English 2025-03-26 00:02:10 +09:00
Ola Hungerford
88ffb5087e Merge pull request #210 from olaservo/add-ui-tests
Add preliminary UI tests
2025-03-25 07:03:30 -07:00
choi sung keun
1f17132ca1 Merge branch 'main' into feat/json-view-component 2025-03-25 22:14:29 +09:00
Pulkit Sharma
0e667acf7d Merge branch 'main' into main 2025-03-25 12:31:01 +05:30
choi sung keun
8e9e5facaf Merge branch 'main' into feat/json-view-component 2025-03-25 09:45:01 +09:00
Ola Hungerford
16b38071e7 Bump version to 0.7.0 2025-03-24 12:36:17 -07:00
Ola Hungerford
25f5bb7620 Merge branch 'main' into add-ui-tests 2025-03-24 12:01:41 -07:00
Pulkit Sharma
6efdcb626f Merge branch 'main' of github.com:pulkitsharma07/inspector 2025-03-24 23:30:57 +05:30
Pulkit Sharma
0656e15a22 add configuration types 2025-03-24 23:30:35 +05:30
Mark Anthony Cianfrani
30e7a4d7b7 Merge branch 'main' into main 2025-03-24 12:59:13 -04:00
cgoing
d204dd6e7e feat: json view component - dark color 2025-03-25 01:56:53 +09:00
Mark Anthony Cianfrani
f2f209dbd3 fix(sidebar): maintain order when changing values 2025-03-24 12:56:41 -04:00
cgoing
f0b28d4760 feat: json view component 2025-03-25 01:48:29 +09:00
Ola Hungerford
65a0d46816 Fix formatting 2025-03-24 09:47:34 -07:00
Ola Hungerford
951db44bad Merge branch 'add-ui-tests' of https://github.com/olaservo/inspector into add-ui-tests 2025-03-24 09:43:52 -07:00
Ola Hungerford
379486b5ea Add failing test for pull/206 2025-03-24 09:43:48 -07:00