code-review SKILL.md skillA
code-review from aethrox/doctrine is an agent skill: a SKILL.md folder an AI agent loads when a task calls for it.
mdr add aethrox/doctrine/code-review@git:20260809.2a45dbbPins git:20260809.2a45dbb in mdr.lock. Pin the fingerprint instead.
- Audit grade
- A, 17 of 17 checks passed
- Kind
- SKILL.md skill
- Repository
- aethrox/doctrine
- Path
skills/code-review/SKILL.md- Versions recorded
- 5
- Latest version
- git:20260809.2a45dbb, committed Aug 9, 2026
- First version recorded
- Aug 9, 2026
- Size
- 6,118 bytes
- Repository stars
- 19
- License
- MIT
Code review discipline: the approval bar is "improves code health," not "perfect," Fowler's code-smell catalog gives design/complexity concerns a name instead of a vague feeling, and small PRs keep review depth from degrading. Use when reviewing a pull request or diff, requesting a review, deciding whether a change is blockable, or leaving review feedback.
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 A reviewer's job is to keep the codebase's health moving forward on net, not to hold every change to a standard of perfection. A change that is better than the current state gets approved, even with room left for future improvement; the improvement doesn't have to happen in this diff, it has to not regress in this diff. ## Phase 1: What a reviewer actually checks In order, roughly by how much a mistake there costs to fix later: 1. **Design.** Does this change belong where it is, does its interface make sense, does it fit the system it's joining? A design problem caught here is cheap; caught after three more features build on top of it, expensive. 2. **Functionality.** Does it do what the author intended, and is that intent actually correct for the user/caller? Read the diff as if you were going to use it, not just as if you were going to compile it. 3. **Complexity.** Could this be simpler? A clever one-liner that takes five minutes to parse is a tax every future reader pays; see Phase 2 for naming the specific complexity smell rather than leaving a vague "this feels complicated." …
Read the whole file at its exact version.
How to install
mdr add aethrox/doctrine/code-review@sha256:46d6cadbd264e57eThe line at the top of this page pins the label git:20260809.2a45dbb, 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_22zbz34kakv6uinv)
4 badge image fetches in 30 days. What the badge shows.
Versions
The registry has recorded 5 versions of this file, all committed on Aug 9, 2026.
| version | committed | commit | size | audit | |
|---|---|---|---|---|---|
| git:20260809.2a45dbb latest | 2026-08-09 | 2a45dbb | 6,118 B | A | view · diff |
| git:20260809.b0bf5b5 | 2026-08-09 | b0bf5b5 | 6,015 B | A | view · diff |
| git:20260809.a266045 | 2026-08-09 | a266045 | 5,998 B | A | view · diff |
| git:20260809.5c2aad2 | 2026-08-09 | 5c2aad2 | 5,992 B | A | view · diff |
| git:20260809.21f29b0 | 2026-08-09 | 21f29b0 | 6,043 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 (6118 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
aethrox/doctrine · 19 stars · license MIT · pushed 2026-09-11 · branch main
API
GET https://markdownregistry.com/api/v1/artifacts/art_22zbz34kakv6uinv GET https://markdownregistry.com/api/v1/resolve?ref=aethrox/doctrine/code-review GET https://markdownregistry.com/api/v1/blob/46d6cadbd264e57e015667b9d52298e923456f3f92adef918b3b5b5ff46c50de
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.
More from aethrox/doctrine
Every file in aethrox/doctrine