# Code Review

> Провести структурированное ревью кода по пяти осям (корректность, читаемость, безопасность, производительность, тесты) перед мержем. Используй при запросах "проверь этот код", "сделай ревью", "посмотри на PR", "готово ли это к мержу". Также применяй проактивно после того как сам написал значимый кусок кода, перед тем как показать его как финальный результат.

- **Type:** Skill
- **Install:** `agentstack add skill-alex7develop-my-agent-skills-code-review`
- **Verified:** Yes — security-reviewed for prompt injection and unsafe behavior
- **Seller:** [Alex7develop](https://agentstack.voostack.com/s/alex7develop)
- **Installs:** 0
- **Category:** [Agent Skills](https://agentstack.voostack.com/c/agent-skills)
- **Latest version:** 0.1.0
- **License:** MIT
- **Upstream author:** [Alex7develop](https://github.com/Alex7develop)
- **Source:** https://github.com/Alex7develop/my-agent-skills/tree/main/skills/code-review
- **Website:** https://skills.sh/Alex7develop/my-agent-skills

## Install

```sh
agentstack add skill-alex7develop-my-agent-skills-code-review
```

Requires the [AgentStack CLI](https://agentstack.voostack.com/docs/cli). Works with Claude Code, Cursor, and any MCP-compatible agent.

## About

# Code Review

Пять осей проверки. Не пытаться "найти хоть что-то" на каждой оси —
если ось не даёт находок, честно писать "нет замечаний", это лучше, чем
притянутые мелочи.

## Пять осей

1. **Корректность.** Делает ли код то, что задумано? Обработаны ли
   граничные случаи (пустой вход, null, переполнение, конкурентный доступ)?
2. **Читаемость.** Поймёт ли код человек, не писавший его, без объяснений?
   Имена, структура, отсутствие "магии".
3. **Безопасность.** Валидация входных данных, инъекции, секреты в коде,
   права доступа, обработка пользовательского ввода.
4. **Производительность.** Есть ли очевидные проблемы (N+1 запросы,
   O(n²) там где легко сделать O(n log n), утечки памяти/хендлов)? Не
   заниматься микрооптимизацией там, где это не нужно.
5. **Тесты.** Покрыты ли изменения тестами? Тестируют ли они поведение,
   а не только "счастливый путь"?

## Процесс

1. Понять контекст изменения (что и зачем меняется) прежде чем оценивать —
   спека или описание PR, если есть.
2. Пройти по каждой из пяти осей отдельно, не смешивая находки.
3. Разделить находки по серьёзности:
   - **Блокирует мерж** — баги, дырки безопасности, потеря данных
   - **Стоит исправить** — читаемость, дублирование, слабое покрытие тестами
   - **Опционально / на вкус** — стилистические предложения
4. Для каждой находки — конкретное место (файл/строка) и конкретное
   предложение, а не общая формулировка вроде "можно лучше".

## Формат вывода

```
## Code Review

### Блокирует мерж
- [файл:строка] ...

### Стоит исправить
- [файл:строка] ...

### Опционально
- ...

### Без замечаний
- 
```

## Анти-паттерны, которых нужно избегать

- Не придумывать проблемы там, где их нет, только чтобы "было что сказать"
- Не давать общие советы без привязки к конкретному коду
- Не путать личные стилистические предпочтения с объективными проблемами —
  явно помечать субъективные замечания как "на вкус"

## Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

- **Author:** [Alex7develop](https://github.com/Alex7develop)
- **Source:** [Alex7develop/my-agent-skills](https://github.com/Alex7develop/my-agent-skills)
- **License:** MIT
- **Homepage:** https://skills.sh/Alex7develop/my-agent-skills

Install and usage instructions live in the source repository linked above.

## Pricing

- **Free** — Free

## Security capabilities

Automated source analysis of v0.1.0 — what this tool can access:

- **Network access:** no
- **Filesystem access:** no
- **Shell / process execution:** no
- **Environment & secrets:** no
- **Dynamic code execution:** no

*"Yes" means the capability is present in the source — more access means more to trust, not that it is unsafe.*


## Versions

- **0.1.0** — security scan: passed — Imported from the upstream source.

## Links

- Listing page: https://agentstack.voostack.com/l/skill-alex7develop-my-agent-skills-code-review
- Seller: https://agentstack.voostack.com/s/alex7develop
- Browse the marketplace: https://agentstack.voostack.com/browse

---
Listed on AgentStack — the marketplace for AI agent skills and MCP servers. Every listing is security-reviewed. Creators keep 70%.