Issue #5: SwiftDataエラーハンドリングの型安全性を改善

[harutiro]

概要

Issue #5「SwiftDataエラーハンドリングの改善」に対応し、文字列ベースのエラー判定を型安全なEnum定義に変更しました。

主な変更内容

1. SwiftDataContainerErrorの追加

• 新しいEnum型を定義してエラーの種類を型安全に管理
• スキーマエラー、ファイルシステムエラー、モデル設定エラー、不明なエラーの4つのケースを定義
• 各エラーケースに適切な日本語メッセージを提供

2. エラーカテゴリ化ロジックの実装

• categorize(_:)メソッドでエラーの内容を分析し、適切なカテゴリに分類
• 文字列ベースの判定から、より構造化されたエラーハンドリングに変更

3. エラー別処理の改善

• スキーマエラー: データベース削除と再作成
• ファイルシステムエラー: ディレクトリ再作成の試行
• モデル設定エラー: より単純な設定で再試行
• 不明なエラー: 通常のリカバリ処理

改善効果

Before (問題のあった実装)

if error.localizedDescription.contains("SwiftDataError") ||
   error.localizedDescription.contains("model") ||
   error.localizedDescription.contains("schema") {
    // エラーハンドリング
}

After (改善後の実装)

let categorizedError = SwiftDataContainerError.categorize(error)
switch categorizedError {
case .schemaError(let originalError):
    // スキーマエラー固有の処理
case .fileSystemError(let originalError):
    // ファイルシステムエラー固有の処理
// ... 他のケース
}

利点

• ✅ ローカライゼーションの影響を受けない堅牢なエラー処理
• ✅ エラーの種類に応じた適切な対応策の実装
• ✅ より詳細なログ出力とエラーメッセージ
• ✅ 型安全性によるコンパイル時のエラー検証
• ✅ 将来的な機能拡張への対応力向上

テスト結果

• ✅ iOS向けビルド成功
• ✅ SwiftFormatによるコードフォーマット適用済み
• ⚠️ 既存のテストに一部問題があるため、今回の変更に関連しないテスト失敗は既知の問題として扱い

関連Issue

Closes #5

🤖 Generated with Claude Code

[coderabbitai]

Warning

Rate limit exceeded

@harutiro has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 9 minutes and 17 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 4da6625 and 50fdf21.

📒 Files selected for processing (1)

• UWBViewerSystem/UWBViewerSystemApp.swift (3 hunks)

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch issue-5-improve-swiftdata-error-handling

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}