Skip to content

Conversation

@stotic-dev
Copy link
Owner

@stotic-dev stotic-dev commented Jan 12, 2026

Summary

  • 家事を未完了に戻す機能を実装
  • 家族グループメンバーの管理機能(CohabitantStore)を追加
  • 家事詳細画面と承認画面に実施者名を表示

関連Issue: #61

Changes

新機能

  • 未完了に戻す機能: 承認待ち状態の家事を未完了に戻せるように実装

    • HouseworkItem.updateIncomplete(): 家事を未完了状態にリセット
    • HouseworkListStore.returnToIncomplete(): 未完了に戻す処理
    • HouseworkDetailActionContent: 未完了に戻すボタンを追加
  • 家族グループメンバー管理: CohabitantStoreを追加し、リアルタイムでメンバー情報を取得・キャッシュ

    • CohabitantStore: 家族グループメンバーのリアルタイム監視とキャッシュ
    • CohabitantMemberList: メンバーリストの管理
    • CohabitantClient: Firestoreとの連携(snapshotListener追加)
  • 実施者名の表示: 家事詳細画面と承認画面に実施者名を表示

    • HouseworkDetailItemListContent: 実施者行を追加
    • HouseworkApprovalView: 実施者名を動的に表示

アーキテクチャ改善

  • StoreファイルをModel/Store/からModel/Domain/配下の適切な場所に移動
  • ファイル構造をドメイン駆動設計に準拠するよう整理

バグ修正・改善

  • FirestoreService.removeSnapshotListnerremoveSnapshotListener(タイポ修正)
  • HouseworkItem.updateState()updatePendingApproval()updateIncomplete()に分割
  • パフォーマンス改善: CohabitantMemberList.missingMemberIds()のSetを使った最適化
  • Task管理の改善: HouseworkListStoreでlistenerTaskをTaskで管理

テスト

  • CohabitantStoreTest: 新規追加
    • addSnapshotListenerIfNeeded_add_member_case: 新しいメンバーの追加テスト
    • addSnapshotListenerIfNeeded_already_exist_member_case: 既存メンバーの重複追加防止テスト
  • HouseworkListStoreTest.returnToIncomplete(): 未完了に戻す処理のテスト追加
  • ObservationHelper: テストヘルパーを追加

Test plan

  • CohabitantStoreTestが成功
  • HouseworkListStoreTestの全テストが成功
  • 手動テスト: 家事を未完了に戻す操作
  • 手動テスト: 家事詳細画面で実施者名が表示される
  • 手動テスト: 家族グループメンバーのリアルタイム更新

🤖 Generated with Claude Code

@stotic-dev stotic-dev self-assigned this Jan 12, 2026
@stotic-dev stotic-dev added the feature 機能追加 label Jan 12, 2026
- スキルの役割を「エントリーポイント」として明確化
- 実際のレビュー処理はios-code-reviewerエージェントに委譲
- Taskツールでios-code-reviewerを起動する手順を追加
- 使用タイミングと注意事項を明記
@stotic-dev-bot
Copy link
Collaborator

stotic-dev-bot commented Jan 12, 2026

Warnings
⚠️ PRの変更行が多すぎます。500行以内にしてね!理想は400行!
⚠️

homete/Views/HouseworkDetailView/SubViews/HouseworkDetailActionContent.swift#L25 - Include a single empty line between switch cases (vertical_whitespace_between_cases)

⚠️

homete/Views/HouseworkDetailView/SubViews/HouseworkDetailActionContent.swift#L32 - Include a single empty line between switch cases (vertical_whitespace_between_cases)

Messages
📖 Project coverage: 30.41%

homete.app: Coverage: 55.74

File Coverage
CohabitantStore.swift 93.06%
CohabitantMemberList.swift 93.75%
HouseworkDetailActionContent.swift 61.48%
HouseworkIndexedDate.swift 100.0%
HouseworkListStore.swift 88.62%
HouseworkItem.swift 100.0%
HouseworkClient.swift 9.52%

hometeTests.xctest: Coverage: 99.02

File Coverage
ObservationHelper.swift 100.0%
HouseworkItemHelper.swift 97.44%
CohabitantStoreTest.swift 100.0%
HouseworkListStoreTest.swift 99.22%

Generated by 🚫 Danger Swift against 63bb760

@stotic-dev stotic-dev merged commit 5174e0c into main Jan 12, 2026
4 checks passed
@stotic-dev stotic-dev deleted the update/impl_return_to_incomplete branch January 12, 2026 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 機能追加

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants