direction_idがNULLのトリップに存在するバス停も正しく取り込むよう修正

[TinyKitten]

GTFSの仕様ではdirection_idはオプショナルであり、NULLの場合でも
バス停は有効な停留所として扱うべきである。

これまではvariant_only_with_neighbors CTEで「direction_id IS NOT NULL」
のトリップにのみ存在するバス停をフィルタリングしていたため、
早81の原宿駅前や渋谷駅東口などのバス停がレスポンスから除外されていた。

Summary by CodeRabbit

リリースノート

• バグ修正
  • ルート変種に属する停留所の取扱いを改善しました。これまではある方向IDに紐づかない停留所が除外されるケースがありましたが、今回の修正でそうした停留所も適切に考慮され、隣接する停留所情報がある場合は優先的に配置されるようになり、ルート表現の一貫性と完全性が向上しました。

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

[coderabbitai]

📝 Walkthrough

ウォークスルー

stationapi/src/import.rsのbuild_stop_route_mappingで、バリアント停留所の選別ロジックが変更され、非NULLのdirection_idを持つトリップの存在を必須とするフィルタ（EXISTS句）が削除されました。代わりにmain_trip_stopsへのLEFT JOINを用い、隣接停留所の有無に基づく優先順位付けを行うORDER BYを追加しています。

変更内容

コホート / ファイル(s)	変更内容
クエリロジック変更 stationapi/src/import.rs	build_stop_route_mappingのクエリからEXISTS/フィルタを削除。main_trip_stopsへのLEFT JOINを追加し、prev/next隣接情報の有無でソートする優先順位付きORDER BYを導入。主系停留所を除外するNOT EXISTSは維持。

シーケンス図

（該当なし）

推定コード レビュー手数

🎯 3 (Moderate) | ⏱️ ~20 minutes

ポエム

🐰 クエリの扉をそっと開けて、
見えなかった停留所が顔を出すよ。
LEFT JOINで隣を確かめ、順序を整え、
小道も大通りも歌い始める。
新しい地図でぴょんと一歩！ 🥕

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	タイトルはPRの主要な変更内容を正確に反映しており、direction_idがNULLのトリップに存在するバス停の取り込みに関する修正という核心を明確に表現している。
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing touches

• 📝 Generate docstrings

---

📜 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 f206adb and 82a2010.

📒 Files selected for processing (1)

• stationapi/src/import.rs

🧰 Additional context used

📓 Path-based instructions (2)

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/import.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/import.rs

🧠 Learnings (1)

📚 Learning: 2025-11-25T10:50:36.694Z

Learnt from: CR
Repo: TrainLCD/StationAPI PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T10:50:36.694Z
Learning: Applies to stationapi/src/use_case/interactor/query.rs : When modifying `QueryInteractor`, ensure enrichment steps (companies, train types, line symbols) still behave as expected and verify helper methods such as `update_station_vec_with_attributes` and `build_route_tree_map`

Applied to files:

• stationapi/src/import.rs

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: Test Suite
• GitHub Check: Test Suite

🔇 Additional comments (1)

stationapi/src/import.rs (1)

1578-1871: I need the review comment to rewrite. Please provide the original review comment within <review_comment> tags or share the feedback you'd like me to verify and rewrite.

---

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

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

stationapi/src/import.rs (1)

1639-1666: direction_idがNULLのトリップに存在するバス停を正しく取り込む修正を承認します。

この変更により、以下の改善が実現されます：

1. NULL direction_idトリップの包含: variant_only_with_neighborsCTE内にdirection_id IS NOT NULLフィルタがないため、NULL direction_idのトリップにのみ存在するバス停（原宿駅前など）が正しく含まれるようになります。

2. 優先順位ベースの選択: LEFT JOINで隣接バス停がメイントリップに存在するかをチェックし、CASE式で優先順位付けすることで、位置推定の精度が最も高いレコードを選択します：

   • 優先度0: 両隣がメイントリップに存在（最良の位置推定）
   • 優先度1: 片隣がメイントリップに存在
   • 優先度2: どちらの隣もメイントリップに存在しない

3. GTFS仕様への準拠: direction_idはオプショナルフィールドであるというGTFS仕様に正しく対応しています。

実装は論理的に正確で、DISTINCT ONとORDER BYの組み合わせにより、各(parent_stop_id, route_id)ペアに対して最適なレコードが選択されます。

コミット前に以下のコマンドを実行してください：

• cargo fmt で Rust コードをフォーマット
• cargo clippy --all-targets --all-features で警告を確認
• cargo test --lib --package stationapi で単体テストを実行

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f206adb and 82a2010.

📒 Files selected for processing (1)

• stationapi/src/import.rs

🧰 Additional context used

📓 Path-based instructions (2)

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/import.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/import.rs

🧠 Learnings (1)

📚 Learning: 2025-11-25T10:50:36.694Z

Learnt from: CR
Repo: TrainLCD/StationAPI PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T10:50:36.694Z
Learning: Applies to stationapi/src/use_case/interactor/query.rs : When modifying `QueryInteractor`, ensure enrichment steps (companies, train types, line symbols) still behave as expected and verify helper methods such as `update_station_vec_with_attributes` and `build_route_tree_map`

Applied to files:

• stationapi/src/import.rs

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: Test Suite
• GitHub Check: Test Suite

🔇 Additional comments (1)

stationapi/src/import.rs (1)

1578-1871: I need the review comment to rewrite. Please provide the original review comment within <review_comment> tags or share the feedback you'd like me to verify and rewrite.