Unverified 提交 46b4637d authored 作者: Will Chen's avatar Will Chen 提交者: GitHub

Update PR Review Responder workflow to do an extra push just in case (#2641)

## Summary Adds a comprehensive GitHub Actions workflow that automatically responds to pull requests with AI-powered code reviews and automated fixes. The workflow handles PR labeling, manages retry logic (up to 3 retries), and integrates with Claude Code for automated problem resolution. ## Test plan - Verify the workflow is properly formatted and no syntax errors exist - Check that the workflow triggers on the correct events (pull_request_target with labels and workflow_run) - Verify that only trusted maintainers (wwwillchen, wwwillchen-bot) can trigger the workflow - Test the retry logic by checking label transitions (cc:request → cc:pending → cc:request:N) - Confirm the workflow creates meaningful git commits through Claude Code - Verify proper cleanup of labels when max retries (4 total) are reached 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2641" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Small workflow change, but it affects automation that can push to PR branches; failure modes are limited to CI/labeling behavior. > > **Overview** > The `PR Review Responder` workflow now runs an additional Claude Code step after `/dyad:pr-fix` to push any local commits that may have been created but not pushed (via a new `/dyad:fast-push` invocation). > > This reduces cases where the workflow would proceed to “check if commits were pushed” and label transitions without the PR branch actually being updated. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 27a3059357b5f2511abc39d1a7c3aedccd92b38c. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Adds a GitHub Actions workflow that automatically reviews pull requests, manages labels/retry logic, and applies fixes via Claude Code. Adds a fallback step to push any unpushed commits to ensure downstream workflows run and PR labels update correctly. <sup>Written for commit 27a3059357b5f2511abc39d1a7c3aedccd92b38c. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. -->
上级 40060a82
......@@ -250,6 +250,22 @@ jobs:
/dyad:pr-fix ${{ steps.pr-info.outputs.pr_number }}
- name: Push any unpushed changes
# In case the previous step accidentally forgot to push its changes
if: steps.pr-info.outputs.should_continue == 'true' && always()
uses: anthropics/claude-code-action@v1
with:
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
github_token: ${{ secrets.GITHUB_TOKEN }}
claude_args: --model claude-opus-4-6
prompt: |
The previous step may have made local commits but
accidentally forgot to push them to the origin remote. Run the following skill to
push any unpushed changes. If there are no unpushed commits, that's fine — just
exit successfully.
/dyad:fast-push
- name: Check if commits were pushed
# Use always() to ensure we detect commits even if Claude Code fails partway through.
# The push itself (made with PR_CONTENTS_RW_GITHUB_TOKEN) triggers downstream workflows
......
Markdown 格式
0%
您添加了 0 到此讨论。请谨慎行事。
请先完成此评论的编辑!
注册 或者 后发表评论