buildNpmPackage: support finalAttrs through lib.extendMkDerivation

[JohnRTitor]

Inspired from #382550.

Nixfmt is done on the 2nd commit to allow easier commit by commit review.

[drupol]

You might be interested in this pattern: #386757 (comment)

[ShamrockLee]

You might be interested in this pattern: #386757 (comment)

I prefer doing one thing at one time, unless the tasks are tightly coupled with each other. The overriding improvement using finalAttrs can be proceed in a subsequent PR.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/git-blame-ignore-revs-and-build-helper-format-changes/61726/1

[philiptaron]

I love it.

[ShamrockLee]

LGTM!

[drupol]

W000000t!

[philiptaron]

@JohnRTitor: could I requisition you to create the necessary backports to 24.11?

[JohnRTitor]

Ah, my dear sir, your request is received with the utmost regard, and I shall attend to the matter with diligence. Pray rest assured that by tomorrow evening, the necessary backports shall be prepared to your satisfaction.

[drupol]

Stop it guys, you crack me up every time I read these kind of comments.

[nixpkgs-ci]

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

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

git fetch origin release-24.11
git worktree add -d .worktree/backport-390099-to-release-24.11 origin/release-24.11
cd .worktree/backport-390099-to-release-24.11
git switch --create backport-390099-to-release-24.11
git cherry-pick -x 70ae16e6bff005ab5ab40090a0dae4b92fad6a57 e6ca2b65cae19e23191213efc04b6341d61947f6 43bf9e325cd1599d88740e4f52f476cf7afad366