Skip to content

Add GitHub Actions workflow to test install scripts#62

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/add-install-script-tests
Draft

Add GitHub Actions workflow to test install scripts#62
Copilot wants to merge 3 commits intomainfrom
copilot/add-install-script-tests

Conversation

Copy link
Contributor

Copilot AI commented Mar 2, 2026

The two install scripts (install/tools/install.sh, install/skills/install.sh) had no automated testing coverage, leaving regressions undetected on PRs.

Adds .github/workflows/test-install-scripts.yml with a single job running on the [self-hosted, mi3xx] custom runner:

test-install-scripts

Tools install script (install/tools/install.sh):

  • --help output and exit 0
  • --dry-run verifies all 6 tools appear in output
  • --dry-run with --pip-cmd, --repo-url, --ref — confirms custom values propagate
  • Error handling: unknown options exit non-zero

Skills install script (install/skills/install.sh):

  • --help output and exit 0
  • --dry-run for all four targets (agents, codex, cursor, claude)
  • --dry-run --global resolves to $HOME-prefixed path
  • Error handling: invalid targets exit non-zero
Original prompt

This section details on the original issue you should resolve

<issue_title>Add workflow to test skill and install scripts are working correctly</issue_title>
<issue_description>We have two install.sh scripts that we need to test on each PR. Add the necessary tests to make sure the scripts are well-tested.</issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: mawad-amd <112003944+mawad-amd@users.noreply.github.com>
Copilot AI changed the title [WIP] Add workflow to test install scripts functionality Add GitHub Actions workflow to test install scripts Mar 2, 2026
bash install/skills/install.sh --base-url 2>/dev/null && exit 1 || true

- name: Test actual download with local HTTP server
run: |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot this is too complicated. Check the other yamls and use our custom runner. simplify this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simplified in c13f27d — now a single job on [self-hosted, mi3xx] with two focused steps (one per script), matching the style of the existing CI workflow.

Co-authored-by: mawad-amd <112003944+mawad-amd@users.noreply.github.com>
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.

Add workflow to test skill and install scripts are working correctly

2 participants