socket 1.0.42 → 1.0.44

package/dist/cli.js

@@ -3882,6 +3882,13 @@ async function agentFix(pkgEnvDetails, actualTree, alertsMap, installer, {
     spinner?.dedent();
     packumentCache.clear();
   };
+  const getModifiedFiles = async (cwd = process.cwd()) => {
+    const unstagedCResult = await gitUnstagedModifiedFiles(cwd);
+    return unstagedCResult.ok ? unstagedCResult.data.filter(filepath => {
+      const basename = path.basename(filepath);
+      return basename === 'package.json' || basename === pkgEnvDetails.lockName;
+    }) : [];
+  };
   const handleInstallFail = () => {
     cleanupInfoEntriesLoop();
     return {
@@ -3890,6 +3897,9 @@ async function agentFix(pkgEnvDetails, actualTree, alertsMap, installer, {
       cause: `Unexpected condition: ${pkgEnvDetails.agent} install failed`
     };
   };
+  const hasModifiedFiles = async (cwd = process.cwd()) => {
+    return (await getModifiedFiles(cwd)).length > 0;
+  };
   spinner?.stop();
   infoEntriesLoop: for (let i = 0, {
       length
@@ -4044,13 +4054,8 @@ async function agentFix(pkgEnvDetails, actualTree, alertsMap, installer, {
           await afterUpdate(editablePkgJson, packument, oldVersion, newVersion, vulnerableVersionRange, fixConfig);
 
           // eslint-disable-next-line no-await-in-loop
-          const unstagedCResult = await gitUnstagedModifiedFiles(cwd);
-          const moddedFilepaths = unstagedCResult.ok ? unstagedCResult.data.filter(filepath => {
-            const basename = path.basename(filepath);
-            return basename === 'package.json' || basename === pkgEnvDetails.lockName;
-          }) : [];
-          if (!moddedFilepaths.length) {
-            logger.logger.warn('Unexpected condition: Nothing to commit, skipping PR creation.');
+          if (!(await hasModifiedFiles(cwd))) {
+            debug.debugFn('notice', 'skip: nothing to commit, skipping PR creation');
             // Reset things just in case.
             if (fixEnv.isCi) {
               // eslint-disable-next-line no-await-in-loop
@@ -4103,7 +4108,9 @@ async function agentFix(pkgEnvDetails, actualTree, alertsMap, installer, {
             try {
               if (
               // eslint-disable-next-line no-await-in-loop
-              !(await gitCreateAndPushBranch(branch, getSocketCommitMessage(oldPurl, newVersion, workspace), moddedFilepaths, {
+              !(await gitCreateAndPushBranch(branch, getSocketCommitMessage(oldPurl, newVersion, workspace),
+              // eslint-disable-next-line no-await-in-loop
+              await getModifiedFiles(cwd), {
                 cwd,
                 email: fixEnv.gitEmail,
                 user: fixEnv.gitUser
@@ -14303,5 +14310,5 @@ void (async () => {
     await utils.captureException(e);
   }
 })();
-//# debugId=1cc1c955-a609-45d8-a905-6f7ba892424a
+//# debugId=54a399b0-5fc6-4a0b-ac22-11952f37b9d5
 //# sourceMappingURL=cli.js.map