fix: Extract Gator Permission Expiry From Delegation Context #37874
+185
−35
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixes #464 - Permission with expiry showing 'No expiration' - Extract expiry timestamp from permissionResponse.context by decoding delegation - Find TimestampEnforcer caveat and decode the terms field (last 32 hex chars) - Convert BigInt timestamp to readable date format - Add comprehensive test coverage for all edge cases: - Valid expiry extraction - No TimestampEnforcer caveat - Invalid delegation count (0 or >1) - Invalid terms length - Zero timestamp - Decoding errors - Remove unused imports and debug logs
Contributor
|
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. |
Collaborator
✨ Files requiring CODEOWNER review ✨👨🔧 @MetaMask/core-extension-ux (2 files, +179 -29)
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
...mponents/multichain/pages/gator-permissions/components/review-gator-permission-item.test.tsx
Show resolved
Hide resolved
Collaborator
Builds ready [542ada2]
UI Startup Metrics (1214 ± 97 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
…missionItem tests - Add beforeEach hooks to configure decodeDelegations and getDeleGatorEnvironment mocks for NATIVE and ERC20 token test suites - Update test assertions to verify actual expiration date content instead of just element presence - Fix variable naming conflicts in edge case tests to avoid scope shadowing - Ensures tests correctly validate expiration date extraction from delegation context
Collaborator
Builds ready [9261685]
UI Startup Metrics (1261 ± 86 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
MoMannn
requested changes
Nov 17, 2025
Contributor
MoMannn
left a comment
MoMannn
left a comment
There was a problem hiding this comment.
Some tests are failing. Run yarn test:unit:coverage --shard=4/6 locally to check.
ui/components/multichain/pages/gator-permissions/components/review-gator-permission-item.tsx
Outdated
Show resolved
Hide resolved
Contributor
|
Would it make sense to merge this into: #37768 and have a single PR? |
- Created extractExpiryTimestampFromDelegation() in time-utils.ts - Returns expiry timestamp or 0, making it reusable across components - Simplified review-gator-permission-item.tsx from ~60 lines to ~10 lines - Moved comprehensive tests from component to utility tests - Component tests now focus on UI behavior only
Collaborator
Builds ready [741e6c2]
UI Startup Metrics (1213 ± 79 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Jest's it() function doesn't accept timeout as third parameter with async callbacks. Removed invalid timeout arguments from two test cases that were causing TypeScript errors.
Collaborator
Builds ready [d790d8a]
UI Startup Metrics (1223 ± 91 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Jest's beforeAll/afterAll don't accept timeout as second parameter with async callbacks. Removed invalid timeout arguments that were causing issues.
Collaborator
Builds ready [8596ba0]
UI Startup Metrics (1242 ± 101 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
MoMannn
requested changes
Nov 19, 2025
Collaborator
Builds ready [9e54655]
UI Startup Metrics (1234 ± 101 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
…ssions-controller
4 tasks
hanzel98
changed the title
…e utility - Use extractExpiryToReadableDate from time-utils instead of manual extraction - Remove UTC zone specification from convertTimestampToReadableDate - Reduce getExpirationDate function from 26 to 9 lines - Import GatorPermissionRule type for proper typing
Collaborator
Builds ready [574f87a]
UI Startup Metrics (1166 ± 95 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes #464 - Gator permissions with expiry were displaying "No expiration" in the UI.
Root Cause: The expiry timestamp was excluded as part of a sanitization of the permissions response because it was stored in the permission.
Solution:
Changelog
CHANGELOG entry: Fixed gator permissions displaying "No expiration" when expiry timestamp exists
Related issues
Fixes: #464
Manual testing steps
jeff.dripfund.xyzjeff.dripfund.xyzScreenshots/Recordings
Before
Permission showed "No expiration" despite having an expiry timestamp in the delegation context.
After
Permission now correctly displays the expiration date (e.g., "11/13/2026") extracted from the
TimestampEnforcercaveat.Pre-merge author checklist
Pre-merge reviewer checklist
Note
Display correct expiration dates for Gator permissions using
permissionResponse.rules; add tests and bump@metamask/gator-permissions-controllerto ^0.5.0.permissionResponse.ruleswithextractExpiryToReadableDateto render expiration inui/components/multichain/pages/gator-permissions/components/review-gator-permission-item.tsx.getExpirationDate()and remove unused warning.review-gator-permission-item.test.tsx):ruleswithexpiryto mocks and assert exact dates for native/ERC20 stream and periodic permissions.@metamask/gator-permissions-controllerfrom^0.4.0to^0.5.0(updatepackage.jsonandyarn.lock).Written by Cursor Bugbot for commit 574f87a. This will update automatically on new commits. Configure here.
Depends on this one MetaMask/core#7195 Pending to upgrade to the gator-permissions-controller: ^0.6.0