Skip to content

refactor: domain_logicパッケージのFlutter依存を除去し、ThemeModeをThemeSettingとして純粋なDartパッケージ化#575

Merged
blendthink merged 6 commits into
mainfrom
refactor/GH-568
Sep 1, 2025
Merged

refactor: domain_logicパッケージのFlutter依存を除去し、ThemeModeをThemeSettingとして純粋なDartパッケージ化#575
blendthink merged 6 commits into
mainfrom
refactor/GH-568

Conversation

@blendthink
Copy link
Copy Markdown
Contributor

@blendthink blendthink commented Aug 28, 2025

概要

レビュー観点

  • domain_logicパッケージからFlutter依存が適切に除去されているか
  • ThemeModeからThemeSettingへのリネームが適切に行われているか
  • ドメインモデルの分離が適切に実装されているか
  • 既存の機能に影響がないか
  • 純粋なDartパッケージとして動作するか

レビューレベル

  • Lv1: ぱっとみて違和感がないかチェックして Approve する
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
  • Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

  • すぐに見てもらいたい ( hotfix など ) 🚀
  • 今日中に見てもらいたい 🚗
  • 今日〜明日中で見てもらいたい 🚶
  • 数日以内で見てもらいたい 🐢

画像 / 動画

Simulator.Screen.Recording.-.iPhone.16.Pro.-.2025-08-28.at.14.31.27.mp4

確認したこと

  • domain_logicパッケージからFlutter依存が除去されていること
  • ThemeModeがThemeSettingに適切にリネームされていること
  • テーマ設定の変更が正常に動作すること
  • 設定画面でテーマ設定が正しく表示されること
  • アプリ起動時に保存されたテーマ設定が適用されること
  • 純粋なDartパッケージとしてビルドできること

動作確認手順

  1. アプリを起動する
  2. 設定画面を開く
  3. テーマ設定を変更する
  4. アプリを再起動して設定が保持されていることを確認する
  5. packages/domain_logicdart pub getが正常に実行できることを確認する

備考

  • Issue #568の要件に従い、domain_logicパッケージを純粋なDartパッケージとして再構築しました
  • FlutterのThemeModeから分離したThemeSettingドメインモデルを作成し、アーキテクチャの一貫性を向上させました
  • 既存のThemeMode関連のコードを削除し、新しいThemeSettingに置き換えました
  • ドメイン層の純粋性を保ち、他のDartプロジェクトでの再利用が可能になりました

@github-actions github-actions Bot added @apps/app Application development @packages/application packages application package @packages/domain_logic packages domain_logic package @packages/domain_model packages domain_model package @packages/infrastructure packages infrastructure package labels Aug 28, 2025
@yumemi-team-review-requester yumemi-team-review-requester Bot requested review from a team, fa0311 and hidekitux and removed request for a team August 28, 2025 05:29
@github-actions
Copy link
Copy Markdown

Ready for review 🚀

@blendthink blendthink requested review from morikann and trm11tkr and removed request for fa0311 and hidekitux August 28, 2025 05:31
Base automatically changed from refactor/GH-567 to main August 29, 2025 01:54
@blendthink blendthink enabled auto-merge August 29, 2025 01:58
Copy link
Copy Markdown
Contributor

@morikann morikann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご対応ありがとうございます!一点だけコメントさせていただきました🙏

Comment thread apps/app/lib/presentation/providers/theme_setting_provider.dart Outdated
Copy link
Copy Markdown
Contributor

@morikann morikann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM です!ご対応ありがとうございます!

Merged via the queue into main with commit 10390db Sep 1, 2025
16 checks passed
@blendthink blendthink deleted the refactor/GH-568 branch September 1, 2025 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

@apps/app Application development @packages/application packages application package @packages/domain_logic packages domain_logic package @packages/domain_model packages domain_model package @packages/infrastructure packages infrastructure package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improve]: domain_logicパッケージのFlutter依存を除去して純粋なDartパッケージ化

2 participants