Skip to content

fix: add root Dockerfile for Glama MCP evaluation#1

Merged
bch1212 merged 1 commit into
mainfrom
fix/glama-root-dockerfile
Jul 6, 2026
Merged

fix: add root Dockerfile for Glama MCP evaluation#1
bch1212 merged 1 commit into
mainfrom
fix/glama-root-dockerfile

Conversation

@bch1212

@bch1212 bch1212 commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a repository-root Dockerfile that launches the published queryshield-mcp==1.0.1 stdio MCP server
  • document that MCP directory evaluators such as Glama can use it for initialize/tool discovery without needing QUERYSHIELD_API_KEY

Why

punkpeye/awesome-mcp-servers#5852 is currently blocked on Glama quality-score evaluation. The repo already had a Dockerfile under packages/queryshield-mcp/, but Glama/awesome-mcp guidance asks for a proper Dockerfile for evaluation. This PR exposes the same slim MCP image at the repository root so directory evaluators can discover it without guessing the nested package path.

Validation

  • uv run --with queryshield-mcp==1.0.1 python - <<'PY' ... imported queryshield_mcp.server and confirmed the FastMCP server name is QueryShield
  • uv run --with pytest --with sqlglot --with sqlalchemy --with fastapi --with 'uvicorn[standard]' --with cryptography --with fastmcp --with httpx --with pydantic --with pydantic-settings --with python-dotenv --with python-multipart --with stripe --with sendgrid --with redis python -m pytest tests/ -q → 66 passed
  • explicit diff review checks passed: root Dockerfile matches packages/queryshield-mcp/Dockerfile, uses queryshield-mcp==1.0.1, entrypoint is queryshield-mcp, README text is scope-limited, and no secret literals were added
  • git diff --check → clean

Note: local Docker daemon was not running on this host, so I could not run a live docker build; the root Dockerfile is an exact copy of the existing nested MCP Dockerfile and installs the released PyPI package.

@bch1212 bch1212 merged commit bb6deb2 into main Jul 6, 2026
1 check passed
@bch1212 bch1212 deleted the fix/glama-root-dockerfile branch July 6, 2026 02:30
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.

2 participants