Repository navigation
Start the MCP server after a plain uv sync - #304
Merged
Merged
Conversation
Glama builds the server with `uv sync` and starts `uv run proxy-scraper-mcp`. The SDK was only in the mcp extra, so the server exited before listing a tool. It is now a default uv dependency group too, and the docker workflow builds the server the same way. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #303
Glama builds the MCP server itself:
uv sync, thenuv run proxy-scraper-mcp. The MCP SDK was only in themcpextra, which a plainuv syncdoesn't install, so the server printed the install hint and exited, and Glama's build test found no tools. Its page is still on 1.25.0 for that reason.Change
pyproject.toml: the SDK is also a uv dependency group ([dependency-groups] mcp), and[tool.uv] default-groups = ["mcp"]makesuv syncinstall it. The requirement is the same string as the extra. pip and the published package ignore dependency groups, sopip install proxy-scraper-clistill pulls in onlyrichandcertifi.docker.yml: a new step builds the server the way Glama does (uv image pinned by digest,uv sync,uv run proxy-scraper-mcp) and runs the existing stdio smoke test against it.Checked
ghcr.io/astral-sh/uv:python3.12-bookworm-slim:tools: []. After:tools: ['check_proxies', 'fetch_url', 'get_proxies'].pyproject.tomland passes with the change. 1038 tests pass locally, ruff is clean.Not checked: Glama's own build. Its admin page needs a login I don't have in this browser, so the cause is taken from reproducing its documented default build, not from its error log. After the merge, Glama's build has to be started again from its admin page; the hand-edited build step there (
uv sync --extra mcp) is no longer needed.🤖 Generated with Claude Code