GetStationsByLineGroupIdList RPC追加

[TinyKitten]

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

Summary by CodeRabbit

• New Features
  • 複数の路線グループIDをまとめて指定し、一度に複数の駅情報を取得できる機能を追加しました。
  • 複数ID向けのAPIエンドポイント（一覧リクエスト）を追加し、一覧取得が可能になりました。
  • 一括取得に伴う属性付与やフィルタ処理を拡張し、既存のフィルタやレスポンス形式との整合性を維持します。

[coderabbitai]

📝 Walkthrough

Walkthrough

複数の line_group_id を一括で扱うベクトル化メソッド群を追加し、リポジトリ／ユースケース／gRPC コントローラ層を横断して get_by_line_group_id_vec / get_stations_by_line_group_id_vec を実装しています（プロト submodule 参照更新を含む）。

変更内容

Cohort / File(s)	Summary
プロトサブモジュール更新 stationapi/proto	サブモジュール参照を更新（b050805... → 7eede39...）。
ドメイン層リポジトリ stationapi/src/domain/repository/station_repository.rs	トレイトに get_by_line_group_id_vec(&[u32]) を追加。テストモックに対応実装を追加。
インフラ層リポジトリ stationapi/src/infrastructure/station_repository.rs	get_by_line_group_id_vec の実装追加。複数ID向けSQLバインド、結果マッピング、入力順序保持（CASE-based ORDER BY）、空入力時は空ベクタ返却。
ユースケース層 stationapi/src/use_case/traits/query.rs, stationapi/src/use_case/interactor/query.rs	get_stations_by_line_group_id_vec(&[u32], transport_type) を QueryUseCase に追加／実装。取得後に update_station_vec_with_attributes で属性付与するフローを追加。テストモックを拡張。
プレゼンテーション（gRPC） stationapi/src/presentation/controller/grpc.rs	gRPC メソッド get_stations_by_line_group_id_list と GetStationsByLineGroupIdListRequest 型を追加。MockQueryUseCase に get_stations_by_line_group_id_vec を追加し、コントローラ実装でユースケースを呼び出すよう変更。

シーケンス図

sequenceDiagram
    participant Client as gRPC Client
    participant API as StationApi\n(Controller)
    participant UseCase as QueryUseCase\n(Interactor)
    participant StationRepo as StationRepository\n(Infrastructure)
    participant DB as Database

    Client->>API: get_stations_by_line_group_id_list\n(GetStationsByLineGroupIdListRequest)
    API->>UseCase: get_stations_by_line_group_id_vec(line_group_ids, transport_type)
    UseCase->>StationRepo: get_by_line_group_id_vec(line_group_ids)
    StationRepo->>DB: SQL SELECT ... WHERE line_group_cd IN (...)
    DB-->>StationRepo: rows
    StationRepo-->>UseCase: Vec<Station>
    UseCase->>UseCase: update_station_vec_with_attributes(Vec<Station>)
    UseCase-->>API: Vec<Station>
    API-->>Client: MultipleStationResponse

Loading

推定コードレビュー工数

🎯 3 (中程度) | ⏱️ ~25分

関連する可能性のあるPR

• ユニットテスト拡充 #1272: リポジトリ側での get_by_line_group_id_vec 追加とテストモック拡張を扱っており強い関連性あり。
• 駅から半径300m以内のバス停を乗換可能バス停とする #1357: QueryUseCase / gRPC 層のベクトル化メソッド追加に重なる変更を含むため関連。
• 都営バス対応 #1353: インフラ／リポジトリ側でのバルク取得サポート（get_by_line_group_id_vec）を扱う変更と重複あり。

推奨レビュアー

• 10mocy

詩

🐰 みみを立ててコードを掘るよ、
いくつもの線を一緒に辿る旅、
ベクタで集めて属性を添え、
リポジトリは歌い API が返す、
うさぎの跳躍で機能は跳ねる！

🚥 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	プルリクエストのタイトル「GetStationsByLineGroupIdList RPC追加」は、変更セットの主要な変更内容（複数のline_group_idで駅を取得するRPCメソッドの追加）を正確に反映している。

_{✏️ 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 feature/tt-batch

---

No actionable comments were generated in the recent review. 🎉

---

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

[coderabbitai]

Actionable comments posted: 2

🤖 Fix all issues with AI agents

In `@stationapi/proto`:
- Line 1: CI is failing because the stationapi/proto submodule points to an
invalid commit (referenced as 6f5474…) that cannot be fetched; fix by either
updating the submodule pointer in the superproject to a valid, existing commit
and committing that change, or make the missing commit reachable from the
submodule's remote (push the commit or add a tag/branch that includes it), then
re-run git submodule update so stationapi/proto resolves successfully.

In `@stationapi/src/domain/repository/station_repository.rs`:
- Around line 226-239: get_by_line_group_id_vec currently uses unwrap_or(0) on
station.line_group_cd which makes None match when line_group_ids contains 0;
change the filter to only match when station.line_group_cd is Some and its value
(cast to u32) is contained in line_group_ids so behavior matches
get_by_line_group_id (e.g., use station.line_group_cd.map(|v| v as
u32).map_or(false, |gid| line_group_ids.contains(&gid)) or an explicit pattern
match); update the filter closure in get_by_line_group_id_vec to check for
Some(...) rather than using unwrap_or(0).