search

snatalenko / node-cqrs

CQRS backbone with event sourcing for Node.js
https://www.node-cqrs.org
MIT License
74 stars 22 forks source link

Status of ts branch #20

Closed TomKaltz closed 1 year ago

TomKaltz commented 1 year ago

Hi can you let us know the status of the “ts” branch?

snatalenko commented 1 year ago

Hi @TomKaltz, After migrating to TypeScript and making a few minor changes I noticed that there're concurrency-related issues in the events distribution mechanism. Basically, same event can be delivered to an aggregate twice - thru the eventStore.publish and thru eventStore.getAggregateEvents. You can see that by running the integration tests - they're failing on the ts branch. Same issue exist on the main branch, but it's more difficult to reproduce. That's the only one I want to fix before merging the ts into master.