Refactor CI

[mafreud]

Summary by CodeRabbit

• Chores

  • CIワークフロー名を統一し、GitHub Appsのテスト/デプロイ設定を整理。
  • 重複ワークフローを削除して再利用可能なワークフローへ集約。
  • Firebase Functionsのテストを再利用ワークフロー化し、実行手順を簡素化。
  • デプロイジョブから不要な依存インストールを削除し、処理時間を短縮。

• Tests

  • テスト実行を再利用ワークフローへ移行し、安定性と保守性を向上。

[linear]

OP-287 refactor ci

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

GitHub Actions ワークフローを再編：.github/workflows/github-apps-tests.ymlを削除し、Firebase Functions テストを新規の再利用ワークフローへ移動。github-apps-cd.ymlはテスト呼び出し先を差し替え、デプロイ前の個別依存インストールを削除・簡素化した。

Changes

Cohort / File(s)	Summary of Changes
GitHub Apps CD 更新 .github/workflows/github-apps-cd.yml	ワークフロー名を「Deploy GitHub Apps」に変更。GitHub Apps テスト呼び出しを reusable-github-apps-ci.yml へ差し替え。firebase-functions のテストジョブを再利用ワークフロー呼び出しに置換。デプロイジョブから個別の依存インストール手順を削除。
再利用ワークフロー：GitHub Apps .github/workflows/reusable-github-apps-ci.yml	ワークフロー名を「GitHub Apps Tests」へ変更（呼び出し対象の名称整合）。内容自体に大きな設定変更はなし。
再利用ワークフロー：Firebase Functions 追加 .github/workflows/reusable-firebase-functions-ci.yml	新規追加。workflow_call トリガーで test ジョブを定義（working-directory を openci-runner/firebase/functions、checkout、Node 設定（.nvmrc）、npm ci、npm test を実行）。
削除されたワークフロー .github/workflows/github-apps-tests.yml	既存の github-apps-tests.yml を削除（内部で reusable-github-apps-tests.yml を呼び出していた構成を廃止）。

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as 開発者
  participant GA as GitHub Actions
  participant ReuseApps as 再利用: GitHub Apps CI
  participant ReuseFF as 再利用: Firebase Functions CI
  participant Deploy as Deploy ジョブ

  Dev->>GA: push / PR
  GA->>ReuseApps: workflow_call -> GitHub Apps Tests
  ReuseApps-->>GA: テスト結果

  GA->>ReuseFF: workflow_call -> Firebase Functions Tests
  ReuseFF-->>GA: テスト結果

  GA->>Deploy: Deploy Functions (依存インストールなし)
  Deploy-->>GA: デプロイ結果

  Note over ReuseFF,Deploy: Firebase の依存インストールとテストは再利用ワークフロー内で実行

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feat: add testing capabilities for Firebase functions #454: Firebase Functions テスト実行（npm test）や .nvmrc を使う構成変更と重複する領域。
• feat: add Firebase Functions CI/CD workflow with testing and deployment steps #455: GitHub Apps / Firebase Functions のワークフロー呼び出し・再利用化に関する変更と連動。
• refactor: streamline GitHub Actions workflows and remove unnecessary paths #460: 再利用ワークフローへの集約やワークフロー名変更など、同系統の CI リファクタを含む PR。

Poem

ぴょんと跳ねてワークフロー揃えた、
再利用で道はすっきり。
テストは箱へ、デプロイは一直線、
にんじんライトでログが踊る。
うさぎの手で、CI は今日も走る。 🥕🐇

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	「Refactor CI」というタイトルは、GitHub Actions ワークフローの構成を再利用可能なワークフローに切り替えて整理するという主な変更を簡潔に示しており、内容と一致しています。
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

---

📜 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 72d8b86 and 40a0eca.

📒 Files selected for processing (1)

• .github/workflows/github-apps-cd.yml (2 hunks)

---

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