docs(mcp): align checkout/order bindings with cart-mcp pattern#444
Open
sakinaroufid wants to merge 4 commits into
Open
docs(mcp): align checkout/order bindings with cart-mcp pattern#444sakinaroufid wants to merge 4 commits into
sakinaroufid wants to merge 4 commits into
Conversation
Three render-consistency fixes for the MCP transport docs. - checkout-mcp.md: switch per-operation Input/Output Schema sections to schema_fields() macro. Cart-mcp.md already does this; the inline bullet lists in checkout-mcp.md had drifted from the schemas. - checkout-mcp.md, order-mcp.md: drop redundant `meta` from per-op Input Schemas. It's documented once per file in "Request Metadata" under "Protocol Fundamentals". - order-mcp.md: add the protocol-errors-vs-business-outcomes bullets to Error Handling. Cart-mcp.md and checkout-mcp.md got these in Universal-Commerce-Protocol#128 but order-mcp.md was added later (Universal-Commerce-Protocol#276) and missed the pattern. Points at the existing Not Found and Not Authorized tabs under get_order rather than inlining another copy of the same envelope.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Three render-consistency fixes for the MCP transport docs. Brings
checkout-mcp.mdandorder-mcp.mdin line with patternscart-mcp.mdhas been using since #128.checkout-mcp.md: replace inline schema bullets with{{ schema_fields(...) }}in per-operation Input/Output Schema sections. The hand-maintained field lists had drifted from the schemas.checkout-mcp.mdandorder-mcp.md: drop themetaline from per-operation Input Schemas. It's documented once per file in "Request Metadata" under "Protocol Fundamentals".order-mcp.md: add the protocol-errors-vs-business-outcomes bullets to Error Handling. This prose was added tocart-mcp.mdandcheckout-mcp.mdin feat(errors): spec error handling for UCP negotiation failures #128;order-mcp.mdlanded later (feat(order): Get Order operation w/ platform-auth #276) and missed the pattern. Points at the existing Not Found and Not Authorized tabs underget_orderrather than inlining another copy of the same envelope.Category (Required)
Related Issues
None.
Checklist
!for breaking changes).Screenshots / Logs (if applicable)
Will add before/after screenshots of the rendered
order-mcpError Handling section and one of thecheckout-mcpInput/Output Schema sections once I have the local mkdocs preview rendered.