Skip to content

fix(transaction-pay-controller): allow Across perps ETH gas top-up quotes#8493

Open
pedronfigueiredo wants to merge 1 commit intomainfrom
codex/across-perps-eth-top-up
Open

fix(transaction-pay-controller): allow Across perps ETH gas top-up quotes#8493
pedronfigueiredo wants to merge 1 commit intomainfrom
codex/across-perps-eth-top-up

Conversation

@pedronfigueiredo
Copy link
Copy Markdown
Contributor

@pedronfigueiredo pedronfigueiredo commented Apr 16, 2026

Explanation

Across perps deposits originating from the PerpsController were being rejected before any quote call when the route included both the USDC deposit leg and the destination-chain ETH gas top-up leg.

The existing Across perps support check only allowed the Arbitrum USDC leg, so the ETH leg triggered quotes-strategy-unsupported and the whole route was rejected locally before reaching Across.

Widen the perps support gate to allow the native ETH top-up leg while still only normalizing the USDC leg into the HyperCore direct-deposit path. This preserves mixed USDC + ETH perps bundles and adds regression coverage for that route shape.

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Low Risk
Small, well-scoped gating change limited to perpsDeposit quotes on Arbitrum plus new unit coverage; minimal impact outside this route shape.

Overview
Enables Across perps deposit quote bundles on Arbitrum to include an additional native-token (ETH) gas top-up leg without being rejected as unsupported.

Updates isSupportedAcrossPerpsDepositRequest to accept either the Arbitrum USDC direct-deposit leg or the Arbitrum native-token leg, while normalizeAcrossRequest continues to normalize only the USDC leg to the HyperCore USDC-PERPS route. Adds regression tests for both legs and records the fix in the CHANGELOG.

Reviewed by Cursor Bugbot for commit eca412d. Bugbot is set up for automated code reviews on this repo. Configure here.

…otes

Across perps deposits originating from the PerpsController were being rejected before any quote call when the route included both the USDC deposit leg and the destination-chain ETH gas top-up leg.

The existing Across perps support check only allowed the Arbitrum USDC leg, so the ETH leg triggered quotes-strategy-unsupported and the whole route was rejected locally before reaching Across.

Widen the perps support gate to allow the native ETH top-up leg while still only normalizing the USDC leg into the HyperCore direct-deposit path. This preserves mixed USDC + ETH perps bundles and adds regression coverage for that route shape.
@pedronfigueiredo pedronfigueiredo force-pushed the codex/across-perps-eth-top-up branch from 3f95738 to eca412d Compare April 16, 2026 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant