lh-code-review SKILL.md skillA

lh-code-review from jrodrigopuca/local-agents is an agent skill: a SKILL.md folder an AI agent loads when a task calls for it.

Install it, pinned
mdr add jrodrigopuca/local-agents/lh-code-review@v1.1

Pins v1.1 in mdr.lock. Pin the fingerprint instead.

Audit grade
A, 17 of 17 checks passed
Kind
SKILL.md skill
Repository
jrodrigopuca/local-agents
Path
apple-dev/skills/lh-code-review/SKILL.md
Versions recorded
1
Latest version
v1.1, committed Oct 14, 2026
Size
5,909 bytes
Repository stars
2
License
Apache-2.0

Reviewing Swift/SwiftUI code as mentor: the severity system, the Apple-specific defect checklist, and review-as-teaching. Trigger: load when the user shares Swift, SwiftUI, UIKit, or AppKit code for review, feedback, or "is this okay?".

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

_Verified against Xcode 26 / iOS 26 / macOS 26 (September 2026). Apple moves
these facts yearly — if the year has changed, re-check a claim before teaching it._

## Critical Patterns

### 1. Review in three passes, report by severity

Pass 1 — **correctness & safety** (🔴): retain cycles, force unwraps, data
races, main-thread violations, leaked resources. Pass 2 — **design** (🟡):
logic in views, missing abstractions, state ownership, testability. Pass 3 —
**style** (🟢): naming, idiom, formatting. Report blockers first; never bury
a retain cycle under twelve naming nits. Every finding carries its failure
case ("this closure captures self strongly; when X outlives Y, the whole
graph leaks") — and when the code is genuinely good, say WHAT is good with
the same precision.

### 2. The Swift-specific 🔴 checklist

- **Retain cycles**: `self` captured strongly in escaping closures stored by
  `self` (handlers, subscriptions, timers); delegates declared strong. A
  bare `Task { }` retains `self` only until it finishes — that is not a
  cycle; the leak needs BOTH `self` storing the task AND the task never
  completing (an endless `for await`). `[weak self]` sprinkled on every Task
…

Read the whole file at its exact version.

How to install

The exact contents, by fingerprint
mdr add jrodrigopuca/local-agents/lh-code-review@sha256:c7a04205ccbd01da

The line at the top of this page pins the label v1.1, 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.

Badge

mdr badge

[![mdr](https://markdownregistry.com/badge/art_fuepep3ujusrvsax.svg)](https://markdownregistry.com/a/art_fuepep3ujusrvsax)

3 badge image fetches in 30 days. What the badge shows.

Versions

The registry has recorded 1 version of this file, committed on Oct 14, 2026.

versioncommittedcommitsizeaudit
v1.1 latest2026-10-14 9571e8f 5,909 BA view

Audit of the latest version

A  17 of 17 checks passed. Fixed automatic checks with no AI involved, so the same checks give the same answer every run.
  • pass: Frontmatter block present
  • pass: Frontmatter declares a name
  • pass: Frontmatter declares a description
  • pass: Size between 200 bytes and 200 KB (5909 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

GitHub

jrodrigopuca/local-agents · 2 stars · license Apache-2.0 · pushed 2026-10-23 · branch main

API

GET https://markdownregistry.com/api/v1/artifacts/art_fuepep3ujusrvsax
GET https://markdownregistry.com/api/v1/resolve?ref=jrodrigopuca/local-agents/lh-code-review
GET https://markdownregistry.com/api/v1/blob/c7a04205ccbd01da36cac4d7b71e13d2b3970bb96e5424b74c74d226a12c78e2

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 jrodrigopuca/local-agents

AGENTS.md AGENTS.md file
jrodrigopuca/local-agents · AGENTS.md
git:20261022.50892c2 · audit A · 2 stars
AGENTS.md@apple-dev AGENTS.md file
jrodrigopuca/local-agents · apple-dev/AGENTS.md · Use this agent for Swift, iOS, and macOS development as a mentor — it teaches you to solve alone (Socratic first)…
git:20261014.9571e8f · audit A · 2 stars
lh-debugging SKILL.md skill
jrodrigopuca/local-agents · apple-dev/skills/lh-debugging/SKILL.md · Debugging Apple-platform issues as taught method: crashes, leaks, hangs, and weird behavior — Socratic first, tools…
v1.1 · audit A · 2 stars
lh-shipping SKILL.md skill
jrodrigopuca/local-agents · apple-dev/skills/lh-shipping/SKILL.md · Getting apps into users' hands: signing, provisioning, TestFlight, App Review, and macOS-specific distribution…
v1.1 · audit A · 2 stars
lh-state-architecture SKILL.md skill
jrodrigopuca/local-agents · apple-dev/skills/lh-state-architecture/SKILL.md · SwiftUI state management and app architecture: ownership, MVVM shape, navigation, dependency injection, project…
v1.1 · audit A · 2 stars
AGENTS.md@architect AGENTS.md file
jrodrigopuca/local-agents · architect/AGENTS.md · Use this agent for software architecture guidance, design patterns, expensive-to-reverse technical decisions…
git:20261021.e01dd0b · audit A · 2 stars
lh-design-review SKILL.md skill
jrodrigopuca/local-agents · architect/skills/lh-design-review/SKILL.md · How to review a codebase's architecture or a proposed design: dependency direction, boundary integrity, coupling…
v1.1 · audit A · 2 stars
lh-mentoring SKILL.md skill
jrodrigopuca/local-agents · architect/skills/lh-mentoring/SKILL.md · Turning a correction into learning without lecturing. Trigger: load when a real misconception would hurt the user…
v1.1 · audit A · 2 stars
lh-tradeoffs SKILL.md skill
jrodrigopuca/local-agents · architect/skills/lh-tradeoffs/SKILL.md · Method for analyzing and presenting expensive-to-reverse technical decisions: drivers, options, tradeoff table…
v1.1 · audit A · 2 stars
AGENTS.md@data-ml AGENTS.md file
jrodrigopuca/local-agents · data-ml/AGENTS.md · Use this agent for data pipelines, ML modeling, and LLM integration — ingestion/quality, honest evaluation, and…
git:20261014.9571e8f · audit A · 2 stars
lh-data-pipelines SKILL.md skill
jrodrigopuca/local-agents · data-ml/skills/lh-data-pipelines/SKILL.md · Moving data reliably: ingestion, transformation, quality validation, storage, and pipeline design. Trigger: load when…
v1.1 · audit A · 2 stars
lh-llm-integration SKILL.md skill
jrodrigopuca/local-agents · data-ml/skills/lh-llm-integration/SKILL.md · Building reliable product features on LLMs: RAG, prompting-as-code, output validation, evals, and cost/latency…
v1.1 · audit A · 2 stars

Every file in jrodrigopuca/local-agents

Browse by kind, by grade A, or by owner.