docs(spec): correct self_update visibility description - #473
Open
8thEdition wants to merge 1 commit into
Open
Conversation
[why] The self_update descriptor incorrectly promised that get_monitoring could read the stored value, although it is kept only in memory and is not returned by the receipt or monitoring view. [how] - Corrected the authoritative OpenAPI self_update descriptor. - Regenerated the MCP catalog with the actual stderr, receipt, re-exec, and merge semantics. T-151
8thEdition
added
created-by-skill
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.
Reason
The
self_updatespecification incorrectly promised thatget_monitoringcould read the stored value. This change makes the authoritative contract describe the existing behavior without adding a read surface, removing the field, or changing credential-renewal behavior.Builder level: L1 / R0 — reversible specification-only correction; path gate passed at L1.
Changes
spec/openapi.json): describe in-memory retention, stderr summary, bounded receipt behavior, re-exec loss, merge semantics, and the absence ofget_monitoringexposure.spec/mcp-catalog.json): regenerate the catalog from the corrected authoritative specification.Verification
Outcome: Partial
jqparsing of both JSON documents: passed.git diff --check: passed.drift-mcp-catalog,drift-ocapi, anddrift-schema-ts: passed..pre-commit-config.yaml; the hook was therefore unavailable. No--no-verifyflag was used, and this limitation is recorded rather than hidden.Checks