Try to refine CI triggers for AI tools (#3659)

* Try to refine CI triggers for AI tools

* seems anthropic api key not working

Author: Kailai-Wang

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

@@ -2,14 +2,18 @@ name: Claude Code Review
 
 on:
   pull_request:
-    types: [opened, synchronize]
+    types: [opened, reopened, ready_for_review]
     # Optional: Only run on specific file changes
     # paths:
     #   - "src/**/*.ts"
     #   - "src/**/*.tsx"
     #   - "src/**/*.js"
     #   - "src/**/*.jsx"
 
+concurrency:
+  group: '${{ github.workflow }}-${{ github.head_ref || github.ref }}'
+  cancel-in-progress: true
+
 jobs:
   claude-review:
     # Optional: Filter by PR author
@@ -29,9 +33,13 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v4
         with:
-          fetch-depth: 1
+          submodules: recursive
+          fetch-depth: 0
 
       - name: Run Claude Code Review
+        if: |
+          !contains(github.event.pull_request.title, '[skip-review]') &&
+          !contains(github.event.pull_request.title, '[WIP]')
         id: claude-review
         uses: anthropics/claude-code-action@beta
         with:
@@ -50,6 +58,7 @@ jobs:
             - Test coverage
             
             Be constructive and helpful in your feedback.
+            Write inline comments on specific lines where necessary.
 
           # Optional: Use sticky comments to make Claude reuse the same comment on subsequent pushes to the same PR
           # use_sticky_comment: true
@@ -70,9 +79,4 @@ jobs:
 
           # Optional: Add specific tools for running tests or linting
           # allowed_tools: "Bash(npm run test),Bash(npm run lint),Bash(npm run typecheck)"
-          
-          # Optional: Skip review for certain conditions
-          # if: |
-          #   !contains(github.event.pull_request.title, '[skip-review]') &&
-          #   !contains(github.event.pull_request.title, '[WIP]')
 

.github/workflows/claude.yml

@@ -28,7 +28,8 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v4
         with:
-          fetch-depth: 1
+          submodules: recursive
+          fetch-depth: 0
 
       - name: Run Claude Code
         id: claude
@@ -53,10 +54,15 @@ jobs:
           # allowed_tools: "Bash(npm install),Bash(npm run build),Bash(npm run test:*),Bash(npm run lint:*)"
 
           # Optional: Add custom instructions for Claude to customize its behavior for your project
-          # custom_instructions: |
-          #   Follow our coding standards
-          #   Ensure all new code has tests
-          #   Use TypeScript for new files
+          custom_instructions: |
+            If you are asked to review code, focus on:
+            - Code quality and best practices
+            - Potential bugs or issues
+            - Performance considerations
+            - Security concerns
+            - Test coverage
+            Be constructive and helpful in your feedback.
+            Write inline comments on specific lines where necessary.
           
           # Optional: Custom environment variables for Claude
           # claude_env: |

.github/workflows/gemini-cli.yml

@@ -25,6 +25,7 @@ permissions:
   id-token: 'write'
   pull-requests: 'write'
   issues: 'write'
+  actions: 'read'
 
 jobs:
   gemini-cli:

.github/workflows/gemini-pr-review.yml

@@ -4,12 +4,8 @@ on:
   pull_request:
     types:
       - 'opened'
-  pull_request_review_comment:
-    types:
-      - 'created'
-  pull_request_review:
-    types:
-      - 'submitted'
+      - 'reopened'
+      - 'ready_for_review'
   workflow_dispatch:
     inputs:
       pr_number:
@@ -31,6 +27,7 @@ permissions:
   issues: 'write'
   pull-requests: 'write'
   statuses: 'write'
+  actions: 'read'
 
 jobs:
   review-pr: