Fix pnpm-lock.yaml for sqlite-native dependencies#4570
Fix pnpm-lock.yaml for sqlite-native dependencies#4570NicholasKissel wants to merge 1 commit intomainfrom
Conversation
|
🚅 Deployed to the rivet-pr-4570 environment in rivet-frontend
|
PR ReviewThis is a lockfile-only change targeting a CI fix. A couple of observations: Description vs. Diff MismatchThe PR description says it "Regenerates
It would be worth clarifying which sqlite-native entries were missing and confirming the CI failure is actually fixed by this change rather than something else. Unintentional Version DowngradeThe
Both versions satisfy the SummaryLow risk change overall. The lockfile diff is minimal and correct from a semver standpoint. Recommend clarifying the description and confirming CI passes before merging. |
|
Superseded by #4571. The lockfile-only fix didn't work because the @rivetkit/sqlite-native-* packages haven't been published to npm yet. The real fix removes the unpublished optionalDependencies from package.json. |
Summary
pnpm-lock.yamlto include the 5 missing@rivetkit/sqlite-native-*platform dependencies added inrivetkit-typescript/packages/sqlite-native/package.jsonERR_PNPM_OUTDATED_LOCKFILEbuild failure in CITest plan
--frozen-lockfile