socket 0.14.124 → 0.14.125

package/dist/module-sync/cli.js

@@ -917,7 +917,7 @@ function emitBanner(name) {
   logger.logger.error(getAsciiHeader(name))
 }
 function getAsciiHeader(command) {
-  const cliVersion = '0.14.124:5b2103f:da70cd14:pub' // The '@rollup/plugin-replace' will replace "process.env['INLINED_SOCKET_CLI_VERSION_HASH']".
+  const cliVersion = '0.14.125:11530dd:821f8925:pub' // The '@rollup/plugin-replace' will replace "process.env['INLINED_SOCKET_CLI_VERSION_HASH']".
   const nodeVersion = process$1.version
   const apiToken = shadowNpmInject.getDefaultToken()
   const defaultOrg = shadowNpmInject.getConfigValue('defaultOrg')
@@ -4180,7 +4180,6 @@ async function npmFix(
           )
           let error
           let errored = false
-          let installed = false
           let saved = false
 
           // eslint-disable-next-line no-await-in-loop
@@ -4196,7 +4195,6 @@ async function npmFix(
             await install$1(arb.idealTree, {
               cwd
             })
-            installed = true
             if (test) {
               if (!testedSpecs.has(newSpecKey)) {
                 testedSpecs.add(newSpecKey)
@@ -4502,32 +4500,43 @@ async function pnpmFix(
     if (!packument) {
       continue
     }
-    const failedSpecs = new Set()
     const fixedSpecs = new Set()
-    const installedSpecs = new Set()
-    const testedSpecs = new Set()
-    const unavailableSpecs = new Set()
-    const revertedSpecs = new Set()
     for (const pkgJsonPath of pkgJsonPaths) {
+      // Re-read actualTree to avoid lockfile state issues
+      // eslint-disable-next-line no-await-in-loop
+      actualTree = await getActualTree(cwd)
+      const pkgPath = path$1.dirname(pkgJsonPath)
+      const isWorkspaceRoot =
+        pkgJsonPath === pkgEnvDetails.editablePkgJson.filename
+      const workspaceName = isWorkspaceRoot
+        ? 'root'
+        : path$1.relative(rootPath, pkgPath)
+      const editablePkgJson = isWorkspaceRoot
+        ? pkgEnvDetails.editablePkgJson
+        : // eslint-disable-next-line no-await-in-loop
+          await packages.readPackageJson(pkgJsonPath, {
+            editable: true
+          })
+
+      // Get current overrides for revert logic
+      const oldPnpmSection = editablePkgJson.content[PNPM$8]
+      const oldOverrides = oldPnpmSection?.[OVERRIDES$2]
       for (const oldVersion of oldVersions) {
         const oldSpec = `${name}@${oldVersion}`
         const oldPurl = `pkg:npm/${oldSpec}`
+        const node = shadowNpmInject.findPackageNode(
+          actualTree,
+          name,
+          oldVersion
+        )
+        if (!node) {
+          debug.debugLog(`Skipping ${oldSpec}, no node found in ${pkgJsonPath}`)
+          continue
+        }
         for (const {
           firstPatchedVersionIdentifier,
           vulnerableVersionRange
         } of infos) {
-          const node = shadowNpmInject.findPackageNode(
-            actualTree,
-            name,
-            oldVersion
-          )
-          if (!node) {
-            debug.debugLog(
-              `Skipping ${oldSpec}, no node found in arborist.actualTree`,
-              pkgJsonPath
-            )
-            continue
-          }
           const availableVersions = Object.keys(packument.versions)
           const newVersion = shadowNpmInject.findBestPatchVersion(
             node,
@@ -4538,30 +4547,9 @@ async function pnpmFix(
             ? packument.versions[newVersion]
             : undefined
           if (!(newVersion && newVersionPackument)) {
-            if (!unavailableSpecs.has(oldSpec)) {
-              unavailableSpecs.add(oldSpec)
-              spinner?.fail(`No update available for ${oldSpec}`)
-            }
+            spinner?.fail(`No update available for ${oldSpec}`)
             continue
           }
-          const isWorkspaceRoot =
-            pkgJsonPath === pkgEnvDetails.editablePkgJson.filename
-          const workspaceName = isWorkspaceRoot
-            ? ''
-            : path$1.relative(rootPath, path$1.dirname(pkgJsonPath))
-          const workspaceDetails = workspaceName ? ` in ${workspaceName}` : ''
-          const editablePkgJson = isWorkspaceRoot
-            ? pkgEnvDetails.editablePkgJson
-            : // eslint-disable-next-line no-await-in-loop
-              await packages.readPackageJson(pkgJsonPath, {
-                editable: true
-              })
-          const oldPnpm = editablePkgJson.content[PNPM$8]
-          const oldPnpmKeyCount = oldPnpm ? Object.keys(oldPnpm).length : 0
-          const oldOverrides = oldPnpm?.[OVERRIDES$2]
-          const oldOverridesCount = oldOverrides
-            ? Object.keys(oldOverrides).length
-            : 0
           const overrideKey = `${name}@${vulnerableVersionRange}`
           const newVersionRange = shadowNpmInject.applyRange(
             oldOverrides?.[overrideKey] ?? oldVersion,
@@ -4569,14 +4557,20 @@ async function pnpmFix(
             rangeStyle
           )
           const newSpec = `${name}@${newVersionRange}`
-          const newSpecKey = `${workspaceName ? `${workspaceName}>` : ''}${newSpec}`
+          const newSpecKey = `${workspaceName}:${newSpec}`
+          if (fixedSpecs.has(newSpecKey)) {
+            debug.debugLog(
+              `Already fixed ${newSpec} in ${workspaceName}, skipping`
+            )
+            continue
+          }
           const updateData = isWorkspaceRoot
             ? {
                 [PNPM$8]: {
-                  ...oldPnpm,
+                  ...oldPnpmSection,
                   [OVERRIDES$2]: {
-                    [overrideKey]: newVersionRange,
-                    ...oldOverrides
+                    ...oldOverrides,
+                    [overrideKey]: newVersionRange
                   }
                 }
               }
@@ -4584,44 +4578,28 @@ async function pnpmFix(
           const revertData = {
             ...(isWorkspaceRoot
               ? {
-                  [PNPM$8]: oldPnpmKeyCount
-                    ? {
-                        ...oldPnpm,
-                        [OVERRIDES$2]:
-                          oldOverridesCount === 1
-                            ? undefined
-                            : {
-                                [overrideKey]: undefined,
-                                ...oldOverrides
-                              }
-                      }
-                    : undefined
+                  [PNPM$8]: {
+                    ...oldPnpmSection,
+                    [OVERRIDES$2]:
+                      oldOverrides && Object.keys(oldOverrides).length > 1
+                        ? {
+                            ...oldOverrides,
+                            [overrideKey]: undefined
+                          }
+                        : undefined
+                  }
                 }
               : {}),
-            ...(editablePkgJson.content.dependencies
-              ? {
-                  dependencies: editablePkgJson.content.dependencies
-                }
-              : undefined),
-            ...(editablePkgJson.content.optionalDependencies
-              ? {
-                  optionalDependencies:
-                    editablePkgJson.content.optionalDependencies
-                }
-              : undefined),
-            ...(editablePkgJson.content.peerDependencies
-              ? {
-                  peerDependencies: editablePkgJson.content.peerDependencies
-                }
-              : undefined)
+            ...(editablePkgJson.content.dependencies && {
+              dependencies: editablePkgJson.content.dependencies
+            }),
+            ...(editablePkgJson.content.optionalDependencies && {
+              optionalDependencies: editablePkgJson.content.optionalDependencies
+            }),
+            ...(editablePkgJson.content.peerDependencies && {
+              peerDependencies: editablePkgJson.content.peerDependencies
+            })
           }
-          const branch = isCi
-            ? getSocketBranchName(oldPurl, newVersion, workspaceName)
-            : ''
-          const shouldOpenPr = isCi
-            ? // eslint-disable-next-line no-await-in-loop
-              !(await doesPullRequestExistForBranch(owner, repo, branch))
-            : false
           if (updateData) {
             editablePkgJson.update(updateData)
           }
@@ -4633,106 +4611,99 @@ async function pnpmFix(
             rangeStyle
           )
           debug.debugLog(`Updated package.json from node: ${modded}`)
-          let error
-          let errored = false
-          let installed = false
 
           // eslint-disable-next-line no-await-in-loop
           if (!(await editablePkgJson.save())) {
             debug.debugLog(
-              `Skipping nothing changed in ${editablePkgJson.filename}`
+              `No changes saved for ${pkgJsonPath}, skipping install`
             )
             continue
           }
-          if (!installedSpecs.has(newSpecKey)) {
-            installedSpecs.add(newSpecKey)
-            spinner?.info(`Installing ${newSpec}${workspaceDetails}`)
-          }
+          spinner?.info(`Installing ${newSpec} in ${workspaceName}`)
+          let errored = false
+          let error
           try {
             // eslint-disable-next-line no-await-in-loop
             actualTree = await install(pkgEnvDetails, {
               spinner
             })
-            installed = true
             if (test) {
-              if (!testedSpecs.has(newSpecKey)) {
-                testedSpecs.add(newSpecKey)
-                spinner?.info(`Testing ${newSpec}${workspaceDetails}`)
-              }
+              spinner?.info(`Testing ${newSpec} in ${workspaceName}`)
               // eslint-disable-next-line no-await-in-loop
               await npm.runScript(testScript, [], {
                 spinner,
                 stdio: 'ignore'
               })
             }
-            if (!fixedSpecs.has(newSpecKey)) {
-              fixedSpecs.add(newSpecKey)
-              spinner?.successAndStop(`Fixed ${name}${workspaceDetails}`)
-              spinner?.start()
-            }
-          } catch (e) {
-            error = e
-            errored = true
-          }
-          if (
-            !errored &&
-            shouldOpenPr &&
-            // eslint-disable-next-line no-await-in-loop
-            (await gitCreateAndPushBranchIfNeeded(
-              branch,
-              getSocketCommitMessage(oldPurl, newVersion, workspaceName),
-              cwd
-            ))
-          ) {
-            // eslint-disable-next-line no-await-in-loop
-            const prResponse = await openGitHubPullRequest(
-              owner,
-              repo,
-              baseBranch,
-              branch,
+            fixedSpecs.add(newSpecKey)
+            spinner?.successAndStop(`Fixed ${name} in ${workspaceName}`)
+            spinner?.start()
+            const branch = getSocketBranchName(
               oldPurl,
               newVersion,
-              {
-                cwd,
-                workspaceName
-              }
+              workspaceName
             )
-            if (prResponse) {
-              const { data } = prResponse
-              spinner?.info(`PR #${data.number} opened.`)
-              if (autoMerge) {
-                // eslint-disable-next-line no-await-in-loop
-                await enableAutoMerge(data)
+            const shouldOpenPr = isCi
+              ? // eslint-disable-next-line no-await-in-loop
+                !(await doesPullRequestExistForBranch(owner, repo, branch))
+              : false
+            if (
+              isCi &&
+              shouldOpenPr &&
+              // eslint-disable-next-line no-await-in-loop
+              (await gitCreateAndPushBranchIfNeeded(
+                branch,
+                getSocketCommitMessage(oldPurl, newVersion, workspaceName),
+                cwd
+              ))
+            ) {
+              // eslint-disable-next-line no-await-in-loop
+              const prResponse = await openGitHubPullRequest(
+                owner,
+                repo,
+                baseBranch,
+                branch,
+                oldPurl,
+                newVersion,
+                {
+                  cwd,
+                  workspaceName
+                }
+              )
+              if (prResponse) {
+                const { data } = prResponse
+                spinner?.info(`PR #${data.number} opened.`)
+                if (autoMerge) {
+                  // eslint-disable-next-line no-await-in-loop
+                  await enableAutoMerge(data)
+                }
               }
             }
+          } catch (e) {
+            error = e
+            errored = true
           }
           if (errored || isCi) {
-            if (errored) {
-              if (!revertedSpecs.has(newSpecKey)) {
-                revertedSpecs.add(newSpecKey)
-                spinner?.error(`Reverting ${newSpec}${workspaceDetails}`, error)
-              }
-            }
             editablePkgJson.update(revertData)
+
             // eslint-disable-next-line no-await-in-loop
             await Promise.all([
               shadowNpmInject.removeNodeModules(cwd),
               ...(isCi
                 ? [gitCheckoutBaseBranchIfAvailable(baseBranch, cwd)]
                 : []),
-              ...(installed && !isCi ? [editablePkgJson.save()] : [])
+              ...(isCi ? [] : [editablePkgJson.save()])
             ])
+
             // eslint-disable-next-line no-await-in-loop
             actualTree = await install(pkgEnvDetails, {
               spinner
             })
             if (errored) {
-              if (!failedSpecs.has(newSpecKey)) {
-                failedSpecs.add(newSpecKey)
-                spinner?.failAndStop(
-                  `Update failed for ${oldSpec}${workspaceDetails}`
-                )
-              }
+              spinner?.failAndStop(
+                `Update failed for ${oldSpec} in ${workspaceName}`,
+                error
+              )
             }
           }
         }
@@ -12307,7 +12278,7 @@ void (async () => {
   await vendor.updater({
     name: SOCKET_CLI_BIN_NAME,
     // The '@rollup/plugin-replace' will replace "process.env['INLINED_SOCKET_CLI_VERSION']".
-    version: '0.14.124',
+    version: '0.14.125',
     ttl: 86_400_000 /* 24 hours in milliseconds */
   })
   try {
@@ -12375,5 +12346,5 @@ void (async () => {
     await shadowNpmInject.captureException(e)
   }
 })()
-//# debugId=bf160d06-cec8-4139-929e-f01e353b895b
+//# debugId=8c8790a2-a162-431b-99ab-b980237160d8
 //# sourceMappingURL=cli.js.map