Code Review

Reviews C# code for bugs, security issues, and best practices. Use when this capability is needed.

tomevault-io Updated

File contents

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/await properly (avoid .Result, .Wait())
  • Prefer IReadOnlyList<T> over List<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.

tomevault-io/skills-registry/tree/main/elbruno--elbruno-agentskills--code-review commit e0c83ca1ab

Frequently asked questions

npx skillmds@latest add tomevault-io/code-review-245