CourseRegistrationRepositoryのエラーハンドリングでError型もcatchするよう修正#591
CourseRegistrationRepositoryのエラーハンドリングでError型もcatchするよう修正#591devin-ai-integration[bot] wants to merge 1 commit into
Conversation
Co-Authored-By: Kanta Oikawa <iam@kantacky.com>
Original prompt from Kanta
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
案件: Crashlytics Issue 81acd654738b8d5208a131903420f4f8
やったこと
CourseRegistrationRepositoryImplの全メソッド (getCourseRegistrations,registerCourse,unregisterCourse) に、Dart のError型(TypeError,BuiltValueNullFieldError等)もキャッチしてDomainErrorに変換するcatch句を追加確認したこと
on DomainError { rethrow; }→on Exception catchの流れは変更なしExceptionを継承しないError型が発生した場合もDomainError(type: unknown)に変換されることを確認メモ
CourseRegistrationRepositoryImpl.getCourseRegistrationsの fatal issue が検出されたon Exception catchは Dart のError型(TypeError,BuiltValueNullFieldError等)をキャッチしない。APIレスポンスのマッピング中にErrorが発生した場合、キャッチされずにPlatformDispatcher.instance.onErrorまで伝播し、fatal: trueで Crashlytics に報告されていたcatch (e, stackTrace)句を追加し、Error型もDomainErrorに変換してから throw するようにしたLink to Devin session: https://app.devin.ai/sessions/cff41c8980ce479bb7d41f46eac5d536
Requested by: @kantacky