Skip to content

chore: increase uniswap coverage#54

Merged
blurpesec merged 2 commits into
mainfrom
feat/categorize-uniswap-trade
May 18, 2026
Merged

chore: increase uniswap coverage#54
blurpesec merged 2 commits into
mainfrom
feat/categorize-uniswap-trade

Conversation

@blurpesec
Copy link
Copy Markdown
Contributor

@blurpesec blurpesec commented May 16, 2026

Note

Medium Risk
Updates core contract-address heuristics for Uniswap routers, which can change transaction categorization outputs across multiple chains; main risk is mislabeling or missed matches for swaps if any addresses are incorrect.

Overview
Improves Uniswap detection in tx-categorize by expanding the heuristic contract address lists for Uniswap routers (notably UNISWAP_V4_ROUTER) to cover additional networks/versions.

Consolidates Uniswap Universal Router matching under a single UNISWAP_UNIVERSAL_ROUTER name (removing the separate _2 entry), and updates Jest categorization snapshots accordingly (e.g., UNISWAP_EXCHANGE toAddressName).

Reviewed by Cursor Bugbot for commit d566755. Bugbot is set up for automated code reviews on this repo. Configure here.

@blurpesec blurpesec requested a review from a team as a code owner May 16, 2026 01:07
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit fdf32a5. Configure here.

Comment thread packages/tx-categorize/src/txSchemas/heuristicMap.ts Outdated
@blurpesec blurpesec merged commit 4cb3fbb into main May 18, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants