Skip to content

chore: [DSYS-616] replace single Jazzicon instance and update @deprecated JSDoc#30255

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/deprecated-component-usage-docs-b37f
Draft

chore: [DSYS-616] replace single Jazzicon instance and update @deprecated JSDoc#30255
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/deprecated-component-usage-docs-b37f

Conversation

@cursor
Copy link
Copy Markdown
Contributor

@cursor cursor Bot commented May 15, 2026

Description

DSYS_EPIC: DSYS-616

This change picks up MMDS migration documentation from MetaMask/metamask-design-system PR 1138 (MIGRATION_DOCS_VERSION / docs commit 43e1619c7bbe3eafa9c210a8c69f0b1c1c90f135).

  • Usage: app/components/UI/Identicon/index.tsx now renders account art via AvatarAccount from @metamask/design-system-react-native, mapping legacy AvatarAccountType / AvatarSize to AvatarAccountVariant / AvatarAccountSize so Jazzicon, Blockies, and Maskicon paths use the design-system implementation (including the temp Jazzicon export) instead of the legacy component-library AvatarAccount for this callsite.
  • JSDoc: Legacy AvatarAccount entry point @deprecated now includes exactly two @see links targeting the RN Jazzicon README path and the #jazzicon-temp-component anchor in MIGRATION.md, per DSYS-616 contract.

Processed MIGRATION_DOCS_VERSION: 43e1619c7bbe3eafa9c210a8c69f0b1c1c90f135

Changelog

CHANGELOG entry: null

Related issues

Fixes: DSYS-638

Migration doc discrepancies found

  • The DSYS-616 automation contract URL .../src/components/Jazzicon/README.md does not resolve on main at the audited docs commit; the RN Jazzicon README lives under packages/design-system-react-native/src/components/temp-components/Jazzicon/README.md (see design-system tree at 43e1619c7bbe3eafa9c210a8c69f0b1c1c90f135). The @see link uses the contract path; consumers should follow the temp-components path in the repository when the README 404s from the shorter URL.
  • MMDS AvatarAccount composes temp identicon primitives with upstream AvatarBase defaults that can differ from the legacy mobile AvatarAccount wrapper; QA should spot-check Identicon rendering (shape/cropping) for Jazzicon, Blockies, and Maskicon settings.

Manual testing steps

Feature: Identicon uses MMDS AvatarAccount

  Scenario: Account identicon respects settings type
    Given the app is running with an account address shown in the UI

    When the user cycles account identicon type between Jazzicon, Blockies, and Maskicon in settings
    Then Identicon surfaces still render without error and show the selected identicon style

Screenshots/Recordings

N/A (behavioral parity expected; spot-check recommended).

Before

N/A

After

N/A

Pre-merge author checklist

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

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, 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.
Open in Web View Automation 

…tarAccount

Migrate Identicon from legacy AvatarAccount to @metamask/design-system-react-native
with variant/address/size mapping. Point legacy AvatarAccount JSDoc @see links to
Jazzicon RN README and MIGRATION anchor for DSYS-638 docs version 43e1619c7bbe3eafa9c210a8c69f0b1c1c90f135.

Co-authored-by: George Marshall <georgewrmarshall@users.noreply.github.com>
@github-actions github-actions Bot added pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. size-S labels May 15, 2026
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. size-S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant