Comments about AnnotatedCommandHandler

[prolic]

see: https://github.com/mrook/annotations/blob/master/src/AnnotatedCommandHandler.php#L69

As you can see here https://github.com/prooph/event-sourcing/blob/develop/src/AggregateRoot.php#L46 the constructor is already protected. So an aggregate root will only be created by static methods, not by the constructor.

[prolic]

https://github.com/mrook/annotations/blob/master/src/AnnotatedCommandHandler.php#L58-L76

I would do that within the onRouteMessage() method, there is no need to create all handlers upfront, when only a single one is used.

[mrook]

@prolic the idea here is that an aggregate root (especially when written as a POPO) could (should?) have a constructor method as a command handler.

As for the onRouteMessage() comment, my comment re caching in EventHandlerInspector applies here as well.