Code Review Checklist
Checklist interattiva per code review - sicurezza, performance, manutenibilita'.
Code Quality
Variabili, funzioni e classi hanno nomi descrittivi e coerenti.
Ogni funzione/classe ha una sola responsabilita'.
Non ci sono blocchi di codice ripetuti che potrebbero essere estratti.
I commenti spiegano il "perche'", non il "cosa" (gia' visibile nel codice).
Le funzioni sono corte e leggibili senza scorrere.
Security
Tutti gli input esterni sono validati prima dell'uso.
Le query usano parametri bind o ORM, non concatenazione di stringhe.
I dati mostrati all'utente sono escaped/sanitizzati.
I route/metodi protetti verificano l'identita' dell'utente.
API key, password e token usano variabili d'ambiente.
Performance
Le query al database sono ottimizzate (eager loading, batch).
I dati costosi da calcolare o recuperare sono in cache.
Moduli, immagini e componenti pesanti sono caricati lazily.
Nessuna libreria pesante aggiunta senza alternativa leggera valutata.
Testing
Ogni funzione di business logic ha almeno un test.
I test includono valori limite, null, array vuoti.
Le dipendenze esterne sono mockate correttamente nei test.
La copertura del codice modificato e' almeno all'80%.
Maintainability
Gli errori sono gestiti esplicitamente, non inghiottiti silenziosamente.
Operazioni importanti e errori sono loggati con contesto.
README, JSDoc o docstring riflettono le modifiche.
Costanti usano nomi descrittivi invece di valori letterali.
Punteggio
0 / 22 completati
- Input validato alle boundaries
- No SQL Injection possibile
- Output sanitizzato (XSS)
- Autenticazione verificata
- Nessun segreto hardcoded
Legenda Severita'
Come utilizzare Code Review Checklist
Choose a language preset (optional)
Select JavaScript, Python, Java or Go to automatically pre-select relevant checks for that stack, or manually tick each option individually.
Mark verified voices during review
Click on each of the 5 categories (Code Quality, Security, Performance, Testing, Maintainability) as you verify them on the pull request.
Check score and missing critical checks
The right panel shows the overall percentage score and highlights critical checks still to be completed before merging.
Export checklist in Markdown
Use the button "Export .md" to copy into notes (or download) a ready-to-paste Markdown report as a comment in the pull request.
Suggerimenti
- Use the checklist as a standard comment in your team's pull request template to standardize review levels among different developers.
- If security score remains low, block the merge even if other categories are complete: critical checks have absolute priority.
Domande frequenti
What's the difference between this tool and "AI Code Reviewer"?
This tool is a static and manual checklist: no code is sent to a server, mark yourself the items as you review the pull request. "AI Code Reviewer" instead analyzes automatically a block of code stuck together with artificial intelligence.
What do severity levels (critical, high, medium, and low) mean?
Critical blocks merge and must be resolved immediately (e.g., security vulnerabilities). High must be resolved in the current pull request. Medium needs to be set up before the next deployment. Low is a optional improvement that does not block.
Do these language presets completely replace my evaluation?
No, presets only select statistically most relevant voices for that language (e.g. SQL Injection for Python/Java on the backend side). They can still be manually modified based on the actual project requirements under review.
Are data from this checklist saved somewhere?
No, everything that lives only in the current browser session: reloading the page resets the checklist. To preserve the result, use the export to Markdown before closing the tab.