• Will Chen's avatar
    Add /dyad:pr-push step to pr-fix skill (#2612) · 6ae812e5
    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: 's avatarClaude Opus 4.6 <noreply@anthropic.com>
    6ae812e5
名称
最后提交
最后更新
..
dyad 正在载入提交数据...
remember-learnings.md 正在载入提交数据...