KarpelesLab / libwallet / 28282266885

24%

Build Type

push

github

Committed by MagicalTux

Commit Message

wltswap/okx: broadcast swaps through OKX (Solana blockhash fix + EVM MEV)

Route OKX swap broadcasts through OKX's own broadcastTransaction/orderStatus
API instead of broadcasting from the wallet.

Solana: broadcast via Crypto/Okx:broadcastTransaction (base64 signedTx)
instead of a direct sendTransaction to our RPC. OKX's node knows the
blockhash it just embedded, eliminating the node-lag
"Transaction simulation failed: Blockhash not found" preflight rejection
(blockhash valid but our broadcast node hadn't observed it yet). Adds a
bounded retry (3x) that re-fetches the swap tx with a FRESH blockhash,
re-signs, and re-broadcasts on a blockhash/expiry error — no new user
approval since the keys are already in hand. The txid comes from the
spliced slot-0 signature, so the explorer link needs no polling.

EVM: sign locally and broadcast through OKX with enableMevProtection on
(safe to enable unconditionally — OKX ignores it on incompatible chains).
Refactors SignAndSend's EVM tail into a shared signEVM core and adds
Transaction.SignEVMRaw (sign without broadcasting) so the raw signed tx can
be handed to OKX; existing direct-broadcast callers are unchanged.

Also: solanaSplicingSignLocal returns the signature (for the local txid)
and logs sign duration on success; SwapResult gains OrderId so the host can
poll Crypto/Okx:orderStatus for final settlement.

CHANGELOG entry for 0.4.69 (pubspec bump deferred to release after CI green).

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

Coverage Stats

0 of 181 new or added lines in 3 files covered. (0.0%)

3 existing lines in 2 files now uncovered.

4560 of 19005 relevant lines covered (23.99%)

98.85 hits per line