Skip to content

Add Copilot setup workflow#811

Merged
jonobr1 merged 1 commit into
devfrom
alert-copilot-setup
Apr 6, 2026
Merged

Add Copilot setup workflow#811
jonobr1 merged 1 commit into
devfrom
alert-copilot-setup

Conversation

@jonobr1
Copy link
Copy Markdown
Owner

@jonobr1 jonobr1 commented Apr 6, 2026

Add .github/workflows/copilot-setup-steps.yml to run Copilot setup steps. The workflow triggers on manual dispatch and on PRs that modify this file to enable validation and manual testing. It defines the required job name copilot-setup-steps, runs on ubuntu-latest, checks out the repo, sets up Node.js 20 with npm caching, installs dependencies and runs the build.

Add .github/workflows/copilot-setup-steps.yml to run Copilot setup steps. The workflow triggers on manual dispatch and on PRs that modify this file to enable validation and manual testing. It defines the required job name copilot-setup-steps, runs on ubuntu-latest, checks out the repo, sets up Node.js 20 with npm caching, installs dependencies and runs the build.
Copilot AI review requested due to automatic review settings April 6, 2026 20:47
@jonobr1 jonobr1 merged commit 48449b8 into dev Apr 6, 2026
4 checks passed
@jonobr1 jonobr1 deleted the alert-copilot-setup branch April 6, 2026 20:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a GitHub Actions workflow to run the required “Copilot setup steps” job for validation and manual testing.

Changes:

  • Introduces a new workflow triggered by workflow_dispatch and PRs that modify the workflow file.
  • Defines the copilot-setup-steps job to checkout, set up Node.js 20 with npm caching, install deps, and build.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +21 to +26
# checkout@v5
uses: jonobr1/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8

- name: Set up Node.js
# setup-node@v5
uses: jonobr1/setup-node@a0853c24544627f65ddf259abe73b1d18a591444
Copy link

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

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

Using third-party forked actions for checkout and setup-node increases supply-chain risk and makes provenance less clear. Prefer the official actions/checkout and actions/setup-node actions (still pinned to a commit SHA for security), and keep the uses: and inline comments consistent with the chosen actions.

Suggested change
# checkout@v5
uses: jonobr1/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
- name: Set up Node.js
# setup-node@v5
uses: jonobr1/setup-node@a0853c24544627f65ddf259abe73b1d18a591444
# actions/checkout@v4
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
- name: Set up Node.js
# actions/setup-node@v4
uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8

Copilot uses AI. Check for mistakes.
cache: "npm"

- name: Install dependencies
run: npm install
Copy link

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

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

For CI builds, npm ci is typically preferred over npm install because it produces deterministic installs from the lockfile and fails if the lockfile is out of sync. Switching to npm ci improves reproducibility and avoids unintended lockfile/dep resolution drift.

Suggested change
run: npm install
run: npm ci

Copilot uses AI. Check for mistakes.
Comment on lines +21 to +25
# checkout@v5
uses: jonobr1/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8

- name: Set up Node.js
# setup-node@v5
Copy link

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

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

These inline comments are misleading because the workflow is not actually using checkout@v5 / setup-node@v5. Update the comments to match the real action source/version being used (or remove the versioned comments) to avoid confusion during maintenance.

Suggested change
# checkout@v5
uses: jonobr1/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
- name: Set up Node.js
# setup-node@v5
uses: jonobr1/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
- name: Set up Node.js

Copilot uses AI. Check for mistakes.
Comment on lines +21 to +25
# checkout@v5
uses: jonobr1/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8

- name: Set up Node.js
# setup-node@v5
Copy link

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

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

These inline comments are misleading because the workflow is not actually using checkout@v5 / setup-node@v5. Update the comments to match the real action source/version being used (or remove the versioned comments) to avoid confusion during maintenance.

Suggested change
# checkout@v5
uses: jonobr1/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
- name: Set up Node.js
# setup-node@v5
uses: jonobr1/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
- name: Set up Node.js

Copilot uses AI. Check for mistakes.
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.

2 participants