Skip to content

Migrate: Replace quick_check_skill.sh with JavaScript #613

Description

@ashleyshaw

Overview

Migrate skills/design-md-agent/*/scripts/quick_check_skill.sh (appears in 2 locations) from Bash to JavaScript with Jest test coverage.

Current Implementation

The script validates skill structure by checking:

  • Skill directory exists
  • SKILL.md file exists
  • agents/openai.yaml file exists
  • SKILL.md starts with YAML frontmatter (---)
  • Frontmatter name is lowercase hyphen-case
  • Frontmatter description field is present
  • Folder name matches frontmatter name
  • No package noise (macOS, Python, Node artifacts)
  • Optional: warnings for TODO/placeholder text

Locations

  1. skills/design-md-agent/figma-wordpress-skill-creator/scripts/quick_check_skill.sh
  2. skills/design-md-agent/wordpress-plugin-packaging-review/scripts/quick_check_skill.sh

Requirements

  • Create scripts/skill-utils/validateSkillStructure.js (single source of truth)
  • Support both with/without grep exclusion pattern for TODO checks
  • Return exit codes: 0 (pass), 1 (fail), 2 (usage)
  • Jest tests covering all validation rules
  • Console output matching original implementation
  • Remove both .sh files after verification

Related

Part of #[bash-to-js-migration-epic]

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Priority

    None yet

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions