principle-code-review SKILL.md skillA
principle-code-review from lugassawan/swe-workbench is an agent skill: a SKILL.md folder an AI agent loads when a task calls for it.
mdr add lugassawan/swe-workbench/principle-code-review@git:20260924.21f932aPins git:20260924.21f932a in mdr.lock. Pin the fingerprint instead.
- Audit grade
- A, 17 of 17 checks passed
- Kind
- SKILL.md skill
- Repository
- lugassawan/swe-workbench
- Path
skills/principle-code-review/SKILL.md- Versions recorded
- 5
- Latest version
- git:20260924.21f932a, committed Sep 24, 2026
- First version recorded
- Jul 25, 2026
- Size
- 4,477 bytes
- Repository stars
- 3
- License
- MIT
Code review heuristics — five-axis review lens (correctness, security, design integrity, test coverage, comment quality); review comment tone (observation over accusation); nitpick filtering; distinguishing a real finding from linter noise. Auto-load when writing or framing a review comment, reviewing a diff for correctness, or filtering review nitpicks.
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
<!-- preload-canary: SWB-PRELOAD-PRINCIPLE-CODE-REVIEW --> # Code Review Principles for high-signal code review. For tool-specific mechanics (diff-size routing, suggestion-block format, GitHub workflow), see the `swe-workbench:reviewer` agent. ## Five-Axis Review Lens Every review covers five axes: - **Correctness** — off-by-ones, null paths, concurrency races, lost errors, unhandled edge cases, null elements inside externally-deserialized collections (a valid payload can populate them), paired-guard predicate gaps (a check enforced by one sibling but missing in its pair). - **Security** — injection, auth/authz gaps, secrets in code, unsafe deserialization, SSRF, missing input validation at trust boundaries. - **Design integrity** — SOLID violations, leaky abstractions, tight coupling, circular deps, domain logic bleeding into infrastructure. *For complexity / duplication / length, prefer Quality-stage output over subjective comments — see `swe-workbench:workflow-development`.* - **Tests** — missing coverage on new branches, brittle tests, tests that mirror implementation rather than behavior. …
Read the whole file at its exact version.
How to install
mdr add lugassawan/swe-workbench/principle-code-review@sha256:8bd8e05bfe41a7f3The line at the top of this page pins the label git:20260924.21f932a, 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_3wwrgvac6fvhv2ky)
2 badge image fetches in 30 days. What the badge shows.
Versions
The registry has recorded 5 versions of this file, between Jul 25, 2026 and Sep 24, 2026, about one every 15 days.
| version | committed | commit | size | audit | |
|---|---|---|---|---|---|
| git:20260924.21f932a latest | 2026-09-24 | 21f932a | 4,477 B | A | view · diff |
| git:20260828.71a6399 | 2026-08-28 | 71a6399 | 4,460 B | A | view · diff |
| git:20260815.60d7983 | 2026-08-15 | 60d7983 | 4,585 B | A | view · diff |
| git:20260811.925e30f | 2026-08-11 | 925e30f | 4,491 B | A | view · diff |
| git:20260725.074b496 | 2026-07-25 | 074b496 | 4,449 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 (4477 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
lugassawan/swe-workbench · 3 stars · license MIT · pushed 2026-11-07 · branch main
API
GET https://markdownregistry.com/api/v1/artifacts/art_3wwrgvac6fvhv2ky GET https://markdownregistry.com/api/v1/resolve?ref=lugassawan/swe-workbench/principle-code-review GET https://markdownregistry.com/api/v1/blob/8bd8e05bfe41a7f3d1bfd37ae17e9f440692302b6f8a6fd1df4ebba2d26ce995
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.