Skip to content

Conversation

@Brazol
Copy link
Contributor

@Brazol Brazol commented Aug 20, 2025

Summary by CodeRabbit

  • New Features
    • Added option to configure Android audio during initialization via StreamVideoOptions.
  • Bug Fixes
    • SFU stats reporting failures are now handled gracefully to improve stability.
  • Chores
    • Bumped app to 1.10.3+1 and synchronized Stream Video packages to 0.10.3 across the workspace.
  • Documentation
    • Updated changelogs to reflect version 0.10.3 and recent changes.

@Brazol Brazol requested a review from a team as a code owner August 20, 2025 12:38
@coderabbitai
Copy link

coderabbitai bot commented Aug 20, 2025

Walkthrough

Coordinated 0.10.3 release prep: updated package versions and dependencies, adjusted melos bootstrap constraints, updated version constant, added changelog entries, removed unused imports in call session files, and applied minor formatting-only edits in WebRTC-related sources.

Changes

Cohort / File(s) Summary
Version bumps (pubspecs)
dogfooding/pubspec.yaml, packages/stream_video/pubspec.yaml, packages/stream_video_flutter/pubspec.yaml, packages/stream_video_flutter/example/pubspec.yaml, packages/stream_video_noise_cancellation/pubspec.yaml, packages/stream_video_push_notification/pubspec.yaml, packages/stream_video_screen_sharing/pubspec.yaml
Bumped package versions to 0.10.3 and aligned dependencies (stream_video* packages) from ^0.10.2 to ^0.10.3.
Melos bootstrap constraints
melos.yaml
Updated Stream Video-related bootstrap dependencies from ^0.10.2 to ^0.10.3.
Version constant
packages/stream_video/lib/globals.dart
Updated const streamVideoVersion from '0.10.2' to '0.10.3'.
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
Added 0.10.3 entries; noted SFU stats handling and Android audio configuration option; sync notes across packages.
Import cleanup (call session)
packages/stream_video/lib/src/call/session/call_session.dart, packages/stream_video/lib/src/call/session/call_session_factory.dart
Removed unused imports (coordinator API, logging, models, SFU/WebRTC utilities, dynascale). No API/signature changes.
Formatting-only edits
packages/stream_video/lib/src/audio_processing/audio_recognition_webrtc.dart, packages/stream_video/lib/src/webrtc/rtc_manager.dart
Reformatted constructor invocation and added trailing comma; no behavioral changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • renefloor
  • xsahil03x

Poem

Hop, hop—tags align, versions gleam,
0.10.3 hops into the stream.
Unused imports trimmed with care,
Changelogs whisper, “We were there.”
Quiet commas, tidy flow—
A rabbit nods: “Ship it. Go!” 🐇✨

Tip

🔌 Remote MCP (Model Context Protocol) integration is now available!

Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats.

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch release/v0.10.3

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
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@codecov
Copy link

codecov bot commented Aug 20, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 4.83%. Comparing base (b8214d8) to head (5cdea84).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...src/audio_processing/audio_recognition_webrtc.dart 0.00% 1 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1054   +/-   ##
=====================================
  Coverage   4.83%   4.83%           
=====================================
  Files        577     577           
  Lines      38830   38830           
=====================================
  Hits        1877    1877           
  Misses     36953   36953           

☔ 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: 1

🧹 Nitpick comments (1)
packages/stream_video/lib/src/audio_processing/audio_recognition_webrtc.dart (1)

129-133: Keep onSoundStateChanged call sites consistently formatted

Nit: The “speaking=false” branch is still single-line while the “speaking=true” branch is multiline. Consider making both consistent for readability and easier diffs.

Apply:

-          onSoundStateChanged(
-            SoundState(isSpeaking: false, audioLevel: averageAudioLevel),
-          );
+          onSoundStateChanged(
+            SoundState(
+              isSpeaking: false,
+              audioLevel: averageAudioLevel,
+            ),
+          );

Also applies to: 140-141

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

💡 Knowledge Base configuration:

  • MCP integration is disabled by default for public repositories
  • Jira integration is disabled by default for public repositories
  • 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 b8214d8 and 5cdea84.

📒 Files selected for processing (18)
  • 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/lib/src/audio_processing/audio_recognition_webrtc.dart (1 hunks)
  • packages/stream_video/lib/src/call/session/call_session.dart (0 hunks)
  • packages/stream_video/lib/src/call/session/call_session_factory.dart (0 hunks)
  • packages/stream_video/lib/src/webrtc/rtc_manager.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)
💤 Files with no reviewable changes (2)
  • packages/stream_video/lib/src/call/session/call_session_factory.dart
  • packages/stream_video/lib/src/call/session/call_session.dart
🧰 Additional context used
🪛 LanguageTool
packages/stream_video_noise_cancellation/CHANGELOG.md

[grammar] ~1-~1: There might be a mistake here.
Context: ## 0.10.3 * Sync version with stream_video_flutter...

(QB_NEW_EN)

packages/stream_video_screen_sharing/CHANGELOG.md

[grammar] ~1-~1: There might be a mistake here.
Context: ## 0.10.3 * Sync version with stream_video_flutter...

(QB_NEW_EN)

⏰ 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: stream_video_push_notification
  • GitHub Check: stream_video_flutter
  • GitHub Check: stream_video
  • GitHub Check: stream_video_noise_cancellation
  • GitHub Check: analyze
  • GitHub Check: build
  • GitHub Check: analyze_legacy_version
🔇 Additional comments (20)
packages/stream_video_screen_sharing/pubspec.yaml (1)

5-5: Version bump to 0.10.3 looks good.
No issues spotted with the pubspec change for this package.

packages/stream_video_flutter/pubspec.yaml (1)

4-4: No stale 0.10.2 references remain
All of the dependency constraint and version checks passed—pubspecs, globals.dart, melos.yaml, and any example/demo folders contain no lingering 0.10.2 pins. The only occurrences of “0.10.2” are historical entries in each package’s CHANGELOG.md. Ready to merge.

packages/stream_video_screen_sharing/CHANGELOG.md (1)

1-3: Changelog entry for 0.10.3 is consistent with the release.
Matches the coordinated version sync with stream_video_flutter 0.10.3.

packages/stream_video_noise_cancellation/CHANGELOG.md (1)

1-3: Changelog entry for 0.10.3 is consistent with the release.
No issues; aligns with the suite-wide version bump.

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

30-32: Version bump looks correct for the example project dependencies.

The dependencies are correctly updated to ^0.10.3 to align with the coordinated release across the Stream Video packages.

packages/stream_video/pubspec.yaml (1)

4-4: Version bump to 0.10.3 is consistent with the release.

The version update aligns with the coordinated 0.10.3 release across all Stream Video packages.

packages/stream_video_push_notification/CHANGELOG.md (1)

1-3: Changelog entry follows the established pattern.

The 0.10.3 entry correctly follows the pattern used in previous releases for version synchronization with the main stream_video_flutter package.

packages/stream_video/lib/src/webrtc/rtc_manager.dart (1)

782-782: Trailing comma added for code formatting consistency.

This is a minor formatting change that improves Dart code consistency by adding a trailing comma after the closing parenthesis. No functional impact.

packages/stream_video_flutter/CHANGELOG.md (1)

1-8: Changelog properly documents the 0.10.3 release.

The changelog correctly updates the version header to 0.10.3 and documents the key changes:

  • Bug fix for SFU stats reporting failures
  • New Android audio configuration option

The entries are well-formatted and provide clear information for users upgrading to this version.

packages/stream_video/lib/src/audio_processing/audio_recognition_webrtc.dart (1)

129-133: LGTM: formatting-only change

No behavioral changes. The multiline SoundState construction with trailing commas reads well and keeps dartfmt happy.

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

4-4: Version bump to 0.10.3 looks correct

This aligns the user agent and client header with the 0.10.3 release.


4-4: No stale 0.10.2 references remain—everything updated to 0.10.3

• No occurrences of “0.10.2” found across the repo (excluding CHANGELOGs and lockfiles)
• All pubspec.yaml files report version 0.10.3 (and dependency constraints updated to ^0.10.3)
packages/stream_video/lib/globals.dart now defines streamVideoVersion = '0.10.3'
melos.yaml bootstrap constraints reference ^0.10.3

packages/stream_video_push_notification/pubspec.yaml (2)

4-4: Package version bumped to 0.10.3

Release metadata updated appropriately.


24-24: Dependency aligned to stream_video ^0.10.3

Matches the workspace release train.

dogfooding/pubspec.yaml (2)

2-2: Dogfooding app version bumped to 1.10.3+1

Versioning consistent with the SDK release.


40-44: Dogfooding dependencies bumped to ^0.10.3

All four Stream Video dependencies are aligned. Note: dependency_overrides below will point to local paths during dev, which is fine.

packages/stream_video_noise_cancellation/pubspec.yaml (2)

3-3: Package version bumped to 0.10.3

Syncs with the coordinated release.


16-16: stream_video dependency aligned to ^0.10.3

Matches the rest of the workspace.

melos.yaml (2)

26-30: Release dependency bumps to ^0.10.3 look correct.

All five internal packages are aligned to the 0.10.3 release train in melos bootstrap deps. With Dart’s 0.y.z caret semantics, these resolve to >=0.10.3 <0.11.0, which is appropriate for a patch release.


26-30: Versions aligned to 0.10.3

  • No stale ^0.10.2 or "0.10.2" references found across the repo.
  • All five Stream Video package pubspec.yaml files declare version: 0.10.3.
  • packages/stream_video/lib/globals.dart defines
    const String streamVideoVersion = '0.10.3';

No further action required.

@Brazol Brazol merged commit e583759 into main Aug 20, 2025
14 of 15 checks passed
@Brazol Brazol deleted the release/v0.10.3 branch August 20, 2025 12:55
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.

3 participants