Merge pull request #328 from campfirein/feat/ENG-1932

feat: [ENG-1933] update Claude Code PR review workflow for GitHub Flo…

Author: leehpham

.github/workflows/claude-code-pr-review.yml

@@ -4,7 +4,7 @@ on:
   pull_request:
     types: [opened, synchronize, ready_for_review, reopened]
     branches:
-      - develop
+      - main
       - 'proj/**'
 
 concurrency:
@@ -17,41 +17,14 @@ permissions:
 
 jobs:
   auto-review:
-    if: ${{ !github.event.pull_request.draft }}
+    if: ${{ !github.event.pull_request.draft && github.event.pull_request.author_association == 'MEMBER' }}
     runs-on: ubuntu-latest
     timeout-minutes: 10
     steps:
       - uses: actions/checkout@v6
         with:
           fetch-depth: 1
-
-      - name: Detect main branch sync
-        id: detect-sync
-        run: |
-          if [ "${{ github.event.action }}" != "synchronize" ]; then
-            echo "is_main_sync=false" >> "$GITHUB_OUTPUT"
-            exit 0
-          fi
-
-          git fetch origin main --depth=1
-          MAIN_SHA=$(git rev-parse origin/main)
-          PARENTS=$(git log -1 --format='%P')
-          PARENT1=$(echo "$PARENTS" | awk '{print $1}')
-          PARENT2=$(echo "$PARENTS" | awk '{print $2}')
-
-          BEFORE="${{ github.event.before }}"
-
-          if [ -n "$PARENT2" ] && \
-             { { [ "$PARENT1" = "$BEFORE" ] && [ "$PARENT2" = "$MAIN_SHA" ]; } || \
-               { [ "$PARENT2" = "$BEFORE" ] && [ "$PARENT1" = "$MAIN_SHA" ]; }; }; then
-            echo "is_main_sync=true" >> "$GITHUB_OUTPUT"
-            echo "Skipping review: sync from main detected"
-          else
-            echo "is_main_sync=false" >> "$GITHUB_OUTPUT"
-          fi
-
       - uses: anthropics/claude-code-action@v1
-        if: steps.detect-sync.outputs.is_main_sync != 'true'
         with:
           anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
           github_token: ${{ secrets.GITHUB_TOKEN }}