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.
Converted and distributed by TomeVault — claim your Tome and manage your conversions.