chore(fc): mark messages deleted upon room join; add link image preview generation behind hidden beta flag for now

Signed-off-by: Brandon McAnsh <[email protected]>

Author: bmc08gt

flipchatApp/build.gradle.kts

@@ -128,6 +128,7 @@ dependencies {
     implementation(project(":libs:messaging"))
     implementation(project(":libs:network:exchange"))
     implementation(project(":libs:network:connectivity"))
+    implementation(project(":libs:opengraph"))
     implementation(project(":libs:permissions"))
     implementation(project(":libs:quickresponse"))
     implementation(project(":libs:requests"))

flipchatApp/src/main/kotlin/xyz/flipchat/app/MainActivity.kt

@@ -9,6 +9,8 @@ import androidx.activity.compose.setContent
 import androidx.activity.enableEdgeToEdge
 import androidx.compose.runtime.CompositionLocalProvider
 import androidx.fragment.app.FragmentActivity
+import com.getcode.libs.opengraph.LocalOpenGraphParser
+import com.getcode.libs.opengraph.OpenGraphParser
 import com.getcode.network.client.Client
 import com.getcode.network.exchange.ExchangeNull
 import com.getcode.network.exchange.LocalExchange
@@ -68,6 +70,9 @@ class MainActivity : FragmentActivity() {
     @Inject
     lateinit var iapController: BillingClient
 
+    @Inject
+    lateinit var openGraphParser: OpenGraphParser
+
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         handleUncaughtException()
@@ -83,7 +88,8 @@ class MainActivity : FragmentActivity() {
                 LocalUserManager provides userManager,
                 LocalPaymentController provides paymentController,
                 LocalLabs provides betaFeatures,
-                LocalBillingClient provides iapController
+                LocalBillingClient provides iapController,
+                LocalOpenGraphParser provides openGraphParser,
             ) {
                 Rinku {
                     App(tipsEngine = tipsEngine)

flipchatApp/src/main/kotlin/xyz/flipchat/app/beta/LabsController.kt

@@ -67,6 +67,12 @@ sealed interface Lab {
         override val launched: Boolean = false
     }
 
+    data object LinkImages : Lab {
+        override val key: String = "link_image_preview_enabled"
+        override val default: Boolean = false
+        override val launched: Boolean = false
+    }
+
     companion object {
         val entries = listOf(
             ReplyToMessage,

flipchatApp/src/main/kotlin/xyz/flipchat/app/features/beta/BetaFlagsScreen.kt

@@ -106,6 +106,7 @@ private val Lab.title: String
         Lab.DeleteMessage -> "Delete Message Support"
         Lab.OpenCloseRoom -> "Open/Close Rooms"
         Lab.Tipping -> "Tipping"
+        Lab.LinkImages -> "Show Previews for Links"
     }
 
 private val Lab.message: String
@@ -117,4 +118,5 @@ private val Lab.message: String
         Lab.DeleteMessage -> "When enabled, hosts will gain the ability to delete messages"
         Lab.OpenCloseRoom -> "When enabled, hosts will gain the ability to temporarily close (and reopen) their rooms"
         Lab.Tipping -> "When enabled, you'll gain the ability to double tap messages to tip the author"
+        Lab.LinkImages -> "When enabled, links shared in chat will show a preview image for the link"
     }

flipchatApp/src/main/kotlin/xyz/flipchat/app/features/chat/conversation/ConversationChatInput.kt

@@ -202,9 +202,7 @@ private fun RoomOpenControlBar(
         )
 
         CodeButton(
-            text = stringResource(
-                if (isOpen) R.string.action_close else R.string.action_reopen
-            ),
+            text = stringResource(R.string.action_change),
             shape = CircleShape,
             buttonState = ButtonState.Filled,
             overrideContentPadding = true,

flipchatApp/src/main/kotlin/xyz/flipchat/app/features/chat/conversation/ConversationMessages.kt

@@ -163,6 +163,10 @@ internal fun ConversationMessages(
                             navigator.show(MessageTipsSheet(event.tips))
                         }
                     }
+
+                    is MessageListEvent.UnreadStateHandled -> {
+                        dispatchEvent(ConversationViewModel.Event.OnUnreadStateHandled)
+                    }
                 }
             }
         )
@@ -203,7 +207,7 @@ internal fun ConversationMessages(
         }
     }
 
-    HandleMessageChanges(listState = lazyListState, items = messages) { message ->
-        dispatchEvent(ConversationViewModel.Event.MarkDelivered(message.chatMessageId))
-    }
+//    HandleMessageChanges(listState = lazyListState, items = messages) { message ->
+//        dispatchEvent(ConversationViewModel.Event.MarkDelivered(message.chatMessageId))
+//    }
 }

flipchatApp/src/main/kotlin/xyz/flipchat/app/features/chat/conversation/ConversationViewModel.kt

@@ -115,6 +115,7 @@ class ConversationViewModel @Inject constructor(
         val replyEnabled: Boolean,
         val replyMessage: MessageReplyAnchor?,
         val startAtUnread: Boolean,
+        val unreadStateHandled: Boolean,
         val title: String,
         val imageUri: String?,
         val lastSeen: Instant?,
@@ -126,6 +127,7 @@ class ConversationViewModel @Inject constructor(
         val isRoomOpen: Boolean,
         val isOpenCloseEnabled: Boolean,
         val isTippingEnabled: Boolean,
+        val isLinkImagePreviewsEnabled: Boolean,
         val roomInfoArgs: RoomInfoArgs,
         val lastReadMessage: UUID?
     ) {
@@ -140,6 +142,7 @@ class ConversationViewModel @Inject constructor(
                 imageUri = null,
                 conversationId = null,
                 unreadCount = null,
+                unreadStateHandled = false,
                 chattableState = null,
                 lastReadMessage = null,
                 textFieldState = TextFieldState(),
@@ -156,6 +159,7 @@ class ConversationViewModel @Inject constructor(
                 isRoomOpen = true,
                 isOpenCloseEnabled = false,
                 isTippingEnabled = false,
+                isLinkImagePreviewsEnabled = false,
                 roomInfoArgs = RoomInfoArgs(),
             )
         }
@@ -169,6 +173,7 @@ class ConversationViewModel @Inject constructor(
             Event
 
         data class OnInitialUnreadCountDetermined(val count: Int) : Event
+        data object OnUnreadStateHandled: Event
         data class OnTitlesChanged(val title: String, val roomCardTitle: String) : Event
         data class OnUserActivity(val activity: Instant) : Event
         data object SendCash : Event
@@ -183,6 +188,7 @@ class ConversationViewModel @Inject constructor(
         data class OnReplyEnabled(val enabled: Boolean) : Event
         data class OnOpenCloseEnabled(val enabled: Boolean) : Event
         data class OnTippingEnabled(val enabled: Boolean) : Event
+        data class OnLinkImagePreviewsEnabled(val enabled: Boolean) : Event
         data class ReplyTo(val anchor: MessageReplyAnchor) : Event {
             constructor(chatItem: ChatItem.Message) : this(
                 MessageReplyAnchor(
@@ -259,6 +265,10 @@ class ConversationViewModel @Inject constructor(
             .onEach { dispatchEvent(Event.OnTippingEnabled(it)) }
             .launchIn(viewModelScope)
 
+        betaFeatures.observe(Lab.LinkImages)
+            .onEach { dispatchEvent(Event.OnLinkImagePreviewsEnabled(it)) }
+            .launchIn(viewModelScope)
+
         eventFlow
             .filterIsInstance<Event.OnChatIdChanged>()
             .map { it.chatId }
@@ -827,6 +837,8 @@ class ConversationViewModel @Inject constructor(
             val enableReply =
                 currentState.replyEnabled && currentState.chattableState is ChattableState.Enabled
 
+            val enableLinkImages = currentState.isLinkImagePreviewsEnabled
+
             page.map { indice ->
                 val (message, member, contents, reply, tipInfo) = indice
 
@@ -904,6 +916,7 @@ class ConversationViewModel @Inject constructor(
                     enableReply = enableReply && !message.isDeleted,
                     showTimestamp = false,
                     enableTipping = tippingEnabled,
+                    enableLinkImagePreview = enableLinkImages,
                     sender = Sender(
                         id = message.senderId,
                         profileImage = member?.imageUri.takeIf { it.orEmpty().isNotEmpty() },
@@ -1345,6 +1358,8 @@ class ConversationViewModel @Inject constructor(
                 is Event.CancelReply -> { state -> state.copy(replyMessage = null) }
                 is Event.OnOpenCloseEnabled -> { state -> state.copy(isOpenCloseEnabled = event.enabled) }
                 is Event.OnTippingEnabled -> { state -> state.copy(isTippingEnabled = event.enabled) }
+                is Event.OnLinkImagePreviewsEnabled -> { state -> state.copy(isLinkImagePreviewsEnabled = event.enabled) }
+                Event.OnUnreadStateHandled -> { state -> state.copy(unreadStateHandled = true) }
             }
         }
     }

flipchatApp/src/main/kotlin/xyz/flipchat/app/inject/AppModule.kt

@@ -9,6 +9,9 @@ import android.os.Build
 import android.os.VibratorManager
 import android.telephony.TelephonyManager
 import androidx.core.app.NotificationManagerCompat
+import com.getcode.libs.opengraph.OpenGraphCacheProvider
+import com.getcode.libs.opengraph.OpenGraphParser
+import com.getcode.libs.opengraph.cache.CacheProvider
 import com.getcode.services.analytics.AnalyticsService
 import com.getcode.services.analytics.AnalyticsServiceNull
 import com.getcode.util.locale.LocaleHelper
@@ -167,4 +170,16 @@ object AppModule {
     } else {
         GooglePlayBillingClient(context, userManager, purchaseRepository)
     }
+
+    @Singleton
+    @Provides
+    fun providesOpenGraphCache(
+        @ApplicationContext context: Context,
+    ): CacheProvider = OpenGraphCacheProvider(context)
+
+    @Singleton
+    @Provides
+    fun providesOpenGraphParser(
+        cache: CacheProvider
+    ): OpenGraphParser = OpenGraphParser(cacheProvider = cache)
 }

libs/opengraph/.gitignore

@@ -0,0 +1,2 @@
+build/
+.gradle/

libs/opengraph/build.gradle.kts

@@ -0,0 +1,64 @@
+plugins {
+    id(Plugins.android_library)
+    id(Plugins.kotlin_android)
+    id(Plugins.kotlin_kapt)
+    id(Plugins.kotlin_serialization)
+}
+
+android {
+    namespace = "${Android.codeNamespace}.libs.opengraph"
+    compileSdk = Android.compileSdkVersion
+    defaultConfig {
+        minSdk = Android.minSdkVersion
+        targetSdk = Android.targetSdkVersion
+        buildToolsVersion = Android.buildToolsVersion
+        testInstrumentationRunner = Android.testInstrumentationRunner
+    }
+
+    compileOptions {
+        sourceCompatibility(Versions.java)
+        targetCompatibility(Versions.java)
+    }
+
+    java {
+        toolchain {
+            languageVersion.set(JavaLanguageVersion.of(Versions.java))
+        }
+    }
+
+    kotlinOptions {
+        jvmTarget = Versions.java
+        freeCompilerArgs += listOf(
+            "-opt-in=kotlin.ExperimentalUnsignedTypes",
+            "-opt-in=kotlin.RequiresOptIn"
+        )
+    }
+
+    buildFeatures {
+        compose = true
+    }
+
+    composeOptions {
+        kotlinCompilerExtensionVersion = Versions.compose_compiler
+    }
+}
+
+dependencies {
+    //Jetpack compose
+    implementation(platform(Libs.compose_bom))
+    implementation(Libs.compose_ui)
+
+    implementation("org.jsoup:jsoup:1.16.1")
+    implementation(project(":libs:encryption:utils"))
+
+    implementation(Libs.kotlinx_coroutines_core)
+    implementation(Libs.kotlinx_serialization_json)
+    implementation(Libs.inject)
+
+    implementation(Libs.androidx_datastore)
+
+    implementation(Libs.hilt)
+
+    implementation(Libs.timber)
+    implementation(Libs.bugsnag)
+}