Skip to content

fix: 名前検索RPCのローマ字検索をcase-insensitiveに変更#1417

Merged
TinyKitten merged 3 commits intodevfrom
claude/fix-case-insensitive-search-87kp1
Mar 6, 2026
Merged

fix: 名前検索RPCのローマ字検索をcase-insensitiveに変更#1417
TinyKitten merged 3 commits intodevfrom
claude/fix-case-insensitive-search-87kp1

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Mar 6, 2026

Summary

  • get_stations_by_nameクエリのstation_name_rnカラムの比較をLIKEからILIKEに変更し、ローマ字検索時に大文字小文字を区別しないようにした
  • 日本語・中国語・韓国語カラムはcase-sensitivityの影響を受けないためLIKEのまま

Test plan

  • ローマ字で小文字(例: tokyo)で検索して結果が返ることを確認
  • ローマ字で大文字混在(例: Tokyo)で検索して同じ結果が返ることを確認
  • 日本語・中国語・韓国語での検索が従来通り動作することを確認

https://claude.ai/code/session_01B3dLcBJ9hv2Njw6RXD1kwP

Summary by CodeRabbit

バグ修正

  • 駅名検索機能が大文字・小文字を区別しないようになりました。これにより、ユーザーが検索時に大文字と小文字の区別を気にせずに駅名を検索できるようになり、検索の利便性が向上しました。

claude and others added 3 commits March 6, 2026 12:20
get_stations_by_nameクエリのLIKEをILIKEに変更し、
ローマ字(station_name_rn)等の大文字小文字を区別せずに検索できるようにした。

https://claude.ai/code/session_01B3dLcBJ9hv2Njw6RXD1kwP
日本語・中国語・韓国語カラムにはcase-insensitive比較は不要なため、
ILIKEはローマ字カラム(station_name_rn)のみに適用し、
他のカラムはLIKEに戻した。

https://claude.ai/code/session_01B3dLcBJ9hv2Njw6RXD1kwP
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 78443b2f-1abe-4081-9f7d-5143440360a3

📥 Commits

Reviewing files that changed from the base of the PR and between 8fb5c84 and d8f53f3.

📒 Files selected for processing (2)
  • .sqlx/query-5f4de4e07d61f2163cda87863b73fef621d587a5056b9f8f4bf96fcd77ae1017.json
  • stationapi/src/infrastructure/station_repository.rs

📝 Walkthrough

Walkthrough

station_repository.rsにおけるget_by_nameクエリの駅名フィルタリングをLIKEからILIKEに変更し、駅名検索の大文字小文字を区別しないようにしました。

Changes

Cohort / File(s) Summary
Case-insensitive Station Name Filtering
.sqlx/query-5f4de4e07d61f2163cda87863b73fef621d587a5056b9f8f4bf96fcd77ae1017.json, stationapi/src/infrastructure/station_repository.rs
station_name_rnのフィルタリング条件をLIKEからILIKEに変更し、駅名の検索時に大文字小文字を区別しないようにしました。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

fix

Poem

🐰 ウサギが囁くよ、大小区別なく、
駅名をさがして、心やさしく。
ILIKEの魔法で、どの文字も輝く、
検索の旅路、もっと優しく。
小さな変更が、大きな幸い♨

🚥 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 プルリクエストのタイトルはローマ字検索をcase-insensitiveに変更するという主要な変更内容を明確に説明しており、変更内容と完全に関連しています。

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/fix-case-insensitive-search-87kp1

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

@TinyKitten TinyKitten merged commit d602572 into dev Mar 6, 2026
11 checks passed
@TinyKitten TinyKitten deleted the claude/fix-case-insensitive-search-87kp1 branch March 6, 2026 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants