fix: pass arborist constructor to manifest call

milaninfy · milaninfy · commit d79c6de1183d · 2024-10-04T10:42:18.000-04:00
diff --git a/lib/commands/pack.js b/lib/commands/pack.js
@@ -4,6 +4,7 @@ const npa = require('npm-package-arg')
 const { log, output } = require('proc-log')
 const { getContents, logTar } = require('../utils/tar.js')
 const BaseCommand = require('../base-cmd.js')
+const Arborist = require('@npmcli/arborist')
 
 class Pack extends BaseCommand {
   static description = 'Create a tarball from a package'
@@ -34,7 +35,7 @@ class Pack extends BaseCommand {
     const manifests = []
     for (const arg of args) {
       const spec = npa(arg)
-      const manifest = await pacote.manifest(spec, this.npm.flatOptions)
+      const manifest = await pacote.manifest(spec, { ...this.npm.flatOptions, Arborist })
       if (!manifest._id) {
         throw new Error('Invalid package, must have name and version')
       }
diff --git a/workspaces/libnpmexec/lib/index.js b/workspaces/libnpmexec/lib/index.js
@@ -24,7 +24,7 @@ const manifests = new Map()
 
 const getManifest = async (spec, flatOptions) => {
   if (!manifests.has(spec.raw)) {
-    const manifest = await pacote.manifest(spec, { ...flatOptions, preferOnline: true })
+    const manifest = await pacote.manifest(spec, { ...flatOptions, preferOnline: true, Arborist })
     manifests.set(spec.raw, manifest)
   }
   return manifests.get(spec.raw)
diff --git a/workspaces/libnpmpack/lib/index.js b/workspaces/libnpmpack/lib/index.js
@@ -12,7 +12,7 @@ async function pack (spec = 'file:.', opts = {}) {
   // gets spec
   spec = npa(spec)
 
-  const manifest = await pacote.manifest(spec, opts)
+  const manifest = await pacote.manifest(spec, { ...opts, Arborist })
 
   const stdio = opts.foregroundScripts ? 'inherit' : 'pipe'
 

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ const manifests = new Map()`
`24`	`24`
`25`	`25`	`const getManifest = async (spec, flatOptions) => {`
`26`	`26`	`if (!manifests.has(spec.raw)) {`
`27`		`- const manifest = await pacote.manifest(spec, { ...flatOptions, preferOnline: true })`
	`27`	`+ const manifest = await pacote.manifest(spec, { ...flatOptions, preferOnline: true, Arborist })`
`28`	`28`	`manifests.set(spec.raw, manifest)`
`29`	`29`	`}`
`30`	`30`	`return manifests.get(spec.raw)`