Skip to content

fix: align mUSD decimals to 6 in default token metadata#8664

Merged
Prithpal-Sooriya merged 4 commits intomainfrom
fix/musd-decimals-6-assets-controller
May 1, 2026
Merged

fix: align mUSD decimals to 6 in default token metadata#8664
Prithpal-Sooriya merged 4 commits intomainfrom
fix/musd-decimals-6-assets-controller

Conversation

@salimtb
Copy link
Copy Markdown
Contributor

@salimtb salimtb commented May 1, 2026

Set the hardcoded mUSD token metadata to 6 decimals in both assets controllers so seeded defaults match the token contract and render balances correctly.

Explanation

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
Low risk: changes only hardcoded default mUSD metadata/seed values (decimals) and updates/extends unit tests and changelogs; no runtime control flow or security-sensitive logic is altered.

Overview
Fixes the default/seeded MetaMask USD (mUSD) token metadata to use 6 decimals instead of 18 across both @metamask/assets-controller (default tracked asset metadata) and @metamask/assets-controllers (TokensController/TokenDetectionController seeding).

Adds/updates tests to assert the seeded mUSD token is added with decimals: 6, and records the fix in both package changelogs.

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

salimtb added 2 commits May 1, 2026 12:24
Set the hardcoded mUSD token metadata to 6 decimals in both assets controllers so seeded defaults match the token contract and render balances correctly.
@salimtb salimtb force-pushed the fix/musd-decimals-6-assets-controller branch from 0284009 to 45b91b3 Compare May 1, 2026 10:25
@salimtb salimtb marked this pull request as ready for review May 1, 2026 10:34
@salimtb salimtb requested review from a team as code owners May 1, 2026 10:34
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ 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 3ef07f3. Configure here.

Comment thread packages/assets-controllers/src/TokensController.ts
@Prithpal-Sooriya Prithpal-Sooriya added this pull request to the merge queue May 1, 2026
Merged via the queue into main with commit ca69efc May 1, 2026
366 checks passed
@Prithpal-Sooriya Prithpal-Sooriya deleted the fix/musd-decimals-6-assets-controller branch May 1, 2026 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants