vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-07-08 14:53:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(android): preserve numeric invoke error codes (#99591)

Browse Source 
* fix(android): preserve numeric invoke error codes

* fix(android): tighten invoke error code parsing

---------

Co-authored-by: Peter Steinberger <steipete@gmail.com>
This commit is contained in:
Ben.Li
2026-07-04 02:24:50 +08:00
committed by GitHub
parent 43a7fd3839
commit df1452799b
2 changed files with 23 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
apps/android/app/src/main/java/ai/openclaw/app/gateway/InvokeErrorParser.kt
View File

@@ -1,5 +1,7 @@
package ai.openclaw.app.gateway
private val invokeErrorCodePattern = Regex("^[A-Z][A-Z0-9_]*$")
data class ParsedInvokeError(
val code: String,
val message: String,
@@ -24,7 +26,7 @@ fun parseInvokeErrorMessage(raw: String): ParsedInvokeError {
if (parts.size == 2) {
val code = parts[0].trim()
val rest = parts[1].trim()
if (code.isNotEmpty() && code.all { it.isUpperCase() || it == '_' }) {
if (invokeErrorCodePattern.matches(code)) {
return ParsedInvokeError(
code = code,
message = rest.ifEmpty { trimmed },

24
apps/android/app/src/test/java/ai/openclaw/app/gateway/InvokeErrorParserTest.kt
View File

@@ -15,12 +15,28 @@ class InvokeErrorParserTest {
assertEquals("CAMERA_PERMISSION_REQUIRED: grant Camera permission", parsed.prefixedMessage)
}
@Test
fun parseInvokeErrorMessage_parsesNumericCodePrefix() {
val parsed = parseInvokeErrorMessage("A2UI_HOST_UNAVAILABLE: bundled A2UI host not reachable")
assertEquals("A2UI_HOST_UNAVAILABLE", parsed.code)
assertEquals("bundled A2UI host not reachable", parsed.message)
assertTrue(parsed.hadExplicitCode)
}
@Test
fun parseInvokeErrorMessage_rejectsNonCanonicalCodePrefix() {
val parsed = parseInvokeErrorMessage("IllegalStateException: boom")
assertEquals("UNAVAILABLE", parsed.code)
assertEquals("IllegalStateException: boom", parsed.message)
assertFalse(parsed.hadExplicitCode)
listOf(
"IllegalStateException: boom",
"2FAST: boom",
"_PRIVATE: boom",
"CAMERA-PERMISSION: boom",
)
.forEach { raw ->
val parsed = parseInvokeErrorMessage(raw)
assertEquals("UNAVAILABLE", parsed.code)
assertEquals(raw, parsed.message)
assertFalse(parsed.hadExplicitCode)
}
}
@Test