Skip to content

Conversation

mafreud
Copy link
Contributor

@mafreud mafreud commented Oct 5, 2025

Summary by CodeRabbit

  • Chores
    • CI/CD のデプロイ認証に使用するシークレットを更新(開発・本番ともに新しいサービスアカウントへ切り替え)。
    • デプロイパイプラインの安定性とセキュリティを向上。ユーザー向けの機能や挙動の変更はありません。

Copy link

linear bot commented Oct 5, 2025

Copy link
Contributor

coderabbitai bot commented Oct 5, 2025

Walkthrough

GitHub Actions の Firebase デプロイ用認証ステップで使用するシークレットを、dev と production の両デプロイにおいて secrets.FIREBASE_SA_KEY から secrets.GHA_SERVICE_ACCOUNT へ置換。

Changes

Cohort / File(s) Change Summary
CI/CD ワークフローの認証シークレット更新
.github/workflows/github-apps-cd.yml
google-github-actions/auth の credentials_json を dev/production ともに secrets.FIREBASE_SA_KEY から secrets.GHA_SERVICE_ACCOUNT に変更。

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as GitHub Actions Runner
  participant Auth as google-github-actions/auth
  participant GCP as Google Cloud
  participant FB as Firebase Deploy

  Note over Dev,FB: CD ワークフロー(dev / production)

  Dev->>Auth: credentials_json = secrets.GHA_SERVICE_ACCOUNT
  Auth->>GCP: OIDC/SA 認証
  GCP-->>Auth: アクセストークン
  Auth-->>Dev: 認証コンテキスト

  Dev->>FB: firebase deploy (対象環境: dev / production)
  FB-->>Dev: デプロイ結果
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

ぴょんと跳ねて鍵を替え、
夜の配信 そっと支え、
GHA のポケット 新しい合鍵、
dev も本番 迷いなし。
月光の下でデプロイ完了、にんじん祝杯🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed このプルリクエストのタイトルは、GitHub Actions ワークフロー内で Firebase サービスアカウントのシークレットを更新するという主要な変更点を簡潔かつ明確に表現しており、コミットメッセージの慣例である「fix:」プレフィックスも適切に使用されています。そのため、変更内容をスキャンするチームメンバーにとって理解しやすいタイトルになっています。
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch op-298

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between cc8d44a and ba80513.

📒 Files selected for processing (1)
  • .github/workflows/github-apps-cd.yml (1 hunks)

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

@mafreud mafreud merged commit 4f7d52e into develop Oct 5, 2025
4 checks passed
@mafreud mafreud deleted the op-298 branch October 5, 2025 10:00
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