Skip to content

fix: use permissions: write in bonk instead of CODEOWNERS #30926

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mvvmm merged 5 commits into from
May 19, 2026
+2 −2
Merged

fix: use permissions: write in bonk instead of CODEOWNERS #30926

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
1e984a8
fix: remove redundant CODEOWNERS check from bonk-auto-review
mvvmm May 19, 2026
7578db7
fix: use write permission check in bonk-auto-review as defense-in-depth
mvvmm May 19, 2026
4abd4c6
fix: drop permissions input from bonk-auto-review
mvvmm May 19, 2026
74bd169
fix: use permissions: write in bonk-auto-review
mvvmm May 19, 2026
a640177
fix: replace CODEOWNERS permission with write in bonk step
mvvmm May 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/bonk.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ jobs:
model: "cloudflare-ai-gateway/workers-ai/@cf/moonshotai/kimi-k2.6"
agent: docs
mentions: "/bonk"
permissions: CODEOWNERS
permissions: write

bonk-auto-review:
needs: check-codeowner
Expand Down Expand Up @@ -141,7 +141,7 @@ jobs:
with:
model: "cloudflare-ai-gateway/workers-ai/@cf/moonshotai/kimi-k2.6"
agent: docs
permissions: CODEOWNERS
permissions: write
token_permissions: NO_PUSH
prompt: |
Review this pull request. Review the title, description, and diff.
Expand Down