Skip to content

refactor(skills): fold 11 language/shell/k8s husk skills into umbrella references (M2) - #897

Merged
notque merged 1 commit into
mainfrom
m2-fold-language-husks
Jul 25, 2026
Merged

refactor(skills): fold 11 language/shell/k8s husk skills into umbrella references (M2)#897
notque merged 1 commit into
mainfrom
m2-fold-language-husks

Conversation

@notque

@notque notque commented Jul 25, 2026

Copy link
Copy Markdown
Owner

What

Folds 11 husk skills into the umbrella skills that already carry their content. This is a merge, not a removal — every umbrella stays, and no capability is dropped.

Husk Folded into SKILL.md similarity
kotlin-coroutines engineering/kotlin/references/ 0.93
kotlin-testing engineering/kotlin/references/ 0.97
swift-concurrency engineering/swift/references/ 0.87
swift-testing engineering/swift/references/ 0.96
php-quality engineering/php/references/ 0.74 (after unique content merged in)
php-testing engineering/php/references/ 0.77 (after unique content merged in)
fish-shell-config infrastructure/shell-config/references/ 0.82
zsh-shell-config infrastructure/shell-config/references/ 0.91
kubernetes-debugging infrastructure/kubernetes/references/ condensed index
kubernetes-security infrastructure/kubernetes/references/ condensed index
cobalt-core infrastructure/kubernetes/references/cobalt-*.md condensed index

Verification

Every husk references/*.md file was compared with difflib.SequenceMatcher
against its umbrella counterpart before removal:

1.000  crash-diagnosis.md      1.000  network-policies.md
0.997  network-debugging.md    1.000  pod-security.md
1.000  resource-debugging.md   1.000  rbac-patterns.md
1.000  supply-chain.md         1.000  cobalt/concurrency-patterns.md
1.000  cobalt/kvm-exporter.md  1.000  cobalt/testing-patterns.md

The three k8s-cluster husks score low at the SKILL.md level only because the
umbrella's same-named reference is a condensed rewrite of the husk body
(the point of the exercise), while the deep content is preserved verbatim.

Inbound references

Swept all 11 names across agents, skills, hooks, scripts, docs, commands. Updated:
hooks/README.md, docs/*, README.md, scripts/fix-skill-paths.py,
scripts/migrate-skills-to-folders.py, shell-config + kubernetes routing tables,
and the two fish-shell-config FORCE-ROUTE examples in skills/meta/do/SKILL.md
(2-line rename only — no other part of that file is touched).

Skills: 131 -> 120. validate-doc-counts.py agrees.

Gates

  • ruff check / ruff format --check (0.15.12) — pass
  • pytest scripts/tests/ — 4700 passed, 1 skipped, 149 xfailed
  • validate-skill-frontmatter.py — 120 OK
  • validate-index-integrity.py — 0 errors, VERDICT PASS
  • validate-doc-counts.py — all count claims agree

…a references

Folds husks whose content already lives in the umbrella they point at:

- kotlin-coroutines (0.93), kotlin-testing (0.97) -> engineering/kotlin/references/
- swift-concurrency (0.87), swift-testing (0.96)  -> engineering/swift/references/
- php-quality, php-testing                        -> engineering/php/references/
- fish-shell-config (0.87), zsh-shell-config (0.91) -> infrastructure/shell-config/references/
- kubernetes-debugging, kubernetes-security       -> infrastructure/kubernetes/references/
- cobalt-core (promoted_to: kubernetes)           -> infrastructure/kubernetes/references/cobalt-*.md

Every husk reference file was verified byte-identical (difflib ratio 1.000)
against its umbrella counterpart before removal. Where the husk SKILL.md body
held unique guidance not yet in the umbrella (php-quality, php-testing, fish,
zsh), that content was merged into the umbrella reference first; the umbrella
index files are condensed rewrites, which is why SKILL.md-level ratios are
lower than the reference-level ones.

Umbrellas are unchanged in scope - the kubernetes references retain the
private infra knowledge. Inbound references updated across docs, hooks/README,
migration scripts, and shell-config routing tables.

Skills: 131 -> 120.
@github-actions

Copy link
Copy Markdown

Skill-eval coverage

Changed skills in this PR: 6 (426ef925b7f3..617c0514606a).

Report only. CI cannot run evals (the runner needs the claude CLI). Run the ablation locally to get the base->head delta.

Mapped

  • do -> eval evals/new-skills-ab-test. Run locally: make skill-eval-ablation BASE=426ef925b7f3 HEAD=617c0514606a SKILL=do

Uncovered

no eval coverage for changed skill(s): kubernetes, public-web-deploy, service-health-check, shell-process-patterns, skill-creator
A gap to consider, not a failure.

@notque
notque merged commit 20a560e into main Jul 25, 2026
11 checks passed
@notque
notque deleted the m2-fold-language-husks branch July 25, 2026 01:43
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