Skip to content

経路検索の種別モーダルで路線選択と行先表示を修正し0件表示の回帰テストを追加#5292

Merged
TinyKitten merged 1 commit intodevfrom
fix/route-search
Feb 15, 2026
Merged

経路検索の種別モーダルで路線選択と行先表示を修正し0件表示の回帰テストを追加#5292
TinyKitten merged 1 commit intodevfrom
fix/route-search

Conversation

@TinyKitten
Copy link
Copy Markdown
Member

@TinyKitten TinyKitten commented Feb 15, 2026

Summary by CodeRabbit

リリースノート

  • Tests

    • 路線選択ロジックの優先順序を検証するテストスイートを追加
    • 目的地情報に基づくフィルタリング処理の検証を強化
  • Bug Fixes

    • 路線選択の判定順序を改善し、ユーザー選択や保留中の情報をより適切に反映するよう修正
    • 目的地情報がない場合のフォールバック動作を改善

@github-actions github-actions Bot added the react label Feb 15, 2026
@TinyKitten TinyKitten self-assigned this Feb 15, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 15, 2026

📝 Walkthrough

Walkthrough

このプルリクエストは、TrainTypeListModal および関連するコンポーネント(SelectBoundModal、RouteSearchScreen)における路線選択ロジックを改善します。優先度ベースの路線選択と目的地対応のフィルタリング機能を導入し、選択された路線、駅の路線、保留中の路線から最適な路線を決定するようにします。

Changes

Cohort / File(s) Summary
SelectBoundModal の路線選択ロジック
src/components/SelectBoundModal.tsx, src/components/SelectBoundModal.test.tsx
selectedLine を atom 状態から抽出して trainTypeModalLine メモ化計算を導入。優先度ベース(selectedLine → station.line → pendingTrainType.line → line)で TrainTypeListModal に渡す路線を決定します。新しいテストスイートで優先度ロジックをカバーします。
TrainTypeListModal のフィルタリングと表示ロジック
src/components/TrainTypeListModal.tsx, src/components/TrainTypeListModal.test.tsx
目的地対応の 2 段階フィルタリングを実装(第 1 段階:選択路線を含む TrainType、第 2 段階:目的地路線を含む TrainType)。目的地の有無に基づいて title/subtitle の動的な表示制御を追加。落ち戻り処理と複数のエッジケースに対応するテストを追加します。
RouteSearchScreen の路線選択コンテキスト
src/screens/RouteSearchScreen.tsx, src/screens/RouteSearchScreen.test.tsx
currentStationLineForTrainTypeModal メモ化ヘルパーを導入して、現在の駅の路線が利用可能な路線に含まれる場合は優先します。TrainTypeListModal に渡す路線の決定ロジックを統一。新しいテストスイートで路線選択の優先度をカバーします。

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Possibly related PRs

Suggested labels

react

Poem

🐰 路線選択の迷路を、
優先度の道標でスキップ、
目的地ごとに衣替え、
テストで確かめて、
モーダルは今や完璧—ぴょん!

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PRタイトルは、モーダルの路線選択と行先表示の修正、及び回帰テスト追加という変更内容を正確に反映しており、変更セットの主要な目的を明確に示しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into dev

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/route-search

No actionable comments were generated in the recent review. 🎉


Comment @coderabbitai help to get the list of available commands and usage tips.

@TinyKitten TinyKitten merged commit 93b8c64 into dev Feb 15, 2026
6 checks passed
@TinyKitten TinyKitten deleted the fix/route-search branch February 15, 2026 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant