SkillProof tested Claude skills

Code Reviewing

Skrivebeskyttet kodegennemgang med en 4-delt tjekliste og et alvorlighedsgraderet rapportformat

Af huangjia2019 · huangjia2019/claude-code-engineering

Testet · Virker ★ 7.6/10

Code Reviewing — Skrivebeskyttet kodegennemgang med en 4-delt tjekliste og et alvorlighedsgraderet rapportformat

Hvad det gør

En enkeltfils Claude Code skill, der omdanner en ad-hoc "se på min kode"-anmodning til en struktureret gennemgang: en tjekliste, der dækker kvalitet, sikkerhed, ydeevne og vedligeholdbarhed, efterfulgt af et fast rapportformat med Critical/Major/Minor alvorligheds-buckets, styrker, prioriterede anbefalinger og en Approved / Needs Changes dom. Den udløses, når du beder om en kodegennemgang, ønsker feedback på en fil, eller spørger, om kode har sikkerheds- eller kvalitetsproblemer. Værktøjsadgang er begrænset til Read, Grep og Glob, så gennemgangen redigerer eller kører aldrig noget.

Testrapport

Fandt SKILL.md via GitHub tree API (en identisk kopi ligger også på 04-Skills/projects/00-basic-skill/SKILL.md, HTTP 200 på begge rå hentninger); frontmatter parses med name, description og allowed-tools [Read, Grep, Glob], kroppen refererer ikke til eksterne filer eller scripts, and I found no curl|sh, base64 blobs, or injection text. For output skrev jeg min egen usåede 31-linjers Express-router (SQL strengkonkatenering, en N+1 item-loop, en modul-niveau cache, klient-leverede item-priser) i stedet for repoets præ-annoterede src/auth.js, producerede derefter en baseline-gennemgang, før jeg åbnede kroppen og en anden gennemgang, der fulgte den strengt: baseline listede 6 urangerede problemer uden dom, skill-kørslen producerede 13 med linjenumre, alvorligheds-buckets og en "Request Significant Changes" dom, and its checklist surfaced two things the baseline missed outright — no ownership/authorization check on GET /orders/:id, and no tests. Reel friktion at bemærke: beskrivelsen annoncerer "nævner gennemgang af ændringer", but allowed-tools omits Bash, so the skill cannot run git diff to fetch those changes; you must name files yourself. Repoet er en kinesisksproget kursusledsager (~1k stjerner, ingen LICENSE-fil), and the skill is one of ~10 teaching demos, so the README is tutorial-prosa om skill-struktur snarere end brugsdokumentation.

Testet: 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

Kommandoer og eksempelprompter

  • /code-reviewingSkrivebeskyttet kodegennemgang med en 4-delt tjekliste og et alvorlighedsgraderet rapportformat

Skills udløses af almindelige forespørgsler — ingen kommandoer at huske. Efter installationen aktiverer prompter som disse skillen (på engelsk):

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

Læs hele rapporten →