Skip to content

refactor(skills): fold text-animation husk into distinctive-frontend-design (M3) - #898

Merged
notque merged 1 commit into
mainfrom
m3-fold-text-animation
Jul 25, 2026
Merged

refactor(skills): fold text-animation husk into distinctive-frontend-design (M3)#898
notque merged 1 commit into
mainfrom
m3-fold-text-animation

Conversation

@notque

@notque notque commented Jul 25, 2026

Copy link
Copy Markdown
Owner

What

Removes the text-animation husk, whose content already lives in distinctive-frontend-design.

Verified with difflib.SequenceMatcher before removal:

File Umbrella counterpart Ratio
references/roll-text.md distinctive-frontend-design/references/roll-text.md 1.000
references/text-animation-patterns.md .../references/text-animation-patterns.md 1.000
SKILL.md .../references/text-animation.md 0.911

The husk's frontmatter already declared promoted_to: distinctive-frontend-design, all six of its triggers are already duplicated into the umbrella's routing, and the umbrella's reference table dispatches them at SKILL.md:72. No capability is lost.

Inbound references repointed

  • agents/ui-design-engineer.md:159
  • agents/typescript-frontend-engineer.md:136
  • skills/meta/html-artifact/SKILL.md:363
  • scripts/migrate-skills-to-folders.py — stale path-map entry dropped
  • Doc counts 120 -> 119 in README.md, docs/docs-audit.md, docs/for-knowledge-workers.md, docs/for-linkedin.md

Scope note

This is the only merge of the twelve M3 candidates that survived inspection. The other eleven pairs measured 0.02-0.08 similarity and were each found to serve a distinct user intent (generative-vs-auditing, new-vs-existing, diff-vs-whole-repo). They are documented as KEEP-SEPARATE rather than merged — see the track report.

Gates

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

…design (M3)

text-animation declares promoted_to: distinctive-frontend-design and its
payload was already copied into the umbrella. Verified before removal:

  roll-text.md              ratio 1.000 (byte-identical)
  text-animation-patterns.md ratio 1.000 (byte-identical)
  SKILL.md -> references/text-animation.md  ratio 0.911

All six triggers already route to distinctive-frontend-design, whose
reference table dispatches them at SKILL.md:72. Keeping the standalone
copy was pure duplication with drift risk.

Repointed the three real path reads (agents/ui-design-engineer.md,
agents/typescript-frontend-engineer.md, skills/meta/html-artifact/SKILL.md)
at the umbrella's references, and dropped the stale migration-map entry.

Skills: 120 -> 119.
@github-actions

Copy link
Copy Markdown

Skill-eval coverage

Changed skills in this PR: 1 (20a560e536ac..02e9fb75e4cc).

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

Uncovered

no eval coverage for changed skill(s): html-artifact
A gap to consider, not a failure.

@notque
notque merged commit 3d8a0bb into main Jul 25, 2026
11 checks passed
@notque
notque deleted the m3-fold-text-animation branch July 25, 2026 01:59
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