Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 29, 2026

Summary

  • What: Standardize Playwright workflow to use Node 20 and pnpm exclusively
  • Why: Eliminate package manager inconsistency, reduce CI times via caching

Changes

  • Upgrade Node.js from 18 to 20 (matches ci.yml, web-ci.yml, storybook.yml)
  • Add pnpm/action-setup@v3 with version 9 and enable pnpm cache in actions/setup-node
  • Replace all npm commands: npm cipnpm install --frozen-lockfile, npxpnpm exec, npm install --no-savepnpm dlx
  • Remove duplicate pnpm install step and manual corepack setup (handled by pnpm/action-setup)

Migration steps

  1. N/A - workflow-only change

Testing

  • Automated: Workflow syntax validated, code review passed, security scan clean

Rollback plan

  • Revert to commit 9e245ce if workflow fails in CI

Checklist

  • Node.js updated to 20
  • pnpm/action-setup added with caching
  • All npm commands replaced with pnpm
  • Duplicate steps removed
  • YAML syntax validated
  • Security scan passed

Reviewers


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

Copilot AI and others added 2 commits January 29, 2026 22:16
Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>
Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>
Copilot AI changed the title [WIP] Update npm auth configuration for private registry Align Playwright workflow with CI standards: Node 20 and pnpm Jan 29, 2026
Copilot AI requested a review from 73junito January 29, 2026 22:20
Copy link
Owner

@73junito 73junito left a comment

Choose a reason for hiding this comment

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

viewed

@73junito 73junito marked this pull request as ready for review January 30, 2026 00:09
@73junito 73junito merged commit d16c045 into chore/npm-auth-config Jan 30, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants