refactor(common): pass HTMLConverter down as a parameter

android/src/main/kotlin/dev/msfjarvis/claw/android/ui/LobstersApp.kt

@@ -38,7 +38,6 @@ import dev.msfjarvis.claw.android.viewmodel.ClawViewModel
 import dev.msfjarvis.claw.api.LobstersApi
 import dev.msfjarvis.claw.common.comments.CommentsPage
 import dev.msfjarvis.claw.common.comments.HTMLConverter
-import dev.msfjarvis.claw.common.comments.LocalHTMLConverter
 import dev.msfjarvis.claw.common.res.ClawIcons
 import dev.msfjarvis.claw.common.theme.LobstersTheme
 import dev.msfjarvis.claw.common.ui.decorations.ClawAppBar
@@ -75,11 +74,7 @@ fun LobstersApp(
 
   LobstersTheme(
     dynamicColor = true,
-    providedValues =
+    providedValues = arrayOf(LocalUriHandler provides urlLauncher),
-      arrayOf(
-        LocalUriHandler provides urlLauncher,
-        LocalHTMLConverter provides htmlConverter,
-      ),
   ) {
     val currentUiMode = LocalConfiguration.current.uiMode
     val systemBarsColor = MaterialTheme.colorScheme.surfaceColorAtNavigationBarElevation()
@@ -211,6 +206,7 @@ fun LobstersApp(
             postId = postId,
             getDetails = viewModel::getPostComments,
             postActions = postActions,
+            htmlConverter = htmlConverter,
           )
         }
         composable(

common/src/main/kotlin/dev/msfjarvis/claw/common/comments/CommentEntry.kt

@@ -49,9 +49,9 @@ import java.time.temporal.TemporalAccessor
 fun CommentsHeader(
   postDetails: ExtendedPostDetails,
   postActions: PostActions,
+  htmlConverter: HTMLConverter,
   modifier: Modifier = Modifier,
 ) {
-  val htmlConverter = LocalHTMLConverter.current
   val uriHandler = LocalUriHandler.current
 
   Surface(color = MaterialTheme.colorScheme.background, modifier = modifier) {
@@ -133,10 +133,10 @@ fun PostLink(
 @Composable
 fun CommentEntry(
   comment: Comment,
+  htmlConverter: HTMLConverter,
   modifier: Modifier = Modifier,
 ) {
   var expanded by remember(comment) { mutableStateOf(true) }
-  val htmlConverter = LocalHTMLConverter.current
   val uriHandler = LocalUriHandler.current
   Box(
     modifier =

common/src/main/kotlin/dev/msfjarvis/claw/common/comments/Comments.kt

@@ -30,11 +30,18 @@ import dev.msfjarvis.claw.model.ExtendedPostDetails
 private fun CommentsPageInternal(
   details: ExtendedPostDetails,
   postActions: PostActions,
+  htmlConverter: HTMLConverter,
   modifier: Modifier = Modifier,
 ) {
   Surface(color = MaterialTheme.colorScheme.surfaceVariant) {
     LazyColumn(modifier = modifier, contentPadding = PaddingValues(bottom = 24.dp)) {
-      item { CommentsHeader(postDetails = details, postActions = postActions) }
+      item {
+        CommentsHeader(
+          postDetails = details,
+          postActions = postActions,
+          htmlConverter = htmlConverter,
+        )
+      }
 
       if (details.comments.isNotEmpty()) {
         item {
@@ -49,7 +56,7 @@ private fun CommentsPageInternal(
           if (index != 0) {
             Divider()
           }
-          CommentEntry(item)
+          CommentEntry(comment = item, htmlConverter = htmlConverter)
         }
       } else {
         item {
@@ -72,6 +79,7 @@ fun CommentsPage(
   postId: String,
   getDetails: suspend (String) -> ExtendedPostDetails,
   postActions: PostActions,
+  htmlConverter: HTMLConverter,
   modifier: Modifier = Modifier,
 ) {
   val postDetails by
@@ -86,9 +94,10 @@ fun CommentsPage(
   when (postDetails) {
     is Success<*> -> {
       CommentsPageInternal(
-        (postDetails as Success<ExtendedPostDetails>).data,
+        details = (postDetails as Success<ExtendedPostDetails>).data,
-        postActions,
+        postActions = postActions,
-        modifier.fillMaxSize(),
+        htmlConverter = htmlConverter,
+        modifier = modifier.fillMaxSize(),
       )
     }
     is Error -> {

common/src/main/kotlin/dev/msfjarvis/claw/common/comments/HTMLConverter.kt

@@ -1,10 +1,6 @@
 package dev.msfjarvis.claw.common.comments
 
-import androidx.compose.runtime.staticCompositionLocalOf
-
 /** Defines a contract to convert strings of HTML to Markdown. */
 fun interface HTMLConverter {
   fun convertHTMLToMarkdown(html: String): String
 }
-
-val LocalHTMLConverter = staticCompositionLocalOf<HTMLConverter> { error("To be provided") }

detekt-baselines/common.xml

@@ -2,7 +2,6 @@
 <SmellBaseline>
   <ManuallySuppressedIssues></ManuallySuppressedIssues>
   <CurrentIssues>
-    <ID>CompositionLocalAllowlist:HTMLConverter.kt$LocalHTMLConverter</ID>
     <ID>MagicNumber:CommentEntry.kt$16</ID>
     <ID>MagicNumber:LobstersCard.kt$50</ID>
     <ID>MagicNumber:LobstersCard.kt$8</ID>