Load: view .claude/skills/java-code-review/SKILL.md
Systematic code review checklist for Java projects. Covers null safety, exception handling, collections, concurrency, idioms, resource management, API design, and performance.
- "Review this class"
- "Check this PR for issues"
- "Code review the changes in PluginManager"
- "What's wrong with this code?"
> view .claude/skills/java-code-review/SKILL.md
> "Review the changes in src/main/java/org/example/UserService.java"
→ Returns findings grouped by severity (Critical → Minor)
- Null Safety - NPE risks, Optional usage
- Exception Handling - Swallowed exceptions, stack traces
- Collections & Streams - Iteration, mutability
- Concurrency - Thread safety, race conditions
- Java Idioms - equals/hashCode, builders
- Resource Management - try-with-resources
- API Design - Boolean params, validation
- Performance - String concat, N+1 queries
- Works best on focused changes (single class or PR)
- Includes positive feedback section for good practices
- Suggests tests for edge cases found during review