駅名・路線名の表記の不備をまとめて修正 - #1665
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Limit details: You’ve used the included review currently available. Your 63 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. 📝 WalkthroughWalkthroughIPA辞書のキーを修正しました。 Changesデータと走査処理の更新
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to The PR corrects station and line-name data and related lookup behavior, with the reported validation and test checks passing; no actionable merge-blocking risk remains. Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Comment |
概要
Issue #1653 で報告された駅名・路線名の表記不備を修正し、あわせて同 Issue が推奨していたローマ字変換の一括検証を実施して、そこで見つかった綴り誤り・読み仮名誤りをまとめて直しました。
変更の種類
変更内容
Issue #1653 の指摘分
station_nameを麹町(U+9EB9)から麴町(U+9EB4)へ。東京メトロ公式の表記に合わせたNirehana→Nirehara。読み仮名ニレハラ・韓国語니레하라は元から正しかった天空树衬里→晴空塔线。Linerを「裏地」と機械翻訳したものが残っていたline_name_rは既にAbukuma Kyūkō Lineだったが、ASCII 検索用のline_name_rnにマクロンが残っておりlinesByName(name: "Abukuma Kyuko")が 0 件を返していたなお 飯田(飯田線)・学研北生駒(近鉄けいはんな線)・福崎(播但線)・エアポート快特(京急・都営浅草線)の 4 件は、本番 API・本ブランチのいずれでも報告どおりの応答を再現できませんでした。データ側は正しい値を返しているため、この PR では触れていません(アプリ側の描画ロジックの調査が必要)。
ローマ字変換の一括検証(Issue #1653 の推奨事項)
stationapi/src/domain/romaji.rsの変換器で読み仮名からローマ字を導出し、data/3!stations.csvの全 11,147 件と突き合わせました。長音・訓令式などの表記揺れを除いた「音節そのものが食い違うもの」を精査し、明らかな綴り誤りだけを修正しています。Kaebata→Kawabata(川端)、Hama-Atsuma→Hamataura(浜田浦、浜厚真の綴りが入っていた)、Semie→Semine(瀬峰)、Oshiishi→Ochiishi(落石)、Gojogawara→Goshogawara(五所川原)、Kishimoto-Tamade→Kishinosato-Tamade(岸里玉出)、Shimoichimachi→Shimo-imaichi(下今市)、Wakayamaguchi→Wakayama(貴志川線 和歌山)、Kannonji→Kan'onji(予讃線 観音寺、名鉄観音寺「カンノンジ」とは別物)station_name_kが誤っていたものコハタ→コワタ(JR 奈良線の木幡は「こはた」で別)オカチ→オカジロクドジ→ロクドウジニッセン…→ニシセン…(2001 年に読みを変更済み)station_name_rnの大文字残り 5 件 —ENSHuMORI→Enshumori(遠州森)などline_name_rnの不整合 3 件 — 阿武隈急行線・久大本線のマクロン残り、ハピラインふくい線に混入していた簡体字线_r(マクロンあり)と_rn(ASCII)の整合性違反は、これで 8 件から 0 件になりました。コード
stationapi/src/domain/ipa.rs— 誤綴りを前提にしていた IPA 辞書のキーを修正後の綴りへ追従。hatcomuta→hatchomuta、参照されなくなったshicikenjayaを削除(Shichikenjayaは辞書エントリなしで解決できる)src/index.rs—build_sstsのchunks_exact(ROW)をas_chunks::<ROW>()へ。この PR の変更点ではなく、ci.ymlの Clippy が使う stable が新しくなりclippy::chunks_exact_to_as_chunksが入ったことで落ちていたもの。直前の 停車駅・種別・経路のデータ不備をまとめて修正 #1663 は Rust ファイルを触っていないのでci.ymlの path フィルタに掛からず、このジョブ自体が走っていなかったテスト
ローカルの
make devで修正内容を実際に確認したstation(id: 2800615)→name: 麴町、station(id: 1141637)→nameRoman: NireharalinesByName(name: "Abukuma Kyuko")/linesByName(name: "Kyudai")が該当路線を返すようになった(修正前はいずれも 0 件)nameKatakanaとnameIpaの両方が追従することを確認cargo run -p data_validator→[VALID] No errors reported.make ipa-audit→ 未解決トークンは修正前から残る 4 件(disneysea/saishuniryocenter/toyamatoyopet/wespa)のみ。hatchomutaは解消make dataでgenerated/*.csvを再生成済みmake fmtが通ることmake clippyが通ること(wasm32 ターゲットを含む)make testが通ること関連Issue
Refs #1653
スクリーンショット(任意)
Summary by CodeRabbit
改善
不具合修正