Code Reviewing

Skrivskyddad kodgranskning med en 4-delad checklista och ett allvarlighetsgraderat rapportformat

av huangjia2019 · huangjia2019/claude-code-engineering

Testad · Fungerar ★ 7.6/10

Code Reviewing — Skrivskyddad kodgranskning med en 4-delad checklista och ett allvarlighetsgraderat rapportformat

Vad den gör

En Claude Code-färdighet i en enda fil som förvandlar en ad-hoc "look at my code"-förfrågan till en strukturerad granskning: en checklista som täcker kvalitet, säkerhet, prestanda och underhållbarhet, följt av ett fast rapportformat med Critical/Major/Minor allvarlighetskategorier, styrkor, prioriterade rekommendationer och ett Approved / Needs Changes-utlåtande. Den utlöses när du ber om en kodgranskning, vill ha feedback på en fil, eller frågar om kod har säkerhets- eller kvalitetsproblem. Verktygsåtkomst är begränsad till Read, Grep och Glob, så granskningen redigerar eller kör aldrig något.

Testrapport

Hittade SKILL.md via GitHub tree API (en identisk kopia finns också på 04-Skills/projects/00-basic-skill/SKILL.md, HTTP 200 på båda råa hämtningarna); frontmatter parsas med name, description och allowed-tools [Read, Grep, Glob], brödtexten refererar inga externa filer eller skript, och jag hittade ingen curl|sh, base64-blobar eller injektionstext. För utdata skrev jag min egen oseedade 31-raders Express-router (SQL-strängkonkatenering, en N+1-objektloop, en cache på modulnivå, klientlevererade objektpriser) snarare än repots förannoterade src/auth.js, sedan producerade jag en baslinjegranskning innan jag öppnade brödtexten och en andra granskning som följde den strikt: baslinjen listade 6 orankade problem utan dom, skicklighetskörningen producerade 13 med radnummer, allvarlighetskategorier och ett "Request Significant Changes"-utlåtande, och dess checklista upptäckte två saker som baslinjen helt missade — ingen ägar-/auktoriseringskontroll på GET /orders/:id, och inga tester. Verklig friktion att flagga: beskrivningen annonserar "mentions reviewing changes" men allowed-tools utelämnar Bash, så skickligheten kan inte köra git diff för att hämta dessa ändringar; du måste själv namnge filer. Repot är en kinesisk språkkurskompanjon (~1k stjärnor, ingen LICENSE-fil) och färdigheten är en av ~10 undervisningsdemonstrationer, så README är handledningsprosa om färdighetsstruktur snarare än användningsdokumentation.

Testad: 2026-07-21 · Claude Code 2.x (agent harness)

Installation

mkdir -p ~/.claude/skills/code-reviewing
curl -sL -o ~/.claude/skills/code-reviewing/SKILL.md \
  "https://raw.githubusercontent.com/huangjia2019/claude-code-engineering/HEAD/04-Skills/projects/00-basic-skill/.claude/skills/code-reviewing/SKILL.md"
# project-scoped instead: swap ~/.claude for ./.claude
# single file, no dependencies; restart Claude Code to pick it up

Kommandon och exempelprompter

/code-reviewingSkrivskyddad kodgranskning med en 4-delad checklista och ett allvarlighetsgraderat rapportformat

Skills triggas av vanliga förfrågningar — inga kommandon att memorera. Efter installationen aktiverar prompter som dessa skillen (på engelska):

Review this code before I merge it
Give me feedback on this pull request
Check this function for security issues

Läs hela rapporten →