Skip to content

Conversation

@Brazol
Copy link
Contributor

@Brazol Brazol commented Dec 19, 2025

Summary by CodeRabbit

  • New Features

    • Added Android screen audio sharing support with configuration options.
  • Bug Fixes

    • Fixed unnecessary video filter reapplication when camera track is disabled.

✏️ Tip: You can customize this high-level summary in your review settings.

@Brazol Brazol marked this pull request as ready for review December 19, 2025 11:45
@Brazol Brazol requested a review from a team as a code owner December 19, 2025 11:45
@coderabbitai
Copy link

coderabbitai bot commented Dec 19, 2025

Walkthrough

This PR performs a coordinated version bump across the stream_video suite of packages from 1.1.0 to 1.2.0, updates the streamVideoVersion constant in globals.dart, and increments the stream_webrtc_flutter dependency from 2.2.1 to 2.2.2 across all manifests and changelog files.

Changes

Cohort / File(s) Summary
Version bumps in package manifests
packages/stream_video/pubspec.yaml, packages/stream_video_filters/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
Package version updated from 1.1.0 to 1.2.0; stream_webrtc_flutter dependency bumped from ^2.2.1 to ^2.2.2 where applicable
Dependency overrides and example projects
dogfooding/pubspec.yaml, packages/stream_video_flutter/example/pubspec.yaml, melos.yaml
Updated stream_video* package constraints from ^1.1.0 to ^1.2.0 and stream_webrtc_flutter from ^2.2.1 to ^2.2.2
Changelog entries
packages/stream_video/CHANGELOG.md, packages/stream_video_filters/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 or updated version 1.2.0 changelog entries documenting sync with stream_video_flutter 1.2.0 and specific feature notes
Version constant
packages/stream_video/lib/globals.dart
Updated streamVideoVersion constant from '1.1.0' to '1.2.0'

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10–15 minutes

  • Homogeneous changes: the same version-bump pattern is repeated across multiple files, reducing per-file review complexity
  • Low-risk modifications: purely manifest and documentation updates with a single trivial constant value change
  • No functional logic changes or new code to audit

Possibly related PRs

Suggested reviewers

  • renefloor
  • xsahil03x

Poem

🐰 Hoppy hops, version's up!
From 1.1 to 1.2, we sup.
Constants change, deps aligned,
Changelogs gently signed,
A release so well-designed!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is entirely missing; no content was provided by the author despite the repository template requiring multiple detailed sections. Add a comprehensive description following the repository template, including the Goal, Implementation details, Testing section, and completed Contributor Checklist to explain the release objectives and testing performed.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: releasing version 1.2.0, which is directly supported by all file changes showing version bumps from 1.1.0 to 1.2.0 across the repository.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch release/v1.2.0

📜 Recent review details

Configuration used: defaults

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 6850c9e and 88bc337.

📒 Files selected for processing (16)
  • 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 (2 hunks)
  • packages/stream_video_filters/CHANGELOG.md (1 hunks)
  • packages/stream_video_filters/pubspec.yaml (2 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: build
  • GitHub Check: analyze
  • GitHub Check: stream_video_flutter
  • GitHub Check: stream_video_push_notification
  • GitHub Check: stream_video_noise_cancellation
  • GitHub Check: stream_video
🔇 Additional comments (17)
packages/stream_video_push_notification/CHANGELOG.md (1)

1-3: LGTM! Version bump looks good.

The 1.2.0 changelog entry is properly formatted and consistent with the package's established changelog pattern.

packages/stream_video_noise_cancellation/CHANGELOG.md (1)

1-3: LGTM! Changelog entry is consistent.

The 1.2.0 version entry follows the established pattern for this package.

packages/stream_video_flutter/CHANGELOG.md (1)

1-4: LGTM! Release changelog is well-documented.

The 1.2.0 release entry properly documents the Android screen audio sharing feature with a documentation link. The changelog follows the established format with clear categorization.

packages/stream_video_screen_sharing/CHANGELOG.md (1)

1-3: LGTM! Feature documentation is clear.

The 1.2.0 entry clearly documents the Android screen audio sharing feature with usage instructions and a documentation link.

packages/stream_video_filters/CHANGELOG.md (1)

1-2: LGTM! Bug fix is documented.

The 1.2.0 entry appropriately documents the video filter bug fix.

packages/stream_video/CHANGELOG.md (1)

1-3: LGTM! Version sync entry is consistent.

The 1.2.0 changelog entry follows the established pattern for version synchronization.

dogfooding/pubspec.yaml (2)

2-2: LGTM! App version bump is appropriate.

The dogfooding app version is properly incremented to 2.2.0+1 to align with the package releases.


40-44: LGTM! Dependency versions are consistent.

All stream_video_* package dependencies are properly updated to ^1.2.0, maintaining consistency across the release.

melos.yaml (1)

25-30: LGTM! Melos dependencies are properly synchronized.

All dependency versions in the melos configuration are updated consistently:

  • stream_webrtc_flutter bumped to ^2.2.2
  • All stream_video_* packages bumped to ^1.2.0

This ensures coordinated versioning across the monorepo.

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

7-7: Version constant correctly synchronized with package version.

The streamVideoVersion constant is properly updated to '1.2.0' and aligns with the version bump in packages/stream_video/pubspec.yaml.

packages/stream_video_push_notification/pubspec.yaml (1)

4-4: Version and dependency updates are consistent.

Package version and inter-package dependencies are correctly synchronized for the 1.2.0 release.

Also applies to: 24-26

packages/stream_video_filters/pubspec.yaml (1)

4-4: Version and dependency updates are consistent.

Package version and dependencies correctly aligned with the 1.2.0 release.

Also applies to: 17-18

packages/stream_video/pubspec.yaml (1)

4-4: Version and dependency updates are consistent.

Package version correctly bumped to 1.2.0, and stream_webrtc_flutter dependency updated to ^2.2.2, aligning with the coordinated release.

Also applies to: 34-34

packages/stream_video_screen_sharing/pubspec.yaml (1)

5-5: Version bump is consistent with the coordinated release.

Package version correctly updated to 1.2.0 as part of the synchronized release across the stream_video suite.

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

31-34: Example app dependencies updated correctly.

All dependency versions are consistent with the 1.2.0 release. The path overrides below ensure the example uses local packages during development.

packages/stream_video_flutter/pubspec.yaml (1)

4-4: Package version and dependencies aligned correctly.

The version bump and dependency updates are consistent with the coordinated 1.2.0 release across the stream_video package suite.

Also applies to: 27-28

packages/stream_video_noise_cancellation/pubspec.yaml (1)

4-4: External dependency stream_webrtc_flutter cannot be verified.

The version 1.2.0 bump and internal stream_video alignment to ^1.2.0 look correct. However, the external dependency stream_webrtc_flutter 2.2.2 does not appear to exist on pub.dev. Verify the correct package name and version before publishing.


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.

@codecov
Copy link

codecov bot commented Dec 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 6.31%. Comparing base (6850c9e) to head (88bc337).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1136   +/-   ##
=====================================
  Coverage   6.31%   6.31%           
=====================================
  Files        595     595           
  Lines      41081   41081           
=====================================
  Hits        2595    2595           
  Misses     38486   38486           

☔ 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.

@Brazol Brazol merged commit e3e80cd into main Dec 19, 2025
16 of 18 checks passed
@Brazol Brazol deleted the release/v1.2.0 branch December 19, 2025 12:57
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