Skip to content

feat(perps): add multiproject validation skill#48

Draft
abretonc7s wants to merge 4 commits into
mainfrom
perps/multiproject-validation-skill
Draft

feat(perps): add multiproject validation skill#48
abretonc7s wants to merge 4 commits into
mainfrom
perps/multiproject-validation-skill

Conversation

@abretonc7s

Copy link
Copy Markdown
Contributor

Summary

  • Add a Perps multiproject validation skill for checking Core controller changes in Mobile/Extension or client parity across Mobile and Extension.
  • Add component-view test coverage guidance to the Perps review checklist and Extension overlay.
  • Keep the validation workflow default-driven: current checkout as owner, sibling repo discovery, yalc for Core package transport, read-only targets, and explicit cleanup.

Validation

  • node bin/metamask-skills.mjs describe perps/validate-perps-multiproject
  • node bin/metamask-skills.mjs list --domain perps --include perps/validate-perps-multiproject
  • yarn smoke
  • yarn test

Follow-up

  • Extension/mobile farm fixtures should consume this guidance from MetaMask skills instead of copying it into project-specific fixture text.

…iled usage instructions and error handling guidelines
…ect checks

- Added `scripts/perps-validate.sh` to automate the validation process for Core changes in client checkouts (Mobile and Extension).
- Enhanced documentation in `skill.md` with detailed usage instructions and error handling guidelines.
- Emphasized the importance of pre-state awareness and yalc resolution in the validation workflow.
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.

1 participant