Skip to content

Fix E2E PnP Angular workflow ESM resolution by upgrading to Node.js 22#1

Draft
Sawsqr68 with Copilot wants to merge 3 commits into
masterfrom
copilot/fix-action-run-issues
Draft

Fix E2E PnP Angular workflow ESM resolution by upgrading to Node.js 22#1
Sawsqr68 with Copilot wants to merge 3 commits into
masterfrom
copilot/fix-action-run-issues

Conversation

Copilot AI commented Dec 27, 2025

Copy link
Copy Markdown

The E2E PnP Angular workflow was failing when @angular/cli@next attempted to execute with yarn dlx. The failure occurred because yargs@18.0.0 (an Angular CLI dependency) uses ESM imports that Node.js 20's loader cannot resolve correctly in Yarn's PnP environment where packages are stored in ZIP archives.

Changes

  • Remove Node.js 20 version pin from e2e-pnp-angular-workflow.yml
  • Workflow now uses default Node.js 22 (from prepare action) which has improved ESM module resolution

The error manifested as:

Error [ERR_MODULE_NOT_FOUND]: Cannot find module '.../yargs/build/lib/utils/apply-extends.js' 
imported from .../yargs/helpers/helpers.mjs

Node.js 22's ESM loader correctly resolves the relative imports in yargs@18.0.0 within PnP's ZIP-based package structure.

Original prompt

Reference: https://github.com/Sawsqr68/berry_s/actions/runs/20539109592/job/59001079161#step:4:1


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits December 27, 2025 12:47
Co-authored-by: Sawsqr68 <173052944+Sawsqr68@users.noreply.github.com>
Co-authored-by: Sawsqr68 <173052944+Sawsqr68@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix action run issues in GitHub workflow Fix E2E PnP Angular workflow ESM resolution by upgrading to Node.js 22 Dec 27, 2025
Copilot AI requested a review from Sawsqr68 December 27, 2025 12:49
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