← ClaudeAtlas

code-review-checklistlisted

Code review guidelines covering code quality, security, and best practices.
phuonghx/aim-cli · ★ 1 · Code & Development · score 77
Install: claude install-skill phuonghx/aim-cli
# Code Review Checklist ## Quick Review Checklist ### Correctness - [ ] Code does what it's supposed to do - [ ] Edge cases handled - [ ] Error handling in place - [ ] No obvious bugs ### Security - [ ] Input validated and sanitized - [ ] No SQL/NoSQL injection vulnerabilities - [ ] No XSS or CSRF vulnerabilities - [ ] No hardcoded secrets or sensitive credentials - [ ] **AI-Specific:** Protection against Prompt Injection (if applicable) - [ ] **AI-Specific:** Outputs are sanitized before being used in critical sinks ### Performance - [ ] No N+1 queries - [ ] No unnecessary loops - [ ] Appropriate caching - [ ] Bundle size impact considered ### Code Quality - [ ] Clear naming - [ ] DRY - no duplicate code - [ ] SOLID principles followed - [ ] Appropriate abstraction level ### Testing - [ ] Unit tests for new code - [ ] Edge cases tested - [ ] Tests readable and maintainable ### Documentation - [ ] Complex logic commented - [ ] Public APIs documented - [ ] README updated if needed ## AI & LLM Review Patterns ### Logic & Hallucinations - [ ] **Chain of Thought:** Does the logic follow a verifiable path? - [ ] **Edge Cases:** Did the AI account for empty states, timeouts, and partial failures? - [ ] **External State:** Is the code making safe assumptions about file systems or networks? ### Prompt Engineering Review ```markdown // ❌ Vague prompt in code const response = await ai.generate(userInput); // ✅ Structured & Safe prompt const response = await ai.generate({ s