Swift Code Review

Swiftコードの品質レビューを行い、Swift 6時代のベストプラクティスに基づく改善提案を行う。

概要

Swiftコードベースに対して以下の観点でレビューを実施：

• Swift 6 Strict Concurrency対応
• プロトコル指向設計の評価
• 値型 vs 参照型の適切な使用
• エラーハンドリングのベストプラクティス
• Swiftイディオムの適用

実行条件

• Swiftファイル（.swift）が対象
• コードレビュー依頼時
• Swift 6移行準備時
• プルリクエストレビュー時

プロセス

Step 1: コード取得

対象のSwiftコードを取得：

• ファイルパス指定
• ディレクトリ全体
• git diff
• PR差分

Step 2: Swift 6 Strict Concurrency チェック

## Strict Concurrency 観点

### Sendable適合
- [ ] 共有される型にSendable適合があるか
- [ ] @unchecked Sendableの使用は適切か
- [ ] 暗黙的Sendable推論に依存していないか

### Actor分離
- [ ] 適切なActor境界が設定されているか
- [ ] @MainActorの過剰使用がないか
- [ ] nonisolatedの使用は適切か

### データ競合防止
- [ ] 可変状態の共有がないか
- [ ] Task間のデータ受け渡しは安全か
- [ ] クロージャのキャプチャは適切か

Step 3: プロトコル指向設計の評価

## プロトコル設計観点

### 適切な抽象化
- [ ] プロトコルが単一責任を持っているか
- [ ] 過度な抽象化になっていないか
- [ ] Protocol Extensionを活用しているか

### 関連型の設計
- [ ] Associated Typeの制約は適切か
- [ ] Primary Associated Typeを活用しているか（Swift 5.7+）
- [ ] where句による制約は明確か

### 依存性注入
- [ ] プロトコルによる依存性注入が可能か
- [ ] テスタビリティが確保されているか

Step 4: 値型 vs 参照型の評価

## 型の選択観点

### 値型（struct, enum）
- [ ] 不変性が活かされているか
- [ ] CoWの意識があるか
- [ ] 値の意味論が適切か

### 参照型（class）
- [ ] 参照型が本当に必要か
- [ ] 継承は適切に使用されているか
- [ ] finalキーワードが適切に付与されているか

### 型の選択基準
- [ ] 同一性（identity）vs 同等性（equality）
- [ ] 可変性の要件
- [ ] パフォーマンス要件

Step 5: エラーハンドリング評価

## エラーハンドリング観点

### Swift Error Handling
- [ ] throws/try/catchが適切に使用されているか
- [ ] Result型の使用は適切か
- [ ] エラー型は具体的か（Error vs 具体的なError型）

### Optional処理
- [ ] 強制アンラップ（!）の使用は安全か
- [ ] guard letの活用
- [ ] nil合体演算子（??）のデフォルト値は適切か

### 失敗ケース
- [ ] fatalError/preconditionの使用は適切か
- [ ] 回復可能なエラーと回復不能なエラーの区別

Step 6: Swiftイディオム評価

## イディオム観点

### モダンSwift構文
- [ ] if/switch式の活用（Swift 5.9+）
- [ ] 末尾クロージャの適切な使用
- [ ] キーパス（\.）の活用

### コレクション処理
- [ ] map/filter/reduceの適切な使用
- [ ] lazy評価の検討
- [ ] Sequenceプロトコルの活用

### 型安全性
- [ ] タグ付き型（Newtypes）の活用
- [ ] Phantom Typesの検討
- [ ] @frozen/@usableFromInlineの適切な使用

出力形式

# Swift Code Review Report

## 対象
- ファイル: `path/to/file.swift`
- Swift Version: 6.x

## サマリー
| カテゴリ | 問題数 | 重要度 |
|---------|--------|--------|
| Concurrency | 2 | 🔴 High |
| Protocol Design | 1 | 🟡 Medium |
| Type Choice | 0 | 🟢 OK |
| Error Handling | 1 | 🟡 Medium |

## 詳細

### 🔴 [Concurrency] データ競合リスク
**Location**: `UserManager.swift:45`

**Issue**:
クラスプロパティ`users`が複数のTaskからアクセス可能だが、Sendable適合がない。

**現状コード**:
```swift
class UserManager {
    var users: [User] = []  // 危険: 複数Taskからアクセス可能

    func addUser(_ user: User) {
        users.append(user)
    }
}

推奨修正:

actor UserManager {
    private var users: [User] = []

    func addUser(_ user: User) {
        users.append(user)
    }
}

理由: Swift 6のStrict Concurrencyでは、共有可変状態へのアクセスはコンパイルエラーになる。 Actorを使用することで、データ競合を防止できる。

---

🟡 [Protocol] 過度な抽象化

...

## ガードレール

### 必須チェック項目
- Swift 6コンパイラ警告の確認
- Sendable適合の検証
- 強制アンラップの使用箇所確認

### 自動修正の制限
- Actor導入は手動確認必須
- プロトコル設計変更は影響範囲を確認
- エラーハンドリング変更はテスト必須

### レビュー対象外
- 生成コード（SwiftGen等）
- 外部ライブラリ
- テストコード（別スキルで対応）

## 関連スキル

- `swift-concurrency`: 並行処理の詳細レビュー
- `swift-protocol`: プロトコル設計の詳細支援
- `swiftui-component`: SwiftUI固有のレビュー