Skip to content

.NET: feat: adds the executorId as an additional property when running the workflow as agent in dotnet - #7934

Open
Vincent Biret (baywet) wants to merge 9 commits into
mainfrom
feat/workflow-as-agent-executor-id-dotnet
Open

.NET: feat: adds the executorId as an additional property when running the workflow as agent in dotnet#7934
Vincent Biret (baywet) wants to merge 9 commits into
mainfrom
feat/workflow-as-agent-executor-id-dotnet

Conversation

@baywet

@baywet Vincent Biret (baywet) commented Aug 28, 2026

Copy link
Copy Markdown
Member

fixes #4445

Copilot AI balanced review requested due to automatic review settings August 28, 2026 12:53
@agent-framework-automation agent-framework-automation Bot added .NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows labels Aug 28, 2026
@github-actions github-actions Bot changed the title feat: adds the executorId as an additional property when running the workflow as agent in dotnet .NET: feat: adds the executorId as an additional property when running the workflow as agent in dotnet Aug 28, 2026
Copilot AI reviewed Aug 28, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds workflow executor attribution to streamed .NET agent response updates.

Changes:

  • Adds a public executorId metadata key.
  • Attaches executor IDs to response updates.
  • Adds multi-executor streaming coverage.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
WorkflowAgentAdditionalProperties.cs Defines the executor ID key.
AgentResponseUpdateEvent.cs Adds attribution metadata to updates.
WorkflowHostSmokeTests.cs Tests attribution and metadata preservation.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Comment thread dotnet/src/Microsoft.Agents.AI.Workflows/AgentResponseUpdateEvent.cs Outdated
@baywet
chore: linting
5366f35 
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@baywet
Vincent Biret (baywet) requested a balanced review from Copilot August 28, 2026 12:57
github-actions[bot]

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

@baywet
feat: adds the executor id to the response event to match behaviour a…
50606c7 
…cross execution methods

Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
tests: adds a regression test to avoid value being wrong because of p…
5b418c4 
…revious inputs for the executorId

Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
tests: adds unit tests to cover AgentResponseEvent executor id mapping
a96df63 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
Vincent Biret (baywet) requested a balanced review from Copilot August 28, 2026 13:13
@baywet Vincent Biret (baywet) mentioned this pull request Aug 28, 2026
Open

This comment was marked as outdated.

Sign in to view

@baywet
feat: sets the executor id on the workflow session code path
f975eab 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
Vincent Biret (baywet) requested a balanced review from Copilot August 28, 2026 14:09

This comment was marked as outdated.

Sign in to view

@baywet
chore: revert changes on failure paths
edf6435 
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@baywet
chore: reverts executorId on parts code path since it's never used
70b38a4 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
feat: adds executor id additional property on observability update
acffd27 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
Vincent Biret (baywet) requested a balanced review from Copilot August 28, 2026 14:30
Copilot AI reviewed Aug 28, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comment thread dotnet/src/Microsoft.Agents.AI.Workflows/WorkflowSession.cs
Throw.IfNull(message);

return new(message.Role, message.Contents)
return SetExecutorId(new(message.Role, message.Contents)

@baywet Vincent Biret (baywet) Aug 28, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The behaviour pre-existed this change, Tao Chen (@TaoChenOSU) to confirm whether we should fix that or not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments
Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@chetantoshniwal chetantoshniwal Awaiting requested review from chetantoshniwal chetantoshniwal is a code owner
@peibekwe Peter Ibekwe (peibekwe) Awaiting requested review from peibekwe peibekwe is a code owner
@rogerbarreto Roger Barreto (rogerbarreto) Awaiting requested review from rogerbarreto rogerbarreto is a code owner
@SergeyMenshykh SergeyMenshykh Awaiting requested review from SergeyMenshykh SergeyMenshykh is a code owner
@westey-m westey (westey-m) Awaiting requested review from westey-m westey-m is a code owner

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

Assignees

@baywet Vincent Biret (baywet)

Labels

.NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

.NET: [Feature]: Expose ExecutorId for AgentResponseUpdate when using workflow\.AsAgent().RunStreamingAsync()

2 participants

@baywet