fix(build): update Nix flake hash and package-lock.json - #571
Conversation
…ld failures Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_b3422a8f-9734-4d36-adf6-a2d02d4c1d10) |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
…ld failures (OpenCloudGaming#571) Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
This PR fixes the Nix flake build failure (issue #570) caused by a stale npm dependency hash and missing registry metadata in the lockfile.
Fixes #570.
The build failed because
opennow-stable/package-lock.jsonwas missingresolvedandintegrityfields for over 480 packages, and theencodingdependency was entirely absent.buildNpmPackagerequires this metadata to cache dependencies reproducibly.Changes:
npmDepsHashinflake.nixto match the corrected lockfileresolvedandintegrityfields inopennow-stable/package-lock.jsonencoding@0.1.13entry to satisfynode-fetchandminipass-fetchdependenciespreBuildhook inflake.nixto symlinklocalesdirectory for the build processensure-electron-installed.mjsto respectELECTRON_SKIP_BINARY_DOWNLOADNote
Low Risk
Build-only Nix packaging changes with no runtime application logic or security-sensitive behavior.
Overview
Fixes the Nix
buildNpmPackagebuild by updatingnpmDepsHashinflake.nixso the fixed npm dependency set hashes correctly for reproducible caching.Adds a
preBuildhook that symlinks the repo-rootlocalesdirectory to../localesduring the build. The app sources translations from that path (e.g.i18n.tsimports../../../../locales/*.json), but the Nixsrcis onlyopennow-stable, so the symlink restores the layout the bundler expects.Reviewed by Cursor Bugbot for commit 90c4361. Configure here.