cacache 16.1.0 → 16.1.3

package/lib/content/read.js

@@ -13,7 +13,7 @@ async function read (cache, integrity, opts = {}) {
   const { size } = opts
   const { stat, cpath, sri } = await withContentSri(cache, integrity, async (cpath, sri) => {
     // get size
-    const stat = await fs.lstat(cpath)
+    const stat = await fs.stat(cpath)
     return { stat, cpath, sri }
   })
   if (typeof size === 'number' && stat.size !== size) {
@@ -73,15 +73,15 @@ function readStream (cache, integrity, opts = {}) {
   // Set all this up to run on the stream and then just return the stream
   Promise.resolve().then(async () => {
     const { stat, cpath, sri } = await withContentSri(cache, integrity, async (cpath, sri) => {
-      // just lstat to ensure it exists
-      const stat = await fs.lstat(cpath)
+      // just stat to ensure it exists
+      const stat = await fs.stat(cpath)
       return { stat, cpath, sri }
     })
     if (typeof size === 'number' && size !== stat.size) {
       return stream.emit('error', sizeError(size, stat.size))
     }
 
-    readPipeline(cpath, stat.size, sri, stream)
+    return readPipeline(cpath, stat.size, sri, stream)
   }).catch(err => stream.emit('error', err))
 
   return stream
@@ -111,7 +111,7 @@ async function hasContent (cache, integrity) {
 
   try {
     return await withContentSri(cache, integrity, async (cpath, sri) => {
-      const stat = await fs.lstat(cpath)
+      const stat = await fs.stat(cpath)
       return { size: stat.size, sri, stat }
     })
   } catch (err) {
@@ -139,7 +139,7 @@ function hasContentSync (cache, integrity) {
 
   return withContentSriSync(cache, integrity, (cpath, sri) => {
     try {
-      const stat = fs.lstatSync(cpath)
+      const stat = fs.statSync(cpath)
       return { size: stat.size, sri, stat }
     } catch (err) {
       if (err.code === 'ENOENT') {

package/lib/content/write.js

@@ -80,9 +80,11 @@ class CacacheWriteStream extends Flush {
         // defer this one tick by rejecting a promise on it.
         return Promise.reject(e).catch(cb)
       }
+      // eslint-disable-next-line promise/catch-or-return
       this.handleContentP.then(
         (res) => {
           res.integrity && this.emit('integrity', res.integrity)
+          // eslint-disable-next-line promise/always-return
           res.size !== null && this.emit('size', res.size)
           cb()
         },

package/lib/entry-index.js

@@ -285,6 +285,7 @@ function lsStream (cache) {
       }))
     }))
     stream.end()
+    return stream
   }).catch(err => stream.emit('error', err))
 
   return stream

package/lib/get.js

@@ -155,6 +155,7 @@ function getStream (cache, key, opts = {}) {
       stream.unshift(memoStream)
     }
     stream.unshift(src)
+    return stream
   }).catch((err) => stream.emit('error', err))
 
   return stream

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cacache",
-  "version": "16.1.0",
+  "version": "16.1.3",
   "cache-version": {
     "content": "2",
     "index": "5"
@@ -65,11 +65,11 @@
     "rimraf": "^3.0.2",
     "ssri": "^9.0.0",
     "tar": "^6.1.11",
-    "unique-filename": "^1.1.1"
+    "unique-filename": "^2.0.0"
   },
   "devDependencies": {
     "@npmcli/eslint-config": "^3.0.1",
-    "@npmcli/template-oss": "3.4.3",
+    "@npmcli/template-oss": "3.5.0",
     "tap": "^16.0.0"
   },
   "engines": {
@@ -78,7 +78,7 @@
   "templateOSS": {
     "//@npmcli/template-oss": "This file is partially managed by @npmcli/template-oss. Edits may be overwritten.",
     "windowsCI": false,
-    "version": "3.4.3"
+    "version": "3.5.0"
   },
   "author": "GitHub Inc."
 }