[Agents Extension] Remove prompt agent handling by trangevi · Pull Request #8040 · Azure/azure-dev

trangevi · 2026-05-04T23:55:36Z

Prompt agents have been deprioritized, and so much of our recent work has ignored them completely. Having the additional branches just muddies the code and makes it harder to make well scoped changes. When/if prompt agents are added back, we can do it in a way that fits will all of our existing features. This PR removes all of the previous prompt agent code and registry (azureml://) support.

Signed-off-by: trangevi <trangevi@microsoft.com>

Copilot

Pull request overview

This PR removes the (now-deprioritized) prompt-agent pathway from the azure.ai.agents extension, simplifying manifest parsing/mapping and deployment flows to focus on hosted/container-based agents.

Changes:

Removed prompt-agent kind/type handling across agent YAML parsing, request mapping, and service deployment.
Removed registry-manifest download support (and the registry_api package) that primarily existed to materialize prompt-agent manifests.
Refactored manifest-parameter prompting and placeholder injection into agent_yaml and updated tests/fixtures accordingly.

Reviewed changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
cli/azd/extensions/azure.ai.agents/tests/samples/githubMcpAgent/agent.yaml	Removed prompt-agent sample manifest.
cli/azd/extensions/azure.ai.agents/tests/samples/declarativeNoTools/agent.yaml	Removed prompt-agent sample manifest.
cli/azd/extensions/azure.ai.agents/internal/project/service_target_agent.go	Removed prompt-agent deploy branch and prompt-specific display output.
cli/azd/extensions/azure.ai.agents/internal/project/service_target_agent_test.go	Renamed artifact protocol test to remove prompt-agent terminology.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/registry_api/operations.go	Deleted registry manifest client implementation.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/registry_api/models.go	Deleted registry manifest models.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/registry_api/helpers.go	Deleted registry manifest processing + parameter prompting/injection helpers.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/registry_api/helpers_test.go	Deleted tests covering registry helper behaviors.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/yaml.go	Removed `prompt` agent kind and `PromptAgent` schema type.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/testdata/prompt-agent-minimal.yaml	Removed prompt-agent fixture.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/testdata/prompt-agent-full.yaml	Removed prompt-agent fixture.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/testdata/mcp-tools-agent.yaml	Removed prompt-agent fixture that exercised MCP tools.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/testdata/invalid-no-model.yaml	Removed prompt-agent invalid fixture.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/testdata_test.go	Removed prompt-agent fixture validation tests.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/parse.go	Removed prompt-agent extraction/validation paths and tool-connection prompting.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/parameters.go	Added manifest-parameter prompting + placeholder injection utilities in `agent_yaml`.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/map.go	Removed prompt-agent API request mapping path and narrowed supported-kind error text.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/map_test.go	Updated mapping tests to remove prompt-agent cases and references.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_api/models.go	Removed prompt-agent-related API model types/constants (and related agent kinds).
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_api/models_test.go	Updated round-trip tests to match the reduced agent model surface.
cli/azd/extensions/azure.ai.agents/internal/cmd/init.go	Switched parameter prompting to `agent_yaml`, removed registry URL handling, removed prompt-agent tool-connection processing.
cli/azd/extensions/azure.ai.agents/internal/cmd/init_test.go	Removed prompt-agent specific init tests.
cli/azd/extensions/azure.ai.agents/internal/cmd/init_models.go	Removed prompt-agent model-deployment path and moved parameter injection types to `agent_yaml`.

Signed-off-by: trangevi <trangevi@microsoft.com>

wbreza

✅ Approve — Clean, well-scoped removal

This PR successfully removes the deprioritized prompt-agent code path from the \�zure.ai.agents\ extension. Verified:

\PromptAgent\ struct, \AgentKindPrompt\ constant, and \ValidAgentKinds()\ — all correctly updated
\CreatePromptAgentAPIRequest()\ and prompt mapping branch — removed from \map.go\
egistry_api\ package — entirely deleted (models, helpers, operations, tests)
Service target deploy path simplified to hosted-only with updated error messages
Init command — registry URL handling removed, parameter prompting migrated to \�gent_yaml\
All prompt agent test fixtures and sample YAML files cleaned up
No orphaned imports, no broken control flow, no dead code

Minor observations (P2 — non-blocking)

Non-relocated helper functions: 11 helper functions from
egistry_api/helpers.go\ were deleted rather than relocated (e.g., \ConvertAgentDefinition, \ValidateParameterValue, \MergeManifestIntoAgentDefinition). These appear to be prompt-agent-specific since they lived in the registry manifest package — worth a quick confirmation that none are needed for hosted agent flows.
Test coverage proportional to removal: Test lines went from 863 → 85, but the reduction tracks the deleted functionality. The relocated \injectParameterValues\ logic has solid test coverage (8 test cases covering brace formats, spacing, multi-param, empty templates, and round-trip).

Overall: clean PR, good scoping. 👍

jongio

Clean removal - the parameter logic migration to agent_yaml is well-structured and uses the correct YAML library (go.yaml.in/yaml/v3). Tests cover the injection logic thoroughly.

A few notes:

The Copilot bot comments about YAML library mismatch and missing tests are both inaccurate - the new code already uses the right lib and includes tests in parameters_test.go. The ContainerAppAgentDefinition comment is also wrong since that type IS removed in this diff.
The bot's comment about documenting the registry URL removal is valid though - dropping azureml:// support is user-facing and worth a line in the PR description or changelog.
Minor: listen.go:131 has a stale comment ("Only hosted agents get platform-created per-agent identities; prompt agents do not.") that references a concept that no longer exists. Not blocking, but worth cleaning up.

JeffreyCA

A couple of cleanup misses after the prompt-agent removal:

cli/azd/extensions/azure.ai.agents/internal/cmd/init_test.go:1162 still has the "azureml registry URL is manifest" case for azureml://registries/myReg/agentmanifests/myManifest. Since the registry URL download path was removed and downloadAgentYaml() now only accepts local files or GitHub URLs, this test case looks stale and should probably be removed.
cli/azd/extensions/azure.ai.agents/internal/pkg/agents/agent_yaml/yaml.go:186 still says variables are projected into a “prompt agent”. With prompt agents removed, this comment is stale; something like “used to configure the agent dynamically” would avoid implying prompt-agent support still exists.

Signed-off-by: trangevi <trangevi@microsoft.com>

therealjohn

We'll be able to better support prompt agents or similar declarative approaches built on the changes in #7962 #7975 #8049 etc. Approving so we can revisit on a better foundation.

github-actions · 2026-05-05T19:40:03Z

📋 Prioritization Note

Thanks for the contribution! The linked issue isn't in the current milestone yet.
Review may take a bit longer — reach out to @rajeshkamal5050 or @kristenwomack if you'd like to discuss prioritization.

trangevi · 2026-05-05T21:02:06Z

/check-enforcer override

* Initial changes to remove prompt agents Signed-off-by: trangevi <trangevi@microsoft.com> * cleanup Signed-off-by: trangevi <trangevi@microsoft.com> * PR comments Signed-off-by: trangevi <trangevi@microsoft.com> * PR comments Signed-off-by: trangevi <trangevi@microsoft.com> --------- Signed-off-by: trangevi <trangevi@microsoft.com> Co-authored-by: therealjohn <1501196+therealjohn@users.noreply.github.com>

trangevi added 2 commits May 4, 2026 16:29

Initial changes to remove prompt agents

cf437f8

Signed-off-by: trangevi <trangevi@microsoft.com>

cleanup

7cb0cd9

Signed-off-by: trangevi <trangevi@microsoft.com>

trangevi requested a review from Copilot May 4, 2026 23:55

microsoft-github-policy-service Bot assigned trangevi May 4, 2026

Copilot started reviewing on behalf of trangevi May 5, 2026 00:01 View session

Copilot AI reviewed May 5, 2026

View reviewed changes

PR comments

0058920

Signed-off-by: trangevi <trangevi@microsoft.com>

trangevi marked this pull request as ready for review May 5, 2026 00:34

trangevi requested review from JeffreyCA, hemarina, jongio, rajeshkamal5050, tg-msft, therealjohn, trrwilson, vhvb1989, wbreza and weikanglim as code owners May 5, 2026 00:34

wbreza approved these changes May 5, 2026

View reviewed changes

jongio reviewed May 5, 2026

View reviewed changes

JeffreyCA added the ext-agents azure.ai.agents extension label May 5, 2026

JeffreyCA reviewed May 5, 2026

View reviewed changes

JeffreyCA approved these changes May 5, 2026

View reviewed changes

PR comments

172aefa

Signed-off-by: trangevi <trangevi@microsoft.com>

JeffreyCA approved these changes May 5, 2026

View reviewed changes

trangevi enabled auto-merge (squash) May 5, 2026 18:03

therealjohn approved these changes May 5, 2026

View reviewed changes

trangevi linked an issue May 5, 2026 that may be closed by this pull request

[Agent Extension] Remove Prompt Agent handling #8053

Closed

trangevi merged commit 02c9713 into main May 5, 2026
21 of 25 checks passed

trangevi deleted the trangevi/remove-prompt-agents branch May 5, 2026 21:04

This was referenced May 6, 2026

Prepare azure.ai.agents 0.1.30-preview patch release #8082

Merged

[ai-agents] Fix MCP tool field mapping #8038

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Agents Extension] Remove prompt agent handling - #8040

[Agents Extension] Remove prompt agent handling#8040
trangevi merged 4 commits into
mainfrom
trangevi/remove-prompt-agents

trangevi commented May 4, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

wbreza left a comment

Uh oh!

jongio left a comment

Uh oh!

JeffreyCA left a comment

Uh oh!

therealjohn left a comment

Uh oh!

github-actions Bot commented May 5, 2026

Uh oh!

trangevi commented May 5, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Uh oh!

Conversation

trangevi commented May 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

wbreza left a comment

Choose a reason for hiding this comment

✅ Approve — Clean, well-scoped removal

Minor observations (P2 — non-blocking)

Uh oh!

jongio left a comment

Choose a reason for hiding this comment

Uh oh!

JeffreyCA left a comment

Choose a reason for hiding this comment

Uh oh!

therealjohn left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented May 5, 2026

📋 Prioritization Note

Uh oh!

trangevi commented May 5, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

trangevi commented May 4, 2026 •

edited

Loading