-
Notifications
You must be signed in to change notification settings - Fork 841
Merge internal changes #6921
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.
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
Merge internal changes #6921
Conversation
Getting the branch ready for 9.10 release ---- #### AI description (iteration 1) #### PR Classification This pull request prepares the branch for the 9.10 release by updating dependency versions and adjusting build configurations. #### PR Summary The update bumps multiple dependency versions from 9.0.9 to 9.0.10, updates release stabilization properties, and refines build pipeline tasks and NuGet configurations. - **`eng/Version.Details.xml` and `eng/Versions.props`**: Dependency versions and SHA values have been updated to 9.0.10, with build properties modified to enable release stabilization and update LTS version numbers. - **`azure-pipelines.yml`**: The code coverage stage has been removed and stage dependencies adjusted. - **`NuGet.config`**: Package source mappings were removed, and internal feed sources have been appropriately disabled. - **`eng/pipelines/templates/BuildAndTest.yml` & `Directory.Build.props`**: New tasks were added to set up private feed credentials, integration test steps were commented out due to authentication requirements, and NU1507 warnings have been suppressed. <!-- GitOpsUserAgent=GitOps.Apps.Server.pullrequestcopilot -->
* Initial plan * Implement local function name cleanup for AIFunctionFactory Co-authored-by: stephentoub <[email protected]> * Fix lambda name uniqueness by capturing full ordinal suffix Co-authored-by: stephentoub <[email protected]> * Fix analyzer errors by adding comments to empty test methods Co-authored-by: stephentoub <[email protected]> * Change Assert.Contains to Assert.StartsWith for lambda name tests Co-authored-by: stephentoub <[email protected]> * Consolidate regexes and improve test parameter names Co-authored-by: stephentoub <[email protected]> * Keep ordinal suffix for local functions to ensure uniqueness Co-authored-by: stephentoub <[email protected]> * Fix build errors for netstandard2.0 and net462 targets Co-authored-by: stephentoub <[email protected]> * Cleanup --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: stephentoub <[email protected]> Co-authored-by: Stephen Toub <[email protected]>
We're currently filtering down to only AITools that are AIFunctionDeclaration. With this, we'll handle AIFunctionDeclaration wrappers the same as well, and for all other tools emit their name as the type. This can be further extended as otel adds more details about how non-function tools should be represented.
* Update ToChatResponse{Async} to also factor in AuthorName
Originally we were only factoring in Role. Then we also augmented it to factor in MessageId, since updates from different messages are by definition not part of the same message. Now it also makes sense to factor in AuthorName, as different text from different speakers / agents are also by definition not part of the same message.
* Fix useAsync inversion in tests
* add support for background responses Adds a model for supporting background responses (long-running operations), updates the chat completion model to use it, and integrates this functionality into OpenAIResponsesChatClient. --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Stephen Toub <[email protected]> Co-authored-by: westey <[email protected]>
All clean cherry-picks
…6915) It should fail gracefully by ignoring any content it can't output telemetry for rather than throwing.
Clean cherry-pick
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR merges internal changes that update dependency versions across the .NET Extensions repository. The primary purpose is to update Microsoft .NET runtime, ASP.NET Core, and Entity Framework Core packages from version 9.0.9 to 9.0.10, and LTS versions from 8.0.20 to 8.0.21.
Key changes include:
- Updated all .NET 9.0 runtime and extensions packages from 9.0.9 to 9.0.10
- Updated LTS (8.0) packages from 8.0.20 to 8.0.21
- Updated corresponding SHA references for the new package versions
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| eng/Versions.props | Updated version properties for .NET runtime, ASP.NET Core, and Entity Framework packages |
| eng/Version.Details.xml | Updated dependency version details and corresponding SHA commit references |
Merging changes from internal
This should be merged and not squashed.
Microsoft Reviewers: Open in CodeFlow