code-review-workflow

Generate severity-ranked code review feedback for .NET changes using Roslyn MCP analysis.

224|87|Updated Dec 15, 2018
One-click install
npx skills add https://github.com/Resgrid/Core --skill code-review-workflow-resgrid
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review-workflow
Source: https://github.com/Resgrid/Core/tree/main/.opencode/skills/code-review-workflow
Command: npx skills add https://github.com/Resgrid/Core --skill code-review-workflow-resgrid

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It solves the problem of inconsistent, hard-to-scan code reviews by providing a repeatable workflow that verifies correctness, security, performance, architecture compliance, and test coverage.

Core Features & Use Cases

  • MCP-first review flow: Runs Roslyn MCP tools (e.g., antipattern detection, diagnostics, public API discovery, reference blast radius) before manually reading code for context.
  • Severity-based findings: Produces findings separated into Critical, Warnings, and Suggestions so important issues can’t get buried.
  • Architecture + test coverage validation: Checks project dependency direction and enforces module boundary rules, then verifies changed logic is covered by tests.
  • Use case: Review a .NET pull request that changes multiple files (features, infra, API) and needs a risk-ranked, actionable review with architecture and testing implications called out.

Quick Start

Ask it to perform a full PR review for your .NET pull request with the changed C# files and the target projects so it can run MCP diagnostics, assess blast radius, check architecture boundaries, and report missing test coverage.

Frequently Asked Questions about code-review-workflow

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I perform a structured .NET code review for a pull request?

A structured .NET code review uses Roslyn MCP analysis on changed C# files to detect antipatterns, extract public APIs, map reference blast radius, and generate severity-ranked findings. It validates architecture boundaries and test coverage before manual code reading.

What is reference blast radius mapping in C# code review?

Reference blast radius mapping identifies how far changes in C# files propagate across the project graph. It assesses risk by extracting public APIs and tracing references, helping prioritize review focus on changes with wider architectural impact.

How does Roslyn MCP detect antipatterns and compiler diagnostics in C# files?

Roslyn MCP runs automated analysis on changed C# files to detect antipatterns and compiler diagnostics before manual review. This MCP-first execution flow ensures correctness and security issues are caught programmatically, producing structured severity-ranked feedback.

Can I check architecture compliance and module boundaries in a .NET PR review?

Yes, architecture compliance is checked by validating project dependency direction and enforcing module boundary rules within the project graph. This ensures changed files respect architectural constraints and identifies violations before merging.

Does this code review workflow verify test coverage for changed .NET logic?

Yes, test coverage validation maps changed logic to existing tests to identify missing coverage. It runs after Roslyn MCP diagnostics and architecture checks, ensuring modified C# files have corresponding test validation before producing final severity-ranked findings.

What's the best way to review a multi-file .NET pull request with infrastructure and API changes?

Use a severity-based review workflow that runs MCP diagnostics, assesses blast radius across all changed files, checks architecture boundaries, and reports missing test coverage. This produces risk-ranked, actionable feedback separating Critical, Warnings, and Suggestions.