Skip to content

Comments

GetLineByIdList->GetLinesByIdList#1392

Merged
TinyKitten merged 2 commits intodevfrom
fix/batch-lines-rpc
Feb 11, 2026
Merged

GetLineByIdList->GetLinesByIdList#1392
TinyKitten merged 2 commits intodevfrom
fix/batch-lines-rpc

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 11, 2026

現状使っているクライアントはないはずなので影響範囲はない

Summary by CodeRabbit

ドキュメント

  • Lines サービスの API ドキュメントを更新しました。複数 ID 検索メソッドの命名を統一し、メソッド名を GetLineByIdList から GetLinesByIdList に修正しました。

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

ウォークスルー

GetLineByIdList API メソッドを GetLinesByIdList に統一するための変更。ドキュメント、プロトバッファのサブモジュール参照、gRPC コントローラーの実装で対応するメソッド名とリクエスト型を複数形に更新。

変更内容

コホート / ファイル サマリー
ドキュメント更新
AGENTS.md, docs/architecture.md
gRPC エンドポイント API 仕様のドキュメントを更新。GetLineByIdListGetLinesByIdList に統一し、API 表記を複数形に修正。
プロトバッファサブモジュール
stationapi/proto
サブモジュールのコミットハッシュを 7eede39 から d97e808 に更新。
gRPC コントローラー実装
stationapi/src/presentation/controller/grpc.rs
RPC メソッド名 get_line_by_id_listget_lines_by_id_list に、リクエスト型 GetLineByIdListRequestGetLinesByIdListRequest に変更。プロトメッセージのインポートを更新。

推定コードレビュー工数

🎯 2 (Simple) | ⏱️ ~12 分

関連する可能性のある PR

  • reflection対応 #1202: stationapi/src/presentation/controller/grpc.rs のプロトインポートパスの更新と関連する gRPC サーフェース修正。
  • GetRoutesMinimal RPCを実装 #1334: 同じファイルの get_line_by_id_list メソッドを使用・追加する変更と直接競合・関連。

ポエム

🐰 GetLine をしりますか
IdList が IdList で
いや、ByIdList じゃなくて
ByIdLists ですか、了解!
複数形の調和で API は踊る 🎭✨

🚥 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 プルリクエストのタイトルは、メソッド名の変更 GetLineByIdList→GetLinesByIdList という主要な変更を正確に反映しており、複数ファイルの更新内容を簡潔かつ明確に要約しています。

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/batch-lines-rpc

No actionable comments were generated in the recent review. 🎉


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

@TinyKitten TinyKitten self-assigned this Feb 11, 2026
@TinyKitten TinyKitten changed the title GetLineByIdList->GetLinesByIdListnp GetLineByIdList->GetLinesByIdList Feb 11, 2026
@TinyKitten TinyKitten merged commit c82cb7c into dev Feb 11, 2026
10 checks passed
@TinyKitten TinyKitten deleted the fix/batch-lines-rpc branch February 11, 2026 08:53
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