sanyuan-skills
Шесть скиллов для Claude Code: ревью кода по SOLID и безопасности, репетитор Sigma, аудит и создание скиллов, вики из заметок и коуч по чтению книг
- 3 966 ★ репозитория
- Последний коммит
- MIT
Расширенный доступ
Почему такой уровень
- Скилл ревью запускает `git` и поиск по репозиторию и читает ваши изменения целиком
- Скилл skill-forge содержит скрипты Python для создания и упаковки скиллов, которые пишут файлы на диск
Установка
- Работает в
В терминале, через SkillFoxx CLI
npx skillfoxx add skills/sanyuan-skillsОпределит агенты на машине, покажет уровень доступа и зафиксирует версию.
Другие способы установки
Доступ и проверки
Как мы проверяемРасширенный доступИнструмент запускает код или ходит в сеть, а внешние данные только читает.
- Скилл ревью запускает `git` и поиск по репозиторию и читает ваши изменения целиком
- Скилл skill-forge содержит скрипты Python для создания и упаковки скиллов, которые пишут файлы на диск
Это чужой код. Посмотрите файлы в репозитории перед установкой.
Что делает
sanyuan-skills — набор из шести скиллов автора sanyuan0704, устанавливается по одному командой npx skills add. Главный скилл code-review-expert разбирает текущие правки, видимые в `git diff`, глазами старшего инженера. Он смотрит нарушения SOLID, кандидатов на удаление, дыры в безопасности (XSS, инъекции, SSRF, пропуск проверок прав, гонки) и качество кода: обработку ошибок, производительность, граничные условия. Находки получают уровни от P0 (блокирует слияние) до P3 (по желанию), а по умолчанию скилл только пишет отчет и ничего не правит, пока вы не попросите. Остальные скиллы: Sigma ведет занятия как репетитор по схеме обучения Блума с вопросами в сократовском стиле, skill-review проверяет структуру и описание чужих скиллов, skill-forge создает новые по 12 приемам, wiki-ingest собирает из статей и заметок вики с перекрестными ссылками, а book-study работает как коуч для чтения с интервальным повторением.
Для кого. Для разработчиков, которые хотят проверять собственные изменения перед ревью коллег, и для тех, кто пишет и дорабатывает скиллы для агентов.
Подходит, если
- Перед запросом на слияние нужно проверить правки на SOLID, безопасность и обработку ошибок
- Крупные правки свыше 500 строк нужно разобрать по файлам и модулям, а не одним потоком
- Хочется получить план удаления мертвого кода с пометкой «сразу» или «отложить»
- Вы написали скилл и хотите проверить структуру, описание и расход токенов
- Нужен репетитор, который задает вопросы по теме вместо готовых ответов
Не подходит, если
- Нужны автоматические комментарии прямо в запросе на слияние на GitHub: скилл читает локальные правки в `git` и пишет отчет
- Нужен статический анализатор в CI: ревью делает агент, результат зависит от модели и не заменяет линтеры
- В репозитории нет изменений: при пустой выдаче `git diff` скилл спросит, проверять ли индекс или диапазон коммитов
- Вам нужна только установка одного набора правил кода: здесь шесть разных скиллов
Пример запроса к агенту
/code-review-expert Проверь текущие изменения, особенно безопасность и обработку ошибок, и ничего не правьОграничения
Скилл ревью читает `git status`, `git diff` и при необходимости ищет по репозиторию через `rg` или `grep`, поэтому качество зависит от модели агента и объема правок. Автор рассчитывал набор на Claude Code и «другие терминалы агентов», список проверенных сред в описании не приведен. Установка идет командой npx skills add из внешнего пакета, поэтому нужен Node.js. Некоторые скиллы вроде Sigma и book-study рассчитаны на диалог, а не на разовый запрос.
Как отключить. Удалите папку нужного скилла из каталога скиллов агента, например skills/code-review-expert. Остальные скиллы набора независимы и продолжат работать.
Частые вопросы
Как поставить code-review-expert в Claude Code?
Выполните npx skills add sanyuan0704/sanyuan-skills --path skills/code-review-expert. Затем вызывайте /code-review-expert в сессии, где есть незакоммиченные изменения. Остальные скиллы набора ставятся такой же командой с другим значением --path.
Что означают уровни P0–P3 в отчете?
P0 — критичная проблема вроде уязвимости или потери данных, она блокирует слияние. P1 — логическая ошибка или серьезное нарушение SOLID. P2 — запах кода, который можно исправить сейчас или оформить задачей. P3 — стиль и названия, исправлять по желанию.
Исправляет ли скилл код сам?
По умолчанию нет: он работает в режиме отчета. Правки скилл вносит только после вашей просьбы.
Какие проверки безопасности входят в ревью?
Скилл ищет XSS, инъекции SQL и команд, SSRF, обход пути, дыры в проверке прав, утечки секретов в логах и файлах, неограниченные циклы, небезопасную десериализацию, слабую криптографию и гонки при конкурентном доступе. Для каждой находки он описывает возможность эксплуатации и последствия.
Что делает skill-forge?
Это мета-скилл для создания скиллов по 12 проверенным приемам. В нем лежат руководства по архитектуре, описаниям и шаблонам вывода и три скрипта на Python: init_skill.py, package_skill.py и quick_validate.py.
Обсуждение
Вопросов и отзывов пока нет.
Похожие
СкиллыClaude Code Skills, китайский набор
Claude Code Skills 中文精选集
Набор из 21 собственного скилла для Claude Code с китайским выводом: ревью кода, README, тесты API, рефакторинг, профилирование
14 скиллов, которые находят пробелы в задаче до кода: интервью, план, проверка допущений и доказательство исправления бага
Подборка скиллов для Claude Code про рабочий процесс: изоляция в worktree, слой сервисов, доказательства и обзор
Скилл для ревью кода по многим языкам и фреймворкам: архитектура, производительность, безопасность, антипаттерны и типичные ошибки
110 скиллов для Claude Code и Codex и семь агентов Claude Code: разработка, продукт, UI, диагностика и операции, ставится целиком или набором
Скиллы-контроли для кодовых агентов: второй проход по диффу, который ловит типичные ошибки в коде, тестах и документации от ИИ
Сравнение с похожими
Записи с тем же главным запросом: чем они отличаются.
| Запись | Официальный | Авторизация | Звезды репозитория | Главное отличие |
|---|---|---|---|---|
| sanyuan-skills (эта запись) | нет | 3 966 | ||
| brooks-lint | нет | 1 550 | плагин ревью с опорой на классические книги по инженерии и ссылками на источник | |
| code-review-graph | нет | не нужна | 32 463 | MCP-сервер, который строит граф кода и отдает агенту карту репозитория |
| ielts-claude-skills | нет | 330 | четыре бесплатных скилла для Claude Code |
Коротко о README
Описание коротко представляет набор как скиллы продакшен-уровня для Claude Code и других терминалов агентов. Таблица перечисляет шесть скиллов с описанием и командой установки через npx skills add с параметром --path. В разделе быстрого старта показаны команды вызова: /code-review-expert, /sigma, /skill-review, /skill-forge, /wiki-ingest и /book-study.
Бейдж для README
Исходный SKILL.md
--- name: code-review-expert description: "Expert code review of current git changes with a senior engineer lens. Detects SOLID violations, security risks, and proposes actionable improvements." --- # Code Review Expert ## Overview Perform a structured review of the current git changes with focus on SOLID, architecture, removal candidates, and security risks. Default to review-only output unless the user asks to implement changes. ## Severity Levels | Level | Name | Description | Action | |-------|------|-------------|--------| | **P0** | Critical | Security vulnerability, data loss risk, correctness bug | Must block merge | | **P1** | High | Logic error, significant SOLID violation, performance regression | Should fix before merge | | **P2** | Medium | Code smell, maintainability concern, minor SOLID violation | Fix in this PR or create follow-up | | **P3** | Low | Style, naming, minor suggestion | Optional improvement | ## Workflow ### 1) Preflight context - Use `git status -sb`, `git diff --stat`, and `git diff` to scope changes. - If needed, use `rg` or `grep` to find related modules, usages, and contracts. - Identify entry points, ownership boundaries, and critical paths (auth, payments, data writes, network). **Edge cases:** - **No changes**: If `git diff` is empty, inform user and ask if they want to review staged changes or a specific commit range. - **Large diff (>500 lines)**: Summarize by file first, then review in batches by module/feature area. - **Mixed concerns**: Group findings by logical feature, not just file order. ### 2) SOLID + architecture smells - Load `references/solid-checklist.md` for specific prompts. - Look for: - **SRP**: Overloaded modules with unrelated responsibilities. - **OCP**: Frequent edits to add behavior instead of extension points. - **LSP**: Subclasses that break expectations or require type checks. - **ISP**: Wide interfaces with unused methods. - **DIP**: High-level logic tied to low-level implementations. - When you propose a refactor, explain *why* it improves cohesion/coupling and outline a minimal, safe split. - If refactor is non-trivial, propose an incremental plan instead of a large rewrite. ### 3) Removal candidates + iteration plan - Load `references/removal-plan.md` for template. - Identify code that is unused, redundant, or feature-flagged off. - Distinguish **safe delete now** vs **defer with plan**.