chore: switch from npm to pnpm (RetroAchievements#2794)

Author: wescopeland

.github/workflows/ci.yml

@@ -69,14 +69,19 @@ jobs:
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
+      - name: Install pnpm
+        uses: pnpm/action-setup@v4
+        with:
+          version: 9
       - name: Use Node 20
         uses: actions/setup-node@v4
         with:
           node-version: '20'
+          cache: 'pnpm'
       - name: Install
-        run: npm install
+        run: pnpm i
       - name: Type check
-        run: npm run tsc
+        run: pnpm tsc
 
   eslint:
     runs-on: ubuntu-22.04
@@ -86,14 +91,19 @@ jobs:
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
+      - name: Install pnpm
+        uses: pnpm/action-setup@v4
+        with:
+          version: 9
       - name: Use Node 20
         uses: actions/setup-node@v4
         with:
           node-version: '20'
+          cache: 'pnpm'
       - name: Install
-        run: npm install
+        run: pnpm i
       - name: Lint
-        run: npm run lint
+        run: pnpm lint
 
   vitest:
     runs-on: ubuntu-22.04
@@ -103,14 +113,19 @@ jobs:
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
+      - name: Install pnpm
+        uses: pnpm/action-setup@v4
+        with:
+          version: 9
       - name: Use Node 20
         uses: actions/setup-node@v4
         with:
           node-version: '20'
+          cache: 'pnpm'
       - name: Install
-        run: npm install
+        run: pnpm i
       - name: Test
-        run: npm run test
+        run: pnpm test
         env:
           VITE_BUILD_PATH: dist
           APP_URL: https://raweb.test

.gitignore

@@ -28,6 +28,7 @@
 /tmp
 /.fleet
 /.idea
+/.pnpm-store
 /.project
 /.settings
 /.vscode

.hooks/post-merge

@@ -3,7 +3,7 @@
 set -e
 
 php composer.phar install
-npm install
-npm run build
+pnpm install
+pnpm build
 
 printf "\n✅ post-merge OK\n\n"

.hooks/pre-push

@@ -8,14 +8,14 @@ vendor/bin/pint --test
 printf "\n⏳ composer analyse\n"
 vendor/bin/phpstan analyse --memory-limit 512M
 
-printf "\n⏳ npm run lint:eslint\n"
-npm run lint:eslint
+printf "\n⏳ pnpm lint:eslint\n"
+pnpm lint:eslint
 
-printf "\n⏳ npm run tsc\n"
-npm run tsc
+printf "\n⏳ pnpm tsc\n"
+pnpm tsc
 
-printf "\n⏳ npm run test\n"
-npm run test
+printf "\n⏳ pnpm test\n"
+pnpm test
 
 # Check the OS. Windows does not support the --parallel flag.
 if [ "$(uname)" = "Linux" ] || [ "$(uname)" = "Darwin" ]; then

README.md

@@ -10,6 +10,7 @@ It is a Laravel application ([TALL stack](https://tallstack.dev/)).
 - [Composer](https://getcomposer.org/) PHP dependency manager
 - [MariaDB 10](https://mariadb.com/docs/server/)
 - [Node.js 20](https://nodejs.org/)
+- [pnpm 9](https://pnpm.io/)
 
 Validated to run on Windows, macOS, and Linux with any of the setup options below (Docker via Laravel Sail, VM with either nginx or Apache, Laravel Valet on macOS).
 
@@ -131,11 +132,11 @@ AWS_MINIO=true
 ### Build frontend assets
 
 ```shell
-npm install
-npm run build
+pnpm install
+pnpm build
 # Using Sail:
-sail npm install
-sail npm run build
+sail pnpm install
+sail pnpm build
 ```
 
 ### Create filesystem symlinks

docs/CONTRIBUTING.md

@@ -36,7 +36,7 @@ Before submitting a pull request:
 
 - Follow **[PSR-12 Coding Standard](https://www.php-fig.org/psr/psr-12/)**. Run `composer fix` to fix most code style issues automatically.
 
-- Ensure ESLint rules are followed. Run `npm run fix` to fix most code style issues automatically.
+- Ensure ESLint rules are followed. Run `pnpm fix` to fix most code style issues automatically.
 
 - **Add tests!** - Your patch won't be accepted if it doesn't have tests.