[Bug]: Historical events subscription ignoring limit and keys clause

[rsodre]

Dojo.js Version

1.3.0

What happened?

The initial amount of events returned by a historical events subscription using sdk.subscribeEventQuery() does not respect the withLimit() parameter.

What is the expected behavior?

We should be to limit the initial amount of events the subscription returns

Steps to reproduce

I used this query, on https://api.cartridge.gg/x/pistols-sepolia/torii

const query = new QueryBuilder()
  .withClause(
    new PistolsClauseBuilder().keys(
      ["pistols-PlayerActivity"],
      [],
    ).build()
  )
  .withEntityModels([
    "pistols-PlayerActivity",
  ])
  .withLimit(100)

Result (771 events):

Another query, with a keys clause, returns ALL historical events, including the ones that do match the key or models:

new QueryBuilder()
  .withClause(
    new ClauseBuilder().keys(
      ["pistols-LordsReleaseEvent"],
      [season_table_id]
    ).build()
  )
  .withEntityModels(
    ["pistols-LordsReleaseEvent"]
  )
  .includeHashedKeys()

Trace

No response

Is there an existing issue for this?

• I have searched the existing issues and verified no issue exits for this problem.

[rsodre]

Edited adding another case where a keys clause is not being respected. All entities, including the ones not including the query model, are being returned.