-
由 Will Chen 提交于
## Summary - The `/dyad:pr-fix` skill was missing a push step, so Claude would fix review comments and CI issues but never commit/push the changes back to the PR branch - Added `/dyad:pr-push` as an explicit step 3 in the skill (between fix steps and summary comment) - Simplified the workflow prompt in `pr-review-responder.yml` since the skill itself now handles the push ## Test plan - [ ] Trigger the PR Review Responder workflow on a PR with `cc:request` label and verify that changes are actually pushed
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2612" 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** > Documentation/prompt-orchestration changes only; risk is limited to altering the automation flow if the new required `/dyad:pr-push` step behaves unexpectedly. > > **Overview** > Updates the `/dyad:pr-fix` orchestration to **explicitly run `/dyad:pr-push` as a required step** before posting the final PR summary comment, ensuring fixes are actually committed and pushed back to the PR branch. > > Simplifies the `pr-review-responder.yml` Claude prompt by removing redundant detailed rules and relying on the updated skill to execute end-to-end without skipping the push. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 542b65bac4120de76a8759ce3cf3e2e021e4fb98. 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 Add a required /dyad:pr-push step to the /dyad:pr-fix skill so fixes are committed and pushed to the PR branch. Simplifies the pr-review-responder workflow now that pushing is handled by the skill. - **Bug Fixes** - Added /dyad:pr-push as step 3 in pr-fix to commit, run lint/tests, and push to GitHub. - Simplified pr-review-responder.yml prompt to run pr-fix end-to-end without manual push checks. <sup>Written for commit 542b65bac4120de76a8759ce3cf3e2e021e4fb98. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Claude Opus 4.6 <noreply@anthropic.com>
| 名称 |
最后提交
|
最后更新 |
|---|---|---|
| .. | ||
| dyad | ||
| remember-learnings.md |