Skip to content

refactor(skills): move contrast + animation-map scripts into hyperframes skill - #281

Closed
jrusso1020 wants to merge 1 commit into
04-15-fix_readme_correct_skills_table_to_match_actual_skill_namesfrom
04-15-refactor_skills_move_contrast_animation-map_scripts_into_hyperframes_skill
Closed

refactor(skills): move contrast + animation-map scripts into hyperframes skill#281
jrusso1020 wants to merge 1 commit into
04-15-fix_readme_correct_skills_table_to_match_actual_skill_namesfrom
04-15-refactor_skills_move_contrast_animation-map_scripts_into_hyperframes_skill

Conversation

@jrusso1020

@jrusso1020 jrusso1020 commented Apr 15, 2026

Copy link
Copy Markdown
Collaborator

What

Moves skills/hyperframes-animation-map/ and skills/hyperframes-contrast/ scripts into skills/hyperframes/scripts/, and removes the now-empty top-level skill directories.

Why

These two directories had scripts but no SKILL.md — they looked like broken/incomplete skills and wouldn't be installed by npx skills add. They're actually helper scripts invoked by the main hyperframes skill (SKILL.md already references them in the "Quality Checks" section). Moving them under skills/hyperframes/scripts/ makes them part of the skill they belong to, so they get installed alongside it.

How

  • Moved skills/hyperframes-animation-map/scripts/animation-map.mjsskills/hyperframes/scripts/animation-map.mjs
  • Moved skills/hyperframes-contrast/scripts/contrast-report.mjsskills/hyperframes/scripts/contrast-report.mjs
  • Removed empty skills/hyperframes-animation-map/ and skills/hyperframes-contrast/ directories
  • Updated path references in:
    • skills/hyperframes/SKILL.md (animation-map usage example)
    • skills/hyperframes/scripts/animation-map.mjs (usage comment)
    • skills/hyperframes/scripts/contrast-report.mjs (usage comment)
    • packages/cli/src/commands/contrast-audit.browser.js (keep-in-sync comment)

Test plan

  • Verify grep -r "hyperframes-animation-map\|hyperframes-contrast" . --include="*.md" --include="*.mjs" --include="*.js" --include="*.ts" returns no results (excluding node_modules)
  • Verify skills/hyperframes/scripts/ contains both scripts
  • Verify skills/hyperframes/SKILL.md references the new path
  • Documentation updated (if applicable)

Copy link
Copy Markdown
Collaborator Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@jrusso1020
jrusso1020 deleted the branch 04-15-fix_readme_correct_skills_table_to_match_actual_skill_names April 15, 2026 03:36
@jrusso1020 jrusso1020 closed this Apr 15, 2026
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