ダウンロード済みエピーソードのアイコンが消える

[L4Ph]

再現手順

1. エピーソードを開いてダウンロードされる
2. 小説の目次に戻る
3. ダウンロード済みエピーソードのアイコンが表示される
4. Pull to refreshや、再度画面遷移を行う
5. ダウンロード済みエピーソードのアイコンが消える

この問題を調査してください。 @claude

[claude]

Claude finished @L4Ph's task in 3m 41s —— View job

---

調査完了 - 修正計画の承認待ち

ダウンロード済みエピソードのアイコンが消える問題の原因を特定しました。

タスクリスト

• コードベースの構造を理解する
• エピソード一覧の表示ロジックを特定する
• ダウンロード状態の管理方法を調査する
• 問題の原因を特定する
• 修正計画を作成する

---

問題の原因

SWRクライアントのrevalidate()が、watcherより後に不完全なデータを送出してしまう

詳細な動作フロー

1. Pull-to-refresh実行時:

   revalidate() 開始
   ├─ fetcher() 実行: APIからエピソードリストを取得
   │  └─ Episode { isDownloaded: false }  ← デフォルト値
   ├─ onPersist() 実行: DBにメタデータを保存
   │  └─ content カラムは保持される (更新しない)
   └─ emitData(newData) 実行  ← ★問題: isDownloaded: false が送出される★
   
   watcher() (並行動作)
   ├─ DBから watchEpisodesRange() でデータ取得
   │  └─ Episode { isDownloaded: true }  ← 正しい値
   └─ emitData(data) 実行
   
   結果: revalidate() の emitData() が最後に実行されるため、
         isDownloaded: false が最終的な表示値になる
   

2. コードの問題箇所:

   packages/riverpod_swr/lib/src/swr_client.dart:180-194

   Future<void> revalidate() async {
     if (_isFetching || _isDisposed) return;
     _isFetching = true;
     try {
       final newData = await client._executeFetch(key, fetcher, onPersist, options);
       if (_isDisposed) return;
       emitData(newData);  // ← ★問題: watcherがあっても無条件に送出★
     } finally {
       _isFetching = false;
     }
   }

修正方針

方針: watcherが存在する場合、revalidate()ではfetcherのデータを送出せず、onPersist後にwatcherからの最新データを待つ

修正内容

packages/riverpod_swr/lib/src/swr_client.dart の revalidate() メソッドを修正:

Future<void> revalidate() async {
  if (_isFetching || _isDisposed) return;

  _isFetching = true;
  try {
    final newData = await client._executeFetch(key, fetcher, onPersist, options);
    if (_isDisposed) return;
    
    // watcherが存在しない場合のみ、fetcherのデータを送出する
    // watcherがある場合は、onPersist後にwatcherが自動的に最新データを送出する
    if (watcher == null) {
      emitData(newData);
    }
  } on Object catch (e, st) {
    if (_isDisposed) return;
    emitError(e, st);
  } finally {
    _isFetching = false;
  }
}

修正の効果

• watcherが存在する場合:

  • fetcher → onPersist → DBに保存
  • watcher がDB変更を検知 → 正しい isDownloaded 値を送出 ✓

• watcherが存在しない場合:

  • 既存の動作を維持 (後方互換性)

テスト計画

1. エピソードをダウンロード
2. 目次に戻り、ダウンロードアイコンが表示されることを確認
3. Pull-to-refresh実行
4. ダウンロードアイコンが消えないことを確認 ← 修正ポイント
5. 画面遷移して戻る
6. ダウンロードアイコンが表示され続けることを確認

---

この修正でよろしいでしょうか？

承認いただければ、以下の手順で実装を進めます:

1. テストコードの作成 (TDD)
2. 実装
3. Lint チェック (info含む、ゼロ件を確認)
4. コミット

[L4Ph]

デグレしてまた発生した