@socketsecurity/cli-with-sentry 0.14.97 → 0.14.98

package/dist/instrument-with-sentry.js

@@ -41,7 +41,7 @@ const relConstantsPath = './constants'
   Sentry.setTag(
     'version',
     // The '@rollup/plugin-replace' will replace "process.env['INLINED_SOCKET_CLI_VERSION_HASH']".
-    '0.14.97:f7d4687:6a21dfff:pub'
+    '0.14.98:34de472:32aaa5f0:pub'
   )
   const constants = require(relConstantsPath)
   if (constants.ENV.SOCKET_CLI_DEBUG) {
@@ -56,5 +56,5 @@ const relConstantsPath = './constants'
   } = constants
   setSentry(Sentry)
 }
-//# debugId=69b16c6f-575a-4ad6-ad24-7674c1cfa148
+//# debugId=2a41e4d4-7137-41d2-9710-a475fcfeadc1
 //# sourceMappingURL=instrument-with-sentry.js.map

package/dist/instrument-with-sentry.js.map

@@ -1 +1 @@
-{"version":3,"file":"instrument-with-sentry.js","sources":["../../src/instrument-with-sentry.ts"],"sourcesContent":["// This should ONLY be included in the special Sentry build!\n// Otherwise the Sentry dependency won't even be present in the manifest.\n\nimport { logger } from '@socketsecurity/registry/lib/logger'\n\n// Require constants with require(relConstantsPath) instead of require('./constants')\n// so Rollup doesn't generate a constants2.js chunk.\nconst relConstantsPath = './constants'\n// The '@rollup/plugin-replace' will replace \"process.env['INLINED_SOCKET_CLI_SENTRY_BUILD']\".\nif (process.env['INLINED_SOCKET_CLI_SENTRY_BUILD']) {\n  const Sentry = require('@sentry/node')\n  Sentry.init({\n    onFatalError(error: Error) {\n      // Defer module loads until after Sentry.init is called.\n      if (require(relConstantsPath).ENV.SOCKET_CLI_DEBUG) {\n        logger.fail('[DEBUG] [Sentry onFatalError]:', error)\n      }\n    },\n    dsn: 'https://66736701db8e4ffac046bd09fa6aaced@o555220.ingest.us.sentry.io/4508846967619585',\n    enabled: true,\n    integrations: []\n  })\n  Sentry.setTag(\n    'environment',\n    // The '@rollup/plugin-replace' will replace \"process.env['INLINED_SOCKET_CLI_PUBLISHED_BUILD']\".\n    process.env['INLINED_SOCKET_CLI_PUBLISHED_BUILD']\n      ? 'pub'\n      : // The NODE_ENV convention is used by apps to define the runtime environment.\n        // https://nodejs.org/en/learn/getting-started/nodejs-the-difference-between-development-and-production\n        process.env['NODE_ENV']\n  )\n  Sentry.setTag(\n    'version',\n    // The '@rollup/plugin-replace' will replace \"process.env['INLINED_SOCKET_CLI_VERSION_HASH']\".\n    process.env['INLINED_SOCKET_CLI_VERSION_HASH']\n  )\n  const constants = require(relConstantsPath)\n  if (constants.ENV.SOCKET_CLI_DEBUG) {\n    Sentry.setTag('debugging', true)\n    logger.log('[DEBUG] Set up Sentry.')\n  } else {\n    Sentry.setTag('debugging', false)\n  }\n  const {\n    kInternalsSymbol,\n    [kInternalsSymbol as unknown as 'Symbol(kInternalsSymbol)']: { setSentry }\n  } = constants\n  setSentry(Sentry)\n} else if (require(relConstantsPath).ENV.SOCKET_CLI_DEBUG) {\n  logger.log('[DEBUG] Sentry disabled explicitly.')\n}\n"],"names":["logger","dsn","enabled","integrations","process","Sentry","setSentry"],"mappings":";;;;;;;;;;;;;AAAA;AACA;;;AAIA;AACA;AACA;AACA;AACoD;AAClD;;;AAGI;;AAEEA;AACF;;AAEFC;AACAC;AACAC;AACF;;AAGE;AACAC;;AAQA;AACAA;AAEF;AACA;AACEC;AACAL;AACF;AACEK;AACF;;;AAGE;AAA+DC;AAAU;AAC3E;;AAEF","debugId":"69b16c6f-575a-4ad6-ad24-7674c1cfa148"}
+{"version":3,"file":"instrument-with-sentry.js","sources":["../../src/instrument-with-sentry.ts"],"sourcesContent":["// This should ONLY be included in the special Sentry build!\n// Otherwise the Sentry dependency won't even be present in the manifest.\n\nimport { logger } from '@socketsecurity/registry/lib/logger'\n\n// Require constants with require(relConstantsPath) instead of require('./constants')\n// so Rollup doesn't generate a constants2.js chunk.\nconst relConstantsPath = './constants'\n// The '@rollup/plugin-replace' will replace \"process.env['INLINED_SOCKET_CLI_SENTRY_BUILD']\".\nif (process.env['INLINED_SOCKET_CLI_SENTRY_BUILD']) {\n  const Sentry = require('@sentry/node')\n  Sentry.init({\n    onFatalError(error: Error) {\n      // Defer module loads until after Sentry.init is called.\n      if (require(relConstantsPath).ENV.SOCKET_CLI_DEBUG) {\n        logger.fail('[DEBUG] [Sentry onFatalError]:', error)\n      }\n    },\n    dsn: 'https://66736701db8e4ffac046bd09fa6aaced@o555220.ingest.us.sentry.io/4508846967619585',\n    enabled: true,\n    integrations: []\n  })\n  Sentry.setTag(\n    'environment',\n    // The '@rollup/plugin-replace' will replace \"process.env['INLINED_SOCKET_CLI_PUBLISHED_BUILD']\".\n    process.env['INLINED_SOCKET_CLI_PUBLISHED_BUILD']\n      ? 'pub'\n      : // The NODE_ENV convention is used by apps to define the runtime environment.\n        // https://nodejs.org/en/learn/getting-started/nodejs-the-difference-between-development-and-production\n        process.env['NODE_ENV']\n  )\n  Sentry.setTag(\n    'version',\n    // The '@rollup/plugin-replace' will replace \"process.env['INLINED_SOCKET_CLI_VERSION_HASH']\".\n    process.env['INLINED_SOCKET_CLI_VERSION_HASH']\n  )\n  const constants = require(relConstantsPath)\n  if (constants.ENV.SOCKET_CLI_DEBUG) {\n    Sentry.setTag('debugging', true)\n    logger.log('[DEBUG] Set up Sentry.')\n  } else {\n    Sentry.setTag('debugging', false)\n  }\n  const {\n    kInternalsSymbol,\n    [kInternalsSymbol as unknown as 'Symbol(kInternalsSymbol)']: { setSentry }\n  } = constants\n  setSentry(Sentry)\n} else if (require(relConstantsPath).ENV.SOCKET_CLI_DEBUG) {\n  logger.log('[DEBUG] Sentry disabled explicitly.')\n}\n"],"names":["logger","dsn","enabled","integrations","process","Sentry","setSentry"],"mappings":";;;;;;;;;;;;;;;AAAA;AACA;;;AAIA;AACA;AACA;AACA;AACoD;AAClD;;;AAGI;;AAEEA;AACF;;AAEFC;AACAC;AACAC;AACF;;AAGE;AACAC;;AAQA;AACAA;AAEF;AACA;AACEC;AACAL;AACF;AACEK;AACF;;;AAGE;AAA+DC;AAAU;AAC3E;;AAEF","debugId":"2a41e4d4-7137-41d2-9710-a475fcfeadc1"}

package/dist/module-sync/cli.js

@@ -35,6 +35,7 @@ const arrays = require('@socketsecurity/registry/lib/arrays')
 const registry = require('@socketsecurity/registry')
 const npm = require('@socketsecurity/registry/lib/npm')
 const packages = require('@socketsecurity/registry/lib/packages')
+const packageurlJs = require('@socketregistry/packageurl-js')
 const spawn = require('@socketsecurity/registry/lib/spawn')
 const index_cjs = require('@socketregistry/hyrious__bun.lockb/index.cjs')
 const sorts = require('@socketsecurity/registry/lib/sorts')
@@ -899,7 +900,7 @@ function emitBanner(name) {
   logger.logger.error(getAsciiHeader(name))
 }
 function getAsciiHeader(command) {
-  const cliVersion = '0.14.97:f7d4687:6a21dfff:pub' // The '@rollup/plugin-replace' will replace "process.env['INLINED_SOCKET_CLI_VERSION_HASH']".
+  const cliVersion = '0.14.98:34de472:32aaa5f0:pub' // The '@rollup/plugin-replace' will replace "process.env['INLINED_SOCKET_CLI_VERSION_HASH']".
   const nodeVersion = process$1.version
   const apiToken = shadowNpmInject.getDefaultToken()
   const shownToken = apiToken ? getLastFiveOfApiToken(apiToken) : 'no'
@@ -1356,7 +1357,7 @@ async function runCycloneDX(yargvWithYes) {
         await shadowBin(NPX$3, [
           ...yesArgs,
           // The '@rollup/plugin-replace' will replace "process.env['INLINED_SYNP_VERSION']".
-          `synp@${'^1.9.14'}`,
+          `synp@${'1.9.14'}`,
           '--source-file',
           `./${YARN_LOCK}`
         ])
@@ -1368,7 +1369,7 @@ async function runCycloneDX(yargvWithYes) {
   await shadowBin(NPX$3, [
     ...yesArgs,
     // The '@rollup/plugin-replace' will replace "process.env['INLINED_CYCLONEDX_CDXGEN_VERSION']".
-    `@cyclonedx/cdxgen@${'^11.2.3'}`,
+    `@cyclonedx/cdxgen@${'11.2.3'}`,
     ...argvToArray(yargv)
   ])
   if (cleanupPackageLock) {
@@ -3701,6 +3702,12 @@ const cmdDiffScan = {
 }
 
 const { GITHUB_REF_NAME } = constants
+function formatBranchName(str) {
+  return str.replace(/[-_.]+/g, '-').replace(/[-a-zA-Z0-9]+/g, '') ?? ''
+}
+function getPkgNameFromPurlObj(purlObj) {
+  return `${purlObj.namespace ? `${purlObj.namespace}/` : ''}${purlObj.name}`
+}
 async function branchExists(branch, cwd = process.cwd()) {
   try {
     await spawn.spawn(
@@ -3762,8 +3769,28 @@ function getBaseBranch() {
     'main'
   )
 }
-function getSocketBranchName(name, version) {
-  return `socket-fix-${name}-${version.replace(/\./g, '-')}`
+function getSocketBranchName(purl, toVersion) {
+  const purlObj = packageurlJs.PackageURL.fromString(purl)
+  const namespace = formatBranchName(purlObj.namespace ?? '')
+  const name = formatBranchName(purlObj.name)
+  const version = formatBranchName(toVersion)
+  const fullName = `${namespace ? `${namespace}-` : ''}${name}`
+  return `socket-fix-${fullName}-${version}`
+}
+function getSocketPullRequestTitle(purl, toVersion) {
+  const purlObj = packageurlJs.PackageURL.fromString(purl)
+  const pkgName = getPkgNameFromPurlObj(purlObj)
+  return `Bump ${pkgName} from ${purlObj.version} to ${toVersion}`
+}
+function getSocketPullRequestBody(purl, toVersion) {
+  const purlObj = packageurlJs.PackageURL.fromString(purl)
+  const pkgName = getPkgNameFromPurlObj(purlObj)
+  return `Bumps [${pkgName}](https://socket.dev/${purlObj.type}/package/${pkgName}) from ${purlObj.version} to ${toVersion}.`
+}
+function getSocketCommitMessage(purl, toVersion) {
+  const purlObj = packageurlJs.PackageURL.fromString(purl)
+  const pkgName = getPkgNameFromPurlObj(purlObj)
+  return `socket: Bump ${pkgName} from ${purlObj.version} to ${toVersion}`
 }
 
 const { GITHUB_ACTIONS, GITHUB_REPOSITORY, SOCKET_SECURITY_GITHUB_PAT } =
@@ -3778,6 +3805,18 @@ function getOctokit() {
   }
   return _octokit
 }
+let _octokitGraphql
+function getOctokitGraphql() {
+  if (!_octokitGraphql) {
+    _octokitGraphql = vendor.graphql2.defaults({
+      headers: {
+        // Lazily access constants.ENV[SOCKET_SECURITY_GITHUB_PAT].
+        authorization: `token ${constants.ENV[SOCKET_SECURITY_GITHUB_PAT]}`
+      }
+    })
+  }
+  return _octokitGraphql
+}
 async function doesPullRequestExistForBranch(owner, repo, branch) {
   const octokit = getOctokit()
   const { data: prs } = await octokit.pulls.list({
@@ -3788,11 +3827,10 @@ async function doesPullRequestExistForBranch(owner, repo, branch) {
   })
   return prs.length > 0
 }
-async function enableAutoMerge(prResponseData) {
-  const octokit = getOctokit()
-  const { node_id: prId, number: prNumber } = prResponseData
+async function enableAutoMerge({ node_id: prId, number: prNumber }) {
+  const octokitGraphql = getOctokitGraphql()
   try {
-    await octokit.graphql(
+    await octokitGraphql(
       `
       mutation EnableAutoMerge($pullRequestId: ID!) {
         enablePullRequestAutoMerge(input: {
@@ -3813,16 +3851,23 @@ async function enableAutoMerge(prResponseData) {
       }
     )
     logger.logger.info(`Auto-merge enabled for PR #${prNumber}`)
+    return true
   } catch (e) {
-    logger.logger.error(`Failed to enable auto-merge for PR #${prNumber}:`, e)
+    let message = `Failed to enable auto-merge for PR #${prNumber}`
+    if (e instanceof vendor.GraphqlResponseError && e.errors) {
+      const details = e.errors.map(({ message }) => ` - ${message}`).join('\n')
+      message += `:\n${details}`
+    }
+    logger.logger.error(message)
+    return false
   }
 }
-function getGitHubRepoInfo() {
+function getGitHubEnvRepoInfo() {
   // Lazily access constants.ENV[GITHUB_REPOSITORY].
   const ownerSlashRepo = constants.ENV[GITHUB_REPOSITORY]
   const slashIndex = ownerSlashRepo.indexOf('/')
   if (slashIndex === -1) {
-    throw new Error('GITHUB_REPOSITORY environment variable not set')
+    throw new Error('Missing GITHUB_REPOSITORY environment variable')
   }
   return {
     owner: ownerSlashRepo.slice(0, slashIndex),
@@ -3834,8 +3879,8 @@ async function openGitHubPullRequest(
   repo,
   baseBranch,
   branch,
-  name,
-  version,
+  purl,
+  toVersion,
   cwd = process.cwd()
 ) {
   // Lazily access constants.ENV[GITHUB_ACTIONS].
@@ -3850,19 +3895,34 @@ async function openGitHubPullRequest(
       cwd
     })
     const octokit = getOctokit()
-    return await octokit.pulls.create({
-      owner,
-      repo,
-      title: `chore: upgrade ${name} to ${version}`,
-      head: branch,
-      base: baseBranch,
-      body: `[socket] Upgrade \`${name}\` to ${version}`
-    })
-  } else {
-    throw new Error(
-      'Unsupported CI platform or missing GITHUB_ACTIONS environment variable'
-    )
+    try {
+      return await octokit.pulls.create({
+        owner,
+        repo,
+        title: getSocketPullRequestTitle(purl, toVersion),
+        head: branch,
+        base: baseBranch,
+        body: getSocketPullRequestBody(purl, toVersion)
+      })
+    } catch (e) {
+      let message = `Failed to open pull request`
+      if (e instanceof vendor.RequestError) {
+        const restErrors = e.response?.data?.['errors']
+        if (Array.isArray(restErrors)) {
+          const details = restErrors
+            .map(
+              restErr =>
+                `- ${restErr.message ?? `${restErr.resource}.${restErr.field} (${restErr.code})`}`
+            )
+            .join('\n')
+          message += `:\n${details}`
+        }
+      }
+      logger.logger.error(message)
+      return null
+    }
   }
+  throw new Error('Missing GITHUB_ACTIONS environment variable')
 }
 
 const { CI: CI$1, NPM: NPM$f } = constants
@@ -3928,7 +3988,9 @@ async function npmFix(
     for (const spec of specs) {
       const lastAtSignIndex = spec.lastIndexOf('@')
       const name = spec.slice(0, lastAtSignIndex)
-      const oldVersion = spec.slice(lastAtSignIndex + 1)
+      const fromVersion = spec.slice(lastAtSignIndex + 1)
+      const fromSpec = `${name}@${fromVersion}`
+      const fromPurl = `pkg:npm/${fromSpec}`
       for (const {
         firstPatchedVersionIdentifier,
         vulnerableVersionRange
@@ -3940,27 +4002,32 @@ async function npmFix(
         const node = shadowNpmInject.findPackageNode(
           arb.idealTree,
           name,
-          oldVersion
+          fromVersion
         )
         if (!node) {
           continue
         }
-        const oldSpec = `${name}@${oldVersion}`
         if (
           !shadowNpmInject.updateNode(node, packument, vulnerableVersionRange)
         ) {
-          spinner?.failAndStop(`Could not patch ${oldSpec}`)
+          spinner?.failAndStop(`Could not patch ${fromSpec}`)
           return
         }
-        const targetVersion = node.package.version
+        const toVersion = node.package.version
+        const toVersionRange = shadowNpmInject.applyRange(
+          fromVersion,
+          toVersion,
+          rangeStyle
+        )
+        const toSpec = `${name}@${toVersionRange}`
         let branch
         let owner
         let repo
         let shouldOpenPr = false
         // Lazily access constants.ENV[CI].
         if (constants.ENV[CI$1]) {
-          ;({ owner, repo } = getGitHubRepoInfo())
-          branch = getSocketBranchName(name, targetVersion)
+          ;({ owner, repo } = getGitHubEnvRepoInfo())
+          branch = getSocketBranchName(name, toVersion)
           // eslint-disable-next-line no-await-in-loop
           shouldOpenPr = !(await doesPullRequestExistForBranch(
             owner,
@@ -3968,7 +4035,6 @@ async function npmFix(
             branch
           ))
         }
-        const fixSpec = `${name}@^${targetVersion}`
         const revertData = {
           ...(editablePkgJson.content.dependencies
             ? {
@@ -3987,7 +4053,7 @@ async function npmFix(
               }
             : undefined)
         }
-        spinner?.info(`Installing ${fixSpec}`)
+        spinner?.info(`Installing ${toSpec}`)
         const baseBranch = getBaseBranch()
 
         // eslint-disable-next-line no-await-in-loop
@@ -3999,7 +4065,7 @@ async function npmFix(
             editablePkgJson,
             arb.idealTree,
             node,
-            targetVersion,
+            toVersion,
             rangeStyle
           )
           // eslint-disable-next-line no-await-in-loop
@@ -4012,7 +4078,7 @@ async function npmFix(
           })
           installed = true
           if (test) {
-            spinner?.info(`Testing ${fixSpec}`)
+            spinner?.info(`Testing ${toSpec}`)
             // eslint-disable-next-line no-await-in-loop
             await npm.runScript(testScript, [], {
               spinner,
@@ -4022,7 +4088,7 @@ async function npmFix(
           spinner?.successAndStop(`Fixed ${name}`)
           spinner?.start()
         } catch {
-          spinner?.error(`Reverting ${fixSpec}`)
+          spinner?.error(`Reverting ${toSpec}`)
           if (saved) {
             editablePkgJson.update(revertData)
             // eslint-disable-next-line no-await-in-loop
@@ -4034,35 +4100,29 @@ async function npmFix(
               cwd
             })
           }
-          spinner?.failAndStop(`Failed to fix ${oldSpec}`)
+          spinner?.failAndStop(`Failed to fix ${fromSpec}`)
           return
         }
         if (shouldOpenPr) {
-          let prResponse
-          try {
-            // eslint-disable-next-line no-await-in-loop
-            prResponse = await openGitHubPullRequest(
-              owner,
-              repo,
-              baseBranch,
-              branch,
-              name,
-              targetVersion,
-              cwd
-            )
-          } catch (e) {
-            logger.logger.error('Failed to open pull request', e)
-          }
+          // eslint-disable-next-line no-await-in-loop
+          await createAndPushBranchIfNeeded(
+            branch,
+            getSocketCommitMessage(fromPurl, toVersion),
+            cwd
+          )
+          // eslint-disable-next-line no-await-in-loop
+          const prResponse = await openGitHubPullRequest(
+            owner,
+            repo,
+            baseBranch,
+            branch,
+            fromPurl,
+            toVersion,
+            cwd
+          )
           if (prResponse && autoMerge) {
-            try {
-              // eslint-disable-next-line no-await-in-loop
-              await enableAutoMerge(prResponse.data)
-            } catch (e) {
-              logger.logger.error(
-                'Failed to enable auto-merge in pull request',
-                e
-              )
-            }
+            // eslint-disable-next-line no-await-in-loop
+            await enableAutoMerge(prResponse.data)
           }
         }
       }
@@ -4341,7 +4401,9 @@ async function pnpmFix(
     for (const spec of specs) {
       const lastAtSignIndex = spec.lastIndexOf('@')
       const name = spec.slice(0, lastAtSignIndex)
-      const oldVersion = spec.slice(lastAtSignIndex + 1)
+      const fromVersion = spec.slice(lastAtSignIndex + 1)
+      const fromSpec = `${name}@${fromVersion}`
+      const fromPurl = `pkg:npm/${fromSpec}`
       for (const {
         firstPatchedVersionIdentifier,
         vulnerableVersionRange
@@ -4349,33 +4411,45 @@ async function pnpmFix(
         const node = shadowNpmInject.findPackageNode(
           actualTree,
           name,
-          oldVersion
+          fromVersion
         )
         if (!node) {
           continue
         }
-        const oldSpec = `${name}@${oldVersion}`
         const availableVersions = Object.keys(packument.versions)
-        const targetVersion = shadowNpmInject.findBestPatchVersion(
+        const toVersion = shadowNpmInject.findBestPatchVersion(
           node,
           availableVersions,
           vulnerableVersionRange
         )
-        const targetPackument = targetVersion
-          ? packument.versions[targetVersion]
+        const targetPackument = toVersion
+          ? packument.versions[toVersion]
           : undefined
-        if (!(targetVersion && targetPackument)) {
-          spinner?.failAndStop(`Could not patch ${oldSpec}`)
+        if (!(toVersion && targetPackument)) {
+          spinner?.failAndStop(`Could not patch ${fromSpec}`)
           return
         }
+        const oldPnpm = editablePkgJson.content[PNPM$9]
+        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 toVersionRange = shadowNpmInject.applyRange(
+          oldOverrides?.[overrideKey] ?? fromVersion,
+          toVersion,
+          rangeStyle
+        )
+        const toSpec = `${name}@${toVersionRange}`
         let branch
         let owner
         let repo
         let shouldOpenPr = false
         // Lazily access constants.ENV[CI].
         if (constants.ENV[CI]) {
-          ;({ owner, repo } = getGitHubRepoInfo())
-          branch = getSocketBranchName(name, targetVersion)
+          ;({ owner, repo } = getGitHubEnvRepoInfo())
+          branch = getSocketBranchName(name, toVersion)
           // eslint-disable-next-line no-await-in-loop
           shouldOpenPr = !(await doesPullRequestExistForBranch(
             owner,
@@ -4383,24 +4457,11 @@ async function pnpmFix(
             branch
           ))
         }
-        const oldPnpm = editablePkgJson.content[PNPM$9]
-        const oldPnpmKeyCount = oldPnpm ? Object.keys(oldPnpm).length : 0
-        const oldOverrides = oldPnpm?.[OVERRIDES$2]
-        const oldOverridesCount = oldOverrides
-          ? Object.keys(oldOverrides).length
-          : 0
-        const overrideKey = `${node.name}@${vulnerableVersionRange}`
-        const overrideRange = shadowNpmInject.applyRange(
-          oldOverrides?.[overrideKey] ?? targetVersion,
-          targetVersion,
-          rangeStyle
-        )
-        const fixSpec = `${name}@${overrideRange}`
         const updateData = {
           [PNPM$9]: {
             ...oldPnpm,
             [OVERRIDES$2]: {
-              [overrideKey]: overrideRange,
+              [overrideKey]: toVersionRange,
               ...oldOverrides
             }
           }
@@ -4435,7 +4496,7 @@ async function pnpmFix(
               }
             : undefined)
         }
-        spinner?.info(`Installing ${fixSpec}`)
+        spinner?.info(`Installing ${toSpec}`)
         const baseBranch = getBaseBranch()
 
         // eslint-disable-next-line no-await-in-loop
@@ -4448,7 +4509,7 @@ async function pnpmFix(
             editablePkgJson,
             actualTree,
             node,
-            targetVersion,
+            toVersion,
             rangeStyle
           )
           // eslint-disable-next-line no-await-in-loop
@@ -4461,7 +4522,7 @@ async function pnpmFix(
           })
           installed = true
           if (test) {
-            spinner?.info(`Testing ${fixSpec}`)
+            spinner?.info(`Testing ${toSpec}`)
             // eslint-disable-next-line no-await-in-loop
             await npm.runScript(testScript, [], {
               spinner,
@@ -4471,7 +4532,7 @@ async function pnpmFix(
           spinner?.successAndStop(`Fixed ${name}`)
           spinner?.start()
         } catch (e) {
-          spinner?.error(`Reverting ${fixSpec}`, e)
+          spinner?.error(`Reverting ${toSpec}`, e)
           if (saved) {
             editablePkgJson.update(revertData)
             // eslint-disable-next-line no-await-in-loop
@@ -4483,41 +4544,29 @@ async function pnpmFix(
               spinner
             })
           }
-          spinner?.failAndStop(`Failed to fix ${oldSpec}`)
+          spinner?.failAndStop(`Failed to fix ${fromSpec}`)
           return
         }
         if (shouldOpenPr) {
           // eslint-disable-next-line no-await-in-loop
           await createAndPushBranchIfNeeded(
             branch,
-            `fix: upgrade ${name} to ${targetVersion}`,
+            getSocketCommitMessage(fromPurl, toVersion),
+            cwd
+          )
+          // eslint-disable-next-line no-await-in-loop
+          const prResponse = await openGitHubPullRequest(
+            owner,
+            repo,
+            baseBranch,
+            branch,
+            fromPurl,
+            toVersion,
             cwd
           )
-          let prResponse
-          try {
-            // eslint-disable-next-line no-await-in-loop
-            prResponse = await openGitHubPullRequest(
-              owner,
-              repo,
-              baseBranch,
-              branch,
-              name,
-              targetVersion,
-              cwd
-            )
-          } catch (e) {
-            logger.logger.error('Failed to open pull request', e)
-          }
           if (prResponse && autoMerge) {
-            try {
-              // eslint-disable-next-line no-await-in-loop
-              await enableAutoMerge(prResponse.data)
-            } catch (e) {
-              logger.logger.error(
-                'Failed to enable auto-merge in pull request',
-                e
-              )
-            }
+            // eslint-disable-next-line no-await-in-loop
+            await enableAutoMerge(prResponse.data)
           }
         }
       }
@@ -11439,7 +11488,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.97',
+    version: '0.14.98',
     ttl: 86_400_000 /* 24 hours in milliseconds */
   })
   try {
@@ -11507,5 +11556,5 @@ void (async () => {
     await shadowNpmInject.captureException(e)
   }
 })()
-//# debugId=bba1f45e-da3b-4698-9168-d6e8c394995d
+//# debugId=daa49c76-000e-474d-9135-b9bd3769de57
 //# sourceMappingURL=cli.js.map