When another NWC client (e.g. Alby) connected to the same relay calls `get_info`, Damus receives the response and previously threw a DecodingError.typeMismatch, causing an "Oops" error dialog to be shown. Fix: Make `result_type` optional in `WalletConnect.Response`. Unknown result types now decode without throwing — `result_type` and `result` are set to `nil`, and the rest of the existing nil-guarded code paths handle this silently. Adds a test to verify `get_info` (and any future unknown result type) is decoded gracefully. Closes: #2204 Changelog-Fixed: Fixed issue where the app could display an error message when using another NWC wallet in parallel Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: danieldaquino <24692108+danieldaquino@users.noreply.github.com> Signed-off-by: Daniel D’Aquino <daniel@daquino.me>
12 KiB
12 KiB