Skip to content

Upgrade repo commits to account for useDependency removal #52975

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 20 commits into
base: main
Choose a base branch
from
Draft

Upgrade repo commits to account for useDependency removal #52975

wants to merge 20 commits into from
+85,243 −326

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Oct 1, 2025 •
edited
Loading

  • Identify all tsp-location.yaml files that need updating (commit before 6267b64842af3d744c5b092a3f3beef49729ad6d)
  • Create script to update tsp-location.yaml files to the target commit
  • Update 29 tsp-location.yaml files with the target commit hash (excluding neonpostgres, terraform, and computefleet due to issues)
  • Regenerate code for each updated library using dotnet build /t:GenerateCode
    • Successfully regenerated 29 libraries
    • Fixed directory paths for 3 libraries that had been reorganized in specs:
      • Azure.ResourceManager.Fabric
      • Azure.ResourceManager.DeviceProvisioningServices
      • Azure.ResourceManager.RecoveryServicesDataReplication
    • Excluded Azure.ResourceManager.NeonPostgres due to generation issues (will be handled in follow-up)
    • Excluded Azure.ResourceManager.Terraform due to test failures (will be handled in follow-up)
    • Excluded Azure.ResourceManager.ComputeFleet (will be handled in follow-up)
  • Merged from main branch (twice - initial merge and second merge to stay up to date)
  • Reverted all changes to metadata files (.sln, .csproj, .md, .props, .yml, .json, AssemblyInfo.cs) - only tsp-location.yaml and generated code remain changed

Note: The neonpostgres, terraform, and computefleet libraries were excluded from this update and will be addressed in separate follow-up issues.

Original prompt

This section details on the original issue you should resolve

<issue_title>Upgrade repo commits to account for useDependency removal</issue_title>
<issue_description>UseDependency was removed from all specs in Azure/azure-rest-api-specs@6267b64. It will also be removed from the Azure TypeSpec packages so we need to upgrade the tsp-location to reference at least 6267b64842af3d744c5b092a3f3beef49729ad6d. The logic should be, if the current tsp-location.yaml points to a commit after 6267b64842af3d744c5b092a3f3beef49729ad6d, then do nothing, else update it to 6267b64842af3d744c5b092a3f3beef49729ad6d. We would then need to regenerate that library using dotnet build /t:GenerateCode from the src directory. </issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #52974

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot finished work on behalf of JoshLove-msft October 1, 2025 23:54
@JoshLove-msft
Copy link
Member

@copilot merge from main

@Copilot
Merge remote-tracking branch 'origin/main' into copilot/fix-18a989fc-…
82f5ba4 
…d467-42b7-8c40-054fa42769fc

# Conflicts:
#	sdk/communication/Azure.Communication.Messages/tsp-location.yaml
@Copilot Copilot SWE Agent
Copy link
Contributor Author

Copilot AI commented Oct 2, 2025

@copilot merge from main

Merged from main in commit 82f5ba4. Resolved conflict in Azure.Communication.Messages/tsp-location.yaml by keeping the newer commit from main.

Copilot finished work on behalf of JoshLove-msft October 2, 2025 21:41
@Copilot Copilot AI requested a review from JoshLove-msft October 2, 2025 21:41
Copilot finished work on behalf of JoshLove-msft October 2, 2025 21:46
JoshLove-msft
JoshLove-msft reviewed Oct 2, 2025
View reviewed changes
JoshLove-msft
JoshLove-msft reviewed Oct 2, 2025
View reviewed changes
JoshLove-msft
JoshLove-msft reviewed Oct 2, 2025
View reviewed changes
JoshLove-msft
JoshLove-msft reviewed Oct 2, 2025
View reviewed changes
@Copilot Copilot AI requested a review from JoshLove-msft October 2, 2025 21:51
Copilot finished work on behalf of JoshLove-msft October 2, 2025 22:09
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 2, 2025 •
edited
Loading

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Azure.AI.Language.Conversations.Authoring
Azure.AI.Agents.Persistent
Azure.ResourceManager.DependencyMap

@Copilot Copilot AI changed the title [WIP] Upgrade repo commits to account for useDependency removal Upgrade repo commits to account for useDependency removal Oct 2, 2025
@Copilot Copilot AI requested a review from JoshLove-msft October 2, 2025 23:22
Copilot finished work on behalf of JoshLove-msft October 2, 2025 23:22
@JoshLove-msft
Copy link
Member

@copilot merge from main

@Copilot Copilot SWE Agent
Copy link
Contributor Author

Copilot AI commented Oct 3, 2025

@copilot merge from main

Merged from main in commit 6b896fb. No conflicts encountered.

Copilot finished work on behalf of JoshLove-msft October 3, 2025 16:24
@Copilot Copilot AI requested a review from JoshLove-msft October 6, 2025 15:31
Copilot finished work on behalf of JoshLove-msft October 6, 2025 15:31
Copilot finished work on behalf of JoshLove-msft October 6, 2025 17:43
@Copilot Copilot AI requested a review from JoshLove-msft October 6, 2025 17:43
@nick863 @JoshLove-msft
Use type parameter for the tool outputs. (#53034)
d04aab8 
* Use type parameter for the tool outputs.

* Update sdk/ai/Azure.AI.Agents.Persistent/src/Custom/ToolOutput.cs

Co-authored-by: JoshLove-msft <[email protected]>

---------

Co-authored-by: JoshLove-msft <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JoshLove-msft JoshLove-msft Awaiting requested review from JoshLove-msft

@atharvau atharvau Awaiting requested review from atharvau atharvau will be requested when the pull request is marked ready for review atharvau is a code owner

@cjoprey cjoprey Awaiting requested review from cjoprey cjoprey will be requested when the pull request is marked ready for review cjoprey is a code owner

@locomotiveviaduct locomotiveviaduct Awaiting requested review from locomotiveviaduct locomotiveviaduct will be requested when the pull request is marked ready for review locomotiveviaduct is a code owner

@deepakmauryams deepakmauryams Awaiting requested review from deepakmauryams deepakmauryams will be requested when the pull request is marked ready for review deepakmauryams is a code owner

@berndverst berndverst Awaiting requested review from berndverst berndverst will be requested when the pull request is marked ready for review berndverst is a code owner

@kaibocai kaibocai Awaiting requested review from kaibocai kaibocai will be requested when the pull request is marked ready for review kaibocai is a code owner

@torosent torosent Awaiting requested review from torosent torosent will be requested when the pull request is marked ready for review torosent is a code owner

@philliphoff philliphoff Awaiting requested review from philliphoff philliphoff will be requested when the pull request is marked ready for review philliphoff is a code owner

@cgillum cgillum Awaiting requested review from cgillum cgillum will be requested when the pull request is marked ready for review cgillum is a code owner

@ArthurMa1978 ArthurMa1978 Awaiting requested review from ArthurMa1978 ArthurMa1978 will be requested when the pull request is marked ready for review ArthurMa1978 is a code owner

@ArcturusZhang ArcturusZhang Awaiting requested review from ArcturusZhang ArcturusZhang will be requested when the pull request is marked ready for review ArcturusZhang is a code owner

@rhurey rhurey Awaiting requested review from rhurey rhurey will be requested when the pull request is marked ready for review rhurey is a code owner

@dargilco dargilco Awaiting requested review from dargilco dargilco will be requested when the pull request is marked ready for review dargilco is a code owner

@alexathomases alexathomases Awaiting requested review from alexathomases alexathomases will be requested when the pull request is marked ready for review alexathomases is a code owner

@quentinRobinson quentinRobinson Awaiting requested review from quentinRobinson quentinRobinson will be requested when the pull request is marked ready for review quentinRobinson is a code owner

@asaflevi-ms asaflevi-ms Awaiting requested review from asaflevi-ms asaflevi-ms will be requested when the pull request is marked ready for review asaflevi-ms is a code owner

@tomsft tomsft Awaiting requested review from tomsft tomsft will be requested when the pull request is marked ready for review tomsft is a code owner

@koen-mertens koen-mertens Awaiting requested review from koen-mertens koen-mertens will be requested when the pull request is marked ready for review koen-mertens is a code owner

@mengaims mengaims Awaiting requested review from mengaims mengaims will be requested when the pull request is marked ready for review mengaims is a code owner

@wiboris wiboris Awaiting requested review from wiboris wiboris will be requested when the pull request is marked ready for review wiboris is a code owner

@dpwatrous dpwatrous Awaiting requested review from dpwatrous dpwatrous will be requested when the pull request is marked ready for review dpwatrous is a code owner

@wanghoppe wanghoppe Awaiting requested review from wanghoppe wanghoppe will be requested when the pull request is marked ready for review wanghoppe is a code owner

@skapur12 skapur12 Awaiting requested review from skapur12 skapur12 will be requested when the pull request is marked ready for review skapur12 is a code owner

@cRui861 cRui861 Awaiting requested review from cRui861 cRui861 will be requested when the pull request is marked ready for review cRui861 is a code owner

@trangevi trangevi Awaiting requested review from trangevi trangevi will be requested when the pull request is marked ready for review trangevi is a code owner

@glharper glharper Awaiting requested review from glharper glharper will be requested when the pull request is marked ready for review glharper is a code owner

@nick863 nick863 Awaiting requested review from nick863 nick863 will be requested when the pull request is marked ready for review nick863 is a code owner

@jsquire jsquire Awaiting requested review from jsquire jsquire will be requested when the pull request is marked ready for review jsquire is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@JoshLove-msft JoshLove-msft

Copilot code review Copilot

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Upgrade repo commits to account for useDependency removal
3 participants
@Copilot @JoshLove-msft @nick863