Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Jan 26, 2025

People have been requesting longer MCP timeouts to allow for asynchronous use cases. This PR adds a per-server configurable network timeout that's stored in the JSON config. Seem reasonable?

Screenshot 2025-01-26 at 4 36 57 PM


Important

Add per-server configurable network timeout for MCP servers, with UI support and validation.

  • Behavior:
    • Adds per-server configurable network timeout in McpHub.ts and ClineProvider.ts.
    • Timeout is stored in JSON config and can be updated via the UI in McpView.tsx.
    • Default timeout is 60 seconds, configurable between 1 and 3600 seconds.
  • Schema:
    • Updates StdioConfigSchema in McpHub.ts to include timeout with validation.
  • UI:
    • Adds timeout selection dropdown in McpView.tsx for each server.
  • Tests:
    • Adds tests for timeout configuration in McpHub.test.ts.
    • Validates timeout values and ensures correct application in tool calls.

This description was created by Ellipsis for 57518e1. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Jan 26, 2025

⚠️ No Changeset found

Latest commit: b0de2295eee64d08080886cd51fb497fdafdd006

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mrubens mrubens merged commit f47dd2d into main Jan 27, 2025
4 checks passed
@mrubens mrubens deleted the mcp_timeouts branch January 27, 2025 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants