Skip to content

http-client-java, update tcgc 0.58.2 - #8069

Merged
Weidong Xu (weidongxu-microsoft) merged 8 commits into
microsoft:mainfrom
weidongxu-microsoft:http-client-java_update-tcgc-0.58.1
Aug 6, 2025
Merged

http-client-java, update tcgc 0.58.2#8069
Weidong Xu (weidongxu-microsoft) merged 8 commits into
microsoft:mainfrom
weidongxu-microsoft:http-client-java_update-tcgc-0.58.1

Conversation

@weidongxu-microsoft

@weidongxu-microsoftWeidong Xu (weidongxu-microsoft) commented Jul 31, 2025

Copy link
Copy Markdown
Contributor

Code change is to adapt TCGC refactor/break on SdkModelProperty

diff on generated code is from #8069 (review)

current code actually works with Legacy.hierarchyBuilding. no code change on this feature.

@microsoft-github-policy-servicemicrosoft-github-policy-serviceBot added the emitter:client:java Issue for the Java client emitter: @typespec/http-client-java label Jul 31, 2025
@github-actions

Copy link
Copy Markdown
Contributor

No changes needing a change description found.

Weidong Xu (weidongxu-microsoft) added a commit to Azure/autorest.java that referenced this pull request Jul 31, 2025
@azure-sdk

Azure SDK Bot (azure-sdk) commented Jul 31, 2025

Copy link
Copy Markdown
Collaborator

You can try these changes here

🛝 Playground🌐 Website🛝 VSCode Extension

@weidongxu-microsoftWeidong Xu (weidongxu-microsoft) left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

The diff in generated code is due to the removal of accept header from TCGC, if no body has content (@error not included).

Unsure whether this would cause diff on playback tests of data-plane libs.

@weidongxu-microsoft

Weidong Xu (weidongxu-microsoft) commented Jul 31, 2025

Copy link
Copy Markdown
ContributorAuthor

fail in smoke test, tcgc bug Azure/typespec-azure#3088

it generates a model with empty name, for https://github.com/microsoft/typespec/blob/main/packages/http-specs/smoke/todoapp/main.tsp#L79-L82, due to visibility

Weidong Xu (weidongxu-microsoft) added a commit to Azure/autorest.java that referenced this pull request Aug 4, 2025
@weidongxu-microsoftWeidong Xu (weidongxu-microsoft) changed the title http-client-java, update tcgc 0.58.1http-client-java, update tcgc 0.58.2Aug 6, 2025
Merged via the queue into microsoft:main with commit 8c5e806Aug 6, 2025
26 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:client:javaIssue for the Java client emitter: @typespec/http-client-java

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@weidongxu-microsoft@azure-sdk@haolingdong-msft@XiaofeiCao