feat(noise): implement Noise_IK + XXfallback for WA-Web parity

agent_docs/protocol_architecture.md

@@ -125,3 +125,59 @@ wacore/src/iq/
 ```
 
 Each feature file contains: constants, enums (`StringEnum`), request/response structs (`ProtocolNode`), `IqSpec` impls, and unit tests.
+
+## Noise Handshake Patterns
+
+Three Noise patterns coexist, mirroring WA Web's `WAWebOpenChatSocket`:
+
+| Pattern        | When                                                          | State machine               | Cost                              |
+| -------------- | ------------------------------------------------------------- | --------------------------- | --------------------------------- |
+| **XX**         | First connect / pairing / forced fallback                     | `XxHandshakeState`          | 1.5 RTT                           |
+| **IK**         | Reconnect with valid cached `serverStaticPub`                 | `IkHandshakeState`          | 1 RTT, ships 0-RTT login payload  |
+| **XXfallback** | Server rejects in-flight IK (reply has `static != null`)      | `XxFallbackHandshakeState`  | 1 RTT (reuses already-sent eph.)  |
+
+### Selection (`src/handshake.rs::select_pattern`)
+
+```text
+ik_failures >= 1  ───────────────────────────────────────► XX
+no cached server_cert_chain ─────────────────────────────► XX
+leaf.not_after < now OR intermediate.not_after < now ────► XX
+otherwise ──────────────────────────────────────────────► IK with leaf.key
+```
+
+The counter `Client.ik_handshake_failures: AtomicU32` is per-process and
+not persisted (matches WA Web's `K = 0` reset on process start).
+
+### Invalidation policy
+
+| Error                                              | `ik_handshake_failures` | `server_cert_chain`                              |
+| -------------------------------------------------- | ----------------------- | ------------------------------------------------ |
+| Transient (timeout, disconnect, transport)         | unchanged               | unchanged                                        |
+| Crypto-fatal during IK (cert MAC, decrypt, proto)  | `+= 1`                  | cleared via `DeviceCommand::ClearServerCertChain`|
+| XX or XX-fallback failure                          | unchanged               | unchanged (XX never reads the cache)             |
+| Any successful handshake                           | reset to `0`            | repopulated (XX, XX-fallback) or kept (IK Continue)|
+
+Distinguishing transient from crypto-fatal is via `HandshakeError::is_transient()`
+and `HandshakeError::is_crypto_fatal()`. Getting the classification wrong leads
+to either oscillating back to XX needlessly or looping on a stale cache.
+
+### Persisted state (`Device.server_cert_chain`)
+
+`CachedServerCertChain { intermediate, leaf }` with each cert reduced to
+`{ key: [u8; 32], not_before: i64, not_after: i64 }`. Mirrors the JSON
+shape WA Web persists (`PrefsInfoStore.js:setCertificateChain`) — full
+protobuf signatures + issuer serial are validated at reception and then
+dropped, since they are not needed for IK or expiry checks.
+
+### Logs (matching WA Web's `[socket]` lines)
+
+```
+[socket] doFullHandshake: openChatSocket send hello
+[socket] resumeNoiseHandshake started
+[socket] resumeNoiseHandshake send hello
+[socket] resumeNoiseHandshake rcv hello
+[socket] resumeNoiseHandshake deriving secrets
+[socket] resumeNoiseHandshake failed: serverStaticCiphertext not null —
+  doFallbackHandshake continuing handshake with given server hello
+[socket] continueFullHandshakeCore client finish and deriving secrets
+```

src/client.rs

@@ -310,6 +310,14 @@ pub struct Client {
     /// Uses an AtomicBool instead of probing the noise_socket mutex to avoid
     /// TOCTOU races where `try_lock()` fails due to contention, not disconnection.
     is_connected: Arc<AtomicBool>,
+
+    /// Per-process counter of consecutive Noise IK handshake failures, scoped
+    /// to the lifetime of this `Client`. Mirrors `K` in WA Web's
+    /// `WAWebOpenChatSocket` (`ChatSocket.js`): on the first failure within a
+    /// process, the next connect skips IK and falls back to XX so a stale
+    /// cached `serverStaticPublic` doesn't trap us in a loop. Reset to 0 on
+    /// any successful handshake (XX, IK, or XXfallback).
+    pub(crate) ik_handshake_failures: Arc<AtomicU32>,
     /// Terminal shutdown (process-wide). Fired ONLY by `disconnect()`.
     /// Long-lived subscribers that must outlive reconnect cycles (saver,
     /// device registry cleanup) subscribe here.
@@ -732,6 +740,7 @@ impl Client {
             is_connecting: Arc::new(AtomicBool::new(false)),
             is_running: Arc::new(AtomicBool::new(false)),
             is_connected: Arc::new(AtomicBool::new(false)),
+            ik_handshake_failures: Arc::new(AtomicU32::new(0)),
             shutdown_notifier: wacore::runtime::ShutdownNotifier::new(),
             connection_shutdown: std::sync::Mutex::new(wacore::runtime::ShutdownNotifier::new()),
             last_data_received_ms: Arc::new(AtomicU64::new(0)),
@@ -1197,11 +1206,10 @@ impl Client {
         })??;
         debug!("Version fetch and transport connection established.");
 
-        let device_snapshot = self.persistence_manager.get_device_snapshot().await;
-
         let noise_socket = match handshake::do_handshake(
             self.runtime.clone(),
-            &device_snapshot,
+            &self.persistence_manager,
+            &self.ik_handshake_failures,
             transport.clone(),
             &mut transport_events,
         )