libnpmexec 5.0.3 → 5.0.5

package/lib/index.js

@@ -4,7 +4,7 @@ const { mkdir } = require('fs/promises')
 const { promisify } = require('util')
 
 const Arborist = require('@npmcli/arborist')
-const ciDetect = require('@npmcli/ci-detect')
+const ciInfo = require('ci-info')
 const crypto = require('crypto')
 const log = require('proc-log')
 const npa = require('npm-package-arg')
@@ -110,7 +110,7 @@ const exec = async (opts) => {
     return run()
   }
 
-  const needPackageCommandSwap = (args.length > 0) && (packages.length === 0)
+  let 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:
   // - in the local package itself
@@ -126,9 +126,11 @@ const exec = async (opts) => {
     if (localManifest?.bin?.[args[0]]) {
       // we have to install the local package into the npx cache so that its
       // bin links get set up
+      flatOptions.installLinks = false
+      // args[0] will exist when the package is installed
       packages.push(path)
       yes = true
-      flatOptions.installLinks = false
+      needPackageCommandSwap = false
     } else {
       const dir = dirname(dirname(localBin))
       const localBinPath = await localFileExists(dir, args[0], '/')
@@ -242,7 +244,7 @@ const exec = async (opts) => {
           throw new Error('canceled')
         }
 
-        if (noTTY() || ciDetect()) {
+        if (noTTY() || ciInfo.isCI) {
           log.warn('exec', `The following package${
             add.length === 1 ? ' was' : 's were'
           } not found and will be installed: ${

package/lib/run-script.js

@@ -1,5 +1,5 @@
 const chalk = require('chalk')
-const ciDetect = require('@npmcli/ci-detect')
+const ciInfo = require('ci-info')
 const runScript = require('@npmcli/run-script')
 const readPackageJson = require('read-package-json-fast')
 const npmlog = require('npmlog')
@@ -44,10 +44,8 @@ const run = async ({
 
   try {
     if (script === scriptShell) {
-      const isTTY = !noTTY()
-
-      if (isTTY) {
-        if (ciDetect()) {
+      if (!noTTY()) {
+        if (ciInfo.isCI) {
           return log.warn('exec', 'Interactive mode disabled in CI environment')
         }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "libnpmexec",
-  "version": "5.0.3",
+  "version": "5.0.5",
   "files": [
     "bin/",
     "lib/"
@@ -51,17 +51,20 @@
   },
   "devDependencies": {
     "@npmcli/eslint-config": "^4.0.0",
-    "@npmcli/template-oss": "4.9.0",
+    "@npmcli/mock-registry": "^1.0.0",
+    "@npmcli/template-oss": "4.10.0",
     "bin-links": "^4.0.1",
+    "just-extend": "^6.1.1",
+    "just-safe-set": "^4.1.1",
     "minify-registry-metadata": "^2.2.0",
     "mkdirp": "^1.0.4",
     "tap": "^16.0.1"
   },
   "dependencies": {
-    "@npmcli/arborist": "^6.1.2",
-    "@npmcli/ci-detect": "^3.0.1",
+    "@npmcli/arborist": "^6.1.4",
     "@npmcli/run-script": "^6.0.0",
     "chalk": "^4.1.0",
+    "ci-info": "^3.6.1",
     "npm-package-arg": "^10.0.0",
     "npmlog": "^7.0.1",
     "pacote": "^15.0.2",
@@ -73,7 +76,7 @@
   },
   "templateOSS": {
     "//@npmcli/template-oss": "This file is partially managed by @npmcli/template-oss. Edits may be overwritten.",
-    "version": "4.9.0",
+    "version": "4.10.0",
     "content": "../../scripts/template-oss/index.js"
   }
 }