From 3b39918c383ed2e9079e7f36bb6f075845a1a33b Mon Sep 17 00:00:00 2001 From: Patrick Mulligan Date: Fri, 9 Jan 2026 23:33:37 +0100 Subject: [PATCH] refactor(lamassu): remove redundant pnpm rebuild step MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit pnpm rebuild always fails due to signal handling issues and we ignore the failure anyway. Since we explicitly rebuild the problematic native modules (node-expat, iconv, argon2), the pnpm rebuild step is unnecessary. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- modules/lamassu-lnbits.nix | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/modules/lamassu-lnbits.nix b/modules/lamassu-lnbits.nix index 996841e..67795bc 100644 --- a/modules/lamassu-lnbits.nix +++ b/modules/lamassu-lnbits.nix @@ -317,11 +317,10 @@ in [ "$NEEDS_BUILD" = "0" ] && exit 0 - # Install and build + # Install dependencies (without running install scripts) pnpm install --no-frozen-lockfile --ignore-scripts - pnpm rebuild || true - # Rebuild problematic native modules + # Build native modules explicitly (pnpm rebuild has signal handling issues) for module in node-expat iconv; do path=$(find node_modules/.pnpm -name "$module" -type d -path "*/$module" 2>/dev/null | head -1) [ -n "$path" ] && [ -f "$path/binding.gyp" ] && (cd "$path" && npx node-gyp rebuild) || true