Skip to content

feat: add activity mapper guidance - #177

Merged
n3ps merged 2 commits into
mainfrom
add-activity-mappers-skill
Sep 23, 2026
Merged

n3ps merged 2 commits into
mainfrom
add-activity-mappers-skill

Conversation

@n3ps

@n3ps n3ps commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Description

Add concise guidance for Activity List and Activity Item changes so mapping logic starts in @metamask/client-utils/mappers and stays shared across Mobile and Extension.

Type of Change

  • New skill

Skill Details (if adding a new skill)

Provider Name: MetaMask
Skill Name: activity-mappers
Brief Description: Centralize Activity mapping in client-utils and prefer API or non-EVM keyring transaction data.

Checklist

  • I have read the CONTRIBUTING.md guidelines
  • My skill follows the SKILL_TEMPLATE.md format
  • I have tested this skill with an AI agent
  • My skill does not contain any secrets, private keys, or sensitive data
  • I have added appropriate documentation
  • My changes don't break existing skills

Testing

node .github/scripts/lint-skill-entry.mjs domains/ui/skills/activity-mappers/skill.md

Additional Context

None.

@n3ps
n3ps force-pushed the add-activity-mappers-skill branch from c17b848 to c7a6361 Compare September 23, 2026 02:36
@n3ps n3ps changed the title Add Activity mapper guidance feat: add activity mapper guidance Sep 23, 2026
@n3ps
n3ps force-pushed the add-activity-mappers-skill branch from c7a6361 to 072274f Compare September 23, 2026 02:50
@n3ps
n3ps force-pushed the add-activity-mappers-skill branch from 072274f to ad05adc Compare September 23, 2026 02:51
@n3ps
n3ps enabled auto-merge (squash) September 23, 2026 15:54
@n3ps
n3ps merged commit bc63ad6 into main Sep 23, 2026
29 checks passed
@n3ps
n3ps deleted the add-activity-mappers-skill branch September 23, 2026 15:57
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