Skip to content

fix(ci): sync pnpm-lock.yaml with package.json (unblock release) - #443

Merged
JohnMcLear merged 1 commit into
mainfrom
fix/lockfile-sync
Jun 7, 2026
Merged

fix(ci): sync pnpm-lock.yaml with package.json (unblock release)#443
JohnMcLear merged 1 commit into
mainfrom
fix/lockfile-sync

Conversation

@JohnMcLear

Copy link
Copy Markdown
Member

The release / publish-npm job on main is failing at pnpm i --frozen-lockfile with ERR_PNPM_OUTDATED_LOCKFILE: #263 (#428) removed the underscore dependency from package.json but the lockfile still listed it, so every push to main fails to publish.

Regenerated pnpm-lock.yaml to match package.json (drops underscore). Verified pnpm i --frozen-lockfile now exits 0. Tests are unaffected (this only touches the lockfile).

🤖 Generated with Claude Code

#263 (#428) removed the underscore dependency from package.json but left
it in pnpm-lock.yaml, so the release job's `pnpm i --frozen-lockfile`
fails with ERR_PNPM_OUTDATED_LOCKFILE — blocking npm publish on every
push to main. Regenerate the lockfile to match.
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Review Summary by Qodo

Sync pnpm-lock.yaml with package.json and fix CI publish

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Remove outdated underscore dependency from lockfile
• Sync pnpm-lock.yaml with package.json changes
• Add missing libc platform specifiers for Linux binaries
• Unblock npm publish CI job on main branch
Diagram
flowchart LR
  A["package.json<br/>removed underscore"] -->|"sync"| B["pnpm-lock.yaml<br/>regenerated"]
  B -->|"fixes"| C["CI release job<br/>pnpm i --frozen-lockfile"]
  C -->|"enables"| D["npm publish<br/>unblocked"]

Loading

Grey Divider

File Changes

1. pnpm-lock.yaml 🐞 Bug fix +6/-8

Remove underscore and sync lockfile

• Removed underscore@1.13.8 dependency entry and its snapshot
• Added missing libc platform specifiers for Linux-based rspack resolver bindings
• Regenerated lockfile to match current package.json state
• Resolves ERR_PNPM_OUTDATED_LOCKFILE error in CI pipeline

pnpm-lock.yaml


Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Jun 7, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@JohnMcLear
JohnMcLear merged commit b608d53 into main Jun 7, 2026
7 checks passed
@JohnMcLear
JohnMcLear deleted the fix/lockfile-sync branch June 7, 2026 07:39
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