@mentra/crust 3.3.0-dev.839 → 3.3.0-dev.841

package/README.md

@@ -58,3 +58,11 @@ shutdown cancels only that module's outstanding host requests.
 
 Engine callers use the ordinary cloud-client API with a signal. They do not call
 native start/cancel methods or maintain their own request registry.
+
+Android native HTTP failures carry fixed `E_NATIVE_HTTP_*` codes for DNS,
+connection, timeout, TLS, socket, interrupted I/O, generic I/O, or other failures.
+The cloud client records only those allowlisted codes in its existing network
+warning; report submission still rejects with `NETWORK_ERROR`. Exception text,
+URLs, credentials and stacks are not copied into that warning. These categories
+describe a failed request; they do not establish the phone's network routing or
+why its Internet connection was unavailable.

package/android/src/main/java/com/mentra/crust/CrustModule.kt

@@ -23,6 +23,7 @@ import com.mentra.crust.jsc.JSCRuntime
 import com.mentra.crust.jsc.InstalledMiniappManifest
 import com.mentra.crust.jsc.JSCPolyfillBridge
 import com.mentra.crust.http.NativeHttpRequests
+import com.mentra.crust.http.nativeHttpFailureCode
 
 class CrustModule : Module() {
   companion object {
@@ -210,7 +211,7 @@ class CrustModule : Module() {
             )
           )
         },
-        onError = { error -> promise.reject("E_NATIVE_HTTP", error.message ?: "Native HTTP request failed", error) },
+        onError = { error -> promise.reject(nativeHttpFailureCode(error), error.message ?: "Native HTTP request failed", error) },
       )
     }
 

package/android/src/main/java/com/mentra/crust/http/NativeHttpFailure.kt

@@ -0,0 +1,31 @@
+package com.mentra.crust.http
+
+import java.io.IOException
+import java.io.InterruptedIOException
+import java.net.ConnectException
+import java.net.NoRouteToHostException
+import java.net.SocketException
+import java.net.SocketTimeoutException
+import java.net.UnknownHostException
+import javax.net.ssl.SSLException
+
+/** Fixed diagnostic codes only: exception messages can contain private URLs and addresses. */
+internal fun nativeHttpFailureCode(error: Throwable): String {
+    var current: Throwable? = error
+    var fallback = "E_NATIVE_HTTP_OTHER"
+    // Native wrappers can retain the useful cause. Bound traversal even for a cyclic cause chain.
+    repeat(8) {
+        val cause = current ?: return fallback
+        when (cause) {
+            is UnknownHostException -> return "E_NATIVE_HTTP_DNS"
+            is SocketTimeoutException -> return "E_NATIVE_HTTP_TIMEOUT"
+            is SSLException -> return "E_NATIVE_HTTP_TLS"
+            is ConnectException, is NoRouteToHostException -> return "E_NATIVE_HTTP_CONNECT"
+            is SocketException -> return "E_NATIVE_HTTP_SOCKET"
+            is InterruptedIOException -> return "E_NATIVE_HTTP_INTERRUPTED"
+            is IOException -> fallback = "E_NATIVE_HTTP_IO"
+        }
+        current = cause.cause
+    }
+    return fallback
+}

package/android/src/test/java/com/mentra/crust/http/NativeHttpFailureTest.kt

@@ -0,0 +1,44 @@
+package com.mentra.crust.http
+
+import java.io.IOException
+import java.io.InterruptedIOException
+import java.net.ConnectException
+import java.net.NoRouteToHostException
+import java.net.SocketException
+import java.net.SocketTimeoutException
+import java.net.UnknownHostException
+import javax.net.ssl.SSLHandshakeException
+import org.junit.Assert.assertEquals
+import org.junit.Test
+
+class NativeHttpFailureTest {
+    @Test fun `preserves failure category without exception text`() {
+        val privateMessage = "https://private.example?token=secret SSID private-address"
+        val cases = listOf(
+            UnknownHostException(privateMessage) to "E_NATIVE_HTTP_DNS",
+            ConnectException(privateMessage) to "E_NATIVE_HTTP_CONNECT",
+            NoRouteToHostException(privateMessage) to "E_NATIVE_HTTP_CONNECT",
+            SocketTimeoutException(privateMessage) to "E_NATIVE_HTTP_TIMEOUT",
+            SSLHandshakeException(privateMessage) to "E_NATIVE_HTTP_TLS",
+            SocketException(privateMessage) to "E_NATIVE_HTTP_SOCKET",
+            InterruptedIOException(privateMessage) to "E_NATIVE_HTTP_INTERRUPTED",
+            IOException(privateMessage) to "E_NATIVE_HTTP_IO",
+            IllegalStateException(privateMessage) to "E_NATIVE_HTTP_OTHER",
+        )
+        cases.forEach { (error, expected) -> assertEquals(expected, nativeHttpFailureCode(error)) }
+    }
+
+    @Test fun `retains a specific cause inside a generic native wrapper`() {
+        val wrapped = IOException("private wrapper", UnknownHostException("private hostname"))
+        assertEquals("E_NATIVE_HTTP_DNS", nativeHttpFailureCode(wrapped))
+        assertEquals("E_NATIVE_HTTP_IO", nativeHttpFailureCode(IllegalStateException(IOException("private"))))
+    }
+
+    @Test fun `never parses a category from private error text and bounds cyclic causes`() {
+        assertEquals("E_NATIVE_HTTP_OTHER", nativeHttpFailureCode(Exception("E_NATIVE_HTTP_DNS")))
+        val first = Exception("private one")
+        val second = Exception("private two", first)
+        first.initCause(second)
+        assertEquals("E_NATIVE_HTTP_OTHER", nativeHttpFailureCode(first))
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mentra/crust",
-  "version": "3.3.0-dev.839",
+  "version": "3.3.0-dev.841",
   "description": "Mentra Native Module",
   "main": "build/index.js",
   "types": "build/index.d.ts",
@@ -35,7 +35,7 @@
   "license": "Apache-2.0",
   "homepage": "https://github.com/Mentra-Community/MentraOS/tree/dev/mobile/modules/crust#readme",
   "dependencies": {
-    "@mentra/jspolyfill": "3.3.0-dev.839"
+    "@mentra/jspolyfill": "3.3.0-dev.841"
   },
   "devDependencies": {
     "@expo/config-plugins": ">=8.0.0",