Codecov Report

❌ Patch coverage is 89.75410% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.38%. Comparing base (75228ab) to head (0ea6538).
⚠️ Report is 16 commits behind head on master.

@@            Coverage Diff             @@
##           master     #352      +/-   ##
==========================================
+ Coverage   71.68%   72.38%   +0.70%     
==========================================
  Files         142      144       +2     
  Lines        5301     5545     +244     
==========================================
+ Hits         3800     4014     +214     
- Misses       1501     1531      +30     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

High-level explanation

This PR introduces a coverage tracking system that records which time ranges have been successfully fetched from each relay for a given filter.

Queries and streams

The coverage is updated when NDK receive EOSE so it's for queries and streams.
Should we update the coverage on stream close / ndk destroy ? and / or during the stream ?

Optional usecase

Yes it's optional, the dev can use it or not.
Should we add an option to enable auto coverage on queries / streams ?

It's the first brick that we need to implement the sync usecase or to do more efficient queries and streams.

DB structure

Yes, it's independent of events.

Changing environments (new events in a previous query)

The coverage tracks what we asked for, not what exists.
There is no perfect solution in a decentralized network, the coverage usecase is here to get a picture of what time range we already fetched. And so if we want we can not refecth these ranges.
Maybe we need to save the fetched date for each time range ?

thx for explaining!
Could this be integrated into our existing caching mechanism? To me, this sounds like a smarter cache.
Otherwise I am for merging (maybe mark it as @experimental) as long as it is optional. We can then better test it in realworld scenarios.

• add experimental tag
• better naming for coverage usecase
• make it optional in Ndk config