docs: refresh add-aspire-existing-app guidance#693
Merged
IEvangelist merged 16 commits intomainfrom Apr 9, 2026
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
IEvangelist
changed the title
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Refreshes the “Add Aspire to an existing app” guide to emphasize resource modeling/relationships, updates examples to Aspire 13.2.0, and improves Steps rendering when Tabs are embedded.
Changes:
- Expand “Choose your AppHost” with API summary tables for C# and TypeScript.
- Reframe scenarios around workloads/infrastructure/dependencies and add a Docker Compose scenario.
- Update package/version examples to 13.2.0 and convert telemetry guidance into synced language tabs.
- Adjust
.sl-stepsstyling to avoid adding padding to embedded tablists.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/frontend/src/styles/site.css | Excludes tablists from .sl-steps list padding to avoid layout issues when Tabs are inside Steps. |
| src/frontend/src/content/docs/get-started/add-aspire-existing-app.mdx | Rewrites/expands the guide content, updates to 13.2.0 examples, adds Compose scenario, and reorganizes telemetry guidance into language tabs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/frontend/src/content/docs/get-started/add-aspire-existing-app.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/get-started/add-aspire-existing-app.mdx
Outdated
Show resolved
Hide resolved
…app.mdx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…app.mdx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
eerhardt
reviewed
Apr 9, 2026
| | API | Description | | ||
| | --- | --- | | ||
| | `AddContainer()` | Run a prebuilt container image that already exists in your current workflow. | | ||
| | `AddCSharpApp()` | Point a file-based AppHost at an existing C# app or `.csproj` without creating a separate AppHost project graph. | |
Member
There was a problem hiding this comment.
what does "without creating a separate AppHost project graph." mean?
eerhardt
reviewed
Apr 9, 2026
| | `AddExecutable()` | Fall back to a custom command when a dedicated hosting API does not exist yet. | | ||
| | `AddJavaScriptApp()`, `AddNodeApp()`, `AddViteApp()` | Model JavaScript and TypeScript workloads, from general package-script apps to Node entrypoints and Vite frontends. | | ||
| | `AddParameter()` | Define reusable config values and secrets for the resources in your model. | | ||
| | `AddPostgres()`, `AddRedis()` | Add shared infrastructure with first-class integrations and connection wiring. | |
Member
There was a problem hiding this comment.
These seem out of place. why are these 2 special?
eerhardt
reviewed
Apr 9, 2026
| ## Next steps | ||
|
|
||
| After you have Aspire orchestrating the existing app, a few natural next steps are: | ||
| At this point, you have the core workflow: describe the resources your app needs, connect the workloads that depend on them, and let Aspire run the system together during local development. From there, you can deepen the setup incrementally instead of trying to remodel the entire app at once. |
Member
There was a problem hiding this comment.
From there, you can deepen the setup incrementally instead of trying to remodel the entire app at once.
This line feels off.
eerhardt
approved these changes
Apr 9, 2026
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
addCSharpApp()parity.sl-stepslist padding so embedded tabs render correctlyValidation