Merge pull request #253 from hyperweb-io/pnpm

yarn -> pnpm

Author: pyramation

.github/workflows/run-tests.yaml

@@ -54,36 +54,44 @@ jobs:
       - name: checkout
         uses: actions/checkout@v4
 
+      - name: setup pnpm
+        run: |
+          corepack enable
+          corepack prepare pnpm@9.12.1 --activate
+
       - name: deps
-        run: yarn
+        run: pnpm install --frozen-lockfile
 
       - name: build
-        run: yarn build
+        run: pnpm run build
 
       - name: fix lql permissions
-        run: chmod +x node_modules/.bin/lql
+        run: |
+          if [ -f node_modules/.bin/lql ]; then
+            chmod +x node_modules/.bin/lql
+          fi
 
       - name: seed app_user
         run: |
-          yarn lql admin-users bootstrap --yes
-          yarn lql admin-users add --test --yes
+          pnpm exec lql admin-users bootstrap --yes
+          pnpm exec lql admin-users add --test --yes
         env:
           PGHOST: pg_db
           PGPORT: 5432
           PGUSER: postgres
           PGPASSWORD: password
 
       - name: mime-bytes
-        run: cd ./packages/mime-bytes && yarn test
+        run: cd ./packages/mime-bytes && pnpm test
 
       - name: launchql/core
-        run: cd ./packages/core && yarn test
+        run: cd ./packages/core && pnpm test
 
       - name: launchql/cli
-        run: cd ./packages/cli && yarn test
+        run: cd ./packages/cli && pnpm test
 
       - name: launchql/client
-        run: cd ./packages/client && yarn test
+        run: cd ./packages/client && pnpm test
         env:
           TEST_DATABASE_URL: postgres://postgres:password@pg_db:5432/postgres
 
@@ -93,51 +101,51 @@ jobs:
           createdb graphile_test_db
           psql -f sql/test.sql graphile_test_db
           psql -f sql/roles.sql graphile_test_db
-          yarn test
+          pnpm test
 
       - name: pgsql-test
         run: |
           cd ./packages/pgsql-test
-          yarn test
+          pnpm test
 
       - name: launchql/orm
-        run: cd ./packages/orm && yarn test
+        run: cd ./packages/orm && pnpm test
 
       - name: launchql/url-domains
-        run: cd ./packages/url-domains && yarn test
+        run: cd ./packages/url-domains && pnpm test
 
       - name: launchql/uuid-hash
-        run: cd ./packages/uuid-hash && yarn test
+        run: cd ./packages/uuid-hash && pnpm test
 
       - name: launchql/uuid-stream
-        run: cd ./packages/uuid-stream && yarn test
+        run: cd ./packages/uuid-stream && pnpm test
 
       - name: launchql/introspectron
-        run: cd ./packages/introspectron && yarn test
+        run: cd ./packages/introspectron && pnpm test
 
       - name: launchql/query-builder
-        run: cd ./packages/query-builder && yarn test
+        run: cd ./packages/query-builder && pnpm test
 
       - name: launchql/query
-        run: cd ./packages/query && yarn test
+        run: cd ./packages/query && pnpm test
 
       - name: launchql/launchql-gen
-        run: cd ./packages/launchql-gen && yarn test
+        run: cd ./packages/launchql-gen && pnpm test
 
       - name: launchql/pg-ast
-        run: cd ./packages/pg-ast && yarn test
+        run: cd ./packages/pg-ast && pnpm test
 
       - name: launchql/pg-codegen
-        run: cd ./packages/pg-codegen && yarn test
+        run: cd ./packages/pg-codegen && pnpm test
 
       - name: launchql-gen
-        run: cd ./packages/launchql-gen && yarn test
+        run: cd ./packages/launchql-gen && pnpm test
 
       - name: launchql/content-type-stream
-        run: cd ./packages/content-type-stream && yarn test
+        run: cd ./packages/content-type-stream && pnpm test
 
       - name: launchql/s3-streamer
-        run: cd ./packages/s3-streamer && yarn test
+        run: cd ./packages/s3-streamer && pnpm test
         env:
           MINIO_ENDPOINT: http://minio_cdn:9000
           AWS_ACCESS_KEY: minioadmin
@@ -146,4 +154,4 @@ jobs:
           BUCKET_NAME: test-bucket
 
       - name: launchql/upload-names
-        run: cd ./packages/upload-names && yarn test
+        run: cd ./packages/upload-names && pnpm test

.gitignore

@@ -1,7 +1,7 @@
 **/node_modules/
 **/.DS_Store
 **/dist
-**/yarn-error.log
+**/pnpm-debug.log
 lerna-debug.log
 .vscode
 ignore

DEVELOPMENT.md

@@ -20,14 +20,14 @@ docker exec postgres /sql-bin/install.sh
 Then you can run 
 
 ```sh
-yarn
-yarn build
+pnpm install
+pnpm run build
 ```
 
 Then to run a test:
 
 ```sh
 cd packages/core
-yarn test
+pnpm test
 ```
 

GRAPHQL_ISSUES.md

@@ -44,7 +44,7 @@ The following packages now have GraphQL resolutions set to `15.5.2`:
 
 ## Why Resolutions Aren't Sufficient
 
-Yarn's `resolutions` field doesn't always override peer dependency requirements. When packages have conflicting peer dependency ranges, Node.js can still load multiple versions of GraphQL, causing the "duplicate modules" error.
+pnpm's `overrides` field does not always override peer dependency requirements. When packages have conflicting peer dependency ranges, Node.js can still load multiple versions of GraphQL, causing the "duplicate modules" error.
 
 ## Investigation Findings
 
@@ -63,7 +63,7 @@ Yarn's `resolutions` field doesn't always override peer dependency requirements.
 1. **Downgrade graphql-upload**: From `15.0.2` to `13.0.0` (supports GraphQL 15.5.2)
 2. **Handle @pyramation/postgis**: Add to overrides or find alternative package
 3. **Add overrides field**: Stronger enforcement than resolutions
-4. **Clean install**: Remove node_modules and yarn.lock, reinstall
+4. **Clean install**: Remove node_modules and pnpm-lock.yaml, then reinstall with `pnpm install`
 
 ### Option 2: Upgrade GraphQL (Higher Risk)
 - Upgrade to GraphQL 16.x to satisfy `graphql-upload`

Makefile

@@ -14,8 +14,8 @@ ssh:
 	docker exec -it postgres /bin/bash
 
 roles:
-	yarn lql admin-users bootstrap --yes
-	yarn lql admin-users add --test --yes
+	pnpm exec lql admin-users bootstrap --yes
+	pnpm exec lql admin-users add --test --yes
 
 install:
 	docker exec postgres /sql-bin/install.sh

TODO.md

@@ -36,7 +36,7 @@ await proj.deploy(
 - [ ] DB_CWD, db.cwd ???? in types/src
 
 
-- [ ] pnpm instead of yarn
+- [x] switch package management to pnpm
 - [ ] Add tests for pg-ast
 - [ ] bring back csv-to-pg
 - [ ] bootstrap-roles.sql and any other "bootstrap" can be included in actual code for onboarding

__fixtures__/kitchen-sink/.travis.yml

@@ -29,5 +29,5 @@ before_install:
   - while ! docker exec -it postgritty pg_isready -U postgres -h 127.0.0.1; do echo "$(date) - waiting for database to start"; sleep 1; done
 script:
   - cd $PACKAGE
-  - yarn install
-  - yarn test
+  - pnpm install
+  - pnpm test

__fixtures__/kitchen-sink/md.md

@@ -82,10 +82,10 @@ cd packages/myfirstmodule
 skitch maketest
 ```
 
-Then you can use `jest` via `yarn` to test your logic.
+Then you can use `jest` via `pnpm` to test your logic.
 
 ```sh
-yarn test:watch
+pnpm test:watch
 ```
 
 ## what's different
@@ -124,7 +124,7 @@ utility
 You can install an npm module and then bundle it for `plv8`
 
 ```sh
-yarn add my-awesome-npm-module
+pnpm add my-awesome-npm-module
 skitch bundle my-awesome-npm-module awesomeThing
 ```
 

__fixtures__/kitchen-sink/shellscript

@@ -8,7 +8,7 @@ fi
 
 
 
-find . -name yarn.lock | grep -v node_modules | xargs rm
+find . -name pnpm-lock.yaml | grep -v node_modules | xargs rm
 git add .
 git commit -am "$1"
 lerna publish

__fixtures__/sqitch/simple-w-tags/packages/my-first/package.json

@@ -23,8 +23,8 @@
     "copy": "copyfiles -f ../../LICENSE README.md package.json dist",
     "clean": "rimraf dist/**",
     "prepare": "npm run build",
-    "build": "npm run clean; tsc; tsc -p tsconfig.esm.json; npm run copy",
-    "build:dev": "npm run clean; tsc --declarationMap; tsc -p tsconfig.esm.json; npm run copy",
+    "build": "npm run clean; tsc -p tsconfig.json; tsc -p tsconfig.esm.json; npm run copy",
+    "build:dev": "npm run clean; tsc -p tsconfig.json --declarationMap; tsc -p tsconfig.esm.json; npm run copy",
     "lint": "eslint . --fix",
     "test": "jest",
     "test:watch": "jest --watch"