Skip to content

CLI-1871: Add sync-skills GitHub Actions workflow. - #2044

Merged
deepakmishra2 merged 5 commits into
mainfrom
CLI-1871
Sep 9, 2026
Merged

CLI-1871: Add sync-skills GitHub Actions workflow.#2044
deepakmishra2 merged 5 commits into
mainfrom
CLI-1871

Conversation

@deepakmishra2

Copy link
Copy Markdown
Contributor

This pull request introduces a new GitHub Actions workflow, sync-skills.yml, to automate syncing ACLI skills with the acquia-skills repository. The workflow is triggered on releases and manual dispatch, and it handles version resolution, downloads the ACLI binary, generates command data, and dispatches a sync event to the target repository.

Workflow automation:

  • Added .github/workflows/sync-skills.yml to automate syncing ACLI skills to the acquia-skills repository on release or manual trigger, including options for forced syncs.
  • The workflow resolves the correct version tag, reads the Acquia spec version, downloads the appropriate ACLI binary, generates a JSON list of commands, and triggers a repository dispatch event with relevant payload data.

@deepakmishra2 deepakmishra2 self-assigned this Sep 7, 2026
Copilot AI lite review requested due to automatic review settings September 7, 2026 09:32
@codecov

codecov Bot commented Sep 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.76%. Comparing base (09a000b) to head (ab8ebac).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #2044   +/-   ##
=========================================
  Coverage     92.76%   92.76%           
  Complexity     2032     2032           
=========================================
  Files           126      126           
  Lines          7338     7338           
=========================================
  Hits           6807     6807           
  Misses          531      531           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Try the dev build for this PR: https://acquia-cli.s3.amazonaws.com/build/pr/2044/acli.phar

curl -OL https://acquia-cli.s3.amazonaws.com/build/pr/2044/acli.phar
chmod +x acli.phar

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The new workflow is likely to fail in common cases (missing PHP runtime, writing to /usr/local/bin, and manual dispatch tag resolution) and includes an unused command JSON generation step that needs clarification or integration.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

This PR adds a new GitHub Actions workflow (sync-skills.yml) intended to automate syncing Acquia CLI (ACLI) “skills” metadata to the separate acquia-skills repository when a release is published or when manually triggered.

Changes:

  • Introduces a release + workflow_dispatch triggered workflow to resolve an ACLI version, read the Acquia spec version, and download an ACLI release PHAR.
  • Generates a commands JSON file via acli list --format=json.
  • Dispatches a repository_dispatch event to acquia/acquia-skills with version/spec/force payload fields.
File summaries
File Description
.github/workflows/sync-skills.yml Adds an automation workflow to orchestrate version resolution, binary download, and cross-repo dispatch for syncing skills.
Review details

Suppressed comments (1)

.github/workflows/sync-skills.yml:29

  • On manual runs, resolving the tag via github.ref_name can point at a branch name. Consider honoring the new tag input and otherwise resolving the latest release tag so the download step consistently targets a real release asset.
      - name: Resolve version tag
        id: version
        run: |
          if [ "${{ github.event_name }}" = "release" ]; then
            echo "tag=${{ github.event.release.tag_name }}" >> $GITHUB_OUTPUT
          else
            echo "tag=${{ github.ref_name }}" >> $GITHUB_OUTPUT
          fi
  • Files reviewed: 1/1 changed files
  • Comments generated: 4
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/sync-skills.yml Outdated
Comment thread .github/workflows/sync-skills.yml
Comment thread .github/workflows/sync-skills.yml Outdated
Comment thread .github/workflows/sync-skills.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Comment thread .github/workflows/sync-skills.yml Outdated
Comment thread .github/workflows/sync-skills.yml Outdated
Comment thread .github/workflows/sync-skills.yml Outdated
Comment thread .github/workflows/sync-skills.yml Outdated
@deepakmishra2
deepakmishra2 merged commit b70f6f3 into main Sep 9, 2026
26 checks passed
@deepakmishra2
deepakmishra2 deleted the CLI-1871 branch September 9, 2026 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants