Skip to content

Fix duplicate section in README (getTools) #24

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
peteski22 merged 1 commit into from
Jan 15, 2026
Merged

Fix duplicate section in README (getTools) #24

peteski22 merged 1 commit into from
Jan 15, 2026

Conversation

@peteski22
Copy link
Contributor

@peteski22 peteski22 commented Jan 14, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Documentation
    • Updated usage examples to demonstrate iterating over retrieved tools and dynamically accessing server-specific tool collections with counts.

✏️ Tip: You can customize this high-level summary in your review settings.

@peteski22 peteski22 added the documentation Improvements or additions to documentation label Jan 14, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 14, 2026 •
edited
Loading

Walkthrough

The README.md documentation has been updated to replace a static example of client server tool retrieval with new usage snippets demonstrating iterative and dynamic tool discovery patterns across multiple servers.

Changes

Cohort / File(s) Summary
Documentation examples
README.md		 Replaced static tool listing example with dynamic usage patterns: (1) iterating over tools from time.getTools() to log individual tool metadata, and (2) dynamically iterating servers to retrieve and log per-server tool counts via client.servers[serverName].getTools()
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing a duplicate getTools example section from the README documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

📜 Recent review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cd0190b and 8a9e9b9.

📒 Files selected for processing (1)
  • README.md
🔇 Additional comments (1)
README.md (1)

236-248: LGTM! Excellent documentation improvements.

The new examples effectively demonstrate both basic iteration over tools and dynamic server usage patterns. The progression from static to dynamic usage is clear and educational, and the syntax is correct throughout.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.

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.

@peteski22 peteski22 merged commit 1323c65 into main Jan 15, 2026
3 checks passed
@peteski22 peteski22 deleted the peteski22/fix-readme-duplicate branch January 15, 2026 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dpoulopoulos dpoulopoulos dpoulopoulos approved these changes

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@peteski22 @dpoulopoulos