Skip to content

Removing Bun hosting integration - #1582

Open
aaronpowell wants to merge 2 commits into
mainfrom
remove-bun
Open

Removing Bun hosting integration#1582
aaronpowell wants to merge 2 commits into
mainfrom
remove-bun

Conversation

@aaronpowell

Copy link
Copy Markdown
Member

In Aspire 13.5.0 Bun was added to the Aspire.Hosting.JavaScript package, superseding our custom hosting integration. The NuGet package is deprecated and we won't publish one in future releases.

PR Checklist

  • Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
  • Based off latest main branch of toolkit
  • PR doesn't include merge commits (always rebase on top of our main, if needed)
  • New integration
    • Docs are written
    • Added description of major feature to project description for NuGet package (4000 total character limit, so don't push entire description over that)
  • Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Contains NO breaking changes
  • Every new API (including internal ones) has full XML docs
  • Code follows all style conventions

Other information

In Aspire 13.5.0 Bun was added to the Aspire.Hosting.JavaScript package, superseding our custom hosting integration. The NuGet package is deprecated and we won't publish one in future releases.
Copilot AI lite review requested due to automatic review settings September 9, 2026 02:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The root README still contains leftover Bun reference-style links after removing the Bun integrations table entry.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review tier: Lite
Findings: 1 Low severity

New issues introduced by this change (1)
Severity Finding
Low severity README.md — The Bun integration row was removed from the integrations table, but the reference-style links for…
What changed in this PR

Removes the deprecated CommunityToolkit.Aspire.Hosting.Bun hosting integration (and its associated tests/examples) now that Bun support is available in the core Aspire.Hosting.JavaScript package.

Changes:

  • Removed the CommunityToolkit.Aspire.Hosting.Bun source package (including generated API surface) and its test project.
  • Removed the Bun example AppHost (C# and TypeScript) and example Bun API content.
  • Updated repository entry points (solution + root README integrations table) to stop advertising/building the Bun integration.
File Description
tests/​CommunityToolkit.Aspire.Hosting.Bun.Tests/​TypeScriptAppHostTests.cs Removed TypeScript AppHost integration test for Bun.
tests/​CommunityToolkit.Aspire.Hosting.Bun.Tests/​CommunityToolkit.Aspire.Hosting.Bun.Tests.csproj Removed Bun test project definition.
tests/​CommunityToolkit.Aspire.Hosting.Bun.Tests/​AppHostTests.cs Removed Bun AppHost HTTP validation test.
tests/​CommunityToolkit.Aspire.Hosting.Bun.Tests/​AddBunAppTests.cs Removed unit tests for deprecated Bun extension APIs.
src/​CommunityToolkit.Aspire.Hosting.Bun/​README.md Removed Bun integration package README.
src/​CommunityToolkit.Aspire.Hosting.Bun/​CommunityToolkit.Aspire.Hosting.Bun.csproj Removed Bun integration package project.
src/​CommunityToolkit.Aspire.Hosting.Bun/​BunInstallerResource.cs Removed Bun installer resource type.
src/​CommunityToolkit.Aspire.Hosting.Bun/​BunAppResource.cs Removed Bun app resource type.
src/​CommunityToolkit.Aspire.Hosting.Bun/​BunAppExtensions.cs Removed Bun extension methods and defaults logic.
src/​CommunityToolkit.Aspire.Hosting.Bun/​api/​CommunityToolkit.Aspire.Hosting.Bun.cs Removed generated API file for Bun integration.
README.md Removed Bun entry from integrations table (but see comment re: leftover reference links).
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost/​Properties/​launchSettings.json Removed Bun example AppHost launch settings.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost/​Program.cs Removed Bun example AppHost program.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost/​CommunityToolkit.Aspire.Hosting.Bun.AppHost.csproj Removed Bun example AppHost project.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost/​appsettings.json Removed Bun example AppHost configuration.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost.TypeScript/​tsconfig.json Removed Bun TypeScript AppHost config.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost.TypeScript/​package.json Removed Bun TypeScript AppHost package manifest.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost.TypeScript/​package-lock.json Removed Bun TypeScript AppHost lockfile.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost.TypeScript/​eslint.config.mjs Removed Bun TypeScript AppHost lint config.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost.TypeScript/​bun-app/​package.json Removed Bun TypeScript sample app manifest.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost.TypeScript/​bun-app/​index.ts Removed Bun TypeScript sample app code.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost.TypeScript/​aspire.config.json Removed TypeScript AppHost aspire config for Bun sample.
examples/​bun/​CommunityToolkit.Aspire.Hosting.Bun.AppHost.TypeScript/​apphost.mts Removed Bun TypeScript AppHost definition.
examples/​bun/​api/​tsconfig.json Removed Bun API example TypeScript config.
examples/​bun/​api/​README.md Removed Bun API example README.
examples/​bun/​api/​package.json Removed Bun API example package manifest.
examples/​bun/​api/​index.ts Removed Bun API example server.
examples/​bun/​api/​.gitignore Removed Bun API example gitignore.
CommunityToolkit.Aspire.slnx Removed Bun projects from the solution.
Files not reviewed (1)
  • examples/bun/CommunityToolkit.Aspire.Hosting.Bun.AppHost.TypeScript/package-lock.json: Generated file

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread README.md

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The change set consistently removes the deprecated integration and associated references without leaving remaining in-repo usages.

Review tier: Lite
Findings: 1 Low severity

New issues introduced by this change (1)
Severity Finding
Low severity README.md — The repo still includes a “New Aspire Hosting Integration” chatmode template that uses Bun-specific…
Issues resolved since last review (1)
Severity Finding
Low severity README.md — The Bun integration row was removed from the integrations table, but the reference-style links for… View resolved comment
Files not reviewed (1)
  • examples/bun/CommunityToolkit.Aspire.Hosting.Bun.AppHost.TypeScript/package-lock.json: Generated file

Comment thread README.md
Comment on lines 24 to 28
| - **Learn More**: [`Hosting.Azure.Dapr.Redis`](src/CommunityToolkit.Aspire.Hosting.Azure.Dapr.Redis/README.md) <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.Azure.Dapr.Redis](https://img.shields.io/nuget/v/CommunityToolkit.Aspire.Hosting.Azure.Dapr.Redis)](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Azure.Dapr.Redis/) <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.Azure.Dapr.Redis](https://img.shields.io/nuget/vpre/CommunityToolkit.Aspire.Hosting.Azure.Dapr.Redis?label=nuget%20(preview))](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Azure.Dapr.Redis/absoluteLatest) | An Aspire hosting integration for configuring Azure Cache for Redis as a Dapr state store or pub/sub component in your AppHost. |
| - **Learn More**: [`Hosting.Azure.DataApiBuilder`](src/CommunityToolkit.Aspire.Hosting.Azure.DataApiBuilder/README.md) <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.Azure.DataApiBuilder](https://img.shields.io/nuget/v/CommunityToolkit.Aspire.Hosting.Azure.DataApiBuilder)](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Azure.DataApiBuilder/) <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.Azure.DataApiBuilder](https://img.shields.io/nuget/vpre/CommunityToolkit.Aspire.Hosting.Azure.DataApiBuilder?label=nuget%20(preview))](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Azure.DataApiBuilder/absoluteLatest) | An Aspire component leveraging the Data API Builder container. |
| - **Learn More**: [`Hosting.Azure.Extensions`](src/CommunityToolkit.Aspire.Hosting.Azure.Extensions/README.md) <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.Azure.Extensions](https://img.shields.io/nuget/v/CommunityToolkit.Aspire.Hosting.Azure.Extensions)](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Azure.Extensions/) <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.Azure.Extensions](https://img.shields.io/nuget/vpre/CommunityToolkit.Aspire.Hosting.Azure.Extensions?label=nuget%20(preview))](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Azure.Extensions/absoluteLatest) | Azure extensions support for Aspire. |
| - **Learn More**: [`Hosting.Bitwarden.SecretManager`](src/CommunityToolkit.Aspire.Hosting.Bitwarden.SecretManager/README.md) <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.Bitwarden.SecretManager](https://img.shields.io/nuget/v/CommunityToolkit.Aspire.Hosting.Bitwarden.SecretManager)](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Bitwarden.SecretManager/) <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.Bitwarden.SecretManager](https://img.shields.io/nuget/vpre/CommunityToolkit.Aspire.Hosting.Bitwarden.SecretManager?label=nuget%20(preview))](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Bitwarden.SecretManager/absoluteLatest) | An Aspire hosting integration for Bitwarden Secrets Manager. |
| - **Learn More**: [`Hosting.Bun`](src/CommunityToolkit.Aspire.Hosting.Bun/README.md) <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.Bun](https://img.shields.io/nuget/v/CommunityToolkit.Aspire.Hosting.Bun)](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Bun/) <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.Bun](https://img.shields.io/nuget/vpre/CommunityToolkit.Aspire.Hosting.Bun?label=nuget%20(preview))](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Bun/absoluteLatest) | ⚠️ **Deprecated** — An Aspire integration for hosting Bun apps. DEPRECATED: Use Aspire.Hosting.JavaScript and builder.AddBunApp(...) instead. This package will be removed in a future release. |
| - **Learn More**: [`Hosting.Dapr`](src/CommunityToolkit.Aspire.Hosting.Dapr/README.md) <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.Dapr](https://img.shields.io/nuget/v/CommunityToolkit.Aspire.Hosting.Dapr)](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Dapr/) <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.Dapr](https://img.shields.io/nuget/vpre/CommunityToolkit.Aspire.Hosting.Dapr?label=nuget%20(preview))](https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.Dapr/absoluteLatest) | Dapr support for Aspire. |
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants