Remove unneeded CustomChatComponentFactory from compose sample

[gpunto]

Goal

This factory became unneeded after I removed pinned item customizations in #6340

Implementation

Remove the factory and update usages

🎨 UI Changes

None

Testing

The sample should work as before

Summary by CodeRabbit

• Refactor

  • Streamlined chat theme component configuration across sample app activities.
  • Updated component factory implementation in messaging views for consistency.

• Chores

  • Removed unused custom component factory code from sample application.

[github-actions]

PR checklist ✅

All required conditions are satisfied:

• Title length is OK (or ignored by label).
• At least one pr: label exists.
• Sections ### Goal, ### Implementation, and ### Testing are filled.

🎉 Great job! This PR is ready for review.

[github-actions]

SDK Size Comparison 📏

SDK	Before	After	Difference	Status
stream-chat-android-client	5.25 MB	5.82 MB	0.57 MB	🔴
stream-chat-android-ui-components	10.60 MB	10.96 MB	0.36 MB	🟡
stream-chat-android-compose	12.81 MB	12.25 MB	-0.56 MB	🚀

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 1d9a4c22-87ef-4de0-bf4b-3bc8e0497da3

📥 Commits

Reviewing files that changed from the base of the PR and between 4008e90 and 198bd32.

📒 Files selected for processing (4)

• stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/feature/channel/list/ChannelsActivity.kt
• stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/ui/MessagesActivity.kt
• stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/ui/chats/ChatsActivity.kt
• stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/ui/component/CustomChatComponentFactory.kt

💤 Files with no reviewable changes (3)

• stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/feature/channel/list/ChannelsActivity.kt
• stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/ui/chats/ChatsActivity.kt
• stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/ui/component/CustomChatComponentFactory.kt

---

Walkthrough

The sample app refactors component factory usage by removing CustomChatComponentFactory entirely. MessagesActivity switches to using LocationComponentFactory directly, while ChannelsActivity and ChatsActivity stop passing a custom factory, relying instead on ChatTheme's default component factory behavior.

Changes

Cohort / File(s)	Summary
Removed Custom Component Factory stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/feature/channel/list/ChannelsActivity.kt, stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/ui/chats/ChatsActivity.kt	Removed componentFactory = CustomChatComponentFactory() parameter from ChatTheme invocations, eliminating explicit custom component injection.
Switched to Location Factory stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/ui/MessagesActivity.kt	Replaced CustomChatComponentFactory(locationViewModelFactory = ...) with LocationComponentFactory(locationViewModelFactory = ...) in ChatTheme, including updated import.
Deleted Custom Factory Implementation stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/ui/component/CustomChatComponentFactory.kt	Removed entire file containing CustomChatComponentFactory, which previously delegated to LocationComponentFactory and overrode ChannelListItemContent to provide swipe-enabled channel list items.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A factory once grand, now fades away,
Location takes the stage in bright array,
The channels simplified, no custom cast,
Defaults reign supreme at long last! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% 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: removing the CustomChatComponentFactory from the compose sample, which aligns with the primary objective of the changeset.
Description check	✅ Passed	The description provides the required Goal and Implementation sections. UI Changes and Testing sections are included. However, it omits most template items like contributor/reviewer checklists, issue linking, and CLA confirmation.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch rm-unneeded-factory

---

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud