complexity

# .NET Complexity Review ## Trigger On - the team wants to find overly complex methods - cyclomatic complexity thresholds are needed in CI - maintainability metrics or coupling thresholds need to be configured ## Value - produce a concrete project delta: code, docs, config, tests, CI, or review artifact - reduce ambiguity through explicit planning, verification, and final validation skills - leave reusable project context so future tasks are faster and safer ## Do Not Use For - formatting-only work - generic analyzer setup with no complexity policy change ## Inputs - the nearest `AGENTS.md` - current analyzer settings - current maintainability limits ## Quick Start 1. Read the nearest `AGENTS.md` and confirm scope and constraints. 2. Run this skill's `Workflow` through the `Ralph Loop` until outcomes are acceptable. 3. Return the `Required Result Format` with concrete artifacts and verification evidence. ## Workflow 1. Start with the built-in maintainability analyzers before reaching for non-standard tooling. 2. Use these rules deliberately: - `CA1502` for excessive cyclomatic complexity in methods - `CA1505` for low maintainability index - `CA1506` for excessive class coupling - `CA1501` when inheritance depth is also part of the design problem 3. Keep rule severity in the root `.editorconfig`. 4. Keep metric thresholds in a checked-in `CodeMetricsConfig.txt` added as `AdditionalFiles`. 5. Pair analyzer findings with MCAF maintainability limits in `AGENTS.md`. ## Bootstrap When Missing If complexity thresholds are not configured yet: 1. Detect current state: - `rg -n "CA1501|CA1502|CA1505|CA1506|CodeMetricsConfig" -g '.editorconfig' -g '*.csproj' -g 'Directory.Build.*' .` - `rg --files -g 'CodeMetricsConfig.txt'` 2. Add severity entries for `CA1502`, `CA1505`, `CA1506`, and `CA1501` in root `.editorconfig`. 3. Add checked-in `CodeMetricsConfig.txt` and include it as `AdditionalFiles` in project props. 4. Keep maintainability limits aligned with `AGENTS.md`. 5. Run `dotnet build SOLUTION_OR_PROJECT` and return `status: configured` or `status: improved`. 6. If policy relies only on `AGENTS.md` limits with no analyzer gate by design, return `status: not_applicable`. ## Deliver - explicit complexity and maintainability policy - checked-in metric thresholds - CI commands that surface complex methods early ## Validate - method-complexity checks are enabled where the repo wants them - thresholds are versioned in repo, not held in IDE memory - complexity findings map to real refactoring decisions ## Ralph Loop Use the Ralph Loop for every task, including docs, architecture, testing, and tooling work. 1. Plan first (mandatory): - analyze current state - define target outcome, constraints, and risks - write a detailed execution plan - list final validation skills to run at the end, with order and reason 2. Execute one planned step and produce a concrete delta. 3. Review the result and capture findings with actionable next fixes. 4. Apply fixes in small batches and rerun the relevant checks or review steps. 5. Update the plan after each iteration. 6. Repeat until outcomes are acceptable or only explicit exceptions remain. 7. If a dependency is missing, bootstrap it or return `status: not_applicable` with explicit reason and fallback path. ### Required Result Format - `status`: `complete` | `clean` | `improved` | `configured` | `not_applicable` | `blocked` - `plan`: concise plan and current iteration step - `actions_taken`: concrete changes made - `validation_skills`: final skills run, or skipped with reasons - `verification`: commands, checks, or review evidence summary - `remaining`: top unresolved items or `none` For setup-only requests with no execution, return `status: configured` and exact next commands. ## Load References - [references/complexity.md](references/complexity.md) - [references/metrics.md](references/metrics.md) - [references/config.md](references/config.md) ## Example Requests - "Which analyzer finds complex methods in .NET?" - "Add a complexity gate for our C# code." - "Configure cyclomatic complexity thresholds."