cacache 17.1.0 → 17.1.1

package/lib/content/write.js

@@ -15,6 +15,9 @@ const fsm = require('fs-minipass')
 
 module.exports = write
 
+// Cache of move operations in process so we don't duplicate
+const moveOperations = new Map()
+
 async function write (cache, data, opts = {}) {
   const { algorithms, size, integrity } = opts
 
@@ -159,15 +162,23 @@ async function makeTmp (cache, opts) {
 async function moveToDestination (tmp, cache, sri, opts) {
   const destination = contentPath(cache, sri)
   const destDir = path.dirname(destination)
-
-  await fs.mkdir(destDir, { recursive: true })
+  if (moveOperations.has(destination)) {
+    return moveOperations.get(destination)
+  }
+  moveOperations.set(
+    destination,
+    fs.mkdir(destDir, { recursive: true })
+      .then(() => moveFile(tmp.target, destination, { overwrite: false }))
+  )
   try {
-    await moveFile(tmp.target, destination, { overwrite: false })
+    await moveOperations.get(destination)
     tmp.moved = true
   } catch (err) {
     if (!err.message.startsWith('The destination file exists')) {
       throw Object.assign(err, { code: 'EEXIST' })
     }
+  } finally {
+    moveOperations.delete(destination)
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cacache",
-  "version": "17.1.0",
+  "version": "17.1.1",
   "cache-version": {
     "content": "2",
     "index": "5"
@@ -60,7 +60,7 @@
   },
   "devDependencies": {
     "@npmcli/eslint-config": "^4.0.0",
-    "@npmcli/template-oss": "4.14.1",
+    "@npmcli/template-oss": "4.15.1",
     "tap": "^16.0.0"
   },
   "engines": {
@@ -69,7 +69,7 @@
   "templateOSS": {
     "//@npmcli/template-oss": "This file is partially managed by @npmcli/template-oss. Edits may be overwritten.",
     "windowsCI": false,
-    "version": "4.14.1",
+    "version": "4.15.1",
     "publish": "true"
   },
   "author": "GitHub Inc.",