swift-code-reviewer-skill 1.0.0 → 1.1.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +32 -0
- package/README.md +76 -444
- package/SKILL.md +97 -7
- package/package.json +1 -1
- package/references/architecture-patterns.md +275 -0
- package/references/performance-review.md +193 -0
- package/references/review-workflow.md +121 -0
- package/references/swiftui-review-checklist.md +738 -0
package/CHANGELOG.md
CHANGED
|
@@ -5,11 +5,25 @@ All notable changes to the Swift Code Reviewer Agent Skill will be documented in
|
|
|
5
5
|
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
|
|
6
6
|
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
|
|
7
7
|
|
|
8
|
+
## [1.1.1] - 2026-03-24
|
|
9
|
+
|
|
10
|
+
### Fixed
|
|
11
|
+
|
|
12
|
+
- Replace `install-skill.sh` with the correct installer for this skill (previously contained the XcodeBuildMCP installer by mistake)
|
|
13
|
+
- Add `uninstall` command support to `install-skill.sh`
|
|
14
|
+
|
|
15
|
+
## [1.1.0] - 2026-03-16
|
|
16
|
+
|
|
17
|
+
### Added
|
|
18
|
+
|
|
19
|
+
- increase adjusts from Dimillian skill and more scenarios to cover
|
|
20
|
+
|
|
8
21
|
## [1.0.0] - 2026-02-10
|
|
9
22
|
|
|
10
23
|
### Added
|
|
11
24
|
|
|
12
25
|
#### Core Functionality
|
|
26
|
+
|
|
13
27
|
- **Four-phase review workflow**: Context Gathering → Analysis → Report Generation → Delivery
|
|
14
28
|
- **Multi-layer analysis** across 6 core categories:
|
|
15
29
|
- Swift Best Practices (Swift 6+ concurrency, error handling, optionals)
|
|
@@ -20,6 +34,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
20
34
|
- Project-Specific Standards (.claude/CLAUDE.md integration)
|
|
21
35
|
|
|
22
36
|
#### Review Capabilities
|
|
37
|
+
|
|
23
38
|
- **GitHub PR reviews** via `gh` CLI integration
|
|
24
39
|
- **GitLab MR reviews** via `glab` CLI integration
|
|
25
40
|
- **Git diff analysis** for uncommitted changes
|
|
@@ -28,6 +43,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
28
43
|
- **Project standards validation** via .claude/CLAUDE.md
|
|
29
44
|
|
|
30
45
|
#### Feedback System
|
|
46
|
+
|
|
31
47
|
- **Severity classification**: Critical, High, Medium, Low
|
|
32
48
|
- **Positive feedback** for good practices
|
|
33
49
|
- **Refactoring suggestions** for improvements
|
|
@@ -36,6 +52,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
36
52
|
- **Prioritized action items** for structured follow-up
|
|
37
53
|
|
|
38
54
|
#### Reference Documentation (7,700+ lines)
|
|
55
|
+
|
|
39
56
|
- **review-workflow.md** (1,131 lines): Complete review process and git integration
|
|
40
57
|
- **swift-quality-checklist.md** (928 lines): Swift 6+ patterns and best practices
|
|
41
58
|
- **swiftui-review-checklist.md** (909 lines): SwiftUI state management and modern APIs
|
|
@@ -46,12 +63,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
46
63
|
- **custom-guidelines.md** (852 lines): Project standards integration guide
|
|
47
64
|
|
|
48
65
|
#### Integration Features
|
|
66
|
+
|
|
49
67
|
- **swift-best-practices skill** integration for Swift 6+ language patterns
|
|
50
68
|
- **swiftui-expert-skill** integration for SwiftUI best practices
|
|
51
69
|
- **swiftui-performance-audit** integration for performance analysis
|
|
52
70
|
- **Independent operation** with comprehensive built-in checklists
|
|
53
71
|
|
|
54
72
|
#### Platform Support
|
|
73
|
+
|
|
55
74
|
- Swift 6.0+
|
|
56
75
|
- iOS 17+, macOS 14+, watchOS 10+, tvOS 17+, visionOS 1+
|
|
57
76
|
- GitHub and GitLab integration
|
|
@@ -68,6 +87,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
68
87
|
### Features in Detail
|
|
69
88
|
|
|
70
89
|
#### Swift Quality Checks
|
|
90
|
+
|
|
71
91
|
- Actor isolation and MainActor usage
|
|
72
92
|
- Sendable conformance validation
|
|
73
93
|
- Data race prevention
|
|
@@ -78,6 +98,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
78
98
|
- Swift API Design Guidelines compliance
|
|
79
99
|
|
|
80
100
|
#### SwiftUI Checks
|
|
101
|
+
|
|
81
102
|
- @Observable pattern adoption (iOS 17+)
|
|
82
103
|
- Property wrapper selection guide
|
|
83
104
|
- NavigationStack vs NavigationView
|
|
@@ -88,6 +109,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
88
109
|
- Accessibility compliance
|
|
89
110
|
|
|
90
111
|
#### Performance Checks
|
|
112
|
+
|
|
91
113
|
- View update optimization
|
|
92
114
|
- Equatable conformance for ViewModels
|
|
93
115
|
- ForEach identity stability
|
|
@@ -98,6 +120,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
98
120
|
- Retain cycle prevention
|
|
99
121
|
|
|
100
122
|
#### Security Checks
|
|
123
|
+
|
|
101
124
|
- Keychain usage for credentials
|
|
102
125
|
- Biometric authentication
|
|
103
126
|
- HTTPS enforcement
|
|
@@ -109,6 +132,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
109
132
|
- Sensitive data logging prevention
|
|
110
133
|
|
|
111
134
|
#### Architecture Checks
|
|
135
|
+
|
|
112
136
|
- MVVM pattern validation
|
|
113
137
|
- Repository pattern implementation
|
|
114
138
|
- Dependency injection verification
|
|
@@ -119,6 +143,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
119
143
|
- Code organization (MARK comments, extensions)
|
|
120
144
|
|
|
121
145
|
#### Project Standards
|
|
146
|
+
|
|
122
147
|
- .claude/CLAUDE.md parsing and validation
|
|
123
148
|
- Custom architecture pattern validation
|
|
124
149
|
- Design system compliance (colors, fonts, spacing)
|
|
@@ -129,6 +154,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
129
154
|
### Templates and Examples
|
|
130
155
|
|
|
131
156
|
#### Positive Feedback Templates
|
|
157
|
+
|
|
132
158
|
- Modern API adoption
|
|
133
159
|
- Architecture excellence
|
|
134
160
|
- Code quality
|
|
@@ -137,12 +163,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
137
163
|
- Security awareness
|
|
138
164
|
|
|
139
165
|
#### Issue Report Templates
|
|
166
|
+
|
|
140
167
|
- Critical issues (security, crashes, data races)
|
|
141
168
|
- High priority (performance, anti-patterns)
|
|
142
169
|
- Medium priority (code quality, documentation)
|
|
143
170
|
- Low priority (style, suggestions)
|
|
144
171
|
|
|
145
172
|
#### Refactoring Suggestions
|
|
173
|
+
|
|
146
174
|
- Extract subview
|
|
147
175
|
- Simplify complex logic
|
|
148
176
|
- Extract reusable component
|
|
@@ -151,6 +179,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
151
179
|
### Example Review Reports
|
|
152
180
|
|
|
153
181
|
Provided complete examples for:
|
|
182
|
+
|
|
154
183
|
- Reviewing uncommitted changes
|
|
155
184
|
- Reviewing against project standards
|
|
156
185
|
- Pull request reviews
|
|
@@ -181,6 +210,8 @@ Provided complete examples for:
|
|
|
181
210
|
|
|
182
211
|
## Version History Summary
|
|
183
212
|
|
|
213
|
+
- **1.1.1** (2026-03-24): Fix incorrect `install-skill.sh` (was XcodeBuildMCP installer)
|
|
214
|
+
- **1.1.0** (2026-03-16): Increase adjusts from Dimillian skill and more scenarios to cover
|
|
184
215
|
- **1.0.0** (2026-02-10): Initial release with comprehensive review capabilities
|
|
185
216
|
|
|
186
217
|
---
|
|
@@ -209,6 +240,7 @@ See [CONTRIBUTING.md](CONTRIBUTING.md) for guidelines on proposing changes.
|
|
|
209
240
|
---
|
|
210
241
|
|
|
211
242
|
**Note**: This skill follows [Semantic Versioning](https://semver.org/):
|
|
243
|
+
|
|
212
244
|
- **MAJOR**: Incompatible API changes
|
|
213
245
|
- **MINOR**: Backward-compatible functionality additions
|
|
214
246
|
- **PATCH**: Backward-compatible bug fixes
|