Skip to content

Run security reviews automatically for Buzz team members - #6952

Open
jmecom wants to merge 1 commit into
mainfrom
jm/codex-security-review-smoke
Open

Run security reviews automatically for Buzz team members#6952
jmecom wants to merge 1 commit into
mainfrom
jm/codex-security-review-smoke

Conversation

@jmecom

@jmecom jmecom commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Changes security-review authorization to use the pull request author's live calculated block/buzz repository permission. Authors with write or admin access now run automatically. Read-only and outside contributors still require exact SHA-bound manual authorization from a Block organization member.

This avoids relying on author_association, which the Actions token reported incorrectly for a member-authored pull request. The repository permission check includes direct, team, organization, and enterprise grants and fails closed when the author is not a collaborator.

Checked with just ci, just security-review-check, and actionlint .github/workflows/codex-security-review.yml.

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

🔐 Codex Security Review

Status: review required for the current range.

The current range is 0808ab485c39c3c12ef02af2188c65a5145eb99d...b0303286cbc2b3c992d28bcedb8857946e6bc2b1.
A new review must complete for this exact range. When manual authorization
is required, a Block organization member must comment exactly
@buzz-security-review b0303286cbc2b3c992d28bcedb8857946e6bc2b1 to authorize a new review.
Any previous review applies only to its recorded range.

@jmecom
jmecom marked this pull request as ready for review August 27, 2026 18:27
@jmecom
jmecom requested a review from a team as a code owner August 27, 2026 18:27
@jmecom

jmecom commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator Author

@buzz-security-review b273b8f

@github-actions github-actions Bot added the codex-security-review-current The posted Codex security review matches its recorded range. label Aug 27, 2026
Signed-off-by: Jordan Mecom <jm@squareup.com>
@jmecom
jmecom force-pushed the jm/codex-security-review-smoke branch from b273b8f to b030328 Compare August 27, 2026 19:22
@jmecom jmecom changed the title Test security review trigger Run security reviews automatically for Buzz team members Aug 27, 2026
@github-actions github-actions Bot removed the codex-security-review-current The posted Codex security review matches its recorded range. label Aug 27, 2026
@jmecom

jmecom commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator Author

@buzz-security-review b030328

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.

1 participant