Code Review Checklist

Strukturert sjekkliste for PR-gjennomgang: sikkerhet, ytelse, korrekthet, merket med alvorlighetsgrad.

av LeoYeAI · LeoYeAI/openclaw-master-skills

Bestått ★ 9.6/10

Code Review Checklist — Strukturert sjekkliste for PR-gjennomgang: sikkerhet, ytelse, korrekthet, merket med alvorlighetsgrad.

Hva den gjør

En systematisk PR-gjennomgangsmetodikk som dekker sikkerhet, ytelse, korrekthet, vedlikeholdbarhet og testing, hver med en konkret sjekkliste (SQL-injeksjon, IDOR, race conditions, N+1-spørringer, minnelekkasjer, og mer), en tre-pass gjennomgangsprosess, og alvorlighetsgrader Critical/Major/Minor/Nitpick slik at forfattere vet nøyaktig hva som blokkerer merge. Brukes ved gjennomgang av pull requests, fastsettelse av teamets gjennomgangsstandarder, eller opplæring av nye gjennomgåere på hva de skal se etter.

Testrapport

Kjørte den mot et to-endepunkts Express-utdrag: en naiv gjennomgang fanget bare den åpenbare SQL-injeksjonsstrenginterpoleringen, men sjekklisten avdekket fire kritiske funn som den naive gjennomgangen helt savnet — en IDOR på ordre-ruten, et uautentisert endepunkt for tilbakestilling av passord, lagring av passord i klartekst, og ingen rate-begrensning.

Testet på: 2026-07-14 · Claude Code 2.x (agent harness)

Installer

git clone https://github.com/LeoYeAI/openclaw-master-skills
cd openclaw-master-skills
mkdir -p ~/.claude/skills
cp -r skills/code-review ~/.claude/skills/code-review-checklist

Kommandoer og eksempelprompter

/code-review-checklistStrukturert sjekkliste for PR-gjennomgang: sikkerhet, ytelse, korrekthet, merket med alvorlighetsgrad.

Skills utløses av vanlige forespørsler — ingen kommandoer å huske. Etter installasjonen aktiverer prompter som disse skillen (på engelsk):

Review this PR for boundary conditions and error paths
Check this diff for concurrency bugs, not style nits
Run a structured correctness review on this pull request

Full rapport →