• Will Chen's avatar
    Improve Claude Code hooks and skills configuration (#2439) · c692cdc9
    Will Chen 提交于
    ## Summary
    - Add unified `DYAD_DISABLE_CLAUDE_CODE_HOOKS` env var to disable
    permission-request and stop hooks
    - Update permission policy to explicitly allow GitHub comment/resolve
    operations (not destructive)
    - Enhance pr-fix:comments skill to always explicitly resolve threads
    after addressing
    - Add `cc:request` label for non-trivial PRs in pr-push skill
    
    ## Test plan
    - Set `DYAD_DISABLE_CLAUDE_CODE_HOOKS=1` and verify hooks are bypassed
    - Run `/dyad:pr-fix:comments` and verify all addressed threads are
    resolved
    - Run `/dyad:pr-push` on a non-trivial PR and verify `cc:request` label
    is added
    
    #skip-bugbot
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    <!-- devin-review-badge-begin -->
    
    ---
    
    <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2439">
      <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 -->
    
    
    <!-- This is an auto-generated description by cubic. -->
    ---
    ## Summary by cubic
    Unifies hook disabling under DYAD_DISABLE_CLAUDE_CODE_HOOKS, clarifies
    non-destructive GitHub actions in the permission policy, and improves PR
    skills to resolve threads explicitly and request review on non-trivial
    changes.
    
    - **New Features**
    - Single env var to disable both permission-request and stop hooks:
    DYAD_DISABLE_CLAUDE_CODE_HOOKS.
    - Permission policy explicitly allows commenting/replying and resolving
    review threads (non-destructive).
    - pr-fix:comments explicitly resolves addressed threads and verifies all
    trusted threads are handled.
      - pr-push adds the cc:request label for non-trivial PRs.
    
    - **Migration**
    - Replace DISABLE_DYAD_STOP_HOOK with DYAD_DISABLE_CLAUDE_CODE_HOOKS
    (true/1/yes) to disable both hooks.
    
    <sup>Written for commit a09d0458ef620da8b2358facf7712f5c47362030.
    Summary will update on new commits.</sup>
    
    <!-- End of auto-generated description by cubic. -->
    Co-authored-by: 's avatarClaude Opus 4.5 <noreply@anthropic.com>
    c692cdc9
pr-push.md 4.6 KB