近傍バス停の絞り込みを件数の上限より先に行う

[TinyKitten]

概要

#1647 の後続 PR です（base は claude/train-route-bottleneck-wgz31w）。

get_bus_stops_near_stations が、半径以内のバス停を距離順に limit_per_station 件へ切り詰めてから路線を引いていたため、路線を引けないバス停や廃止路線 (e_status != 0) のバス停が上限の枠を埋めると、その分だけ採用件数が減っていました。絞り込みを先に行い、有効なものが limit_per_station 件そろった時点で打ち切るようにします。

変更の種類

• バグ修正
• 新機能
• データの修正・追加
• リファクタリング
• ドキュメント
• CI/CD
• その他

変更内容

• src/repository.rs: get_bus_stops_near_stations を「半径以内を近い順に見る → 有効な路線を持つものだけ残す → limit_per_station 件で打ち切る」順序に変更。並びは従来どおり指定された座標の順、その中では距離順。
• src/index.rs: within_radius の doc コメントにあった「呼び出し側が件数を確定させてから路線で絞る」を実装に合わせて修正。
• docs/nearby-bus-stops.md: 「最大50件のバス停候補を取得し、その中から300m以内のものをフィルタリング」という記述を、trainRoute のボトルネックを解消し stationsNearby の並びを距離順に修正 #1647 以降の実装（半径で切ってから採用）に合わせて更新。

従来の挙動との関係

この順序は移行前の SQL をそのまま写したものでした。LATERAL 側で LIMIT を掛けてから lines を JOIN していたため、路線が引けない行はそこで落ちるだけで、その枠は埋め直されません。

LATERAL (
    SELECT s.* FROM stations s
    WHERE s.e_status = 0 AND s.transport_type = $5
    ORDER BY point(s.lat, s.lon) <-> point(ic.lat, ic.lon)
    LIMIT $4
) s
JOIN lines AS l ON s.line_cd = l.line_cd AND l.e_status = 0

data/*.csv にはバスデータが含まれないためこのリポジトリのデータでは差分は出ませんが、鉄道側には e_status != 0 の路線に紐づく駅が 15 件あり、バス側で同じ状態が起きれば近傍バス停・近傍バス路線が黙って減ります。GTFS 由来のバス停と路線が食い違わない限り応答は変わりません。

テスト

実行したコマンドと結果:

• make fmt — 成功
• make clippy — 成功（wasm32 ターゲットを含む）
• make test — 成功（418 件）
• make check — 成功

get_bus_stops_near_stations は埋め込み CSV から作る静的な索引を直接引くため、廃止路線のバス停を差し込んだケースの単体テストは追加していません（バスデータを含むフィクスチャが必要になります）。

• make fmt が通ること
• make clippy が通ること（wasm32 ターゲットを含む）
• make test が通ること

関連Issue

スクリーンショット（任意）

---

Generated by Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c2aceada-a6d1-4f6a-9da3-66092b4c920d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

Warning

Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption.

---

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