Skip to content

fix(skills): use CLAUDE_PLUGIN_ROOT in hook paths#1043

Open
Gujiassh wants to merge 2 commits intogarrytan:mainfrom
Gujiassh:fix/claude-plugin-root-hooks
Open

fix(skills): use CLAUDE_PLUGIN_ROOT in hook paths#1043
Gujiassh wants to merge 2 commits intogarrytan:mainfrom
Gujiassh:fix/claude-plugin-root-hooks

Conversation

@Gujiassh
Copy link
Copy Markdown

Summary

  • switch the generated hook paths to use
  • keep the skill docs/templates and generator test aligned with the current plugin root contract

@Gujiassh Gujiassh force-pushed the fix/claude-plugin-root-hooks branch from 06fb53c to 355efff Compare April 19, 2026 03:09
@Gujiassh
Copy link
Copy Markdown
Author

Follow-up maintenance on this lane:

  • rebased fix/claude-plugin-root-hooks onto current main
  • refreshed the ship golden baselines required by the newer host-config regression checks
  • force-pushed the updated branch head 355efffa

Validation run on the refreshed branch:

  • bun test test/host-config.test.ts
  • bun test test/gen-skill-docs.test.ts

This should clear the stale merge-base drift that had the PR in a dirty state.

@Gujiassh
Copy link
Copy Markdown
Author

Follow-up maintenance on this lane:\n\n- rebased onto current \n- refreshed the ship golden baselines required by the newer host-config regression checks\n- force-pushed the updated branch head \n\nValidation run on the refreshed branch:\n- bun test v1.3.11 (af24e281)\n- bun test v1.3.11 (af24e281)
GENERATED: SKILL.md
GENERATED: investigate/SKILL.md
GENERATED: design-review/SKILL.md
GENERATED: canary/SKILL.md
GENERATED: learn/SKILL.md
GENERATED: qa-only/SKILL.md
GENERATED: checkpoint/SKILL.md
GENERATED: autoplan/SKILL.md
GENERATED: design-consultation/SKILL.md
GENERATED: setup-deploy/SKILL.md
GENERATED: open-gstack-browser/SKILL.md
GENERATED: document-release/SKILL.md
GENERATED: land-and-deploy/SKILL.md
GENERATED: freeze/SKILL.md
GENERATED: guard/SKILL.md
GENERATED: pair-agent/SKILL.md
GENERATED: gstack-upgrade/SKILL.md
GENERATED: cso/SKILL.md
GENERATED: design-html/SKILL.md
GENERATED: design-shotgun/SKILL.md
GENERATED: setup-browser-cookies/SKILL.md
GENERATED: careful/SKILL.md
GENERATED: devex-review/SKILL.md
GENERATED: ship/SKILL.md
GENERATED: unfreeze/SKILL.md
GENERATED: office-hours/SKILL.md
GENERATED: plan-eng-review/SKILL.md
GENERATED: plan-ceo-review/SKILL.md
GENERATED: plan-design-review/SKILL.md
GENERATED: review/SKILL.md
GENERATED: codex/SKILL.md
GENERATED: qa/SKILL.md
GENERATED: benchmark/SKILL.md
GENERATED: retro/SKILL.md
GENERATED: browse/SKILL.md
GENERATED: plan-devex-review/SKILL.md
GENERATED: health/SKILL.md

Token Budget (claude host)
════════════════════════════════════════════════════════════
ship 2544 lines ~ 32133 tokens
plan-ceo-review 1838 lines ~ 27668 tokens
plan-devex-review 1834 lines ~ 23126 tokens
office-hours 1826 lines ~ 25057 tokens
design-review 1597 lines ~ 19450 tokens
land-and-deploy 1587 lines ~ 19548 tokens
plan-design-review 1530 lines ~ 21731 tokens
review 1468 lines ~ 18915 tokens
retro 1468 lines ~ 17203 tokens
autoplan 1465 lines ~ 18476 tokens
plan-eng-review 1432 lines ~ 20851 tokens
qa 1420 lines ~ 16103 tokens
design-consultation 1266 lines ~ 16390 tokens
cso 1226 lines ~ 16697 tokens
design-html 1181 lines ~ 13481 tokens
codex 1076 lines ~ 13726 tokens
devex-review 1035 lines ~ 13887 tokens
qa-only 1010 lines ~ 12078 tokens
design-shotgun 955 lines ~ 11479 tokens
document-release 926 lines ~ 11083 tokens
SKILL.md 882 lines ~ 9585 tokens
pair-agent 826 lines ~ 9736 tokens
checkpoint 814 lines ~ 9283 tokens
canary 807 lines ~ 9725 tokens
health 802 lines ~ 9539 tokens
investigate 776 lines ~ 9807 tokens
open-gstack-browser 771 lines ~ 9528 tokens
browse 763 lines ~ 8522 tokens
setup-deploy 736 lines ~ 8973 tokens
learn 708 lines ~ 8436 tokens
benchmark 674 lines ~ 7274 tokens
setup-browser-cookies 523 lines ~ 5731 tokens
gstack-upgrade 276 lines ~ 2672 tokens
freeze 83 lines ~ 751 tokens
guard 83 lines ~ 789 tokens
careful 60 lines ~ 615 tokens
unfreeze 41 lines ~ 340 tokens
────────────────────────────────────────────────────────────
TOTAL 38309 lines ~480388 tokens

GENERATED: SKILL.md
GENERATED: investigate/SKILL.md
GENERATED: design-review/SKILL.md
GENERATED: canary/SKILL.md
GENERATED: learn/SKILL.md
GENERATED: qa-only/SKILL.md
GENERATED: autoplan/SKILL.md
GENERATED: design-consultation/SKILL.md
GENERATED: setup-deploy/SKILL.md
GENERATED: open-gstack-browser/SKILL.md
GENERATED: document-release/SKILL.md
GENERATED: land-and-deploy/SKILL.md
GENERATED: freeze/SKILL.md
GENERATED: guard/SKILL.md
GENERATED: pair-agent/SKILL.md
GENERATED: gstack-upgrade/SKILL.md
GENERATED: cso/SKILL.md
GENERATED: design-html/SKILL.md
GENERATED: design-shotgun/SKILL.md
GENERATED: setup-browser-cookies/SKILL.md
GENERATED: careful/SKILL.md
GENERATED: devex-review/SKILL.md
GENERATED: ship/SKILL.md
GENERATED: unfreeze/SKILL.md
GENERATED: office-hours/SKILL.md
GENERATED: plan-eng-review/SKILL.md
GENERATED: plan-tune/SKILL.md
GENERATED: plan-ceo-review/SKILL.md
GENERATED: plan-design-review/SKILL.md
GENERATED: context-restore/SKILL.md
GENERATED: review/SKILL.md
GENERATED: codex/SKILL.md
GENERATED: context-save/SKILL.md
GENERATED: qa/SKILL.md
GENERATED: benchmark/SKILL.md
GENERATED: retro/SKILL.md
GENERATED: browse/SKILL.md
GENERATED: plan-devex-review/SKILL.md
GENERATED: health/SKILL.md

Token Budget (claude host)
════════════════════════════════════════════════════════════
ship 2882 lines ~ 36671 tokens
plan-ceo-review 2057 lines ~ 30527 tokens
office-hours 2039 lines ~ 27828 tokens
plan-devex-review 2023 lines ~ 25437 tokens
design-review 1937 lines ~ 24066 tokens
plan-design-review 1808 lines ~ 25371 tokens
land-and-deploy 1776 lines ~ 21851 tokens
autoplan 1711 lines ~ 21556 tokens
retro 1680 lines ~ 19775 tokens
review 1671 lines ~ 21341 tokens
plan-eng-review 1625 lines ~ 23194 tokens
qa 1613 lines ~ 18403 tokens
design-consultation 1459 lines ~ 18704 tokens
design-html 1455 lines ~ 16937 tokens
cso 1419 lines ~ 18999 tokens
codex 1348 lines ~ 17266 tokens
design-shotgun 1229 lines ~ 14941 tokens
devex-review 1224 lines ~ 16194 tokens
qa-only 1199 lines ~ 14380 tokens
document-release 1115 lines ~ 13389 tokens
plan-tune 1080 lines ~ 13175 tokens
pair-agent 1015 lines ~ 12040 tokens
canary 996 lines ~ 12031 tokens
health 991 lines ~ 11842 tokens
investigate 979 lines ~ 12290 tokens
context-save 971 lines ~ 11536 tokens
open-gstack-browser 960 lines ~ 11834 tokens
SKILL.md 926 lines ~ 10247 tokens
setup-deploy 925 lines ~ 11276 tokens
learn 897 lines ~ 10739 tokens
browse 857 lines ~ 9837 tokens
context-restore 853 lines ~ 10635 tokens
benchmark 714 lines ~ 7729 tokens
setup-browser-cookies 563 lines ~ 6187 tokens
gstack-upgrade 280 lines ~ 2693 tokens
freeze 87 lines ~ 774 tokens
guard 87 lines ~ 810 tokens
careful 64 lines ~ 632 tokens
unfreeze 45 lines ~ 361 tokens
────────────────────────────────────────────────────────────
TOTAL 46560 lines ~583498 tokens\n\nThis should clear the stale merge-base drift that had the PR in a dirty state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant