Skip to content

Conversation

caarlos0
Copy link
Member

@caarlos0 caarlos0 commented Aug 8, 2025

If a MCP gets stuck starting/connecting, chatting will not work.

This makes it so it'll timeout after 5s (we can discuss the time), and then everything works accordingly.

If a MCP gets stuck starting/connecting, chatting will not work.

This makes it so it'll timeout after 5s (we can discuss the time), and
then everything works accordingly.

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 self-assigned this Aug 8, 2025
@caarlos0 caarlos0 requested a review from kujtimiihoxha as a code owner August 8, 2025 18:08
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Co-authored-by: Andrey Nering <andreynering@users.noreply.github.com>
@caarlos0 caarlos0 merged commit 42a346c into main Aug 8, 2025
19 checks passed
@caarlos0 caarlos0 deleted the mcp-timeout branch August 8, 2025 19:18
scalabl3 pushed a commit to metaphori-ai/crush that referenced this pull request Aug 19, 2025
* fix(mcp): give a time limit for startup

If a MCP gets stuck starting/connecting, chatting will not work.

This makes it so it'll timeout after 5s (we can discuss the time), and
then everything works accordingly.

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix: spinner

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* Update mcp-tools.go

Co-authored-by: Andrey Nering <andreynering@users.noreply.github.com>

---------

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Co-authored-by: Andrey Nering <andreynering@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants