n8n: 1.9.3 -> 1.46.0, repackage with pnpm.fetchDeps

[gepbird]

Since #290715 was merged, it's much easier to package and maintain pnpm projects.

Description of changes

• replace all the node-env, node-composition stuff with pnpm.fetchDeps and pnpm.configHook
• upgrade n8n to latest version
• move to pkgs/by-name
• set N8N_RELEASE_TYPE to stable
• remove old patches and build inputs that seem no longer necessary
  • oracledb and @sap/hana-client removal on aarch64: these don't exist in node_modules anymore and aarch64 build works
  • pkgs.which: probably was unused before, there were no references to it
  • NIX_CFLAGS_COMPILE = "-DNODE_API_EXPERIMENTAL_NOGC_ENV_OPT_OUT";: builds without this with node 20
• remove update script: there's no update script for similarly packaged apps, and updating is fairly simple
• format with nixfmt-rfc-style

Issues I encountered and solved

Error: cannot find '/build/source/node_modules/.pnpm/[email protected]/node_modules/sqlite3/build/Release/node_sqlite3.node

pnpm.configHook calls pnpm install with --ignore-scripts that skips downloading/building node-sqlite3 bindings. A solution is building it manually: (use node-gyp from nixpkgs rather than from pnpm installed node_modules to avoid other errors)

https://github.com/NixOS/nixpkgs/blob/077adff3243b078a0f4a49421038d6f3f16fdf37/pkgs/by-name/n8/n8n/package.nix#L65-L67

turbo run build: Failed to create APIClient: Unable to set up TLS. builder error: No such file or directory

In turbo release 1.12.4, they switched to using rustls-tls-native-roots rust crate which tries to open a file from SSL_CERT_FILE environment variable, which is set to /no-cert-file.crt in a nix build environment. Adding cacert to build inputs will set this env var to a real file, fixing the issue.

ERR_PNPM_BAD_PM_VERSION This project is configured to use v9.1.4 of pnpm. Your current pnpm is v9.1.1.

Patching package.json to replace pnpm version to the current fixes it:

https://github.com/NixOS/nixpkgs/blob/077adff3243b078a0f4a49421038d6f3f16fdf37/pkgs/by-name/n8/n8n/package.nix#L35

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[natsukium]

Result of nixpkgs-review pr 319310 at 32950b65 run on x86_64-linux 1

1 package built successfully:

• n8n

---

Result of nixpkgs-review pr 319310 at 32950b65 run on aarch64-darwin 1

1 package built successfully:

• n8n

[natsukium]

It works fine on x86_64-linux.
LGTM, thanks!

[gepbird]

In the last few hours I updated this package from 1.45.1 to 1.46.0 and removed the pnpm version patch.

Tested it on x86_64 and aarch64 linux, works as before.

[natsukium]

LGTM, thanks for your work!

resolved