reviewing-codelisted

# Code Review ## Quick Commands ```bash # Run ALL checks at once (tests + linting) bpsai-pair ci # Validate project structure bpsai-pair validate # See what changed git diff main...HEAD --stat git diff main...HEAD ``` ## Review Output Format ```markdown ## Code Review: [Description] ### Summary Brief assessment. ### Must Fix 1. **[File:Line]** - Issue and fix ### Should Fix 1. **[File:Line]** - Suggestion ### Consider 1. **[File:Line]** - Optional improvement ### Positive Notes - What was done well ### Verdict - [ ] Approve - [ ] Approve with comments - [ ] Request changes ``` ## Project-Specific Checks - Type hints on public functions - Docstrings on public interfaces - No hardcoded values (use config) - Tests for new functionality - Mock external services (Trello, GitHub APIs) - Follow existing patterns in codebase ## Quick Checks ```bash # Find debug statements git diff main...HEAD | grep -E "print\(|breakpoint|pdb" # Find TODOs in changes git diff main...HEAD --name-only | xargs grep -n "TODO\|FIXME" # Check for secrets git diff main...HEAD | grep -iE "password|secret|api.?key|token" ```