@npmcli/arborist 7.5.3 → 8.0.0

package/lib/arborist/build-ideal-tree.js

@@ -195,7 +195,10 @@ module.exports = cls => class IdealTreeBuilder extends cls {
     for (const node of this.idealTree.inventory.values()) {
       if (!node.optional) {
         try {
-          checkEngine(node.package, npmVersion, nodeVersion, this.options.force)
+          // if devEngines is present in the root node we ignore the engines check
+          if (!(node.isRoot && node.package.devEngines)) {
+            checkEngine(node.package, npmVersion, nodeVersion, this.options.force)
+          }
         } catch (err) {
           if (engineStrict) {
             throw err

package/lib/calc-dep-flags.js

@@ -31,6 +31,10 @@ const calcDepFlagsStep = (node) => {
 
   // for links, map their hierarchy appropriately
   if (node.isLink) {
+    // node.target can be null, we check to ensure it's not null before proceeding
+    if (node.target == null) {
+      return node
+    }
     node.target.dev = node.dev
     node.target.optional = node.optional
     node.target.devOptional = node.devOptional
@@ -97,15 +101,18 @@ const unsetFlag = (node, flag) => {
       tree: node,
       visit: node => {
         node.extraneous = node[flag] = false
-        if (node.isLink) {
+        if (node.isLink && node.target) {
           node.target.extraneous = node.target[flag] = false
         }
       },
       getChildren: node => {
         const children = []
-        for (const edge of node.target.edgesOut.values()) {
-          if (edge.to && edge.to[flag] &&
-            (flag !== 'peer' && edge.type === 'peer' || edge.type === 'prod')
+        const targetNode = node.isLink && node.target ? node.target : node
+        for (const edge of targetNode.edgesOut.values()) {
+          if (
+            edge.to &&
+            edge.to[flag] &&
+            ((flag !== 'peer' && edge.type === 'peer') || edge.type === 'prod')
           ) {
             children.push(edge.to)
           }

package/lib/shrinkwrap.js

@@ -50,6 +50,7 @@ const versionFromTgz = require('./version-from-tgz.js')
 const npa = require('npm-package-arg')
 const pkgJson = require('@npmcli/package-json')
 const parseJSON = require('parse-conflict-json')
+const nameFromFolder = require('@npmcli/name-from-folder')
 
 const stringify = require('json-stringify-nice')
 const swKeyOrder = [
@@ -233,7 +234,8 @@ class Shrinkwrap {
     // root to help prevent churn based on the name of the directory the
     // project is in
     const pname = node.packageName
-    if (pname && (node === node.root || pname !== node.name)) {
+    // when Target package name and Target node share the same name, we include the name, target node should have name as per realpath.
+    if (pname && (node === node.root || pname !== node.name || nameFromFolder(node.realpath) !== pname)) {
       meta.name = pname
     }
 

package/package.json

@@ -1,49 +1,49 @@
 {
   "name": "@npmcli/arborist",
-  "version": "7.5.3",
+  "version": "8.0.0",
   "description": "Manage node_modules trees",
   "dependencies": {
     "@isaacs/string-locale-compare": "^1.1.0",
-    "@npmcli/fs": "^3.1.1",
-    "@npmcli/installed-package-contents": "^2.1.0",
-    "@npmcli/map-workspaces": "^3.0.2",
-    "@npmcli/metavuln-calculator": "^7.1.1",
-    "@npmcli/name-from-folder": "^2.0.0",
-    "@npmcli/node-gyp": "^3.0.0",
-    "@npmcli/package-json": "^5.1.0",
-    "@npmcli/query": "^3.1.0",
-    "@npmcli/redact": "^2.0.0",
-    "@npmcli/run-script": "^8.1.0",
-    "bin-links": "^4.0.4",
-    "cacache": "^18.0.3",
+    "@npmcli/fs": "^4.0.0",
+    "@npmcli/installed-package-contents": "^3.0.0",
+    "@npmcli/map-workspaces": "^4.0.1",
+    "@npmcli/metavuln-calculator": "^8.0.0",
+    "@npmcli/name-from-folder": "^3.0.0",
+    "@npmcli/node-gyp": "^4.0.0",
+    "@npmcli/package-json": "^6.0.1",
+    "@npmcli/query": "^4.0.0",
+    "@npmcli/redact": "^3.0.0",
+    "@npmcli/run-script": "^9.0.1",
+    "bin-links": "^5.0.0",
+    "cacache": "^19.0.1",
     "common-ancestor-path": "^1.0.1",
-    "hosted-git-info": "^7.0.2",
-    "json-parse-even-better-errors": "^3.0.2",
+    "hosted-git-info": "^8.0.0",
+    "json-parse-even-better-errors": "^4.0.0",
     "json-stringify-nice": "^1.1.4",
     "lru-cache": "^10.2.2",
     "minimatch": "^9.0.4",
-    "nopt": "^7.2.1",
-    "npm-install-checks": "^6.2.0",
-    "npm-package-arg": "^11.0.2",
-    "npm-pick-manifest": "^9.0.1",
-    "npm-registry-fetch": "^17.0.1",
-    "pacote": "^18.0.6",
-    "parse-conflict-json": "^3.0.0",
-    "proc-log": "^4.2.0",
-    "proggy": "^2.0.0",
+    "nopt": "^8.0.0",
+    "npm-install-checks": "^7.1.0",
+    "npm-package-arg": "^12.0.0",
+    "npm-pick-manifest": "^10.0.0",
+    "npm-registry-fetch": "^18.0.1",
+    "pacote": "^19.0.0",
+    "parse-conflict-json": "^4.0.0",
+    "proc-log": "^5.0.0",
+    "proggy": "^3.0.0",
     "promise-all-reject-late": "^1.0.0",
     "promise-call-limit": "^3.0.1",
-    "read-package-json-fast": "^3.0.2",
+    "read-package-json-fast": "^4.0.0",
     "semver": "^7.3.7",
-    "ssri": "^10.0.6",
+    "ssri": "^12.0.0",
     "treeverse": "^3.0.0",
     "walk-up-path": "^3.0.1"
   },
   "devDependencies": {
-    "@npmcli/eslint-config": "^4.0.0",
-    "@npmcli/template-oss": "4.22.0",
+    "@npmcli/eslint-config": "^5.0.1",
+    "@npmcli/template-oss": "4.23.3",
     "benchmark": "^2.1.4",
-    "minify-registry-metadata": "^3.0.0",
+    "minify-registry-metadata": "^4.0.0",
     "nock": "^13.3.3",
     "tap": "^16.3.8",
     "tar-stream": "^3.0.0",
@@ -54,12 +54,13 @@
     "posttest": "npm run lint",
     "snap": "tap",
     "test-proxy": "ARBORIST_TEST_PROXY=1 tap --snapshot",
-    "lint": "eslint \"**/*.{js,cjs,ts,mjs,jsx,tsx}\"",
-    "lintfix": "npm run lint -- --fix",
+    "lint": "npm run eslint",
+    "lintfix": "npm run eslint -- --fix",
     "benchmark": "node scripts/benchmark.js",
     "benchclean": "rm -rf scripts/benchmark/*/",
     "postlint": "template-oss-check",
-    "template-oss-apply": "template-oss-apply --force"
+    "template-oss-apply": "template-oss-apply --force",
+    "eslint": "eslint \"**/*.{js,cjs,ts,mjs,jsx,tsx}\""
   },
   "repository": {
     "type": "git",
@@ -88,11 +89,11 @@
     ]
   },
   "engines": {
-    "node": "^16.14.0 || >=18.0.0"
+    "node": "^18.17.0 || >=20.5.0"
   },
   "templateOSS": {
     "//@npmcli/template-oss": "This file is partially managed by @npmcli/template-oss. Edits may be overwritten.",
-    "version": "4.22.0",
+    "version": "4.23.3",
     "content": "../../scripts/template-oss/index.js"
   }
 }