# PR Checklist MCP (Obbligatoria)

Usare questa checklist in ogni PR che modifica server MCP, tool schema, skill o documentazione collegata.

## Check obbligatori

- [ ] **Compatibilita' backward dei tool name**: i nomi dei tool esistenti restano invariati (salvo migrazione esplicita e documentata).
- [ ] **Validazione schema input**: ogni campo `type: "array"` dichiara sempre `items` (anche per array annidati).
- [ ] **Aggiornamento test e documentazione**: i test/smoke test rilevanti sono aggiornati ed e' stata aggiornata la documentazione locale.
- [ ] **Check JSON eval skill**: eseguire `node scripts/check-skill-evals-json.js` e verificare che tutti i `skills/*/evals/evals.json` siano JSON validi.
- [ ] **Sync subagents portabili da canonico**: eseguire `node scripts/sync-portable-agents.js` per rigenerare target runtime Codex + VS Code/Copilot.
- [ ] **Verifica subagents portabili (Codex + VS Code)**: eseguire `node scripts/check-agents-doc.js` per confermare source of truth canonica, coerenza semantica tra canonico/derivati, e assenza di riferimenti hardcoded al repository nei profili portabili.
- [ ] **Verifica fallback/compatibilita' I/O**: `project_path` resta disponibile come fallback e `save_path` continua a funzionare senza regressioni.
- [ ] **Rischio compatibilita' cross-host**: la PR indica esplicitamente il rischio/impatto su Codex, Copilot e client MCP con validazione schema severa.
- [ ] **Nota riavvio server**: quando cambia codice MCP, la PR include una nota esplicita che richiede il riavvio del server.

## Nota operativa consigliata nella PR

Aggiungere una sezione breve, per esempio:

- **Compatibilita'**: Nessuna rottura backward dei tool name / oppure migrazione documentata.
- **Cross-host risk**: Basso/Medio/Alto + motivazione.
- **Restart richiesto**: Sì/No (Sì se e' cambiato codice MCP).

## Riferimenti

- Guida viva: `docs/mcp-skills-agents-development-guide.md`
- Storico milestone completate: `docs/completed-milestones-mcp-skills.md`