@automattic/vip 2.33.0 → 2.34.0

package/CHANGELOG.md

@@ -1,5 +1,34 @@
 ## Changelog
 
+### 2.34.0
+
+- #1503 chore(dev-deps): update nock to 13.3.3
+- #1502 chore(dev-deps): update flow-bin to 0.216.1
+- #1501 chore(deps): update jest-related packages
+- #1500 chore(dev-deps): update babel-related packages
+- #1498 ci: run tests and checks in parallel
+- #1462 Fixed Slowlogs local unit test
+- #1488 fix: do not abort if a temporary directory cannot be removed on exit
+- #1495 refactor: convert `media-import` to TypeScript
+- #1492 refactor: convert `site-import` directory to TypeScript
+
+### 2.33.0
+
+- #1475 Updating CONTRIBUTING.md with new publishing procedure
+- #1477 Fetch environment name from server to fix environment selector label
+- #1479 chore(dev-deps): update jest-related packages to 29.6.2
+- #1480 chore(deps): update node-fetch to 2.6.12
+- #1481 chore(deps): update socket.io-client to 4.7.2
+- #1482 chore(deps): update enquirer to 2.4.1
+- #1478 Fix False UP status for multisite dev-envs
+- #1483 Disable Windows patch
+- #1458 Add warning confirmation when there's not enough space on disk. by
+- #1487 fix(dev-env): display ports exposed by services
+- #1490 fix: CLI not respecting proxy settings when creating local dev environment by
+- #1491 chore(dev-deps): update eslint from 8.43.0 to 8.47.0
+- #1486 chore(dev-deps): update babel-related packages
+- #1484 feat(backup): Add vip backup db command
+
 ### 2.32.4
 
 - #1470 Rename prepare-release.yml to npm-prepare-release.yml

package/dist/lib/media-import/media-file-import.js

@@ -7,11 +7,6 @@ exports.SUPPORTED_MEDIA_FILE_IMPORT_SITE_TYPES = exports.MEDIA_IMPORT_FILE_SIZE_
 exports.currentUserCanImportForApp = currentUserCanImportForApp;
 exports.isSupportedApp = void 0;
 var _fileSize = require("../../lib/constants/file-size");
-/**
- * 
- * @format
- */
-
 /**
  * Internal dependencies
  */

package/dist/lib/media-import/progress.js

@@ -7,13 +7,14 @@ exports.MediaImportProgressTracker = void 0;
 var _singleLineLog = require("single-line-log");
 var _status = require("../../lib/media-import/status");
 var _format = require("../../lib/cli/format");
-/** @format */
 /**
  * External dependencies
  */
+
 /**
  * Internal dependencies
  */
+
 const PRINT_INTERVAL = process.env.DEBUG ? 5000 : 200; // How often the report is printed. Mainly affects the "spinner" animation.
 
 class MediaImportProgressTracker {
@@ -26,15 +27,20 @@ class MediaImportProgressTracker {
   constructor(status) {
     this.runningSprite = new _format.RunningSprite();
     this.hasFailure = false;
-    this.status = Object.assign({}, status);
+    this.status = {
+      ...status
+    };
     this.prefix = '';
     this.suffix = '';
+    this.hasPrinted = false;
   }
   setStatus(status) {
     if ('FAILED' === status.status) {
       this.hasFailure = true;
     }
-    this.status = Object.assign({}, status);
+    this.status = {
+      ...status
+    };
   }
   startPrinting(prePrintCallback = () => {}) {
     this.printInterval = setInterval(() => {
@@ -54,7 +60,7 @@ class MediaImportProgressTracker {
       this.hasPrinted = true;
       _singleLineLog.stdout.clear();
     }
-    const statusIcon = (0, _status.getGlyphForStatus)(this.status.status, this.runningSprite);
+    const statusIcon = (0, _status.getGlyphForStatus)(this.status.status ?? '', this.runningSprite);
     let logs;
     if (this.status.filesProcessed && this.status.filesTotal) {
       const progressPercentage = Math.floor(this.status.filesProcessed / this.status.filesTotal * 100);
@@ -62,7 +68,7 @@ class MediaImportProgressTracker {
     }
 
     // Output the logs
-    (0, _singleLineLog.stdout)(`${this.prefix || ''}${logs || ''}${this.suffix || ''}`);
+    (0, _singleLineLog.stdout)(`${this.prefix || ''}${logs ?? ''}${this.suffix || ''}`);
     if (clearAfter) {
       // Break out of the "Single log line" buffer
       _singleLineLog.stdout.clear();

package/dist/lib/media-import/status.js

@@ -6,23 +6,14 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 exports.getGlyphForStatus = getGlyphForStatus;
 exports.mediaImportCheckStatus = mediaImportCheckStatus;
+var _promises = require("node:fs/promises");
+var _nodePath = require("node:path");
 var _chalk = _interopRequireDefault(require("chalk"));
 var _graphqlTag = _interopRequireDefault(require("graphql-tag"));
-var _fs = require("fs");
-var path = _interopRequireWildcard(require("path"));
 var _api = _interopRequireDefault(require("../../lib/api"));
 var _mediaFileImport = require("../../lib/media-import/media-file-import");
-var _progress = require("../../lib/media-import/progress");
 var _format = require("../../lib/cli/format");
-var _format2 = require("../cli/format");
-function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function (nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
-function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-/**
- * 
- * @format
- */
-
 /**
  * External dependencies
  */
@@ -62,6 +53,7 @@ const IMPORT_MEDIA_PROGRESS_QUERY = (0, _graphqlTag.default)`
 	}
 `;
 async function getStatus(api, appId, envId) {
+  var _response$data$app;
   const response = await api.query({
     query: IMPORT_MEDIA_PROGRESS_QUERY,
     variables: {
@@ -70,21 +62,15 @@ async function getStatus(api, appId, envId) {
     },
     fetchPolicy: 'network-only'
   });
-  const {
-    data: {
-      app: {
-        environments
-      }
-    }
-  } = response;
+  const environments = (_response$data$app = response.data.app) === null || _response$data$app === void 0 ? void 0 : _response$data$app.environments;
   if (!(environments !== null && environments !== void 0 && environments.length)) {
     throw new Error('Unable to determine import status from environment');
   }
   const [environment] = environments;
   const {
     mediaImportStatus
-  } = environment;
-  return mediaImportStatus;
+  } = environment ?? {};
+  return mediaImportStatus ?? null;
 }
 function getGlyphForStatus(status, runningSprite) {
   switch (status) {
@@ -113,10 +99,11 @@ function buildErrorMessage(importFailed) {
   if ('FAILED' === importFailed.status) {
     const globalFailureDetails = importFailed.failureDetails;
     if (globalFailureDetails) {
+      var _globalFailureDetails;
       message += `${_chalk.default.red('Import failed at status: ')}`;
       message += `${_chalk.default.redBright.bold(globalFailureDetails.previousStatus)}\n`;
       message += _chalk.default.red('Errors:');
-      globalFailureDetails.globalErrors.forEach(value => {
+      (_globalFailureDetails = globalFailureDetails.globalErrors) === null || _globalFailureDetails === void 0 || _globalFailureDetails.forEach(value => {
         message += `\n\t- ${_chalk.default.redBright.bold(value)}`;
       });
       return message;
@@ -131,16 +118,17 @@ function buildFileErrors(fileErrors, exportFileErrorsToJson) {
   if (exportFileErrorsToJson) {
     const fileErrorsToExport = fileErrors.map(fileError => {
       return {
-        fileName: fileError.fileName,
-        errors: fileError.errors
+        fileName: fileError === null || fileError === void 0 ? void 0 : fileError.fileName,
+        errors: fileError === null || fileError === void 0 ? void 0 : fileError.errors
       };
     });
     return (0, _format.formatData)(fileErrorsToExport, 'json');
   }
   let errorString = '';
   for (const fileError of fileErrors) {
-    errorString += `File Name: ${fileError.fileName}`;
-    errorString += `\n\nErrors:\n\t- ${fileError.errors}\n\n\n\n`;
+    var _fileError$errors;
+    errorString += `File Name: ${(fileError === null || fileError === void 0 ? void 0 : fileError.fileName) ?? 'N/A'}`;
+    errorString += `\n\nErrors:\n\t- ${(fileError === null || fileError === void 0 || (_fileError$errors = fileError.errors) === null || _fileError$errors === void 0 ? void 0 : _fileError$errors.join(', ')) ?? 'unknown error'}\n\n\n\n`;
   }
   return errorString;
 }
@@ -177,7 +165,7 @@ async function mediaImportCheckStatus({
     const suffix = `
 =============================================================
 Status: ${statusMessage}
-App: ${app.name} (${(0, _format.formatEnvironment)(env.type)})
+App: ${app.name ?? 'N/A'} (${(0, _format.formatEnvironment)(env.type ?? 'N/A')})
 =============================================================
 ${maybeExitPrompt}
 `;
@@ -188,13 +176,15 @@ ${maybeExitPrompt}
     progressTracker.print();
   };
   progressTracker.startPrinting(setSuffixAndPrint);
-  const getResults = () => new Promise((resolve, reject) => {
+  const getResults = () =>
+  // eslint-disable-next-line @typescript-eslint/no-shadow
+  new Promise((resolve, reject) => {
     let startDate = Date.now();
     let pollIntervalDecreasing = false;
     const checkStatus = async pollInterval => {
-      let mediaImportStatus;
+      let mediaImportStatus = null;
       try {
-        mediaImportStatus = await getStatus(api, app.id, env.id);
+        mediaImportStatus = await getStatus(api, app.id ?? -1, env.id ?? -1);
         if (!mediaImportStatus) {
           return reject({
             error: 'Requested app/environment is not available for this operation. If you think this is not correct, please contact Support.'
@@ -202,12 +192,10 @@ ${maybeExitPrompt}
         }
       } catch (error) {
         return reject({
-          error
+          error: error.message
         });
       }
-      const {
-        status
-      } = mediaImportStatus;
+      const status = mediaImportStatus.status ?? 'unknown';
       const failedMediaImport = 'FAILED' === status;
       if (failedMediaImport) {
         progressTracker.setStatus(mediaImportStatus);
@@ -244,23 +232,23 @@ ${maybeExitPrompt}
   try {
     var _results$failureDetai;
     const results = await getResults();
-    overallStatus = (results === null || results === void 0 ? void 0 : results.status) || 'unknown';
+    overallStatus = results.status ?? 'unknown';
     progressTracker.stopPrinting();
     setProgressTrackerSuffix();
     progressTracker.print();
     const fileErrors = (_results$failureDetai = results.failureDetails) === null || _results$failureDetai === void 0 ? void 0 : _results$failureDetai.fileErrors;
     if (!!fileErrors && fileErrors.length > 0) {
       progressTracker.suffix += `${_chalk.default.yellow(`⚠️  ${fileErrors.length} file error(s) have been extracted`)}`;
-      if (results.filesTotal - results.filesProcessed !== fileErrors.length) {
+      if ((results.filesTotal ?? 0) - (results.filesProcessed ?? 0) !== fileErrors.length) {
         progressTracker.suffix += `. ${_chalk.default.italic.yellow('File-errors report size threshold reached.')}`;
       }
       const formattedData = buildFileErrors(fileErrors, exportFileErrorsToJson);
-      const errorsFile = `media-import-${app.name}-${Date.now()}${exportFileErrorsToJson ? '.json' : '.txt'}`;
+      const errorsFile = `media-import-${app.name ?? ''}-${Date.now()}${exportFileErrorsToJson ? '.json' : '.txt'}`;
       try {
-        await _fs.promises.writeFile(errorsFile, formattedData);
-        progressTracker.suffix += `\n\n${_chalk.default.yellow(`All errors have been exported to ${_chalk.default.bold(path.resolve(errorsFile))}`)}\n\n`;
+        await (0, _promises.writeFile)(errorsFile, formattedData);
+        progressTracker.suffix += `\n\n${_chalk.default.yellow(`All errors have been exported to ${_chalk.default.bold((0, _nodePath.resolve)(errorsFile))}`)}\n\n`;
       } catch (writeFileErr) {
-        progressTracker.suffix += `\n\n${_chalk.default.red(`Could not export errors to file\n${writeFileErr}`)}\n\n`;
+        progressTracker.suffix += `\n\n${_chalk.default.red(`Could not export errors to file\n${writeFileErr.message}`)}\n\n`;
       }
     }
 

package/dist/lib/site-import/db-file-import.js

@@ -7,11 +7,6 @@ exports.SYNC_STATUS_NOT_SYNCING = exports.SUPPORTED_DB_FILE_IMPORT_SITE_TYPES =
 exports.currentUserCanImportForApp = currentUserCanImportForApp;
 exports.isSupportedApp = exports.isImportingBlockedBySync = void 0;
 var _fileSize = require("../../lib/constants/file-size");
-/**
- * 
- * @format
- */
-
 /**
  * Internal dependencies
  */
@@ -32,9 +27,8 @@ const isSupportedApp = ({
 exports.isSupportedApp = isSupportedApp;
 const SYNC_STATUS_NOT_SYNCING = 'not_syncing';
 exports.SYNC_STATUS_NOT_SYNCING = SYNC_STATUS_NOT_SYNCING;
-const isImportingBlockedBySync = ({
-  syncProgress: {
-    status
-  }
-}) => status !== SYNC_STATUS_NOT_SYNCING;
+const isImportingBlockedBySync = env => {
+  var _env$syncProgress;
+  return ((_env$syncProgress = env.syncProgress) === null || _env$syncProgress === void 0 ? void 0 : _env$syncProgress.status) !== SYNC_STATUS_NOT_SYNCING;
+};
 exports.isImportingBlockedBySync = isImportingBlockedBySync;

package/dist/lib/site-import/status.js

@@ -10,17 +10,12 @@ var _graphqlTag = _interopRequireDefault(require("graphql-tag"));
 var _debug = _interopRequireDefault(require("debug"));
 var _api = _interopRequireDefault(require("../../lib/api"));
 var _dbFileImport = require("../../lib/site-import/db-file-import");
-var _progress = require("../../lib/cli/progress");
 var exit = _interopRequireWildcard(require("../../lib/cli/exit"));
 var _format = require("../../lib/cli/format");
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function (nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-/**
- * 
- * @format
- */
-
+/* eslint-disable complexity */
 /**
  * External dependencies
  */
@@ -72,6 +67,7 @@ const IMPORT_SQL_PROGRESS_QUERY = (0, _graphqlTag.default)`
 	}
 `;
 async function getStatus(api, appId, envId) {
+  var _response$data$app;
   const response = await api.query({
     query: IMPORT_SQL_PROGRESS_QUERY,
     variables: {
@@ -80,17 +76,11 @@ async function getStatus(api, appId, envId) {
     },
     fetchPolicy: 'network-only'
   });
-  const {
-    data: {
-      app: {
-        environments
-      }
-    }
-  } = response;
-  if (!(environments !== null && environments !== void 0 && environments.length)) {
+  const environments = ((_response$data$app = response.data.app) === null || _response$data$app === void 0 ? void 0 : _response$data$app.environments) ?? [];
+  if (!environments.length) {
     throw new Error('Unable to determine import status from environment');
   }
-  const [environment] = environments;
+  const environment = environments[0] ?? {};
   const {
     importStatus,
     jobs,
@@ -99,7 +89,7 @@ async function getStatus(api, appId, envId) {
   if (!environment.isK8sResident && !(jobs !== null && jobs !== void 0 && jobs.length)) {
     return {};
   }
-  const [importJob] = jobs;
+  const [importJob] = jobs ?? [];
   return {
     importStatus,
     importJob,
@@ -178,6 +168,7 @@ async function importSqlCheckStatus({
   let completedAt;
   let overallStatus = 'Checking...';
   const setProgressTrackerSuffix = () => {
+    var _env$primaryDomain;
     const sprite = (0, _format.getGlyphForStatus)(overallStatus, progressTracker.runningSprite);
     const formattedCreatedAt = createdAt ? `${new Date(createdAt).toLocaleString()} (${createdAt})` : 'TBD';
     const formattedCompletedAt = createdAt && completedAt ? `${new Date(completedAt).toLocaleString()} (${completedAt})` : 'TBD';
@@ -185,7 +176,7 @@ async function importSqlCheckStatus({
     let statusMessage;
     switch (overallStatus) {
       case 'success':
-        statusMessage = `Success ${sprite} imported data should be visible on your site ${env.primaryDomain.name}.`;
+        statusMessage = `Success ${sprite} imported data should be visible on your site ${((_env$primaryDomain = env.primaryDomain) === null || _env$primaryDomain === void 0 ? void 0 : _env$primaryDomain.name) ?? 'N/A'}.`;
         break;
       case 'running':
         if (progressTracker.allStepsSucceeded()) {
@@ -205,7 +196,7 @@ SQL Import Completed: ${formattedCompletedAt}`;
     const suffix = `
 =============================================================
 Status: ${statusMessage}
-Site: ${app.name} (${(0, _format.formatEnvironment)(env.type)})${maybeTimestamps}
+Site: ${app.name ?? 'N/A'} (${(0, _format.formatEnvironment)(env.type ?? 'N/A')})${maybeTimestamps}
 =============================================================
 ${maybeExitPrompt}
 `;
@@ -221,7 +212,7 @@ ${maybeExitPrompt}
       var _importJob$progress, _importJob$progress2;
       let status;
       try {
-        status = await getStatus(api, app.id, env.id);
+        status = await getStatus(api, app.id ?? -1, env.id ?? -1);
       } catch (error) {
         return reject({
           error
@@ -246,13 +237,15 @@ ${maybeExitPrompt}
 
           // if the progress meta isn't filled out yet, wait until it is.
           if (!statusSteps) {
-            return setTimeout(checkStatus, IMPORT_SQL_PROGRESS_POLL_INTERVAL); // NOSONAR
+            return setTimeout(() => {
+              void checkStatus();
+            }, IMPORT_SQL_PROGRESS_POLL_INTERVAL);
           }
-
           jobSteps = statusSteps.map(step => {
+            var _step$name;
             return {
               id: step.name,
-              name: (0, _format.capitalize)(step.name.replace(/_/g, ' ')),
+              name: (0, _format.capitalize)((_step$name = step.name) === null || _step$name === void 0 ? void 0 : _step$name.replace(/_/g, ' ')),
               status: step.result
             };
           });
@@ -267,7 +260,8 @@ ${maybeExitPrompt}
             result
           }) => result === 'success')) {
             jobStatus = 'success';
-            importJob.completedAt = new Date(Math.max(...statusSteps.map(step => step.finished_at), 0) * 1000).toUTCString();
+            const timestamps = statusSteps.map(step => step.finished_at).filter(Boolean);
+            importJob.completedAt = new Date(Math.max(...timestamps, 0) * 1000).toUTCString();
           }
           if (importStatus !== null && importStatus !== void 0 && (_importStatus$progres2 = importStatus.progress) !== null && _importStatus$progres2 !== void 0 && _importStatus$progres2.started_at) {
             importJob.createdAt = new Date(importStatus.progress.started_at * 1000).toUTCString();
@@ -288,7 +282,7 @@ ${maybeExitPrompt}
         dbOperationInProgress,
         importInProgress,
         progress: importStepProgress
-      } = importStatus;
+      } = importStatus ?? {};
       debug({
         jobStatus,
         completedAt,
@@ -299,14 +293,15 @@ ${maybeExitPrompt}
       });
       let jobCreationTime;
       try {
-        jobCreationTime = new Date(createdAt).getTime();
+        jobCreationTime = new Date(createdAt ?? '').getTime();
       } catch (err) {
         debug('Unable to parse createdAt to a Date');
       }
       let failedImportStep;
-      if (jobCreationTime && (importStepProgress === null || importStepProgress === void 0 ? void 0 : importStepProgress.started_at) * 1000 >= jobCreationTime) {
+      if (jobCreationTime && ((importStepProgress === null || importStepProgress === void 0 ? void 0 : importStepProgress.started_at) ?? 0) * 1000 >= jobCreationTime) {
+        var _importStepProgress$s;
         // The contents of the `import_progress` meta are pertinent to the most recent import job
-        failedImportStep = importStepProgress.steps.find(step => (step === null || step === void 0 ? void 0 : step.result) === 'failed' && 1000 * (step === null || step === void 0 ? void 0 : step.started_at) > new Date(createdAt).getTime());
+        failedImportStep = importStepProgress === null || importStepProgress === void 0 || (_importStepProgress$s = importStepProgress.steps) === null || _importStepProgress$s === void 0 ? void 0 : _importStepProgress$s.find(step => (step === null || step === void 0 ? void 0 : step.result) === 'failed' && 1000 * (step.started_at ?? 0) > new Date(createdAt ?? '').getTime());
       }
       if (!jobSteps.length) {
         return reject({
@@ -317,9 +312,7 @@ ${maybeExitPrompt}
       if (failedImportStep) {
         // The server marks the step as a success as per the host action, demote it to 'failed'
         const _jobSteps = [...jobSteps];
-        const failedJobStepIndex = _jobSteps.findIndex(({
-          id
-        }) => id === 'import');
+        const failedJobStepIndex = _jobSteps.findIndex(step => (step === null || step === void 0 ? void 0 : step.id) === 'import');
         _jobSteps[failedJobStepIndex] = {
           ..._jobSteps[failedJobStepIndex],
           status: 'failed'
@@ -332,7 +325,6 @@ ${maybeExitPrompt}
           commandOutput: failedImportStep.output,
           error: 'Import step failed',
           stepName: failedImportStep.name,
-          errorText: failedImportStep.error,
           launched
         });
       }
@@ -349,7 +341,9 @@ ${maybeExitPrompt}
         return resolve(importJob);
       }
       overallStatus = 'running';
-      setTimeout(checkStatus, IMPORT_SQL_PROGRESS_POLL_INTERVAL); // NOSONAR
+      setTimeout(() => {
+        void checkStatus();
+      }, IMPORT_SQL_PROGRESS_POLL_INTERVAL);
     };
 
     // Kick off the check
@@ -361,7 +355,7 @@ ${maybeExitPrompt}
       overallStatus = results;
     } else {
       var _results$progress;
-      overallStatus = (results === null || results === void 0 || (_results$progress = results.progress) === null || _results$progress === void 0 ? void 0 : _results$progress.status) || 'unknown';
+      overallStatus = ((_results$progress = results.progress) === null || _results$progress === void 0 ? void 0 : _results$progress.status) ?? 'unknown';
       // This shouldn't be 'unknown'...what should we do here?
     }
 

package/dist/lib/utils.js

@@ -56,11 +56,16 @@ function makeTempDir(prefix = 'vip-cli') {
   const tempDir = _fs.default.mkdtempSync(_path.default.join(_os.default.tmpdir(), `${prefix}-`));
   debug(`Created a directory to hold temporary files: ${tempDir}`);
   process.on('exit', () => {
-    _fs.default.rmSync(tempDir, {
-      recursive: true,
-      force: true
-    });
-    debug(`Removed temporary directory: ${tempDir}`);
+    try {
+      _fs.default.rmSync(tempDir, {
+        recursive: true,
+        force: true,
+        maxRetries: 10
+      });
+      debug(`Removed temporary directory: ${tempDir}`);
+    } catch (err) {
+      console.warn(`Failed to remove temporary directory ${tempDir} (${err.message})`);
+    }
   });
   return tempDir;
 }