Safeguard all call class's public methods against NPE

[rahul-lohra]

Goal

Closes #AND-1350

We got a crash with stacktrace

Fatal Exception: java.lang.NullPointerException
Attempt to invoke virtual method 'java.lang.Class java.lang.Object.getClass()' on a null object reference
io.getstream.video.android.core.Call._join$stream_video_android_core_release (Call.kt:788)
io.getstream.video.android.core.Call$_join$1.invokeSuspend (Call.kt:17)
kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith (ContinuationImpl.kt:34)

Since we were unable to reproduce the crash with the exact stack trace, we decided to defensively guard all public methods of the Call class against potential NullPointerExceptions.

Implementation

Wrap all Call class's public method which returns Result with try-catch

@OptIn(ExperimentalContracts::class)
internal inline fun <T : Any> runResultCatchingCancellable(block: () -> Result<T>): Result<T> {
    contract { callsInPlace(block, InvocationKind.AT_MOST_ONCE) }
    return try {
        block()
    } catch (ce: CancellationException) {
        throw ce
    } catch (npe: NullPointerException) {
        Result.Failure(Error.ThrowableError(npe.message ?: "NullPointerException caught", npe))
    }
}

🎨 UI Changes

None

Testing

Smoke test video/audio call

[github-actions]

PR checklist ✅

All required conditions are satisfied:

• Title length is OK (or ignored by label).
• At least one pr: label exists.
• Sections ### Goal, ### Implementation, and ### Testing are filled, or the PR is bot-authored.
• An issue is linked (Linear ticket or GitHub issue), or the PR is bot-authored.

🎉 Great job! This PR is ready for review.

[github-actions]

SDK Size Comparison 📏

SDK	Before	After	Difference	Status
stream-video-android-core	12.27 MB	12.29 MB	0.02 MB	🟢
stream-video-android-ui-xml	5.68 MB	5.70 MB	0.02 MB	🟢
stream-video-android-ui-compose	6.20 MB	6.20 MB	0.00 MB	🟢

[sonarqubecloud]

Quality Gate failed

Failed conditions
14.1% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

[rahul-lohra]

Closing this PR — the NPE crash is already resolved in #1740, released in v1.29.0.

Note on reproducing the exact stack trace

The reported message only appears on a release build with AGP 9+:

java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.Class java.lang.Object.getClass()' on a null object reference

Why: session.value!! is a null-assertion that R8 compiles differently per version. On AGP 9+, R8 rewrites it into an Object.getClass() call, so a null value produces the getClass() message above. On AGP 8, R8 kept it as Intrinsics.checkNotNull, which throws a plain NullPointerException without that text — same crash, different wording. This is why it couldn't be reproduced with the exact message on our AGP 8 builds.