refactor: rewrite popup PendingTransactionWatcher to reactive useEffect pattern

[janek26]

Rewrite popup PendingTransactionWatcher to reactive useEffect pattern

The popup's PendingTransactionWatcher used usePoll + useWatchPendingTransactions to actively fetch receipts on a timer. Now that the background handles receipt checking, the popup no longer needs to poll — it just needs to react when the background marks a tx as confirmed.

This replaces the polling hook with a reactive useEffect that watches the pending store for non-pending (mined) transactions. When it sees one, it triggers React Query cache refetches, waits for the tx to appear in the consolidated cache, and then removes it from the pending store. The old useWatchPendingTransactions.ts (323 lines) is deleted.

Design choices & review focus

• Reactive, not polling — The useEffect fires when minedTransactions changes (derived from usePendingTransactionsStore). Since the background writes to Chrome storage and the popup subscribes via Zustand, the popup sees updates automatically without its own timer.
• Same cleanup delays — The 1.5s initial wait and 5s/16s follow-up refetches are preserved from the old implementation. These account for backend ingestion lag (the consolidated endpoint may not have the tx immediately after on-chain confirmation).
• Two removal paths preserved — Supported-chain txs are removed only after they appear in the consolidated cache. Custom-chain txs are removed immediately (they aren't in the consolidated endpoint). This prevents the tx list from flickering.
• invalidationTimeoutsRef cleanup — Timeouts are tracked in a ref and cleared on effect cleanup to prevent stale refetches if the address changes or component unmounts mid-cycle.

---

PR-Codex overview

This PR focuses on enhancing the PendingTransactionWatcher component to handle confirmed and failed transactions more effectively, updating the pending transactions in the store and improving the cleanup process for transactions.

Detailed summary

• Deleted useWatchPendingTransactions hook.
• Integrated transaction updates directly in watchPendingTransactions.
• Enhanced PendingTransactionWatcher to:
  • Use currentCurrency and supportedChainIds.
  • Refetch queries for confirmed transactions.
  • Remove transactions from the pending store based on cache checks.
  • Added cleanup delays for transaction removal.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[janek26]

• feat: implement wallet_getCapabilities #2215 : 2 dependent PRs (#2216 , #2234 )
• feat: add EIP-5792 inpage method stubs and provider upgrade #2214
• refactor: move pending tx cleanup from useEffect to store subscription #2250
• refactor: rewrite popup PendingTransactionWatcher to reactive useEffect pattern #2243 👈 (View in Graphite)
• feat: add chain-specific tiered polling schedule with timeout handling #2242
• feat: move pending transaction receipt checking to background service worker #2241
• master

---

How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

[janek26]

Merge activity

• Mar 19, 3:43 PM UTC: @janek26 merged this pull request with Graphite.