CDワークフローを構築

[masaya-osuga]

No description provided.

[devin-ai-integration]

レビュー

全体的に CD ワークフローの初期構築として良い方向です。いくつか気になる点を挙げます。

---

1. prd vs prod の不一致（Bug）

workflow_dispatch の選択肢に prd がありますが、suffix 判定は "$ENVIRONMENT" = "prod" をチェックしています。prd を選択した場合、suffix が空にならず -prd になり、Cloud Run リソース名が意図しないものになります。

options:
  - dev
  - stg
  - prd   # ← ここ

if [ "$ENVIRONMENT" = "prod" ]; then   # ← "prd" とマッチしない

prd → prod に統一するか、条件を "prd" に合わせる必要があります。

2. Actions のタグ参照 → SHA 固定推奨

actions/checkout@v6, google-github-actions/auth@v2 等がタグ参照です。サプライチェーン攻撃防止のため、SHA 固定（+ タグをコメント併記）が推奨されます。PR #21 ではこの対応がされているので、こちらも合わせるとよさそうです。

3. secrets vs vars の使い分け

GCP_PROJECT_ID が secrets で参照されていますが、project ID は機密情報ではないので vars の方が適切では？ PR #21 では vars.PROJECT_ID に切り替わっています。

4. concurrency group の null 問題

group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.environment_name }}

push イベント時は github.event.inputs.environment_name が空文字列になり、group 名末尾が - で終わります。機能上は問題ありませんが、意図的かどうか確認したいです。

5. CI 変更について

• concurrency group の ci- prefix 削除、mise バージョンピン解除は問題ないと思います。
• mise のバージョンピンを外した場合、CIの再現性が下がるリスクがありますが、404 問題の回避としては妥当です。

6. Dockerfile の変更

announcement-api と user-api のビルド追加は OK です。atlas-loader がビルド対象に含まれていない点は意図的と理解しています（ローカルツールのため？）。