Fix detekt violations in web_view components

Author: AlvaroBrey

purchases/src/main/kotlin/com/revenuecat/purchases/paywalls/components/WebViewComponent.kt

@@ -5,7 +5,6 @@ import com.revenuecat.purchases.InternalRevenueCatAPI
 import com.revenuecat.purchases.paywalls.components.properties.Size
 import com.revenuecat.purchases.paywalls.components.properties.SizeConstraint
 import com.revenuecat.purchases.paywalls.components.properties.SizeConstraint.Fill
-import com.revenuecat.purchases.paywalls.components.StackComponent
 import dev.drewhamilton.poko.Poko
 import kotlinx.serialization.SerialName
 import kotlinx.serialization.Serializable

ui/revenuecatui/src/main/kotlin/com/revenuecat/purchases/ui/revenuecatui/components/webview/WebViewComponentView.kt

@@ -20,8 +20,6 @@ import androidx.compose.runtime.setValue
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.viewinterop.AndroidView
 import com.revenuecat.purchases.paywalls.components.properties.Size
-import com.revenuecat.purchases.paywalls.components.properties.SizeConstraint
-import com.revenuecat.purchases.paywalls.components.properties.SizeConstraint.Fill
 import com.revenuecat.purchases.paywalls.components.properties.SizeConstraint.Fit
 import com.revenuecat.purchases.paywalls.components.properties.SizeConstraint.Fixed
 import com.revenuecat.purchases.ui.revenuecatui.components.PaywallAction
@@ -34,7 +32,7 @@ import java.net.URL
 
 @JvmSynthetic
 @Composable
-@Suppress("LongMethod")
+@Suppress("LongMethod", "ReturnCount")
 internal fun WebViewComponentView(
     style: WebViewComponentStyle,
     state: PaywallState.Loaded.Components,

ui/revenuecatui/src/main/kotlin/com/revenuecat/purchases/ui/revenuecatui/components/webview/WebViewEnvelope.kt

@@ -36,7 +36,7 @@ internal object WebViewEnvelope {
         KIND_ERROR,
     )
 
-    data class Parsed(
+    internal data class Parsed(
         val kind: String,
         val protocolVersion: Int,
         val componentId: String,
@@ -46,7 +46,7 @@ internal object WebViewEnvelope {
         val error: String?,
     )
 
-    @Suppress("ReturnCount")
+    @Suppress("ReturnCount", "CyclomaticComplexMethod")
     fun parse(rawJson: String): Parsed? {
         val json = try {
             JSONObject(rawJson)
@@ -90,6 +90,7 @@ internal object WebViewEnvelope {
         )
     }
 
+    @Suppress("LongParameterList")
     fun build(
         kind: String,
         protocolVersion: Int,

ui/revenuecatui/src/main/kotlin/com/revenuecat/purchases/ui/revenuecatui/components/webview/WebViewJavaScriptBridge.kt

@@ -114,7 +114,10 @@ internal class WebViewJavaScriptBridge(
         val webView = webViewRef.get() ?: return
 
         if (json.toByteArray(Charsets.UTF_8).size > WebViewMessageParser.MAX_PAYLOAD_BYTES) {
-            Logger.w("Dropping inbound web view message: payload exceeds ${WebViewMessageParser.MAX_PAYLOAD_BYTES} bytes.")
+            Logger.w(
+                "Dropping inbound web view message: payload exceeds " +
+                    "${WebViewMessageParser.MAX_PAYLOAD_BYTES} bytes.",
+            )
             return
         }
 
@@ -127,7 +130,8 @@ internal class WebViewJavaScriptBridge(
             WebViewEnvelope.KIND_CONNECT -> {
                 if (!isOriginTrusted(webView, allowBeforeNavigation = true)) {
                     Logger.w(
-                        "Dropping inbound web view connect: current origin does not match the resolved component origin.",
+                        "Dropping inbound web view connect: current origin does not match the " +
+                            "resolved component origin.",
                     )
                     return
                 }
@@ -138,7 +142,8 @@ internal class WebViewJavaScriptBridge(
             -> {
                 if (!isOriginTrusted(webView, allowBeforeNavigation = false)) {
                     Logger.w(
-                        "Dropping inbound web view message: current origin does not match the resolved component origin.",
+                        "Dropping inbound web view message: current origin does not match the " +
+                            "resolved component origin.",
                     )
                     return
                 }
@@ -194,6 +199,7 @@ internal class WebViewJavaScriptBridge(
     }
 
     @MainThread
+    @Suppress("ReturnCount")
     private fun handleAppFrame(rawJson: String) {
         if (!channelOpen) return
 

ui/revenuecatui/src/main/kotlin/com/revenuecat/purchases/ui/revenuecatui/components/webview/WebViewMessageParser.kt

@@ -23,7 +23,7 @@ internal object WebViewMessageParser {
     const val MAX_PAYLOAD_BYTES: Int = 65_536
     const val MAX_NESTING_DEPTH: Int = 16
 
-    data class ParsedAppMessage(
+    internal data class ParsedAppMessage(
         val message: PaywallWebViewMessage,
         /** Set when the inbound frame is a transport `request` that expects a `response`. */
         val requestId: String? = null,