Skip to content

.Net: Removed experimental flags for stable items#13013

Merged
markwallace-microsoft merged 3 commits intomicrosoft:mainfrom
dmytrostruk:remove-experimental-flags
Aug 27, 2025
Merged

.Net: Removed experimental flags for stable items#13013
markwallace-microsoft merged 3 commits intomicrosoft:mainfrom
dmytrostruk:remove-experimental-flags

Conversation

@dmytrostruk
Copy link
Copy Markdown
Member

Motivation and Context

Resolves: #12982

Removed experimental flags in following items:

  1. ChatClientExtensions
  2. ChatClientBuilderExtensions
  3. AIFunctionExtensions
  4. ChatCompletionServiceExtensions
  5. AuthorName property in ChatMessageContent and StreamingChatMessageContent models.
  6. KernelFunctionExtensions
  7. KernelPlugin

Contribution Checklist

@dmytrostruk dmytrostruk self-assigned this Aug 26, 2025
@dmytrostruk dmytrostruk requested a review from a team as a code owner August 26, 2025 21:33
@moonbox3 moonbox3 added .NET Issue or Pull requests regarding .NET code kernel.core labels Aug 26, 2025
@markwallace-microsoft markwallace-microsoft added this pull request to the merge queue Aug 27, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 27, 2025
@markwallace-microsoft
Copy link
Copy Markdown
Contributor

Merge failed due to unrelated integration test failure

[xUnit.net 00:06:35.62]         --- End of stack trace from previous location ---
  Failed SemanticKernel.IntegrationTests.Agents.CommonInterfaceConformance.AgentWithTextSearchBehaviorConformance.AzureAIAgentWithTextSearchProviderTests.TextSearchBehaviorStateIsUsedByAgentInternalAsync(question: "What was Acme Corp's revenue in Q1 2025?", expectedResult: "12.5", ragResults: ["Acme Corp reported a revenue of $12.5 million in Q"···, "This represents a 15% increase from Q4 2024."]) [3 m 59 s]
  Error Message:
   Azure.RequestFailedException : Service invocation timed out. 
Request: POST obotoken.vienna-eastus.svc/obotoken/v1.0/saveusertoken/v2 
 Message: Unable to read data from the transport connection: Operation canceled. Time waited: 00:00:10.0006724

@markwallace-microsoft markwallace-microsoft merged commit e65a0ab into microsoft:main Aug 27, 2025
19 checks passed
jcruzmot-te pushed a commit to thousandeyes/aia-semantic-kernel that referenced this pull request Sep 15, 2025
### Motivation and Context

<!-- Thank you for your contribution to the semantic-kernel repo!
Please help reviewers and future users, providing the following
information:
  1. Why is this change required?
  2. What problem does it solve?
  3. What scenario does it contribute to?
  4. If it fixes an open issue, please link to the issue here.
-->

Resolves: microsoft#12982

Removed experimental flags in following items:
1. `ChatClientExtensions`
2. `ChatClientBuilderExtensions`
3. `AIFunctionExtensions`
4. `ChatCompletionServiceExtensions`
5. `AuthorName` property in `ChatMessageContent` and
`StreamingChatMessageContent` models.
6. `KernelFunctionExtensions`
7. `KernelPlugin`

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [x] The code builds clean without any errors or warnings
- [x] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kernel.core .NET Issue or Pull requests regarding .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

.Net: New Feature: Adding Experimental Function to Production

5 participants