csharp-dotnet-review

What problem does it solve?

Many .NET/C# projects accumulate architecture anti-patterns, EF Core inefficiencies, security misconfigurations, and performance regressions that are time-consuming to detect manually. This Skill centralizes a checklist-driven review to quickly surface critical issues across ASP.NET Core, EF Core, Minimal APIs, and related project files so teams can prioritize fixes and reduce production risk.

Core Features & Use Cases

• Architecture & Patterns: Evaluates separation of concerns, dependency injection usage, CQRS/MediatR patterns, repository abstractions, and configuration best practices.
• EF Core & Data Access: Detects N+1 queries, missing AsNoTracking, index suggestions, migration hygiene, and connection pooling issues.
• Security & Compliance: Flags missing authorization, insecure secrets handling, CORS misconfiguration, and SQL injection risks from concatenated queries.
• Performance & Operations: Recommends caching, appropriate JSON serializer settings, pagination, background task design, and high-throughput API patterns.
• Output & Integration: Produces a Health Score, a prioritized list of Critical Issues, actionable Improvements with examples, and Architecture Notes suitable for PR comments or design docs.

Quick Start

Run a review of the repository's C# source files and csproj metadata to generate a Health Score, list critical issues, and receive recommended fixes.