Skip to content

ワークフローの top-level permissions を明示して default write 権限をやめられるようにする - #34

Merged
krrrr38 merged 1 commit into
mainfrom
explicit-workflow-permissions
Aug 30, 2026
Merged

ワークフローの top-level permissions を明示して default write 権限をやめられるようにする#34
krrrr38 merged 1 commit into
mainfrom
explicit-workflow-permissions

Conversation

@krrrr38

@krrrr38 krrrr38 commented Aug 30, 2026

Copy link
Copy Markdown
Member

Summary

リポジトリ設定の default_workflow_permissions: write をやめられるようにするため、GITHUB_TOKEN の権限をワークフロー側で完結させました。

  • action-lint.yml / labeler.yml に top-level の permissions: contents: read を追加(他のワークフローには既に定義済み)
  • 全 job には既に permissions が明示されていたため、job 側の変更はなし

現状の権限棚卸し

ワークフロー / job permissions 理由
action-lint / actionlint checks: write, contents: read, pull-requests: write reviewdog が PR レビューコメントを投稿
action-lint / ghalint contents: read checkout と public repo からの gh release download
ci / changes contents: read, pull-requests: read dorny/paths-filter が PR の差分を取得
ci / 各言語, all-checks contents: read checkout のみ
test-*.yml / build contents: read checkout のみ
dependabot-auto-merge / auto-merge contents: write, pull-requests: write gh pr merge --auto
labeler / triage contents: read, pull-requests: write ラベル付与
release / package contents: write gh release create / delete --cleanup-tag

write 権限が必要なのは action-lint(reviewdog) / dependabot-auto-merge / labeler / release の 4 job だけで、いずれも明示済みです。

マージ後にやること

Settings → Actions → General → Workflow permissions を Read repository contents and packages permissions に変更する。

gh api -X PUT repos/folio-sec/wrap-api-example/actions/permissions/workflow \
  -f default_workflow_permissions=read

同画面の "Allow GitHub Actions to create and approve pull requests" は別設定のため影響なし。

Test plan

  • ローカルで ghalint run が exit 0
  • この PR で action-lint (actionlint / ghalint) が通ること
  • labeler がラベルを付与できること
  • 設定変更後に release ワークフローを手動実行し、release の作成・削除ができること
  • 設定変更後に dependabot PR で auto merge が有効化されること

🤖 Generated with Claude Code

https://claude.ai/code/session_01SppdKxgYtCGigXC8dZyxob

@github-actions github-actions Bot added the ci label Aug 30, 2026
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SppdKxgYtCGigXC8dZyxob
Signed-off-by: krrrr38 <k.kaizu38@gmail.com>
@krrrr38
krrrr38 force-pushed the explicit-workflow-permissions branch from a84cf3f to 254f766 Compare August 30, 2026 23:32
@krrrr38
krrrr38 merged commit 16fe130 into main Aug 30, 2026
15 checks passed
@krrrr38
krrrr38 deleted the explicit-workflow-permissions branch August 30, 2026 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant