code-review SKILL.md skillA
code-review from dzhng/skills is an agent skill: a SKILL.md folder an AI agent loads when a task calls for it.
mdr add dzhng/skills/code-review@git:20261021.8e74252Pins git:20261021.8e74252 in mdr.lock. Pin the fingerprint instead.
- Audit grade
- A, 17 of 17 checks passed
- Kind
- SKILL.md skill
- Repository
- dzhng/skills
- Path
skills/engineering/code-review/SKILL.md- Versions recorded
- 6
- Latest version
- git:20261021.8e74252, committed Oct 21, 2026
- First version recorded
- Jul 8, 2026
- Size
- 22,327 bytes
- Repository stars
- 1,377
- License
- MIT
Review changed code for naming, stale references, unnecessary complexity, and comment quality. Use after completing implementation work, before committing, or when the user asks to review or audit code.
Collected from public GitHub. Every version is saved exactly as it was and never changes. Install it at an exact version with the mdr tool, and every file is checked before it reaches your agent. The main file is checked against the SHA-256 fingerprint recorded here, and the others against the fingerprints GitHub keeps for its source commit. The automatic checks below grade the latest version, and the same checks always give the same file the same grade.
What the file says
# Code Review Review the diff or specified files against these principles. ## 1. Names must reflect current reality - Variable and function names should describe what they ARE, not what they used to be. - If the underlying mechanism changed (e.g. FUSE → NFS), all related names must update. - Ask: would a new reader be confused by this name? ## 2. No stale references - After refactoring, grep for references to the old approach — dead detection logic, abandoned feature flags, comments mentioning removed code. - If something was tried and reverted, remove ALL traces. The codebase should look like the current approach was always the plan. ## 3. Simplify detection and guard logic - A gate like "can this feature run" should check the ONE thing that actually matters. - Don't chain fallback detections (binary exists OR source exists OR toolchain exists) when one check covers it. ## 4. Comments document WHY, not WHAT HAPPENED - **Keep:** non-obvious technical discoveries, platform quirks, "if you remove this, X breaks because Y." - Good: `// com.apple.provenance causes SIGKILL when spawned as child process` - Good: `// umount while server is alive panics the macOS NFS client` …
Read the whole file at its exact version.
How to install
mdr add dzhng/skills/code-review@sha256:5b7c5de0b735d878The line at the top of this page pins the label git:20261021.8e74252, which is what most people want. This one pins the sha256 fingerprint of the contents instead, so it survives a label being moved. Either way the exact version is written to mdr.lock, and mdr install downloads and checks those contents again, so it installs them exactly or stops with an error.
[](https://markdownregistry.com/a/art_ryfullyjb6osl2hs)
4 badge image fetches in 30 days. What the badge shows.
Versions
The registry has recorded 6 versions of this file, between Jul 8, 2026 and Oct 21, 2026, about one every 21 days.
| version | committed | commit | size | audit | |
|---|---|---|---|---|---|
| git:20261021.8e74252 latest | 2026-10-21 | 8e74252 | 22,327 B | A | view · diff |
| git:20260805.969b8da | 2026-08-05 | 969b8da | 22,358 B | A | view · diff |
| git:20260719.8325c8f | 2026-07-19 | 8325c8f | 20,451 B | A | view · diff |
| git:20260719.ced8b61 | 2026-07-19 | ced8b61 | 19,401 B | A | view · diff |
| git:20260717.ff8d634 | 2026-07-17 | ff8d634 | 17,484 B | A | view · diff |
| git:20260708.f7ace9d | 2026-07-08 | f7ace9d | 14,291 B | A | view |
Audit of the latest version
- pass: Frontmatter block present
- pass: Frontmatter declares a name
- pass: Frontmatter declares a description
- pass: Size between 200 bytes and 200 KB (22327 bytes)
- pass: No zero-width or bidi control characters
- pass: No instruction hidden inside an HTML comment
- pass: No link to an exfiltration or paste host
- pass: No credential-shaped string
- pass: No instruction to send local credentials anywhere
- pass: No text hidden with inline styles
- pass: No prompt-injection phrasing
- pass: No curl or wget piped into a shell
- pass: No recursive delete of root, home or parent
- pass: No instruction to read or print local credentials
- pass: No base64 blob over 200 characters
- pass: No link to a raw IP address
- pass: No script tag
Source
dzhng/skills · 1,377 stars · license MIT · pushed 2026-10-21 · branch main
API
GET https://markdownregistry.com/api/v1/artifacts/art_ryfullyjb6osl2hs GET https://markdownregistry.com/api/v1/resolve?ref=dzhng/skills/code-review GET https://markdownregistry.com/api/v1/blob/5b7c5de0b735d8788466bc575e8877843c08b1ea02de29fc50fb6124bad03012
Your AI assistant does the legwork. You hear about the deals worth your word. Hand yours the standing instructions at modelranch.com and it joins the network that reads files like this one.
Guides: the SKILL.md format and its required fields · what to check before you install a skill · how to pin a skill to an exact version · where to find agent skills · keeping a team on the same skills · which agent file does what.