feat(threads): filter out notifications from threads we haven't subscribed to

[bnjbvr]

Based on #5455, this makes it possible to filter out notifications for a thread we haven't subscribed to.

The rules are the following:

• if the thread is subscribed to: keep the notification
• if the thread has no subscription status:
  • if the new event would cause a subscription: keep the notification
  • if the new event wouldn't cause a subscription: dismiss the notification
• if the thread is unsubscribed from: dismiss the notification

Part of #4869.

[codecov]

Codecov Report

❌ Patch coverage is 88.18898% with 30 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.90%. Comparing base (1be8b42) to head (8f1177f).
⚠️ Report is 52 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
crates/matrix-sdk-ui/src/notification_client.rs	82.14%	8 Missing and 7 partials ⚠️
crates/matrix-sdk-sqlite/src/state_store.rs	77.55%	2 Missing and 9 partials ⚠️
crates/matrix-sdk/src/room/mod.rs	88.46%	0 Missing and 3 partials ⚠️
crates/matrix-sdk-base/src/store/mod.rs	93.33%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5461      +/-   ##
==========================================
+ Coverage   88.89%   88.90%   +0.01%     
==========================================
  Files         333      333              
  Lines       92121    92325     +204     
  Branches    92121    92325     +204     
==========================================
+ Hits        81893    82084     +191     
+ Misses       6393     6390       -3     
- Partials     3835     3851      +16     

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

[codspeed-hq]

CodSpeed Performance Report

Merging #5461 will not alter performance

_{Comparing bnjbvr/client-mode-thread-subs (8f1177f) with main (8091094)}

Summary

✅ 6 untouched benchmarks

[bnjbvr]

Closing: we don't need this, since the evaluation of push rules does it for us (duh! but now I know better about push rules).

The only remaining interesting things have been / will be extracted to other PRs:

• refactor(notification client): misc simplifications #5494 for the simplifications spotted during this work
• the boolean for filtering out unsubscribed threads will go in refactor(threads): add support for MSC4306 latest changes and push rules #5483