Code Review
Провести структурированное ревью кода по пяти осям (корректность, читаемость, безопасность, производительность, тесты) перед мержем. Используй при запросах "проверь этот код", "сделай ревью", "посмотри на PR", "готово ли это к мержу". Также применяй проактивно после того как сам написал значимый кусок кода, перед тем как показать его как финальный результат.
Install
$ agentstack add skill-alex7develop-my-agent-skills-code-review ✓ scanned · ✓ verified, works with Claude Code, Cursor, and more.
Security review
✓ PassedNo issues found. Passed automated security review. · v0.1.0 How review works →
- ✓ Prompt-injection patterns
- ✓ Secret / credential exfiltration
- ✓ Dangerous shell & filesystem operations
- ✓ Untrusted network calls
- ✓ Known-malicious package signatures
What it can access
- ✓ Network access No
- ✓ Filesystem access No
- ✓ Shell / process execution No
- ✓ Environment & secrets No
- ✓ Dynamic code execution No
From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.
Verified badge
Passed review? Show it. Paste this badge into your README, it links to the public security report.
Links to your public security report.[](https://agentstack.voostack.com/security/report/skill-alex7develop-my-agent-skills-code-review) Reliability & compatibility
Declared compatibility
Compatibility is declared by the source manifest. End-to-end runtime verification is coming, see below.
We're building live execution health for every listing: tool-call success rate, median latency, uptime, and last-checked timestamps, measured, not self-reported. It isn't live yet, so we don't show numbers we can't stand behind.
How agent discovery & health will work →About
Code Review
Пять осей проверки. Не пытаться "найти хоть что-то" на каждой оси — если ось не даёт находок, честно писать "нет замечаний", это лучше, чем притянутые мелочи.
Пять осей
- Корректность. Делает ли код то, что задумано? Обработаны ли
граничные случаи (пустой вход, null, переполнение, конкурентный доступ)?
- Читаемость. Поймёт ли код человек, не писавший его, без объяснений?
Имена, структура, отсутствие "магии".
- Безопасность. Валидация входных данных, инъекции, секреты в коде,
права доступа, обработка пользовательского ввода.
- Производительность. Есть ли очевидные проблемы (N+1 запросы,
O(n²) там где легко сделать O(n log n), утечки памяти/хендлов)? Не заниматься микрооптимизацией там, где это не нужно.
- Тесты. Покрыты ли изменения тестами? Тестируют ли они поведение,
а не только "счастливый путь"?
Процесс
- Понять контекст изменения (что и зачем меняется) прежде чем оценивать —
спека или описание PR, если есть.
- Пройти по каждой из пяти осей отдельно, не смешивая находки.
- Разделить находки по серьёзности:
- Блокирует мерж — баги, дырки безопасности, потеря данных
- Стоит исправить — читаемость, дублирование, слабое покрытие тестами
- Опционально / на вкус — стилистические предложения
- Для каждой находки — конкретное место (файл/строка) и конкретное
предложение, а не общая формулировка вроде "можно лучше".
Формат вывода
## 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
- Source: 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.
Reviews
No reviews yet, be the first.
Write a review
Versions
- v0.1.0 Imported from the upstream source.