Skip to content

Drop redundant roles key from sweep frontmatter#168

Merged
theletterf merged 1 commit into
mainfrom
fix-sweep-roles-warning
May 22, 2026
Merged

Drop redundant roles key from sweep frontmatter#168
theletterf merged 1 commit into
mainfrom
fix-sweep-roles-warning

Conversation

@theletterf
Copy link
Copy Markdown
Member

Summary

  • Removes the explicit roles: [admin, maintainer, write] from the on: frontmatter of all 7 sweep workflows
  • This key is the gh-aw default, so it has no effect on policy
  • Its presence caused a spurious warning in the pre_activation job when github-actions[bot] triggered a sweep: the bot passes via bots:, but gh-aw still evaluated the roles: check and warned about permission none

Follows #167 which added bots: ["github-actions[bot]"].

Test plan

  • Pre-commit passes (compile hook produces no diff)
  • No pre_activation warning on bot-triggered sweep runs

🤖 Generated with Claude Code

roles: [admin, maintainer, write] is the gh-aw default; leaving it
explicit caused a warning in pre_activation when github-actions[bot]
triggered the workflow (the bot passes via bots:, but the roles check
still ran and warned about permission 'none'). Removing the key
silences the warning without changing the effective policy.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@theletterf theletterf requested a review from a team as a code owner May 22, 2026 07:45
@theletterf theletterf requested a review from Mpdreamz May 22, 2026 07:45
@theletterf theletterf self-assigned this May 22, 2026
@theletterf theletterf added the fix label May 22, 2026
@theletterf theletterf merged commit 281c0cd into main May 22, 2026
4 checks passed
@theletterf theletterf deleted the fix-sweep-roles-warning branch May 22, 2026 07:47
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