tomevault-io Code Review
Review Checklist
- Correctness — Does it do what it's supposed to?
- Edge cases — What happens with null, empty, boundary values?
- Error handling — Are errors caught and handled properly?
- Security — Any injection, exposure, or auth issues?
- Performance — Any obvious bottlenecks or N+1 queries?
- Readability — Clear naming, reasonable complexity, good comments?
- Testing — Are tests present and meaningful?
PR Review via GitHub
# List open PRs
gh pr list
# View PR details
gh pr view NUMBER
# View PR diff
gh pr diff NUMBER
# Add review comment
gh pr review NUMBER --comment --body "COMMENT"
# Approve
gh pr review NUMBER --approve
# Request changes
gh pr review NUMBER --request-changes --body "REASON"
Output Format
For each issue found:
- Severity: 🔴 Critical / 🟡 Warning / 🔵 Suggestion
- Location: file:line
- Issue: What's wrong
- Fix: How to fix it
Source: JansenAnalytics/claudex — distributed by TomeVault.