Skip to content

Migrate: Replace ci-design-md-check.sh with JavaScript #612

@ashleyshaw

Description

@ashleyshaw

Overview

Migrate skills/design-md-agent/design-md-generator/scripts/ci-design-md-check.sh from Bash to JavaScript with Jest test coverage.

Current Implementation

The script:

  • Runs DESIGN.md validation via validate-design-md.sh
  • Executes the design.md CLI lint command in JSON format
  • Parses JSON results and creates a PR comment with summary and top findings
  • Exits with error code if lint errors found

Requirements

  • Create scripts/design-md-agent/ciDesignMdCheck.js equivalent
  • Leverage the validateDesignMd module (see related issue)
  • Parse JSON lint output and extract summary stats
  • Generate PR comment markdown with findings (limit to top 10)
  • Jest tests with mocked design.md CLI outputs
  • Handle both jq-available and non-available environments
  • Proper error handling and exit codes
  • Remove ci-design-md-check.sh after verification

Related

  • Part of #[bash-to-js-migration-epic]
  • Depends on: #[validate-design-md-issue]

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