Skip to content

fix: clarify MCP tool metadata for Glama#23

Merged
simongonzalezdc merged 1 commit into
mainfrom
codex/glama-mcp-metadata-checkyourself
Jun 7, 2026
Merged

fix: clarify MCP tool metadata for Glama#23
simongonzalezdc merged 1 commit into
mainfrom
codex/glama-mcp-metadata-checkyourself

Conversation

@simongonzalezdc

@simongonzalezdc simongonzalezdc commented Jun 7, 2026

Copy link
Copy Markdown
Member

BLUF

Tightens the CheckYourself MCP tool metadata that Glama scores: explicit auth/network/rate-limit/side-effect language on every tool, and accurate findings input schemas for score/backlog/next.

What changed

  • Added a shared MCP description safety footer: no auth, local-only inputs, no network calls, no file modification, no external rate limits.
  • Changed findings input schema for score/backlog/next from object-only to object-or-array, matching the existing normalizer behavior.
  • Added a regression test that exercises tools/list over stdio.

Verification

  • python3 -m unittest discover -s tests -k test_mcp_stdio_exposes_thin_tools
  • python3 -m py_compile tools/validate_public.py tools/checkyourself.py tests/test_checkyourself_cli.py
  • python3 -m unittest discover -s tests
  • python3 tools/validate_public.py
  • python3 tools/checkyourself.py describe --format json

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

@simongonzalezdc simongonzalezdc merged commit 98fecc2 into main Jun 7, 2026
2 checks passed
@simongonzalezdc simongonzalezdc deleted the codex/glama-mcp-metadata-checkyourself branch June 7, 2026 02:07
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.

1 participant