Skip to content

Conversation

reegnz
Copy link
Contributor

@reegnz reegnz commented Aug 22, 2025

The change allows the LM Studio mcp config found under ~/.lmstudio/mcp.json
to be discovered and updated with the toolhive-managed mcp servers.

Fixes #1529

The change allows the LM Studio mcp config found under `~/.lmstudio/mcp.json`
to be discovered and updated with the toolhive-managed mcp servers.
@eleftherias eleftherias self-assigned this Aug 25, 2025
@eleftherias
Copy link
Member

Thanks @reegnz!
Just a couple of comments before we merge this.

  1. Could you run task docs to ensure the documentation gets updated with the new client
  2. We have 2 tests that check the format for each client, the first one is here https://github.com/stacklok/toolhive/blob/main/pkg/client/config_test.go#L275 and the second one is here https://github.com/stacklok/toolhive/blob/main/pkg/client/config_test.go#L329

@eleftherias
Copy link
Member

Thanks @reegnz! I included a small commit to fix the linting.

@eleftherias eleftherias merged commit b00ff1f into stacklok:main Aug 28, 2025
13 checks passed
@reegnz
Copy link
Contributor Author

reegnz commented Aug 28, 2025

@eleftherias thanks! Sorry but I couldn't get around to adding the tests this week.

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.

LM Studio Support managing mcp.json
4 participants