Maciej Bratek

ioutil.TempFile adds a random int as suffix, which confuses ImageMagick. Rename the result file so that the file ends with the extension. Also, cleanup with deferred os.Remove after you're done...

Idea: Separate `eventstore/${driver.go}` into `eventstore/${driver}` packages

2

If there's more drivers, then perhaps ```go import "esja/pkg/eventstore/sql" sql.NewStore[MyAggregate](...) ``` would be better than ```go import "esja/pkg/eventstore" eventstore.NewSQLStore[MyAggregate](...) ``` Besides, we could have separate `go.mod`s for those drivers, so...

https://github.com/ThreeDotsLabs/esja/pull/1#discussion_r1040072232 We could validate that the aggregate versions connected to the events check out. On the other hand, maybe we should trust the event store even if the versions don't...