dotnet-strict

name	dotnet-strict
description	Apply strict .NET/C# coding standards to a project or solution. Adds Roslynator and Meziantou analyzers, a comprehensive .editorconfig with 80+ diagnostic rules, naming conventions, and performance warnings. Use when the user wants to enforce strict code quality, set up analyzers, or add an .editorconfig to a .NET project.
disable-model-invocation	true
argument-hint	[solution or project path]

What This Does

Applies a strict, production-grade coding standard to a .NET project by:

1. Adding analyzer NuGet packages to the project(s)
2. Creating a comprehensive .editorconfig at the solution root
3. Verifying the setup compiles

Step 1: Determine Target

If $ARGUMENTS is provided, use it as the solution/project path. Otherwise, look for a .sln or .csproj in the current directory.

Identify all .csproj files that should get analyzers (typically src/ projects, not test projects).

Step 2: Add Analyzer Packages

Add these analyzer packages to each source project (not test projects):

dotnet add <project.csproj> package Roslynator.Analyzers
dotnet add <project.csproj> package Meziantou.Analyzer

These should be added as development-only dependencies. After adding, verify the package references include PrivateAssets="all" so they don't flow to consumers:

<PackageReference Include="Roslynator.Analyzers" Version="*">
  <PrivateAssets>all</PrivateAssets>
  <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>
<PackageReference Include="Meziantou.Analyzer" Version="*">
  <PrivateAssets>all</PrivateAssets>
  <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>

Step 3: Create .editorconfig

Create a .editorconfig file at the solution root (next to the .sln file, or at the repo root). If one already exists, merge the rules — do not overwrite existing customizations.

The .editorconfig should contain the following rules:

; Strict .NET coding standards
; Based on ASYNKRON production configuration
root = true

[*]
indent_style = space
charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = true

[*.cs]
indent_size = 4
dotnet_sort_system_directives_first = true

; --- Code Style ---

; Don't use 'this.' qualifier
dotnet_style_qualification_for_field = false:suggestion
dotnet_style_qualification_for_property = false:suggestion

; Use int over Int32
dotnet_style_predefined_type_for_locals_parameters_members = true:warning
dotnet_style_predefined_type_for_member_access = true:warning

; Require var
csharp_style_var_for_built_in_types = true:warning
csharp_style_var_when_type_is_apparent = true:warning
csharp_style_var_elsewhere = true:suggestion

; Disallow throw expressions
csharp_style_throw_expression = false:suggestion

; Braces on new lines (Allman style)
csharp_new_line_before_open_brace = all
csharp_new_line_before_else = true
csharp_new_line_before_catch = true
csharp_new_line_before_finally = true
 = 
 = 


 = file_scoped


 = 


 = public, private, protected, internal, static, extern, new, virtual, abstract, sealed, override, readonly, unsafe, volatile, async:suggestion


 = 
 = warning


 = non_public




 = suggestion
 = constant_fields
 = pascal_case_style
 = field
 = const
 = pascal_case


 = suggestion
 = static_fields
 = pascal_case_style
 = 
 = field
 = public, private, internal
 = static


 = suggestion
 = private_internal_fields
 = camel_case_underscore_style
 = 
 = field
 = private, internal
 = _
 = camel_case




 = warning

 = error

 = none

 = none

 = warning

 = warning

 = warning

 = warning




 = warning

 = warning

 = warning

 = warning

 = warning

 = warning

 = warning




 = none

 = warning

 = warning

 = warning

 = warning

 = warning

 = warning

 = warning

 = warning

 = warning

 = warning

 = warning

 = suggestion

 = warning

 = warning

 = warning

 = warning




 = warning

 = warning




 = warning
 = private, internal
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = error
 = warning
 = warning


 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning


 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = suggestion
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning
 = warning


 = 


 = 


 = 
 = lf



 = silent

Step 4: Build and Verify

dotnet build <solution-or-project>

The first build after adding analyzers will likely produce many warnings. This is expected — it shows the analyzers are working.

Report a summary of the warnings found, grouped by category:

• How many performance warnings (CA18xx)
• How many unused code warnings (RCS1163, RCS1213, CA1811, etc.)
• How many style warnings (IDE0xxx)
• How many Meziantou warnings (MA0xxx)

Step 5: Optional — Auto-fix

Suggest running roslynator to auto-fix what it can:

roslynator fix <solution-or-project>

What the Rules Cover

Performance (CA18xx): Use Span/Memory over substring, prefer Count property over Count(), use char overloads, seal internal types, prefer TryGetValue, avoid zero-length arrays.

Dead Code (RCS1163, RCS1213, CA1811, IDE0051): Unused parameters, unused type parameters, unused members, uncalled private code.

Correctness (CA2xxx): Rethrow to preserve stack, don't raise reserved exceptions, forward CancellationToken, avoid infinite recursion, use ValueTask correctly.

Culture Safety (MA0074, MA0006, CA1305): Avoid implicit culture-sensitive methods, use String.Equals, specify IFormatProvider.

Modern C# Style: File-scoped namespaces, pattern matching, null propagation, coalesce expressions, var everywhere, braces always.

Naming: Constants PascalCase, static fields PascalCase, private fields _camelCase.