Skip to content

Conversation

@maxkahan
Copy link
Collaborator

@maxkahan maxkahan commented Jan 30, 2026

  • When the release process was changed to match Vision Agents' release process more closely, the logic to bump the version was deleted.
  • Instead of hard-coding the version string, it now dynamically retrieves the package version.
  • Also skip specific test test_integration_test_user_details_in_response when getting a rate limit error.

Summary by CodeRabbit

  • Tests

    • Enhanced test resilience by automatically skipping tests when rate-limiting is encountered, improving reliability of the test suite during API constraint scenarios.
  • Chores

    • Updated version management to dynamically retrieve package version metadata instead of using a static string.

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

@coderabbitai
Copy link

coderabbitai bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

The pull request introduces a dynamic version retrieval mechanism in the package metadata and adds a test utility decorator to skip integration tests when rate-limit errors occur.

Changes

Cohort / File(s) Summary
Dynamic Version Retrieval
getstream/version.py
Replaced static version string with dynamic retrieval from package metadata using importlib.metadata.version().
Test Rate-Limit Handling
tests/conftest.py
Added skip_on_rate_limit decorator that catches StreamWSConnectionError and skips tests when rate-limit indicators are detected in error messages.
Decorator Application
tests/rtc/coordinator/test_connect.py
Applied skip_on_rate_limit decorator to test_integration_test_user_details_in_response test function.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A version now dances with metadata's grace,
While tests skip away from the rate-limit race,
With decorators and imports so neat,
Our package's heartbeat stays ever fleet! 🐇✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix version numbering' accurately describes the main change: replacing hard-coded version strings with dynamic package metadata retrieval in getstream/version.py.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-version-numbering

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!


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.

@maxkahan maxkahan merged commit 8b9d8c4 into main Jan 30, 2026
13 checks passed
@maxkahan maxkahan deleted the fix-version-numbering branch January 30, 2026 17:38
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