From 4a423c66e7f9b2b5c7ac15e49191a310217028a3 Mon Sep 17 00:00:00 2001 From: Malte Viering Date: Tue, 12 May 2026 12:28:59 +0000 Subject: [PATCH] Fix claude-assistant workflow: enforce allowlist and remove review trigger - Add missing 'if' condition on claude job to actually gate on the allowlist check output. Previously the check ran but was never used. - Remove pull_request_review trigger to avoid firing on approvals/ request_changes. Inline review comments and issue comments still work. --- .github/workflows/claude-assistant.yaml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/.github/workflows/claude-assistant.yaml b/.github/workflows/claude-assistant.yaml index 20d939630..496809a04 100644 --- a/.github/workflows/claude-assistant.yaml +++ b/.github/workflows/claude-assistant.yaml @@ -7,8 +7,6 @@ on: types: [opened, assigned] pull_request_review_comment: types: [created] - pull_request_review: - types: [submitted] jobs: check-allowlist: @@ -31,6 +29,7 @@ jobs: claude: needs: check-allowlist + if: needs.check-allowlist.outputs.allowed == 'true' runs-on: ubuntu-latest permissions: contents: write