Repository navigation
Consolidate SDK service compatibility metadata - #12
Merged
Merged
Conversation
5 tasks done
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
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.
Summary
Use one exact Agent Service revision and one OpenAPI fingerprint for the whole SDK. Remove the four feature-specific revision exports and eight duplicate package fields, together with their repeated runner checks. Update the installed-package checks, frozen-contract readers, documentation and release summary to use the shared values.
Cleanup 3 of 4, based on merged #11. The selected service revision, OpenAPI bytes and supported workflows do not change.
Public behavior verified
Evidence
Before: four feature groups repeated the same service identity and fingerprint. After: the same immutable contract and service evidence is checked against one shared declaration. Existing coverage is retained, and packed-consumer assertions now reject the removed fields.
Merge Danger
Door: two-way. Blast Radius: compatibility metadata.
Callers of the removed Provisioning, Realtime, Lifecycle or Text revision constants must use
TIDB_LINK_REVISION; package readers usetidbLinkRevisionandtidbLinkOpenApiSha256. A public npm lookup returned 404, which does not rule out private or Git consumers. No compatibility aliases, npm publication, release tags or service changes are included.