feat(STX-472): add gas-included metrics properties#28571
feat(STX-472): add gas-included metrics properties#28571rarquevaux wants to merge 1 commit intomainfrom
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. |
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection: This hook is used in No other areas are impacted: no navigation changes, no controller changes, no shared components affected. The test file changes are unit test updates to verify the new analytics assertions. Risk is low since the functional behavior is unchanged. Performance Test Selection: |
|
|
✅ E2E Fixture Validation — Schema is up to date |



Description
Same changes than MetaMask/metamask-extension#41581
Changelog
CHANGELOG entry: add gas-included metrics properties
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
Low risk: adds a confirmation metrics side-effect alongside existing auto-selection logic and updates unit tests accordingly, with minimal impact on transaction behavior.
Overview
Adds confirmation metrics when
useAutomaticGasFeeTokenSelectautomatically picks a gas fee token. When auto-selection triggers, it now callssetConfirmationMetricwithautomatic_fee_token_selection: trueand the selected token symbol, and expands the effect dependencies accordingly.Updates
useAutomaticGasFeeTokenSelect.test.tsto mockuseConfirmationMetricEventsand assert metrics are emitted only when an automatic selection occurs (and not on other paths or rerenders).Reviewed by Cursor Bugbot for commit 82c4bc8. Bugbot is set up for automated code reviews on this repo. Configure here.