swift-code-reviewer-skill 1.1.0 → 1.2.0
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 +47 -6
- package/README.md +89 -472
- package/SKILL.md +74 -7
- package/bin/install.js +86 -21
- package/package.json +15 -2
- package/templates/agents/swift-code-reviewer.md +78 -0
- package/templates/commands/review.md +56 -0
package/CHANGELOG.md
CHANGED
|
@@ -5,11 +5,40 @@ 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.2.0] - 2026-04-10
|
|
9
|
+
|
|
10
|
+
### Added
|
|
11
|
+
|
|
12
|
+
- **`init` command**: Scaffold review agent + `/review` slash command into any project via `npx swift-code-reviewer-skill init`
|
|
13
|
+
- **templates/ directory**: Bundled agent (`swift-code-reviewer.md`) and command (`review.md`) templates
|
|
14
|
+
- `setup` alias for `init` in both installers
|
|
15
|
+
- `npx skills add` as primary installation method in README
|
|
16
|
+
|
|
17
|
+
### Changed
|
|
18
|
+
|
|
19
|
+
- README rewritten with `npx skills add` as primary install, `init` as step 2
|
|
20
|
+
- `install-skill.sh` and `bin/install.js` now show `init` hint after install
|
|
21
|
+
- Post-install message updated to guide users toward project scaffolding
|
|
22
|
+
|
|
23
|
+
## [1.1.1] - 2026-03-24
|
|
24
|
+
|
|
25
|
+
### Fixed
|
|
26
|
+
|
|
27
|
+
- Replace `install-skill.sh` with the correct installer for this skill (previously contained the XcodeBuildMCP installer by mistake)
|
|
28
|
+
- Add `uninstall` command support to `install-skill.sh`
|
|
29
|
+
|
|
30
|
+
## [1.1.0] - 2026-03-16
|
|
31
|
+
|
|
32
|
+
### Added
|
|
33
|
+
|
|
34
|
+
- increase adjusts from Dimillian skill and more scenarios to cover
|
|
35
|
+
|
|
8
36
|
## [1.0.0] - 2026-02-10
|
|
9
37
|
|
|
10
38
|
### Added
|
|
11
39
|
|
|
12
40
|
#### Core Functionality
|
|
41
|
+
|
|
13
42
|
- **Four-phase review workflow**: Context Gathering → Analysis → Report Generation → Delivery
|
|
14
43
|
- **Multi-layer analysis** across 6 core categories:
|
|
15
44
|
- Swift Best Practices (Swift 6+ concurrency, error handling, optionals)
|
|
@@ -20,6 +49,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
20
49
|
- Project-Specific Standards (.claude/CLAUDE.md integration)
|
|
21
50
|
|
|
22
51
|
#### Review Capabilities
|
|
52
|
+
|
|
23
53
|
- **GitHub PR reviews** via `gh` CLI integration
|
|
24
54
|
- **GitLab MR reviews** via `glab` CLI integration
|
|
25
55
|
- **Git diff analysis** for uncommitted changes
|
|
@@ -28,6 +58,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
28
58
|
- **Project standards validation** via .claude/CLAUDE.md
|
|
29
59
|
|
|
30
60
|
#### Feedback System
|
|
61
|
+
|
|
31
62
|
- **Severity classification**: Critical, High, Medium, Low
|
|
32
63
|
- **Positive feedback** for good practices
|
|
33
64
|
- **Refactoring suggestions** for improvements
|
|
@@ -36,6 +67,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
36
67
|
- **Prioritized action items** for structured follow-up
|
|
37
68
|
|
|
38
69
|
#### Reference Documentation (7,700+ lines)
|
|
70
|
+
|
|
39
71
|
- **review-workflow.md** (1,131 lines): Complete review process and git integration
|
|
40
72
|
- **swift-quality-checklist.md** (928 lines): Swift 6+ patterns and best practices
|
|
41
73
|
- **swiftui-review-checklist.md** (909 lines): SwiftUI state management and modern APIs
|
|
@@ -46,12 +78,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
46
78
|
- **custom-guidelines.md** (852 lines): Project standards integration guide
|
|
47
79
|
|
|
48
80
|
#### Integration Features
|
|
81
|
+
|
|
49
82
|
- **swift-best-practices skill** integration for Swift 6+ language patterns
|
|
50
83
|
- **swiftui-expert-skill** integration for SwiftUI best practices
|
|
51
84
|
- **swiftui-performance-audit** integration for performance analysis
|
|
52
85
|
- **Independent operation** with comprehensive built-in checklists
|
|
53
86
|
|
|
54
87
|
#### Platform Support
|
|
88
|
+
|
|
55
89
|
- Swift 6.0+
|
|
56
90
|
- iOS 17+, macOS 14+, watchOS 10+, tvOS 17+, visionOS 1+
|
|
57
91
|
- GitHub and GitLab integration
|
|
@@ -68,6 +102,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
68
102
|
### Features in Detail
|
|
69
103
|
|
|
70
104
|
#### Swift Quality Checks
|
|
105
|
+
|
|
71
106
|
- Actor isolation and MainActor usage
|
|
72
107
|
- Sendable conformance validation
|
|
73
108
|
- Data race prevention
|
|
@@ -78,6 +113,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
78
113
|
- Swift API Design Guidelines compliance
|
|
79
114
|
|
|
80
115
|
#### SwiftUI Checks
|
|
116
|
+
|
|
81
117
|
- @Observable pattern adoption (iOS 17+)
|
|
82
118
|
- Property wrapper selection guide
|
|
83
119
|
- NavigationStack vs NavigationView
|
|
@@ -88,6 +124,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
88
124
|
- Accessibility compliance
|
|
89
125
|
|
|
90
126
|
#### Performance Checks
|
|
127
|
+
|
|
91
128
|
- View update optimization
|
|
92
129
|
- Equatable conformance for ViewModels
|
|
93
130
|
- ForEach identity stability
|
|
@@ -98,6 +135,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
98
135
|
- Retain cycle prevention
|
|
99
136
|
|
|
100
137
|
#### Security Checks
|
|
138
|
+
|
|
101
139
|
- Keychain usage for credentials
|
|
102
140
|
- Biometric authentication
|
|
103
141
|
- HTTPS enforcement
|
|
@@ -109,6 +147,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
109
147
|
- Sensitive data logging prevention
|
|
110
148
|
|
|
111
149
|
#### Architecture Checks
|
|
150
|
+
|
|
112
151
|
- MVVM pattern validation
|
|
113
152
|
- Repository pattern implementation
|
|
114
153
|
- Dependency injection verification
|
|
@@ -119,6 +158,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
119
158
|
- Code organization (MARK comments, extensions)
|
|
120
159
|
|
|
121
160
|
#### Project Standards
|
|
161
|
+
|
|
122
162
|
- .claude/CLAUDE.md parsing and validation
|
|
123
163
|
- Custom architecture pattern validation
|
|
124
164
|
- Design system compliance (colors, fonts, spacing)
|
|
@@ -129,6 +169,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
129
169
|
### Templates and Examples
|
|
130
170
|
|
|
131
171
|
#### Positive Feedback Templates
|
|
172
|
+
|
|
132
173
|
- Modern API adoption
|
|
133
174
|
- Architecture excellence
|
|
134
175
|
- Code quality
|
|
@@ -137,12 +178,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
137
178
|
- Security awareness
|
|
138
179
|
|
|
139
180
|
#### Issue Report Templates
|
|
181
|
+
|
|
140
182
|
- Critical issues (security, crashes, data races)
|
|
141
183
|
- High priority (performance, anti-patterns)
|
|
142
184
|
- Medium priority (code quality, documentation)
|
|
143
185
|
- Low priority (style, suggestions)
|
|
144
186
|
|
|
145
187
|
#### Refactoring Suggestions
|
|
188
|
+
|
|
146
189
|
- Extract subview
|
|
147
190
|
- Simplify complex logic
|
|
148
191
|
- Extract reusable component
|
|
@@ -151,18 +194,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
151
194
|
### Example Review Reports
|
|
152
195
|
|
|
153
196
|
Provided complete examples for:
|
|
197
|
+
|
|
154
198
|
- Reviewing uncommitted changes
|
|
155
199
|
- Reviewing against project standards
|
|
156
200
|
- Pull request reviews
|
|
157
201
|
- File-specific reviews
|
|
158
202
|
- Multi-file reviews
|
|
159
203
|
|
|
160
|
-
## [1.1.0] - 2026-03-16
|
|
161
|
-
|
|
162
|
-
### Added
|
|
163
|
-
|
|
164
|
-
- increase adjusts from Dimillian skill and more scenarios to cover
|
|
165
|
-
|
|
166
204
|
## [Unreleased]
|
|
167
205
|
|
|
168
206
|
### Planned Features
|
|
@@ -187,6 +225,8 @@ Provided complete examples for:
|
|
|
187
225
|
|
|
188
226
|
## Version History Summary
|
|
189
227
|
|
|
228
|
+
- **1.2.0** (2026-04-10): Add `init` command with agent + /review scaffolding, `npx skills` as primary install
|
|
229
|
+
- **1.1.1** (2026-03-24): Fix incorrect `install-skill.sh` (was XcodeBuildMCP installer)
|
|
190
230
|
- **1.1.0** (2026-03-16): Increase adjusts from Dimillian skill and more scenarios to cover
|
|
191
231
|
- **1.0.0** (2026-02-10): Initial release with comprehensive review capabilities
|
|
192
232
|
|
|
@@ -216,6 +256,7 @@ See [CONTRIBUTING.md](CONTRIBUTING.md) for guidelines on proposing changes.
|
|
|
216
256
|
---
|
|
217
257
|
|
|
218
258
|
**Note**: This skill follows [Semantic Versioning](https://semver.org/):
|
|
259
|
+
|
|
219
260
|
- **MAJOR**: Incompatible API changes
|
|
220
261
|
- **MINOR**: Backward-compatible functionality additions
|
|
221
262
|
- **PATCH**: Backward-compatible bug fixes
|