Skip to content

feat: add assets domain skills - #131

Merged
Prithpal-Sooriya merged 4 commits into
mainfrom
feat/add-initial-asset-domain-skills
Aug 19, 2026
Merged

feat: add assets domain skills#131
Prithpal-Sooriya merged 4 commits into
mainfrom
feat/add-initial-asset-domain-skills

Conversation

@Prithpal-Sooriya

Copy link
Copy Markdown
Contributor

Description

Initial setup for @MetaMask/metamask-assets skills.

Type of Change

  • New skill
  • Skill improvement/update
  • Bug fix
  • Documentation update
  • Other (please describe):

Skill Details (if adding a new skill)

Provider Name: Assets
Skill Name & Desc:

  • acceptance-criteria-doc - generate Acceptance Criteria scaffold in Confluence for PM/QA/Eng. Useful for QA checklists, UAT, and EPIC DoD
  • react-compiler-migration - internal tool to migrate UI/Hooks to be react-compiler compatible. Used internally until ready to assimilate to other skills and teams. We'll be piping this to our AI board for cleaner RC migration execution.

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

Additional Context

- Introduced a new section in the README.md for the `assets` domain, outlining its skills and purpose within the MM product engineering team.
- Added a new skill documentation file for `acceptance-criteria-doc`, detailing the process for generating acceptance criteria documents for Jira epics, including structure, inputs, and workflow.
- Created a new skill documentation file for `react-compiler-migration`, providing guidelines for migrating files to optimize them for the React Compiler.
- Added a reference patterns document for `react-compiler-migration`, containing validated patterns for common compiler issues and their solutions.
- Updated CODEOWNERS to include the new `assets` domain for better ownership tracking.
@Prithpal-Sooriya
Prithpal-Sooriya enabled auto-merge (squash) August 18, 2026 16:51

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Refined this, see examples here:
MetaMask/metamask-mobile#34895

Comment thread domains/assets/skills/acceptance-criteria-doc/skill.md Outdated
Comment thread domains/assets/skills/react-compiler-migration/skill.md Outdated
Comment thread domains/assets/skills/react-compiler-migration/skill.md Outdated
cursoragent and others added 2 commits August 19, 2026 11:48
Start the skill at experimental until it has been run outside Assets,
per review feedback.

Co-authored-by: Prithpal Sooriya <prithpal.sooriya@gmail.com>
Start the skill at experimental until it has been run outside Assets,
per review feedback.

Co-authored-by: Prithpal Sooriya <prithpal.sooriya@gmail.com>
@Prithpal-Sooriya
Prithpal-Sooriya merged commit 7320fe2 into main Aug 19, 2026
28 checks passed
@Prithpal-Sooriya
Prithpal-Sooriya deleted the feat/add-initial-asset-domain-skills branch August 19, 2026 15:16
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.

3 participants