Develop

.coderabbit.yaml

@@ -157,17 +157,18 @@ reviews:
       - name: check conventions
         enabled: true
         instructions: >
-          Review the changed .cs files against the conventions in CLAUDE.md.
-          Check specifically: (1) value objects use const int MaxLength not static readonly,
-          (2) domain event handlers inject only IEventBus and source payload from the
-          domain event (no repository reload), (3) every route parent ID is passed
-          through to the command, (4) IDOR guards return opaque NotFound errors.
+          Review the changed .cs files for conventions not covered by pre-merge checks.
+          Check: (1) aggregates expose behaviour through methods only — no public setters
+          outside the aggregate root, (2) domain errors use Error.NotFound / Error.Problem /
+          Error.Conflict / Error.Failure from Eternelle.Common.Domain (never raw exceptions),
+          (3) handlers return Result or Result<T> — no throw for business rule violations,
+          (4) cross-module references use plain Guid, never a value object from another module.
           Post a concise list of any violations found; if none, confirm compliance.
 
   pre_merge_checks:
     override_requested_reviewers_only: false
     docstrings:
-      mode: off
+      mode: warning
     title:
       mode: warning
       requirements: "Title should reference the Linear ticket (e.g. Eternelle-123) and be concise."

.gitignore

@@ -54,3 +54,6 @@ appsettings.local.json
 /create-issues-guest-photos-retry.sh
 /create-issues-guest-photos.sh
 /create-issues.sh
+/docs/superpowers
+/.claude/settings.local.json
+/.files/eternelle-realm.json

CLAUDE.md

@@ -1,8 +1,9 @@
 # CLAUDE.md — Eternelle
 
 Modular monolith backend in **.NET 10**, Clean Architecture + full DDD.
-Database design: `docs/database-design.md`.
+System architecture: `docs/architecture.md`.
 Module conventions (authoritative): `docs/module-conventions.md`.
+Per-module reference docs: `docs/modules/` (Weddings, Users, Catalog, Tenancy).
 
 ## Reference Implementations (Milan Jovanovic)
 
@@ -36,7 +37,7 @@ src/
         └── Eternelle.Modules.Weddings.IntegrationEvents/   ← integration event classes only
 ```
 
-Future modules follow the same **five-project** structure: **Identity → Tenancy → RSVP → Guestbook → Catalog**.
+Future modules follow the same **five-project** structure: **Users → Catalog → Tenancy → RSVP → Guestbook**. Users, Catalog, and Tenancy are complete.
 
 ---
 
@@ -45,11 +46,11 @@ Future modules follow the same **five-project** structure: **Identity → Tenanc
 | Module    | Domain | Application | Infrastructure | Presentation | IntegrationEvents |
 |-----------|--------|-------------|----------------|--------------|-------------------|
 | Weddings  | ✅      | ✅           | ✅              | ✅            | ✅                 |
-| Identity  | ❌      | ❌           | ❌              | ❌            | ❌                 |
-| Tenancy   | ❌      | ❌           | ❌              | ❌            | ❌                 |
+| Users     | ✅      | ✅           | ✅              | ✅            | ✅                 |
+| Catalog   | ✅      | ✅           | ✅              | ✅            | ✅                 |
+| Tenancy   | ✅      | ✅           | ✅              | ✅            | ✅                 |
 | RSVP      | ❌      | ❌           | ❌              | ❌            | ❌                 |
 | Guestbook | ❌      | ❌           | ❌              | ❌            | ❌                 |
-| Catalog   | ❌      | ❌           | ❌              | ❌            | ❌                 |
 
 ---
 
@@ -69,10 +70,10 @@ Future modules follow the same **five-project** structure: **Identity → Tenanc
 **Aggregate roots** extend `Eternelle.Common.Domain.Entity`.
 
 ```csharp
-// Strongly-typed ID — always a readonly record struct
+// Strongly-typed ID — always a readonly record struct, UUIDv7 via the BCL
 public readonly record struct WeddingId(Guid Value)
 {
-    public static WeddingId New() => new(Guid.NewGuid());
+    public static WeddingId New() => new(Guid.CreateVersion7());
     public static WeddingId Empty => new(Guid.Empty);
 }
 
@@ -269,14 +270,10 @@ internal sealed class CreateWeddingEndpoint : IEndpoint
 
 ---
 
-## Next Module: Identity
+## Next Module: RSVP
 
-Module name: **`Eternelle.Modules.Identity`** — not IAM.  
-Schema constant: `public static class Schemas { public const string Identity = "identity"; }`
-
-Rationale: matches `.NET` ecosystem convention (ASP.NET Core Identity), correct scope (auth + user profile), avoids acronym in namespace, already referenced as the example in `docs/module-conventions.md`.
-
-Cross-module data flow: `WeddingCreatedIntegrationEvent` → Identity handler stores a local copy of `WeddingId`/`TenantId`/`WeddingDate` in the `identity` schema. Identity never queries the `wedding` schema directly.
+Module name: **`Eternelle.Modules.Rsvp`**  
+Schema constant: `public static class Schemas { public const string Rsvp = "rsvp"; }`
 
 ---
 

CONTRIBUTING.md

@@ -6,26 +6,30 @@ This document describes the architecture rules, coding conventions, and patterns
 
 ## Layer rules
 
-Each module has four layers. The dependency rule is strict and enforced by project references.
+Each module is a **five-project** vertical slice: `Domain`, `Application`, `Infrastructure`, `Presentation`, and `IntegrationEvents`. The dependency rule is strict and enforced by project references — only a module's `IntegrationEvents` project may be referenced from outside the module. See [`docs/module-conventions.md`](docs/module-conventions.md) for the full reference graph.
 
 ### Domain
 
 **Zero dependencies on Application, Infrastructure, or any framework** (no EF Core, MediatR, Dapper, ASP.NET).
 
 - Aggregates expose behaviour through methods only — no public setters outside the aggregate root.
 - Owned entities (e.g. `EntourageMember`, `Partner`) are created and mutated exclusively through their aggregate root. Their factory and mutation methods are `internal`.
-- Value objects are `sealed record` or `sealed class` with a private constructor and a `static Create()` factory returning `Result<T>`.
-- String length limits are defined as `public static readonly int` constants on the owning type, not as magic numbers:
+- Value objects are `sealed record` with a private constructor, a `static Create()` factory returning `Result<T>`, an `internal static FromPersistence()` (trusts the DB, skips validation), and a sibling `{Vo}Errors` class.
+- String length limits are defined as a `public const int MaxLength` on the value object, not as magic numbers:
+
   ```csharp
-  // On EntourageMember:
-  public static readonly int MaxNameLength = 150;
-  public static readonly int MaxRoleLength = 100;
+  // On PersonName:
+  public const int MaxLength = 150;
   ```
+
+  `const` (not `static readonly`) is correct here: this is a monorepo where every project compiles from source, so there is no cross-assembly binary-inlining concern.
 - Domain errors use the `Error` type from `Eternelle.Common.Domain`:
+
   ```csharp
   public static Error NotFound(WeddingId id) =>
       Error.NotFound("Weddings.NotFound", $"Wedding {id.Value} was not found");
   ```
+
 - Domain events are raised via `Entity.Raise()` and are plain records implementing `IDomainEvent`.
 
 ### Application
@@ -35,21 +39,26 @@ Each module has four layers. The dependency rule is strict and enforced by proje
 - Commands implement `ICommand` (void) or `ICommand<TResponse>`.
 - Queries implement `IQuery<TResponse>`.
 - All handlers are `internal sealed class`.
-- Every command must have a matching `AbstractValidator<TCommand>`. `MaximumLength` rules **must** reference domain constants:
+- Every command must have a matching `AbstractValidator<TCommand>`. `MaximumLength` rules **must** reference the value object's `MaxLength` constant (the single source of truth):
+
   ```csharp
   RuleFor(c => c.Name)
       .NotEmpty()
-      .MaximumLength(EntourageMember.MaxNameLength);
+      .MaximumLength(PersonName.MaxLength);
   ```
+
 - Nullable fields are validated conditionally:
+
   ```csharp
   RuleFor(c => c.Subtitle)
-      .MaximumLength(EntourageGroup.MaxSubtitleLength)
+      .MaximumLength(GroupSubtitle.MaxLength)
       .When(c => c.Subtitle is not null);
   ```
+
 - **Query handlers use Dapper** via `IDbConnectionFactory` with `CommandDefinition` (always pass `cancellationToken`). Never use EF Core in a query handler.
 - **Command handlers use domain repository interfaces** (`IWeddingRepository`, etc.) and `IUnitOfWork`. Call `entourageGroupRepository.Update(group)` after mutating owned entities, then `await unitOfWork.SaveChangesAsync(cancellationToken)`.
 - Reorder handlers **must** validate list completeness before iterating using a three-condition HashSet check:
+
   ```csharp
   HashSet<Guid> providedIds = [.. command.StoryMomentIds];
   if (command.StoryMomentIds.Count != providedIds.Count ||   // duplicate check
@@ -65,15 +74,20 @@ Each module has four layers. The dependency rule is strict and enforced by proje
 - EF Core entity configurations go in `IEntityTypeConfiguration<T>` classes, one per aggregate root.
 - `UseSnakeCaseNamingConvention()` is active. Use explicit `HasColumnName()` only when the snake_case default is wrong.
 - All strongly-typed ID value objects must have a `ValueConverter` registered globally in `ConfigureConventions`, not per-property:
+
   ```csharp
   builder.Properties<WeddingId>().HaveConversion<WeddingIdConverter>();
   ```
+
 - String length constants from the domain must be referenced in `HasMaxLength()` — no magic numbers:
+
   ```csharp
-  .HasMaxLength(EntourageMember.MaxNameLength)
+  .HasMaxLength(PersonName.MaxLength)
   ```
+
+- Enums are persisted as integers via `HasConversion<int>()` (with a `CHECK` or unique constraint where a closed set must be enforced at the DB).
 - Repositories are `internal sealed class` implementing domain interfaces.
-- All tables live in the `wedding` schema. Declare it via `modelBuilder.HasDefaultSchema(Schemas.Weddings)`.
+- Each module declares its own schema via `modelBuilder.HasDefaultSchema(Schemas.{Module})` (`wedding`, `users`, `catalog`).
 
 ### Presentation
 
@@ -105,16 +119,17 @@ if (result.IsFailure) return result;
 
 ## ID types
 
-Every aggregate and owned entity uses a strongly-typed ID record:
+Every aggregate and owned entity uses a strongly-typed ID `readonly record struct`:
 
 ```csharp
-public record WeddingId(Guid Value)
+public readonly record struct WeddingId(Guid Value)
 {
-    public static WeddingId New() => new(Uuid7.NewUuid7().ToGuid());
+    public static WeddingId New() => new(Guid.CreateVersion7());
+    public override string ToString() => Value.ToString();
 }
 ```
 
-IDs use **UUIDv7** (time-ordered) via `SourceFlow.Stores.EntityFramework`. This keeps database index locality while remaining globally unique.
+IDs use **UUIDv7** (time-ordered) via the BCL `Guid.CreateVersion7()` (.NET 9+). This keeps database index locality while remaining globally unique. A `ValueConverter` for each ID type is registered globally in `ConfigureConventions`.
 
 ---
 
@@ -209,7 +224,7 @@ migrationBuilder.Sql("""
 - All handler classes are `internal sealed`.
 - Command and query records are `public sealed record`.
 - Endpoint classes are `internal sealed` implementing `IEndpoint`.
-- String constants on domain types use `public static readonly int`, not `const int`, because `const` fields are inlined at compile time and can cause issues across assembly boundaries.
+- String length limits on value objects use `public const int MaxLength`. `const` is correct in this monorepo (all projects compile from source); `static readonly` would only be needed if domain types shipped as binary NuGet packages to independently-deployed consumers.
 - Nullable reference types are enabled project-wide. Never suppress the nullable warning without a comment explaining why.
 - `using` directives are auto-managed by the IDE (file-scoped namespaces, no redundant usings).
 - Diagnostic suppressions (`NoWarn`, `#pragma warning disable`) go in `.editorconfig`, not in `.csproj` files or inline in source.

Directory.Packages.props

@@ -17,7 +17,7 @@
     <PackageVersion Include="MassTransit" Version="9.1.0" />
     <PackageVersion Include="MassTransit.Redis" Version="9.1.0" />
     <PackageVersion Include="MediatR" Version="14.1.0" />
-    <PackageVersion Include="Microsoft.AspNetCore.Authentication.JwtBearer" Version="10.0.7" />
+    <PackageVersion Include="Microsoft.AspNetCore.Authentication.JwtBearer" Version="10.0.8" />
     <PackageVersion Include="Microsoft.AspNetCore.Http" Version="2.3.9" />
     <PackageVersion Include="Microsoft.AspNetCore.OpenApi" Version="10.0.7" />
     <PackageVersion Include="Microsoft.EntityFrameworkCore.Tools" Version="10.0.4" />

Eternelle.slnx

@@ -1,4 +1,34 @@
 <Solution>
+  <Folder Name="/docs/">
+    <File Path="docs/architecture.md" />
+    <File Path="docs/feature-gift-contributions.md" />
+    <File Path="docs/feature-guest-photo-upload.md" />
+    <File Path="docs/module-conventions.md" />
+  </Folder>
+  <Folder Name="/docs/modules/">
+    <File Path="docs/modules/album.md" />
+    <File Path="docs/modules/catalog.md" />
+    <File Path="docs/modules/README.md" />
+    <File Path="docs/modules/rsvp.md" />
+    <File Path="docs/modules/users.md" />
+    <File Path="docs/modules/weddings.md" />
+  </Folder>
+  <Folder Name="/docs/superpowers/" />
+  <Folder Name="/docs/superpowers/plans/">
+    <File Path="docs/superpowers/plans/2026-05-20-users-module.md" />
+    <File Path="docs/superpowers/plans/2026-05-27-catalog-module.md" />
+    <File Path="docs/superpowers/plans/2026-05-28-tenancy-module.md" />
+    <File Path="docs/superpowers/plans/2026-05-29-rsvp-module.md" />
+    <File Path="docs/superpowers/plans/2026-05-30-album-module.md" />
+  </Folder>
+  <Folder Name="/docs/superpowers/specs/">
+    <File Path="docs/superpowers/specs/2026-05-20-users-module-design.md" />
+    <File Path="docs/superpowers/specs/2026-05-27-catalog-module-design.md" />
+    <File Path="docs/superpowers/specs/2026-05-28-tenancy-module-design.md" />
+    <File Path="docs/superpowers/specs/2026-05-29-rsvp-module-design.md" />
+    <File Path="docs/superpowers/specs/2026-05-30-album-module-design.md" />
+    <File Path="docs/superpowers/specs/2026-05-30-guestbook-module-design.md" />
+  </Folder>
   <Folder Name="/Solution Items/">
     <File Path=".editorconfig" />
     <File Path="Directory.Build.props" />
@@ -15,7 +45,42 @@
     <Project Path="src/Common/Eternelle.Common.Presentation/Eternelle.Common.Presentation.csproj" Id="54cfc49d-e0bb-44a0-84ac-ee35515c1b5e" />
   </Folder>
   <Folder Name="/src/Modules/" />
-  <Folder Name="/src/Modules/Wedding/">
+  <Folder Name="/src/Modules/Album/">
+    <Project Path="src/Modules/Album/Eternelle.Modules.Album.Application/Eternelle.Modules.Album.Application.csproj" />
+    <Project Path="src/Modules/Album/Eternelle.Modules.Album.Domain/Eternelle.Modules.Album.Domain.csproj" />
+    <Project Path="src/Modules/Album/Eternelle.Modules.Album.Infrastructure/Eternelle.Modules.Album.Infrastructure.csproj" />
+    <Project Path="src/Modules/Album/Eternelle.Modules.Album.IntegrationEvents/Eternelle.Modules.Album.IntegrationEvents.csproj" />
+    <Project Path="src/Modules/Album/Eternelle.Modules.Album.Presentation/Eternelle.Modules.Album.Presentation.csproj" />
+  </Folder>
+  <Folder Name="/src/Modules/Catalog/">
+    <Project Path="src/Modules/Catalog/Eternelle.Modules.Catalog.Application/Eternelle.Modules.Catalog.Application.csproj" />
+    <Project Path="src/Modules/Catalog/Eternelle.Modules.Catalog.Domain/Eternelle.Modules.Catalog.Domain.csproj" />
+    <Project Path="src/Modules/Catalog/Eternelle.Modules.Catalog.Infrastructure/Eternelle.Modules.Catalog.Infrastructure.csproj" />
+    <Project Path="src/Modules/Catalog/Eternelle.Modules.Catalog.IntegrationEvents/Eternelle.Modules.Catalog.IntegrationEvents.csproj" />
+    <Project Path="src/Modules/Catalog/Eternelle.Modules.Catalog.Presentation/Eternelle.Modules.Catalog.Presentation.csproj" />
+  </Folder>
+  <Folder Name="/src/Modules/Rsvp/">
+    <Project Path="src/Modules/Rsvp/Eternelle.Modules.Rsvp.Application/Eternelle.Modules.Rsvp.Application.csproj" Id="584d8710-9449-4431-9359-59ec784eb276" />
+    <Project Path="src/Modules/Rsvp/Eternelle.Modules.Rsvp.Domain/Eternelle.Modules.Rsvp.Domain.csproj" Id="73e928a0-583f-4b26-93de-d541914305d4" />
+    <Project Path="src/Modules/Rsvp/Eternelle.Modules.Rsvp.Infrastructure/Eternelle.Modules.Rsvp.Infrastructure.csproj" Id="2c5fbf2f-a4ee-46ed-beb7-d76408d75222" />
+    <Project Path="src/Modules/Rsvp/Eternelle.Modules.Rsvp.IntegrationEvents/Eternelle.Modules.Rsvp.IntegrationEvents.csproj" Id="55dc7f38-3a59-4077-8249-46530f103444" />
+    <Project Path="src/Modules/Rsvp/Eternelle.Modules.Rsvp.Presentation/Eternelle.Modules.Rsvp.Presentation.csproj" Id="4f8b50a7-2c4a-46eb-b83a-63007eab9e69" />
+  </Folder>
+  <Folder Name="/src/Modules/Tenancy/">
+    <Project Path="src/Modules/Tenancy/Eternelle.Modules.Tenancy.Application/Eternelle.Modules.Tenancy.Application.csproj" />
+    <Project Path="src/Modules/Tenancy/Eternelle.Modules.Tenancy.Domain/Eternelle.Modules.Tenancy.Domain.csproj" />
+    <Project Path="src/Modules/Tenancy/Eternelle.Modules.Tenancy.Infrastructure/Eternelle.Modules.Tenancy.Infrastructure.csproj" />
+    <Project Path="src/Modules/Tenancy/Eternelle.Modules.Tenancy.IntegrationEvents/Eternelle.Modules.Tenancy.IntegrationEvents.csproj" />
+    <Project Path="src/Modules/Tenancy/Eternelle.Modules.Tenancy.Presentation/Eternelle.Modules.Tenancy.Presentation.csproj" />
+  </Folder>
+  <Folder Name="/src/Modules/Users/">
+    <Project Path="src/Modules/Users/Eternelle.Modules.Users.Application/Eternelle.Modules.Users.Application.csproj" Id="27530601-1743-4d73-80d3-bddbf28d8eb2" />
+    <Project Path="src/Modules/Users/Eternelle.Modules.Users.Domain/Eternelle.Modules.Users.Domain.csproj" Id="823bb7bc-2382-4588-a404-73ae27f49b02" />
+    <Project Path="src/Modules/Users/Eternelle.Modules.Users.Infrastructure/Eternelle.Modules.Users.Infrastructure.csproj" Id="290675fd-c75d-44e0-8cdc-52999a9ead97" />
+    <Project Path="src/Modules/Users/Eternelle.Modules.Users.IntegrationEvents/Eternelle.Modules.Users.IntegrationEvents.csproj" Id="f4f38cd7-8af7-44a6-a6aa-4b57bb9dc00d" />
+    <Project Path="src/Modules/Users/Eternelle.Modules.Users.Presentation/Eternelle.Modules.Users.Presentation.csproj" Id="d6dde47a-faab-4b24-9738-6fe5d9e71738" />
+  </Folder>
+  <Folder Name="/src/Modules/Weddings/">
     <Project Path="src/Modules/Weddings/Eternelle.Modules.Weddings.Application/Eternelle.Modules.Weddings.Application.csproj" Id="a6f4ff32-35b6-40bf-a034-4aff406678ea" />
     <Project Path="src/Modules/Weddings/Eternelle.Modules.Weddings.Domain/Eternelle.Modules.Weddings.Domain.csproj" Id="8ca34ec3-032b-4439-bc9f-318df0f9e87a" />
     <Project Path="src/Modules/Weddings/Eternelle.Modules.Weddings.Infrastructure/Eternelle.Modules.Weddings.Infrastructure.csproj" Id="f7f463ef-3373-4063-9909-487fb52a34e0" />