diff --git a/node_modules/@npmcli/arborist/LICENSE b/node_modules/@npmcli/arborist/LICENSE deleted file mode 100644 index 13ff5b9eac5dc..0000000000000 --- a/node_modules/@npmcli/arborist/LICENSE +++ /dev/null @@ -1,22 +0,0 @@ -The ISC License - -Copyright npm, Inc. - -Permission to use, copy, modify, and/or distribute this software for any -purpose with or without fee is hereby granted, provided that the above -copyright notice and this permission notice appear in all copies. - -THE SOFTWARE IS PROVIDED "AS IS" AND NPM DISCLAIMS ALL WARRANTIES WITH -REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY -AND FITNESS. IN NO EVENT SHALL NPM BE LIABLE FOR ANY SPECIAL, DIRECT, -INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM -LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE -OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR -PERFORMANCE OF THIS SOFTWARE. - ---- - -Files and metadata contained in `test/fixtures/registry-mocks/content` are -downloaded from the public npm registry. These are the property of their -respective owners. The use and distribution of said artifacts are covered by -their associated licenses. diff --git a/node_modules/@npmcli/arborist/LICENSE.md b/node_modules/@npmcli/arborist/LICENSE.md new file mode 100644 index 0000000000000..5fc208ff122e0 --- /dev/null +++ b/node_modules/@npmcli/arborist/LICENSE.md @@ -0,0 +1,20 @@ + + +ISC License + +Copyright npm, Inc. + +Permission to use, copy, modify, and/or distribute this +software for any purpose with or without fee is hereby +granted, provided that the above copyright notice and this +permission notice appear in all copies. + +THE SOFTWARE IS PROVIDED "AS IS" AND NPM DISCLAIMS ALL +WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL +IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO +EVENT SHALL NPM BE LIABLE FOR ANY SPECIAL, DIRECT, +INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES +WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, +WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER +TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE +USE OR PERFORMANCE OF THIS SOFTWARE. diff --git a/node_modules/@npmcli/arborist/bin/prune.js b/node_modules/@npmcli/arborist/bin/prune.js index 4809a992388aa..e11858c209f25 100644 --- a/node_modules/@npmcli/arborist/bin/prune.js +++ b/node_modules/@npmcli/arborist/bin/prune.js @@ -6,7 +6,7 @@ require('./lib/logging.js') require('./lib/timers.js') const printDiff = diff => { - const {depth} = require('treeverse') + const { depth } = require('treeverse') depth({ tree: diff, visit: d => { diff --git a/node_modules/@npmcli/arborist/bin/reify.js b/node_modules/@npmcli/arborist/bin/reify.js index 803bac978c69a..9dc26d2b96835 100644 --- a/node_modules/@npmcli/arborist/bin/reify.js +++ b/node_modules/@npmcli/arborist/bin/reify.js @@ -6,7 +6,7 @@ require('./lib/logging.js') require('./lib/timers.js') const printDiff = diff => { - const {depth} = require('treeverse') + const { depth } = require('treeverse') depth({ tree: diff, visit: d => { diff --git a/node_modules/@npmcli/arborist/lib/add-rm-pkg-deps.js b/node_modules/@npmcli/arborist/lib/add-rm-pkg-deps.js index 3c1cbd44abcc9..ae813186fb712 100644 --- a/node_modules/@npmcli/arborist/lib/add-rm-pkg-deps.js +++ b/node_modules/@npmcli/arborist/lib/add-rm-pkg-deps.js @@ -2,9 +2,9 @@ const localeCompare = require('@isaacs/string-locale-compare')('en') -const add = ({pkg, add, saveBundle, saveType, log}) => { +const add = ({ pkg, add, saveBundle, saveType, log }) => { for (const spec of add) { - addSingle({pkg, spec, saveBundle, saveType, log}) + addSingle({ pkg, spec, saveBundle, saveType, log }) } return pkg @@ -20,7 +20,7 @@ const saveTypeMap = new Map([ ['peer', 'peerDependencies'], ]) -const addSingle = ({pkg, spec, saveBundle, saveType, log}) => { +const addSingle = ({ pkg, spec, saveBundle, saveType, log }) => { const { name, rawSpec } = spec // if the user does not give us a type, we infer which type(s) diff --git a/node_modules/@npmcli/arborist/lib/arborist/build-ideal-tree.js b/node_modules/@npmcli/arborist/lib/arborist/build-ideal-tree.js index 4071deac5a780..aa37acfe52d31 100644 --- a/node_modules/@npmcli/arborist/lib/arborist/build-ideal-tree.js +++ b/node_modules/@npmcli/arborist/lib/arborist/build-ideal-tree.js @@ -31,7 +31,7 @@ const Node = require('../node.js') const Link = require('../link.js') const addRmPkgDeps = require('../add-rm-pkg-deps.js') const optionalSet = require('../optional-set.js') -const {checkEngine, checkPlatform} = require('npm-install-checks') +const { checkEngine, checkPlatform } = require('npm-install-checks') const relpath = require('../relpath.js') @@ -311,7 +311,7 @@ module.exports = cls => class IdealTreeBuilder extends cls { ? Shrinkwrap.reset({ path: this.path, lockfileVersion: this.options.lockfileVersion, - }).then(meta => Object.assign(root, {meta})) + }).then(meta => Object.assign(root, { meta })) : this.loadVirtual({ root })) // if we don't have a lockfile to go from, then start with the @@ -492,7 +492,7 @@ module.exports = cls => class IdealTreeBuilder extends cls { // This returns a promise because we might not have the name yet, // and need to call pacote.manifest to find the name. - [_add] (tree, {add, saveType = null, saveBundle = false}) { + [_add] (tree, { add, saveType = null, saveBundle = false }) { // get the name for each of the specs in the list. // ie, doing `foo@bar` we just return foo // but if it's a url or git, we don't know the name until we @@ -936,7 +936,7 @@ This is a one-time fix-up, please be patient... } }) - tasks.push({edge, dep}) + tasks.push({ edge, dep }) } const placeDeps = tasks @@ -1271,7 +1271,7 @@ This is a one-time fix-up, please be patient... // we typically only install non-optional peers, but we have to // factor them into the peerSet so that we can avoid conflicts .filter(e => e.peer && !(e.valid && e.to)) - .sort(({name: a}, {name: b}) => localeCompare(a, b)) + .sort(({ name: a }, { name: b }) => localeCompare(a, b)) for (const edge of peerEdges) { // already placed this one, and we're happy with it. @@ -1280,7 +1280,7 @@ This is a one-time fix-up, please be patient... } const parentEdge = node.parent.edgesOut.get(edge.name) - const {isProjectRoot, isWorkspace} = node.parent.sourceReference + const { isProjectRoot, isWorkspace } = node.parent.sourceReference const isMine = isProjectRoot || isWorkspace const conflictOK = this[_force] || !isMine && !this[_strictPeerDeps] diff --git a/node_modules/@npmcli/arborist/lib/arborist/index.js b/node_modules/@npmcli/arborist/lib/arborist/index.js index 21d4ec5a36cf8..de625e9b7684f 100644 --- a/node_modules/@npmcli/arborist/lib/arborist/index.js +++ b/node_modules/@npmcli/arborist/lib/arborist/index.js @@ -26,9 +26,10 @@ // the base class, so that the overall voltron class is easier to test and // cover, and separation of concerns can be maintained. -const {resolve} = require('path') -const {homedir} = require('os') +const { resolve } = require('path') +const { homedir } = require('os') const procLog = require('proc-log') +const { depth } = require('treeverse') const { saveTypeMap } = require('../add-rm-pkg-deps.js') const mixins = [ @@ -88,6 +89,9 @@ class Arborist extends Base { process.emit('timeEnd', 'arborist:ctor') } + // TODO: We should change these to static functions instead + // of methods for the next major version + // returns an array of the actual nodes for all the workspaces workspaceNodes (tree, workspaces) { return getWorkspaceNodes(tree, workspaces, this.log) @@ -103,15 +107,15 @@ class Arborist extends Base { } } } - const set = new Set(wsNodes) + const wsDepSet = new Set(wsNodes) const extraneous = new Set() - for (const node of set) { + for (const node of wsDepSet) { for (const edge of node.edgesOut.values()) { const dep = edge.to if (dep) { - set.add(dep) + wsDepSet.add(dep) if (dep.isLink) { - set.add(dep.target) + wsDepSet.add(dep.target) } } } @@ -122,28 +126,36 @@ class Arborist extends Base { } } for (const extra of extraneous) { - set.add(extra) + wsDepSet.add(extra) } - return set + return wsDepSet } + // returns a set of root dependencies, excluding depdencies that are + // exclusively workspace dependencies excludeWorkspacesDependencySet (tree) { - const set = new Set() - for (const edge of tree.edgesOut.values()) { - if (edge.type !== 'workspace' && edge.to) { - set.add(edge.to) - } - } - for (const node of set) { - for (const edge of node.edgesOut.values()) { - if (edge.to) { - set.add(edge.to) + const rootDepSet = new Set() + depth({ + tree, + visit: node => { + for (const { to } of node.edgesOut.values()) { + if (!to || to.isWorkspace) { + continue + } + for (const edgeIn of to.edgesIn.values()) { + if (edgeIn.from.isRoot || rootDepSet.has(edgeIn.from)) { + rootDepSet.add(to) + } + } } - } - } - - return set + return node + }, + filter: node => node, + getChildren: (node, tree) => + [...tree.edgesOut.values()].map(edge => edge.to), + }) + return rootDepSet } } diff --git a/node_modules/@npmcli/arborist/lib/arborist/load-actual.js b/node_modules/@npmcli/arborist/lib/arborist/load-actual.js index 5d8fea85399a4..a232bf32b32d0 100644 --- a/node_modules/@npmcli/arborist/lib/arborist/load-actual.js +++ b/node_modules/@npmcli/arborist/lib/arborist/load-actual.js @@ -1,9 +1,9 @@ // mix-in implementing the loadActual method -const {relative, dirname, resolve, join, normalize} = require('path') +const { relative, dirname, resolve, join, normalize } = require('path') const rpj = require('read-package-json-fast') -const {promisify} = require('util') +const { promisify } = require('util') const readdir = promisify(require('readdir-scoped-modules')) const walkUp = require('walk-up-path') const ancestorPath = require('common-ancestor-path') @@ -128,7 +128,7 @@ module.exports = cls => class ActualLoader extends cls { pkg: {}, global, }) - return this[_loadActualActually]({root, ignoreMissing, global}) + return this[_loadActualActually]({ root, ignoreMissing, global }) } // not in global mode, hidden lockfile is allowed, load root pkg too @@ -163,7 +163,7 @@ module.exports = cls => class ActualLoader extends cls { // we can't easily get a ref to Arborist in this module, without // creating a circular reference, since this class is a mixin used // to build up the Arborist class itself. - await new this.constructor({...this.options}).loadVirtual({ + await new this.constructor({ ...this.options }).loadVirtual({ root: this[_actualTree], }) await this[_loadWorkspaces](this[_actualTree]) diff --git a/node_modules/@npmcli/arborist/lib/arborist/load-virtual.js b/node_modules/@npmcli/arborist/lib/arborist/load-virtual.js index ed0e47daf58ab..7761380e9f71f 100644 --- a/node_modules/@npmcli/arborist/lib/arborist/load-virtual.js +++ b/node_modules/@npmcli/arborist/lib/arborist/load-virtual.js @@ -1,7 +1,7 @@ // mixin providing the loadVirtual method const localeCompare = require('@isaacs/string-locale-compare')('en') -const {resolve} = require('path') +const { resolve } = require('path') const nameFromFolder = require('@npmcli/name-from-folder') const consistentResolve = require('../consistent-resolve.js') @@ -97,7 +97,7 @@ module.exports = cls => class VirtualLoader extends cls { this[checkRootEdges](s, root) root.meta = s this.virtualTree = root - const {links, nodes} = this[resolveNodes](s, root) + const { links, nodes } = this[resolveNodes](s, root) await this[resolveLinks](links, nodes) if (!(s.originalLockfileVersion >= 2)) { this[assignBundles](nodes) @@ -208,7 +208,7 @@ module.exports = cls => class VirtualLoader extends cls { nodes.set(location, this[loadNode](location, meta)) } } - return {links, nodes} + return { links, nodes } } // links is the set of metadata, and nodes is the map of non-Link nodes @@ -240,7 +240,7 @@ module.exports = cls => class VirtualLoader extends cls { if (!location || node.isLink && !node.target.location) { continue } - const { name, parent, package: { inBundle }} = node + const { name, parent, package: { inBundle } } = node if (!parent) { continue diff --git a/node_modules/@npmcli/arborist/lib/arborist/rebuild.js b/node_modules/@npmcli/arborist/lib/arborist/rebuild.js index 4f3fc1b9c3ba1..1a05e52735376 100644 --- a/node_modules/@npmcli/arborist/lib/arborist/rebuild.js +++ b/node_modules/@npmcli/arborist/lib/arborist/rebuild.js @@ -2,13 +2,13 @@ // bundle building needed. Called by reify, and by `npm rebuild`. const localeCompare = require('@isaacs/string-locale-compare')('en') -const {depth: dfwalk} = require('treeverse') +const { depth: dfwalk } = require('treeverse') const promiseAllRejectLate = require('promise-all-reject-late') const rpj = require('read-package-json-fast') const binLinks = require('bin-links') const runScript = require('@npmcli/run-script') const promiseCallLimit = require('promise-call-limit') -const {resolve} = require('path') +const { resolve } = require('path') const { isNodeGypPackage, defaultGypInstallScript, @@ -220,7 +220,7 @@ module.exports = cls => class Builder extends cls { } if (this[_oldMeta] === null) { - const {root: {meta}} = node + const { root: { meta } } = node this[_oldMeta] = meta && meta.loadedFromDisk && !(meta.originalLockfileVersion >= 2) } @@ -242,7 +242,7 @@ module.exports = cls => class Builder extends cls { const pkg = await rpj(node.path + '/package.json').catch(() => ({})) set.delete(node) - const {scripts = {}} = pkg + const { scripts = {} } = pkg node.package.scripts = scripts return this[_addToBuildSet](node, set, true) } @@ -319,9 +319,9 @@ module.exports = cls => class Builder extends cls { } const p = runScript(runOpts).catch(er => { const { code, signal } = er - this.log.info('run', pkg._id, event, {code, signal}) + this.log.info('run', pkg._id, event, { code, signal }) throw er - }).then(({args, code, signal, stdout, stderr}) => { + }).then(({ args, code, signal, stdout, stderr }) => { this.scriptsRun.add({ pkg, path, @@ -333,7 +333,7 @@ module.exports = cls => class Builder extends cls { stdout, stderr, }) - this.log.info('run', pkg._id, event, {code, signal}) + this.log.info('run', pkg._id, event, { code, signal }) }) await (this[_doHandleOptionalFailure] diff --git a/node_modules/@npmcli/arborist/lib/arborist/reify.js b/node_modules/@npmcli/arborist/lib/arborist/reify.js index 978743b38255f..547e54ac37670 100644 --- a/node_modules/@npmcli/arborist/lib/arborist/reify.js +++ b/node_modules/@npmcli/arborist/lib/arborist/reify.js @@ -3,15 +3,15 @@ const onExit = require('../signal-handling.js') const pacote = require('pacote') const AuditReport = require('../audit-report.js') -const {subset, intersects} = require('semver') +const { subset, intersects } = require('semver') const npa = require('npm-package-arg') const debug = require('../debug.js') const walkUp = require('walk-up-path') -const {dirname, resolve, relative} = require('path') -const {depth: dfwalk} = require('treeverse') +const { dirname, resolve, relative } = require('path') +const { depth: dfwalk } = require('treeverse') const fs = require('fs') -const {promisify} = require('util') +const { promisify } = require('util') const lstat = promisify(fs.lstat) const symlink = promisify(fs.symlink) const mkdirp = require('mkdirp-infer-owner') @@ -188,7 +188,7 @@ module.exports = cls => class Reifier extends cls { // ok, we're about to start touching the fs. need to roll back // if we get an early termination. let reifyTerminated = null - const removeHandler = onExit(({signal}) => { + const removeHandler = onExit(({ signal }) => { // only call once. if signal hits twice, we just terminate removeHandler() reifyTerminated = Object.assign(new Error('process terminated'), { @@ -352,7 +352,7 @@ module.exports = cls => class Reifier extends cls { if (includeRootDeps) { // add all non-workspace nodes to filterNodes for (const tree of [this.idealTree, this.actualTree]) { - for (const {type, to} of tree.edgesOut.values()) { + for (const { type, to } of tree.edgesOut.values()) { if (type !== 'workspace' && to) { filterNodes.push(to) } @@ -686,7 +686,7 @@ module.exports = cls => class Reifier extends cls { } [_warnDeprecated] (node) { - const {_id, deprecated} = node.package + const { _id, deprecated } = node.package if (deprecated) { this.log.warn('deprecated', `${_id}: ${deprecated}`) } @@ -1159,7 +1159,7 @@ module.exports = cls => class Reifier extends cls { const edge = addTree.edgesOut.get(name) const pkg = addTree.package const req = npa.resolve(name, edge.spec, addTree.realpath) - const {rawSpec, subSpec} = req + const { rawSpec, subSpec } = req const spec = subSpec ? subSpec.rawSpec : rawSpec const child = edge.to diff --git a/node_modules/@npmcli/arborist/lib/audit-report.js b/node_modules/@npmcli/arborist/lib/audit-report.js index de97cdc29e3a5..53fc5b17ade58 100644 --- a/node_modules/@npmcli/arborist/lib/audit-report.js +++ b/node_modules/@npmcli/arborist/lib/audit-report.js @@ -265,7 +265,7 @@ class AuditReport extends Map { avoid: vuln.range, avoidStrict: true, }) - return {name, version, isSemVerMajor} + return { name, version, isSemVerMajor } } catch (er) { return false } @@ -285,7 +285,7 @@ class AuditReport extends Map { } const bulk = {} - const {advisories} = report + const { advisories } = report for (const advisory of Object.values(advisories)) { const { id, @@ -296,7 +296,7 @@ class AuditReport extends Map { module_name: name, } = advisory bulk[name] = bulk[name] || [] - bulk[name].push({id, url, title, severity, vulnerable_versions}) + bulk[name].push({ id, url, title, severity, vulnerable_versions }) } return bulk diff --git a/node_modules/@npmcli/arborist/lib/calc-dep-flags.js b/node_modules/@npmcli/arborist/lib/calc-dep-flags.js index 95ecc8a617b08..2f33981419d0c 100644 --- a/node_modules/@npmcli/arborist/lib/calc-dep-flags.js +++ b/node_modules/@npmcli/arborist/lib/calc-dep-flags.js @@ -38,7 +38,7 @@ const calcDepFlagsStep = (node) => { return calcDepFlagsStep(node.target) } - node.edgesOut.forEach(({peer, optional, dev, to}) => { + node.edgesOut.forEach(({ peer, optional, dev, to }) => { // if the dep is missing, then its flags are already maximally unset if (!to) { return diff --git a/node_modules/@npmcli/arborist/lib/can-place-dep.js b/node_modules/@npmcli/arborist/lib/can-place-dep.js index 2278e25e3b0f3..1a3ccff669227 100644 --- a/node_modules/@npmcli/arborist/lib/can-place-dep.js +++ b/node_modules/@npmcli/arborist/lib/can-place-dep.js @@ -78,7 +78,7 @@ class CanPlaceDep { } this._treeSnapshot = JSON.stringify([...target.root.inventory.entries()] - .map(([loc, {packageName, version, resolved}]) => { + .map(([loc, { packageName, version, resolved }]) => { return [loc, packageName, version, resolved] }).sort(([a], [b]) => localeCompare(a, b))) }) @@ -118,7 +118,7 @@ class CanPlaceDep { debug(() => { const treeSnapshot = JSON.stringify([...target.root.inventory.entries()] - .map(([loc, {packageName, version, resolved}]) => { + .map(([loc, { packageName, version, resolved }]) => { return [loc, packageName, version, resolved] }).sort(([a], [b]) => localeCompare(a, b))) /* istanbul ignore if */ diff --git a/node_modules/@npmcli/arborist/lib/dep-valid.js b/node_modules/@npmcli/arborist/lib/dep-valid.js index d80437f20c8e4..2c837ae888448 100644 --- a/node_modules/@npmcli/arborist/lib/dep-valid.js +++ b/node_modules/@npmcli/arborist/lib/dep-valid.js @@ -6,7 +6,7 @@ const semver = require('semver') const npa = require('npm-package-arg') -const {relative} = require('path') +const { relative } = require('path') const fromPath = require('./from-path.js') const depValid = (child, requested, requestor) => { diff --git a/node_modules/@npmcli/arborist/lib/diff.js b/node_modules/@npmcli/arborist/lib/diff.js index 0d17bde9583ac..0387773c29754 100644 --- a/node_modules/@npmcli/arborist/lib/diff.js +++ b/node_modules/@npmcli/arborist/lib/diff.js @@ -5,13 +5,13 @@ // Thus, the root Diff node is the shallowest change required // for a given branch of the tree being mutated. -const {depth} = require('treeverse') -const {existsSync} = require('fs') +const { depth } = require('treeverse') +const { existsSync } = require('fs') const ssri = require('ssri') class Diff { - constructor ({actual, ideal, filterSet, shrinkwrapInflated}) { + constructor ({ actual, ideal, filterSet, shrinkwrapInflated }) { this.filterSet = filterSet this.shrinkwrapInflated = shrinkwrapInflated this.children = [] @@ -94,14 +94,14 @@ class Diff { } return depth({ - tree: new Diff({actual, ideal, filterSet, shrinkwrapInflated}), + tree: new Diff({ actual, ideal, filterSet, shrinkwrapInflated }), getChildren, leave, }) } } -const getAction = ({actual, ideal}) => { +const getAction = ({ actual, ideal }) => { if (!ideal) { return 'REMOVE' } @@ -237,7 +237,7 @@ const diffNode = ({ return } - const action = getAction({actual, ideal}) + const action = getAction({ actual, ideal }) // if it's a match, then get its children // otherwise, this is the child diff node @@ -245,7 +245,7 @@ const diffNode = ({ if (action === 'REMOVE') { removed.push(actual) } - children.push(new Diff({actual, ideal, filterSet, shrinkwrapInflated})) + children.push(new Diff({ actual, ideal, filterSet, shrinkwrapInflated })) } else { unchanged.push(ideal) // !*! Weird dirty hack warning !*! diff --git a/node_modules/@npmcli/arborist/lib/from-path.js b/node_modules/@npmcli/arborist/lib/from-path.js index 08be583d1a49d..2a3617844c001 100644 --- a/node_modules/@npmcli/arborist/lib/from-path.js +++ b/node_modules/@npmcli/arborist/lib/from-path.js @@ -3,7 +3,7 @@ // end up getting installed. directory (ie, symlink) deps also need // to be resolved based on their targets, but that's what realpath is -const {dirname} = require('path') +const { dirname } = require('path') const npa = require('npm-package-arg') const fromPath = (node, spec) => diff --git a/node_modules/@npmcli/arborist/lib/link.js b/node_modules/@npmcli/arborist/lib/link.js index 0289e04151ef5..dcce8c0d3dfa5 100644 --- a/node_modules/@npmcli/arborist/lib/link.js +++ b/node_modules/@npmcli/arborist/lib/link.js @@ -3,7 +3,7 @@ const relpath = require('./relpath.js') const Node = require('./node.js') const _loadDeps = Symbol.for('Arborist.Node._loadDeps') const _target = Symbol.for('_target') -const {dirname} = require('path') +const { dirname } = require('path') // defined by Node class const _delistFromMeta = Symbol.for('_delistFromMeta') const _refreshLocation = Symbol.for('_refreshLocation') diff --git a/node_modules/@npmcli/arborist/lib/node.js b/node_modules/@npmcli/arborist/lib/node.js index a872f24805b59..d311b6a837817 100644 --- a/node_modules/@npmcli/arborist/lib/node.js +++ b/node_modules/@npmcli/arborist/lib/node.js @@ -32,15 +32,15 @@ const semver = require('semver') const nameFromFolder = require('@npmcli/name-from-folder') const Edge = require('./edge.js') const Inventory = require('./inventory.js') -const {normalize} = require('read-package-json-fast') -const {getPaths: getBinPaths} = require('bin-links') +const { normalize } = require('read-package-json-fast') +const { getPaths: getBinPaths } = require('bin-links') const npa = require('npm-package-arg') const debug = require('./debug.js') const gatherDepSet = require('./gather-dep-set.js') const treeCheck = require('./tree-check.js') const walkUp = require('walk-up-path') -const {resolve, relative, dirname, basename} = require('path') +const { resolve, relative, dirname, basename } = require('path') const util = require('util') const _package = Symbol('_package') const _parent = Symbol('_parent') @@ -291,8 +291,8 @@ class Node { } get hasInstallScript () { - const {hasInstallScript, scripts} = this.package - const {install, preinstall, postinstall} = scripts || {} + const { hasInstallScript, scripts } = this.package + const { install, preinstall, postinstall } = scripts || {} return !!(hasInstallScript || install || preinstall || postinstall) } @@ -376,7 +376,7 @@ class Node { } if (this.root.sourceReference) { - const {name, version} = this.root.package + const { name, version } = this.root.package why.whileInstalling = { name, version, diff --git a/node_modules/@npmcli/arborist/lib/printable.js b/node_modules/@npmcli/arborist/lib/printable.js index ba2a25b195aee..018e569b1d2f1 100644 --- a/node_modules/@npmcli/arborist/lib/printable.js +++ b/node_modules/@npmcli/arborist/lib/printable.js @@ -87,7 +87,7 @@ class ArboristNode { // fsChildren sorted by path if (tree.fsChildren.size) { this.fsChildren = new Set([...tree.fsChildren] - .sort(({path: a}, {path: b}) => localeCompare(a, b)) + .sort(({ path: a }, { path: b }) => localeCompare(a, b)) .map(tree => printableTree(tree, path))) } @@ -114,7 +114,7 @@ class ArboristLink extends ArboristNode { } } -const treeError = ({code, path}) => ({ +const treeError = ({ code, path }) => ({ code, ...(path ? { path } : {}), }) diff --git a/node_modules/@npmcli/arborist/lib/relpath.js b/node_modules/@npmcli/arborist/lib/relpath.js index 33e12ff6f6ca8..e1960cfec02c7 100644 --- a/node_modules/@npmcli/arborist/lib/relpath.js +++ b/node_modules/@npmcli/arborist/lib/relpath.js @@ -1,3 +1,3 @@ -const {relative} = require('path') +const { relative } = require('path') const relpath = (from, to) => relative(from, to).replace(/\\/g, '/') module.exports = relpath diff --git a/node_modules/@npmcli/arborist/lib/retire-path.js b/node_modules/@npmcli/arborist/lib/retire-path.js index a6ce867d78940..5d583b151440d 100644 --- a/node_modules/@npmcli/arborist/lib/retire-path.js +++ b/node_modules/@npmcli/arborist/lib/retire-path.js @@ -1,5 +1,5 @@ const crypto = require('crypto') -const {dirname, basename, resolve} = require('path') +const { dirname, basename, resolve } = require('path') // use sha1 because it's faster, and collisions extremely unlikely anyway const pathSafeHash = s => diff --git a/node_modules/@npmcli/arborist/lib/shrinkwrap.js b/node_modules/@npmcli/arborist/lib/shrinkwrap.js index e7dd435ca34fe..2f0c0877cf8e0 100644 --- a/node_modules/@npmcli/arborist/lib/shrinkwrap.js +++ b/node_modules/@npmcli/arborist/lib/shrinkwrap.js @@ -35,7 +35,7 @@ const mismatch = (a, b) => a && b && a !== b const procLog = require('proc-log') const YarnLock = require('./yarn-lock.js') -const {promisify} = require('util') +const { promisify } = require('util') const rimraf = promisify(require('rimraf')) const fs = require('fs') const readFile = promisify(fs.readFile) @@ -180,7 +180,7 @@ const assertNoNewer = async (path, data, lockTime, dir = path, seen = null) => { const parent = isParent ? dir : resolve(dir, 'node_modules') const children = dir === path - ? Promise.resolve([{name: 'node_modules', isDirectory: () => true }]) + ? Promise.resolve([{ name: 'node_modules', isDirectory: () => true }]) : readdir(parent, { withFileTypes: true }) return children.catch(() => []) @@ -366,7 +366,7 @@ class Shrinkwrap { if (fromYarn && fromYarn.version) { // if it's the yarn or npm default registry, use the version as // our effective spec. if it's any other kind of thing, use that. - const {resolved, version, integrity} = fromYarn + const { resolved, version, integrity } = fromYarn const isYarnReg = spec.registry && yarnRegRe.test(resolved) const isnpmReg = spec.registry && !isYarnReg && npmRegRe.test(resolved) const isReg = isnpmReg || isYarnReg @@ -1062,7 +1062,7 @@ class Shrinkwrap { } // now we walk the children, putting them in the 'dependencies' object - const {children} = node.target + const { children } = node.target if (!children.size) { delete lock.dependencies } else { diff --git a/node_modules/@npmcli/arborist/lib/version-from-tgz.js b/node_modules/@npmcli/arborist/lib/version-from-tgz.js index d5d8f7345c9bb..cdb59b7d4dcad 100644 --- a/node_modules/@npmcli/arborist/lib/version-from-tgz.js +++ b/node_modules/@npmcli/arborist/lib/version-from-tgz.js @@ -1,7 +1,7 @@ /* eslint node/no-deprecated-api: "off" */ const semver = require('semver') -const {basename} = require('path') -const {parse} = require('url') +const { basename } = require('path') +const { parse } = require('url') module.exports = (name, tgz) => { const base = basename(tgz) if (!base.endsWith('.tgz')) { diff --git a/node_modules/@npmcli/arborist/lib/vuln.js b/node_modules/@npmcli/arborist/lib/vuln.js index a818cf318eb89..5b1483e183d5e 100644 --- a/node_modules/@npmcli/arborist/lib/vuln.js +++ b/node_modules/@npmcli/arborist/lib/vuln.js @@ -11,7 +11,7 @@ // @npmcli/metavuln-calculator // - via: dependency vulns which cause this one -const {satisfies, simplifyRange} = require('semver') +const { satisfies, simplifyRange } = require('semver') const semverOpt = { loose: true, includePrerelease: true } const localeCompare = require('@isaacs/string-locale-compare')('en') diff --git a/node_modules/@npmcli/arborist/lib/yarn-lock.js b/node_modules/@npmcli/arborist/lib/yarn-lock.js index 1eed0664083ac..5119a757a7e4b 100644 --- a/node_modules/@npmcli/arborist/lib/yarn-lock.js +++ b/node_modules/@npmcli/arborist/lib/yarn-lock.js @@ -30,8 +30,8 @@ const localeCompare = require('@isaacs/string-locale-compare')('en') const consistentResolve = require('./consistent-resolve.js') -const {dirname} = require('path') -const {breadth} = require('treeverse') +const { dirname } = require('path') +const { breadth } = require('treeverse') // sort a key/value object into a string of JSON stringified keys and vals const sortKV = obj => Object.keys(obj) diff --git a/node_modules/@npmcli/arborist/package.json b/node_modules/@npmcli/arborist/package.json index dfe2b6f08f56a..34d38572d38d7 100644 --- a/node_modules/@npmcli/arborist/package.json +++ b/node_modules/@npmcli/arborist/package.json @@ -1,15 +1,15 @@ { "name": "@npmcli/arborist", - "version": "4.0.4", + "version": "4.0.5", "description": "Manage node_modules trees", "dependencies": { - "@isaacs/string-locale-compare": "^1.0.1", + "@isaacs/string-locale-compare": "^1.1.0", "@npmcli/installed-package-contents": "^1.0.7", "@npmcli/map-workspaces": "^2.0.0", "@npmcli/metavuln-calculator": "^2.0.0", "@npmcli/move-file": "^1.1.0", "@npmcli/name-from-folder": "^1.0.1", - "@npmcli/node-gyp": "^1.0.1", + "@npmcli/node-gyp": "^1.0.3", "@npmcli/package-json": "^1.0.1", "@npmcli/run-script": "^2.0.0", "bin-links": "^2.3.0", @@ -23,7 +23,7 @@ "npm-package-arg": "^8.1.5", "npm-pick-manifest": "^6.1.0", "npm-registry-fetch": "^11.0.0", - "pacote": "^12.0.0", + "pacote": "^12.0.2", "parse-conflict-json": "^1.1.1", "proc-log": "^1.0.0", "promise-all-reject-late": "^1.0.0", @@ -37,11 +37,11 @@ "walk-up-path": "^1.0.0" }, "devDependencies": { - "@npmcli/lint": "^1.0.2", + "@npmcli/template-oss": "^2.3.0", "benchmark": "^2.1.4", "chalk": "^4.1.0", "minify-registry-metadata": "^2.1.0", - "tap": "^15.0.9", + "tap": "^15.1.2", "tcompare": "^5.0.6" }, "scripts": { @@ -54,21 +54,22 @@ "postversion": "npm publish", "prepublishOnly": "git push origin --follow-tags", "eslint": "eslint", - "lint": "npm run npmclilint -- \"lib/**/*.*js\" \"bin/**/*.*js\" \"test/*.*js\" \"test/arborist/*.*js\"", + "lint": "eslint '**/*.js'", "lintfix": "npm run lint -- --fix", "benchmark": "node scripts/benchmark.js", "benchclean": "rm -rf scripts/benchmark/*/", - "npmclilint": "npmcli-lint" + "npmclilint": "npmcli-lint", + "postlint": "npm-template-check" }, "repository": { "type": "git", "url": "https://github.com/npm/arborist" }, - "author": "Isaac Z. Schlueter (http://blog.izs.me/)", + "author": "GitHub Inc.", "license": "ISC", "files": [ - "lib/**/*.js", - "bin/**/*.js" + "bin", + "lib" ], "main": "lib/index.js", "bin": { @@ -91,5 +92,10 @@ }, "engines": { "node": "^12.13.0 || ^14.15.0 || >=16" - } + }, + "templateVersion": "2.3.0", + "eslintIgnore": [ + "test/fixtures/", + "!test/fixtures/*.js" + ] } diff --git a/node_modules/@npmcli/node-gyp/package.json b/node_modules/@npmcli/node-gyp/package.json index 37ecb87a2bdc0..299a453b66cd3 100644 --- a/node_modules/@npmcli/node-gyp/package.json +++ b/node_modules/@npmcli/node-gyp/package.json @@ -1,6 +1,6 @@ { "name": "@npmcli/node-gyp", - "version": "1.0.2", + "version": "1.0.3", "description": "Tools for dealing with node-gyp packages", "scripts": { "test": "tap", @@ -8,6 +8,10 @@ "postversion": "npm publish", "prepublishOnly": "git push origin --follow-tags" }, + "repository": { + "type": "git", + "url": "https://github.com/npm/node-gyp.git" + }, "keywords": [ "npm", "cli", diff --git a/package-lock.json b/package-lock.json index 31698c8e74b99..4aff6e2fe12c2 100644 --- a/package-lock.json +++ b/package-lock.json @@ -86,7 +86,7 @@ ], "dependencies": { "@isaacs/string-locale-compare": "^1.1.0", - "@npmcli/arborist": "^4.0.4", + "@npmcli/arborist": "^4.0.5", "@npmcli/ci-detect": "^1.4.0", "@npmcli/config": "^2.3.0", "@npmcli/map-workspaces": "^2.0.0", @@ -799,18 +799,18 @@ } }, "node_modules/@npmcli/arborist": { - "version": "4.0.4", - "resolved": "https://registry.npmjs.org/@npmcli/arborist/-/arborist-4.0.4.tgz", - "integrity": "sha512-5hRkiHF9zu62z6a7CJqhVG5CFUVnbYqvrrcxxEmhxFgyH2ovICyULOrj7nF4VBlfzp7OPu/rveV2ts9iYrn74g==", + "version": "4.0.5", + "resolved": "https://registry.npmjs.org/@npmcli/arborist/-/arborist-4.0.5.tgz", + "integrity": "sha512-WR2cqxzjsvmHJ9sKCdqBYG/qeiAXB9ev1iq1W2Rry7LxeJ7eDtTr4mOWe/TBvp6xFzevGecQc2YEWwExTuLZLg==", "inBundle": true, "dependencies": { - "@isaacs/string-locale-compare": "^1.0.1", + "@isaacs/string-locale-compare": "^1.1.0", "@npmcli/installed-package-contents": "^1.0.7", "@npmcli/map-workspaces": "^2.0.0", "@npmcli/metavuln-calculator": "^2.0.0", "@npmcli/move-file": "^1.1.0", "@npmcli/name-from-folder": "^1.0.1", - "@npmcli/node-gyp": "^1.0.1", + "@npmcli/node-gyp": "^1.0.3", "@npmcli/package-json": "^1.0.1", "@npmcli/run-script": "^2.0.0", "bin-links": "^2.3.0", @@ -824,7 +824,7 @@ "npm-package-arg": "^8.1.5", "npm-pick-manifest": "^6.1.0", "npm-registry-fetch": "^11.0.0", - "pacote": "^12.0.0", + "pacote": "^12.0.2", "parse-conflict-json": "^1.1.1", "proc-log": "^1.0.0", "promise-all-reject-late": "^1.0.0", @@ -988,9 +988,9 @@ "inBundle": true }, "node_modules/@npmcli/node-gyp": { - "version": "1.0.2", - "resolved": "https://registry.npmjs.org/@npmcli/node-gyp/-/node-gyp-1.0.2.tgz", - "integrity": "sha512-yrJUe6reVMpktcvagumoqD9r08fH1iRo01gn1u0zoCApa9lnZGEigVKUd2hzsCId4gdtkZZIVscLhNxMECKgRg==", + "version": "1.0.3", + "resolved": "https://registry.npmjs.org/@npmcli/node-gyp/-/node-gyp-1.0.3.tgz", + "integrity": "sha512-fnkhw+fmX65kiLqk6E3BFLXNC26rUhK90zVwe2yncPliVT/Qos3xjhTLE59Df8KnPlcwIERXKVlU1bXoUQ+liA==", "inBundle": true }, "node_modules/@npmcli/package-json": { @@ -10683,17 +10683,17 @@ "dev": true }, "@npmcli/arborist": { - "version": "4.0.4", - "resolved": "https://registry.npmjs.org/@npmcli/arborist/-/arborist-4.0.4.tgz", - "integrity": "sha512-5hRkiHF9zu62z6a7CJqhVG5CFUVnbYqvrrcxxEmhxFgyH2ovICyULOrj7nF4VBlfzp7OPu/rveV2ts9iYrn74g==", + "version": "4.0.5", + "resolved": "https://registry.npmjs.org/@npmcli/arborist/-/arborist-4.0.5.tgz", + "integrity": "sha512-WR2cqxzjsvmHJ9sKCdqBYG/qeiAXB9ev1iq1W2Rry7LxeJ7eDtTr4mOWe/TBvp6xFzevGecQc2YEWwExTuLZLg==", "requires": { - "@isaacs/string-locale-compare": "^1.0.1", + "@isaacs/string-locale-compare": "^1.1.0", "@npmcli/installed-package-contents": "^1.0.7", "@npmcli/map-workspaces": "^2.0.0", "@npmcli/metavuln-calculator": "^2.0.0", "@npmcli/move-file": "^1.1.0", "@npmcli/name-from-folder": "^1.0.1", - "@npmcli/node-gyp": "^1.0.1", + "@npmcli/node-gyp": "^1.0.3", "@npmcli/package-json": "^1.0.1", "@npmcli/run-script": "^2.0.0", "bin-links": "^2.3.0", @@ -10707,7 +10707,7 @@ "npm-package-arg": "^8.1.5", "npm-pick-manifest": "^6.1.0", "npm-registry-fetch": "^11.0.0", - "pacote": "^12.0.0", + "pacote": "^12.0.2", "parse-conflict-json": "^1.1.1", "proc-log": "^1.0.0", "promise-all-reject-late": "^1.0.0", @@ -10825,9 +10825,9 @@ "integrity": "sha512-qq3oEfcLFwNfEYOQ8HLimRGKlD8WSeGEdtUa7hmzpR8Sa7haL1KVQrvgO6wqMjhWFFVjgtrh1gIxDz+P8sjUaA==" }, "@npmcli/node-gyp": { - "version": "1.0.2", - "resolved": "https://registry.npmjs.org/@npmcli/node-gyp/-/node-gyp-1.0.2.tgz", - "integrity": "sha512-yrJUe6reVMpktcvagumoqD9r08fH1iRo01gn1u0zoCApa9lnZGEigVKUd2hzsCId4gdtkZZIVscLhNxMECKgRg==" + "version": "1.0.3", + "resolved": "https://registry.npmjs.org/@npmcli/node-gyp/-/node-gyp-1.0.3.tgz", + "integrity": "sha512-fnkhw+fmX65kiLqk6E3BFLXNC26rUhK90zVwe2yncPliVT/Qos3xjhTLE59Df8KnPlcwIERXKVlU1bXoUQ+liA==" }, "@npmcli/package-json": { "version": "1.0.1", diff --git a/package.json b/package.json index 3c5c3dae73bb7..a363015432975 100644 --- a/package.json +++ b/package.json @@ -55,7 +55,7 @@ }, "dependencies": { "@isaacs/string-locale-compare": "^1.1.0", - "@npmcli/arborist": "^4.0.4", + "@npmcli/arborist": "^4.0.5", "@npmcli/ci-detect": "^1.4.0", "@npmcli/config": "^2.3.0", "@npmcli/map-workspaces": "^2.0.0",