share important convo with contacts

[aaravgarg]

If convo over 30 mins, send notif to suggest sharing convo with contacts

trim.95379F87-4C7C-480D-B5CF-16AF3DF6E26E.MOV

[gemini-code-assist]

Code Review

This pull request introduces a new feature to notify users about important (long) conversations and allows them to share a summary via SMS with their contacts. This includes adding contacts permissions, a new dependency for accessing contacts, a new UI for contact selection, and backend logic to trigger these notifications. The changes are extensive and well-implemented for the most part.

My main feedback is on the backend notification sending logic. The new and modified functions for sending data-only FCM messages are inefficient and have regressed in error handling, specifically in removing invalid device tokens. I've suggested refactoring these to use an existing helper function that handles batching and token cleanup correctly.

[mdmohsin7]

@aaravgarg conflicts, also why does it take so long after clicking share with 1 contact button

[aaravgarg]

@aaravgarg conflicts, also why does it take so long after clicking share with 1 contact button

in video or locally? video coz my phone sucks

[aaravgarg]

@mdmohsin7 does this look good?

[mdmohsin7]

@mdmohsin7 does this look good?

yes, pls test once deployed

[aaravgarg]

@mdmohsin7 does this look good?

yes, pls test once deployed

Yeah, please let me know when deployed.

[mdmohsin7]

Already deployed bro, try the latest testflight