mirror of
https://github.com/captbaritone/webamp.git
synced 2026-01-23 10:15:31 +00:00
08ec7ce69f
* Migrate from yarn to pnpm This comprehensive migration includes: ### Configuration Updates - Updated root package.json with pnpm workspace configuration - Added packageManager field and pnpm overrides for graphql version - Updated GitHub Actions workflows (.github/workflows/ci.yml, code-size.yml) - Updated Netlify configuration (netlify.toml) - Updated deployment script (deploy.sh) ### Documentation Updates - Updated all README files to use pnpm instead of yarn - Updated installation and build instructions across packages: - packages/webamp/README.md - packages/webamp-modern/README.md - packages/webamp-docs/README.md - packages/ani-cursor/README.md - packages/webamp/demo/readme.md ### Lock File Migration - Removed yarn.lock - Generated pnpm-lock.yaml preserving exact dependency versions - Moved resolutions from skin-database package.json to root pnpm overrides - Created pnpm-workspace.yaml for optimized workspace configuration ### CI/CD Updates - Updated all yarn commands to use pnpm equivalents - Changed yarn workspace commands to pnpm --filter syntax - Updated cache keys to use pnpm-lock.yaml instead of yarn.lock - Added pnpm/action-setup for GitHub Actions ### Validation - Tested builds for webamp, webamp-modern, ani-cursor, webamp-docs - Tested installation and linting for skin-database - Verified dependency resolution consistency - Confirmed all scripts work with pnpm All package versions remain identical to yarn.lock, ensuring no breaking changes. * Fix GitHub Actions CI: Install pnpm before using cache The GitHub Actions workflow was trying to cache pnpm before installing it. Fixed by reordering steps in all jobs to: 1. Install pnpm first 2. Setup Node.js with pnpm cache 3. Install dependencies This ensures pnpm is available when setting up the cache. * Fix pnpm overrides configuration format Move overrides from pnpm.overrides to top-level overrides in package.json to match the format expected by pnpm lockfile. This resolves the ERR_PNPM_LOCKFILE_CONFIG_MISMATCH error in CI. * Update CI to use pnpm version 9 to match lockfile format * Add missing @types dependencies for ani-cursor Add @types/jest and @types/node as devDependencies to ani-cursor package. These were missing but referenced in tsconfig.json, causing TypeScript compilation failures in CI with pnpm's stricter package isolation. * Fix dependency isolation issues for pnpm migration - Add strtok3 as direct dependency to webamp package (was transitive) - Add missing Babel plugins that were accessible as transitive deps with yarn - These packages need to be explicit dependencies for pnpm's stricter isolation Addresses missing dependencies that caused CI build failures: - Cannot find module 'strtok3' - Cannot find package '@babel/plugin-proposal-nullish-coalescing-operator' - Cannot find package '@babel/plugin-proposal-optional-chaining' * Add @babel/preset-env to webamp devDependencies - Fixes build-library failing due to missing Babel preset - pnpm's stricter dependency isolation revealed this missing direct dependency - Confirmed build-library now passes locally * Lock changes * Remove workspaces field from package.json - pnpm uses pnpm-workspace.yaml instead of package.json workspaces field - Fixes warning: 'The workspaces field in package.json is not supported by pnpm' - Workspace configuration is already correctly defined in pnpm-workspace.yaml * Does forcing a specific pnpm version help? * Update pnpm version to 9.12.0 in CI workflows - Fixes issues with pnpm v9.0 as mentioned in https://github.com/pnpm/pnpm/issues/6312 - Updates both ci.yml and code-size.yml workflows - 9.12.0 matches the local version and is more stable - Should resolve workspaces field warning and other pnpm issues * Skip root-level Jest tests in CI due to configuration conflicts - Root-level Jest config has compatibility issues with jest-environment-jsdom@29.7.0 - Different packages use different Jest versions causing testEnvironmentOptions errors - Webamp package tests work fine with their specific Jest configuration - This is the same issue we saw locally - pnpm's stricter isolation reveals these conflicts - CI only needs webamp tests to pass for the migration validation * Add missing Babel plugins for build-library - Add @babel/plugin-proposal-object-rest-spread - Add @babel/plugin-syntax-dynamic-import - These were missing dependencies revealed by pnpm's stricter isolation - Fixes build-library errors in CI * Upgrade Jest to v29.7.0 to fix test environment issues - Upgrade from Jest 27.5.1 to 29.7.0 to match webamp package version - Add jest-environment-jsdom as direct dependency - Fixes 'Cannot read properties of undefined (reading testEnvironmentOptions)' error - pnpm's stricter isolation revealed version conflicts between packages - Tests now run properly but some snapshots need updating due to format changes * Re-enable Jest tests in CI - Jest environment issues are now fixed with v29.7.0 upgrade - Tests work properly with the updated configuration - Some packages may have snapshot format changes but tests pass * Clean up lock * Clean up Yarn cruft * Update snapshots * Fix compressed size workflow for pnpm - Add Node.js setup step (required for pnpm) - Add pnpm install step to install dependencies before build - Update checkout action from v2 to v4 - Ensure dependencies are available before running deploy script
26 lines
654 B
YAML
26 lines
654 B
YAML
name: Compressed Size
|
|
|
|
on: [pull_request]
|
|
|
|
jobs:
|
|
build:
|
|
runs-on: ubuntu-latest
|
|
|
|
steps:
|
|
- uses: actions/checkout@v4
|
|
- name: Install pnpm
|
|
uses: pnpm/action-setup@v2
|
|
with:
|
|
version: 9.12.0
|
|
- name: Use Node.js 20.x
|
|
uses: actions/setup-node@v4
|
|
with:
|
|
node-version: 20.x
|
|
cache: "pnpm"
|
|
- name: Install Dependencies
|
|
run: pnpm install --frozen-lockfile
|
|
- uses: preactjs/compressed-size-action@v2
|
|
with:
|
|
repo-token: "${{ secrets.GITHUB_TOKEN }}"
|
|
build-script: "deploy"
|
|
pattern: "./packages/webamp/built/*bundle.min.js"
|