Ios code review

Skill edfenton/claude-skills/ios/ios-code-review

A collection of Claude Code skills for iOS, MERN, NEAN, and shared development workflows

Install

npx -y skills add edfenton/claude-skills --skill ios-code-review

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

2 stars2 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its author says it does

Copied from the file, not written here

Review iOS code for compliance with standards, NFRs, and security policy.

SKILL.md

1.5 KB, as published. Nobody here has run it

Purpose

Review code against ios-std, ios-nfr, and ios-sec policies. Report issues, then (with approval) fix and run tests to confirm.

Arguments

--paths <glob> — Limit review scope (default: whole repo)
--no-fix — Report only, don't offer to fix

Workflow

1. Automated gates

./scripts/lint.sh      # SwiftLint
./scripts/format.sh    # SwiftFormat (check mode)

If scripts don't exist, note that repo doesn't match scaffold expectations.

2. Policy review

Review against:

ios-std — MVVM boundaries, SwiftUI conventions, project structure
ios-nfr — performance, accessibility, offline correctness, app store readiness
ios-sec — keychain usage, ATS, deep link validation, logging hygiene

For each issue, note:

Category: std | nfr | sec
Severity: must-fix | should-fix | nice-to-have
File and location
What's wrong and how to fix it

3. Report results

Summary of automated gate results + policy findings grouped by severity.

4–5. Approval gate, fix and confirm

See /shared-review-workflow for severity definitions, approval gate protocol, and fix constraints. Run /ios-unit-test to confirm no regressions after fixes.

Reference

For review checklists and common issues, see reference/ios-code-review-reference.md