Merge pull request #332 from campfirein/feat/ENG-1933

feat: [ENG-1933] include OWNER in author_association check and enhanc…

Author: leehpham

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

@@ -16,8 +16,22 @@ permissions:
   pull-requests: write
 
 jobs:
+  debug:
+    runs-on: ubuntu-latest
+    steps:
+      - run: |
+          echo "draft: ${{ github.event.pull_request.draft }}"
+          echo "author_association: ${{ github.event.pull_request.author_association }}"
+
   auto-review:
-    if: ${{ !github.event.pull_request.draft && github.event.pull_request.author_association == 'MEMBER' }}
+    # CONTRIBUTOR: GitHub reports CONTRIBUTOR instead of MEMBER for org members
+    # whose repo access comes from org base permissions rather than team assignment.
+    # TODO: Remove CONTRIBUTOR once all org members are assigned to repo teams.
+    if: >-
+      ${{
+        !github.event.pull_request.draft &&
+        contains(fromJSON('["MEMBER", "OWNER", "CONTRIBUTOR"]'), github.event.pull_request.author_association)
+      }}
     runs-on: ubuntu-latest
     timeout-minutes: 10
     steps:
@@ -35,5 +49,8 @@ jobs:
           prompt: |
             REPO: ${{ github.repository }}
             PR NUMBER: ${{ github.event.pull_request.number }}
-            
-            Perform code review. Provide detailed feedback using inline comments for specific issues. Apply Conventional Comments.
+
+            Review this PR for correctness, security, and code quality. Use Conventional Comments.
+
+            Always provide inline comments for specific issues.
+            Post a general summary comment with an overall score (0-5, where 0 is poor and 5 is excellent).