feat(ci): migrate from Claude to GitHub Copilot code review#588
feat(ci): migrate from Claude to GitHub Copilot code review#588JasonXuDeveloper merged 5 commits intomasterfrom
Conversation
- Remove claude-code-review.yml and claude.yml workflows - Update auto-approve.yml to check Copilot inline comments instead of Claude - Copilot code review is now configured via repository ruleset This change reduces CI costs and avoids potential account suspension from running Claude ~100 times/day across different GitHub runners. Co-Authored-By: Claude Opus 4.5 <[email protected]> Signed-off-by: JasonXuDeveloper - 傑 <[email protected]>
There was a problem hiding this comment.
Pull request overview
This PR migrates from Claude-based code review (via CI workflows) to GitHub Copilot code review (via repository rulesets). The change is motivated by avoiding potential account suspension from running Claude reviews ~100 times/day on different GitHub runners.
Changes:
- Removed two Claude workflow files that triggered reviews via CI
- Updated auto-approve workflow to check for Copilot inline comments instead of Claude bot comments
- Changed workflow trigger to no longer wait for "Claude Code Review" workflow completion
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.github/workflows/claude.yml |
Deleted - removed Claude interactive bot workflow |
.github/workflows/claude-code-review.yml |
Deleted - removed automated Claude PR review workflow |
.github/workflows/auto-approve.yml |
Updated logic to detect Copilot review comments and removed Claude workflow from triggers |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Unity Test Results✅ EditMode: All tests passed Unity Version: 2022.3.55f1 ✅ All tests passed! The PR is ready for review. View workflow run |
…hanges - Check Copilot review existence before checking comments (fixes race condition) - Only trigger Unity tests for unity-tests.yml and pr-tests.yml changes - Other CI workflow changes (auto-approve, etc.) no longer run Unity tests Co-Authored-By: Claude Opus 4.5 <[email protected]> Signed-off-by: JasonXuDeveloper - 傑 <[email protected]>
- Reviews: copilot-pull-request-reviewer[bot] - Comments: Copilot Co-Authored-By: Claude Opus 4.5 <[email protected]> Signed-off-by: JasonXuDeveloper - 傑 <[email protected]>
Resolved threads should not block auto-approve. Use GraphQL to count only unresolved review threads from Copilot. Co-Authored-By: Claude Opus 4.5 <[email protected]> Signed-off-by: JasonXuDeveloper - 傑 <[email protected]>
Co-Authored-By: Claude Opus 4.5 <[email protected]> Signed-off-by: JasonXuDeveloper - 傑 <[email protected]>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
claude-code-review.ymlandclaude.ymlworkflowsauto-approve.ymlto check Copilot reviews instead of ClaudeWhy
Running Claude code review via CI (~100 times/day on different GitHub runners) may trigger account suspension due to suspicious activity patterns. GitHub Copilot code review via rulesets is the native, integrated solution.
Changes
.github/workflows/claude-code-review.yml.github/workflows/claude.yml.github/workflows/auto-approve.ymlcopilot[bot]review existence + inline comments.github/workflows/pr-tests.ymlHow Copilot Review Works
copilot_code_reviewenabledUnity Test Skip Logic
Tests now only run when these paths change:
UnityProject/Packages/com.jasonxudeveloper.jengine.core/**UnityProject/Packages/com.jasonxudeveloper.jengine.util/**UnityProject/Assets/Tests/**.github/workflows/unity-tests.yml.github/workflows/pr-tests.ymlCI-only changes (auto-approve, labeler, etc.) will skip Unity tests.
Test plan
🤖 Generated with Claude Code