Add agent skills for working on ty#25422
Merged
Merged
Conversation
carljm
approved these changes
May 27, 2026
carljm
left a comment
Contributor
There was a problem hiding this comment.
I don't know how to evaluate all the Claude JSON stuff, but the skills themselves look good to me, and I'd find them useful. Thank you!
anishgirianish
pushed a commit
to anishgirianish/ruff
that referenced
this pull request
May 28, 2026
## Summary This PR adds a `.agents/skills` directory to the repo, and adds three skills to help agents work on ty more effectively: - A skill to help them write better ty diagnostics - A skill to help them minimize a single ecosystem change more effectively - A skill to help them analyze and summarise ecosystem reports more effectively The skills have been designed so that they will (hopefully) work well whether a contributor is using Codex or Claude as their agent locally. Codex should discover these skills automatically if you're working on the repo; if you use Claude locally, you _should_ hopefully be prompted by Claude to install the local skills due to them being listed as recommended skills in `.claude/settings.json`. ## Test Plan I ran `/review` locally with Codex, instructing it to be as nitpicky as possible, until it could find no more issues with these skills.
AlexWaygood
added a commit
that referenced
this pull request
May 29, 2026
## Summary A bunch of followup changes from #25422. Mostly triggered by using the skill for the first time to analyze the report in #24761. - Move ty-specific guidance from `AGENTS.md` to a new local `working-on-ty` skill. Link to the other ty-specific skills from that skill. - Add more keyword triggers to various skills so that agents are more likely to automatically figure out that a skill is appropriate (suggested by @dhruvmanila). - Add more guidance to the ecosystem-minimization skill (and tweak the script for setting up a primer project) so that agents are likely to reproduce results in a deterministic and helpful way. ## Test Plan I repeatedly ran `/review` with Codex on this until the only remaining issues were trivial.
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.
Summary
This PR adds a
.agents/skillsdirectory to the repo, and adds three skills to help agents work on ty more effectively:The skills have been designed so that they will (hopefully) work well whether a contributor is using Codex or Claude as their agent locally. Codex should discover these skills automatically if you're working on the repo; if you use Claude locally, you should hopefully be prompted by Claude to install the local skills due to them being listed as recommended skills in
.claude/settings.json.Test Plan
I ran
/reviewlocally with Codex, instructing it to be as nitpicky as possible, until it could find no more issues with these skills.