Skip to content

proto の変更をクライアントリポジトリへ通知するワークフローを追加 - #68

Merged
kantacky merged 7 commits into
feature/admin-protofrom
add-release-proto-workflow
Oct 11, 2026
Merged

kantacky merged 7 commits into
feature/admin-protofrom
add-release-proto-workflow

Conversation

@kantacky

@kantacky kantacky commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

概要

proto 定義の変更をクライアントリポジトリ(app・web・admin-web)へ通知する release-proto ワークフローを追加します。

変更点

  • .github/workflows/release-proto.yml を追加
    • 発火条件: main で proto/**・buf.yaml・buf.gen.yaml が変わったとき、または手動実行(workflow_dispatch)
    • matrix で app・web・admin-web それぞれに POST /repos/{owner}/{repo}/dispatches を送り、event_type: update-proto でクライアント側の update-proto ワークフローを発火する
    • client_payload.sha に元のコミットの SHA を渡す
    • トークンは actions/create-github-app-token で、対象リポジトリごとに contents: write に絞って発行する

事前準備

  • このリポジトリに vars.APP_ID と secrets.APP_PRIVATE_KEY を登録する
  • GitHub App を app・web・admin-web にインストールし、Contents の書き込み権限を付与する

確認内容

  • マージ後、release-proto を手動実行して 3 リポジトリの update-proto が動くことを確認する

🤖 Generated with Claude Code

https://claude.ai/code/session_016M34xEcz4rFzq43PMoZZG5

kantacky and others added 6 commits October 8, 2026 18:22
タスク名や説明文の表記が揃っておらず、admin-bff-api / app-bff-api は
portless 経由で起動できないなど、運用が分かりにくかったため整理した。

- setup を bootstrap に改名し、rulesync を先に実行する
- generate:openapi を generate に改名し、buf generate も実行する
- API 起動タスクを run:<name> に統一し、全 API を portless 経由にする
- lint タスクに go vet と gormuuidlint をまとめる
- 不要になった clean タスクや migrate 系バイナリのビルドを削除する
- 説明文を日本語に統一し、README のタスク一覧を更新する

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mise タスクの整理で setup が bootstrap に、lint:gormuuid が lint に
統合されたため、CI が存在しないタスクを呼び出して失敗する状態だった。

- mise setup を mise bootstrap に置き換える
- go vet と gormuuid のステップを mise run lint に統合する

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
タグ指定のままだと参照先が書き換えられるリスクがあるため、
全ワークフローの action をコミット SHA で固定した。

- actions/checkout を v7.0.1 に更新する
- jdx/mise-action を v5.1.1 に更新する
- google-github-actions 系はすでに最新のため据え置く

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
タスクの実行方法を他の手順と揃え、mise run 経由で呼び出す表記にした。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
他のタスクと同様に mise tasks の一覧で用途が分かるよう、
rulesync タスクにも日本語の説明文を付けた。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
main で proto 定義が変わったときに、app・web・admin-web の
update-proto ワークフローを repository_dispatch で発火する。
トークンは GitHub App から対象リポジトリごとに contents:write に
絞って発行する。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016M34xEcz4rFzq43PMoZZG5
@kantacky kantacky self-assigned this Oct 8, 2026
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Deploying fun-dotto-server-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5303d47
Status: ✅  Deploy successful!
Preview URL: https://6fed5d0c.fun-dotto-server-docs.pages.dev
Branch Preview URL: https://add-release-proto-workflow.fun-dotto-server-docs.pages.dev

View logs

@kantacky
kantacky changed the base branch from main to mise-tasks October 8, 2026 14:21
@kantacky
kantacky added this pull request to stack #66 October 8, 2026 14:21
@kantacky
kantacky marked this pull request as ready for review October 8, 2026 14:21
@kantacky kantacky changed the title Admin API の Protobuf 定義と proto 更新を通知するワークフローを追加 proto の変更をクライアントリポジトリへ通知するワークフローを追加 Oct 8, 2026
proto の変更をクライアントへ反映する際の手順を揃えるため、
release-proto の手動実行、完了待ち、発火先の update-proto の
確認と結果報告までを rulesync のスキルとして定義した。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016M34xEcz4rFzq43PMoZZG5
@kantacky
kantacky added this pull request to the merge queue Oct 11, 2026
Merged via the queue into main with commit d6ca69e Oct 11, 2026
4 checks passed
Base automatically changed from mise-tasks to feature/admin-proto October 11, 2026 05:03
@kantacky
kantacky deleted the add-release-proto-workflow branch October 11, 2026 05:03
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.

2 participants