Fix error code mapping for all routing errors

The issue was that Mesh.Routing_Error enum reverse lookup by number wasn't working for all error codes (especially 32-38). This caused "error 38" to show instead of "RATE_LIMIT_EXCEEDED".

Created ROUTING_ERROR_NAMES mapping with all 17 error codes from mesh.proto:
- 0-9: NONE, NO_ROUTE, GOT_NAK, TIMEOUT, NO_INTERFACE, MAX_RETRANSMIT, NO_CHANNEL, TOO_LARGE, NO_RESPONSE, DUTY_CYCLE_LIMIT
- 32-38: BAD_REQUEST, NOT_AUTHORIZED, PKI_FAILED, PKI_UNKNOWN_PUBKEY, ADMIN_BAD_SESSION_KEY, ADMIN_PUBLIC_KEY_UNAUTHORIZED, RATE_LIMIT_EXCEEDED

Updated both DMPanel and ChatPanel formatErrorReason functions to use exact enum name matches first, then fallback to substring matches.

Now all error codes correctly map to human-readable labels:
- error 38 → "rate limited"
- error 37 → "admin no auth"
- etc.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: statico

src/ui/App.tsx

@@ -32,6 +32,27 @@ import { Logger } from "../logger";
 
 const BROADCAST_ADDR = 0xFFFFFFFF;
 
+// Map error codes to enum names for routing errors
+const ROUTING_ERROR_NAMES: Record<number, string> = {
+  0: "NONE",
+  1: "NO_ROUTE",
+  2: "GOT_NAK",
+  3: "TIMEOUT",
+  4: "NO_INTERFACE",
+  5: "MAX_RETRANSMIT",
+  6: "NO_CHANNEL",
+  7: "TOO_LARGE",
+  8: "NO_RESPONSE",
+  9: "DUTY_CYCLE_LIMIT",
+  32: "BAD_REQUEST",
+  33: "NOT_AUTHORIZED",
+  34: "PKI_FAILED",
+  35: "PKI_UNKNOWN_PUBKEY",
+  36: "ADMIN_BAD_SESSION_KEY",
+  37: "ADMIN_PUBLIC_KEY_UNAUTHORIZED",
+  38: "RATE_LIMIT_EXCEEDED",
+};
+
 type AppMode = "packets" | "nodes" | "chat" | "dm" | "config" | "log" | "meshview";
 
 export interface ChannelInfo {
@@ -500,7 +521,7 @@ export function App({ address, packetStore, nodeStore, skipConfig = false, skipN
         if (routing.variant?.case === "errorReason" && routing.variant.value !== undefined) {
           const isAck = routing.variant.value === Mesh.Routing_Error.NONE;
           const newStatus: db.MessageStatus = isAck ? "acked" : "error";
-          const errorReason = isAck ? undefined : (Mesh.Routing_Error[routing.variant.value] || `error_${routing.variant.value}`);
+          const errorReason = isAck ? undefined : (ROUTING_ERROR_NAMES[routing.variant.value] || `error_${routing.variant.value}`);
           Logger.info("App", "Received routing response", {
             requestId: packet.requestId,
             from: mp.from,

src/ui/components/ChatPanel.tsx

@@ -378,22 +378,28 @@ function MessageRow({ message, nodeStore, isOwn, isSelected, width, meshViewConf
   const formatErrorReason = (reason?: string): string => {
     if (!reason) return "failed";
     const lowerReason = reason.toLowerCase().replace(/_/g, " ");
-    if (lowerReason.includes("max retransmit")) return "max retries";
-    if (lowerReason.includes("no route")) return "no route";
-    if (lowerReason.includes("got nak")) return "rejected";
-    if (lowerReason.includes("timeout")) return "timeout";
-    if (lowerReason.includes("no interface")) return "no interface";
-    if (lowerReason.includes("too large")) return "too large";
-    if (lowerReason.includes("no channel")) return "no channel";
-    if (lowerReason.includes("duty cycle")) return "duty limit";
-    if (lowerReason.includes("bad request")) return "bad request";
-    if (lowerReason.includes("not authorized")) return "no auth";
-    if (lowerReason.includes("no response")) return "no response";
-    if (lowerReason.includes("pki")) return "pki failed";
-    if (lowerReason.includes("public key")) return "unknown key";
-    if (lowerReason.includes("admin") && lowerReason.includes("session")) return "bad session";
-    if (lowerReason.includes("admin") && lowerReason.includes("unauthorized")) return "admin no auth";
+
+    // Direct enum name matches
+    if (lowerReason === "max retransmit") return "max retries";
+    if (lowerReason === "no route") return "no route";
+    if (lowerReason === "got nak") return "rejected";
+    if (lowerReason === "timeout") return "timeout";
+    if (lowerReason === "no interface") return "no interface";
+    if (lowerReason === "too large") return "too large";
+    if (lowerReason === "no channel") return "no channel";
+    if (lowerReason === "no response") return "no response";
+    if (lowerReason === "duty cycle limit") return "duty limit";
+    if (lowerReason === "bad request") return "bad request";
+    if (lowerReason === "not authorized") return "no auth";
+    if (lowerReason === "pki failed") return "pki failed";
+    if (lowerReason === "pki unknown pubkey") return "unknown key";
+    if (lowerReason === "admin bad session key") return "bad session";
+    if (lowerReason === "admin public key unauthorized") return "admin no auth";
+    if (lowerReason === "rate limit exceeded") return "rate limited";
+
+    // Fallback to substring matches for legacy reasons
     if (lowerReason.includes("rate limit")) return "rate limited";
+
     return lowerReason.slice(0, 12);
   };
 

src/ui/components/DMPanel.tsx

@@ -245,22 +245,28 @@ function MessageRow({ message, nodeStore, isOwn, isSelected, textWidth, meshView
   const formatErrorReason = (reason?: string): string => {
     if (!reason) return "failed";
     const lowerReason = reason.toLowerCase().replace(/_/g, " ");
-    if (lowerReason.includes("max retransmit")) return "max retries";
-    if (lowerReason.includes("no route")) return "no route";
-    if (lowerReason.includes("got nak")) return "rejected";
-    if (lowerReason.includes("timeout")) return "timeout";
-    if (lowerReason.includes("no interface")) return "no interface";
-    if (lowerReason.includes("too large")) return "too large";
-    if (lowerReason.includes("no channel")) return "no channel";
-    if (lowerReason.includes("duty cycle")) return "duty limit";
-    if (lowerReason.includes("bad request")) return "bad request";
-    if (lowerReason.includes("not authorized")) return "no auth";
-    if (lowerReason.includes("no response")) return "no response";
-    if (lowerReason.includes("pki")) return "pki failed";
-    if (lowerReason.includes("public key")) return "unknown key";
-    if (lowerReason.includes("admin") && lowerReason.includes("session")) return "bad session";
-    if (lowerReason.includes("admin") && lowerReason.includes("unauthorized")) return "admin no auth";
+
+    // Direct enum name matches
+    if (lowerReason === "max retransmit") return "max retries";
+    if (lowerReason === "no route") return "no route";
+    if (lowerReason === "got nak") return "rejected";
+    if (lowerReason === "timeout") return "timeout";
+    if (lowerReason === "no interface") return "no interface";
+    if (lowerReason === "too large") return "too large";
+    if (lowerReason === "no channel") return "no channel";
+    if (lowerReason === "no response") return "no response";
+    if (lowerReason === "duty cycle limit") return "duty limit";
+    if (lowerReason === "bad request") return "bad request";
+    if (lowerReason === "not authorized") return "no auth";
+    if (lowerReason === "pki failed") return "pki failed";
+    if (lowerReason === "pki unknown pubkey") return "unknown key";
+    if (lowerReason === "admin bad session key") return "bad session";
+    if (lowerReason === "admin public key unauthorized") return "admin no auth";
+    if (lowerReason === "rate limit exceeded") return "rate limited";
+
+    // Fallback to substring matches for legacy reasons
     if (lowerReason.includes("rate limit")) return "rate limited";
+
     return lowerReason.slice(0, 12);
   };