Skip to content

docs(skills): Aggregat-Event-Publikations-Modell (Phase A) nachziehen#16

Merged
Headgent merged 1 commit into
developfrom
fix/260613_event-publication-skill-sync
Jun 13, 2026
Merged

docs(skills): Aggregat-Event-Publikations-Modell (Phase A) nachziehen#16
Headgent merged 1 commit into
developfrom
fix/260613_event-publication-skill-sync

Conversation

@Headgent

Copy link
Copy Markdown
Contributor

Update-Release via do-git-update. Zielversion: v1.0.5 (patch).

Doku-Sync der 4 Event-relevanten Skills auf das Aggregat-Event-Publikations-Modell (Phase A):

  • Aggregat-CommandHandler sammelt sein Event (addEvent($event, EventScope::Internal)), kein eager Dispatch.
  • Publikation nach Commit durch den Aufrufer / eine Process-Node.
  • platform-implementation §7, rules-testing §C, platform-usage (2 Stellen), platform-cookbook §1.
  • EventRouter + eventDispatcher()-Override bleiben unverändert (E2, dev-owned Transport).

Bezug: builder docs/prd/PRD_aggregate_event_publication.md (E1).

Aggregat-CommandHandler sammeln ihr Event (addEvent($event, EventScope::Internal))
und dispatchen NICHT mehr eager; die Publikation nach erfolgreichem Commit ist
Sache des Aufrufers / einer Process-Node. Sync auf das neue Modell:

- platform-implementation §7: addEvent collects-only + EventScope + getEvents(?EventScope)
- rules-testing §C: Event-Emission-Test auf $response->getEvents(EventScope::Internal)
  statt EventCollector am Dispatcher (Handler dispatcht nicht mehr)
- platform-usage: getEvents collected; Publikation post-Commit durch Aufrufer/Process-Node
- platform-cookbook §1: Klarstellung Handler collects only

EventRouter + eventDispatcher()-Override bleiben unveraendert (E2, dev-owned Transport).
Bezug: builder docs/prd/PRD_aggregate_event_publication.md (E1).
@Headgent Headgent merged commit 48a7287 into develop Jun 13, 2026
4 checks passed
@Headgent Headgent deleted the fix/260613_event-publication-skill-sync branch June 13, 2026 15:16
@Headgent Headgent mentioned this pull request Jun 13, 2026
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.

1 participant