docs: fix UTCP/MCP protocol clarity and markdown syntax #3
Conversation
There was a problem hiding this comment.
Pull request overview
This PR improves documentation clarity around the UTCP/MCP protocol relationship and fixes markdown syntax issues across README.md and PROJECT_OVERVIEW.md.
Changes:
- Clarified that rs-utcp is a UTCP (Universal Tool Calling Protocol) implementation and its relationship to MCP
- Fixed markdown URL formatting by wrapping URLs in angle brackets for proper rendering
- Improved markdown table formatting and alignment for better readability
- Updated project title casing from "Thulp" to "thulp"
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| README.md | Updated title casing, clarified UTCP/MCP relationship, fixed URL formatting, improved table alignment, reformatted code blocks, and updated acknowledgments |
| docs/PROJECT_OVERVIEW.md | Fixed URL formatting by wrapping URLs in angle brackets |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - **Type-Safe Parameters**: Strongly-typed parameter validation with JSON Schema support | ||
| - **Unified Tool Abstraction**: Consistent interface for defining, validating, | ||
| and executing tools | ||
| - **MCP Integration**: Full Model Context Protocol support via `rs-utcp` UTCP |
There was a problem hiding this comment.
Redundant wording: using both "rs-utcp" (the Rust implementation) and "UTCP" (the protocol) creates awkward phrasing. Consider simplifying to either "via the rs-utcp implementation" or "via UTCP (Universal Tool Calling Protocol)" to avoid the repetition.
| - **MCP Integration**: Full Model Context Protocol support via `rs-utcp` UTCP | |
| - **MCP Integration**: Full Model Context Protocol support via the `rs-utcp` |
| - **rs-utcp**: MCP protocol implementation | ||
| - **rs-utcp**: UTCP protocol implementation (includes MCP transport) | ||
| - **Anthropic**: Model Context Protocol specification | ||
| - **UTCP**: Universal Tool Calling Protocol |
There was a problem hiding this comment.
The acknowledgment entry for "UTCP: Universal Tool Calling Protocol" is unusual. Acknowledgments typically credit people, organizations, or specific projects/implementations, not protocols themselves. Consider removing this line since UTCP is already acknowledged through the rs-utcp entry above.
| - **UTCP**: Universal Tool Calling Protocol |
| | ------------------ | ------------------------------------------------- | | ||
| | **thulp-query** | Query DSL for searching and filtering tools | | ||
| | **thulp-skills** | Multi-step workflow composition and execution | | ||
| | **thulp-workspace**| Workspace and execution context management | |
There was a problem hiding this comment.
Missing space after the pipe character in the table. The formatting should be "| thulp-workspace |" with a space before the closing pipe to match the formatting of other rows in the table.
| | **thulp-workspace**| Workspace and execution context management | | |
| | **thulp-workspace** | Workspace and execution context management | |
bkataru
added
documentation
|
@copilot open a new pull request to apply changes based on the comments in this thread |
Summary