docs(mcp): document output schemas and read-only annotations in README - #51
Conversation
Cover what shipped in 2.4.0: every tool declares an outputSchema mirroring the library's TypeScript types (with nullable fields for legitimately absent values), and tools carry readOnlyHint except microlink_function. Also note that PRO parameters now say so in their own schema descriptions. Signed-off-by: Kiko Beats <2096101+Kikobeats@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe MCP README now documents tool output schemas, read-only annotations, and the labeling of parameters that require a paid plan. ChangesMCP README updates
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other Merge Risk: ⚪ Minimal · up to This PR only documents existing MCP behavior and introduces no runtime changes or merge-blocking risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Coverage Report for CI Build 34701993503Warning No base build found for commit Coverage: 79.358%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
|
@coderabbitai review |
✅ Action performedReview finished.
|
What
The README's "Response shape" section predates the 2.4.0 protocol work. This adds:
outputSchemaforstructuredContent.data, mirroring the library's TypeScript types (Asset,Metadata,Embed,FunctionResult, ...); error results are exempt from output validation; legitimately absent fields are nullable (logowith no detected brand logo,markdownwith no selector match, ...).readOnlyHint: true(they only fetch/transform public URLs), exceptmicrolink_function, which executes user-supplied code.Docs only, no code changes.
Note
Low Risk
Documentation-only README edits with no runtime or API behavior changes.
Overview
Updates the Response shape section in
packages/mcp/README.mdso it matches the 2.4.0 MCP protocol behavior.It now documents that each tool exposes an MCP
outputSchemaforstructuredContent.data(aligned with library types likeAsset,Metadata,Embed,FunctionResult), with errors exempt from output validation and optional fields marked nullable. It also notesreadOnlyHint: trueon fetch/transform tools, withmicrolink_functionexcluded because it runs user-supplied code.PRO paid-plan parameters are described as labeled in per-parameter schema descriptions (not only in external Microlink docs). The failure-response bullet is unchanged in substance; it sits after the new bullets in the list.
Reviewed by Cursor Bugbot for commit ea3f82d. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by CodeRabbit
microlink_function.PROin their schema descriptions.