Skip to content

ci: validate pnpm override resolution - #3005

Open
lntutor wants to merge 4 commits into
Chainlit:mainfrom
lntutor:fix/pnpm-override-validation
Open

ci: validate pnpm override resolution#3005
lntutor wants to merge 4 commits into
Chainlit:mainfrom
lntutor:fix/pnpm-override-validation

Conversation

@lntutor

@lntutor lntutor commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary by cubic

Validates pnpm override resolution in CI to prevent lockfile drift. Previously we did not verify overrides; now CI fails if resolved versions don’t satisfy override targets, including parent>dependency selectors, with prerelease-aware matching.

  • Details

    • Adds scripts/check-pnpm-overrides.mjs and tests (scripts/check-pnpm-overrides.test.mjs); new scripts: check-pnpm-overrides and test:check-pnpm-overrides.
    • CI workflow adds matrix steps to run pnpm test:check-pnpm-overrides and pnpm check-pnpm-overrides.
    • Skips non-semver override targets; validates across root and workspace lockfiles.
  • Dependencies

    • Adds semver and yaml.

Written for commit 38a6487. Summary will update on new commits.

Review in cubic

Co-Authored-By: OpenAI Codex <noreply@openai.com>
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. dev-tooling Relating to developer/contributor toolings. labels Jul 31, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 5 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread scripts/check-pnpm-overrides.mjs Outdated
Comment thread scripts/check-pnpm-overrides.mjs Outdated
Comment thread package.json
Co-Authored-By: OpenAI Codex <noreply@openai.com>
@lntutor

lntutor commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

Current head bca01879de29ef7807ff5ba76b3a1ba457266032 is mergeable and fully green. The three review findings on this PR are resolved on this exact head, and there are no unresolved review threads remaining. This looks ready for maintainer review when convenient.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev-tooling Relating to developer/contributor toolings. size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants