Preconfirmation notification UX + swap speed optimizations#85
Merged
techgangboss merged 22 commits intomainfrom Mar 25, 2026
Merged
Preconfirmation notification UX + swap speed optimizations#85techgangboss merged 22 commits intomainfrom
techgangboss merged 22 commits intomainfrom
Conversation
Redesign swap notification experience to make preconfirmations the hero moment: - Celebration animation with particle burst, glow ring, and spring-bounce Fast logo - "Preconfirmed in X.Xs" speed timer — the shareable, tweet-able differentiator - Share on X button with @Fast_Protocol mention and elapsed time - Tokens Available auto-dismisses after 5s with progress bar + balance flash - Clickable balance text fills max amount (reserves 0.01 ETH for gas) - Adaptive polling: 100ms initial → 500ms (catches sub-second preconfs) - Removed redundant getFreshGasFees() call (-300ms before wallet popup) - Removed duplicate polling from useSwapConfirmation (halves API calls) - Pre-warmed tx-timeout config, abort signal propagation - Standardized "preconfirmed" (no hyphen) across all UI code Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Gas estimation can revert if the quote is stale or liquidity shifted between quote and execution. Instead of blocking the user with an opaque "Execution reverted" error before the wallet popup, fall back to a generous 500k gas limit so the tx can still be attempted. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replaces the silent gas fallback with a user-friendly error message when gas estimation reverts. The revert means the tx would fail on-chain, so we should block it — not burn the user's gas on a doomed tx. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Single card transitions in place instead of swapping between components: - Confirmed: green checkmark badge on Fast logo corner, title changes to "Tokens Available", green accent line, auto-dismiss countdown - Share on X: floating pill popup that slides in 2s after preconfirmed, positioned below the card - Dismiss: subtle 5px circle in top-right corner, only visible on hover - Explorer link: minimal icon in right column, no button chrome Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Web Audio API synthesized sound — no external files: - Quick bandpass-filtered noise sweep (whoosh, 150ms) - Rising two-note sine chime (C6→E6, ~200ms) - Total ~300ms, subtle volume, respects browser autoplay policy Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Three-note rising major arpeggio, ~350ms. Game-like dopamine hit. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Speed: bold badge to the right (blue/green bg pill with Xs value) - Explorer link: larger icon, color-matched to state (blue/green) - Share on X: flush strip below card body with border-t separator, slides in after 1.5s, feels like a natural extension of the card - Dismiss X: bare glyph, no circle/bubble, hover-only opacity Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The /api/fastswap proxy was a pure pass-through adding 100-300ms (serverless cold start + extra hop). FastRPC CORS allows direct client calls. This cuts the "Swapping..." spinner duration for ERC-20 permit path swaps. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The previous implementation used fromBalanceValue (parseFloat) which could lose precision and produce floating point artifacts like 0.08990300000000001. Now uses fromBalance.value (BigInt wei) with formatUnits for exact representation. Also fixes disabled check to use raw balance instead of parsed float. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- /api/og/preconfirm: edge-rendered OG image showing speed, token pair, fire emojis (3 for <1s, 2 for 1-4s, 1 for >4s), Fast Swaps branding - /share/preconfirm: meta page that serves OG tags for Twitter card preview, then redirects to main app - Tweet now includes fire emojis, token pair, and links to share page so Twitter renders the visual card with the speed number The OG card makes the speed claim verifiable — the number in the image matches the tweet text, adding credibility to the preconfirmation speed. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…confirmed Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
OG card: removed emoji, clean layout — speed number, "Swap Preconfirmed" label, accent line, fastprotocol.io branding. No pair names. Tweet: natural message variations based on speed tier: <1s: "Didn't even see a spinner" / "That's a Fast Swap" 1-4s: "Before the block even landed" / "Hit different" >4s: "Still faster than waiting for a block" Fire emoji at end of text only (1-3 based on speed). No URL in tweet — just the tag and the OG image card. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…thereum/@ethereumfdn Speed + mev rewards are the two highlights. Tweets now naturally weave in Ethereum mainnet context and mev surplus being returned to the swapper. Tags rotate between @ethereum and @ethereumfdn. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…or slow tier Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ring Twitter requires a URL in the tweet to render OG cards. Using the separate &url= param in the intent so the link is present but the tweet text stays clean. Over 10s tweets omit the speed number. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Console logs to measure exactly where the 10s goes: - [Permit Path] FastRPC /fastswap response time (ms) - [Toast] Dead time between placeholder creation and real hash arrival - [TxConfirmation] Time from polling start to preconfirmed detection Open browser console during a swap to see the breakdown. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The mctransactions DB lags ~15s behind the actual preconfirmation. Now polls FastRPC's mevcommit_getTransactionCommitments JSON-RPC in parallel — the node knows about commitments instantly since it received them directly from the provider. Three sources now race in parallel: 1. mevcommit_getTransactionCommitments (fastest — node has it) 2. eth_getTransactionReceipt via FastRPC 3. mctransactions DB (slowest, kept as fallback) Note: ideal solution is eth_sendRawTransactionSync which returns the commitment in the same call, but requires backend changes to the /fastswap relayer endpoint for the Permit path. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
wagmi's http() with no URL resolves to the chain's default public RPC (eth.merkle.io for mainnet) which has no CORS headers, causing ~40+ console errors per session. Alchemy + Ankr + 1rpc are sufficient fallbacks. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Need to see what the endpoint returns for Permit path txs — the 0.6s ETH path worked but the ERC-20 Permit path still took 9.2s. The node may not track commitments for relayer-submitted txs the same way. Also includes the eth.merkle.io fix from previous commit. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Strip all timing/polling debug logs added during development: [Permit Path], [Toast], [TxConfirmation], [CommitmentPoll] Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…, dead state Critical fixes from code audit: - Sanitize `time` query param in OG image + share page (parseFloat + range check) to prevent metadata injection via crafted URLs - Remove false positive fallback in fetchCommitmentStatus — only trust array results with actual commitment objects, not any truthy value - Remove dead isSuccess/isConfirming state from useSwapConfirmation — confirmation is now owned by SwapToast, these were never set after removing the duplicate polling hook Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/api/og/preconfirmmevcommit_getTransactionCommitmentsas fastest polling source — queries FastRPC node directly instead of waiting for mctransactions DB (~15s L1 inclusion lag)/api/fastswapwas a pure pass-through, FastRPC CORS allows direct client callsgetFreshGasFees()before wallet popup (result was unused, saved 200-500ms)useSwapConfirmation+SwapToastboth ran independent polling for the same hasheth.merkle.iofallback in wagmi transports (no CORS, ~40 console errors/session)Gating integrity
Zero changes to whitelisting/gating files. Verified:
SwapOrLandingGate.tsx— untoucheduse-gate-status.ts— untouchedmiddleware.ts— untouched/api/gate/*,/api/whitelist/*— untoucheduse-whitelist.ts,gate-data.ts— untouchedAll changes are inside the swap execution path (post-gate).
Known issue
ERC-20 Permit path swaps don't get actual preconfirmations — the
/fastswaprelayer appears to bypass FastRPC's commitment pipeline.mevcommit_getTransactionCommitmentsreturns errors for relayer-submitted txs. Posted to #claws for backend team investigation.Test plan
🤖 Generated with Claude Code