docs(crewai-tools): fix MCP connection typo#5792
Conversation
📝 WalkthroughWalkthroughA single line in the MCP documentation was updated to correct a duplicate word and clarify the role of the ChangesMCP Context Manager Documentation Clarification
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~1 minute Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
AI assistance disclosure: Kimi CLI was used for the single-line documentation edit, and I reviewed the change before submitting. The repository requires the |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@lib/crewai-tools/README.md`:
- Line 86: The README wording uses the fused term "contextmanager" — update the
prose to the standard two-word phrase "context manager" where it describes the
`with` statement (e.g., change "contextmanager (`with` statement)" to "context
manager (`with` statement)") so the documentation reads correctly; search for
occurrences in lib/crewai-tools/README.md and replace accordingly, preserving
the inline code formatting around `with`.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: af8c00b8-5c04-4316-ab68-3e99158c8e67
📒 Files selected for processing (1)
lib/crewai-tools/README.md
| ### Option 1: Fully managed connection | ||
|
|
||
| In this scenario we use a contextmanager (`with` statement) to start and stop the the connection with the MCP server. | ||
| In this scenario we use a contextmanager (`with` statement) to start and stop the connection with the MCP server. |
There was a problem hiding this comment.
Use “context manager” (two words) in docs prose.
Small wording tweak for correctness/readability: “context manager” is the standard term in narrative text.
Suggested edit
-In this scenario we use a contextmanager (`with` statement) to start and stop the connection with the MCP server.
+In this scenario, we use a context manager (`with` statement) to start and stop the connection with the MCP server.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| In this scenario we use a contextmanager (`with` statement) to start and stop the connection with the MCP server. | |
| In this scenario, we use a context manager (`with` statement) to start and stop the connection with the MCP server. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@lib/crewai-tools/README.md` at line 86, The README wording uses the fused
term "contextmanager" — update the prose to the standard two-word phrase
"context manager" where it describes the `with` statement (e.g., change
"contextmanager (`with` statement)" to "context manager (`with` statement)") so
the documentation reads correctly; search for occurrences in
lib/crewai-tools/README.md and replace accordingly, preserving the inline code
formatting around `with`.
Fixes a duplicated word in the CrewAI tools MCP README section.
Docs-only change; no tests needed.
AI assistance: Kimi CLI was used for the single-line edit; reviewed before submission.
Summary by CodeRabbit