[dylanjeffers]

Summary

Fixes #14745: the desktop app (reported on the Linux AppImage, 1.5.180) crashes on launch with Cannot find module 'node-fetch'.

packages/web/src/electron.js requires node-fetch, but node-fetch is only a devDependency of @audius/web, so electron-builder leaves it out of app.asar. Electron 33 ships Node 20, which has fetch built in, so this drops the require.

Changes

• Remove require('node-fetch') and use the global fetch.
• The web-update download used data.body.pipe(), which only exists on node-fetch responses. It now uses pipeline(Readable.fromWeb(data.body), fs.createWriteStream(...)).
• If S3 returns a non-2xx, the download logs and skips the update instead of writing the error page to disk as the .tar.gz.

node-fetch stays in devDependencies even though nothing else in web uses it. Removing it changes package-lock.json, which triggers Mobile CI/CD and an automatic OTA on main. That can go in a separate cleanup.

Testing

• node --check and eslint pass on electron.js.
• Ran the download logic under the installed Electron 33.0.0 binary in Node mode (Node 20.18.0) against a local server:
  • fetch and Readable.fromWeb are available.
  • A 5 MB file streams to disk byte-for-byte.
  • A 404 reports ok=false.
  • .text() works for the package.json check.
• Not tested: a packaged AppImage build.

— 🏹 Legolas

🤖 Generated with Claude Code

[changeset-bot]

⚠️ No Changeset found

Latest commit: bfc06c3

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR