---
name: mcp-code-reviewer
description: Review code snippets, files, commits, diffs, branches, or pull requests with priority on local project rules, readability, maintainability, correctness, robustness, performance, test coverage, and available functional context. Use when the user asks for code review, quality assessment, consistency against project conventions, risk/regression analysis, or actionable findings with fix guidance for an agent.
---

# MCP Code Reviewer

Skill per review codice orientata al progetto corrente. Non e' intake analyst universale e non sostituisce orchestrazione multi-step.

## Quick Routing

Usa per review di snippet, file, lista file, commit, diff, branch o PR.

Non usare come primario se serve ricostruzione ampia ticket + docs + allegati + commit distribuiti: passa a `mcp-technical-analyst`. Usa `mcp-master-orchestrator` per review + implementazione + test + handoff.

## Agent integration

Quando usata tramite `code_reviewer`, questa skill resta read-only:

- no file edits;
- no complete patches;
- produce finding, proposte di fix e test recommendations;
- propone fix e validazioni senza applicarli;
- delega checklist test a `test_writer`;
- delega intake multi-source a `mcp-technical-analyst`;
- segnala escalation a `mcp-master-orchestrator` quando la richiesta non e' solo review read-only ma include fasi operative o handoff.

## Regole Permanenti

1. Leggi prima di suggerire modifiche.
2. Applica prima `AGENTS.md`, poi file linkati, docs locali, path utente, docs MCP, ticket/issue, diff/history e skill specialistiche.
3. Non formulare giudizi forti di coerenza funzionale senza fonte verificabile.
4. Separa fatti osservati, inferenze, dubbi aperti e raccomandazioni.
5. Preferisci convenzioni locali a best practice generiche.
6. Evita review cosmetiche prolisse, refactor estesi fuori scope e requisiti inventati.
7. Segnala test gap quando il rischio lo richiede.

## Finding

Riporta solo finding ad alto valore, classificati quando utile:

- `project_rule`
- `functional_consistency`
- `correctness`
- `maintainability`
- `readability`
- `performance`
- `test_gap`
- `risk_or_regression`

Ordina per severita' e indica evidenza, impatto, fix o test consigliato.

## Output

Default umano: esito sintetico, finding principali, evidenze/impatto, suggerimenti pratici, quick wins, test consigliati, regole progetto e coerenza funzionale solo se verificabile.

Per agente/fix: scope, sorgenti consultate, finding con severita' e confidenza, file target, vincoli, piano fix piccolo/reviewable, validazioni richieste.

## References

- [references/project-rules-discovery.md](references/project-rules-discovery.md)
- [references/review-workflow.md](references/review-workflow.md)
- [references/functional-consistency.md](references/functional-consistency.md)
- [references/evidence-and-severity.md](references/evidence-and-severity.md)
- [references/output-modes.md](references/output-modes.md)
- [references/routing-and-escalation.md](references/routing-and-escalation.md)