Fix prompt handling causing duplicate messages to be shown for export sql and dev-env sync

[abdullah-kasim]

Description

Fixes an issue with prompt handling that introduces a regression with the promptless flow.

The bug is as per below:

$ vip export sql @0000.production
✓ Latest backup found with timestamp 2023-11-06T10:54:14.000Z
Exporting database backup with timestamp 2023-11-06T10:54:14.000Z
✓ Preparing for backup download
✓ Creating backup copy
⠦ Checking if there's enough storage <-- shouldn't repeat
○ Requesting download link <-- shouldn't repeat
○ Downloading file <-- shouldn't repeat
✓ Checking if there's enough storage
✓ Requesting download link
✓ Downloading file - 100.00% (52.34 KB/52.34 KB)
File saved to /some/path/some-backup.sql.gz

This PR removes the lines that say 'shouldn't repeat'

Pull request checklist

• Update SETUP.md with any new environmental variables.
• Update the documentation.
• Manually test the relevant changes.
• Follow the pull request checklist
• Add/update automated tests as needed.

New release checklist

• Automated tests pass.
• The Preparing for release checklist is completed.

Steps to Test

1. Check out PR.
2. Make sure you have a dev-env running - see here
3. Run npm run build:watch
4. Run ./dist/bin/vip-dev-env-sync-sql.js @0000.production --slug=local-env. Replace 0000 with your app. Bug should no longer be there.
5. Run ./dist/bin/vip-export-sql.js @0000.production. Bug should no longer be there.

For regression test - do step 4 and 5 again, but force the prompt to always show by overriding the conditionals at here, here, and here.

[aswasif007]

Reviewed and tested. Worked fine for me. Thanks for addressing this.

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud