バス機能をReducer + Freezed State + Repositoryパターンにリファクタリング by kantacky · Pull Request #513 · fun-dotto/dotto-flutter

kantacky · 2026-04-06T09:49:27Z

やったこと

バス機能のアーキテクチャをReducer + Freezed State + Repositoryパターンにリファクタリング
- ドメインモデル（BusStop, BusTripStop, BusTrip）を@freezedクラスに変換
- BusRepositoryをシングルトンからabstract + Implパターンに変更し、lib/repository/に移動
- 7つの個別コントローラーを1つのBusReducer + BusState（freezed）に統合
- Widget層をBusReducer経由のデータ取得に移行（BusScreenをHookConsumerWidget化）
- BusCardからprovider依存を除去し、パラメータでデータを受け取る形に変更
- 旧コントローラー（7ファイル）と��リポジトリを削除

確認したこと

task build でコード生成が正常に完了すること
task format でフォーマットが通ること
task analyze でバス関連ファイルにエラー・警告がないこと

UI 差分

UI変更なし（内部リファクタリングのみ）

メモ

BusRepository.changeDirectionOnCurrentLocationは未使用だったため削除
formatDurationはトップレベル関数としてbus_reducer.dartに配置

ドメインモデルをimmutableなfreezedクラスに移行し、 copyWithや等値比較を自動生成できるようにする。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

シングルトンを廃止し、WidgetRef依存を除去することで他のRepositoryと統一されたパターンに揃える。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

個別のUI状態コントローラーを1つのfreezed Stateと Reducerに集約し、状態管理を一元化する。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

BusScreenをHookConsumerWidgetに変更し、BusCardはパラメータでデータを受け取る形にしてprovider依存を除去する。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

BusReducerへの移行が完了したため、不要になった 7つの個別コントローラーとfeature内リポジトリを削除する。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

kantacky · 2026-04-06T11:15:23Z

動作確認済み

Copilot

Pull request overview

バス機能の状態管理・ドメインモデル・データ取得層を整理し、Reducer + Freezed State + Repository パターンへ移行するためのリファクタリングです。

Changes:

バス関連ドメイン（BusStop/BusTrip/BusTripStop）を @freezed 化し、lib/domain 配下へ移動
既存の複数Riverpodコントローラを BusReducer + BusState に統合し、画面側は busReducerProvider 経由に集約
BusRepository を abstract + Impl に再構成し、lib/repository/ に移設・Provider登録

Reviewed changes

Copilot reviewed 22 out of 23 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
lib/repository/repository_provider.dart	BusRepository の Provider を追加
lib/repository/bus_repository.dart	バスデータ取得の Repository 抽象/実装を追加
lib/feature/root/root_screen.dart	Bus画面 import を新しい `bus_screen.dart` に更新
lib/feature/bus/widget/bus_card_home.dart	旧ホーム用バスカード（旧Provider依存）を削除
lib/feature/bus/repository/bus_repository.dart	旧シングルトンRepositoryを削除
lib/feature/bus/domain/bus_stop.dart	旧feature配下のBusStopモデルを削除（domainへ移動）
lib/feature/bus/controller/my_bus_stop_controller.dart	旧コントローラ削除（Reducerへ統合）
lib/feature/bus/controller/bus_stops_controller.dart	旧コントローラ削除（Reducerへ統合）
lib/feature/bus/controller/bus_polling_controller.dart	旧コントローラ削除（Reducerへ統合）
lib/feature/bus/controller/bus_is_weekday_controller.dart	旧コントローラ削除（Reducerへ統合）
lib/feature/bus/controller/bus_is_to_controller.dart	旧コントローラ削除（Reducerへ統合）
lib/feature/bus/controller/bus_is_scrolled_controller.dart	旧コントローラ削除（Reducerへ統合）
lib/feature/bus/controller/bus_data_controller.dart	旧コントローラ削除（Reducerへ統合）
lib/feature/bus/bus.dart	旧BusScreen実装を削除（`bus_screen.dart` へ）
lib/feature/bus/bus_timetable.dart	新domainモデル/format関数参照へ更新
lib/feature/bus/bus_stop_select.dart	busReducerProvider 経由でバス停選択に更新
lib/feature/bus/bus_state.dart	Freezed の BusState を追加
lib/feature/bus/bus_screen.dart	HookConsumerWidget ベースの新Bus画面を追加
lib/feature/bus/bus_reducer.dart	BusReducer（AsyncNotifier + polling + actions）を追加
lib/feature/bus/bus_card.dart	Provider依存を一部削減しつつ、Reducer操作へ接続
lib/domain/bus_type.dart	BusType を domain に追加
lib/domain/bus_trip.dart	BusTrip/BusTripStop を Freezed 化して domain へ移行
lib/domain/bus_stop.dart	BusStop を Freezed 化して domain へ移行

Comments suppressed due to low confidence (2)

lib/feature/bus/bus_timetable.dart:2

時刻フォーマットのためだけに bus_reducer.dart に依存しており、UI層がReducer/Provider定義へ引きずられる構造になっています。formatDuration は純粋関数なので、busのutil（例: bus_utils.dart）等に移してUI側はそれだけをimportする方が依存関係が明確になります。
lib/feature/bus/bus_card.dart:69
PR説明では「BusCardからprovider依存を除去」とありますが、この箇所で busReducerProvider を直接readしておりWidgetが状態変更を握っています。onSwapPressed のようなコールバックをBusCardの引数で受け取り、親（BusScreen等）側で toggleDirection() を呼ぶ形にすると、BusCardをプレゼンテーショナルに保てます。

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-06T11:19:38Z

lib/feature/bus/bus_reducer.dart

+    if (savedId != null) {
+      final found = allStops.where((s) => s.id == savedId);
+      if (found.isNotEmpty) return found.first;
+    }


保存済みのバス停IDが無い/見つからない場合に _defaultBusStop をそのまま返しており、Firebaseから取得した allStops 側の最新情報（名称・routeList・selectable等）と不整合になる可能性があります。旧実装は常にFirebaseの stop(id=14013) を採用していたので、フォールバック時も allStops から該当IDを検索して返し、見つからない場合のみ _defaultBusStop に落とす形にすると安全です。

Suggested change

}

}

final defaultFound = allStops.where((s) => s.id == _defaultBusStop.id);

if (defaultFound.isNotEmpty) return defaultFound.first;

Copilot · 2026-04-06T11:19:39Z

lib/feature/bus/bus_screen.dart

+        WidgetsBinding.instance.addPostFrameCallback((_) {
+          if (state.isScrolled) return;
+          final currentContext = busKey.currentContext;
+          if (currentContext == null) return;
+          final box = currentContext.findRenderObject()! as RenderBox;


addPostFrameCallback を build 内で毎回登録しているため、currentTime 更新などでリビルドが発生するたびにコールバックが積まれます（isScrolled で早期returnはするものの登録自体は行われます）。スクロール処理は useEffect 等で isScrolled / tripWidgets 変更時のみ1回登録する形にすると不要なフレーム処理を避けられます。

github-actions · 2026-04-06T11:20:13Z

	2 Warnings
⚠️	このPRは大きすぎます (1008 lines)。小さなPRに分割することを検討してください。
⚠️	アプリコードが変更されていますが、テストが追加されていません。テストの追加を検討してください。

	3 Messages
📖	削除されたファイル: lib/feature/bus/bus.dart, lib/feature/bus/controller/bus_data_controller.dart, lib/feature/bus/controller/bus_is_scrolled_controller.dart, lib/feature/bus/controller/bus_is_to_controller.dart, lib/feature/bus/controller/bus_is_weekday_controller.dart, lib/feature/bus/controller/bus_polling_controller.dart, lib/feature/bus/controller/bus_stops_controller.dart, lib/feature/bus/controller/my_bus_stop_controller.dart, lib/feature/bus/domain/bus_stop.dart, lib/feature/bus/repository/bus_repository.dart, lib/feature/bus/widget/bus_card_home.dart
📖	追加されたファイル: lib/domain/bus_stop.dart, lib/feature/bus/bus_reducer.dart, lib/feature/bus/bus_screen.dart, lib/feature/bus/bus_state.dart, lib/repository/bus_repository.dart
📖	Dangerのチェックが完了しました! 🎉

Generated by 🚫 Danger

kantacky and others added 5 commits April 6, 2026 18:44

BusStop, BusTripStop, BusTripをfreezedクラスに変換

f5a0d44

ドメインモデルをimmutableなfreezedクラスに移行し、 copyWithや等値比較を自動生成できるようにする。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

BusRepositoryをabstract+Implパターンに変更しrepository層に移動

62e24a5

シングルトンを廃止し、WidgetRef依存を除去することで他のRepositoryと統一されたパターンに揃える。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

BusStateとBusReducerを作成し7つのコントローラーを統合

55ac8a8

個別のUI状態コントローラーを1つのfreezed Stateと Reducerに集約し、状態管理を一元化する。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Widget層をBusReducerに移行しprovider依存を一本化

70f2375

BusScreenをHookConsumerWidgetに変更し、BusCardはパラメータでデータを受け取る形にしてprovider依存を除去する。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

旧コントローラーと旧リポジトリを削除

abc3204

BusReducerへの移行が完了したため、不要になった 7つの個別コントローラーとfeature内リポジトリを削除する。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

kantacky added this to the 260409_ milestone Apr 6, 2026

kantacky self-assigned this Apr 6, 2026

kantacky added 3 commits April 6, 2026 20:04

Merge branch 'main' into refactor/bus-feature-architecture

3cac89f

Reorganize

354833e

Reorder default parameters in BusState for clarity

d42c3b7

kantacky marked this pull request as ready for review April 6, 2026 11:15

kantacky requested review from a team, KAZZ00777, Copilot, hikaru-0602, masaya-osuga, mikaminasaki and tanakarinna April 6, 2026 11:15

Copilot started reviewing on behalf of kantacky April 6, 2026 11:15 View session

Copilot AI reviewed Apr 6, 2026

View reviewed changes

kantacky merged commit 71da17f into main Apr 6, 2026
2 checks passed

kantacky deleted the refactor/bus-feature-architecture branch April 6, 2026 11:47

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

バス機能をReducer + Freezed State + Repositoryパターンにリファクタリング#513

バス機能をReducer + Freezed State + Repositoryパターンにリファクタリング#513
kantacky merged 8 commits intomainfrom
refactor/bus-feature-architecture

kantacky commented Apr 6, 2026

Uh oh!

kantacky commented Apr 6, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 6, 2026

Uh oh!

Copilot AI Apr 6, 2026

Uh oh!

github-actions bot commented Apr 6, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

-    }
+    }
+    final defaultFound = allStops.where((s) => s.id == _defaultBusStop.id);
+    if (defaultFound.isNotEmpty) return defaultFound.first;

Conversation

kantacky commented Apr 6, 2026

やったこと

確認したこと

UI 差分

メモ

Uh oh!

kantacky commented Apr 6, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Apr 6, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants