# code-review

> Comprehensive code review checklist and guidelines. Use when reviewing code changes or PRs.

- Author: erik1908
- Repository: quadero-com/multi-agent-dev-system
- Version: 20260131120238
- Stars: 0
- Forks: 0
- Last Updated: 2026-02-06
- Source: https://github.com/quadero-com/multi-agent-dev-system
- Web: https://mule.run/skillshub/@@quadero-com/multi-agent-dev-system~code-review:20260131120238

---

---
name: code-review
description: Comprehensive code review checklist and guidelines. Use when reviewing code changes or PRs.
disable-model-invocation: false
---

# Code Review Checklist

## 1. Correctness
- [ ] Does it solve the intended problem?
- [ ] Are edge cases handled?
- [ ] Is error handling appropriate?

## 2. Security
- [ ] Input validation present?
- [ ] No SQL injection vulnerabilities?
- [ ] No XSS vulnerabilities?
- [ ] Secrets not hardcoded?
- [ ] Proper authentication/authorization?

## 3. Performance
- [ ] No N+1 queries?
- [ ] Appropriate indexing?
- [ ] No unnecessary computations?
- [ ] Caching considered?

## 4. Maintainability
- [ ] Code is readable?
- [ ] Functions are small and focused?
- [ ] Naming is clear?
- [ ] No code duplication?

## 5. Testing
- [ ] Unit tests added?
- [ ] Edge cases tested?
- [ ] Integration tests if needed?
- [ ] Test names are descriptive?

## 6. Documentation
- [ ] Complex logic explained?
- [ ] API docs updated?
- [ ] README updated if needed?

## Review Response Template

```markdown
## Summary
[Overall assessment]

## Strengths
- [What's good about this code]

## Required Changes
- [ ] [Must fix before merge]

## Suggestions
- [ ] [Nice to have improvements]

## Questions
- [Clarifications needed]
```