pacote 9.1.0 → 9.1.1

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+<a name="9.1.1"></a>
+## [9.1.1](https://github.com/zkat/pacote/compare/v9.1.0...v9.1.1) (2018-10-26)
+
+
+### Bug Fixes
+
+* **deps:** bump protoduck to remove CC0-1.0 license in dep ([3d9d9a6](https://github.com/zkat/pacote/commit/3d9d9a6))
+* **git:** Fix temp directory permissions for git fetcher ([#159](https://github.com/zkat/pacote/issues/159)) ([97c3aeb](https://github.com/zkat/pacote/commit/97c3aeb))
+* **packument:** group together all registry specs in silly log ([2333a17](https://github.com/zkat/pacote/commit/2333a17))
+* **standard:** standard --fix ([0ecb188](https://github.com/zkat/pacote/commit/0ecb188))
+
+
+
 <a name="9.1.0"></a>
 # [9.1.0](https://github.com/zkat/pacote/compare/v9.0.0...v9.1.0) (2018-08-15)
 

package/lib/fetch.js

@@ -8,7 +8,7 @@ const Fetcher = duck.define(['spec', 'opts', 'manifest'], {
   tarball: ['spec', 'opts'],
   fromManifest: ['manifest', 'spec', 'opts'],
   clearMemoized () {}
-}, {name: 'Fetcher'})
+}, { name: 'Fetcher' })
 module.exports = Fetcher
 
 module.exports.packument = packument

package/lib/fetchers/directory.js

@@ -34,11 +34,11 @@ Fetcher.impl(fetchDirectory, {
     const pkgPath = path.join(spec.fetchSpec, 'package.json')
     const srPath = path.join(spec.fetchSpec, 'npm-shrinkwrap.json')
     return BB.join(
-      readFileAsync(pkgPath).then(JSON.parse).catch({code: 'ENOENT'}, err => {
+      readFileAsync(pkgPath).then(JSON.parse).catch({ code: 'ENOENT' }, err => {
         err.code = 'ENOPACKAGEJSON'
         throw err
       }),
-      readFileAsync(srPath).then(JSON.parse).catch({code: 'ENOENT'}, () => null),
+      readFileAsync(srPath).then(JSON.parse).catch({ code: 'ENOENT' }, () => null),
       (pkg, sr) => {
         pkg._shrinkwrap = sr
         pkg._hasShrinkwrap = !!sr
@@ -50,7 +50,7 @@ Fetcher.impl(fetchDirectory, {
     ).then(pkg => {
       if (!pkg.bin && pkg.directories && pkg.directories.bin) {
         const dirBin = pkg.directories.bin
-        return glob(path.join(spec.fetchSpec, dirBin, '/**'), {nodir: true}).then(matches => {
+        return glob(path.join(spec.fetchSpec, dirBin, '/**'), { nodir: true }).then(matches => {
           matches.forEach(filePath => {
             const relative = path.relative(spec.fetchSpec, filePath)
             if (relative && relative[0] !== '.') {

package/lib/fetchers/file.js

@@ -43,9 +43,9 @@ Fetcher.impl(fetchFile, {
               opts.cache, `pacote:tarball:file:${src}`, data, {
                 integrity: opts.integrity
               }
-            ).then(integrity => ({data, integrity}))
+            ).then(integrity => ({ data, integrity }))
           } else {
-            return {data}
+            return { data }
           }
         }).then(info => {
           if (info.integrity) { stream.emit('integrity', info.integrity) }

package/lib/fetchers/git.js

@@ -157,7 +157,7 @@ function resolve (url, spec, name, opts) {
 function withTmp (opts, cb) {
   if (opts.cache) {
     // cacache has a special facility for working in a tmp dir
-    return cacache.tmp.withTmp(opts.cache, {tmpPrefix: 'git-clone'}, cb)
+    return cacache.tmp.withTmp(opts.cache, { tmpPrefix: 'git-clone', uid: opts.uid, gid: opts.gid }, cb)
   } else {
     const tmpDir = path.join(osenv.tmpdir(), 'pacote-git-tmp')
     const tmpName = uniqueFilename(tmpDir, 'git-clone')

package/lib/fetchers/registry/tarball.js

@@ -48,7 +48,7 @@ function tarball (spec, opts) {
 module.exports.fromManifest = fromManifest
 function fromManifest (manifest, spec, opts) {
   opts = optCheck(opts)
-  if (spec.scope) { opts = opts.concat({scope: spec.scope}) }
+  if (spec.scope) { opts = opts.concat({ scope: spec.scope }) }
   const stream = new PassThrough()
   const registry = fetch.pickRegistry(spec, opts)
   const uri = getTarballUrl(spec, registry, manifest, opts)

package/lib/finalize-manifest.js

@@ -156,7 +156,7 @@ function tarballedProps (pkg, spec, opts) {
       needsShrinkwrap && jsonFromStream('npm-shrinkwrap.json', extracted),
       needsManifest && jsonFromStream('package.json', extracted),
       needsBin && getPaths(extracted),
-      needsHash && ssri.fromStream(tarStream, {algorithms: ['sha1', 'sha512']}),
+      needsHash && ssri.fromStream(tarStream, { algorithms: ['sha1', 'sha512'] }),
       needsExtract && pipe(tarStream, extracted),
       (sr, mani, paths, hash) => {
         if (needsManifest && !mani) {

package/lib/util/git.js

@@ -74,10 +74,10 @@ function fullClone (repo, committish, target, opts) {
   if (process.platform === 'win32') {
     gitArgs.push('--config', 'core.longpaths=true')
   }
-  return execGit(gitArgs, {cwd: target}).then(() => {
-    return execGit(['init'], {cwd: target})
+  return execGit(gitArgs, { cwd: target }).then(() => {
+    return execGit(['init'], { cwd: target })
   }).then(() => {
-    return execGit(['checkout', committish || 'HEAD'], {cwd: target})
+    return execGit(['checkout', committish || 'HEAD'], { cwd: target })
   }).then(() => {
     return updateSubmodules(target, opts)
   }).then(() => headSha(target, opts))
@@ -110,7 +110,7 @@ function updateSubmodules (localRepo, opts) {
 
 function headSha (repo, opts) {
   opts = optCheck(opts)
-  return execGit(['rev-parse', '--revs-only', 'HEAD'], {cwd: repo}, opts).spread(stdout => {
+  return execGit(['rev-parse', '--revs-only', 'HEAD'], { cwd: repo }, opts).spread(stdout => {
     return stdout.trim()
   })
 }
@@ -139,7 +139,7 @@ function revs (repo, opts) {
         if (!ref) { return revs } // ???
         if (ref.endsWith(CARET_BRACES)) { return revs } // refs/tags/x^{} crap
         const type = refType(line)
-        const doc = {sha, ref, type}
+        const doc = { sha, ref, type }
 
         revs.refs[ref] = doc
         // We can check out shallow clones on specific SHAs if we have a ref
@@ -157,7 +157,7 @@ function revs (repo, opts) {
         }
 
         return revs
-      }, {versions: {}, 'dist-tags': {}, refs: {}, shas: {}})
+      }, { versions: {}, 'dist-tags': {}, refs: {}, shas: {} })
     }, err => {
       err.message = `Error while executing:\n${GITPATH} ls-remote -h -t ${repo}\n\n${err.stderr}\n${err.message}`
       throw err

package/lib/util/opt-check.js

@@ -12,28 +12,28 @@ module.exports = figgyPudding({
   dirPacker: {},
   dmode: {},
   fmode: {},
-  'fetch-retries': {default: 2},
-  'fetch-retry-factor': {default: 10},
-  'fetch-retry-maxtimeout': {default: 60000},
-  'fetch-retry-mintimeout': {default: 10000},
+  'fetch-retries': { default: 2 },
+  'fetch-retry-factor': { default: 10 },
+  'fetch-retry-maxtimeout': { default: 60000 },
+  'fetch-retry-mintimeout': { default: 10000 },
   fullMetadata: 'full-metadata',
-  'full-metadata': {default: false},
+  'full-metadata': { default: false },
   gid: {},
-  includeDeprecated: {default: true},
+  includeDeprecated: { default: true },
   'include-deprecated': 'includeDeprecated',
   integrity: {},
-  log: {default: logger},
+  log: { default: logger },
   memoize: {},
   offline: {},
   preferOffline: 'prefer-offline',
   'prefer-offline': {},
   preferOnline: 'prefer-online',
   'prefer-online': {},
-  registry: {default: 'https://registry.npmjs.org/'},
+  registry: { default: 'https://registry.npmjs.org/' },
   resolved: {},
   retry: {},
   scope: {},
-  tag: {default: 'latest'},
+  tag: { default: 'latest' },
   uid: {},
   umask: {},
   where: {}

package/lib/util/pack-dir.js

@@ -33,7 +33,7 @@ function packDir (manifest, label, dir, target, opts) {
 }
 
 function mkPacker (dir) {
-  return packlist({path: dir}).then(files => {
+  return packlist({ path: dir }).then(files => {
     return tar.c({
       cwd: dir,
       gzip: true,

package/lib/with-tarball-stream.js

@@ -36,7 +36,7 @@ function withTarballStream (spec, opts, streamHandler) {
       const file = path.resolve(opts.where || '.', opts.resolved.substr(5))
       return statAsync(file)
         .then(() => {
-          const verifier = ssri.integrityStream({integrity: opts.integrity})
+          const verifier = ssri.integrityStream({ integrity: opts.integrity })
           const stream = fs.createReadStream(file)
             .on('error', err => verifier.emit('error', err))
             .pipe(verifier)
@@ -50,7 +50,7 @@ function withTarballStream (spec, opts, streamHandler) {
           throw err
         })
     })
-    : BB.reject(Object.assign(new Error('no file!'), {code: 'ENOENT'}))
+    : BB.reject(Object.assign(new Error('no file!'), { code: 'ENOENT' }))
 
   const tryDigest = tryFile
     .catch(err => {
@@ -96,10 +96,10 @@ function withTarballStream (spec, opts, streamHandler) {
           const tardata = fetch.tarball(spec, opts)
           if (!opts.resolved) {
             tardata.on('manifest', m => {
-              opts = opts.concat({resolved: m._resolved})
+              opts = opts.concat({ resolved: m._resolved })
             })
             tardata.on('integrity', i => {
-              opts = opts.concat({integrity: i})
+              opts = opts.concat({ integrity: i })
             })
           }
           return BB.try(() => streamHandler(tardata))
@@ -117,7 +117,7 @@ function withTarballStream (spec, opts, streamHandler) {
                 throw err
               }
             })
-        }, {retries: 1}))
+        }, { retries: 1 }))
       }
     })
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pacote",
-  "version": "9.1.0",
+  "version": "9.1.1",
   "description": "JavaScript package downloader",
   "main": "index.js",
   "files": [
@@ -41,44 +41,44 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "bluebird": "^3.5.1",
-    "cacache": "^11.0.2",
-    "figgy-pudding": "^3.2.1",
-    "get-stream": "^3.0.0",
-    "glob": "^7.1.2",
+    "bluebird": "^3.5.2",
+    "cacache": "^11.2.0",
+    "figgy-pudding": "^3.5.1",
+    "get-stream": "^4.1.0",
+    "glob": "^7.1.3",
     "lru-cache": "^4.1.3",
     "make-fetch-happen": "^4.0.1",
     "minimatch": "^3.0.4",
-    "minipass": "^2.3.3",
+    "minipass": "^2.3.5",
     "mississippi": "^3.0.0",
     "mkdirp": "^0.5.1",
     "normalize-package-data": "^2.4.0",
     "npm-package-arg": "^6.1.0",
-    "npm-packlist": "^1.1.10",
+    "npm-packlist": "^1.1.12",
     "npm-pick-manifest": "^2.1.0",
-    "npm-registry-fetch": "^3.0.0",
+    "npm-registry-fetch": "^3.8.0",
     "osenv": "^0.1.5",
     "promise-inflight": "^1.0.1",
     "promise-retry": "^1.1.1",
-    "protoduck": "^5.0.0",
+    "protoduck": "^5.0.1",
     "rimraf": "^2.6.2",
     "safe-buffer": "^5.1.2",
-    "semver": "^5.5.0",
-    "ssri": "^6.0.0",
-    "tar": "^4.4.3",
-    "unique-filename": "^1.1.0",
-    "which": "^1.3.0"
+    "semver": "^5.6.0",
+    "ssri": "^6.0.1",
+    "tar": "^4.4.6",
+    "unique-filename": "^1.1.1",
+    "which": "^1.3.1"
   },
   "devDependencies": {
-    "nock": "^9.2.6",
+    "nock": "^10.0.1",
     "npmlog": "^4.1.2",
-    "nyc": "^11.8.0",
+    "nyc": "^13.1.0",
     "require-inject": "^1.4.3",
-    "standard": "^11.0.1",
+    "standard": "^12.0.1",
     "standard-version": "^4.4.0",
-    "tacks": "^1.2.6",
+    "tacks": "^1.2.7",
     "tap": "^12.0.1",
-    "tar-stream": "^1.6.1",
+    "tar-stream": "^1.6.2",
     "weallbehave": "^1.2.0",
     "weallcontribute": "^1.0.7"
   }

package/packument.js

@@ -23,7 +23,7 @@ function packument (spec, opts) {
     return fetchPackument(spec, opts)
   }).then(p => {
     const elapsedTime = Date.now() - startTime
-    opts.log.silly('pacote', `${spec.type} packument for ${spec.name}@${spec.saveSpec || spec.fetchSpec} fetched in ${elapsedTime}ms`)
+    opts.log.silly('pacote', `${spec.registry ? 'registry' : spec.type} packument for ${spec.name}@${spec.saveSpec || spec.fetchSpec} fetched in ${elapsedTime}ms`)
     return p
   })
 }

package/prefetch.js

@@ -15,7 +15,7 @@ function prefetch (spec, opts) {
   const startTime = Date.now()
   if (!opts.cache) {
     opts.log.info('prefetch', 'skipping prefetch: no cache provided')
-    return BB.resolve({spec})
+    return BB.resolve({ spec })
   }
   if (opts.integrity && !opts.preferOnline) {
     opts.log.silly('prefetch', 'checking if', opts.integrity, 'is already cached')