Block a user
A V3 path with bytes after its last hop names a token the router does not pay
fix: a V3 exact-out swap step shows its tokens and amounts (closes #601)
Review: FAIL (needs-rework)
src/shared/uniswap.js,decodeV3SwapExactOut(): the token spent is taken from the last 20 bytes of the path, but the router pays the token that ends the…
fix: TxApproval prices a site's ETH send from its exact amount on its own network (closes #614)
- The USD price is now described once, in the TxApproval Behavior paragraph, for both the Value and Network fee lines: the price on the network the transaction was populated on, none for a…
A V3_SWAP_EXACT_OUT step is named but never decoded, so its tokens show as not named in the calldata
Built in sneak/AutistMask#636: a V3_SWAP_EXACT_OUT step is now read the same way as a V2_SWAP_EXACT_OUT, with its path read from the token it buys to the token it…
fix: a V3 exact-out swap step shows its tokens and amounts (closes #601)
fix: TxApproval prices a site's ETH send from its exact amount on its own network (closes #614)
Review: FAIL (needs-rework)
README.md, TxApproval, Elements, the "Network fee (max)" line. The PR now prices the fee at the price on the transaction's network (showTxFee()passes the…
A V3_SWAP_EXACT_OUT step is named but never decoded, so its tokens show as not named in the calldata
Plan: one PR to next, in decode() in src/shared/uniswap.js. A decodeV3SwapExactOut() beside decodeV3SwapExactIn() reads (recipient, amountOut, amountInMax, path, payerIsUser), with the…
A V4 step shows its swap action's amount, not the larger SETTLE the user pays
A swap split across two steps the user pays for shows only the first step's amount
fix: a swap split across steps the user pays for shows every amount it spends (closes #625)
fix: a swap split across steps the user pays for shows every amount it spends (closes #625)
Review: PASS
Model: opus-5-5
fix: TxApproval prices a site's ETH send from its exact amount on its own network (closes #614)
Added: TxApproval's Network fee line now takes its USD price from the transaction's network (showTxFee() passes the chain id to formatFee()), with a test in tests/approvalValueUsd.test.js.
M…
fix: a swap split across steps the user pays for shows every amount it spends (closes #625)
Rework for sneak/AutistMask#626 (comment):
- The sentence from rule 2 now follows the three rules in README Display Consistency and in the comment above…