Skip to content

Conversation

@ttypic
Copy link
Contributor

@ttypic ttypic commented Jan 21, 2026

Renamed files for consistency across SDK

Summary by CodeRabbit

  • Chores
    • Reorganized internal module structure to improve code organization and maintainability. Updated import paths across the codebase. No changes to public APIs or end-user functionality.

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

@ttypic ttypic requested a review from owenpearson January 21, 2026 18:57
@coderabbitai
Copy link

coderabbitai bot commented Jan 21, 2026

Walkthrough

Module import paths are refactored across source and test files, renaming modules from underscore-separated format (realtime_channel, default_vcdiff_decoder) to camelCase/joined format (realtimechannel, defaultvcdiffdecoder) for consistency.

Changes

Cohort / File(s) Summary
VCDiff Decoder Module Rename
ably/__init__.py
Import path for AblyVCDiffDecoder updated from ably.vcdiff.default_vcdiff_decoder to ably.vcdiff.defaultvcdiffdecoder
Realtime Channel Source Imports
ably/realtime/realtime.py,
ably/realtime/realtimepresence.py
Import paths for channel-related entities updated from ably.realtime.realtime_channel to ably.realtime.realtimechannel
Realtime Channel Test Imports
test/ably/realtime/realtimechannel_*.py
Import paths for ChannelOptions, ChannelState, and RealtimeChannel updated from ably.realtime.realtime_channel to ably.realtime.realtimechannel across four test files

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Files renamed, no underscore fuss,
Camel case paths now guide us,
Consistency blooms, consistency grows,
The refactored module tree shows,
A tidier Python, version three glows! ✨

🚥 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 describes the main change: renaming two files (realtime_channel.py and default_vcdiff_decoder.py) for consistency.
Linked Issues check ✅ Passed The PR successfully addresses the coding requirement from AIT-305 to rename realtime_channel and default_vcdiff_decoder files for consistency before Python v3 release.
Out of Scope Changes check ✅ Passed All changes are directly related to renaming two files and updating their import paths; no unrelated modifications are present.
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 docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch AIT-305/rename-files

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
Member

@owenpearson owenpearson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants