Skip to content

Comments

Hotfix/merge dev#1394

Merged
TinyKitten merged 8 commits intomasterfrom
hotfix/merge-dev
Feb 11, 2026
Merged

Hotfix/merge dev#1394
TinyKitten merged 8 commits intomasterfrom
hotfix/merge-dev

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 11, 2026

間違えてmasterにsquash mergeしてしまった

TinyKitten and others added 8 commits February 8, 2026 04:21
複数のline_idを一括で処理するバッチRPCを追加。
既存のGetStationByIdList/GetLineByIdListと同じバッチパターンに従う。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
GetStationsByLineGroupIdList RPC追加
* get_stations_by_line_group_id_vecで列車種別を一括取得してセット

N+1問題を回避しつつ、train_type_repository.get_by_line_group_id_vecで
複数line_group_idの列車種別を1クエリで取得し、top-levelとlines[].station
の両方にセットするよう修正。

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

* get_stations_by_line_group_id_vecのユニットテストを追加

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

* ネストされた駅のtrain_typeをis_some()で明示的に検証するよう修正

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

* train_typeをtrain_type_mapから常に上書きしてリーク防止

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

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* GetLineByIdList->GetLinesByIdList

* cargo fmt
@TinyKitten TinyKitten self-assigned this Feb 11, 2026
@TinyKitten TinyKitten marked this pull request as ready for review February 11, 2026 08:57
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 11, 2026

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.

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
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hotfix/merge-dev

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

@TinyKitten TinyKitten merged commit 5b51755 into master Feb 11, 2026
10 checks passed
@TinyKitten TinyKitten deleted the hotfix/merge-dev branch February 11, 2026 09:02
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.

1 participant