[gpunto]

Goal

Closes AND-1701

Move develop-v2 to Kotlin 2.3 so the SDK can use kotlin.time.Instant (replacing ThreeTenABP is AND-1702).

Implementation

• kotlin and kotlinTestJunit: 2.2.0 → 2.3.21.
• ksp: 2.2.0-2.0.2 → 2.3.11. KSP versions are independent of Kotlin since 2.3.0. 2.3.12 requires AGP 8.12, and we are on 8.10.1.
• hilt: 2.56.2 → 2.58. 2.56.2 can't read Kotlin 2.3 metadata, and 2.59 and later require AGP 9.
• TraceRecordTest: explicit arrayOf<Any?>, because Kotlin 2.3 rejects a reified type parameter inferred as an intersection type.

Testing

• ./gradlew assembleDebug apiCheck testDebugUnitTest passes, and the API dumps are unchanged.

Summary by CodeRabbit

• Chores

  • Updated Kotlin, KSP, and Hilt tooling to newer versions for improved compatibility and maintenance.

• Tests

  • Improved serialization test coverage for records containing nullable values.

[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.38 MB	12.66 MB	0.29 MB	🟡
stream-video-android-ui-xml	5.66 MB	5.72 MB	0.06 MB	🟢
stream-video-android-ui-compose	6.19 MB	3.16 MB	-3.03 MB	🚀

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 7d1aab5c-cdf2-4ca1-8c03-ab048da0bbba

📥 Commits

Reviewing files that changed from the base of the PR and between 88d73e1 and c3ed03c.

📒 Files selected for processing: 2

• gradle/libs.versions.toml
• stream-video-android-core/src/test/kotlin/io/getstream/video/android/core/trace/TraceRecordTest.kt

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

📝 Walkthrough

The version catalog updates Kotlin and Kotlin test JUnit to 2.3.21, KSP to 2.3.11, and Hilt to 2.58. Two serialization test assertions now use explicitly nullable expected arrays.

Suggested reviewers: aleksandar-apostolov

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to c3ed0

The dependency and serialization test updates are ready to merge with normal checks.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 1 files. (1 skipped: 1 …	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Title check		The title clearly summarizes the primary change: the Kotlin upgrade with matching KSP and Hilt version updates.
Description check		The description includes the required Goal, Implementation, and Testing sections. It explains the version changes, compatibility constraints, test updates, linked issues, and validation results. The C…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description includes the required Goal, Implementation, and Testing sections. It explains the version changes, compatibility constraints, test updates, linked issues, and validation results. The Checklist section is omitted, but the description is otherwise complete.

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 1 files. (1 skipped: 1 unsupported.)

Resolution

Write docstrings for the functions missing them to satisfy the coverage threshold.

✨ Finishing Touches: 💡 2

📝 Generate docstrings: 💡

• Commit to this branch
• Create a new PR

🛠️ Fix failing CI checks: 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

_{A rabbit checks the version list,
Then nibbles through each test.
Nullable arrays hop into place,
While Kotlin takes a step.
The rabbit twitches both its ears,
And finds the changes dressed.}

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