gnosisguild / zodiac-modifier-roles / 29594382083

96%

main: 96%

push

github

fix(app): reliable RPC for all chains + record-apply & wallet fixes (#499)

* fix(app): route wagmi transports through gnosisguild RPC

The wagmi config passed `chains` without `transports`, so viem fell back
to each chain's default public RPC (e.g. https://eth.merkle.io for
mainnet), which now returns 401/429 and blocks CORS from the browser.
Point every configured chain at https://rpc.gnosisguild.org/<chainId>.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* feat(app): skip legacy-flow modal when applying a record

Applying a record is the supported flow, but the diff page shows the
legacy-flow migration modal purely based on the post timestamp, so a
freshly applied record was hitting the warning. Have the record-apply
navigation append ?legacy-unlock, and treat that param as fully
suppressing the modal (previously it only made it dismissable).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* chore(app): bump wallet stack to latest to debug WalletConnect modal

WalletConnect connect modal hangs on loading. Bump wallet-connection deps
to latest to pick up connector fixes (notably @wagmi/connectors 7.2.1 ->
8.0.22, a major bump):

- wagmi 3.5.0 -> 3.7.1
- viem 2.47.0 -> 2.55.2
- connectkit 1.9.1 -> 1.9.2
- @tanstack/react-query 5.85.8 -> 5.101.2

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* fix(app): pin all chains to gnosisguild RPC at the chain level

Override each chain's default rpcUrls to https://rpc.gnosisguild.org/<id>
so every code path that falls back to chain.rpcUrls (rather than an
explicit transport) uses our reliable endpoint instead of viem's public
defaults (e.g. eth.merkle.io, which rate-limits and blocks CORS) — this
includes wallet add/switch-network prompts and any bare http() client.

With the chain defaults pinned, the explicit wagmi `transports` are
redundant: connectkit's getDefaultConfig falls back to http() per chain,
which now resolves to our RPC. Dro... (continued)

401 of 414 branches covered (96.86%)

Branch coverage included in aggregate %.

695 of 732 relevant lines covered (94.95%)

234.66 hits per line