@socketsecurity/cli-with-sentry 1.0.33 → 1.0.35

package/dist/cli.js

@@ -3091,7 +3091,7 @@ function getSocketPullRequestTitle(purl, newVersion, workspace) {
 async function gitCleanFdx(cwd = process.cwd()) {
   const stdioIgnoreOptions = {
     cwd,
-    stdio: 'ignore'
+    stdio: debug.isDebug('stdio') ? 'inherit' : 'ignore'
   };
   // TODO: propagate CResult?
   await spawn.spawn('git', ['clean', '-fdx'], stdioIgnoreOptions);
@@ -3109,16 +3109,8 @@ async function gitCreateAndPushBranch(branch, commitMsg, filepaths, options) {
   };
   const stdioIgnoreOptions = {
     cwd,
-    stdio: 'inherit'
+    stdio: debug.isDebug('stdio') ? 'inherit' : 'ignore'
   };
-  logger.logger.dir({
-    branch,
-    user,
-    email,
-    cwd,
-    filepaths,
-    commitMsg
-  });
   try {
     await gitEnsureIdentity(user, email, cwd);
     await spawn.spawn('git', ['checkout', '-b', branch], stdioIgnoreOptions);
@@ -3127,14 +3119,10 @@ async function gitCreateAndPushBranch(branch, commitMsg, filepaths, options) {
     await spawn.spawn('git', ['push', '--force', '--set-upstream', 'origin', branch], stdioIgnoreOptions);
     return true;
   } catch (e) {
-    logger.logger.dir({
+    debug.debugFn('error', `caught: git push --force --set-upstream origin ${branch} failed`);
+    debug.debugDir('inspect', {
       error: e
     });
-    // debugFn(
-    //   'error',
-    //   `caught: git push --force --set-upstream origin ${branch} failed`,
-    // )
-    // debugDir('inspect', { error: e })
   }
   try {
     // Will throw with exit code 1 if branch does not exist.
@@ -3183,7 +3171,7 @@ async function gitRepoInfo(cwd = process.cwd()) {
 async function gitEnsureIdentity(name, email, cwd = process.cwd()) {
   const stdioIgnoreOptions = {
     cwd,
-    stdio: 'ignore'
+    stdio: debug.isDebug('stdio') ? 'inherit' : 'ignore'
   };
   const stdioPipeOptions = {
     cwd
@@ -3228,7 +3216,7 @@ async function gitResetAndClean(branch = 'HEAD', cwd = process.cwd()) {
 async function gitResetHard(branch = 'HEAD', cwd = process.cwd()) {
   const stdioIgnoreOptions = {
     cwd,
-    stdio: 'ignore'
+    stdio: debug.isDebug('stdio') ? 'inherit' : 'ignore'
   };
   await spawn.spawn('git', ['reset', '--hard', branch], stdioIgnoreOptions);
 }
@@ -3238,13 +3226,10 @@ async function gitUnstagedModifiedFiles(cwd = process.cwd()) {
       cwd
     };
     const changedFilesDetails = (await spawn.spawn('git', ['diff', '--name-only'], stdioPipeOptions)).stdout;
-    const rawRelPaths = changedFilesDetails.split('\n') ?? [];
-    console.log({
-      rawRelPaths
-    });
+    const relPaths = changedFilesDetails.split('\n') ?? [];
     return {
       ok: true,
-      data: rawRelPaths.map(p => path$1.normalizePath(p))
+      data: relPaths.map(p => path$1.normalizePath(p))
     };
   } catch (e) {
     debug.debugFn('error', 'caught: git diff --name-only failed');
@@ -3644,7 +3629,7 @@ async function openPr(owner, repo, branch, purl, newVersion, options) {
 async function setGitRemoteGithubRepoUrl(owner, repo, token, cwd = process.cwd()) {
   const stdioIgnoreOptions = {
     cwd,
-    stdio: 'ignore'
+    stdio: debug.isDebug('stdio') ? 'inherit' : 'ignore'
   };
   const {
     host
@@ -3936,10 +3921,7 @@ async function agentFix(pkgEnvDetails, actualTree, alertsMap, installer, {
           cwd,
           spinner
         });
-        const maybeLockSrc = maybeActualTree ?
-        // eslint-disable-next-line no-await-in-loop
-        await utils.readLockfile(pkgEnvDetails.lockPath) : null;
-        if (maybeActualTree && maybeLockSrc) {
+        if (maybeActualTree && fs$1.existsSync(pkgEnvDetails.lockPath)) {
           actualTree = maybeActualTree;
         }
       }
@@ -4054,6 +4036,9 @@ async function agentFix(pkgEnvDetails, actualTree, alertsMap, installer, {
             }
             continue infosLoop;
           }
+
+          // eslint-disable-next-line no-await-in-loop
+          const lockSrc = await utils.readLockfile(pkgEnvDetails.lockPath);
           if (!hasAnnouncedWorkspace) {
             hasAnnouncedWorkspace = true;
             workspaceLogCallCount = logger.logger.logCallCount;
@@ -4069,19 +4054,10 @@ async function agentFix(pkgEnvDetails, actualTree, alertsMap, installer, {
               cwd,
               spinner
             });
-            // eslint-disable-next-line no-await-in-loop
-            const unstagedCResult = await gitUnstagedModifiedFiles(cwd);
-            console.log('after installer', unstagedCResult);
-            const maybeLockSrc = maybeActualTree ?
-            // eslint-disable-next-line no-await-in-loop
-            await utils.readLockfile(pkgEnvDetails.lockPath) : null;
-            if (maybeActualTree && maybeLockSrc) {
+            if (maybeActualTree && fs$1.existsSync(pkgEnvDetails.lockPath)) {
               actualTree = maybeActualTree;
               // eslint-disable-next-line no-await-in-loop
               await afterInstall(editablePkgJson, packument, oldVersion, newVersion, vulnerableVersionRange, fixConfig);
-              // eslint-disable-next-line no-await-in-loop
-              const unstagedCResult = await gitUnstagedModifiedFiles(cwd);
-              console.log('after afterInstall', unstagedCResult);
               if (test) {
                 spinner?.info(`Testing ${newId} in ${workspace}.`);
                 // eslint-disable-next-line no-await-in-loop
@@ -4103,12 +4079,13 @@ async function agentFix(pkgEnvDetails, actualTree, alertsMap, installer, {
 
           // Check repoInfo to make TypeScript happy.
           if (!errored && fixEnv.isCi && fixEnv.repoInfo) {
+            // Rewrite files in case the install reverted them.
             // 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;
-            }) : [];
+            await editablePkgJson.save({
+              ignoreWhitespace: true
+            });
+            // eslint-disable-next-line no-await-in-loop
+            await fs$1.promises.writeFile(pkgEnvDetails.lockPath, lockSrc, 'utf8');
             try {
               if (
               // eslint-disable-next-line no-await-in-loop
@@ -4125,10 +4102,7 @@ async function agentFix(pkgEnvDetails, actualTree, alertsMap, installer, {
                   cwd,
                   spinner
                 });
-                const maybeLockSrc = maybeActualTree ?
-                // eslint-disable-next-line no-await-in-loop
-                await utils.readLockfile(pkgEnvDetails.lockPath) : null;
-                if (maybeActualTree && maybeLockSrc) {
+                if (maybeActualTree && fs$1.existsSync(pkgEnvDetails.lockPath)) {
                   actualTree = maybeActualTree;
                   continue infosLoop;
                 }
@@ -14289,5 +14263,5 @@ void (async () => {
     await utils.captureException(e);
   }
 })();
-//# debugId=edbafe48-120d-4ca9-ac23-1f9ab4197a31
+//# debugId=4d4efe6d-3f8e-4bd8-b5f9-0987e88a59d4
 //# sourceMappingURL=cli.js.map