I would appreciate this fix, if there is nothing against it.

It is changing the behaviour, so to keep our Semantic Versioning promise we cannot release it in 2.x series. If that's needed in your current project, I'd suggest patching this functionality specifically in the application.

Adding version check, either in runtime or a as a unit test would keep everyone alert when upgrading RES to see if the patch applies (it should, we're going to keep SemVer promise).

raise "check RubyEventStore::Projection#read_scope implementation drift and bump version check" unless RubyEventStore::VERSION == "2.9.0"

module AsOfReadScope
  def read_scope(*, **)
    super.as_of
  end
end

RubyEventStore::Projection.prepend(AsOfReadScope)

I don't know if we really need time_sort_by=nil

There's a guaranteed monotonicity by the means of sequence numbers in the database table. It is mostly the same as the monotonicity of created_at timestamp.

However sorting by timestamp is less accurate when:

• timestamp resolution is low (i.e. seconds)
• multiple events share same timestamp, when appended in batches

I'd still keep sorting by ids and timestamps separate.

Makes sense and is another good reason for the redesign of projections.

Thanks @pawelpacana for the monkey patch, it is working as expected.