ARCH-6: Adopt single aggregate-per-command mapping contract #6

New issue

Closed

opened 2026-05-06 21:14:39 -07:00 by jwilger · 1 comment

jwilger commented

2026-05-06 21:14:39 -07:00

Owner

Roadmap key: ARCH-6

Depends on: ARCH-2

Acceptance criteria

Model and architecture define mandatory command shape, router self-registration conventions, middleware ordering, and event-model fields required for strict completeness to validate command ownership.

Source

Created from .kilo/plans/1778116681638-nimble-star.md.

Roadmap key: `ARCH-6` Depends on: [ARCH-2](https://git.johnwilger.com/jwilger/anything_app/issues/2) ## Acceptance criteria - Model and architecture define mandatory command shape, router self-registration conventions, middleware ordering, and event-model fields required for strict completeness to validate command ownership. ## Source Created from `.kilo/plans/1778116681638-nimble-star.md`.

jwilger added this to the Architecture Contract milestone

2026-05-06 21:14:39 -07:00

jwilger added the

enhancement

label

2026-05-06 21:14:39 -07:00

jwilger added a new dependency

2026-05-06 21:23:37 -07:00

#2 ARCH-2: Define event model artifact schema and glossary

jwilger added a new dependency

2026-05-06 21:23:38 -07:00

#12 BASE-6: Add command-per-command dispatch scaffolding

jwilger referenced this issue

2026-05-06 23:28:13 -07:00

BASE-6: Add command-per-command dispatch scaffolding #12

jwilger referenced this issue

2026-05-06 23:35:12 -07:00

ROAD-4: Add roadmap index issue #57

jwilger commented

2026-05-07 11:11:15 -07:00

Author

Owner

Completed issue #6 on branch issue-6-single-aggregate-command-contract.

Summary:

Documented the mandatory single aggregate-per-command contract in docs/ARCHITECTURE.md and event_model/schema.md, including command module shape, aggregate module/name separation, router self-registration, middleware ordering, lifespan, and idempotency requirements.
Added a focused contract test for the strict completeness validator.
Updated the validator to reject state-change slices missing command ownership metadata, invalid aggregate pattern, non-self-registering router metadata, wrong middleware order, and missing idempotency keys for external-input commands.
Aligned bootstrap state-change YAML slices with aggregate.module, distinct aggregate names, and stable idempotency keys where required.

Verification:

npm run test:event-model-contract
npm run check:event-model
mix format --check-formatted
MIX_ENV=test mix compile --warnings-as-errors --force
MIX_ENV=prod mix compile --warnings-as-errors --force
mix test --warnings-as-errors

Skipped unavailable gates:

mix dialyzer --halt-exit-status (task not installed)
mix credo --strict (task not installed)
mix sobelow (task not installed)

Completed issue #6 on branch `issue-6-single-aggregate-command-contract`. Summary: - Documented the mandatory single aggregate-per-command contract in `docs/ARCHITECTURE.md` and `event_model/schema.md`, including command module shape, aggregate module/name separation, router self-registration, middleware ordering, lifespan, and idempotency requirements. - Added a focused contract test for the strict completeness validator. - Updated the validator to reject state-change slices missing command ownership metadata, invalid aggregate pattern, non-self-registering router metadata, wrong middleware order, and missing idempotency keys for external-input commands. - Aligned bootstrap state-change YAML slices with `aggregate.module`, distinct aggregate names, and stable idempotency keys where required. Verification: - `npm run test:event-model-contract` - `npm run check:event-model` - `mix format --check-formatted` - `MIX_ENV=test mix compile --warnings-as-errors --force` - `MIX_ENV=prod mix compile --warnings-as-errors --force` - `mix test --warnings-as-errors` Skipped unavailable gates: - `mix dialyzer --halt-exit-status` (task not installed) - `mix credo --strict` (task not installed) - `mix sobelow` (task not installed)

jwilger closed this issue

2026-05-07 11:11:20 -07:00

jwilger referenced this issue from a pull request that will close it,

2026-05-07 11:12:22 -07:00

ARCH-6: Adopt single aggregate-per-command mapping contract #65

jwilger referenced this issue from a commit

2026-05-07 11:24:47 -07:00

ARCH-6: Adopt single aggregate-per-command mapping contract (#65)