• 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
名称
最后提交
最后更新
.claude 正在载入提交数据...
.cursor/rules 正在载入提交数据...
.devcontainer 正在载入提交数据...
.github 正在载入提交数据...
.husky 正在载入提交数据...
assets 正在载入提交数据...
docs 正在载入提交数据...
drizzle 正在载入提交数据...
e2e-tests 正在载入提交数据...
makers 正在载入提交数据...
packages/@dyad-sh 正在载入提交数据...
scaffold 正在载入提交数据...
scripts 正在载入提交数据...
shared 正在载入提交数据...
src 正在载入提交数据...
testing 正在载入提交数据...
tools 正在载入提交数据...
worker 正在载入提交数据...
workers/tsc 正在载入提交数据...
.cursorignore 正在载入提交数据...
.env.example 正在载入提交数据...
.eslintrc.json 正在载入提交数据...
.gitattributes 正在载入提交数据...
.gitignore 正在载入提交数据...
.oxfmtrc.json 正在载入提交数据...
.oxlintrc.json 正在载入提交数据...
.prettierignore 正在载入提交数据...
.prettierrc 正在载入提交数据...
AGENTS.md 正在载入提交数据...
CLA.md 正在载入提交数据...
CLAUDE.md 正在载入提交数据...
CONTRIBUTING.md 正在载入提交数据...
LICENSE 正在载入提交数据...
README.md 正在载入提交数据...
SECURITY.md 正在载入提交数据...
biome.json 正在载入提交数据...
components.json 正在载入提交数据...
drizzle.config.ts 正在载入提交数据...
forge.config.ts 正在载入提交数据...
forge.env.d.ts 正在载入提交数据...
index.html 正在载入提交数据...
lint-staged.config.js 正在载入提交数据...
merge.config.ts 正在载入提交数据...
package-lock.json 正在载入提交数据...
package.json 正在载入提交数据...
playwright.config.ts 正在载入提交数据...
tsconfig.app.json 正在载入提交数据...
tsconfig.json 正在载入提交数据...
tsconfig.node.json 正在载入提交数据...
vite.main.config.mts 正在载入提交数据...
vite.preload.config.mts 正在载入提交数据...
vite.renderer.config.mts 正在载入提交数据...
vite.worker.config.mts 正在载入提交数据...
vitest.config.ts 正在载入提交数据...
windowsSign.ts 正在载入提交数据...