Skip to content

Harden package installs with pnpm and CI updates#1838

Open
mcgrew-oai wants to merge 6 commits intomasterfrom
dev/codex/package-manager-safety-dry-run-20260423145803
Open

Harden package installs with pnpm and CI updates#1838
mcgrew-oai wants to merge 6 commits intomasterfrom
dev/codex/package-manager-safety-dry-run-20260423145803

Conversation

@mcgrew-oai
Copy link
Copy Markdown

@mcgrew-oai mcgrew-oai commented Apr 23, 2026

Summary

This PR migrates the repo's primary package manager from Yarn to pnpm and updates install paths in CI and ecosystem tests to use committed, repo-scoped package-manager configuration.

Changes in this PR:

  • migrate the root repo and examples/ to pinned pnpm via Corepack
  • add pnpm-workspace.yaml and commit pnpm-lock.yaml
  • make ./scripts/bootstrap verify the pnpm hardening settings and always install with --frozen-lockfile
  • update scripts and GitHub Actions workflows to use corepack pnpm
  • keep npm ecosystem fixtures on npm, with per-project .npmrc config, but run them with repo-pinned npm@11.12.1 so the npm hardening keys are actually respected
  • add Bun install config and verify bun install --frozen-lockfile in CI
  • remove stale root Yarn and examples/ package-lock artifacts

Why

To reduce package supply chain risk by making installs use committed, repo-scoped package-manager configuration in both local development and CI.

This moves the primary repo workflow onto pinned pnpm, makes bootstrap and CI use the same hardened install path, and keeps explicit config in the npm and Bun compatibility fixtures instead of relying on default package-manager behavior.

Testing

Ran locally:

  • ./scripts/bootstrap
  • corepack pnpm build
  • corepack pnpm lint
  • env -u OPENAI_API_KEY ./scripts/test
  • bun install --frozen-lockfile in ecosystem-tests/bun
  • corepack pnpm tsn ecosystem-tests/cli.ts --retry=0 browser-direct-import ts-browser-webpack

@mcgrew-oai mcgrew-oai marked this pull request as ready for review April 23, 2026 22:03
@mcgrew-oai mcgrew-oai requested a review from a team as a code owner April 23, 2026 22:03
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7d40a03040

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/bootstrap Outdated
Comment thread ecosystem-tests/browser-direct-import/.npmrc Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant