Merge pull request #255 from launchql/feat/pnpm

Feat/pnpm

Author: pyramation

.github/workflows/run-tests.yaml

@@ -43,9 +43,9 @@ jobs:
           --health-interval 10s
           --health-timeout 5s
           --health-retries 5
-    
+
     steps:
-       # TODO remove deps on git config
+      # TODO remove deps on git config
       - name: Configure Git (for tests)
         run: |
           git config --global user.name "CI Test User"
@@ -54,36 +54,38 @@ jobs:
       - name: checkout
         uses: actions/checkout@v4
 
+      - name: setup pnpm
+        run: |
+          corepack enable
+          corepack prepare [email protected] --activate
+
       - name: deps
-        run: yarn
+        run: pnpm install --frozen-lockfile
 
       - name: build
-        run: yarn build
-
-      - name: fix lql permissions
-        run: chmod +x node_modules/.bin/lql
+        run: pnpm run build
 
       - name: seed app_user
         run: |
-          yarn lql admin-users bootstrap --yes
-          yarn lql admin-users add --test --yes
+          pnpm --filter @launchql/cli exec node dist/index.js admin-users bootstrap --yes
+          pnpm --filter @launchql/cli exec node dist/index.js 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,57 +95,57 @@ 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
           AWS_SECRET_KEY: minioadmin
           AWS_REGION: us-east-1
           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,8 @@
 **/node_modules/
 **/.DS_Store
 **/dist
-**/yarn-error.log
+**/pnpm-debug.log
 lerna-debug.log
 .vscode
-ignore
+ignore
+packages/pgsql-test/output/

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
 ```
 

Dockerfile

@@ -16,16 +16,16 @@ RUN set -eux; \
       ca-certificates curl git python3 make g++; \
     update-ca-certificates || true; \
     corepack enable; \
-    corepack prepare [email protected] --activate; \
+    corepack prepare [email protected] --activate; \
     rm -rf /var/lib/apt/lists/*
 
 # Copy full repo (build context must be repo root when building this image)
 COPY . .
 
 # Install and build all workspaces
 RUN set -eux; \
-    yarn install --frozen-lockfile; \
-    yarn build
+    pnpm install --frozen-lockfile; \
+    pnpm run build
 
 ################################################################################
 FROM ${BASE}:${BASE_VERSION} AS launchql
@@ -39,7 +39,7 @@ RUN set -eux; \
     apt-get install -y --no-install-recommends ca-certificates postgresql-client; \
     update-ca-certificates || true; \
     corepack enable; \
-    corepack prepare [email protected] --activate; \
+    corepack prepare [email protected] --activate; \
     rm -rf /var/lib/apt/lists/*
 
 # Copy built repo from builder

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