Skip to content

fix: run post-clone hook before worktree-post-create during clone#233

Merged
avihut merged 2 commits intomasterfrom
hook-order
Feb 18, 2026
Merged

fix: run post-clone hook before worktree-post-create during clone#233
avihut merged 2 commits intomasterfrom
hook-order

Conversation

@avihut
Copy link
Owner

@avihut avihut commented Feb 18, 2026

Summary

  • Swap hook execution order during git worktree-clone so post-clone runs before worktree-post-create
  • post-clone is the repo bootstrap hook (install toolchains like pnpm, bun, uv) and should run first so worktree-post-create (per-worktree setup like pnpm install) can depend on those tools being present
  • Document the execution order in the hooks guide and SKILL.md

Test plan

  • mise run clippy passes with zero warnings
  • mise run fmt produces no changes
  • mise run test-unit passes (333 tests)
  • Manual test: clone a repo with both post-clone and worktree-post-create hooks, verify post-clone runs first

🤖 Generated with Claude Code

avihut and others added 2 commits February 18, 2026 21:39
post-clone is the repo bootstrap hook (install toolchains like pnpm,
bun, uv) and should run before worktree-post-create, which handles
per-worktree setup that may depend on those tools being present.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Clarify that post-clone runs before worktree-post-create during
git worktree-clone, so foundational tools are installed first.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@avihut avihut added this to the v1.1.0 milestone Feb 18, 2026
@avihut avihut added bug Something isn't working fix Bug fix labels Feb 18, 2026
@avihut avihut self-assigned this Feb 18, 2026
@avihut avihut merged commit 90b3676 into master Feb 18, 2026
6 checks passed
@avihut avihut deleted the hook-order branch February 18, 2026 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant