---

name: csharp-review description: > Code review against C# 14 standards. Four modes: standards, performance, architecture, full. Use for "/review", "check code", "performance review". user-invocable: true allowed-tools:

• Read
• Grep
• Glob metadata: author: csharp-standards version: "4.0.0"

---

Code Review

Review C# code against modern standards with four specialized modes.

Review Modes

Quick Start

1. Determine mode from user request
2. Scan with grep for obvious violations
3. Read files for deep analysis
4. Generate report with findings

Grep Patterns (Quick Scan)

Copy# Public classes (should be internal)
grep -rn "public class\|public sealed class" --include="*.cs" | grep -v "abstract\|record"

# Unsealed classes
grep -rn "internal class" --include="*.cs" | grep -v "sealed\|abstract\|static\|partial"

# DateTime.Now (forbidden)
grep -rn "DateTime\.\(Now\|UtcNow\|Today\)" --include="*.cs"

# == null (use 'is null')
grep -rn "== null\|!= null" --include="*.cs"

# async void (forbidden)
grep -rn "async void" --include="*.cs"

# new HttpClient (use IHttpClientFactory)
grep -rn "new HttpClient()" --include="*.cs"

# FluentAssertions (use Shouldly)
grep -rn "\.Should()\|FluentAssertions" --include="*.cs"

# Legacy collections
grep -rn "new List<\|new Dictionary<\|new HashSet<" --include="*.cs"

Report Format

Copy## Code Review Report

**Mode:** [standards|performance|architecture|full]
**Scope:** [files/folders reviewed]

### Critical (MUST fix)
- `File.cs:LINE` — [Category] Description

### Warnings (SHOULD fix)
- `File.cs:LINE` — [Category] Description

### Suggestions
- `File.cs:LINE` — Consider: description

### Compliant Patterns
- Good use of ErrorOr in OrderService
- Proper sealed internal classes in Domain/

### Summary
X critical, Y warnings, Z suggestions
Overall: [PASS|NEEDS WORK|FAIL]

Exclusions

Skip these paths:

• **/Migrations/**
• **/obj/**
• **/bin/**
• **/*.Designer.cs
• **/*.g.cs

Checklists

Load detailed checklist for deep review: