feat: do not alter file ownership

BREAKING CHANGE: this package no longer attempts to change file ownership automatically
npm · Oct 17, 2022 · 58065bc · 58065bc
1 parent 475e9b6
commit 58065bc
Show file tree

Hide file tree

Showing 5 changed files with 4 additions and 7 deletions.
diff --git a/DEPENDENCIES.md b/DEPENDENCIES.md
@@ -544,7 +544,6 @@ graph LR;
   libnpmexec-->bin-links;
   libnpmexec-->chalk;
   libnpmexec-->minify-registry-metadata;
-  libnpmexec-->mkdirp-infer-owner;
   libnpmexec-->mkdirp;
   libnpmexec-->npm-package-arg;
   libnpmexec-->npmcli-arborist["@npmcli/arborist"];

diff --git a/package-lock.json b/package-lock.json
@@ -13862,7 +13862,6 @@
         "@npmcli/fs": "^2.1.1",
         "@npmcli/run-script": "^4.2.0",
         "chalk": "^4.1.0",
-        "mkdirp-infer-owner": "^2.0.0",
         "npm-package-arg": "^9.0.1",
         "npmlog": "^6.0.2",
         "pacote": "^14.0.0",

diff --git a/workspaces/libnpmexec/lib/file-exists.js b/workspaces/libnpmexec/lib/file-exists.js
@@ -1,10 +1,10 @@
 const { resolve } = require('path')
-const fs = require('@npmcli/fs')
+const { stat } = require('fs/promises')
 const walkUp = require('walk-up-path')
 
 const fileExists = async (file) => {
   try {
-    const res = await fs.stat(file)
+    const res = await stat(file)
     return res.isFile()
   } catch {
     return false

diff --git a/workspaces/libnpmexec/lib/index.js b/workspaces/libnpmexec/lib/index.js
@@ -1,12 +1,12 @@
 'use strict'
 
+const { mkdir } = require('fs/promises')
 const { promisify } = require('util')
 
 const Arborist = require('@npmcli/arborist')
 const ciDetect = require('@npmcli/ci-detect')
 const crypto = require('crypto')
 const log = require('proc-log')
-const mkdirp = require('mkdirp-infer-owner')
 const npa = require('npm-package-arg')
 const npmlog = require('npmlog')
 const pacote = require('pacote')
@@ -205,7 +205,7 @@ const exec = async (opts) => {
       .digest('hex')
       .slice(0, 16)
     const installDir = resolve(npxCache, hash)
-    await mkdirp(installDir)
+    await mkdir(installDir, { recursive: true })
     const npxArb = new Arborist({
       ...flatOptions,
       path: installDir,

diff --git a/workspaces/libnpmexec/package.json b/workspaces/libnpmexec/package.json
@@ -63,7 +63,6 @@
     "@npmcli/fs": "^2.1.1",
     "@npmcli/run-script": "^4.2.0",
     "chalk": "^4.1.0",
-    "mkdirp-infer-owner": "^2.0.0",
     "npm-package-arg": "^9.0.1",
     "npmlog": "^6.0.2",
     "pacote": "^14.0.0",