Skip to content

Conversation

@ReubenBond
Copy link
Member

Motivation and Context

Currently, if we have both an agent and a workflow with the same id, the backend does not know which one the frontend is interested in when it requests entity info, so it must choose arbitrarily.

This PR adds an optional 'type' parameter which lets the backend distinguish.

This enables representing a workflow as both an agent (using .AsAIAgent()) and as a workflow, with both views on the entity being visible in DevUI.

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.

Copilot AI review requested due to automatic review settings November 7, 2025 23:44
@github-actions github-actions bot changed the title DevUI: support having both an agent and a workflow with the same in discovery .NET: DevUI: support having both an agent and a workflow with the same in discovery Nov 7, 2025
@github-actions github-actions bot changed the title .NET: DevUI: support having both an agent and a workflow with the same in discovery Python: DevUI: support having both an agent and a workflow with the same in discovery Nov 7, 2025
@markwallace-microsoft
Copy link
Member

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
TOTAL13035193985% 
report-only-changed-files is enabled. No files were changed during this commit :)

Python Unit Test Overview

Tests Skipped Failures Errors Time
1740 107 💤 0 ❌ 0 🔥 35.198s ⏱️

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants