Skip to content

fix(setup): teach the update skill's drift check about the dispatcher hook flavour #1322

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
potiuk merged 1 commit into from
Sep 21, 2026
+12 −5
Merged

fix(setup): teach the update skill's drift check about the dispatcher hook flavour #1322

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions docs/mode-economics.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ special-token spellings counted as ordinary text.
Coverage: **75 of 75 local `skills/*/SKILL.md` files**.
External `source.md` redirects and harness symlinks are excluded.

Measurement manifest SHA-256: `1ae8680e2a68bddd8a2d9ce0a761b055cebac5ea74312ba2a05196b6f3ba3cd9`.
Measurement manifest SHA-256: `d4084d97d203d0df2895a57eea81b74334fcb357c4ca6db7a8b2918ab579a026`.

| Skill file | Measured tokens | Source SHA-256 (first 16 characters) |
|---|---:|---|
Expand Down Expand Up @@ -161,7 +161,7 @@ Measurement manifest SHA-256: `1ae8680e2a68bddd8a2d9ce0a761b055cebac5ea74312ba2a
| [setup](../skills/setup/SKILL.md) | 8,724 | `82788542bb240309` |
| [setup-isolated-setup-doctor](../skills/setup-isolated-setup-doctor/SKILL.md) | 7,950 | `3fa5d728fa080ed0` |
| [setup-isolated-setup-install](../skills/setup-isolated-setup-install/SKILL.md) | 11,278 | `418ea9794077c6e9` |
| [setup-isolated-setup-update](../skills/setup-isolated-setup-update/SKILL.md) | 5,111 | `96096e6ac10407d3` |
| [setup-isolated-setup-update](../skills/setup-isolated-setup-update/SKILL.md) | 5,214 | `97dabfc08b915b4a` |
| [setup-isolated-setup-verify](../skills/setup-isolated-setup-verify/SKILL.md) | 8,502 | `6f0a2594801bdf27` |
| [setup-override-upstream](../skills/setup-override-upstream/SKILL.md) | 4,012 | `fb583feb56b7f77c` |
| [setup-privacy-llm](../skills/setup-privacy-llm/SKILL.md) | 2,145 | `0e27b542a1656846` |
Expand Down
13 changes: 10 additions & 3 deletions plugins/magpie-setup/skills/isolated-setup-update/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -209,9 +209,16 @@ Walk each:
*only when whole-user scope is in effect, detected via
`git config --global --get core.hooksPath` resolving to
`~/.claude/git-hooks`* —
`~/.claude/git-hooks/post-checkout` for the universal
post-checkout hook), `diff` the user copy against the
framework's source-of-truth in `tools/agent-isolation/`.
the contents of `~/.claude/git-hooks/`, whose shape depends on
the flavour Step P.3 installed: the **simple** flavour puts a
copy of `git-global-post-checkout.sh` at `post-checkout`, while
the **dispatcher** flavour installs `git-hook-dispatcher.sh`
there and symlinks every hook name to it, superseding the
standalone post-checkout script — diff whichever script is
present against its own source, and read the hook-name symlinks
as the installed shape rather than as drift), `diff` the user
copy against the framework's source-of-truth in
`tools/agent-isolation/`.
Report any drift as a unified diff; do not re-`cp`. The
re-install path for each is
[`setup-isolated-setup-install`](../isolated-setup-install/SKILL.md)
Expand Down