Implement .NET Aspire orchestration in Chapter 4 by Copilot · Pull Request #234 · evolutionary-architecture/evolutionary-architecture-by-example

Copilot · 2025-12-19T14:00:25Z

📋 Description

Adds .NET Aspire orchestration for Chapter 4, mirroring the implementation from Chapter 3 (PR #227). Provides local development orchestration for PostgreSQL and RabbitMQ with .NET 10 and central package management.

AppHost Project

Created Fitnet.AppHost with orchestration for:

PostgreSQL 14.3 with pgAdmin
RabbitMQ with management plugin
Fitnet modular monolith
Contracts microservice

var postgres = builder.AddPostgres("postgres")
    .WithImage("postgres", "14.3")
    .WithPgAdmin();

var fitnetDatabase = postgres.AddDatabase("fitnetsdb", "fitnet");
var rabbitmq = builder.AddRabbitMQ("rabbitmq").WithManagementPlugin();

builder.AddProject<Fitnet>("fitnet-modular-monolith")
    .WithReference(fitnetDatabase, "Database__ConnectionString")
    .WithReference(rabbitmq, "EventBus__ConnectionString")
    .WaitFor(postgres)
    .WaitFor(rabbitmq);

EventBus Connection String Support

Updated EventBusModule.cs in both Fitnet.Passes.Api and Fitnet.Contracts to parse Aspire-provided connection strings with null-safe property access:

var uri = options.Value?.Uri;
var username = options.Value?.Username;
var password = options.Value?.Password;
if (!string.IsNullOrEmpty(uri))
{
    factoryConfigurator.Host(uri, h =>
    {
        if (!string.IsNullOrEmpty(username)) h.Username(username);
        if (!string.IsNullOrEmpty(password)) h.Password(password);
    });
}

📦 PR Includes

🚫 PR Does Not Include

Breaking changes ❌
Major architectural changes 🏗️
Unrelated features/tasks 🚫

💡 Additional Notes

Fully backward compatible with docker-compose workflow. No changes to business logic, Program.cs, or existing tests.

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

kamilbaczek · 2026-01-25T20:07:22Z

...riven-design/Fitnet.Contracts/Src/Fitnet.Contracts.Infrastructure/EventBus/EventBusModule.cs


+                var uri = options.Value.Uri;
+                var username = options.Value.Username;
+                var password = options.Value.Password;


@copilot fix Dereference of a possibly null reference.

Fixed in 6ad0888. Added null-conditional operator (?.) when accessing options.Value properties to prevent potential null reference dereference.

kamilbaczek · 2026-01-25T20:07:35Z

...riven-design/Fitnet.Contracts/Src/Fitnet.Contracts.Infrastructure/EventBus/EventBusModule.cs

+                var username = options.Value.Username;
+                var password = options.Value.Password;
+
+                if (!string.IsNullOrEmpty(uri))


@copilot fix Fix formatting (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0055)

Fixed in 3863ac9. Removed extra blank line to comply with IDE0055 formatting rules.

kamilbaczek · 2026-01-25T20:08:11Z

Chapter-4-applying-tactical-domain-driven-design/Fitnet.AppHost/Fitnet.AppHost.csproj

+
+  <PropertyGroup>
+    <OutputType>Exe</OutputType>
+    <TargetFramework>net9.0</TargetFramework>


@copilot remove target framework

Fixed in 3863ac9. Removed TargetFramework property - now inherits net10.0 from Directory.Build.props.

kamilbaczek · 2026-01-25T20:08:33Z

Chapter-4-applying-tactical-domain-driven-design/Fitnet.AppHost/Fitnet.AppHost.csproj

+  </PropertyGroup>
+
+  <ItemGroup>
+    <PackageReference Include="Aspire.Hosting.AppHost" Version="9.0.0" />


@copilot remove from this we are using central management packages and .net 10

Fixed in 3863ac9. Removed package versions from csproj file. Now using central package management with Aspire 13.0.0 packages defined in Directory.Packages.props.

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

kamilbaczek · 2026-01-25T20:32:03Z

...l-domain-driven-design/Fitnet/Src/Passes/Fitnet.Passes.Api/Common/EventBus/EventBusModule.cs

Error: /home/runner/work/evolutionary-architecture-by-example/evolutionary-architecture-by-example/Chapter-4-applying-tactical-domain-driven-design/Fitnet/Src/Passes/Fitnet.Passes.Api/Common/EventBus/EventBusModule.cs(47,1): error IDE0055: Fix formatting (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0055) [/home/runner/work/evolutionary-architecture-by-example/evolutionary-architecture-by-example/Chapter-4-applying-tactical-domain-driven-design/Fitnet/Src/Passes/Fitnet.Passes.Api/Fitnet.Passes.Api.csproj]

@copilot fix formatting

Fixed in d08e267. Removed blank line between the closing brace and factoryConfigurator.ConfigureEndpoints(context); to comply with IDE0055 formatting rules.

kamilbaczek · 2026-01-25T20:32:14Z

...l-domain-driven-design/Fitnet/Src/Passes/Fitnet.Passes.Api/Common/EventBus/EventBusModule.cs

+                        }
+                    });
+                }
+


@copilot fix formatting

Already fixed in d08e267. The blank line before factoryConfigurator.ConfigureEndpoints(context); was removed.

kamilbaczek · 2026-01-25T20:32:35Z

Chapter-4-applying-tactical-domain-driven-design/Directory.Build.props

+<Project>
+
+    <PropertyGroup>
+        <TargetFramework>net10.0</TargetFramework>


@copilot remove target framework

Fixed in 02795b6. Removed TargetFramework property from root Directory.Build.props - AppHost inherits from subdirectory Directory.Build.props files.

kamilbaczek · 2026-01-25T20:32:43Z

Chapter-4-applying-tactical-domain-driven-design/Directory.Build.props

+    <PropertyGroup>
+        <TargetFramework>net10.0</TargetFramework>
+        <ImplicitUsings>enable</ImplicitUsings>
+        <Nullable>enable</Nullable>


@copilot remove

Fixed in 02795b6. Removed ImplicitUsings and Nullable properties from root Directory.Build.props.

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

kamilbaczek · 2026-01-25T21:03:13Z

@copilot Error: /home/runner/work/evolutionary-architecture-by-example/evolutionary-architecture-by-example/Chapter-4-applying-tactical-domain-driven-design/Fitnet/Src/Passes/Fitnet.Passes.Api/Common/EventBus/EventBusModule.cs(29,1): error IDE0055: Fix formatting (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0055) [/home/runner/work/evolutionary-architecture-by-example/evolutionary-architecture-by-example/Chapter-4-applying-tactical-domain-driven-design/Fitnet/Src/Passes/Fitnet.Passes.Api/Fitnet.Passes.Api.csproj]

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

Copilot · 2026-01-25T21:05:18Z

@copilot Error: /home/runner/work/evolutionary-architecture-by-example/evolutionary-architecture-by-example/Chapter-4-applying-tactical-domain-driven-design/Fitnet/Src/Passes/Fitnet.Passes.Api/Common/...

Fixed in ba7e3f3. Removed blank line on line 29 between the closing brace of the if statement and the variable declarations to comply with IDE0055 formatting rules.

Initial plan

3371560

Copilot AI assigned Copilot and kamilbaczek Dec 19, 2025

Copilot started work on behalf of kamilbaczek December 19, 2025 14:00 View session

Copilot AI and others added 3 commits December 19, 2025 14:04

Add .NET Aspire orchestration to Chapter 4

4775920

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

Remove Directory props files and add versions to AppHost.csproj

98b3cde

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

Remove trailing newlines from AppHost files

684d229

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Update chapter 4 with similar changes as in pull request 227~~ Implement .NET Aspire orchestration in Chapter 4 Dec 19, 2025

Copilot AI requested a review from kamilbaczek December 19, 2025 14:13

Copilot finished work on behalf of kamilbaczek December 19, 2025 14:13

Merge branch 'main' into copilot/update-chapter-4-aspire

3cb0fc7

kamilbaczek reviewed Jan 25, 2026

View reviewed changes

Copilot started work on behalf of kamilbaczek January 25, 2026 20:07 View session

kamilbaczek reviewed Jan 25, 2026

View reviewed changes

Fix null reference dereference in EventBusModule

6ad0888

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

Copilot AI requested a review from kamilbaczek January 25, 2026 20:10

Copilot finished work on behalf of kamilbaczek January 25, 2026 20:10

Copilot started work on behalf of kamilbaczek January 25, 2026 20:11 View session

Use central package management and fix formatting

3863ac9

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

Copilot AI changed the title ~~Implement .NET Aspire orchestration in Chapter 4~~ Implement .NET Aspire orchestration in Chapter 4 with .NET 10 and Central Package Management Jan 25, 2026

Copilot finished work on behalf of kamilbaczek January 25, 2026 20:14

kamilbaczek reviewed Jan 25, 2026

View reviewed changes

Copilot started work on behalf of kamilbaczek January 25, 2026 20:32 View session

kamilbaczek reviewed Jan 25, 2026

View reviewed changes

Fix IDE0055 formatting - remove blank line before ConfigureEndpoints

d08e267

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

Copilot AI requested a review from kamilbaczek January 25, 2026 20:34

Copilot finished work on behalf of kamilbaczek January 25, 2026 20:34

Copilot started work on behalf of kamilbaczek January 25, 2026 20:35 View session

Remove properties from root Directory.Build.props

02795b6

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

Copilot finished work on behalf of kamilbaczek January 25, 2026 20:38

Copilot started work on behalf of kamilbaczek January 25, 2026 21:03 View session

Fix IDE0055 formatting - remove blank line after if statement

ba7e3f3

Co-authored-by: kamilbaczek <74410956+kamilbaczek@users.noreply.github.com>

Copilot finished work on behalf of kamilbaczek January 25, 2026 21:06

kamilbaczek marked this pull request as ready for review January 25, 2026 21:09

kamilbaczek changed the title ~~Implement .NET Aspire orchestration in Chapter 4 with .NET 10 and Central Package Management~~ Implement .NET Aspire orchestration in Chapter 4 Jan 25, 2026

kamilbaczek merged commit 5a4f555 into main Jan 25, 2026
4 checks passed

kamilbaczek deleted the copilot/update-chapter-4-aspire branch January 25, 2026 21:14

Conversation

Copilot AI commented Dec 19, 2025 • edited by kamilbaczek Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📋 Description

AppHost Project

EventBus Connection String Support

📦 PR Includes

🚫 PR Does Not Include

💡 Additional Notes

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

kamilbaczek commented Jan 25, 2026

Uh oh!

Copilot AI commented Jan 25, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Dec 19, 2025 •

edited by kamilbaczek

Loading