Skip to content

Conversation

@Brazol
Copy link
Contributor

@Brazol Brazol commented Nov 24, 2025

What's Changed

Full Changelog: v0.11.2...v1.0.0

Summary by CodeRabbit

  • New Features

    • v1.0.0 release across multiple packages; dependency upgrades to 1.0.0
    • Ringing: notify specific call members; improved Android/iOS interruption and audio focus handling
    • Swift Package Manager (SPM) support for iOS; video filters moved to a dedicated package
  • Breaking Changes

    • Renamed ringing/push notification APIs and related configuration fields
    • Removal/deprecation of several legacy push/CallKit options; migration notes provided
  • Bug Fixes

    • Fixed premature disposal when handling ringing notifications

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

@Brazol Brazol requested a review from a team as a code owner November 24, 2025 20:45
@coderabbitai
Copy link

coderabbitai bot commented Nov 24, 2025

Walkthrough

Coordinated release bump from 0.11.2 → 1.0.0 across stream-video packages: updated package versions and internal dependency constraints, updated the streamVideoVersion constant, and added 1.0.0 changelog entries documenting CallKit → Ringing API renames, deprecations, and SPM notes.

Changes

Cohort / File(s) Summary
Root config / dogfooding
dogfooding/pubspec.yaml, melos.yaml
Updated stream_video* dependency constraints from ^0.11.2^1.0.0.
Core package metadata
packages/stream_video/pubspec.yaml, packages/stream_video/lib/globals.dart
Bumped package version 0.11.21.0.0; updated streamVideoVersion constant to '1.0.0'.
Core changelog
packages/stream_video/CHANGELOG.md
Replaced Unreleased with 1.0.0; documented Breaking changes (CallKit→Ringing renames), Added ringing APIs and Android audio-focus notes, and Fixed premature disposal.
Flutter package metadata & example
packages/stream_video_flutter/pubspec.yaml, packages/stream_video_flutter/example/pubspec.yaml
Bumped package version to 1.0.0 and updated dependency stream_video^1.0.0.
Flutter changelog
packages/stream_video_flutter/CHANGELOG.md
Added 1.0.0 section describing CallKit→Ringing API renames, pushParams → pushConfiguration, nameCaller → callerName, deprecations, and SPM notes.
Filters package
packages/stream_video_filters/pubspec.yaml, packages/stream_video_filters/CHANGELOG.md
Bumped to 1.0.0; dependency stream_video^1.0.0; changelog header updated.
Noise cancellation package
packages/stream_video_noise_cancellation/pubspec.yaml, packages/stream_video_noise_cancellation/CHANGELOG.md
Version bumped to 1.0.0; dependency stream_video^1.0.0; changelog adds SPM note.
Push notification package
packages/stream_video_push_notification/pubspec.yaml, packages/stream_video_push_notification/CHANGELOG.md
Version bumped to 1.0.0; dependencies updated to ^1.0.0; changelog documents CallKit→Ringing renames and pushParams → pushConfiguration.
Screen sharing package
packages/stream_video_screen_sharing/pubspec.yaml, packages/stream_video_screen_sharing/CHANGELOG.md
Version bumped to 1.0.0; changelog adds SPM note.

Sequence Diagram(s)

sequenceDiagram
    participant App
    participant StreamVideo
    participant Ringing as RingingService

    Note over App,StreamVideo: Release v1.0.0 — metadata & API renames
    App->>StreamVideo: initialize(pushConfiguration)
    Note right of StreamVideo: replaces pushParams

    rect rgb(235,245,255)
        App->>Ringing: observe(onRingingEvent)
        Note right of Ringing: (was onCallKitEvent / CallKitEvent)
        Ringing->>App: emit RingingEvent
    end

    rect rgb(245,255,235)
        App->>StreamVideo: call.ring(userIds)
        StreamVideo->>Ringing: trigger ringing for userIds
    end
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

  • Pay extra attention to changelog/API rename consistency (onCallKitEvent → onRingingEvent and related observe* renames).
  • Verify push notification API changes (pushParams → pushConfiguration, nameCaller → callerName) are reflected where applicable.
  • Confirm all pubspec version/dependency bumps are uniform and streamVideoVersion in globals.dart matches.

Possibly related PRs

Suggested reviewers

  • renefloor
  • xsahil03x

Poem

🐰 We hopped from .11 to one-oh-oh,
Rings now chirp where CallKits used to go.
pushParams traded for a clearer name,
Packages aligned and changelogs tame.
SPM joins the hop — a tiny victory hop!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description provides a comprehensive list of changes included in the release but does not follow the repository's required template structure with Goal, Implementation details, Testing, and Contributor Checklist sections. Add the required template sections: fill in the Goal section explaining why v1.0.0 is being released, add Implementation details, Testing instructions, and complete the Contributor and Reviewer Checklists as specified in the repository template.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The PR title 'chore(repo): Release v1.0.0' clearly and concisely summarizes the main change—a version release to 1.0.0 across multiple packages.
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.0.0

📜 Recent 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 bdaa01a and 2bb326c.

📒 Files selected for processing (4)
  • packages/stream_video_flutter/CHANGELOG.md (1 hunks)
  • packages/stream_video_noise_cancellation/CHANGELOG.md (1 hunks)
  • packages/stream_video_push_notification/CHANGELOG.md (1 hunks)
  • packages/stream_video_screen_sharing/CHANGELOG.md (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
  • packages/stream_video_screen_sharing/CHANGELOG.md
  • packages/stream_video_noise_cancellation/CHANGELOG.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
packages/stream_video_flutter/CHANGELOG.md

11-11: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


12-12: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


14-14: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


15-15: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


16-16: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


17-17: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


18-18: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


19-19: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)

⏰ 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). (9)
  • GitHub Check: analyze
  • GitHub Check: build
  • GitHub Check: stream_video_push_notification
  • GitHub Check: stream_video_flutter
  • GitHub Check: stream_video_noise_cancellation
  • GitHub Check: stream_video_screen_sharing
  • GitHub Check: analyze_legacy_version
  • GitHub Check: stream_video
  • GitHub Check: iOS Build (Dogfooding)

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

@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: 0

🧹 Nitpick comments (2)
packages/stream_video_push_notification/CHANGELOG.md (1)

22-23: Expand the SPM support section with migration guidance.

The Swift Package Manager support is mentioned but lacks context for developers evaluating the 1.0.0 upgrade. For a significant feature addition in a major release, consider adding:

  • Whether SPM is now the recommended integration method
  • Any migration steps from existing CocoaPods/manual integration
  • Link to SPM setup documentation (if available)

This aligns with the level of detail provided in the breaking changes sections and improves user experience during adoption.

packages/stream_video/CHANGELOG.md (1)

1-33: LGTM! Well-documented breaking changes.

The 1.0.0 release notes comprehensively document breaking changes in CallKit/Ringing APIs with clear migration paths, deprecated member removals, new features, and bug fixes. The changelog provides excellent guidance for users upgrading to 1.0.0.

Optional: Fix markdown indentation for consistency.

The static analysis tool flagged unordered list indentation issues on lines 11-16. While this doesn't affect readability, fixing it would improve markdown consistency:

 * **API renames and type changes**
-    - `onCallKitEvent` is now `onRingingEvent`
-    - `observeCoreCallKitEvents` is now `observeCoreRingingEvents`
-    - `observeCallAcceptCallKitEvent` is now `observeCallAcceptRingingEvent`
-    - `observeCallDeclinedCallKitEvent` is now `observeCallDeclinedRingingEvent`
-    - `observeCallEndedCallKitEvent` is now `observeCallEndedRingingEvent`
-    - The `CallKitEvent` type is now `RingingEvent`
+  - `onCallKitEvent` is now `onRingingEvent`
+  - `observeCoreCallKitEvents` is now `observeCoreRingingEvents`
+  - `observeCallAcceptCallKitEvent` is now `observeCallAcceptRingingEvent`
+  - `observeCallDeclinedCallKitEvent` is now `observeCallDeclinedRingingEvent`
+  - `observeCallEndedCallKitEvent` is now `observeCallEndedRingingEvent`
+  - The `CallKitEvent` type is now `RingingEvent`
📜 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 4f7a90c and 9202cfe.

📒 Files selected for processing (16)
  • dogfooding/pubspec.yaml (2 hunks)
  • melos.yaml (1 hunks)
  • packages/stream_video/CHANGELOG.md (2 hunks)
  • packages/stream_video/lib/globals.dart (1 hunks)
  • packages/stream_video/pubspec.yaml (1 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)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
packages/stream_video/CHANGELOG.md

11-11: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


12-12: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


14-14: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


15-15: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


16-16: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)

⏰ 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). (8)
  • GitHub Check: build
  • GitHub Check: analyze
  • GitHub Check: stream_video_noise_cancellation
  • GitHub Check: stream_video_push_notification
  • GitHub Check: stream_video
  • GitHub Check: stream_video_flutter
  • GitHub Check: analyze_legacy_version
  • GitHub Check: iOS Build (Dogfooding)
🔇 Additional comments (20)
packages/stream_video_push_notification/CHANGELOG.md (1)

3-20: Well-structured breaking changes documentation with clear migration path.

The CallKit/Ringing breaking changes section is comprehensive and provides developers with everything needed to migrate:

  • Clear explanation of the dependency removal
  • Specific configuration migration steps
  • Complete list of removed properties with context
  • Full API rename mappings

This sets a good example for communicating breaking changes in major releases.

packages/stream_video_screen_sharing/CHANGELOG.md (1)

1-3: Changelog entry properly formatted for 1.0.0 release.

The new version entry is correctly positioned at the top of the file, follows consistent markdown formatting, and aligns with the PR objective of documenting SPM support for iOS. The structure matches the existing changelog pattern.

Verify that this changelog entry captures all relevant changes for the 1.0.0 release of the stream_video_screen_sharing package. If there are additional breaking changes or features specific to this package (beyond the SPM support), ensure they are documented here.

packages/stream_video_filters/CHANGELOG.md (1)

1-1: LGTM!

The version header is correctly updated to 1.0.0, aligning with the coordinated release across all stream-video packages.

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

5-5: LGTM!

The version constant is correctly updated to 1.0.0, which will propagate to the user-agent headers used by the SDK.

melos.yaml (1)

26-30: LGTM!

The bootstrap dependency versions are correctly updated to ^1.0.0 across all stream-video packages, ensuring consistent dependency resolution throughout the monorepo.

packages/stream_video_noise_cancellation/CHANGELOG.md (1)

1-4: LGTM!

The 1.0.0 release is properly documented with the addition of Swift Package Manager support for iOS, which improves the developer experience for iOS integration.

packages/stream_video_flutter/CHANGELOG.md (1)

1-49: LGTM! Comprehensive release documentation.

The 1.0.0 changelog thoroughly documents:

  • CallKit/Ringing breaking changes with clear migration paths
  • Video filter extraction to the new stream_video_filters package
  • Swift Package Manager (SPM) support for iOS
  • Deprecated member removals with migration guidance
  • Performance improvements and bug fixes

This provides excellent guidance for developers upgrading to 1.0.0.

dogfooding/pubspec.yaml (2)

2-2: LGTM!

The dogfooding app version is appropriately updated. It's normal for the example/dogfooding app to have independent versioning from the SDK packages.


40-44: LGTM!

All stream-video dependencies are correctly updated to ^1.0.0, ensuring the dogfooding app uses the new release versions.

packages/stream_video_filters/pubspec.yaml (2)

4-4: LGTM!

The package version is correctly updated to 1.0.0, aligning with the coordinated release.


17-17: LGTM!

The stream_video dependency is correctly updated to ^1.0.0, ensuring compatibility with the new release.

packages/stream_video_noise_cancellation/pubspec.yaml (2)

4-4: Consistent version bump for coordinated 1.0.0 release.

Version is properly updated to 1.0.0, and the stream_video dependency is correctly constrained to ^1.0.0, aligning with the release across the monorepo.


17-17: Internal dependency properly updated.

stream_video dependency correctly updated to ^1.0.0 to match the major version bump in this coordinated release.

packages/stream_video_screen_sharing/pubspec.yaml (1)

5-5: Consistent version bump for coordinated 1.0.0 release.

Version is properly updated to 1.0.0, maintaining consistency with other packages in the monorepo release.

packages/stream_video_push_notification/pubspec.yaml (2)

4-4: Consistent version bump for coordinated 1.0.0 release.

Version properly updated to 1.0.0, aligning with the major version release across the monorepo.


24-25: Internal dependencies correctly updated to ^1.0.0.

Both stream_video and stream_video_flutter dependencies are properly constrained to ^1.0.0, maintaining consistency with this coordinated major version release.

packages/stream_video/pubspec.yaml (1)

4-4: Consistent version bump for coordinated 1.0.0 release.

Version properly updated to 1.0.0. As the base package, no internal stream_video dependencies are present, and external dependencies remain stable, which is appropriate for a major version release.

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

31-33: Example app dependencies properly updated for 1.0.0 release.

All three stream_video packages are consistently updated to ^1.0.0. The dependency overrides (lines 36–42) correctly retain path-based references for local development, allowing the example to test against local package changes.

packages/stream_video_flutter/pubspec.yaml (2)

4-4: Consistent version bump for coordinated 1.0.0 release.

Version properly updated to 1.0.0, maintaining alignment with the broader monorepo release.


27-27: Base dependency properly updated for major version bump.

stream_video dependency correctly constrained to ^1.0.0, ensuring the UI package stays synchronized with the base client package.

@codecov
Copy link

codecov bot commented Nov 24, 2025

Codecov Report

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

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1115   +/-   ##
=====================================
  Coverage   6.31%   6.31%           
=====================================
  Files        593     593           
  Lines      40703   40703           
=====================================
  Hits        2570    2570           
  Misses     38133   38133           

☔ 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_flutter/CHANGELOG.md (1)

24-33: Deprecated members section duplicates stream_video/CHANGELOG.md content.

The deprecated members documentation (lines 24-33) is identical to the corresponding section in packages/stream_video/CHANGELOG.md (lines 16-25). While this may be intentional for consistency across packages, consider using a shared reference or note if the content is expected to diverge in future updates.

📜 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 90ed5f6 and 97a272a.

📒 Files selected for processing (5)
  • packages/stream_video/CHANGELOG.md (2 hunks)
  • packages/stream_video_flutter/CHANGELOG.md (1 hunks)
  • packages/stream_video_noise_cancellation/CHANGELOG.md (1 hunks)
  • packages/stream_video_push_notification/CHANGELOG.md (1 hunks)
  • packages/stream_video_screen_sharing/CHANGELOG.md (1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
packages/stream_video/CHANGELOG.md

9-9: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


10-10: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


11-11: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


12-12: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


13-13: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


14-14: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


26-26: Inconsistent indentation for list items at the same level
Expected: 0; Actual: 1

(MD005, list-indent)


26-26: Unordered list indentation
Expected: 0; Actual: 1

(MD007, ul-indent)

packages/stream_video_flutter/CHANGELOG.md

11-11: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


12-12: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


14-14: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


15-15: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


16-16: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


17-17: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


18-18: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


19-19: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


37-37: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)

⏰ 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: analyze
  • GitHub Check: build
  • GitHub Check: stream_video_noise_cancellation
  • GitHub Check: stream_video
  • GitHub Check: stream_video_push_notification
  • GitHub Check: stream_video_flutter
  • GitHub Check: iOS Build (Dogfooding)
🔇 Additional comments (8)
packages/stream_video_noise_cancellation/CHANGELOG.md (1)

1-5: LGTM!

The 1.0.0 changelog entry is well-structured with clear SPM support documentation for iOS. The external link to Flutter SPM docs is helpful for users.

packages/stream_video_screen_sharing/CHANGELOG.md (1)

1-5: LGTM!

Consistent with the other package changelogs. The 1.0.0 SPM support entry is clear and includes helpful setup instructions.

packages/stream_video/CHANGELOG.md (2)

1-25: Well-documented breaking changes and new features.

The 1.0.0 changelog entry comprehensively documents the CallKit/Ringing API migration, deprecated members with clear migration paths, and new functionality. The structure is clear and helpful for users upgrading to 1.0.0.


27-37: Approve added features and fixes.

The new features (Call.ring(), audio playout control, interrupt handling) and fixes are clearly documented. The SPM support note is appropriate.

packages/stream_video_push_notification/CHANGELOG.md (2)

1-25: Comprehensive migration guide for push notifications.

The breaking changes documentation clearly outlines the shift from flutter_callkit_incoming dependency, configuration parameter changes (pushParams → pushConfiguration), and API renames. Migration paths are explicit and helpful.


3-20: API renames are consistent across all changelogs.

Verification confirms all documented API renames are consistently documented across packages/stream_video_push_notification, packages/stream_video, and packages/stream_video_flutter CHANGELOGs. Configuration and parameter changes are clearly communicated to users.

packages/stream_video_flutter/CHANGELOG.md (2)

1-53: Comprehensive 1.0.0 release notes with clear migration guidance.

The changelog documents the major migration from CallKit to Ringing API, the relocation of video filters to a separate package, and significant PiP improvements. The structure is clear and provides helpful context for users upgrading.


21-23: Clearly document the video filter package relocation.

The note that video filter functionality has moved to stream_video_filters is important for users. This migration path is well-documented and helpful.

@xsahil03x
Copy link
Member

@coderabbitai review

@coderabbitai
Copy link

coderabbitai bot commented Nov 25, 2025

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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

📜 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 97a272a and c411efb.

📒 Files selected for processing (2)
  • packages/stream_video/CHANGELOG.md (2 hunks)
  • packages/stream_video_flutter/CHANGELOG.md (1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
packages/stream_video_flutter/CHANGELOG.md

11-11: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


12-12: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


14-14: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


15-15: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


16-16: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


17-17: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


18-18: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


19-19: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


38-38: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)

packages/stream_video/CHANGELOG.md

9-9: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


10-10: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


11-11: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


12-12: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


13-13: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


14-14: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)

🔇 Additional comments (2)
packages/stream_video_flutter/CHANGELOG.md (1)

1-54: Release notes content is comprehensive and well-structured.

The v1.0.0 changelog clearly documents breaking changes with migration guidance, new features, improvements, and bug fixes. API renames are listed explicitly (e.g., onCallKitEventonRingingEvent), deprecation paths are noted, and the SPM support addition is well-explained with relevant links.

packages/stream_video/CHANGELOG.md (1)

1-37: Well-documented 1.0.0 release with clear migration guidance.

The changelog effectively communicates the major breaking changes (CallKit/Ringing API renames), provides clear deprecation guidance with migration paths, documents new features (ring specific members, audio playout controls, Android interruption handling), and includes a bug fix note. The structure is clear and aligns with the coordinated ecosystem release outlined in the PR objectives.

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

♻️ Duplicate comments (1)
packages/stream_video_flutter/CHANGELOG.md (1)

10-20: Fix markdown list indentation to conform to linting standards.

This is the same markdown indentation issue previously flagged in an earlier review. Nested list items use 4-space indentation instead of the required 2-space indentation. Apply the same fix as documented in the prior review:

* **Removed properties:**
-    * The deprecated `callerCustomizationCallback` and `backgroundVoipCallHandler` have been fully removed from `StreamVideoPushNotificationManager`.
-    * The previously used `appName` field in `pushParams` has been removed as it was deprecated. On iOS, the app's product name from build settings is now used instead.
+  * The deprecated `callerCustomizationCallback` and `backgroundVoipCallHandler` have been fully removed from `StreamVideoPushNotificationManager`.
+  * The previously used `appName` field in `pushParams` has been removed as it was deprecated. On iOS, the app's product name from build settings is now used instead.
* **API renames and type changes**
-    - `onCallKitEvent` is now `onRingingEvent`
-    - `observeCoreCallKitEvents` is now `observeCoreRingingEvents`
-    - `observeCallAcceptCallKitEvent` is now `observeCallAcceptRingingEvent`
-    - `observeCallDeclinedCallKitEvent` is now `observeCallDeclinedRingingEvent`
-    - `observeCallEndedCallKitEvent` is now `observeCallEndedRingingEvent`
-    - The `CallKitEvent` type is now `RingingEvent`
+  - `onCallKitEvent` is now `onRingingEvent`
+  - `observeCoreCallKitEvents` is now `observeCoreRingingEvents`
+  - `observeCallAcceptCallKitEvent` is now `observeCallAcceptRingingEvent`
+  - `observeCallDeclinedCallKitEvent` is now `observeCallDeclinedRingingEvent`
+  - `observeCallEndedCallKitEvent` is now `observeCallEndedRingingEvent`
+  - The `CallKitEvent` type is now `RingingEvent`
📜 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 c411efb and bdaa01a.

📒 Files selected for processing (4)
  • packages/stream_video_flutter/CHANGELOG.md (1 hunks)
  • packages/stream_video_noise_cancellation/CHANGELOG.md (1 hunks)
  • packages/stream_video_push_notification/CHANGELOG.md (1 hunks)
  • packages/stream_video_screen_sharing/CHANGELOG.md (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
  • packages/stream_video_noise_cancellation/CHANGELOG.md
  • packages/stream_video_screen_sharing/CHANGELOG.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
packages/stream_video_push_notification/CHANGELOG.md

11-11: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


12-12: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


14-14: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


15-15: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


16-16: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


17-17: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


18-18: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


19-19: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)

packages/stream_video_flutter/CHANGELOG.md

11-11: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


12-12: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


14-14: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


15-15: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


16-16: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


17-17: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


18-18: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)


19-19: Unordered list indentation
Expected: 2; Actual: 4

(MD007, ul-indent)

⏰ 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). (8)
  • GitHub Check: analyze
  • GitHub Check: stream_video_noise_cancellation
  • GitHub Check: stream_video_push_notification
  • GitHub Check: stream_video
  • GitHub Check: stream_video_flutter
  • GitHub Check: analyze_legacy_version
  • GitHub Check: build
  • GitHub Check: iOS Build (Dogfooding)

@Brazol Brazol merged commit ad461d0 into main Nov 25, 2025
16 checks passed
@Brazol Brazol deleted the release/v1.0.0 branch November 25, 2025 17:36
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