Restore hosted agent ATS compatibility by sebastienros · Pull Request #18980 · microsoft/aspire

Sébastien Ros (sebastienros) · 2026-08-03T23:20:27Z

Description

Restores the shipped polyglot asHostedAgent(project, options?) capability so existing AppHosts continue to use the Responses protocol version 2.0.0 defaults. The explicit protocol form is now exposed as the additive asHostedAgentWithProtocol capability instead of replacing the existing signature.

The two ATS adapters use distinct internal CLR names and capability IDs, preserving the existing public C# overloads without introducing overload ambiguity. TypeScript, Python, Go, and Java validation AppHosts exercise both generated methods.

Related reviews:

User-facing usage

C# callers continue to use the existing overloads:

agent.AsHostedAgent(project, HostedAgentProtocol.Invocations, "1.0.0");

Polyglot callers can retain the existing defaulted form or opt into explicit protocol selection:

await agent.asHostedAgent(project, options);
await agent.asHostedAgentWithProtocol(
    project,
    HostedAgentProtocol.Invocations,
    "1.0.0",
    options);

Validation:

Built Aspire.Hosting.Foundry with integration analyzers enabled.
Passed all 117 Aspire.Hosting.Foundry.Tests.
Generated and compiled the Foundry validation AppHosts for TypeScript, Python, Go, and Java.

Fixes # (issue)

Checklist

Is this feature complete?
- Yes. Ready to ship.
- No. Follow-up changes expected.
Are you including unit tests for the changes and scenario tests if relevant?
- Yes
- No
Did you add public API?
- Yes
  - If yes, did you have an API Review for it?
    - Yes
    - No
  - Did you add <remarks /> and <code /> elements on your triple slash comments?
    - Yes
    - No
- No
Does the change make any security assumptions or guarantees?
- Yes
  - If yes, have you done a threat model and had a security review?
    - Yes
    - No
- No

Keep the original asHostedAgent export with Responses 2.0.0 defaults and expose explicit protocol selection through a distinct capability. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c8bf16f9-3a59-4062-bd20-818fb4e70df9

github-actions · 2026-08-03T23:20:44Z

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 18980

Or

Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 18980"

Copilot

Pull request overview

Restores ATS compatibility for Foundry hosted agents while retaining explicit protocol selection.

Changes:

Restores default Responses 2.0.0 polyglot calls.
Adds asHostedAgentWithProtocol.
Exercises both APIs across four polyglot AppHosts.

Reviewed changes

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

Show a summary per file

File	Description
`src/Aspire.Hosting.Foundry/HostedAgent/HostedAgentBuilderExtension.cs`	Adds distinct compatibility and explicit-protocol ATS adapters.
`tests/Aspire.Hosting.Foundry.Tests/HostedAgentExtensionTests.cs`	Updates adapter tests; restored path needs focused regression coverage.
`tests/PolyglotAppHosts/Aspire.Hosting.Foundry/TypeScript/apphost.mts`	Validates both generated TypeScript methods.
`tests/PolyglotAppHosts/Aspire.Hosting.Foundry/Python/apphost.py`	Validates both generated Python methods.
`tests/PolyglotAppHosts/Aspire.Hosting.Foundry/Go/apphost.go`	Validates both generated Go methods.
`tests/PolyglotAppHosts/Aspire.Hosting.Foundry/Java/AppHost.java`	Validates both generated Java methods.

Add direct behavior coverage for legacy defaults and option application. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c8bf16f9-3a59-4062-bd20-818fb4e70df9

Copilot

Pull request overview

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

Suppressed comments (2)

src/Aspire.Hosting.Foundry/HostedAgent/HostedAgentBuilderExtension.cs:134

Restoring the defaulted polyglot entry point makes the existing public overload documentation at lines 80–83 inaccurate: it still says polyglot hosts must declare the protocol and version explicitly. Update those remarks and the AspireExportIgnore reason to describe asHostedAgent as the defaulted form and asHostedAgentWithProtocol as the explicit form.

        this IResourceBuilder<T> builder,
        IResourceBuilder<AzureCognitiveServicesProjectResource> project,
        HostedAgentOptions? options = null)

src/Aspire.Hosting.Foundry/HostedAgent/HostedAgentBuilderExtension.cs:160

This export adds a new user-facing generated-SDK capability, but the PR checklist says no public API was added. Mark the public-API item as Yes and record the ATS API review for this additive method so the PR metadata matches the change.

    [AspireExport("asHostedAgentWithProtocol")]

Give the explicit-protocol hosted agent a distinct HTTP target port in every polyglot validation AppHost. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c8bf16f9-3a59-4062-bd20-818fb4e70df9

Sébastien Ros (sebastienros) · 2026-08-04T00:19:40Z

cc Tommaso Stocchi (@tommasodotNET)

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

Suppressed comments (1)

src/Aspire.Hosting.Foundry/HostedAgent/HostedAgentBuilderExtension.cs:134

Restoring this defaulted ATS adapter makes the public AsHostedAgent(project, configure) documentation at lines 80–83 stale: it still says polyglot hosts must supply protocol and version explicitly, and the AspireExportIgnore reason repeats that claim. Update those remarks/reason to explain that the callback overload is replaced by the options DTO for polyglot callers, while asHostedAgentWithProtocol is the explicit-protocol alternative.

        this IResourceBuilder<T> builder,
        IResourceBuilder<AzureCognitiveServicesProjectResource> project,
        HostedAgentOptions? options = null)

github-actions · 2026-08-04T01:12:08Z

Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt.

James Newton-King (JamesNK)

1 documentation accuracy issue found. The implementation and regression coverage otherwise look correct; local Aspire.Hosting.Foundry.Tests passed 118/118 and the relevant CI checks are passing.

Document the defaulted and explicit protocol ATS entry points consistently. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c8bf16f9-3a59-4062-bd20-818fb4e70df9

github-actions · 2026-08-04T04:06:37Z

Tests selector (audit mode)

The full test matrix and all jobs still run in audit mode. The tests and jobs below are what selective CI would run under enforcement.

3 / 100 test projects · 3 jobs, from 6 changed files.

Selected test projects (3 / 100)

Aspire.Hosting.Azure.Kubernetes.Tests, Aspire.Hosting.Azure.Tests, Aspire.Hosting.Foundry.Tests

Selected jobs (3)

extension-e2e, polyglot, typescript-api-compat

How these were chosen — grouped by what changed

🔧 src/Aspire.Hosting.Foundry/HostedAgent/HostedAgentBuilderExtension.cs (changed source)
→ 1 directly: Aspire.Hosting.Foundry.Tests
→ 2 via the project graph: Aspire.Hosting.Azure.Kubernetes.Tests (2 hops), Aspire.Hosting.Azure.Tests

🧪 tests/Aspire.Hosting.Foundry.Tests/HostedAgentExtensionTests.cs (changed test)
→ 1 directly: Aspire.Hosting.Foundry.Tests

Job reasons

Job	Triggered by
`extension-e2e`	• `src/Aspire.Hosting.Foundry/HostedAgent/HostedAgentBuilderExtension.cs` • affected project `Aspire.Hosting.Foundry`
`polyglot`	`tests/PolyglotAppHosts/Aspire.Hosting.Foundry/Go/apphost.go`, `tests/PolyglotAppHosts/Aspire.Hosting.Foundry/Java/AppHost.java`, `tests/PolyglotAppHosts/Aspire.Hosting.Foundry/Python/apphost.py`, `tests/PolyglotAppHosts/Aspire.Hosting.Foundry/TypeScript/apphost.mts`
`typescript-api-compat`	affected project `Aspire.Hosting.Foundry`

Selection computed for commit f97860a.

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

github-actions · 2026-08-04T04:33:14Z

Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt.

aspire-repo-bot · 2026-08-04T17:13:13Z

⚠️ Documentation was required for this change, but a docs PR could not be drafted automatically.

Documentation was required (signal: pr_body_has_user_facing_section) but the draft PR could not be created. The create_pull_request safe-output tool failed twice with "failed to generate patch: ERR_SYSTEM: Git command failed with status 1", likely due to the shallow-clone environment. The intended change was to add a Foundry hosted agent ATS compatibility restored entry to src/frontend/src/content/docs/whats-new/aspire-13-5.mdx documenting the restored asHostedAgent polyglot export (Responses protocol 2.0.0 defaults) and the new asHostedAgentWithProtocol export for explicit protocol/version selection.

See the workflow run for details: https://github.com/microsoft/aspire/actions/runs/30932312207

Copilot AI balanced review requested due to automatic review settings August 3, 2026 23:20

Sébastien Ros (sebastienros) marked this pull request as draft August 3, 2026 23:20

github-actions Bot added the area-integrations Issues pertaining to Aspire Integrations packages label Aug 3, 2026

Copilot started reviewing on behalf of Sébastien Ros (sebastienros) August 3, 2026 23:21 View session

This comment has been minimized.

Sign in to view

Copilot AI reviewed Aug 3, 2026

View reviewed changes

Comment thread tests/Aspire.Hosting.Foundry.Tests/HostedAgentExtensionTests.cs

Test hosted agent ATS compatibility path

6573adc

Add direct behavior coverage for legacy defaults and option application. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c8bf16f9-3a59-4062-bd20-818fb4e70df9

Copilot AI review requested due to automatic review settings August 3, 2026 23:56

Copilot started reviewing on behalf of Sébastien Ros (sebastienros) August 3, 2026 23:58 View session

This comment has been minimized.

Sign in to view

Copilot AI reviewed Aug 4, 2026

View reviewed changes

Avoid hosted agent fixture port collisions

44127e1

Give the explicit-protocol hosted agent a distinct HTTP target port in every polyglot validation AppHost. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c8bf16f9-3a59-4062-bd20-818fb4e70df9

Copilot AI review requested due to automatic review settings August 4, 2026 00:15

Copilot started reviewing on behalf of Sébastien Ros (sebastienros) August 4, 2026 00:16 View session

Sébastien Ros (sebastienros) requested a review from Tommaso Stocchi (tommasodotNET) August 4, 2026 00:20

Copilot AI reviewed Aug 4, 2026

View reviewed changes

This comment has been minimized.

Sign in to view

Sébastien Ros (sebastienros) marked this pull request as ready for review August 4, 2026 01:57

James Newton-King (JamesNK) approved these changes Aug 4, 2026

View reviewed changes

Comment thread src/Aspire.Hosting.Foundry/HostedAgent/HostedAgentBuilderExtension.cs

Align hosted agent polyglot guidance

f97860a

Document the defaulted and explicit protocol ATS entry points consistently. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c8bf16f9-3a59-4062-bd20-818fb4e70df9

Copilot AI review requested due to automatic review settings August 4, 2026 04:02

Sébastien Ros (sebastienros) marked this pull request as draft August 4, 2026 04:02

Copilot started reviewing on behalf of Sébastien Ros (sebastienros) August 4, 2026 04:03 View session

Copilot AI reviewed Aug 4, 2026

View reviewed changes

Sébastien Ros (sebastienros) marked this pull request as ready for review August 4, 2026 17:05

Sébastien Ros (sebastienros) merged commit 30190f9 into main Aug 4, 2026
676 of 680 checks passed

Sébastien Ros (sebastienros) deleted the sebros/restore-hosted-agent-ats branch August 4, 2026 17:05

github-actions Bot added this to the 13.5 milestone Aug 4, 2026

David Pine (IEvangelist) mentioned this pull request Aug 6, 2026

PR Documentation Check fails to create docs PRs for release branches #19054

Open

50 tasks

Uh oh!

Conversation

Sébastien Ros (sebastienros) commented Aug 3, 2026

Description

User-facing usage

Checklist

Uh oh!

github-actions Bot commented Aug 3, 2026

Uh oh!

This comment has been minimized.

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

This comment has been minimized.

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Sébastien Ros (sebastienros) commented Aug 4, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

This comment has been minimized.

github-actions Bot commented Aug 4, 2026

Uh oh!

James Newton-King (JamesNK) left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions Bot commented Aug 4, 2026

Tests selector (audit mode)

Selected test projects (3 / 100)

Selected jobs (3)

Job reasons

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

github-actions Bot commented Aug 4, 2026

Uh oh!

Uh oh!

aspire-repo-bot Bot commented Aug 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants