Skip to content

fix(build): update Nix flake hash and package-lock.json - #571

Merged
zortos293 merged 1 commit into
devfrom
capy/fix-nix-flake-hash
Jun 30, 2026
Merged

fix(build): update Nix flake hash and package-lock.json#571
zortos293 merged 1 commit into
devfrom
capy/fix-nix-flake-hash

Conversation

@zortos293

@zortos293 zortos293 commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

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.json was missing resolved and integrity fields for over 480 packages, and the encoding dependency was entirely absent. buildNpmPackage requires this metadata to cache dependencies reproducibly.

Changes:

  • Updated npmDepsHash in flake.nix to match the corrected lockfile
  • Populated missing resolved and integrity fields in opennow-stable/package-lock.json
  • Added encoding@0.1.13 entry to satisfy node-fetch and minipass-fetch dependencies
  • Added preBuild hook in flake.nix to symlink locales directory for the build process
  • Updated ensure-electron-installed.mjs to respect ELECTRON_SKIP_BINARY_DOWNLOAD

Open OPE-239 OPE-239


Note

Low Risk
Build-only Nix packaging changes with no runtime application logic or security-sensitive behavior.

Overview
Fixes the Nix buildNpmPackage build by updating npmDepsHash in flake.nix so the fixed npm dependency set hashes correctly for reproducible caching.

Adds a preBuild hook that symlinks the repo-root locales directory to ../locales during the build. The app sources translations from that path (e.g. i18n.ts imports ../../../../locales/*.json), but the Nix src is only opennow-stable, so the symlink restores the layout the bundler expects.

Reviewed by Cursor Bugbot for commit 90c4361. Configure here.

…ld failures

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
@zortos293 zortos293 added the capy Generated by capy.ai label Jun 30, 2026 — with Image Capy AI
@cursor

cursor Bot commented Jun 30, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot 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)

@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 66a99eef-e409-434d-a3a4-131325e34c39

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch capy/fix-nix-flake-hash

Comment @coderabbitai help to get the list of available commands.

@zortos293
zortos293 merged commit b3c2230 into dev Jun 30, 2026
17 checks passed
Chizuui pushed a commit to Chizuui/OpenNOW-Modified that referenced this pull request Aug 11, 2026
…ld failures (OpenCloudGaming#571)

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

capy Generated by capy.ai

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant