Fix exception handling in Kotlin request executor (#1025)

Added comprehensive exception handling to prevent UnexpectedUniFFICallbackError
crashes when network exceptions occur. The executor now properly catches and
maps all exceptions to appropriate Rust error types.

Without proper exception handling, uncaught exceptions from OkHttp would crash
the UniFFI callback layer instead of being returned as proper errors to Rust.
This was causing API discovery and other network operations to fail with
UnexpectedUniFFICallbackError instead of meaningful error messages.

Changes:
- Add ConnectException handler to map connection failures to HttpError
- Add catch-all Exception handler to map unexpected errors to GenericError
- Add testLocalSite integration test to verify localhost API discovery works

Author: oguzkocer

native/kotlin/api/kotlin/src/integrationTest/kotlin/ApiUrlDiscoveryTest.kt

@@ -27,6 +27,15 @@ import kotlin.test.assertContains
 class ApiUrlDiscoveryTest {
     private val loginClient: WpLoginClient = WpLoginClient()
 
+    @Test
+    fun testLocalSite() = runTest {
+        assertEquals(
+            "http://localhost/wp-admin/authorize-application.php",
+            loginClient.apiDiscovery("http://localhost")
+                .assertSuccess().applicationPasswordsAuthenticationUrl.url()
+        )
+    }
+
     @Test // Spec Example 1
     fun testValidSiteWorksCorrectly() = runTest {
         assertEquals(

native/kotlin/api/kotlin/src/main/kotlin/rs/wordpress/api/kotlin/WpRequestExecutor.kt

@@ -25,6 +25,7 @@ import uniffi.wp_api.WpNetworkRequest
 import uniffi.wp_api.WpNetworkResponse
 import uniffi.wp_api.parseCertificate
 import java.io.File
+import java.net.ConnectException
 import java.net.NoRouteToHostException
 import java.net.UnknownHostException
 import javax.net.ssl.HttpsURLConnection
@@ -130,7 +131,9 @@ class WpRequestExecutor(
         )
     }
 
-    @Suppress("ThrowsCount")
+    // We intentionally catch all exceptions to prevent UniFFI callback crashes.
+    // All exceptions are converted to proper Rust error types rather than being swallowed.
+    @Suppress("ThrowsCount", "TooGenericExceptionCaught", "SwallowedException")
     private fun executeRequestSafely(
         urlRequest: Request,
         requestUrl: String,
@@ -162,6 +165,18 @@ class WpRequestExecutor(
             throw requestExecutionFailedWith(RequestExecutionErrorReason.unknownHost(e))
         } catch (e: NoRouteToHostException) {
             throw requestExecutionFailedWith(RequestExecutionErrorReason.noRouteToHost(e))
+        } catch (e: ConnectException) {
+            throw requestExecutionFailedWith(
+                RequestExecutionErrorReason.HttpError(
+                    reason = "Connection failed: ${e.localizedMessage}"
+                )
+            )
+        } catch (e: Exception) {
+            throw requestExecutionFailedWith(
+                RequestExecutionErrorReason.GenericError(
+                    errorMessage = e.localizedMessage ?: e.toString()
+                )
+            )
         }
     }