webcord(-vencord): bump electron to v37

[bengsparks]

Tracking PR / issue: #458485.
I managed to build the package on darwin as well, but I'm not able to test it right now.

Things done

• Built on platform:
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• Tested, as applicable:
  • NixOS tests in nixos/tests.
  • Package tests at passthru.tests.
  • Tests in lib/tests or pkgs/test for functions and "core" functionality.
• Ran nixpkgs-review on this PR. See nixpkgs-review usage.
• Tested basic functionality of all binary files, usually in ./result/bin/.
• Nixpkgs Release Notes
  • Package update: when the change is major or breaking.
• NixOS Release Notes
  • Module addition: when adding a new NixOS module.
  • Module update: when the change is significant.
• Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

---

Add a 👍 reaction to pull requests you find important.

[bengsparks]

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 459739
Commit: 7603716186557469fd7757f72e2d5209ff0d7317

---

x86_64-linux

✅ 2 packages built:

• webcord
• webcord-vencord

---

aarch64-linux

✅ 2 packages built:

• webcord
• webcord-vencord

[NotAShelf]

Why was the update script removed?

[bengsparks]

Because it didnt work 🤷 I ran nix-shell maintainers/scripts/update.nix on the webcord package, and nothing was bumped. It is my understanding that the update scripts exist to bump their package, not their dependencies?

[huantianad]

That's to be expected; the update script will only do something if upstream releases a new version. When that happens, it will update the pinned electron version to whatever upstream sets it to.

[niklaskorz]

If upstream does not manage to leave a certain version of Electron behind before it reaches EOL (as has happened here), I agree the custom update script does more harm than good.

[huantianad]

It does no harm though? Like the normal upgrade script would've also done nothing...

[eclairevoyant]

I disagree with dropping the update script, the whole point was to make electron bumps automatic.
See 7030416

Since the electron version is directly in the package expression now, the file and regex just needs to be changed slightly.

sed -r "s|(electron_)[0-9]+|\1$electronVersion|" -i pkgs/by-name/we/webcord/package.nix

Please drop bengsparks@7603716 and apply the above.
The same update script would have to be copied into webcord-vencord's directory and the path adjusted of course.

And then we should update the two in-tandem.
(Actually I will probably just propose a new PR at this point.)

[bengsparks]

I'll drop the commit and apply the changes 👍

[eclairevoyant]

This ensures they're updated together

[eclairevoyant]

The same update script would have to be copied into webcord-vencord's directory and the path adjusted of course.

This was incorrect on my part, I forgot webcord-vencord was just an override of webcord. So we'll want to not-set any update script for webcord-vencord.
My change above should suffice instead.

[nixpkgs-ci]

Backport failed for release-25.05, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin release-25.05
git worktree add -d .worktree/backport-459739-to-release-25.05 origin/release-25.05
cd .worktree/backport-459739-to-release-25.05
git switch --create backport-459739-to-release-25.05
git cherry-pick -x 30e8e817989c7ae2b427dca7f978fff6db3b934e 9eec9c1c29d6a0f98a1de2a840bd880c163c857d