RailAndBusが指定された場合はRailを先頭にソート by TinyKitten · Pull Request #1379 · TrainLCD/StationAPI

TinyKitten · 2026-01-15T15:48:39Z

Summary by CodeRabbit

リリースノート

バグ修正
- RailAndBusフィルター選択時に、駅検索結果の表示順序を改善しました。鉄道駅がバス駅の前に表示されるようになり、各交通機関タイプ内での距離順序は保持されます。

_{✏️ Tip: You can customize this high-level summary in your review settings.}

coderabbitai · 2026-01-15T15:48:50Z

📝 Walkthrough

概要

gRPCコントローラーでTransportTypeのインポートを追加し、駅コレクションをミュータブル化。RailAndBusフィルター使用時に、距離順序を保持しながら鉄道を路線バスの前に安定的に並び替えるロジックを実装した。

変更内容

コホート / ファイル	変更サマリー
TransportType関連の拡張インポート `stationapi/src/presentation/controller/grpc.rs`	ドメインGTFSエンティティからTransportTypeFilterに加えてTransportTypeをインポート
RailAndBus時の駅並び替え機能 `stationapi/src/presentation/controller/grpc.rs`	座標で取得した駅をミュータブル化し、RailAndBusフィルター使用時に大文字小文字を区別しない安定ソートで鉄道を路線バスより前に配置

推定レビュー時間

🎯 2 (Simple) | ⏱️ ~10分

ウサギの詩

🚄 線路と走る乗りもの
並ぶ距離も大切だが
鉄路がまず前へ
かしこき並び替えで
旅人の迷い晴らす ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PRのタイトルは変更内容の主要な目的を正確に反映しており、RailAndBusフィルター使用時にRailを先頭にソートする動作が明確に記述されています。

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

✨ Finishing touches

📝 Generate docstrings

🧹 Recent nitpick comments

stationapi/src/presentation/controller/grpc.rs (1)
135-142: ソートロジックは正しいですが、ユニットテストの追加を検討してください。

Rustの sort_by は安定ソートなので、コメント通り距離順は維持されます。ロジックも正しいです。

ただし、この新しいソート動作をカバーするユニットテストが見当たりません。RailAndBus フィルタ時に Rail が Bus より前にソートされることを検証するテストを追加することをお勧めします。
📝 テスト追加の提案
#[tokio::test]
async fn test_get_stations_by_coordinates_rail_and_bus_sorts_rail_first() {
    // Bus が先、Rail が後の順序で返されるケースをシミュレート
    let bus_station1 = create_test_station(1, TransportType::Bus);
    let rail_station1 = create_test_station(2, TransportType::Rail);
    let bus_station2 = create_test_station(3, TransportType::Bus);
    let rail_station2 = create_test_station(4, TransportType::Rail);
    
    let mock = MockQueryUseCase::with_stations(vec![
        bus_station1.clone(),
        rail_station1.clone(),
        bus_station2.clone(),
        rail_station2.clone(),
    ]);

    let result = mock
        .get_stations_by_coordinates(35.0, 139.0, Some(10), TransportTypeFilter::RailAndBus)
        .await
        .unwrap();

    // Rail が先頭にソートされていることを確認
    assert_eq!(result[0].transport_type, TransportType::Rail);
    assert_eq!(result[1].transport_type, TransportType::Rail);
    assert_eq!(result[2].transport_type, TransportType::Bus);
    assert_eq!(result[3].transport_type, TransportType::Bus);
    
    // 安定ソートにより、同じタイプ内での順序が維持されていることを確認
    assert_eq!(result[0].station_cd, 2); // rail_station1
    assert_eq!(result[1].station_cd, 4); // rail_station2
    assert_eq!(result[2].station_cd, 1); // bus_station1
    assert_eq!(result[3].station_cd, 3); // bus_station2
}
注意: このテストは MockQueryUseCase を直接テストしているため、実際のソートロジック（MyApi::get_stations_by_coordinates 内）をテストするには、MyApi のインスタンスを使ったテストが必要になります。

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5b51e72 and 343f359.

📒 Files selected for processing (1)

stationapi/src/presentation/controller/grpc.rs

🧰 Additional context used

📓 Path-based instructions (3)

stationapi/src/presentation/controller/**/*.rs

📄 CodeRabbit inference engine (AGENTS.md)

gRPC presentation layer controller should wire service implementations (e.g., MyApi) to generated server types in presentation/controller/grpc.rs

Files:

stationapi/src/presentation/controller/grpc.rs

stationapi/src/**/*.rs

📄 CodeRabbit inference engine (AGENTS.md)

Run cargo test --lib --package stationapi or make test-unit for unit tests focusing on entities and repository mocks without database

Files:

stationapi/src/presentation/controller/grpc.rs

**/*.rs

📄 CodeRabbit inference engine (AGENTS.md)

**/*.rs: Before committing, run cargo fmt on all Rust code
Before committing, run cargo clippy --all-targets --all-features and resolve new Clippy warnings unless covered by existing #![allow] attributes

Files:

stationapi/src/presentation/controller/grpc.rs

🧠 Learnings (7)