[docs] Update Bun integration docs for first-party Aspire.Hosting.JavaScript support#1354
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the Aspire documentation for Bun hosting to reflect its move from the Community Toolkit package to the first-party Aspire.Hosting.JavaScript integration, and ensures the JavaScript hosting reference lists the Bun resource type.
Changes:
- Updates the Bun integration page to reference
Aspire.Hosting.JavaScript, adds an Aspire 13.4+ note, and includes TypeScript AppHost (addBunApp) examples alongside C#. - Updates the JavaScript hosting reference to include
BunAppResource(AddBunApp/addBunApp) with a cross-link to the Bun page.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/frontend/src/content/docs/integrations/frameworks/bun-apps.mdx | Switches Bun docs to first-party Aspire.Hosting.JavaScript, adds note about deprecation of Community Toolkit package, and adds TypeScript AppHost examples. |
| src/frontend/src/content/docs/integrations/frameworks/javascript.mdx | Adds BunAppResource to the JavaScript hosting integration resource-type list and links to the Bun integration page. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Frontend HTML artifact readyThe latest frontend build uploaded the This comment updates automatically when a new frontend build artifact is uploaded. |
|
|
CI has been retriggered multiple times and is still failing on the current PR head. This looks likely to be caused by changes in this PR rather than transient infra flakiness. |
CI Failure: Forbidden WordThe Forbidden Words check is failing due to a linting rule violation in \src/frontend/src/content/docs/integrations/frameworks/bun-apps.mdx\ (line 35):
The phrase *\�pp host* (lowercase, two words) is forbidden — it must be written as *\AppHost* (PascalCase). Please update line 35 to use \AppHost\ instead of \�pp host\ to fix this CI failure. |
|
CI has already been retriggered multiple times (latest attempt: 4) and is still failing on Check forbidden words. This appears to be caused by the current PR changes rather than a transient runner issue, so this PR likely needs an update before another rerun. |
|
CI has been retriggered several times and is still failing due to changes in this PR. This likely needs PR-content fixes before CI will pass consistently. |
|
[docs-from-code-ci-note] |
|
[docs-from-code-retry-note] CI has been retriggered multiple times and is still failing for this PR revision. This likely indicates a PR-content issue rather than a transient runner failure, so the PR changes should be adjusted to resolve the failing checks. |
|
[docs-from-code automation] repeated CI reruns |
|
[docs-from-code-automation] repeated CI failure note posted CI has been retriggered multiple times for this PR and the failures continue on the same changeset. This likely needs a PR-side fix rather than another rerun. |
|
We've retriggered CI multiple times (latest run attempt: 17), and it's still failing in Forbidden Words (Check forbidden words). The repeated failure is in the forbidden-words content check, which is typically caused by wording in this PR. Could someone update this PR to address the failing check directly? |
|
CI has been retriggered multiple times and appears to be failing due to changes in this PR. Please review the failing jobs and address the PR changes causing the failures. |
|
CI has been retriggered multiple times and is still failing on this PR. This likely requires changes in the PR content rather than another rerun. |
|
We've retriggered CI multiple times on this PR, and failures have persisted. This appears to be caused by the current PR changes rather than a transient CI issue, so this PR likely needs an update to resolve the failing checks. |
Supersedes #1153.
This replacement targets
elease/13.5 because the original
elease/13.4 base branch no longer exists on microsoft/aspire.dev.
Documents changes from microsoft/aspire#17700 (
@aspire-repo-bot[bot])Targeting
release/13.4based on the source PR milestone13.4.x.Why this PR is needed
Aspire 13.4 promotes Bun hosting to a first-party integration (
BunAppResource) in theAspire.Hosting.JavaScriptpackage. The existingintegrations/frameworks/bun-apps.mdxpage still referenced the deprecatedCommunityToolkit.Aspire.Hosting.Bunpackage. Theintegrations/frameworks/javascript.mdxresource-type list was also missingBunAppResource.What changed
bun-apps.mdx: Removed the Community Toolkit badge and Community Toolkit package reference. Updated the install command toAspire.Hosting.JavaScript. Added anAsidenote directing readers who are on Aspire < 13.4 to the Community Toolkit and those on 13.4+ to the official package. Added TypeScript AppHost (addBunApp) examples alongside the existing C# examples. Updated "See also" links to point toAspire.Hosting.JavaScriptNuGet and the JavaScript hosting reference page.javascript.mdx: AddedBunAppResource(added withAddBunApp/addBunApp) to the integration's resource-type list, with a cross-link to the Bun integration page.Files modified
src/frontend/src/content/docs/integrations/frameworks/bun-apps.mdx— updatedsrc/frontend/src/content/docs/integrations/frameworks/javascript.mdx— updated