diff --git a/client/src/App.tsx b/client/src/App.tsx index e2aab1b..4f99ffd 100644 --- a/client/src/App.tsx +++ b/client/src/App.tsx @@ -105,8 +105,7 @@ const App = () => { Object.entries(mergedConfig).forEach(([key, value]) => { mergedConfig[key as keyof InspectorConfig] = { ...value, - description: - DEFAULT_INSPECTOR_CONFIG[key as keyof InspectorConfig].description, + label: DEFAULT_INSPECTOR_CONFIG[key as keyof InspectorConfig].label, }; }); diff --git a/client/src/components/Sidebar.tsx b/client/src/components/Sidebar.tsx index 2ef54b1..4633d64 100644 --- a/client/src/components/Sidebar.tsx +++ b/client/src/components/Sidebar.tsx @@ -326,7 +326,7 @@ const Sidebar = ({
diff --git a/client/src/components/__tests__/Sidebar.test.tsx b/client/src/components/__tests__/Sidebar.test.tsx index f373a41..8e72b00 100644 --- a/client/src/components/__tests__/Sidebar.test.tsx +++ b/client/src/components/__tests__/Sidebar.test.tsx @@ -343,7 +343,8 @@ describe("Sidebar Environment Variables", () => { expect(setConfig).toHaveBeenCalledWith( expect.objectContaining({ MCP_SERVER_REQUEST_TIMEOUT: { - description: "Request Timeout", + label: "Request Timeout", + description: "Timeout for requests to the MCP server (ms)", value: 5000, }, }), @@ -366,7 +367,9 @@ describe("Sidebar Environment Variables", () => { expect(setConfig).toHaveBeenCalledWith( expect.objectContaining({ MCP_PROXY_FULL_ADDRESS: { - description: "Inspector Proxy Address", + label: "Inspector Proxy Address", + description: + "Set this if you are running the MCP Inspector Proxy on a non-default address. Example: http://10.1.1.22:5577", value: "http://localhost:8080", }, }), @@ -389,7 +392,9 @@ describe("Sidebar Environment Variables", () => { expect(setConfig).toHaveBeenCalledWith( expect.objectContaining({ MCP_REQUEST_MAX_TOTAL_TIMEOUT: { - description: "Maximum Total Timeout", + label: "Maximum Total Timeout", + description: + "Maximum total timeout for requests sent to the MCP server (ms) (Use with progress notifications)", value: 10000, }, }), @@ -410,7 +415,8 @@ describe("Sidebar Environment Variables", () => { expect(setConfig).toHaveBeenCalledWith( expect.objectContaining({ MCP_SERVER_REQUEST_TIMEOUT: { - description: "Request Timeout", + label: "Request Timeout", + description: "Timeout for requests to the MCP server (ms)", value: 0, }, }), @@ -455,7 +461,8 @@ describe("Sidebar Environment Variables", () => { expect(setConfig).toHaveBeenLastCalledWith( expect.objectContaining({ MCP_SERVER_REQUEST_TIMEOUT: { - description: "Request Timeout", + label: "Request Timeout", + description: "Timeout for requests to the MCP server (ms)", value: 3000, }, }), diff --git a/client/src/lib/configurationTypes.ts b/client/src/lib/configurationTypes.ts index b4dd86e..7c74bb7 100644 --- a/client/src/lib/configurationTypes.ts +++ b/client/src/lib/configurationTypes.ts @@ -1,4 +1,5 @@ export type ConfigItem = { + label: string; description: string; value: string | number | boolean; }; diff --git a/client/src/lib/constants.ts b/client/src/lib/constants.ts index 1a772f5..e7fa14c 100644 --- a/client/src/lib/constants.ts +++ b/client/src/lib/constants.ts @@ -22,19 +22,25 @@ export const DEFAULT_MCP_PROXY_LISTEN_PORT = "6277"; **/ export const DEFAULT_INSPECTOR_CONFIG: InspectorConfig = { MCP_SERVER_REQUEST_TIMEOUT: { - description: "Request Timeout", + label: "Request Timeout", + description: "Timeout for requests to the MCP server (ms)", value: 10000, }, MCP_REQUEST_TIMEOUT_RESET_ON_PROGRESS: { - description: "Reset Timeout on Progress", + label: "Reset Timeout on Progress", + description: "Reset timeout on progress notifications", value: true, }, MCP_REQUEST_MAX_TOTAL_TIMEOUT: { - description: "Maximum Total Timeout", + label: "Maximum Total Timeout", + description: + "Maximum total timeout for requests sent to the MCP server (ms) (Use with progress notifications)", value: 60000, }, MCP_PROXY_FULL_ADDRESS: { - description: "Inspector Proxy Address", + label: "Inspector Proxy Address", + description: + "Set this if you are running the MCP Inspector Proxy on a non-default address. Example: http://10.1.1.22:5577", value: "", }, } as const;