fix: align perps advanced chart price precision

[geositta]

Description

This PR aligns Perps Advanced Charts price precision with the existing Lightweight Charts behavior, especially for markets with very small prices like PEPE and market specific price precision like LIT. Advanced Charts now receives Perps market precision from Hyperliquid szDecimals, uses that precision for price scale labels and price pills, and resolves Perps symbols with enough internal price scale precision for tiny-price assets. Auto-close TP/SL labels also use market-aware Perps price formatting so values like SL $2.1946 are preserved instead of being rounded to two decimals.

Changelog

CHANGELOG entry: Fixed Y-axis labels and price pill parity on perps Advanced Charts

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/TAT-3499

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I've included tests if applicable
• I've documented my code using JSDoc format if applicable
• I've applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Performance checks (if applicable)

• I've tested on Android
  • Ideally on a mid-range device; emulator is acceptable
• I've tested with a power user scenario
  • Use these power-user SRPs to import wallets with many accounts and tokens
• I've instrumented key operations with Sentry traces for production performance metrics
  • See trace() for usage and addToken for an example

For performance guidelines and tooling, see the Performance Guide.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

---

Note

Medium Risk
Touches shared Advanced Chart WebView logic and Perps market-details display; behavior is opt-in via priceDecimals/szDecimals with broad test coverage and no auth or payment paths.

Overview
Perps Advanced Charts now match Lightweight Charts and Hyperliquid price rules by threading market szDecimals into priceDecimals (6 - szDecimals) on AdvancedChart, including Y-axis labels, last-price pills, and symbol pricescale / variable_tick_size when precision is configured.

AdvancedChart gains an optional priceDecimals prop baked into WebView CONFIG; the WebView applies custom price formatters (alongside existing subscript notation) and a perps-oriented tick ladder for tiny and large prices.

Perps UI forwards szDecimals from market details to PerpsAdvancedChart and PerpsPositionCard. Auto-close TP/SL copy switches from generic fiat rounding to formatPerpsPrice so values like $2.1946 are not collapsed to two decimals.

^{Reviewed by Cursor Bugbot for commit eabc62a. Bugbot is set up for automated code reviews on this repo. Configure here.}

[github-actions]

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.

[mm-token-exchange-service]

PR template — items to address before "Ready for review"

Warnings — informational, address before merging:

• Manual testing steps still contain template content (the Gherkin example title or a [...] placeholder). Replace with real steps, or write N/A — <reason>.
• Pre-merge author checklist has unchecked items (e.g. "I've tested on Android"). Every box must be consciously checked — see docs/readme/ready-for-review.md.

See docs/readme/ready-for-review.md for the full Definition of Ready for Review.

[github-actions]

🔍 Smart E2E Test Selection

• Selected E2E tags: SmokePerps, SmokeWalletPlatform, SmokeConfirmations
• Selected Performance tags: @PerformancePreps
• Risk Level: medium
• AI Confidence: 88%

click to see 🤖 AI reasoning details

E2E Test Selection:
All 15 changed files are within the Perps feature area and the shared AdvancedChart component:

1. Perps price precision feature: The PR adds szDecimals (Hyperliquid size decimals) propagation from PerpsMarketDetailsView → PerpsAdvancedChart and PerpsPositionCard. This drives a new priceDecimals prop in the chart and a new formatPerpsPrice() utility for TP/SL display.

2. AdvancedChart changes: New priceDecimals prop added to AdvancedChartProps, AdvancedChartTemplate, and the WebView chart logic (chartLogic.js/chartLogicString.ts). New functions getConfiguredPriceDecimals(), formatPriceWithConfiguredDecimals(), PERPS_VARIABLE_TICK_SIZE, getPriceScale(), getVariableTickSize() affect chart price scale and tick size when priceDecimals is configured. The priceDecimals prop is optional and defaults to null, so non-Perps chart usage (Predictions, Tokens) is unaffected.

3. formatUtils.ts: New formatPerpsPrice() function using formatHyperLiquidPrice from @metamask/perps-controller for accurate Hyperliquid price precision.

Tag selection rationale:

• SmokePerps: Directly tests Perps functionality including market details view, position cards, and chart display — all directly modified.
• SmokeWalletPlatform: Per tag description, Perps is a section inside the Trending tab; changes to Perps views (PerpsMarketDetailsView, PerpsPositionCard) affect Trending.
• SmokeConfirmations: Per SmokePerps tag description, "Add Funds deposits are on-chain transactions — when selecting SmokePerps, also select SmokeConfirmations."

No changes to core Engine, controllers, navigation, browser, accounts, swaps, staking, or other unrelated areas.

Performance Test Selection:
The PerpsMarketDetailsView and PerpsAdvancedChart are directly modified with new price formatting logic and chart configuration changes (new tick size arrays, price scale computation). The @PerformancePreps tag covers perps market loading and position management, which includes the market details view. While the changes are primarily display/formatting improvements, the chart WebView logic changes (new PERPS_VARIABLE_TICK_SIZE, getPriceScale, getVariableTickSize) could affect chart rendering performance in the perps market details view. Conservatively selecting this tag to validate no performance regression.

View GitHub Actions results

[cursor]

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 eabc62a. Configure here.}

[cursor]

Chart subscript diverges TP labels

Medium Severity

With perps priceDecimals enabled, the Advanced Chart custom formatter still prefers subscript notation for very small prices before market-aware decimals run. Auto-close TP/SL now uses formatPerpsPrice / Hyperliquid formatting without subscript. Y-axis and price pills can disagree with TP/SL text on tiny-price markets like PEPE.

Additional Locations (1)

• app/components/UI/Perps/components/PerpsPositionCard/PerpsPositionCard.tsx#L523-L533

 

^{Reviewed by Cursor Bugbot for commit eabc62a. Configure here.}

[sonarqubecloud]

Quality Gate failed

Failed conditions
64.1% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

[github-actions]

⚡ Performance Test Results

ℹ️ Performance test results are currently non-blocking and will not block this PR.

✅ All tests passed · 2 tests · 1 device

📱 Devices tested (1)

Android: Google Pixel 8 Pro (v14.0)

✅ Passed Tests (2)

Test	Platform	Device	Duration	Team	Recording
Perps add funds	Android	Google Pixel 8 Pro (v14.0)	9.11s	@mm-perps-engineering-team	📹 Watch
Perps open position and close it	Android	Google Pixel 8 Pro (v14.0)	16.76s	@mm-perps-engineering-team	📹 Watch

---

Branch: perps/fix-adv-charts-y-axis · Build: Normal · Commit: 2d2f214 · View full run