bump node-gyp to v10

[branchv]

Fixes #1752

[YasharF]

Hey there, I am curious. What kind of tests have run to verify the version bump is not creating other issues, regressions, etc. ?

[niklasf]

Can confirm this solves the issue, builds, and works at least for basic database operations. Tested with --target_arch arm --build-from-source.

[demiankatz]

I wonder if this might also fix #1762, since the deprecated dependency is coming in courtesy of node-gyp.

[gladykov]

This would also fix GHSA-2p57-rm9w-gvfp. All dependencies are fixed down the line.

Current dependency chain to affected package:

[netroy]

I compiled this branch, and then updated n8n to use file: versions of sqlite3, and all integration tests still passed.
Considering that n8n's backend tests do a lot of database calls, I don't think there is going to be any major regression caused by this upgrade.
The upgrade also cleaned up the dependency tree a little bit as a side-bonus 🎉 .

@daniellockyer Can we please have this merged 🙏🏽

[DamienCassou]

Is it possible to get this merged?

[DamienCassou]

@daniellockyer what is the reason for closing this PR? Has the issue been solved differently?

[daniellockyer]

Sorry no, the PR was automatically closed with a branch change 🫤 This PR is blocked because it would bump our Node requirement, which we can't do without a major

[branchv]

I see, would it be possible to release a new major version then? This impacts distributions which build packages from source (such as Homebrew), and sqlite3 is a common dependency of many node packages

[branchv]

Trying this again in #1834