refactor(ffi): remove two fields in EventTimelineItem

[bnjbvr]

• is_local is the opposite of is_remote. I hope all foreign languages have a ! operator 🤡
• the two lazy providers can be unified into a single one, avoiding a bit of boilerplate.

Split from #4100.

[jmartinesp]

is_local is the opposite of is_remote. I hope all foreign languages have a ! operator 🤡

This will do!

val isLocal: Boolean = is_remote.toInt() - 1 == 0

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.69%. Comparing base (85682ac) to head (b07a634).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4110      +/-   ##
==========================================
- Coverage   84.71%   84.69%   -0.03%     
==========================================
  Files         269      269              
  Lines       28816    28816              
==========================================
- Hits        24411    24405       -6     
- Misses       4405     4411       +6     

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

[jmartinesp]

LGTM, thanks!

I liked the previous *Provider way of splitting responsibilities better, but we'll probably add wrappers for them once they're in Kotlin/Swift anyway, so we can split them there if we want to, so it's no big deal.