fix: resolve LateInitializationError for userAddedSubscription

[4555jan]

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Changes made:

• it just a small change the error causeing keyword late just removed it
• Removed fromuserAddedSubscription` field declaration
• fields are now properly nullable and use null-safe operators (?.)

Fixes #560

Type of change

• Bug fix (non-breaking CHANGE which fixes an issue)

How Has This Been Tested?

Tested the pairing flow:

1. Started quick match/choose partner
2. Canceled the request before pairing
3. Verified no crash occurs
4. Confirmed the app returns to home screen properly

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules
• I have checked my code and corrected any misspellings

Maintainer Checklist

• closes LateInitializationError in PairChatController (userAddedSubscription not initialized) #560
• Tag the PR with the appropriate labels

[github-actions]

🎉 Welcome @4555jan!
Thank you for your pull request! Our team will review it soon. 🔍

• Please ensure your PR follows the contribution guidelines. ✅
• All automated tests should pass before merging. 🔄
• If this PR fixes an issue, link it in the description. 🔗

We appreciate your contribution! 🚀

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[M4dhav]

The "Changes Made" mentions

fields are now properly nullable and use null-safe operators (?.)

However the only change seems to be the removal of the late operator. Please confirm this and also confirm if you have verified all Pair Chat functionalities to be working

[4555jan]

Yes, the main change was the removal of the late keyword for userAddedSubscription. These fields were already declared as nullable (RealtimeSubscription?), and the code was already using null-safe calls (like ?.close()), so no further change to null-safety operators was necessary. The reason this change was needed is that using late with nullable fields caused a LateInitializationError when calling .close() before initialization. Removing late ensures they are properly initialized to null, allowing the existing ?. calls to safely handle them. @M4dhav if you want i can make video before and after to show how i tested

[M4dhav]

A video is not necessary. Thank you for the contribution!

[github-actions]

✅ PR Closed - Thank You, @4555jan!

• If this PR was merged: Congratulations! Your contribution is now part of the project. 🚀
• If this PR was closed without merging: Don’t worry! You can always improve it and submit again. 💪

We appreciate your effort and look forward to more contributions from you! 🤝