[Automated][tcgc] Update emitter guideline for versionsEnum, SSE metadata, and boolean encode - #10
Draft
github-actions[bot] wants to merge 1 commit into
Draft
Conversation
…boolean encode Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
Incremental documentation update for TCGC based on 8 source commits since the last checkpoint. Most commits were pure formatting/eslint-comment changes; this PR covers the three substantive, user-observable additions plus the removal of a linter rule, and records lessons in the knowledge base.
Changes
Emitter developer docs (
guideline.md):SdkClientType.versionsEnum— documented the new field exposing the service's API-versionsSdkEnumTypeon each client (undefined for unversioned/multi-service clients).SdkSseMetadata/SdkSseEventMetadatacarried on HTTP body parameters and responses fortext/event-stream(@events) streams, alongsidestreamMetadata.@encode(string)— updated the built-in types note;@encode(string)now applies tobooleanin addition to integer kinds, settingencode: "string".Knowledge base (
knowledge/tcgc.md):no-unnamed-typeslinter rule was removed (and unregistered) and that the remaining rule.mdbodies were expanded (Impact / Diagnostic Message / How to Fix / Suppression). Regeneratingreference/rules/*.mdrequires a fullpnpm regen-docsbuild and was out of scope for this constrained run.Metadata (
knowledge/tcgc.meta.json):update-meta.tsso the next incremental run starts from the correct point.Notes
No Spector specs were added — these are emitter-consumed type-graph metadata and type
encodebehavior covered by existing unit tests, not new wire-level scenarios. No<ClientTabs>blocks were added or changed.