skilld
All skills
github avatar

/dotnet-design-pattern-review

@caab1f6 official
by githubgithub/awesome-copilot40k stars
5,040

Review the C#/.NET code for design pattern implementation and suggest improvements.

Ask your Agent

Use this Skill: https://skilld.dev/gh/github/awesome-copilot/dotnet-design-pattern-review

This session only. Nothing lands on disk.

SKILL.md

≈28 tokens always: the name and description. ≈785 when used: this file.

.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?
  • SOLID Principles: Single Responsibility, Open/Closed, Liskov Substitution, Interface Segregation, Dependency Inversion violations?
  • Performance: Proper async/await, resource disposal, ConfigureAwait(false), parallel processing opportunities?
  • Maintainability: Clear separation of concerns, consistent error handling, proper configuration usage?
  • Testability: Dependencies abstracted via interfaces, mockable components, async testability, AAA pattern compatibility?
  • Security: Input validation, secure credential handling, parameterized queries, safe exception handling?
  • Documentation: XML docs for public APIs, parameter/return descriptions, resource file organization?
  • Code Clarity: Meaningful names reflecting domain concepts, clear intent through patterns, self-explanatory structure?
  • Clean Code: Consistent style, appropriate method/class size, minimal complexity, eliminated duplication?

Improvement Focus Areas

  • Command Handlers: Validation in base class, consistent error handling, proper resource management
  • Factories: Dependency configuration, service provider integration, disposal patterns
  • Providers: Connection management, async patterns, exception handling and logging
  • Configuration: Data annotations, validation attributes, secure sensitive value handling
  • AI/ML Integration: Semantic Kernel patterns, structured output handling, model configuration

Provide specific, actionable recommendations for improvements aligned with the project's architecture and .NET best practices.

Source: SKILL.md on GitHub

Third-party checks

1 warning17d5 checks · Risk SAFE
  • Gen Agent Trust Hub17d

    The skill is designed to perform code reviews for .NET/C# projects and is generally safe. It identifies a potential vulnerability to indirect prompt injection because it processes user-provided code selections without explicit boundary markers or sanitization instructions.

  • Socket17d

    No alerts

  • Snyk17d

    Risk: LOW · No issues

  • Runlayer7mo

    1/1 file flagged

  • ZeroLeaks5mo

    Score: 93/100 · 2 sections analyzed

Provenance

Signed by skilld at caab1f6. This ties the file your Agent reads to that commit on GitHub. It does not review the instructions.

Last checked against GitHub 18 hours ago.

Activeupdated 7 months ago

Topics

  • csharp
  • dotnet
  • design-patterns
  • code-review
  • dependency-injection
  • architecture
  • solid-principles
  • command-pattern
  • repository-pattern
  • best-practices

README badge

README badge for github/awesome-copilot/dotnet-design-pattern-review

What it does

Reviews C# and .NET code for adherence to design patterns including Command, Factory, Dependency Injection, Repository, and Provider patterns, then provides specific recommendations aligned with SOLID principles and .NET best practices. Targets .NET projects using primary constructors, async/await, and ResourceManager for localization, without modifying code.

Generated from the current SKILL.md.

Frequently asked

Does this skill modify code or just review it?
This skill only reviews code and provides recommendations. It does not make any changes to the codebase.
What .NET patterns does this skill check for?
It reviews implementation of Command, Factory, Dependency Injection, Repository, Provider, and Resource patterns, plus SOLID principles and GoF design patterns.
Does this work with C# only or other .NET languages?
The skill targets C#/.NET code and assumes C# syntax conventions like primary constructors and ResourceManager.
Can this skill review AI/ML integration code?
Yes. It includes a focus area for Semantic Kernel patterns and structured output handling in the context of AI/ML integration.

Generated from the current SKILL.md. These answers refresh after source changes.