Agent Skill
2/7/2026

code-review

This skill should be used when the user asks to "провести ревью", "проверить код", "найти проблемы в коде", "code review", "найти баги", "проверить на ошибки", "analyze code quality", "ревью PR", "review pull request", or needs guidance on concurrency issues, security vulnerabilities, or language-specific code quality for Go, Java, Python.

C
conte777
0GitHub Stars
1Views
npx skills add Conte777/config_for_claude_code

SKILL.md

Namecode-review
DescriptionThis skill should be used when the user asks to "провести ревью", "проверить код", "найти проблемы в коде", "code review", "найти баги", "проверить на ошибки", "analyze code quality", "ревью PR", "review pull request", or needs guidance on concurrency issues, security vulnerabilities, or language-specific code quality for Go, Java, Python.

name: code-review description: This skill should be used when the user asks to "провести ревью", "проверить код", "найти проблемы в коде", "code review", "найти баги", "проверить на ошибки", "analyze code quality", "ревью PR", "review pull request", or needs guidance on concurrency issues, security vulnerabilities, or language-specific code quality for Go, Java, Python. allowed-tools: Bash, Read, Grep, Glob, LSP, WebFetch, WebSearch, mcp__context7__resolve-library-id, mcp__context7__query-docs

Code Review Skill

Скил для проведения code review с поддержкой Go, Java, Python и их популярных фреймворков.

Overview

Этот скил предоставляет структурированный подход к code review с фокусом на:

  • Security — уязвимости, утечки секретов, инъекции
  • Concurrency — race conditions, deadlocks, утечки goroutine/потоков
  • Performance — сложность алгоритмов, memory leaks, N+1 queries
  • Best Practices — идиоматичный код, обработка ошибок, тестируемость

Review Process

Step 1: Detect Language & Frameworks

Определи язык и фреймворки по расширениям файлов и импортам:

ИндикаторЯзык/Фреймворк
.go, go.modGo
fx.New, fx.ModuleGo + Uber FX
.java, pom.xml, build.gradleJava
@SpringBootApplication, @RestControllerJava + Spring
.py, requirements.txt, pyproject.tomlPython
FastAPI, @app.getPython + FastAPI

Step 2: Apply Checks

Загрузи соответствующие reference файлы и примени проверки:

  1. Всегда: references/common-patterns.md — security, race conditions, performance
  2. По языку:
    • Go: references/go-patterns.md
    • Go + FX: references/go-fx-patterns.md
    • Java: references/java-patterns.md
    • Java + Spring: references/java-spring-patterns.md
    • Python: references/python-patterns.md
    • Python + FastAPI: references/python-fastapi-patterns.md

Step 3: Generate Report

Сформируй отчёт на русском языке по шаблону из examples/review-report-template.md.

Severity Levels

УровеньEmojiКритерииДействие
CRITICAL🔴Security vulnerability, data loss, production crashТребует немедленного исправления
HIGH🟠Race condition, resource leak, significant bug, blocking in asyncИсправить до merge
LOW🔵Performance issue, code smell, minor improvementПо возможности

Common Checks Quick Reference

Security Checks

ПроверкаОписание
Input ValidationВалидация пользовательского ввода
SQL InjectionПараметризованные запросы vs конкатенация
Secret ExposureСекреты в коде, логах, ошибках
Path TraversalПроверка путей к файлам

Concurrency Checks

ПроверкаОписание
Race ConditionsДоступ к shared state без синхронизации
DeadlocksПорядок блокировок, вложенные локи
Resource LeaksНезакрытые goroutines/threads/connections
TOCTOUTime-of-check to time-of-use

Performance Checks

ПроверкаОписание
ComplexityO(n²) и хуже в hot paths
MemoryАллокации в циклах, unbounded caches
I/OBlocking в async, отсутствие pooling
QueriesN+1, отсутствие индексов

Language Highlights

Go

  • Goroutine leaks: unbuffered channels, missing done signal
  • Defer: defer in loops, resource cleanup order
  • Errors: %w wrapping, unchecked errors
  • Context: propagation, cancellation handling

Java

  • Null safety: NPE риски, Optional misuse
  • Synchronized: double-checked locking, lock ordering
  • Resources: try-with-resources, connection leaks
  • Streams: parallel stream pitfalls, side effects

Python

  • GIL: threading for I/O vs multiprocessing for CPU
  • Context managers: manual cleanup vs with
  • Async: blocking in async, proper await usage
  • Types: Optional без проверки, missing hints

Workflow

  1. Получи список файлов для ревью (от пользователя или через git diff)
  2. Для каждого файла:
    • Определи язык и фреймворки
    • Загрузи соответствующие references
    • Прочитай файл и проанализируй
  3. Собери все findings в единый отчёт
  4. Отсортируй по severity (CRITICAL → LOW)
  5. Выведи отчёт на русском языке

Additional Resources

Детальные паттерны и anti-patterns находятся в:

  • references/common-patterns.md — общие проверки
  • references/go-patterns.md — Go специфика
  • references/go-fx-patterns.md — Uber FX паттерны
  • references/java-patterns.md — Java специфика
  • references/java-spring-patterns.md — Spring паттерны
  • references/python-patterns.md — Python специфика
  • references/python-fastapi-patterns.md — FastAPI паттерны
  • examples/review-report-template.md — шаблон отчёта
Skills Info
Original Name:code-reviewAuthor:conte777