Skip to content

Fix: Bridge Txs#44

Merged
blurpesec merged 5 commits into
mainfrom
fix/bridge-deposit
May 4, 2026
Merged

Fix: Bridge Txs#44
blurpesec merged 5 commits into
mainfrom
fix/bridge-deposit

Conversation

@blurpesec
Copy link
Copy Markdown
Contributor

@blurpesec blurpesec commented May 4, 2026

Note

Medium Risk
Renames core Action values and many transactionType strings for bridge transactions, which can break downstream consumers expecting the old BRIDGE_IN/BRIDGE_OUT identifiers. Scope is broad but largely mechanical across schemas, heuristics, i18n, and snapshots.

Overview
Bridge tx categorization is renamed and standardized. The Action enum replaces BRIDGE_IN/BRIDGE_OUT with BRIDGE_DEPOSIT/BRIDGE_WITHDRAW, and bridge-related schema metadata (txSchemas.json, txSchemas/ethereum.ts, txSchemas/evmCompatible.ts) and heuristic determinants (txSchemas/heuristicMap.ts) are updated to emit the new action names.

User-facing strings and tests are updated to match. All locale translation keys (both localization and localizationV2) switch to the new action keys, and test fixtures/snapshots (testCases.mock.ts, txCategorize.test.ts.snap) rename affected transactionType cases (e.g., *_BRIDGE_IN*_BRIDGE_DEPOSIT, *_BRIDGE_OUT*_BRIDGE_WITHDRAW) to reflect the new categorization outputs.

Reviewed by Cursor Bugbot for commit 606fd31. 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 4, 2026 20:10
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 76dad4c. Configure here.

Comment thread packages/tx-categorize/src/localizationV2/locales/en/translation.json Outdated
erwolff
erwolff previously approved these changes May 4, 2026
@blurpesec blurpesec merged commit 20c7347 into main May 4, 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