Skip to content

.Net - Update and Fixes for AzureAIAgent - #12348

Merged
Chris (crickman) merged 4 commits into
mainfrom
update-agents-azurefoundry
Jun 3, 2025
Merged

Chris (crickman) merged 4 commits into
mainfrom
update-agents-azurefoundry

Conversation

@crickman

@crickman Chris (crickman) commented Jun 3, 2025 •

Copy link
Copy Markdown
Contributor

Motivation and Context

Address customer input and discovered issues for AzureAIAgent

Description

Address several issues:

  • Support image input
  • Null reference when function calling parameter is null
  • Null reference when using BingGroundingTool with streaming
  • Add samples for response-format
  • Fix connection resolution for declarative bing tool

Fixes: #12288
Fixes: #12351
Fixes: #12339

Contribution Checklist

@crickman Chris (crickman) self-assigned this Jun 3, 2025
@crickman
Chris (crickman) requested a review from a team as a code owner June 3, 2025 05:02
@crickman Chris (crickman) added .NET Issue or Pull requests regarding .NET code agents labels Jun 3, 2025
@github-actions github-actions Bot changed the title .NET - Update and Fixes for AzureAIAgent .Net - Update and Fixes for AzureAIAgent Jun 3, 2025
@crickman Chris (crickman) moved this to Sprint: In Review in Semantic Kernel Jun 3, 2025
@crickman
Chris (crickman) added this pull request to the merge queue Jun 3, 2025
Merged via the queue into main with commit 6683aff Jun 3, 2025
@crickman
Chris (crickman) deleted the update-agents-azurefoundry branch June 3, 2025 15:30
@github-project-automation github-project-automation Bot moved this from Sprint: In Review to Sprint: Done in Semantic Kernel Jun 3, 2025
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.
-->

Address customer input and discovered issues for `AzureAIAgent`

### Description
<!-- Describe your changes, the overall approach, the underlying design.
These notes will help understanding how your code works. Thanks! -->

Address several issues:

- Support image input
- Null reference when function calling parameter is null
- Null reference when using BingGroundingTool with streaming
- Add samples for response-format
- Fix connection resolution for declarative bing tool

Fixes: microsoft#12288

### 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 😄
Anton Dziatkovskii (tonydzi) pushed a commit to tonydzi/semantic-kernel that referenced this pull request Nov 2, 2026
…ADME (microsoft#14699)

## Motivation

The sample tables in `dotnet/samples/GettingStartedWithAgents/README.md`
contain 20 broken links:

1. **18 links use a wrong path prefix** — they were written as
`./dotnet/samples/GettingStartedWithAgents/...`, but the README itself
already lives in `dotnet/samples/GettingStartedWithAgents/`, so the
links resolve to a doubled path and 404 on GitHub. The BedrockAgent,
CopilotStudioAgent and Orchestration sections in the same file use
correct relative paths (e.g. `./BedrockAgent/...`), which makes the
inconsistency clear.
2. **2 links point to renamed files** in the `Orchestration/` folder:
`Step02a_Sequential.cs` is now `Step02a_SequentialCancellation.cs` and
`Step04b_HandoffWithStructuredInput.cs` is now
`Step04a_HandoffWithStructuredInput.cs`.
3. **1 entry points to a removed sample**: `Step03_AzureAIAgent_Chat.cs`
was removed and replaced by `Step03_AzureAIAgent_Vision.cs` in microsoft#12348
(both changes in the same commit), but the README row was never updated.

## Changes

All in `dotnet/samples/GettingStartedWithAgents/README.md` (20 lines):

- Agent section (7 links): strip the wrong
`./dotnet/samples/GettingStartedWithAgents/` prefix.
- OpenAIAssistant section (5 links): same prefix fix.
- AzureAIAgent section (6 links): same prefix fix; additionally the
Step03 row now points to `./AzureAIAgent/Step03_AzureAIAgent_Vision.cs`
with its title and description updated to match the replacement sample.
- Orchestration section (2 links): update to the renamed
`Step02a_SequentialCancellation.cs` and
`Step04a_HandoffWithStructuredInput.cs`.

## Verification

- Every new link target was verified to exist on the current `main` tree
(file listing of `dotnet/samples/GettingStartedWithAgents/`,
`OpenAIAssistant/`, `AzureAIAgent/`, `Orchestration/`).
- The removal of `Step03_AzureAIAgent_Chat.cs` and addition of
`Step03_AzureAIAgent_Vision.cs` were confirmed via the commit history of
the `AzureAIAgent` folder (microsoft#12348); the new description mirrors the
wording of the OpenAIAssistant Vision row.
- No text outside the affected table rows was modified.

> [!NOTE]
> This documentation fix was prepared with AI assistance (an automated
contribution workflow). Happy to adjust anything — e.g. if you prefer
keeping the old `Step03_AzureAIAgent_Chat` row title or handling it
differently.

This branch was previously deployed

1 inactive deployment
integration — 5c57865d Deployed Jun 3, 2025 by crickman
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents .NET Issue or Pull requests regarding .NET code

Projects

Archived in project

3 participants