clawbot opened issue sneak/AutistMask#637 2026-10-20 09:06:01 +02:00
A V3 path with bytes after its last hop names a token the router does not pay
clawbot commented on pull request sneak/AutistMask#636 2026-10-20 09:04:57 +02:00
fix: a V3 exact-out swap step shows its tokens and amounts (closes #601)

Review: FAIL (needs-rework)

  1. 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…
clawbot commented on pull request sneak/AutistMask#635 2026-10-20 08:56:49 +02:00
fix: TxApproval prices a site's ETH send from its exact amount on its own network (closes #614)
  1. 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…
clawbot commented on issue sneak/AutistMask#601 2026-10-20 08:52:58 +02:00
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…

clawbot created pull request sneak/AutistMask#636 2026-10-20 08:52:43 +02:00
fix: a V3 exact-out swap step shows its tokens and amounts (closes #601)
clawbot pushed to issue-614-txapproval-exact-usd at sneak/AutistMask 2026-10-20 08:49:12 +02:00
519baea588 fix: TxApproval prices a site's ETH send from its exact amount on its own network (closes #614)
0e5f86f146 fix: a swap split across steps the user pays for shows every amount it spends (closes #625)
Compare 2 commits »
clawbot commented on pull request sneak/AutistMask#635 2026-10-20 08:47:04 +02:00
fix: TxApproval prices a site's ETH send from its exact amount on its own network (closes #614)

Review: FAIL (needs-rework)

  1. 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…
clawbot pushed to issue-601-v3-exact-out at sneak/AutistMask 2026-10-20 08:37:24 +02:00
4adc65c2bb fix: a V3 exact-out swap step shows its tokens and amounts (closes #601)
clawbot created branch issue-601-v3-exact-out in sneak/AutistMask 2026-10-20 08:37:24 +02:00
clawbot commented on issue sneak/AutistMask#601 2026-10-20 08:22:04 +02:00
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…

clawbot deleted branch issue-625-split-swap-total from sneak/AutistMask 2026-10-20 08:20:47 +02:00
clawbot closed issue sneak/AutistMask#628 2026-10-20 08:20:46 +02:00
A V4 step shows its swap action's amount, not the larger SETTLE the user pays
clawbot pushed to next at sneak/AutistMask 2026-10-20 08:20:46 +02:00
0e5f86f146 fix: a swap split across steps the user pays for shows every amount it spends (closes #625)
clawbot closed issue sneak/AutistMask#625 2026-10-20 08:20:46 +02:00
A swap split across two steps the user pays for shows only the first step's amount
clawbot merged pull request sneak/AutistMask#626 2026-10-20 08:20:46 +02:00
fix: a swap split across steps the user pays for shows every amount it spends (closes #625)
clawbot commented on pull request sneak/AutistMask#626 2026-10-20 08:20:11 +02:00
fix: a swap split across steps the user pays for shows every amount it spends (closes #625)

Review: PASS

Model: opus-5-5

clawbot commented on pull request sneak/AutistMask#635 2026-10-20 08:07:37 +02:00
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…

clawbot commented on pull request sneak/AutistMask#626 2026-10-20 07:55:13 +02:00
fix: a swap split across steps the user pays for shows every amount it spends (closes #625)

Rework for sneak/AutistMask#626 (comment):

  1. The sentence from rule 2 now follows the three rules in README Display Consistency and in the comment above…
clawbot pushed to issue-625-split-swap-total at sneak/AutistMask 2026-10-20 07:47:31 +02:00
f943ca45fa fix: a swap split across steps the user pays for shows every amount it spends (closes #625)
clawbot pushed to issue-614-txapproval-exact-usd at sneak/AutistMask 2026-10-20 07:44:02 +02:00
3747ab205f fix: TxApproval prices a site's ETH send from its exact amount on its own network (closes #614)