# Implementation Review (Code Review)

> No need to run all checks; do not use pylint warnings. Feel free to further analyse any mentioned files and/or the file structure.

- Skill: `tools-only/implementation-review-code-review` (Agent Skill, multi-file: 3 files)
- Install (CLI): `npx skillmds@latest add tools-only/implementation-review-code-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/tools-only/implementation-review-code-review/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: tools-only (https://skillmd.com/u/tools-only)
- Updated: 2026-09-29
- Page: https://skillmd.com/skills/tools-only/implementation-review-code-review

---


# Implementation Review (Code Review)

**First, ensure we're up to date:**
```bash
git fetch
git status
```

Confirm and display the current feature branch name.

---

**Then run the code review:**

## Code Review Request

Run this command to get the changes to review:
```bash
git diff --unified=5 --no-prefix main...HEAD -- . ":(exclude)pr_info/.conversations/**"
```

No need to run all checks; do not use pylint warnings. Feel free to further analyse any mentioned files and/or the file structure.

### Focus Areas:
- Logic errors or bugs
- Tests for `__main__` functions should be removed (not needed)
- Unnecessary debug code or print statements
- Code that could break existing functionality
- Compliance with existing architecture principles, see `docs/architecture/ARCHITECTURE.md`

### Output Format:
1. **Summary** - What changed (1-2 sentences)
2. **Critical Issues** - Must fix before merging
3. **Suggestions** - Nice to have improvements
4. **Good** - What works well

Do not perform any action. Just present the code review.