internal/handlers tests take about 26 seconds, over the 20-second target
PASS: the internal/handlers tests now run in parallel, one lock covers every use of the global viper instance, each test still has its own server and database, and the package is well under the…
Review: FAIL (needs-rework)
decode()insrc/shared/uniswap.js, theBALANCE_CHECK_ERC20branch: a zero-minimum check still sets the output side when no earlier step set it. The check…
Step 2, the routing table in the database, is up for review in sneak/routewatch#186. Nothing uses it yet.
Two figures for the later steps: reading a full table at…
Review: FAIL (needs-rework)
onChainSwitch()insrc/shared/chainSwitch.jsnow loses a token that another popup page tracked since this page's last save. Applying the switch to the…
Plan: one PR to next, in the BALANCE_CHECK_ERC20 branch of decode() in src/shared/uniswap.js. A balance check with a zero minimum never changes Token Out. A check with a positive minimum…
Review: PASS
The reworded test comment and the token and amount sentence in the PR body and commit message are accurate and resolve the earlier finding and note.
Model: opus-5-5