Skip to content

[codex] Migrate tooling to pnpm #109

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
javorosas merged 1 commit into from
Jun 24, 2026
+3,160 −5,163
Merged

[codex] Migrate tooling to pnpm #109

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
111 changes: 51 additions & 60 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,25 +12,25 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v7

- name: Setup pnpm
uses: pnpm/action-setup@v6

- name: Setup Node
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: 22
cache: npm

- name: Pin npm version
run: npm i -g npm@10.9.2
node-version: 24
cache: pnpm

- name: Install dependencies
run: npm ci
run: pnpm install --frozen-lockfile

- name: Build package
run: npm run build
run: pnpm run build

- name: Upload dist artifact
uses: actions/upload-artifact@v4
uses: actions/upload-artifact@v7
with:
name: dist-artifact
path: dist
Expand All @@ -42,15 +42,15 @@ jobs:
needs: build-dist
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v7

- name: Setup Node
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: 18

- name: Download dist artifact
uses: actions/download-artifact@v4
uses: actions/download-artifact@v8
with:
name: dist-artifact
path: dist
Expand All @@ -60,100 +60,91 @@ jobs:

node-runtime:
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
node-version: [20, 22, 24]

steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v7

- name: Setup pnpm
uses: pnpm/action-setup@v6

- name: Setup Node
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: ${{ matrix.node-version }}
cache: npm

- name: Pin npm version
run: npm i -g npm@10.9.2
node-version: 24
cache: pnpm

- name: Install dependencies
run: npm ci
run: pnpm install --frozen-lockfile

- name: Run Node runtime tests
run: npm run test:node
run: pnpm run test:node

web-runtime:
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
node-version: [24]

steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v7

- name: Setup pnpm
uses: pnpm/action-setup@v6

- name: Setup Node
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: ${{ matrix.node-version }}
cache: npm

- name: Pin npm version
run: npm i -g npm@10.9.2
node-version: 24
cache: pnpm

- name: Install dependencies
run: npm ci
run: pnpm install --frozen-lockfile

- name: Run web-simulated runtime tests
run: npm run test:web
run: pnpm run test:web

build-and-types:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v7

- name: Setup pnpm
uses: pnpm/action-setup@v6

- name: Setup Node
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: 22
cache: npm

- name: Pin npm version
run: npm i -g npm@10.9.2
node-version: 24
cache: pnpm

- name: Install dependencies
run: npm ci
run: pnpm install --frozen-lockfile

- name: Build package
run: npm run build
run: pnpm run build

- name: Run type contract tests
run: npm run test:types
run: pnpm run test:types

browser-smoke:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v7

- name: Setup pnpm
uses: pnpm/action-setup@v6

- name: Setup Node
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: 22
cache: npm

- name: Pin npm version
run: npm i -g npm@10.9.2
node-version: 24
cache: pnpm

- name: Install dependencies
run: npm ci
run: pnpm install --frozen-lockfile

- name: Install Playwright browser
run: npx playwright install --with-deps chromium
run: pnpm exec playwright install --with-deps chromium

- name: Run browser smoke tests
run: npm run test:browser
run: pnpm run test:browser
14 changes: 9 additions & 5 deletions .github/workflows/deploy.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,19 +15,23 @@ jobs:

steps:
- name: Checkout repository
uses: actions/checkout@v4
uses: actions/checkout@v7

- name: Setup pnpm
uses: pnpm/action-setup@v6

- name: Set up Node.js
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: "24"
registry-url: "https://registry.npmjs.org"
cache: pnpm

- name: Install dependencies
run: npm ci
run: pnpm install --frozen-lockfile

- name: Build project
run: npm run build
run: pnpm run build

- name: Check if version exists
id: check-version
Expand All @@ -41,7 +45,7 @@ jobs:
- name: Publish to npm
id: publish_step
if: env.exists == 'false'
run: npm publish --provenance --access public
run: pnpm publish --provenance --access public

- name: Notify Slack (success)
if: env.exists == 'false' && steps.publish_step.outcome == 'success'
Expand Down
Loading