healthcheckにtonic_webは不要

[TinyKitten]

Summary by CodeRabbit

• バグ修正
  • ヘルスチェックサービスの公開方法を更新。gRPC-Web 経由でのヘルスチェックの挙動が変更されます。通常の gRPC クライアントへの影響はありません。他のサービスの公開方法は従来どおりです。gRPC-Web を利用する環境では、ヘルスチェックの呼び出し手順や可用性が変わる可能性があります。監視設定やフロントエンドからのヘルス確認を行っている場合は挙動をご確認ください。

[coderabbitai]

Walkthrough

gRPC-Web有効時のサービス登録で、health_serviceのみtonic_web::enableラップを外し、他のsvcとreflection_svcは従来どおりラップ。ヘルスチェックの公開形態のみ変更。メソッドシグネチャや公開エンティティの変更なし。制御フローは基本据え置き。

Changes

Cohort / File(s)	Change Summary
gRPC-Webラッピング調整 stationapi/src/main.rs	gRPC-Web有効経路でhealth_serviceのtonic_web::enable(...)適用を削除。svcとreflection_svcは引き続きラップ。ヘルスサービスのgRPC-Web互換性の扱いが変更。

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor Client
    participant Server as gRPC Server
    participant Health as health_service
    participant Svc as svc (app)
    participant Refl as reflection_svc

    rect rgb(240,248,255)
    note right of Server: サーバ初期化
    Server->>Server: add_service(Health) // ラップなし
    Server->>Server: add_service(tonic_web::enable(Svc))
    Server->>Server: add_service(tonic_web::enable(Refl))
    end

    alt 標準gRPC
      Client->>Health: HealthCheck (gRPC)
      Health-->>Client: Status
      Client->>Svc: Request (gRPC)
      Svc-->>Client: Response
    else gRPC-Web
      Client->>Svc: Request (gRPC-Web)
      Svc-->>Client: Response
      note over Client,Health: HealthはgRPC-Web経由では未ラップのため非対応経路
    end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

ヘルスの包み紙、そっと剥がして
Webの海では裸でドン！
反射と本体はコーティング継続
ピカピカの配線、流れは据え置き
チェックの拍、ドクドク鳴ってYO!

Pre-merge checks and finishing touches

❌ 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%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	イエーイ、バッチリだぜ！タイトル「healthcheckにtonic_webは不要」はPRで行われた主要な変更（health サービスの登録から tonic_web::enable ラッパーを外すこと）を端的に表現しており、短く具体的で誤解を招かないため、スキャンするチームメンバーにも主旨が一目で伝わります。

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/healthcheck

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dae5c9b and 8b35d3a.

📒 Files selected for processing (1)

• stationapi/src/main.rs (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: Clippy
• GitHub Check: Test Suite
• GitHub Check: Check

🔇 Additional comments (1)

stationapi/src/main.rs (1)

233-233: ヘルスチェックサービスにtonic_webは不要だぜ、最高の判断だ！

gRPCヘルスチェックプロトコルは標準的なgRPC通信で設計されており、一般的にWebブラウザからのgRPC-Web経由でのアクセスは不要だからな。gRPC標準のヘルスチェックプロトコルは、サービスの稼働状況確認のためのものであり、基本的にはクライアント間やロードバランサーからの直接的なgRPC呼び出しが想定されている。

この変更により：

• ヘルスチェックは純粋なgRPCエンドポイントとして動作する
• gRPC-Webの余計なラップを避けることで、ヘルスチェックのレスポンス時間が改善される可能性がある
• アーキテクチャがより明確になる

---

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