fix(catalog): one counting rule across surfaces; correct screencast tool count#82
Merged
Conversation
…ool count [skip version] The catalog hero summed all tools (14 repos / 404 MCP tools) while README, CLAUDE.md, and the GitHub About line counted only active+maintenance repos (11 / 378), so the same page showed two different totals (D3). Separately the registry recorded screencast-mcp at 10 MCP tools while its server registers, and its manifest declares, 25 (D4). - aggregate_stats and the hero renderStats now both count every non-archived tool, so the hero, README, CLAUDE.md, and About all derive from one rule: 14 repos / 189 skills / 77 rules / 419 MCP tools. - Cards now carry a status badge (experimental / beta / maintenance / deprecated) so non-active tools are surfaced, not hidden. - registry.json screencast-mcp mcpTools 10 -> 25, verified against the 25 server.tool() registrations in C:\Dev\screencast-mcp/src/tools (its mcp-tools.json already declared 25 and matches the server, so the manifest needed no change). - New test compares registry mcpTools to each locally-checked-out repo's manifest length; it skips when no sibling checkouts exist, as full-fleet validation runs CI-side where the repos can be cloned. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: TMHSDigital <TMHospitalityStrategies@gmail.com>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
github-actions
Bot
added
documentation
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.
Summary
Reconciles the catalog count divergence and the screencast tool-count error from
site-audit.md: D3 and D4. No VERSION bump ([skip version]).Findings addressed
aggregate_statsand the herorenderStatsnow both count every non-archived tool, so all four surfaces agree: 14 repos / 189 skills / 77 rules / 419 MCP tools. Cards now show a status badge (experimental/beta/maintenance/deprecated) so nothing is hidden.registry.jsonrecorded screencast-mcp at 10 MCP tools; its server registers 25 (src/tools/*.ts, oneserver.tool()each) andmcp-tools.jsonalready declares 25. Corrected the registry to 25. The manifest matched the server, so no change was needed in the tool repo (no cross-repo PR required).New CI check
tests/test_registry_manifest_counts.pycompares the registrymcpToolsvalue to each locally-checked-out tool repo'smcp-tools.jsonlength. It runs in the existing pytest job, skips when no sibling checkouts exist, and notes that full-fleet validation runs CI-side where repos can be cloned.Verification
python scripts/sync_from_registry.py --checkpasses.betabadge (matching its tool-site); cfx-mcp and local-ai showexperimental; active tools show no status badge.🤖 Generated with Claude Code