Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
Org のチーム管理とプロフィールページの自動更新基盤を導入します。
terraform/: GitHub の team とメンバーシップをコード管理(既存 team はimports.tfで import)scripts/+.github/workflows/: team メンバーを取得し、Org プロフィールのメンバー一覧を自動再生成profile/README.md: メンバー一覧を自動生成ブロック化(Core Members / OB・OG 含む全メンバーの2段構成)レビューと修正
マルチエージェントレビュー(29エージェント)を実施し、指摘された以下の問題を修正済みです。
fetched_atが毎回更新され、メンバー変更がなくても6時間ごとに無意味な同期PRが作られ続けるfetched_atを廃止。メンバー変更時のみ差分が発生gh workflow runに App token を使用しており、repo 権限がなく 403 で失敗するGITHUB_TOKENを使用(workflow_dispatchはGITHUB_TOKEN起動でも run が作られる)data/core_members.yamlを再生成gh api失敗時に stderr が握りつぶされ、原因(403/404/rate limit)が CI ログに出ないre.subの置換文字列にメンバー名を直接埋め込んでおり、\1等を含む入力で誤動作.gitignoreのterraform/*.tfplanが CI の生成物terraform/tfplanにマッチしないまた、live の team と
locals.tfを突き合わせ、active-member-2026に実在するMTowaがlocals.tfに漏れていたのを修正しました(このまま apply すると team から削除されるところでした)。検証済み
uv run scripts/render_members.py --check✅terraform fmt -check/terraform validate✅(lock ファイルは linux_amd64 ハッシュ込みでコミット)actionlint✅Sakiho-Ishiiは現在 org メンバーではありません(招待も未送信)。locals.tfの 2025 に含まれているため、apply 時に org への招待が送信されます。意図していない場合はlocals.tfから削除してください(招待が承認されるまで plan に差分が出続けます)。active-member-YYYY→core-members-YYYY、members2022/2023→core-members-2022/2023。旧 slug を参照している設定(他リポジトリの CODEOWNERS・branch protection 等)があれば壊れます。all-membersteam には現在 Shion1305 のみが所属しており、apply で15名が追加されます(意図どおりのはず)。TF_GH_APP_ID/TF_GH_APP_PRIVATE_KEY/SYNC_GH_APP_ID/SYNC_GH_APP_PRIVATE_KEY)。org レベルで設定済みか確認してください。未設定だと tf-plan がこの PR 上で失敗します。今後の改善候補(このPRでは未実施)
github_team/github_team_members/ import ブロックの年度ごとのコピペをfor_each+ map で一本化(Terraform 1.7+ で import の for_each が利用可能。新年度追加が1行で済むようになる)