[T-225] ingest_telemetry：self_update 說明不再教成員去讀回（取代 #473） - #507
Draft
8thEdition wants to merge 2 commits into
Draft
8thEdition wants to merge 2 commits into
8thEdition wants to merge 2 commits into
Conversation
… back [why] The self_update field description ended with "Call get_monitoring when you need to read the entry back", but the server only writes entry["self_update"] into the in-memory telemetry store and logs one stderr line; no read path (get_monitoring included) returns it, and it is lost on re-exec. Redo of PR #473 on current main, which conflicted. [how] Rewrite the field description in the x-mcp legacy descriptor in spec/openapi.json and regenerate spec/mcp-catalog.json. Compared with #473: say no read tool at all returns it (not only get_monitoring), keep the misspelled-sub-key consequence, and list the reporter id the stderr line prints. gen-ocapi and gen:api produce no change. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
[why] Independent review: the server's stderr line prints ? for a missing sub-key, so the server side can see it; only the caller cannot. [how] Narrow the clause to the caller in the openapi legacy descriptor and regenerate spec/mcp-catalog.json. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2 tasks
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.
Why
The
ingest_telemetrydescription told callers to "call get_monitoring when you need to read the entry back" forself_update. That entry lives only in the server's in-memory telemetry store: no read tool returns it, the receipt does not echo it, and it is gone on re-exec. An agent following the sentence silently concludes there is no update record.This supersedes #473, which made the same correction (hand-written source, then regenerated) but no longer applies cleanly to main.
How
self_updatefield description in thespec/openapi.jsonlegacy descriptor; regeneratedspec/mcp-catalog.json.Verification
drift-*targets rc=0;bin/tests/mcp-catalog-generator.shgreen.🤖 Generated with Claude Code