Skip to content

Conversation

Brazol
Copy link
Contributor

@Brazol Brazol commented Oct 1, 2025

Summary by CodeRabbit

  • Breaking Changes

    • Call stats payload restructured; some CallState fields removed. New minimum Flutter and updated Android build requirements.
  • New Features

    • Added statsReporter exposing currentMetrics (includes battery level and device thermal status).
  • Bug Fixes

    • Resolved leave-call failures with custom data, Android audio configuration issues, and a video rendering problem.
  • Chores

    • Bumped packages and aligned stream video-related dependencies to 0.11.0.
  • Documentation

    • Updated changelogs with migration guidance and clearer requirements.

@Brazol Brazol requested a review from a team as a code owner October 1, 2025 14:09
Copy link

coderabbitai bot commented Oct 1, 2025

Walkthrough

Bump workspace packages to 0.11.0, update stream_video-related dependency ranges, update changelogs to 0.11.0 with Android/Flutter/Dart requirements and breaking-change notes, and increment the streamVideoVersion constant in globals.dart. No runtime logic edits beyond version strings.

Changes

Cohort / File(s) Summary
Workspace & dogfooding pubspecs
melos.yaml, dogfooding/pubspec.yaml, packages/stream_video/pubspec.yaml, packages/stream_video_flutter/pubspec.yaml, packages/stream_video_noise_cancellation/pubspec.yaml, packages/stream_video_push_notification/pubspec.yaml, packages/stream_video_screen_sharing/pubspec.yaml, packages/stream_video_flutter/example/pubspec.yaml
Bump package versions to 0.11.0 and align dependencies (upgrade stream_video and related packages from ^0.10.4^0.11.0).
Changelogs
packages/stream_video/CHANGELOG.md, packages/stream_video_flutter/CHANGELOG.md, packages/stream_video_noise_cancellation/CHANGELOG.md, packages/stream_video_push_notification/CHANGELOG.md, packages/stream_video_screen_sharing/CHANGELOG.md
Replace “Unreleased” with 0.11.0 entries; add Android/Flutter/Dart minimums and document breaking changes (stats payload shape, statsReporter/currentMetrics) and fixes.
Version constant
packages/stream_video/lib/globals.dart
Update const String streamVideoVersion from '0.10.4' to '0.11.0' (affects user-agent string).

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • renefloor
  • xsahil03x

Poem

I thump my paws—0.11 hops in flight,
Versions aligned, changelogs polished bright.
Stats tucked neat in bundles and rows,
Android asks taller tools as it grows.
I nibble a merge, then bound into night. 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The pull request description is missing entirely and does not follow the repository’s required template which includes sections for the goal, implementation details, testing instructions, and contributor checklist. Please add a pull request description using the repository template by filling in the 🎯 Goal, 🛠 Implementation details, 🧪 Testing sections and completing the contributor checklist to explain the purpose and testing of these version bumps.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly indicates that this PR is a chore to release version v0.11.0 and aligns with the primary change of bumping package versions for the v0.11.0 release.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch release/v0.11.0

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

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

Copy link

codecov bot commented Oct 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 4.80%. Comparing base (0623534) to head (9f546d8).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1072   +/-   ##
=====================================
  Coverage   4.80%   4.80%           
=====================================
  Files        591     591           
  Lines      40395   40395           
=====================================
  Hits        1941    1941           
  Misses     38454   38454           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🧹 Nitpick comments (5)
packages/stream_video_noise_cancellation/CHANGELOG.md (1)

7-11: Use consistent comparison operators across items

AGP/Gradle use “>=” while Kotlin is a plain “2.2.0”. Consider “Kotlin >= 2.2.0” (if older patch versions work) or “Kotlin = 2.2.0” if it must be pinned.

packages/stream_video_flutter/CHANGELOG.md (1)

18-27: Add a short migration snippet for Call.stats rename

Include a before/after code sample to reduce upgrade friction (mapping publisherStatspublisherStatsBundle, etc.).

+// Before
+final stats = await call.stats.first;
+final pub = stats.publisherStats;
+
+// After (0.11.0)
+final stats = await call.stats.first;
+final pub = stats.publisherStatsBundle;
packages/stream_video/CHANGELOG.md (2)

18-27: Clarify migration path for removed fields

Add a brief mapping table (old CallState fields → new surfaces), e.g., publisherStats/subscriberStatscall.stats stream, aggregated → call.statsReporter?.currentMetrics.


7-11: Android requirements consistency and completeness

  • Rename “compileSDK 36” to “compileSdk 36.”
  • Add “JDK 17+ required” to the requirements list.
  • AGP 8.12.1 and Gradle 8.13 are released and compatible (AGP 8.x requires JDK 17).
packages/stream_video_noise_cancellation/pubspec.yaml (1)

16-18: Align plugin_platform_interface version across plugins

This package uses plugin_platform_interface: ^2.0.2, while stream_video_screen_sharing uses ^2.1.8. Consider bumping here to ^2.1.8 to avoid duplicate transitive resolutions and keep SDK guidance consistent.

-  plugin_platform_interface: ^2.0.2
+  plugin_platform_interface: ^2.1.8
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 0623534 and a7894b9.

📒 Files selected for processing (14)
  • dogfooding/pubspec.yaml (2 hunks)
  • melos.yaml (1 hunks)
  • packages/stream_video/CHANGELOG.md (1 hunks)
  • packages/stream_video/lib/globals.dart (1 hunks)
  • packages/stream_video/pubspec.yaml (1 hunks)
  • packages/stream_video_flutter/CHANGELOG.md (1 hunks)
  • packages/stream_video_flutter/example/pubspec.yaml (1 hunks)
  • packages/stream_video_flutter/pubspec.yaml (2 hunks)
  • packages/stream_video_noise_cancellation/CHANGELOG.md (1 hunks)
  • packages/stream_video_noise_cancellation/pubspec.yaml (2 hunks)
  • packages/stream_video_push_notification/CHANGELOG.md (1 hunks)
  • packages/stream_video_push_notification/pubspec.yaml (2 hunks)
  • packages/stream_video_screen_sharing/CHANGELOG.md (1 hunks)
  • packages/stream_video_screen_sharing/pubspec.yaml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)
  • GitHub Check: analyze
  • GitHub Check: build
  • GitHub Check: stream_video
  • GitHub Check: stream_video_push_notification
  • GitHub Check: stream_video_noise_cancellation
  • GitHub Check: stream_video_flutter
🔇 Additional comments (12)
packages/stream_video/pubspec.yaml (1)

4-4: LGTM! Version bump aligns with the 0.11.0 release.

The version update is consistent with the coordinated release across the stream_video package ecosystem.

packages/stream_video_push_notification/CHANGELOG.md (1)

1-15: LGTM! Well-documented breaking changes for the 0.11.0 release.

The CHANGELOG entry clearly communicates the breaking changes for Android development with specific version requirements, making it easy for developers to understand what they need to update.

packages/stream_video_flutter/example/pubspec.yaml (1)

30-32: LGTM! Dependency versions correctly updated for the 0.11.0 release.

The example app dependencies are properly aligned with the new release version.

packages/stream_video/lib/globals.dart (1)

4-4: LGTM! Version constant updated for the 0.11.0 release.

This change ensures the SDK reports the correct version in user-agent strings and version checks.

packages/stream_video_screen_sharing/CHANGELOG.md (1)

1-15: LGTM! Consistent breaking changes documentation for the 0.11.0 release.

The CHANGELOG entry maintains consistency with other packages in the ecosystem and clearly documents the Android breaking changes and SDK requirements.

packages/stream_video_noise_cancellation/pubspec.yaml (1)

9-12: Env constraints match changelog — LGTM

Dart ^3.8.0 and Flutter >=3.32.0 align with the 0.11.0 notes.

packages/stream_video_screen_sharing/pubspec.yaml (2)

5-5: Version bump looks good

0.10.4 → 0.11.0 with unchanged API surface.


10-12: Env constraints consistent

Dart ^3.8.0 and Flutter >=3.32.0 match 0.11.0 requirements.

dogfooding/pubspec.yaml (1)

2-2: Flutter SDK unavailable in sandbox—please verify locally
I couldn’t run flutter pub get or dart analyze --fatal-infos . here (Flutter isn’t installed). Please run those commands in your local environment to confirm there are no errors.

packages/stream_video_push_notification/pubspec.yaml (1)

4-4: Approve version bump and CHANGELOG entry
CHANGELOG.md contains a ## 0.11.0 entry that matches the pubspec version.

melos.yaml (1)

26-30: Deps aligned to ^0.11.0; no ^0.10.x constraints found, global version constant updated.

packages/stream_video_flutter/pubspec.yaml (1)

4-4: Approve version bump to 0.11.0; verify intl constraint

Confirm with Flutter 3.32 whether the intl: <=0.21.0 cap in packages/stream_video_flutter/pubspec.yaml is still required to avoid unnecessary resolution conflicts.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between a7894b9 and 9f546d8.

📒 Files selected for processing (2)
  • packages/stream_video/CHANGELOG.md (1 hunks)
  • packages/stream_video_flutter/CHANGELOG.md (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
  • GitHub Check: build
  • GitHub Check: analyze
  • GitHub Check: stream_video_noise_cancellation
  • GitHub Check: stream_video_flutter
  • GitHub Check: analyze_legacy_version
  • GitHub Check: stream_video
  • GitHub Check: stream_video_push_notification

Comment on lines +8 to +11
> - Minimum compileSDK 36
> - Android Gradle Plugin >=8.12.1
> - Gradle wrapper >=8.13
> - Kotlin 2.2.0
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion | 🟠 Major

🧩 Analysis chain

Please align the Android tooling requirements with the actual project configuration.

The changelog still lists “compileSDK 36”, “Android Gradle Plugin >=8.12.1”, and “Gradle wrapper >=8.13”, but the repository now uses compileSdk = 36, AGP 8.13.0, and Gradle 9.1.0. Please fix the casing (compileSdk), bump the AGP/Gradle minimums, and add the mandatory JDK 17 requirement so the docs accurately match the build setup.

To confirm the current versions in the repo, run:


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "Android Gradle Plugin version(s):"
rg -nP "com\.android\.tools\.build:gradle" -C2

echo -e "\nGradle wrapper distribution:"
fd gradle-wrapper.properties --type f --exec sh -c 'echo "==> $1"; cat "$1"' sh

Length of output: 1747


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "compileSdk version(s):"
rg -nP "compileSdk(Version)?\s*=\s*\d+" -C1

Length of output: 22


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "Searching for compileSdkVersion assignments:"
rg -nP "compileSdk(Version)?\s+\d+" -n

echo -e "\nInspecting stream_video_flutter build.gradle:"
FILE=$(fd --type f --full-path "packages/stream_video_flutter/android/build.gradle" || echo "")
if [ -n "$FILE" ]; then
  echo "==> $FILE"
  rg -nP "compileSdk|com.android.tools.build:gradle" -n "$FILE"
else
  echo "stream_video_flutter build.gradle not found"
fi

Length of output: 44


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "All 'compileSdk' occurrences in repo:"
rg -n "compileSdk" -n || true

echo -e "\nGradle build files under stream_video_flutter:"
fd --type f --full-path "packages/stream_video_flutter/android" -e gradle -e kts

echo -e "\nInspect each for compileSdk and AGP:"
for file in $(fd --type f --full-path "packages/stream_video_flutter/android" -e gradle -e kts); do
  echo "==> $file"
  rg -n "compileSdk" -n "$file" || true
  rg -n "com.android.tools.build:gradle" -n "$file" || true
done

Length of output: 910


Align changelog with actual Android tooling versions
CHANGELOG.md (lines 8–11): correct casing (compileSdk), and list the real minimums—AGP ≥ 8.12.1 and Gradle wrapper ≥ 8.13 (compileSdkVersion is set via flutter.compileSdkVersion).

🤖 Prompt for AI Agents
In packages/stream_video_flutter/CHANGELOG.md around lines 8 to 11, update the
Android tooling entries to use the correct casing and actual minimums: change
"compileSDK" to "compileSdk" and note that compileSdkVersion is controlled via
flutter.compileSdkVersion; ensure entries state "Android Gradle Plugin ≥ 8.12.1"
and "Gradle wrapper ≥ 8.13" exactly as the minimums.

@Brazol Brazol merged commit 36ae8cf into main Oct 2, 2025
14 of 15 checks passed
@Brazol Brazol deleted the release/v0.11.0 branch October 2, 2025 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants