Add polyglot exports for Aspire.Hosting.Orleans#14891
Merged
sebastienros merged 8 commits intorelease/13.2from Mar 5, 2026
Merged
Add polyglot exports for Aspire.Hosting.Orleans#14891sebastienros merged 8 commits intorelease/13.2from
sebastienros merged 8 commits intorelease/13.2from
Conversation
Contributor
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 14891Or
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 14891" |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds ATS/polyglot export coverage for Aspire.Hosting.Orleans so TypeScript AppHosts can configure Orleans resources via generated capabilities, and includes a TypeScript ValidationAppHost to compile-check the generated SDK surface.
Changes:
- Annotates Orleans resource types and extension methods with
[AspireExport]and marks non-polyglot-friendly overloads with[AspireExportIgnore]. - Adds Orleans TypeScript ValidationAppHost (tsconfig/package manifests, generated
.modules, and sampleapphost.ts) to validate codegen/typing. - Exposes Orleans “silo” and “client” reference wiring as separate polyglot capabilities.
Reviewed changes
Copilot reviewed 12 out of 14 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Aspire.Hosting.Orleans/OrleansServiceExtensions.cs | Adds [AspireExport] coverage for Orleans configuration APIs and reference wiring; adds ignore annotations for non-polyglot overloads. |
| src/Aspire.Hosting.Orleans/OrleansServiceClientExtensions.cs | Exports the Orleans client reference wiring capability for polyglot AppHosts. |
| src/Aspire.Hosting.Orleans/OrleansServiceClient.cs | Marks Orleans client type as an ATS-exported type. |
| src/Aspire.Hosting.Orleans/OrleansService.cs | Marks Orleans service type as an ATS-exported type. |
| playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/tsconfig.json | TypeScript compiler configuration for Orleans ValidationAppHost. |
| playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/package.json | NPM manifest for the Orleans ValidationAppHost. |
| playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/package-lock.json | Dependency lockfile for Orleans ValidationAppHost. |
| playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/apphost.ts | TypeScript AppHost script exercising the exported Orleans capabilities. |
| playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/apphost.run.json | Run profile/environment variables for the Orleans ValidationAppHost. |
| playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/.modules/transport.ts | Generated ATS transport/runtime support for the ValidationAppHost. |
| playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/.modules/base.ts | Generated base types/helpers for the ValidationAppHost. |
| playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/.modules/.codegen-hash | Tracks codegen hash for generated .modules output. |
| playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/.aspire/settings.json | Configures the ValidationAppHost as a TypeScript AppHost and references the Orleans package. |
Files not reviewed (1)
- playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/package-lock.json: Language not supported
playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/apphost.ts
Outdated
Show resolved
Hide resolved
76 tasks
Contributor
🎬 CLI E2E Test RecordingsThe following terminal recordings are available for commit
📹 Recordings uploaded automatically from CI run #22697464622 |
IEvangelist
previously requested changes
Mar 3, 2026
playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/apphost.ts
Outdated
Show resolved
Hide resolved
davidfowl
previously approved these changes
Mar 4, 2026
davidfowl
reviewed
Mar 4, 2026
playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/.aspire/settings.json
Outdated
Show resolved
Hide resolved
davidfowl
reviewed
Mar 4, 2026
playground/polyglot/TypeScript/Aspire.Hosting.Orleans/ValidationAppHost/.aspire/settings.json
Outdated
Show resolved
Hide resolved
davidfowl
approved these changes
Mar 5, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Rename withSiloReference → withOrleansReference and withClientReference → withOrleansClientReference to follow the pattern used by other integrations (e.g., withServiceReference for IResourceWithServiceDiscovery). Update playground ValidationAppHost and regenerate TypeScript SDK. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Removed sdkVersion and cleared Aspire.Hosting.Orleans version.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
f6e70b9 to
f47772f
Compare
eerhardt
pushed a commit
to eerhardt/aspire
that referenced
this pull request
Mar 7, 2026
* Add AspireExport coverage for Orleans Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update src/Aspire.Hosting.Orleans/OrleansServiceExtensions.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update src/Aspire.Hosting.Orleans/OrleansServiceClientExtensions.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Fix XML documentation return tags in OrleansServiceExtensions * Address review feedback for polyglot-orleans Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Rename Orleans WithReference export IDs to match naming conventions Rename withSiloReference → withOrleansReference and withClientReference → withOrleansClientReference to follow the pattern used by other integrations (e.g., withServiceReference for IResourceWithServiceDiscovery). Update playground ValidationAppHost and regenerate TypeScript SDK. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Cleanup settings.json Removed sdkVersion and cleared Aspire.Hosting.Orleans version. * Regenerate Orleans playground after rebase on release/13.2 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI
pushed a commit
that referenced
this pull request
Mar 10, 2026
* Add AspireExport coverage for Orleans Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update src/Aspire.Hosting.Orleans/OrleansServiceExtensions.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update src/Aspire.Hosting.Orleans/OrleansServiceClientExtensions.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Fix XML documentation return tags in OrleansServiceExtensions * Address review feedback for polyglot-orleans Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Rename Orleans WithReference export IDs to match naming conventions Rename withSiloReference → withOrleansReference and withClientReference → withOrleansClientReference to follow the pattern used by other integrations (e.g., withServiceReference for IResourceWithServiceDiscovery). Update playground ValidationAppHost and regenerate TypeScript SDK. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Cleanup settings.json Removed sdkVersion and cleared Aspire.Hosting.Orleans version. * Regenerate Orleans playground after rebase on release/13.2 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Add polyglot AspireExport coverage for Aspire.Hosting.Orleans.
Related to #14069.
Validation:
Fixes # (issue)
Checklist
<remarks />and<code />elements on your triple slash comments?aspire.devissue: