Skip to content

feat(client-utils): add shared mobile activity types - #9916

Queued
n3ps wants to merge 1 commit into
mainfrom
feat/client-utils-mobile-activity-types
Queued

feat(client-utils): add shared mobile activity types#9916
n3ps wants to merge 1 commit into
mainfrom
feat/client-utils-mobile-activity-types

Conversation

@n3ps

@n3ps n3ps commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Explanation

Mobile still extends ActivityKind and ActivityItem locally for staking, prediction, and perps activity. This adds those remaining variants to @metamask/client-utils so Mobile can remove its temporary type shim.

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

Cursor Bugbot is generating a summary for commit e68fb6d. Configure here.

@n3ps
n3ps requested review from a team as code owners August 20, 2026 03:51
@n3ps
n3ps deployed to default-branch August 20, 2026 03:51 — with GitHub Actions Active
@n3ps
n3ps requested a review from a team as a code owner August 20, 2026 03:51
Add the staking, prediction, and perps activity kinds and item variants
that MetaMask Mobile still declares locally, so its temporary type shim
can be removed. Mapper behavior is unchanged, keeping Extension's
existing `stakingDeposit` to `deposit` mapping intact.

Co-authored-by: Cursor <cursoragent@cursor.com>
@n3ps
n3ps force-pushed the feat/client-utils-mobile-activity-types branch from 8389d9d to 926ffcb Compare August 20, 2026 04:11
@n3ps
n3ps enabled auto-merge August 20, 2026 04:12
@n3ps
n3ps added this pull request to the merge queue Aug 20, 2026
Any commits made after this event will not be merged.
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