[panicboat]

Summary

共通の workflow の中からだけ呼ぶ 3 つの workflow を、接頭辞 internal-- で区別し、中身に合う名前に変えます。

Why

reusable--* には、他のリポジトリから呼ぶものと、resolver などの中から $/ でだけ呼ぶものが混ざっていて、ファイル名から区別できません。
reusable workflow は .github/workflows の直下にしか置けず、サブディレクトリはサポートされていないので、.github/internal には移せません。
そのため、置き場所ではなくファイル名の接頭辞で区別します。
あわせて、中身と合っていなかった名前を、やっていることに合わせて変えます。

Before / After

• 共通の workflow の中からだけ呼ぶ workflow の名前
  • Before: reusable--container-builder・reusable--kubernetes-builder・reusable--kubernetes-hydrator
  • After: internal--image-builder・internal--kustomize-builder・internal--helmfile-hydrator
    • kubernetes-builder の本体は head と base の kustomize build なので、kustomize-builder にします。
      • diff を PR にコメントするのは、build の結果を比べる後段です。
    • kubernetes-hydrator は helmfile で manifest を描画するだけなので、helmfile-hydrator にします。
• 他のリポジトリから呼ぶ reusable--* は変わりません。

Decisions

• reusable--terragrunt-runner は internal-- にしません。
  • monorepo の deploy-dystopia-lambda が直接呼んでいるためです。
• $/ で指している workflow が実在することを確かめるテストを足しました。
  • 名前を変えたときの付け替え漏れは、呼び出し側の PR で初めて落ちるので、ここで先に検出します。
• refactor なので release は出ません。
  • 呼び出し側から見た動作は変わらないので、次の feat・fix の release で一緒に届けば十分です。
• concurrency の group 名、scripts/kubernetes-hydrate/、kustomize の job の名前 Kustomize Diff は変えていません。
  • ファイル名とは別の識別子で、この PR の範囲を名前の区別に留めるためです。
  • job の名前は PR のコメントの見出しと check の名前に出るもので、そこで伝えたいのは diff です。

Impact

• 利用規約
• プライバシーポリシー
• ドキュメント
  • README に internal--* の説明を足しました。

Verification

• VERIFIED: bash scripts/workflows/test.sh で 6 件すべて PASS しました。
• VERIFIED: 参照先を存在しない名前に書き換えると、test_nested_calls_resolve_to_existing_workflows が missing: $/.github/workflows/internal--image-builderX.yaml で FAIL することを確かめました。
• VERIFIED: CI と同じ scripts/*/test.sh・scripts/*/*.test.sh・scripts/kubernetes-hydrate/tests/*.test.sh がすべて通りました。
• VERIFIED: monorepo・platform・deploy-actions の workflow を grep して、3 つの workflow を直接呼んでいる箇所がないことを確かめました。
• REASONED: $/ は呼び出し元と同じ commit に解決されるので、古い SHA に固定したリポジトリは古い名前を、新しい SHA に固定したリポジトリは新しい名前を参照し、どちらも壊れません。
• REASONED: check の名前は job の名前から作られ、ファイル名は使わないので、必須チェックは変わりません。

Relations

• feat: add the shared workflows, scripts, and renovate preset #3: 共通の workflow

🤖 Generated with Claude Code

https://claude.ai/code/session_01R7jQZQj1VyeBbYR7P1ssV7

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 827f0bfc-5dd2-47b9-9416-2d2cc3db5f6c

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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