dotnet-design-pattern-review

Installation
Summary

Comprehensive C#/.NET design pattern analysis with actionable improvement recommendations.

  • Evaluates six core patterns: Command, Factory, Dependency Injection, Repository, Provider, and Resource patterns against GoF and SOLID principles
  • Analyzes architecture compliance including namespace conventions, project separation, async/await patterns, and structured logging implementation
  • Reviews code quality across security, testability, maintainability, and performance dimensions with specific focus on primary constructors and ConfigureAwait usage
  • Identifies missing or incorrectly implemented patterns and suggests concrete refactoring strategies aligned with .NET best practices
  • Checks XML documentation, resource file organization, input validation, and credential handling without modifying the original code
SKILL.md

.NET/C# Design Pattern Review

Review the C#/.NET code in ${selection} for design pattern implementation and suggest improvements for the solution/project. Do not make any changes to the code, just provide a review.

Required Design Patterns

  • Command Pattern: Generic base classes (CommandHandler<TOptions>), ICommandHandler<TOptions> interface, CommandHandlerOptions inheritance, static SetupCommand(IHost host) methods
  • Factory Pattern: Complex object creation service provider integration
  • Dependency Injection: Primary constructor syntax, ArgumentNullException null checks, interface abstractions, proper service lifetimes
  • Repository Pattern: Async data access interfaces provider abstractions for connections
  • Provider Pattern: External service abstractions (database, AI), clear contracts, configuration handling
  • Resource Pattern: ResourceManager for localized messages, separate .resx files (LogMessages, ErrorMessages)

Review Checklist

  • Design Patterns: Identify patterns used. Are Command Handler, Factory, Provider, and Repository patterns correctly implemented? Missing beneficial patterns?
  • Architecture: Follow namespace conventions ({Core|Console|App|Service}.{Feature})? Proper separation between Core/Console projects? Modular and readable?
  • .NET Best Practices: Primary constructors, async/await with Task returns, ResourceManager usage, structured logging, strongly-typed configuration?
  • GoF Patterns: Command, Factory, Template Method, Strategy patterns correctly implemented?
Related skills

More from github/awesome-copilot

Installs
9.7K
GitHub Stars
32.8K
First Seen
Feb 25, 2026