Code Review
When to use
Use this skill when the user asks you to review C# code, check for bugs, or audit for security issues.
Review Checklist
Bug Patterns
- •Null reference risks (missing null checks, nullable misuse)
- •Resource leaks (undisposed IDisposable objects)
- •Off-by-one errors in loops and indexing
- •Unhandled exceptions or swallowed catch blocks
Security Issues
- •SQL injection (string concatenation in queries)
- •Hardcoded secrets or connection strings
- •Path traversal vulnerabilities
- •Insecure deserialization
Best Practices
- •Follow C# naming conventions (PascalCase for public, camelCase for private)
- •Use
async/awaitproperly (avoid.Result,.Wait()) - •Prefer
IReadOnlyList<T>overList<T>in public APIs - •Use pattern matching and modern C# features
Output Format
Provide a summary with severity levels: Critical, Warning, Info.