Skip to content

fix: correct tool count badge in README (73 → actual count)#225

Merged
himerus merged 1 commit intodevfrom
feature/fix-correct-tool-count-badge-in-readme
Mar 27, 2026
Merged

fix: correct tool count badge in README (73 → actual count)#225
himerus merged 1 commit intodevfrom
feature/fix-correct-tool-count-badge-in-readme

Conversation

@himerus
Copy link
Copy Markdown
Contributor

@himerus himerus commented Mar 27, 2026

Summary

Fix Tool Count Badge Discrepancy

The README badge currently shows '73 tools' but the actual implementation has a different count. The badge and tool catalog table in README need to match the real number of registered MCP tools.

Implementation

  1. Count actual tools registered in src/mcp/index.ts (the server.tool() calls)
  2. Update the badge in README.md to show the correct count
  3. Verify the tool catalog tables in README list every tool exactly once
  4. Cross-reference with integrati...

Recovered automatically by Automaker post-agent hook

Summary by CodeRabbit

  • Tests
    • Updated integration test expectations for the tools endpoint to reflect an additional tool now available when token path configuration is enabled. The expected tool count increased from 72 total tools (70 core + 2 token-based) to 73 total tools (71 core + 2 token-based).

@himerus himerus merged commit 3214e7c into dev Mar 27, 2026
4 of 7 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 27, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 57739d84-bc67-4736-a952-35aed6779470

📥 Commits

Reviewing files that changed from the base of the PR and between d1369e4 and a76cb30.

📒 Files selected for processing (1)
  • tests/integration/server.test.ts

Walkthrough

A single integration test was updated to reflect a change in expected tool count. The tools/list endpoint expectation when MCP_WC_TOKENS_PATH is configured now expects 71 core tools instead of 70, with one additional tool name added to the assertions.

Changes

Cohort / File(s) Summary
Test Update
tests/integration/server.test.ts
Updated expected tool count from 70 to 71 core tools and added one additional tool name to the expected tools list for the tools/list endpoint when MCP_WC_TOKENS_PATH is configured.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

skip-changeset

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/fix-correct-tool-count-badge-in-readme

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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