fix(android): keep BaseUrlInterceptor.intercept under detekt ReturnCount

aec10ce7 added a third early return (the placeholder-host bail)
on top of the existing unparseable-baseUrl elvis return, tripping
detekt's ReturnCount ceiling of 2 on the dev test workflow.

Refactor: keep the placeholder-host early bail (it preserves the
no-op cost for external URLs — no AuthStore read, no URL parse),
fold the unparseable-baseUrl case into a `?:` that falls back to
the original URL. Result is two returns and identical observable
behavior — placeholder hosts get rewritten when baseUrl parses,
fall through unchanged when it doesn't.

Existing unit tests cover all three paths and continue to assert
the same outputs.

android/app/src/main/java/com/fabledsword/minstrel/api/BaseUrlInterceptor.kt

@@ -38,14 +38,18 @@ class BaseUrlInterceptor @Inject constructor(
 
     override fun intercept(chain: Interceptor.Chain): Response {
         val original = chain.request()
+        // Early-bail keeps the no-op path for external URLs cheap
+        // (no AuthStore read, no URL parse).
         if (original.url.host != PLACEHOLDER_HOST) return chain.proceed(original)
-        val baseUrl = authStore.baseUrl.value.toHttpUrlOrNull()
-            ?: return chain.proceed(original)
-        val rewritten: HttpUrl = original.url.newBuilder()
-            .scheme(baseUrl.scheme)
-            .host(baseUrl.host)
-            .port(baseUrl.port)
-            .build()
+        // Unparseable baseUrl folds into the same proceed path — keeps
+        // detekt's ReturnCount happy by avoiding a second early return.
+        val rewritten: HttpUrl = authStore.baseUrl.value.toHttpUrlOrNull()?.let { baseUrl ->
+            original.url.newBuilder()
+                .scheme(baseUrl.scheme)
+                .host(baseUrl.host)
+                .port(baseUrl.port)
+                .build()
+        } ?: original.url
         return chain.proceed(original.newBuilder().url(rewritten).build())
     }