Simplify MessagePositionHandler logic and API

[gpunto]

🎯 Goal

While working on the message redesign, I noticed we could simplify the logic of MessagePositionHandler & related code

🛠 Implementation details

• Changed output from List<MessagePosition> to MessagePosition since a message can't be simultaneously in multiple positions and we were always using one position from the list anyway
• Refactor the logic of the default handler to be easier to understand

🎨 UI Changes

None

🧪 Testing

The message position affects the styling, so you should see no styling differences in the sample as this PR doesn't change behavior

Summary by CodeRabbit

• Refactor
  • Streamlined message position handling to use single values instead of lists for improved code clarity.
  • Updated message styling to use explicit theme parameters for consistent bubble rendering.
  • Consolidated message rendering logic across components for better maintainability.

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

[gpunto]

@coderabbitai review

[coderabbitai]

✅ 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.

[coderabbitai]

Walkthrough

This PR refactors message position handling from a list to a single enum value and updates message bubble styling APIs to accept explicit theme parameters instead of inferring them from message ownership. The groupPosition property type changes from List<MessagePosition> to MessagePosition, with cascading updates across handlers, state classes, tests, and UI components.

Changes

Cohort / File(s)	Summary
Message Bubble Theming API Updates stream-chat-android-compose/src/main/java/io/getstream/chat/android/compose/ui/components/messages/MessageBubble.kt	Changed getMessageBubbleColor signature to accept explicit MessageTheme parameter instead of deriving from message ownership; removes InternalStreamChatApi dependency.
Message Position Logic Simplification stream-chat-android-compose/src/main/java/io/getstream/chat/android/compose/ui/components/messages/PollMessageContent.kt, stream-chat-android-ui-common/src/main/kotlin/io/getstream/chat/android/ui/common/feature/messages/list/MessageListController.kt, stream-chat-android-ui-common/src/main/kotlin/io/getstream/chat/android/ui/common/feature/messages/list/MessagePositionHandler.kt	Replaced list-based position containment checks with direct single-value enum matching; handleMessagePosition now returns MessagePosition instead of List<MessagePosition>.
Message Item State Type Refactoring stream-chat-android-ui-common/src/main/kotlin/io/getstream/chat/android/ui/common/state/messages/list/MessageListItemState.kt, stream-chat-android-ui-common/api/stream-chat-android-ui-common.api	Changed MessageItemState.groupPosition from List<MessagePosition> to MessagePosition with default MessagePosition.NONE; updates constructors, copy methods, and getters accordingly.
Compose UI Integration & Theme Usage stream-chat-android-compose/src/main/java/io/getstream/chat/android/compose/ui/messages/list/MessageItem.kt	Updated getMessageBubbleShape and getMessageBubbleColor calls to pass MessageTheme parameter; refactored shape selection logic to use direct theme mapping instead of ownership-based branching.
Test Updates stream-chat-android-compose/src/test/kotlin/io/getstream/chat/android/compose/ui/messages/MessageListTest.kt, stream-chat-android-ui-common/src/test/kotlin/io/getstream/chat/android/ui/common/feature/messages/list/MessageListControllerTests.kt, stream-chat-android-ui-common/src/test/kotlin/io/getstream/chat/android/ui/common/feature/messages/list/MessagePositionHandlerTest.kt	Adjusted assertions to expect single MessagePosition values instead of lists; updated expected types in grouping and position handler tests.
UI Components Adapter stream-chat-android-ui-components/src/main/kotlin/io/getstream/chat/android/ui/utils/extensions/MessageListItem.kt	Wraps single groupPosition value in listOf() when constructing MessageListItem.MessageItem to maintain list-based downstream API.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• Update design of quoted message #6084 — Refactors MessageTheme API surface (removing quoted-related fields), directly related to theme parameter changes in message bubble functions.
• Update design of composer link preview #6066 — Expands theme-driven message bubble styling surface that this PR integrates with.

Suggested labels

compose, ui-components

Suggested reviewers

• VelikovPetar
• andremion

Poem

🐰 A single position hops so true,
No lists to confuse the message queue,
Themes now guide the bubble's art,
Ownership deferred, simplicity's heart!
Chirp chirp—the refactoring's complete! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% 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
Title check	✅ Passed	The title accurately summarizes the main change: simplifying MessagePositionHandler logic and API by converting from List to single MessagePosition.
Description check	✅ Passed	The description covers Goal and Implementation details clearly, but lacks Testing details and UI Changes are marked as None without explanation. Most critical sections are present.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

SDK Size Comparison 📏

SDK	Before	After	Difference	Status
stream-chat-android-client	5.25 MB	5.25 MB	0.00 MB	🟢
stream-chat-android-offline	5.48 MB	5.48 MB	0.00 MB	🟢
stream-chat-android-ui-components	10.62 MB	10.61 MB	-0.01 MB	🚀
stream-chat-android-compose	12.84 MB	11.64 MB	-1.20 MB	🚀

[gpunto]

@coderabbitai review

[coderabbitai]

✅ 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.

[sonarqubecloud]

Quality Gate failed

Failed conditions
77.6% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud