chore: add /check-rs skill to repo for shared use#20
Merged
Conversation
Vendor the check-rs Rust quality-pass skill (fmt, clippy, DRY/complexity/test review agents, then fix-up) into .claude/skills so every contributor using this repo has /check-rs available without local agent setup. Verified clean before commit: cargo fmt --check, cargo clippy --all-targets -- -D warnings (0 warnings), cargo test (140 + 9 doc-tests passing). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
1 participant
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.
What
Vendors the
/check-rsRust quality-pass skill into the repository at.claude/skills/check-rs/SKILL.mdso it ships with the repo. Previously thisskill only existed in individual contributors' local
~/.agents/skillsdirectory; committing it makes
/check-rsavailable to anyone who clones thisrepo.
The skill
/check-rsperforms a comprehensive quality pass on recently changed Rust code:cargo fmt+cargo clippy -- -D warnings(fixes before reviewing)#[allow(...)]suppressions)cargo fmt --check,cargo clippy -- -D warnings,cargo testmust all passVerification
This change is docs/tooling only (no Rust source touched). Ran the skill's
tooling + verification phases against the project to confirm it's clear:
cargo fmt --check— cleancargo clippy --all-targets -- -D warnings— 0 warningscargo test— 140 tests + 9 doc-tests passing, 0 failed🤖 Generated with Claude Code