feat(samples): Update chat sdk in dogfooding#1253
Conversation
renefloor
changed the title
|
Warning Review limit reached
More reviews will be available in 58 minutes and 35 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughStream Chat dark theme moved into ThemeData.extensions; StreamMessageInput replaced with StreamMessageComposer in the call screen; Dart/Flutter SDK constraints and several dependency versions were bumped across manifests with a file_picker override; generated plugin lists removed ChangesTheme Integration and Component Updates
🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Currently legacy_analyze requires Flutter 3.32.0, which is over a year old. We can easily set the minimum to 3.38.0, which means we support the last 3 stables and dropping 3.32 and 3.35. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@dogfooding/lib/app/app_content.dart`:
- Line 286: Replace the deprecated/invalid chat.StreamTheme.dark() call with the
v10 theming API: use chat.StreamChatThemeData.dark() (or
StreamChatThemeData.dark() from the same package) and pass that into the widget
expecting a StreamChatThemeData; also update imports if necessary to reference
StreamChatThemeData and remove StreamTheme usage so the app uses the new
stream_chat_flutter v10 theming API.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 73f67c1a-d622-40ec-967e-ef7c9e46322d
📒 Files selected for processing (8)
dogfooding/lib/app/app_content.dartdogfooding/lib/router/router.dartdogfooding/lib/screens/call_screen.dartdogfooding/pubspec.yamlmelos.yamlpackages/stream_video_flutter/example/linux/flutter/generated_plugins.cmakepackages/stream_video_flutter/example/pubspec.yamlpackages/stream_video_flutter/example/windows/flutter/generated_plugins.cmake
💤 Files with no reviewable changes (3)
- packages/stream_video_flutter/example/windows/flutter/generated_plugins.cmake
- packages/stream_video_flutter/example/linux/flutter/generated_plugins.cmake
- dogfooding/lib/router/router.dart
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
dogfooding/pubspec.yaml (1)
47-47:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winBeta dependency in production warrants caution.
The
file_pickeroverride uses a beta version (12.0.0-beta.5). While the PR objectives explain this is necessary for win32 compatibility, beta versions in production may introduce instability.Monitor the
file_pickerpackage for a stable 12.0.0 release and update this override once available. Consider adding a TODO comment or tracking issue to remove the override when the stable version is released.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@dogfooding/pubspec.yaml` at line 47, Update the pubspec entry that pins file_picker to the beta (the "file_picker: ^12.0.0-beta.5" line) by adding a TODO comment and/or create a tracking issue reference to monitor for a stable 12.0.0 release; when the stable 12.0.0 is published, remove the override and update the version constraint to the stable release. Ensure the TODO or issue ID is clearly adjacent to the file_picker entry so future maintainers can find and remove the beta pin promptly.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@dogfooding/pubspec.yaml`:
- Line 47: Update the pubspec entry that pins file_picker to the beta (the
"file_picker: ^12.0.0-beta.5" line) by adding a TODO comment and/or create a
tracking issue reference to monitor for a stable 12.0.0 release; when the stable
12.0.0 is published, remove the override and update the version constraint to
the stable release. Ensure the TODO or issue ID is clearly adjacent to the
file_picker entry so future maintainers can find and remove the beta pin
promptly.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 7a679d49-f569-4e94-a3ec-6306cbabeddb
📒 Files selected for processing (20)
.github/workflows/app-distribute.yml.github/workflows/legacy_version_analyze.yml.github/workflows/stream_video_flutter_workflow.ymldogfooding/pubspec.yamlmelos.yamlpackages/stream_video/CHANGELOG.mdpackages/stream_video/pubspec.yamlpackages/stream_video_filters/CHANGELOG.mdpackages/stream_video_filters/pubspec.yamlpackages/stream_video_flutter/CHANGELOG.mdpackages/stream_video_flutter/example/pubspec.yamlpackages/stream_video_flutter/pubspec.yamlpackages/stream_video_noise_cancellation/CHANGELOG.mdpackages/stream_video_noise_cancellation/example/pubspec.yamlpackages/stream_video_noise_cancellation/pubspec.yamlpackages/stream_video_push_notification/CHANGELOG.mdpackages/stream_video_push_notification/example/pubspec.yamlpackages/stream_video_push_notification/pubspec.yamlpackages/stream_video_screen_sharing/CHANGELOG.mdpackages/stream_video_screen_sharing/pubspec.yaml
✅ Files skipped from review due to trivial changes (15)
- packages/stream_video_filters/CHANGELOG.md
- packages/stream_video_push_notification/example/pubspec.yaml
- packages/stream_video/pubspec.yaml
- packages/stream_video_screen_sharing/pubspec.yaml
- packages/stream_video_noise_cancellation/pubspec.yaml
- packages/stream_video_push_notification/pubspec.yaml
- packages/stream_video_noise_cancellation/example/pubspec.yaml
- packages/stream_video_push_notification/CHANGELOG.md
- packages/stream_video/CHANGELOG.md
- packages/stream_video_flutter/CHANGELOG.md
- packages/stream_video_screen_sharing/CHANGELOG.md
- packages/stream_video_noise_cancellation/CHANGELOG.md
- packages/stream_video_flutter/pubspec.yaml
- .github/workflows/legacy_version_analyze.yml
- packages/stream_video_filters/pubspec.yaml
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1253 +/- ##
=====================================
Coverage 9.68% 9.68%
=====================================
Files 676 676
Lines 49689 49689
=====================================
Hits 4810 4810
Misses 44879 44879 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
2 participants
🎯 Goal
Update chat sdk to v10
🛠 Implementation details
Chat update was relatively easy, but we have a dependency issue with win32. Most dependencies need latest win32 now, but for file_picker that's only supported in the latest beta.
☑️Contributor Checklist
General
☑️Reviewer Checklist
Summary by CodeRabbit
New Features
Chores