Skip to content

.NET Workflows - Remove empty message injection in declarative workflow agent provider.#3819

Merged
crickman merged 2 commits intomainfrom
crickman/fix-workflows-declarative-remove-empty-message
Feb 10, 2026
Merged

.NET Workflows - Remove empty message injection in declarative workflow agent provider.#3819
crickman merged 2 commits intomainfrom
crickman/fix-workflows-declarative-remove-empty-message

Conversation

@crickman
Copy link
Contributor

Motivation and Context

Removing workaround for SDK limitation.

Description

SDK no longers requires an input message when conversation-id is present.

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

@crickman crickman requested a review from peibekwe February 10, 2026 19:24
@crickman crickman self-assigned this Feb 10, 2026
@crickman crickman added .NET workflows Related to Workflows in agent-framework labels Feb 10, 2026
Copilot AI review requested due to automatic review settings February 10, 2026 19:24
@crickman crickman added v1.0 Features being tracked for the version 1.0 GA declarative-workflow labels Feb 10, 2026
@crickman crickman enabled auto-merge February 10, 2026 19:27
Copy link
Contributor

Copilot AI left a 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 removes a workaround in the .NET declarative workflow Azure AI agent provider that injected an empty user message when no input messages were provided, aligning behavior with the updated SDK capability to run with a conversation id and no new input.

Changes:

  • Stop injecting new ChatMessage(ChatRole.User, string.Empty) when messages is null.
  • Invoke the agent with an empty message collection instead (RunStreamingAsync([] , ...)).

@crickman crickman added this pull request to the merge queue Feb 10, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 10, 2026
@crickman crickman moved this to In Review in Agent Framework Feb 10, 2026
@crickman crickman enabled auto-merge February 10, 2026 21:15
@crickman crickman added this pull request to the merge queue Feb 10, 2026
Merged via the queue into main with commit f407f72 Feb 10, 2026
14 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in Agent Framework Feb 10, 2026
@crickman crickman deleted the crickman/fix-workflows-declarative-remove-empty-message branch February 11, 2026 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

declarative-workflow .NET v1.0 Features being tracked for the version 1.0 GA workflows Related to Workflows in agent-framework

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants