Skip to content

feat: classify ERC-20 send transactions - #146

Merged
0xpantera merged 1 commit into
mainfrom
feature/issue-142-erc20-send-transaction
Jun 24, 2026
Merged

feat: classify ERC-20 send transactions#146
0xpantera merged 1 commit into
mainfrom
feature/issue-142-erc20-send-transaction

Conversation

@0xpantera

Copy link
Copy Markdown
Collaborator

Summary

  • classify narrow ERC-20 transfer(address,uint256) and approve(address,uint256) calldata in eth_sendTransaction
  • keep unknown/malformed calldata fail-closed and refuse native value attached to ERC-20 calldata
  • admit ERC-20 sends/approvals in signerd as human-review transactions and broadcast token sends as exact transfer calls
  • extend extension and WalletBeat transaction QA coverage

Test Plan

  • cargo fmt --all --check
  • just check
  • cargo test --workspace
  • pnpm run qa:extension
  • pnpm run qa:walletbeat
  • pnpm run qa:walletbeat:signatures
  • pnpm run qa:walletbeat:transactions
  • git diff --check

Closes #142
Advances #93

@0xpantera
0xpantera force-pushed the feature/issue-142-erc20-send-transaction branch from 7554e94 to 27ea2bc Compare June 24, 2026 12:52
@0xpantera 0xpantera self-assigned this Jun 24, 2026
@0xpantera
0xpantera merged commit 7b1d3f6 into main Jun 24, 2026
5 checks passed
@0xpantera
0xpantera deleted the feature/issue-142-erc20-send-transaction branch June 24, 2026 13:05
hellno added a commit that referenced this pull request Jul 10, 2026
…-bridge doc (#220)

- ADR 0006: accept the shipped 127.0.0.1:8765 loopback bridge as the alpha
  transport (temporary/experimental); native messaging stays the post-audit
  target; priority is the two ends (core + extension). Resolves the #202 decision.
- browser-bridge.md: correct two stale claims — the bridge forwards the full
  EIP-1193 set (personal_sign / eth_signTypedData_v4 / eth_sendTransaction /
  EIP-5792), not 'only 3 methods', and those are no longer 'not implemented'
  (#139/#141/#146/#148, attributed via #198).
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.

Browser bridge: classify ERC-20 transfer/approve eth_sendTransaction requests

1 participant