libnpmexec 5.0.1 → 5.0.3

package/lib/index.js

@@ -79,7 +79,6 @@ const exec = async (opts) => {
   const {
     args = [],
     call = '',
-    color = false,
     localBin = resolve('./node_modules/.bin'),
     locationMsg = undefined,
     globalBin = '',
@@ -97,7 +96,6 @@ const exec = async (opts) => {
   const run = () => runScript({
     args,
     call,
-    color,
     flatOptions,
     locationMsg,
     output,
@@ -114,20 +112,36 @@ const exec = async (opts) => {
 
   const needPackageCommandSwap = (args.length > 0) && (packages.length === 0)
   // If they asked for a command w/o specifying a package, see if there is a
-  // bin that directly matches that name either globally or in the local tree.
+  // bin that directly matches that name:
+  // - in the local package itself
+  // - in the local tree
+  // - globally
   if (needPackageCommandSwap) {
-    const dir = dirname(dirname(localBin))
-    const localBinPath = await localFileExists(dir, args[0], '/')
-    if (localBinPath) {
-      binPaths.push(localBinPath)
-      return await run()
-    } else if (globalPath && await fileExists(`${globalBin}/${args[0]}`)) {
-      binPaths.push(globalBin)
-      return await run()
+    let localManifest
+    try {
+      localManifest = await pacote.manifest(path, flatOptions)
+    } catch {
+      // no local package.json? no problem, move one.
+    }
+    if (localManifest?.bin?.[args[0]]) {
+      // we have to install the local package into the npx cache so that its
+      // bin links get set up
+      packages.push(path)
+      yes = true
+      flatOptions.installLinks = false
+    } else {
+      const dir = dirname(dirname(localBin))
+      const localBinPath = await localFileExists(dir, args[0], '/')
+      if (localBinPath) {
+        binPaths.push(localBinPath)
+        return await run()
+      } else if (globalPath && await fileExists(`${globalBin}/${args[0]}`)) {
+        binPaths.push(globalBin)
+        return await run()
+      }
+      // We swap out args[0] with the bin from the manifest later
+      packages.push(args[0])
     }
-
-    // We swap out args[0] with the bin from the manifest later
-    packages.push(args[0])
   }
 
   // Resolve any directory specs so that the npx directory is unique to the

package/lib/run-script.js

@@ -15,7 +15,6 @@ const nocolor = {
 const run = async ({
   args,
   call,
-  color,
   flatOptions,
   locationMsg,
   output = () => {},
@@ -26,6 +25,7 @@ const run = async ({
 }) => {
   // turn list of args into command string
   const script = call || args.shift() || scriptShell
+  const color = !!flatOptions.color
   const colorize = color ? chalk : nocolor
 
   // do the fakey runScript dance
@@ -68,7 +68,6 @@ const run = async ({
       banner: false,
       // we always run in cwd, not --prefix
       path: runPath,
-      stdioString: true,
       binPaths,
       event: 'npx',
       args,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "libnpmexec",
-  "version": "5.0.1",
+  "version": "5.0.3",
   "files": [
     "bin/",
     "lib/"
@@ -51,16 +51,16 @@
   },
   "devDependencies": {
     "@npmcli/eslint-config": "^4.0.0",
-    "@npmcli/template-oss": "4.6.2",
+    "@npmcli/template-oss": "4.9.0",
     "bin-links": "^4.0.1",
     "minify-registry-metadata": "^2.2.0",
     "mkdirp": "^1.0.4",
     "tap": "^16.0.1"
   },
   "dependencies": {
-    "@npmcli/arborist": "^6.1.0",
-    "@npmcli/ci-detect": "^3.0.0",
-    "@npmcli/run-script": "^5.0.0",
+    "@npmcli/arborist": "^6.1.2",
+    "@npmcli/ci-detect": "^3.0.1",
+    "@npmcli/run-script": "^6.0.0",
     "chalk": "^4.1.0",
     "npm-package-arg": "^10.0.0",
     "npmlog": "^7.0.1",
@@ -73,7 +73,7 @@
   },
   "templateOSS": {
     "//@npmcli/template-oss": "This file is partially managed by @npmcli/template-oss. Edits may be overwritten.",
-    "version": "4.6.2",
+    "version": "4.9.0",
     "content": "../../scripts/template-oss/index.js"
   }
 }