fixup! feat: add confirmation steps to any write operation

Author: RafaelGSS

lib/security-release/security-release.js

@@ -35,16 +35,14 @@ function formatCommand(command, args) {
 }
 
 export async function confirmSecurityStep(cli, action, detail) {
-  const lines = [`Allow action: ${action}?`];
+  let message = `Allow action: ${action}?`;
   if (detail) {
-    lines.push('', detail);
+    message += `\n\n${detail}`;
   }
-  const message = lines.join('\n');
 
   const allowed = await cli.prompt(message, { defaultAnswer: false });
   if (!allowed) {
-    cli.info(`Aborted: ${action}.`);
-    process.exit(0);
+    throw new Error(`Aborted: ${action}.`);
   }
 }
 
@@ -97,18 +95,10 @@ export async function commitAndPushVulnerabilitiesJSON(
 
   if (Array.isArray(filePath)) {
     for (const currentPath of filePath) {
-      await runSecurityGitCommand(
-        cli,
-        ['add', currentPath],
-        `This stages ${currentPath} for the security release commit.`
-      );
+      runSync('git', ['add', currentPath]);
     }
   } else {
-    await runSecurityGitCommand(
-      cli,
-      ['add', filePath],
-      `This stages ${filePath} for the security release commit.`
-    );
+    runSync('git', ['add', filePath]);
   }
 
   const staged = runSync('git', ['diff', '--name-only', '--cached']).trim();