Skip to content

feat: warn on infinite approval risks#155

Merged
0xpantera merged 1 commit into
mainfrom
feature/issue-151-infinite-approval-permit-warnings
Jun 25, 2026
Merged

feat: warn on infinite approval risks#155
0xpantera merged 1 commit into
mainfrom
feature/issue-151-infinite-approval-permit-warnings

Conversation

@0xpantera

Copy link
Copy Markdown
Collaborator

Summary

  • add first-class unlimited ERC-20 approval risk metadata to approve review cards
  • extract Permit-style EIP-712 owner/spender/value/deadline review metadata and flag unlimited/long-lived permits
  • update Activity rendering, WalletBeat safety matrix, and signerd-backed local-chain QA for the two WalletBeat safety: infinite approval and permit risk warnings #151 WalletBeat safety fixtures

Closes #151

Test Plan

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

@0xpantera
0xpantera force-pushed the feature/issue-151-infinite-approval-permit-warnings branch from dad9b3b to ba07f00 Compare June 25, 2026 12:17
@0xpantera 0xpantera self-assigned this Jun 25, 2026
@0xpantera
0xpantera merged commit 14e726f into main Jun 25, 2026
5 checks passed
@0xpantera
0xpantera deleted the feature/issue-151-infinite-approval-permit-warnings branch June 25, 2026 12:27
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.

WalletBeat safety: infinite approval and permit risk warnings

1 participant