code-review

Code Review (Backend)

Overview

Language-aware code review that delegates to the active language module for language-specific checks while enforcing the team's merged coding guidelines.

Process

1. Get resolved config from config-resolver (language, framework, guidelines)

2. Run LSP diagnostics:

   • LSP.getDiagnostics on all changed files
   • Any errors → blocker severity

3. Check against merged guidelines:

   • For each ## section in the resolved guidelines, verify code compliance
   • Map violations to severity: ## Forbidden rules → blocker, all others → suggestion

4. Delegate to language module:

   • Invoke the <lang>-coding skill from the active language module
   • This checks language-specific idioms (e.g., Go error handling, C# async patterns)

5. Check framework conventions:

   • Invoke the <lang>-frameworks skill
   • This checks framework patterns (e.g., ASP.NET controller conventions, Spring annotations)

6. Report:

   • Use the same severity format as /backend-review
   • Blockers must be fixed before the agent considers the task complete
   • Suggestions are reported but do not block completion

Automatic vs Manual

This skill runs automatically at the end of every coding task (routing flow step 6). For manual on-demand review, use the /backend-review command instead.