Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 4, 2025

EventStore was deprecated in #895. This PR completes the removal by deleting all associated files and references.

Changes

  • Removed packages:

    • src/CommunityToolkit.Aspire.Hosting.EventStore/
    • src/CommunityToolkit.Aspire.EventStore/
    • examples/eventstore/
  • Updated references:

    • README.md: Removed integration table entries and badge links
    • Directory.Packages.props: Removed AspNetCore.HealthChecks.EventStore.gRPC, EventStore.Client.Extensions.OpenTelemetry, EventStore.Client.Grpc.Streams
    • CommunityToolkit.Aspire.slnx: Removed project references

Total: 27 files deleted, 1,163 lines removed

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • dnceng.pkgs.visualstudio.com
    • Triggering command: dotnet build --no-restore (dns block)
  • k4kvsblobprodcus344.vsblob.vsassets.io
    • Triggering command: dotnet build --no-restore (dns block)
    • Triggering command: dotnet restore (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Remove the EventStore integrations and all associated files as that was deprecated in #895


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Remove EventStore integrations and associated files Remove deprecated EventStore integrations Nov 4, 2025
Copilot AI requested a review from aaronpowell November 4, 2025 03:55
Copilot finished work on behalf of aaronpowell November 4, 2025 03:55
@aaronpowell aaronpowell added this to the 13 milestone Nov 4, 2025
@aaronpowell aaronpowell marked this pull request as ready for review November 4, 2025 04:06
README.md Outdated
| - **Learn More**: [`Hosting.Rust`][rust-integration-docs] <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.Rust][rust-shields]][rust-nuget] <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.Rust][rust-shields-preview]][rust-nuget-preview] | A hosting integration for the Rust apps. |
| - **Learn More**: [`Hosting.Bun`][bun-integration-docs] <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.Bun][bun-shields]][bun-nuget] <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.Bun][bun-shields-preview]][bun-nuget-preview] | A hosting integration for the Bun apps. |
| - **Learn More**: [`Hosting.Python.Extensions`][python-ext-integration-docs] <br /> - Stable 📦: [![CommunityToolkit.Aspire.Python.Extensions][python-ext-shields]][python-ext-nuget] <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.Python.Extensions][python-ext-shields-preview]][python-ext-nuget-preview] | An integration that contains some additional extensions for running python applications |
| - **Learn More**: [`Hosting.EventStore`][eventstore-integration-docs] <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.EventStore][eventstore-shields]][eventstore-nuget] <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.EventStore][eventstore-shields-preview]][eventstore-nuget-preview] | An Aspire hosting integration leveraging the [EventStore](https://eventstore.com) container. |
Copy link
Contributor

Choose a reason for hiding this comment

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

I suspect changes in this file are going to conflict with the latest changes in main branch.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah it will, I'll deal with that merge conflict when everything is sync'ed together.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

Code Coverage

Package Line Rate Branch Rate Complexity Health
CommunityToolkit.Aspire.GoFeatureFlag 100% 97% 74
CommunityToolkit.Aspire.Hosting.ActiveMQ 79% 40% 100
CommunityToolkit.Aspire.Hosting.ActiveMQ.MassTransit 1% 0% 14
CommunityToolkit.Aspire.Hosting.Adminer 73% 50% 20
CommunityToolkit.Aspire.Hosting.Azure.Dapr 29% 7% 124
CommunityToolkit.Aspire.Hosting.Azure.Dapr.Redis 61% 34% 76
CommunityToolkit.Aspire.Hosting.Azure.DataApiBuilder 100% 100% 22
CommunityToolkit.Aspire.Hosting.Bun 96% 83% 28
CommunityToolkit.Aspire.Hosting.Dapr 60% 36% 836
CommunityToolkit.Aspire.Hosting.DbGate 94% 50% 18
CommunityToolkit.Aspire.Hosting.Deno 98% 85% 44
CommunityToolkit.Aspire.Hosting.Flagd 100% 100% 24
CommunityToolkit.Aspire.Hosting.GoFeatureFlag 95% 73% 28
CommunityToolkit.Aspire.Hosting.Golang 87% 70% 28
CommunityToolkit.Aspire.Hosting.Java 70% 75% 130
CommunityToolkit.Aspire.Hosting.k6 58% 12% 20
CommunityToolkit.Aspire.Hosting.Keycloak.Extensions 100% 100% 22
CommunityToolkit.Aspire.Hosting.KurrentDB 79% 75% 26
CommunityToolkit.Aspire.Hosting.LavinMQ 78% 50% 18
CommunityToolkit.Aspire.Hosting.LavinMQ.MassTransit 1% 0% 14
CommunityToolkit.Aspire.Hosting.MailPit 91% 50% 14
CommunityToolkit.Aspire.Hosting.McpInspector 89% 55% 150
CommunityToolkit.Aspire.Hosting.Meilisearch 73% 57% 50
CommunityToolkit.Aspire.Hosting.Minio 93% 75% 48
CommunityToolkit.Aspire.Hosting.MongoDB.Extensions 96% 83% 36
CommunityToolkit.Aspire.Hosting.MySql.Extensions 100% 88% 78
CommunityToolkit.Aspire.Hosting.Ngrok 52% 35% 82
CommunityToolkit.Aspire.Hosting.NodeJS.Extensions 97% 87% 168
CommunityToolkit.Aspire.Hosting.Ollama 67% 70% 174
CommunityToolkit.Aspire.Hosting.OpenTelemetryCollector 78% 61% 77
CommunityToolkit.Aspire.Hosting.PapercutSmtp 92% 50% 10
CommunityToolkit.Aspire.Hosting.PostgreSQL.Extensions 99% 89% 84
CommunityToolkit.Aspire.Hosting.Python.Extensions 39% 32% 90
CommunityToolkit.Aspire.Hosting.RavenDB 63% 49% 136
CommunityToolkit.Aspire.Hosting.Redis.Extensions 100% 71% 48
CommunityToolkit.Aspire.Hosting.Rust 94% 83% 16
CommunityToolkit.Aspire.Hosting.Solr 96% 100% 14
CommunityToolkit.Aspire.Hosting.SqlDatabaseProjects 52% 49% 206
CommunityToolkit.Aspire.Hosting.Sqlite 99% 89% 42
CommunityToolkit.Aspire.Hosting.SqlServer.Extensions 100% 87% 78
CommunityToolkit.Aspire.Hosting.SurrealDb 54% 38% 244
CommunityToolkit.Aspire.KurrentDB 94% 92% 54
CommunityToolkit.Aspire.MassTransit.RabbitMQ 100% 100% 30
CommunityToolkit.Aspire.Meilisearch 97% 92% 68
CommunityToolkit.Aspire.Microsoft.Data.Sqlite 89% 85% 52
CommunityToolkit.Aspire.Microsoft.EntityFrameworkCore.Sqlite 61% 58% 114
CommunityToolkit.Aspire.Minio.Client 73% 59% 88
CommunityToolkit.Aspire.OllamaSharp 77% 71% 132
CommunityToolkit.Aspire.RavenDB.Client 60% 53% 237
CommunityToolkit.Aspire.SurrealDb 79% 63% 78
Summary 69% (6901 / 9942) 56% (1962 / 3518) 4364

Minimum allowed line rate is 60%

@aaronpowell aaronpowell merged commit 3522e4a into aspire-13 Nov 5, 2025
105 checks passed
@aaronpowell aaronpowell deleted the copilot/remove-eventstore-integrations branch November 5, 2025 00:10
aaronpowell added a commit that referenced this pull request Nov 6, 2025
* baseline branch for the next major release

* Updating nightly

* Remove deprecated IDistributedApplicationLifecycleHook interface (#915)

* Initial plan

* Replace IDistributedApplicationLifecycleHook with event-based patterns

Co-authored-by: aaronpowell <[email protected]>

* Add tests for installer resources and update Dapr tests

Co-authored-by: aaronpowell <[email protected]>

* Refactor Dapr to use IDistributedApplicationEventingSubscriber

Co-authored-by: aaronpowell <[email protected]>

* Fixing build issue

* Updating the endpoint port

* Disabling some tests so we can move forward in other parts of the Aspire 13 update

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: aaronpowell <[email protected]>
Co-authored-by: Aaron Powell <[email protected]>

* Adjusting the version of keycloak

* Updating aspire nightly

* New nightly

* Remove npm functionality moved to Aspire 13 (#928)

* Initial plan

* Remove AddViteApp and npm-specific functionality (moved to Aspire 13)

Co-authored-by: aaronpowell <[email protected]>

* Update comments and tests to reflect npm removal

Co-authored-by: aaronpowell <[email protected]>

* Update MONOREPO.md to remove npm references

Co-authored-by: aaronpowell <[email protected]>

* Adding missing types

* Bad copilot code

* Fixing missing namespace

* Handling port forwarding

* removing more stuff that has been migrated to aspire core

* Reworking how package installing is handled

* daily update

* Adding back the annotation that is no longer moving to core

* Forgot to install packages

* adding a bunch more annotations to provide enough metadata

* Removing old demo apps

* Disabling python tests

* Removing a testing change

* Updating to the latest nightly and dealing with type name changes

* Removing type that was can repurpose from aspire core

* breaking the tests down so they are easier to read

* Removing legacy docs

* Fixing failing tests

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: aaronpowell <[email protected]>
Co-authored-by: Aaron Powell <[email protected]>

* Fixing failing test

* Remove deprecated EventStore integrations (#946)

* Initial plan

* Remove EventStore integrations and all associated files

Co-authored-by: aaronpowell <[email protected]>

* Remove EventStore project references from solution file

Co-authored-by: aaronpowell <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: aaronpowell <[email protected]>
Co-authored-by: Aaron Powell <[email protected]>
Co-authored-by: Fredi Machado <[email protected]>

* Creating a new agent for generating hosting integrations

* Creating a new agent for generating client integrations

* Expanded the hosting agent a bit

* Removing the legacy approach to checking resource health and updated the last usages of it

* Bumping go and removing pnpm setup

* maybe we need an explicit health check

* Using the new JavaScriptAppResource so we can leverage the new methods on inspector

Getting rid of some custom code but making it so we run from local by default

* Disabling tests on Windows due to modelcontextprotocol/inspector#893

---------

Co-authored-by: Copilot <[email protected]>
Co-authored-by: aaronpowell <[email protected]>
Co-authored-by: Fredi Machado <[email protected]>
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