Repository navigation
STUDIO.md: Drop the skills list that agents already discover from the installed skills - #5195
Merged
Merged
Conversation
… installed skills
chubes4
marked this pull request as ready for review
October 20, 2026 02:49
1 task
chubes4
added a commit
that referenced
this pull request
Oct 20, 2026
## Related issues - Follows #5195: drop `STUDIO.md` instructions that repeat something else. ## How AI was used in this PR Claude (via the Pi coding agent in a Roadie session) found the repeated rule, wrote the change, and drafted this description. I reviewed the change and the reasoning. ## Proposed Changes `STUDIO.md` says "don't edit `wp-includes/` or `wp-admin/`" five times: in the Constraints table, twice in "Use hooks, not direct edits" (the rule and a code comment), in "When to Create vs. Modify", and again in "Studio-Specific Notes". This keeps it once, in "Use hooks, not direct edits", the only place that gives the reason (core edits are overwritten on every WordPress update) and shows the alternative (the `add_action` example). The other four go. That paragraph now reads as what to do: extend through hooks in a plugin or theme, and leave core as shipped. Net: 2 lines added, 10 removed, in `skills/STUDIO.md` only. It's a managed file, so existing sites pick it up the next time they start (`updateManagedInstructionFiles`). ## Testing Instructions 1. Create a site and open its `STUDIO.md`. 2. `grep -n "wp-includes" STUDIO.md` returns one line, under "Use hooks, not direct edits". `npx vitest run packages/common/lib/tests/agent-skills.test.ts` passes (17). No eval: Studio Code doesn't load this file, which only reaches external agents opened in a site folder. The rule itself is unchanged; only its copies are removed. ## Pre-merge Checklist - [ ] Have you checked for TypeScript, React or other console errors?
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.
Related issues
How AI was used in this PR
Claude (via the Pi coding agent in a Roadie session) traced how skills reach a site, found the stale list, wrote the change, and drafted this description. I reviewed the change and the reasoning.
Proposed Changes
STUDIO.md's "Available Skills" section lists six skills as installed in every site. Only the skills a user picks in Settings → Skills are installed (installAiInstructionsToSitecopies justselectedSkills), and they can be added or removed later. So the list often names skills the site doesn't have, and it's the same for every site.Agents don't need it: Claude Code finds skills in
.claude/skills/and Codex in.agents/skills/, each from the skill's own name and description, and they see exactly what's installed.This removes the section:
studio wpprefix note goes. It repeats the file's first section, "WP-CLI in WordPress Studio".wp-project-triagenote stays, moved under "Studio-Specific Notes", since nothing else says it.Net: 16 lines removed, 1 added.
STUDIO.mdis a managed file, so existing sites get the change the next time they start (updateManagedInstructionFiles).Testing Instructions
STUDIO.md: no "Available Skills" section, and the triage note sits under "Studio-Specific Notes".ls <site>/.claude/skills <site>/.agents/skillsshows onlystudio-cli, which is what agents discover. On trunk,STUDIO.mdalso names the five WordPress skills that aren't there.npx vitest run packages/common/lib/tests/agent-skills.test.tspasses (17). No eval: this file isn't loaded by Studio Code. It only reaches external agents opened in a site folder.Pre-merge Checklist