Commit 7418947
chore: bump ramps-controller to 13.2.0 (#28879)
<!--
Please submit this PR as a draft initially.
Do not mark it as "Ready for review" until the template has been
completely filled out, and PR status checks have passed at least once.
-->
## **Description**
Bumps `@metamask/ramps-controller` from `^13.1.0` to `^13.2.0` to pick
up the `TransakService.verifyUserOtp` no-retry policy fix (TRAM-3442).
**What changed in ramps-controller 13.2.0:**
- **Fixed** — `TransakService.verifyUserOtp` no longer retries on
failure, preventing single-use OTP attempts from being silently consumed
when consumers configure a non-zero `maxRetries` in `policyOptions`
([core#8468](MetaMask/core#8468))
- **Changed** — Bump `@metamask/base-controller` from `^9.0.1` to
`^9.1.0` ([core#8457](MetaMask/core#8457))
**Core release PR:** MetaMask/core#8476
**Scope of this PR:** bumps the `@metamask/ramps-controller` dependency
version in `package.json`. No mobile source code changes.
## **Changelog**
CHANGELOG entry: null
## **Related issues**
Fixes:
[TRAM-3442](https://consensyssoftware.atlassian.net/browse/TRAM-3442)
## **Manual testing steps**
```gherkin
Feature: OTP verification no-retry policy
Scenario: user enters an invalid OTP during Transak onramp
Given user has initiated a Transak fiat-to-crypto purchase
And user has received an OTP via SMS/email
When user enters an incorrect OTP code
Then the app surfaces the verification failure immediately
And no automatic retry is performed against the Transak API
And the user's remaining OTP attempt count decrements by exactly one
Scenario: user enters a valid OTP during Transak onramp
Given user has initiated a Transak fiat-to-crypto purchase
And user has received an OTP via SMS/email
When user enters the correct OTP code
Then verification succeeds on the first attempt
And the onramp flow proceeds to the next step
```
## **Screenshots/Recordings**
N/A — dependency bump only, no UI changes.
## **Pre-merge author checklist**
- [x] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I've included tests if applicable — N/A, dependency bump only
- [x] I've documented my code using [JSDoc](https://jsdoc.app/) format
if applicable — N/A
- [x] I've applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.
## **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.
[TRAM-3442]:
https://consensyssoftware.atlassian.net/browse/TRAM-3442?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> **Medium Risk**
> Dependency-only change, but it impacts the fiat ramps stack; behavior
changes in the controller could affect onramp/offramp flows and should
be smoke-tested.
>
> **Overview**
> Updates the `@metamask/ramps-controller` dependency from `^13.1.0` to
`^13.2.0`.
>
> Regenerates `yarn.lock` to pull in `@metamask/ramps-controller@13.2.0`
and its updated transitive deps (notably
`@metamask/base-controller@9.1.0` / `@metamask/messenger@^1.1.1`), with
no application source changes.
>
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
9ea81a7. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
---------
Co-authored-by: Darius Costolas <10818970+meltingice1337@users.noreply.github.com>
Co-authored-by: Amitabh Aggarwal <aggarwal.amitabh@gmail.com>1 parent ff903fd commit 7418947
2 files changed
Lines changed: 12 additions & 12 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
293 | 293 | | |
294 | 294 | | |
295 | 295 | | |
296 | | - | |
| 296 | + | |
297 | 297 | | |
298 | 298 | | |
299 | 299 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8099 | 8099 | | |
8100 | 8100 | | |
8101 | 8101 | | |
8102 | | - | |
8103 | | - | |
8104 | | - | |
| 8102 | + | |
| 8103 | + | |
| 8104 | + | |
8105 | 8105 | | |
8106 | | - | |
| 8106 | + | |
8107 | 8107 | | |
8108 | 8108 | | |
8109 | | - | |
| 8109 | + | |
8110 | 8110 | | |
8111 | 8111 | | |
8112 | 8112 | | |
| |||
9678 | 9678 | | |
9679 | 9679 | | |
9680 | 9680 | | |
9681 | | - | |
9682 | | - | |
9683 | | - | |
| 9681 | + | |
| 9682 | + | |
| 9683 | + | |
9684 | 9684 | | |
9685 | | - | |
| 9685 | + | |
9686 | 9686 | | |
9687 | 9687 | | |
9688 | | - | |
| 9688 | + | |
9689 | 9689 | | |
9690 | 9690 | | |
9691 | 9691 | | |
| |||
35917 | 35917 | | |
35918 | 35918 | | |
35919 | 35919 | | |
35920 | | - | |
| 35920 | + | |
35921 | 35921 | | |
35922 | 35922 | | |
35923 | 35923 | | |
| |||
0 commit comments