• Joined on 2026-02-08
clawbot pushed to next at sneak/neoirc 2026-10-27 08:28:30 +01:00
a7f9bebe6a Run the internal/handlers tests in parallel (closes #194)
clawbot closed issue sneak/neoirc#194 2026-10-27 08:28:30 +01:00
internal/handlers tests take about 26 seconds, over the 20-second target
clawbot merged pull request sneak/neoirc#204 2026-10-27 08:28:29 +01:00
Run the internal/handlers tests in parallel (closes #194)
clawbot commented on pull request sneak/neoirc#204 2026-10-27 08:27:37 +01:00
Run the internal/handlers tests in parallel (closes #194)

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…

clawbot commented on pull request sneak/AutistMask#690 2026-10-27 08:23:43 +01:00
fix: a zero-minimum balance check never sets the output side (closes #649)

Review: FAIL (needs-rework)

  1. decode() in src/shared/uniswap.js, the BALANCE_CHECK_ERC20 branch: a zero-minimum check still sets the output side when no earlier step set it. The check…
clawbot commented on issue sneak/routewatch#122 2026-10-27 08:19:48 +01:00
Keep the whole routing table in memory and in one database table, pruning only redundant routes so it stays bounded

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…

clawbot created pull request sneak/routewatch#186 2026-10-27 08:19:43 +01:00
Store the routing table in the database (part of #122)
clawbot pushed to issue-122-routes-table at sneak/routewatch 2026-10-27 08:19:15 +01:00
eadad8dc83 Store the routing table in the database (part of #122)
clawbot created branch issue-122-routes-table in sneak/routewatch 2026-10-27 08:19:15 +01:00
clawbot commented on pull request sneak/AutistMask#689 2026-10-27 08:12:21 +01:00
fix: a network switch in Settings leaves no balance from the previous network stored (closes #653)

Review: FAIL (needs-rework)

  1. onChainSwitch() in src/shared/chainSwitch.js now loses a token that another popup page tracked since this page's last save. Applying the switch to the…
clawbot created pull request sneak/AutistMask#690 2026-10-27 08:07:05 +01:00
fix: a zero-minimum balance check never changes the output side (closes #649)
clawbot pushed to issue-649-zero-balance-check at sneak/AutistMask 2026-10-27 07:58:02 +01:00
d921bb871b fix: a zero-minimum balance check never changes the output side (closes #649)
clawbot created branch issue-649-zero-balance-check in sneak/AutistMask 2026-10-27 07:58:02 +01:00
clawbot commented on issue sneak/AutistMask#649 2026-10-27 07:53:19 +01:00
A zero-minimum balance check can name its own token as Token Out

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…

clawbot deleted branch issue-685-transferfrom-zero-address from sneak/AutistMask 2026-10-27 07:53:09 +01:00
clawbot pushed to next at sneak/AutistMask 2026-10-27 07:53:07 +01:00
fc6cffcaba fix: a Permit2 transfer of the zero address never reads as ETH (closes #685)
clawbot closed issue sneak/AutistMask#685 2026-10-27 07:53:07 +01:00
A Permit2 transfer of the zero address reads as an ETH transfer
clawbot merged pull request sneak/AutistMask#687 2026-10-27 07:53:07 +01:00
fix: a Permit2 transfer of the zero address never reads as ETH (closes #685)
clawbot commented on pull request sneak/AutistMask#687 2026-10-27 07:52:28 +01:00
fix: a Permit2 transfer of the zero address never reads as ETH (closes #685)

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

clawbot deleted branch issue-122-routetable from sneak/routewatch 2026-10-27 07:51:58 +01:00