Skip to content

Bump getstream minimum version to 3.1.1#465

Merged
aliev merged 2 commits intomainfrom
bump/getstream-3.1.1
Apr 6, 2026
Merged

Bump getstream minimum version to 3.1.1#465
aliev merged 2 commits intomainfrom
bump/getstream-3.1.1

Conversation

@aliev
Copy link
Copy Markdown
Member

@aliev aliev commented Apr 6, 2026

Why

GetStream v3.1.1 fixes a memory leak in drain proxy cleanup. Without proper cleanup, dead proxies continue queuing video frames, causing out-of-memory errors in agents that process video feeds.

Fixes GetStream/stream-py#233

Changes

  • Bump getstream minimum version from 3.0.1 to 3.1.1

Summary by CodeRabbit

  • Chores
    • Updated dependency version to ensure compatibility and maintain stability.

GetStream v3.1.1 fixes a memory leak in drain proxy cleanup (GetStream/stream-py#233).
Without proper cleanup, dead proxies continue queuing video frames, causing OOM in
agents that process video feeds.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 6, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d433d82c-cf40-4b6c-a0ac-b7fc4c018554

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
📝 Walkthrough

Walkthrough

Updated the minimum required version of the getstream[webrtc,telemetry] dependency in agents-core from 3.0.1 to 3.1.1, maintaining the upper bound at version 4. This is a configuration-only change affecting dependency constraints.

Changes

Cohort / File(s) Summary
Dependency Version Update
agents-core/pyproject.toml
Bumped minimum version constraint for getstream[webrtc,telemetry] from >=3.0.1,<4 to >=3.1.1,<4.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

Getstream rises in minor key,
A version whisper, 3.1 calls,
Dependencies shift, austere and slight—
What ghosts in the code do we feed?
Silent, the upgrades multiply.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: bumping the getstream minimum version from 3.0.1 to 3.1.1.
Linked Issues check ✅ Passed The PR addresses the root cause of issue #233 by bumping getstream to v3.1.1, which includes the memory leak fix for drain proxy cleanup required to prevent relay queue leaks.
Out of Scope Changes check ✅ Passed The PR contains only the necessary dependency version update with no extraneous changes unrelated to the stated objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bump/getstream-3.1.1

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.

@aliev aliev marked this pull request as ready for review April 6, 2026 10:07
Copy link
Copy Markdown

@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

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@agents-core/pyproject.toml`:
- Line 24: Update the getstream dependency declaration that currently reads
getstream[webrtc,telemetry]>=3.0.1,<4 to require the patched release by changing
it to getstream[webrtc,telemetry]>=3.1.1,<4 so the buggy 3.0.x series cannot be
resolved when this plugin is installed independently; locate the dependency line
in the plugin's pyproject.toml (the getstream[webrtc,telemetry] requirement) and
update the version specifier accordingly.
🪄 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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 524c2054-9d28-430a-ac0b-5edeff3f6c2f

📥 Commits

Reviewing files that changed from the base of the PR and between 982ee4b and da1c49e.

📒 Files selected for processing (1)
  • agents-core/pyproject.toml

@aliev aliev merged commit b94e391 into main Apr 6, 2026
6 checks passed
@aliev aliev deleted the bump/getstream-3.1.1 branch April 6, 2026 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants