Repository navigation
docs: point agents at the content guidelines skill - #39056
Draft
alidotforrest wants to merge 2 commits into
Draft
alidotforrest wants to merge 2 commits into
alidotforrest wants to merge 2 commits into
Conversation
Tell coding agents to read mms-content-guidelines before changing user-facing copy, and to edit the skill source in MetaMask/skills rather than the generated copy. Co-authored-by: alidotforrest <alidotforrest@users.noreply.github.com>
|
Open
6 of 11 tasks
Co-authored-by: alidotforrest <alidotforrest@users.noreply.github.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Description
Coding agents working in this repo do not currently have a durable pointer to the MetaMask content guidelines, so user-facing copy can be written without those rules. This adds a short section to
AGENTS.mdtelling agents to read and follow.claude/skills/mms-content-guidelines/SKILL.mdbefore writing or changing user-facing strings, to runyarn skillsif that file is missing, to edit English source strings inlocales/languages/en.jsonrather than the generated skill copy, and to change rules in MetaMask/skills rather than in this repo.The guidelines live in MetaMask/skills at
domains/ui/skills/content-guidelines/and sync into this repo as the generatedmms-content-guidelinesskill. That skill is the agent-readable implementation of the MetaMask Content Design Style Guide. The section also tells agents to flag any conflict between the skill and the Style Guide to the content owner rather than invent a rule. The updated guidelines are in MetaMask/skills#221, which is not merged yet. This pointer becomes useful once that PR merges andyarn skillsruns. No app strings or code change in this PR.Changelog
CHANGELOG entry: null
Related issues
Fixes:
Refs: MetaMask/skills#221
Manual testing steps
N/A — agent documentation only; no app behavior to exercise.
Screenshots/Recordings
Before
N/A
After
N/A
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist