Skip to content

[Automated][tcgc][skip chg] document wireType and @clientDefaultValue type validation - #18

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
doc-update/tcgc-incremental-20260824-700e124f1186c1f7
Draft

github-actions[bot] wants to merge 1 commit into
mainfrom
doc-update/tcgc-incremental-20260824-700e124f1186c1f7

Conversation

@github-actions

Copy link
Copy Markdown

Summary

Incremental TCGC documentation update covering source changes since the last run (checkout 9d5bfaac).

Doc-relevant changes analyzed

  • SdkBuiltInType.wireType (interfaces/types): new property holding the built-in type a value is serialized as on the wire when @encode specifies a target encode-as type; string/url kinds can now be encoded as other built-in types.
  • client-default-value-type-mismatch diagnostic: @clientDefaultValue now validates that the value type matches the target property/parameter type (or the @alternateType when present).
  • New C# linter rules (csharp-model-suffix, csharp-use-standard-acronyms) and the reworded @operationGroup deprecation note — the auto-generated reference docs (reference/linter.md, reference/decorators.md) were already regenerated in the source PRs, so no further action was required.
  • Type-only import refactors and test-only commits — no documentation impact.

Changes

  • guideline.md: documented the new wireType property on the SdkBuiltInType bullet (emitter developer docs).
  • 08types.mdx: added a "Value Type Validation" subsection under @clientDefaultValue describing the new diagnostic and how @alternateType affects validation.
  • knowledge base: recorded lessons for wireType, the new diagnostic, the C# linter rules, and the @operationGroup doc-comment change.
  • metadata: updated tcgc.meta.json to the checkout commit.

No versioned package content changed, so no changelog entry was added.

Generated by Documentation Update Agent · 162.2 AIC · ⌖ 14.3 AIC · ⊞ 8.3K · ◷

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants