Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 6, 2026

The workflow was caching and installing npm dependencies from pkg/workflow/js/ which doesn't contain a package.json. JavaScript dependencies are in actions/setup/js/.

Changes

  • Updated cache-dependency-path from pkg/workflow/js/package-lock.json to actions/setup/js/package-lock.json
  • Updated npm install step to run at actions/setup/js directory instead of pkg/workflow/js

Aligns with the pattern used in .github/workflows/ci.yml.

Original prompt

Fix the copilot-setup-steps node js installation. https://github.com/githubnext/gh-aw/actions/runs/20756955042/job/59601516188

Run actions/setup-node@395ad3262231945c25e8478fd5baf05154b1d79f
Found in cache @ /opt/hostedtoolcache/node/24.12.0/x64
Environment details
  (node:2185) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
  (Use `node --trace-deprecation ...` to show where the warning was created)
  node: v24.12.0
  npm: 11.6.2
  yarn: 1.22.22
/opt/hostedtoolcache/node/24.12.0/x64/bin/npm config get cache
/home/runner/.npm
Error: Some specified paths were not resolved, unable to cache dependencies.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix copilot setup steps Node.js installation Fix copilot-setup-steps workflow npm cache path Jan 6, 2026
Copilot AI requested a review from pelikhan January 6, 2026 18:02
cache: true
go-version-file: go.mod
- name: Install JavaScript dependencies
run: cd pkg/workflow/js && npm ci
Copy link
Contributor

Choose a reason for hiding this comment

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

@copilot keep but run at actions/setup/js/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Restored the npm ci step to run at actions/setup/js/ (commit 8a145d4)

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI requested a review from pelikhan January 6, 2026 18:11
@pelikhan pelikhan marked this pull request as ready for review January 6, 2026 18:38
@pelikhan pelikhan merged commit f3eef77 into main Jan 6, 2026
85 checks passed
@pelikhan pelikhan deleted the copilot/fix-copilot-setup-steps branch January 6, 2026 18:39
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