Skip to content

Normalization in-consistancy for discriminated union during Serializer/DeSerializer#3652

Merged
v-jiaodi merged 5 commits intoAzure:mainfrom
v-jiaodi:fix-issue3651
Dec 24, 2025
Merged

Normalization in-consistancy for discriminated union during Serializer/DeSerializer#3652
v-jiaodi merged 5 commits intoAzure:mainfrom
v-jiaodi:fix-issue3651

Conversation

@v-jiaodi
Copy link
Copy Markdown
Member

fixes #3651

@MaryGao MaryGao changed the title Discriminated Union should not be normalized for Serializer/DeSerializer Normalization in-consistancy for discriminated union during Serializer/DeSerializer Dec 23, 2025
Comment thread packages/typespec-ts/test/modularUnit/scenarios/models/unionArrayTypeReference.md Outdated
propEncoded: string;
propNotNormalizeModel: FOO;
propNormalizeModel: FOOBAR;
propRecordOfUnionArrayNotNormalize: Record<NFVIs[]>;
Copy link
Copy Markdown
Member

@MaryGao MaryGao Dec 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

propRecordOfUnionArrayNotNormalize: NFVIs[];
propRecordOfUnionArrayNotNormalize: Record<NFVIs>;

@MaryGao MaryGao marked this pull request as ready for review December 24, 2025 02:45
Copy link
Copy Markdown
Member

@MaryGao MaryGao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pls check there is no build issue.

@v-jiaodi
Copy link
Copy Markdown
Member Author

check smoke case and no build issue.
image

@v-jiaodi v-jiaodi merged commit 0c80e14 into Azure:main Dec 24, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[tsp migration] Discriminated Union should not be normalized for Serializer/DeSerializer

2 participants