Skip to content

chore: remove Glama integration#88

Merged
cmeans-claude-dev[bot] merged 2 commits into
mainfrom
chore/remove-glama
May 2, 2026
Merged

chore: remove Glama integration#88
cmeans-claude-dev[bot] merged 2 commits into
mainfrom
chore/remove-glama

Conversation

@cmeans-claude-dev
Copy link
Copy Markdown
Contributor

@cmeans-claude-dev cmeans-claude-dev Bot commented May 2, 2026

Summary

Drops the Glama score badge from README.md and deletes the glama.json config file at the repo root.

Why

Glama's Maintenance sub-score is driven primarily by issues responded to in the last N months, which conflicts with this project's use of GitHub issues as a planned roadmap. The baggage tracker (#51) plus per-item issues (#33–47, #75/76, etc.) are TODO entries — work we know we want to do — not unanswered support requests. Glama can't tell the difference, so the score reads as "0 of 33 issues responded to" → C grade on Maintenance, even though every one of those issues is intentionally tracked and being worked through (we just merged six of them in v0.5.2 alone).

The badge therefore mis-represents project health to anyone landing on the README, and the heuristic isn't tunable from our side. Removing rather than gaming the metric (e.g. bulk-commenting "tracked elsewhere" on every open issue) or adding a workflow note that wouldn't fix the perception for casual readers.

What's left

PyPI, GitHub release, and the MCP registry remain the canonical distribution and discovery surfaces. The README still carries the standard PyPI / Python-versions / License / Tests / Coverage / Downloads badges plus the per-installer and per-OS download breakdowns from cmeans/pypi-winnow-downloads.

Files changed

  • README.md — drops the single [![Glama](...)] line
  • glama.json — deleted (was just { "$schema": "...", "maintainers": ["cmeans"] })
  • CHANGELOG.md### Changed entry under ## Unreleased explaining the rationale

Test plan

  • CI green
  • grep -in glama README.md returns nothing
  • glama.json no longer exists at repo root

🤖 Generated with Claude Code

Drops the Glama score badge from README.md and deletes the
glama.json config file at the repo root.

Glama's "Maintenance" sub-score is driven primarily by
`issues responded to in the last N months`, which conflicts
with this project's use of GitHub issues as a planned
roadmap — the baggage tracker (#51) plus per-item issues
are TODO entries, not unanswered support requests. The
badge therefore mis-represents project health to anyone
landing on the README, and the heuristic isn't tunable
from our side.

Removing rather than gaming the metric or adding a workflow
note. PyPI / GitHub release / MCP registry remain the
canonical distribution and discovery surfaces.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added the Awaiting CI Dev complete, waiting for CI/Codecov to pass before QA label May 2, 2026
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added Ready for QA Dev work complete — QA can begin review and removed Awaiting CI Dev complete, waiting for CI/Codecov to pass before QA labels May 2, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Owner

@cmeans cmeans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cmeans cmeans added QA Active QA is actively reviewing; Dev should not push changes and removed Ready for QA Dev work complete — QA can begin review labels May 2, 2026
Copy link
Copy Markdown
Owner

@cmeans cmeans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA Round 1 — PASS (no findings)

Tight chore PR. Diff is exactly what the description says: 3 files / +4 / -5.

Verified

  • README.md: drops the single [![Glama](...)] badge line; all other badges (PyPI, Python versions, License, Tests, Coverage, Downloads, per-installer, per-OS) preserved.
  • glama.json: deleted. ls glama.json → "No such file or directory". The file was minimal anyway ($schema + maintainers: ["cmeans"]); nothing else referenced it.
  • CHANGELOG.md: new ### Changed entry under ## Unreleased, correctly slotted ABOVE ## 0.5.2 (2026-05-01). Standard Keep a Changelog category. First PR after the v0.5.2 release introduces the section per project convention. Rationale text matches the PR body — the heuristic conflict between issue-as-roadmap and Glama's "issues responded to" metric is honestly stated.
  • Repo-wide grep -in glama turns up two residual hits, both intentional / accurate:
    • .claude/settings.local.json:75 — gitignored (git check-ignore confirms), local-only Claude settings; not part of the repo.
    • scripts/sync-server-json.py:5 — docstring noting server.json is consumed by "the MCP Registry and Glama.ai". Glama still scans server.json from the repo whether or not we maintain a glama.json or display a badge, so this comment is still factually correct. Not flagging.

Test plan checkboxes (all 3 pre-validated locally)

  • CI green — 12/12 required incl. vdsm SUCCESS on c342a9e.
  • grep -in glama README.md returns nothing (exit 1, no match).
  • glama.json no longer exists at repo root.

Local stack

  • uv run pytest → 581 passed, 112 deselected, 96.19% coverage. (Same baseline as post-#85; this PR doesn't touch any code paths.)
  • uv run ruff check, ruff format --check, mypy src/ all clean.

Disposition

Ready for QA Signoff applied as the final act. Awaiting maintainer's QA Approved.

If you also delist on Glama's admin side (separate from this PR), the scripts/sync-server-json.py:5 docstring becomes the only remaining repo reference and would warrant a one-line drop of "and Glama.ai" — but that's a follow-up, not a blocker for this PR.

@cmeans
Copy link
Copy Markdown
Owner

cmeans commented May 2, 2026

Applying Ready for QA Signoff — clean chore PR, no findings. Diff exactly matches the description: README badge line removed, glama.json deleted, CHANGELOG ## Unreleased ### Changed entry added with the rationale we discussed earlier. Local 581/96.19%, ruff/format/mypy clean, CI 12/12 green. Test-plan checkboxes 1-3 flipped (all pre-validated). Heads-up: if you also delist on Glama's admin side, scripts/sync-server-json.py:5 docstring should drop "and Glama.ai" as a follow-up — not blocking this PR.

@cmeans cmeans added Ready for QA Signoff QA passed — ready for maintainer final review and merge QA Approved Manual QA testing completed and passed and removed QA Active QA is actively reviewing; Dev should not push changes Ready for QA Signoff QA passed — ready for maintainer final review and merge labels May 2, 2026
@cmeans-claude-dev cmeans-claude-dev Bot merged commit 3e9d96a into main May 2, 2026
38 checks passed
@cmeans-claude-dev cmeans-claude-dev Bot deleted the chore/remove-glama branch May 2, 2026 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

QA Approved Manual QA testing completed and passed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants