Breaks when using `npm:...` version syntax

[ursi]

This dependency would not get picked up for me.

"latest-snabbdom-release": "npm:snabbdom@2.1.0"

When I run node2nix -l package-lock.json on this package.json after generating package-lock.json using npm i

{
  "name": "test",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "latest-snabbdom-release": "npm:snabbdom@2.1.0"
  }
}

then run nix-build -A nodeDependencies, the result/lib/node_modules folder is not made. If other dependencies are added, the node_modules folder will be in the derivation, and contain everything except latest-snabbdom-release.

[datakurre]

For me, a patch like this, seems to work

+++ b/nix/node-env.nix
@@ -135,6 +135,9 @@ let
           function replaceDependencies(dependencies) {
               if(typeof dependencies == "object" && dependencies !== null) {
                   for(var dependency in dependencies) {
+                      if (dependencies[dependency].startsWith("npm:")) {
+                        dependency = dependencies[dependency].substring(4).split("@")[0];
+                      }
                       var resolvedVersion = resolveDependencyVersion(process.cwd(), dependency);

                       if(resolvedVersion === null) 

[svanderburg]

There are 2 PRs for introducing npm: aliases, but the problem is that they extend the functionality on the wrong level causing them also to be used as version specifiers.

The proper solution is to change the functionality that treats all package specifiers as dependency specifiers, and interpret the aliases there.

[towry]

2024 now, can not use @vue/language-service because it use npm:....