Skip to content

chore(repo): resolve and analyze on Flutter 3.50 - #1616

Merged
renefloor merged 1 commit into
v2from
ci/flutter-3-50-v2
Nov 11, 2026
Merged

renefloor merged 1 commit into
v2from
ci/flutter-3-50-v2

Conversation

@renefloor

@renefloor renefloor commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

🎯 Goal

Make v2 resolve and pass analysis on Flutter 3.50 (tested on 3.50.0-0.4.pre), so the beta check and a later Flutter bump don't break CI.

🛠 Implementation details

  • stream_video dev dependency freezed now allows 4.x (">=3.0.0 <5.0.0"). On 3.50 flutter_test pins test_api 0.7.14, so the webrtc packages that depend on test need test >=1.31.2, which needs analyzer >=13. freezed 3 tops out at analyzer 12, so pub get failed. On the current Flutter (3.47.x) the lock file is unchanged and freezed 3 is still used, so generated files don't change.
  • webrtc/stream_webrtc_flutter/example: ignore deprecated_member_use. Flutter 3.50 deprecates the material.dart/cupertino.dart imports, and webrtc:analyze runs with --fatal-infos. The SDK packages are analyzed with --no-fatal-warnings, so their deprecation infos don't fail CI.

The deprecated_member_use ignore is reverted as part of FLU-922, the migration to material_ui/cupertino_ui.

🧪 Testing

With Flutter 3.50.0-0.4.pre: flutter pub get, melos run webrtc:analyze, melos run analyze:error and melos run format:verify pass. With 3.47.4: flutter pub get passes with no lock file changes.

🤖 Generated with Claude Code

Allow freezed 4, needed for pub get to resolve on Flutter 3.50, and
ignore the material/cupertino import deprecations in the webrtc example,
which webrtc:analyze runs with --fatal-infos.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 1d954448-5105-4d9e-9a55-6c37fd22238a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • 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

@github-actions github-actions Bot added the v2 label Nov 10, 2026
@renefloor
renefloor marked this pull request as ready for review November 10, 2026 11:58
@renefloor
renefloor requested a review from a team as a code owner November 10, 2026 11:58
@github-actions

github-actions Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

SDK Size

title v2 branch diff status
android_stream_video 15.14 MB 15.14 MB 0 KB 🟢
android_stream_video_flutter 17.48 MB 17.48 MB 0 KB 🟢
pub_stream_video 1.0 MB 1.0 MB 0 KB 🟢
pub_stream_video_filters 0.04 MB 0.04 MB 0 KB 🟢
pub_stream_video_flutter 1.0 MB 1.0 MB 0 KB 🟢
pub_stream_video_noise_cancellation 0.05 MB 0.05 MB 0 KB 🟢
pub_stream_video_push_notification 0.36 MB 0.36 MB 0 KB 🟢
pub_stream_video_screen_sharing 0.01 MB 0.01 MB 0 KB 🟢

Measured on fcc4991.

@github-actions

Copy link
Copy Markdown

Size Breakdown

What each SDK adds to an empty release app, per Dart package and native library. Dart sizes are uncompressed symbol sizes. Open the sdk-size-analysis artifacts of this run in the DevTools App Size tool to drill down further.

This run does not measure ios.

Measured on fcc4991.

android · stream_video: 15.1 MB (0)
Component Size Δ vs v2
native:libjingle_peerconnection_so.so 11.96 MB 0
package:stream_video 1.11 MB 0
classes.dex 261.9 KB 0
dart:@unknown 260.1 KB 0
dart:_http 154.1 KB 0
package:protobuf 146.0 KB 0
dart:mixin_deduplication 143.8 KB 0
native:libdartjni.so 121.7 KB 0
dart:io 99.3 KB 0
dart:async 75.6 KB 0
package:stream_core 69.5 KB 0
package:stream_webrtc_flutter 55.3 KB 0
package:rxdart 54.3 KB 0
package:dio 53.2 KB 0
dart:core 52.2 KB 0
other (71) 537.7 KB 0
android · stream_video_flutter: 17.41 MB (0)
Component Size Δ vs v2
native:libjingle_peerconnection_so.so 11.96 MB 0
package:stream_video 1.16 MB 0
package:flutter 699.1 KB 0
classes.dex 454.8 KB 0
dart:@unknown 434.9 KB 0
dart:mixin_deduplication 362.7 KB 0
package:stream_video_flutter 205.7 KB 0
dart:_http 154.1 KB 0
package:protobuf 145.2 KB 0
dart:io 136.5 KB 0
package:stream_core_flutter 124.2 KB 0
native:libdartjni.so 121.7 KB 0
assets 118.5 KB 0
package:hive_ce 99.2 KB 0
dart:@shared 84.2 KB 0
other (99) 1.22 MB 0

@renefloor renefloor added the ready Pull request ready to be reviewed label Nov 10, 2026
@codecov

codecov Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.05%. Comparing base (e8abe0d) to head (fcc4991).
⚠️ Report is 3 commits behind head on v2.

Additional details and impacted files
@@           Coverage Diff           @@
##               v2    #1616   +/-   ##
=======================================
  Coverage   66.05%   66.05%           
=======================================
  Files         467      467           
  Lines       27361    27361           
=======================================
  Hits        18072    18072           
  Misses       9289     9289           

☔ View full report in Codecov by Harness.
📢 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.

@github-actions github-actions Bot added approved Approved by a reviewer and removed ready Pull request ready to be reviewed labels Nov 10, 2026
@renefloor
renefloor merged commit 85619fb into v2 Nov 11, 2026
63 checks passed
@renefloor
renefloor deleted the ci/flutter-3-50-v2 branch November 11, 2026 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Approved by a reviewer v2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants