tools.func: use safe variable expansion in check_for_gh_release RETURN trap

2026-07-02 04:02:13 +02:00 · 2026-07-01 21:53:51 +02:00
1 changed files with 2 additions and 8 deletions
@@ -2647,9 +2647,9 @@ check_for_gh_release() {

  ensure_dependencies jq

-  local gh_check_json
+  local gh_check_json=""
  gh_check_json=$(mktemp /tmp/tools-gh-check-XXXXXX) || return 73
-  trap 'rm -f "$gh_check_json"' RETURN
+  trap 'rm -f "${gh_check_json:-}"' RETURN

  # Build auth header if token is available
  local header_args=()
@@ -7657,12 +7657,6 @@ setup_nodejs() {
    fi
  fi

-  # pnpm v10+ blocks dependency build scripts by default (ERR_PNPM_IGNORED_BUILDS).
-  # In a container environment all installed packages are trusted, so we enable builds globally.
-  if command -v pnpm >/dev/null 2>&1; then
-    pnpm config set --global dangerouslyAllowAllBuilds true >/dev/null 2>&1 || true
-  fi
-
  if [[ "$NODE_COREPACK_ENABLE" == "1" ]] && ((wants_corepack)) && command -v corepack >/dev/null 2>&1; then
    msg_info "Enabling corepack"
    if $STD corepack enable 2>/dev/null; then