fix: address PR review feedback — storage fallback, pending request leak, snapshot refresh

- loadToken() now falls back to async storage when secure store unavailable
- connect() rejects pending requests before closing socket (matches disconnect())
- requiresSnapshotRefresh returns false for thread.archived/unarchived

Author: sethwebster

apps/mobile/src/lib/orchestration.ts

@@ -393,6 +393,8 @@ export function requiresSnapshotRefresh(event: OrchestrationEvent): boolean {
     case "project.deleted":
     case "thread.created":
     case "thread.deleted":
+    case "thread.archived":
+    case "thread.unarchived":
     case "thread.meta-updated":
     case "thread.runtime-mode-set":
     case "thread.interaction-mode-set":

apps/mobile/src/lib/remoteClient.ts

@@ -214,6 +214,7 @@ export class RemoteClient {
     this.disposed = false;
     this.reconnectAttempt = 0;
     this.clearReconnectTimer();
+    this.rejectPendingRequests(new Error("Reconnecting."));
     this.closeSocket();
     this.openWebSocket();
   }

apps/mobile/src/lib/storage.ts

@@ -95,10 +95,13 @@ async function loadToken(): Promise<string> {
 
   const secureStore = await loadSecureStore();
   try {
-    return secureStore ? ((await secureStore.getItemAsync(CONNECTION_TOKEN_KEY)) ?? "") : "";
+    if (secureStore) {
+      return (await secureStore.getItemAsync(CONNECTION_TOKEN_KEY)) ?? "";
+    }
   } catch {
-    return (await readStorageItem(CONNECTION_TOKEN_KEY)) ?? "";
+    // fall through to async storage
   }
+  return (await readStorageItem(CONNECTION_TOKEN_KEY)) ?? "";
 }
 
 async function storeToken(token: string): Promise<void> {