Skip to content

Comments

refactor: rename and version service transport definitions#154

Merged
igrigorik merged 1 commit intomainfrom
rename-and-version-service-transport-schemas
Feb 6, 2026
Merged

refactor: rename and version service transport definitions#154
igrigorik merged 1 commit intomainfrom
rename-and-version-service-transport-schemas

Conversation

@lemonmade
Copy link
Contributor

@lemonmade lemonmade commented Feb 4, 2026

Description

This PR updates the names and post-processing applied to the transport schema files (currently, just the files in https://github.com/Universal-Commerce-Protocol/ucp/tree/main/source/services/shopping).

Before #125, there was additional processing that was used to generate the published spec files. This processing renamed the files, creating a set of x.{openapi,openrpc}.json files that get published (e.g., embedded, MCP). The embedded file underwent particular processing — the source file for it is not an OpenRPC doc at all, but was turned into one as part of this script.

This PR applies the processing that was previously applied when generating the final spec files directly to the schemas themselves. Mostly, this is just renaming the files, but for the embedded schema, more changes are made to generate a valid OpenRPC file (add openrpc, info, and server fields, move delegations to x-delegations as it is not a standard field). I have also added one additional bit of post-processing to all the files, to ensure that their respective info.version fields correctly represent the UCP version to which they apply.

@lemonmade lemonmade requested review from a team, igrigorik and yanheChen February 4, 2026 16:16
@lemonmade lemonmade changed the title Rename and version service transport definitions refactor: rename and version service transport definitions Feb 4, 2026
@lemonmade lemonmade force-pushed the rename-and-version-service-transport-schemas branch from 5257c4f to 523cb0f Compare February 4, 2026 16:20
Copy link
Contributor

@igrigorik igrigorik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great catch 👍

@igrigorik igrigorik added this to the Working Draft milestone Feb 4, 2026
@lemonmade lemonmade force-pushed the rename-and-version-service-transport-schemas branch from 523cb0f to bcaa730 Compare February 5, 2026 14:58
@lemonmade lemonmade force-pushed the rename-and-version-service-transport-schemas branch from bcaa730 to 2fb3a07 Compare February 5, 2026 15:12
@igrigorik igrigorik merged commit d3ee50a into main Feb 6, 2026
14 checks passed
@igrigorik igrigorik deleted the rename-and-version-service-transport-schemas branch February 6, 2026 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants