feat: adds new wallet_requestExecutionPermissions permission type token-approval-revocation#42841
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
✨ Files requiring CODEOWNER review ✨✅ @MetaMask/confirmations (4 files, +119 -25)
👨🔧 @MetaMask/core-extension-ux (6 files, +120 -99)
👨🔧 @MetaMask/delegation (10 files, +307 -75)
📜 @MetaMask/policy-reviewers (8 files, +160 -8)
Tip Follow the policy review process outlined in the LavaMoat Policy Review Process doc before expecting an approval from Policy Reviewers. |
d1b5fe2 to
736cb8b
Compare
erc20-token-revocation permission with token-approval-revocation permission
736cb8b to
523ef8a
Compare
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning MetaMask internal reviewing guidelines:
|
523ef8a to
4adf21b
Compare
…ion permission - bump @metamask/7715-permission-types from ^0.6.0 to ^0.7.1 - bump @metamask/gator-permissions-controller from ^4.10 to ^4.2.0 - bump @metamask/gator-permissions-snap from ^2.1.0 to ^2.2.0
4adf21b to
242fc25
Compare
Builds ready [aa168a3]
⚡ Performance Benchmarks (Total: 🟢 14 pass · 🟡 9 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
9f37da4 to
93a865c
Compare
93a865c to
10c928c
Compare
erc20-token-revocation permission with token-approval-revocation permissionwallet_requestExecutionPermissions permission type token-approval-revocation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ 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 30f3162. Configure here.
Builds ready [30f3162]
⚡ Performance Benchmarks (Total: 🟢 12 pass · 🟡 13 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
…g fix: destructuring.
Mrtenz
left a comment
There was a problem hiding this comment.
Policy changes look good to me.
Builds ready [db4dfe3]
⚡ Performance Benchmarks (Total: 🟢 9 pass · 🟡 16 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
db4dfe3 to
4f29680
Compare
Mrtenz
left a comment
There was a problem hiding this comment.
Policy changes look good to me.
|
Builds ready [4f29680]
⚡ Performance Benchmarks (Total: 🟢 12 pass · 🟡 12 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|




Description
Adds new permission type
token-approval-revocationthat can be granted viawallet_requestExecutionPermissions.Also bumps the following packages:
If a permission appears in permission details (Dapp Connections / All Permissions) where we do not know the permission type, we render it as "Unknown permission type".
Changelog
CHANGELOG entry: adds new ERC-7715 permission type
token-approval-revocationthat can be granted viawallet_requestExecutionPermissionsRelated issues
Fixes:
Manual testing steps
Screenshots/Recordings
All primitives enabled
Permission picker
Signature confirmation
Subset of primitives enabled
Permission picker
Signature confirmation
Dapp Connections
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Medium Risk
Updates permission-type handling and confirmation/review rendering for execution permissions, which can affect how revocation requests are described and validated to users. Risk is mitigated by test updates, but incorrect schema/mapping could lead to confusing or missing permission details.
Overview
Adds a new ERC-7715 execution permission type,
token-approval-revocation, replacing the priorerc20-token-revocationidentifier across feature flags, selectors, tests, and confirmation copy.Updates permission detail schemas/renderers to support revocation method breakdown (all primitives vs a list of enabled methods), introduces new schema element types (
raw-text,list), and adds a safe fallback schema that renders unknown permission types as “Unknown permission type” while logging via Sentry.Bumps related dependencies (
@metamask/7715-permission-types,@metamask/gator-permissions-controller,@metamask/gator-permissions-snap) and adjusts LavaMoat policies/build env (GATOR_ENABLED_PERMISSION_TYPES) accordingly.Reviewed by Cursor Bugbot for commit 4f29680. Bugbot is set up for automated code reviews on this repo. Configure here.