We are storing a lot of module related data in SessionObject. It would be good to move storage of those values to modules (adding proper resetting mechanism) and reduce load on SessionObject which holds those data in a dictionary (map) to which access needs to be synchronized.

We are doing a lot of tasks in async way, which is good but causes a lot of synchronization issues. It would be good to review that.

We should also consider improvements to the EventBus. It works pretty well, but in many places it could be replaced by Publish-Subscribe pattern which would be more efficient and better in a long term use.