@pnp/cli-microsoft365 5.8.0-beta.9cf7616 → 5.8.0-beta.9e55865

package/dist/m365/spfx/commands/project/project-upgrade.js

@@ -77,196 +77,190 @@ class SpfxProjectUpgradeCommand extends base_project_command_1.BaseProjectComman
     get description() {
         return 'Upgrades SharePoint Framework project to the specified version';
     }
-    commandAction(logger, args, cb) {
-        this.projectRootPath = this.getProjectRoot(process.cwd());
-        if (this.projectRootPath === null) {
-            cb(new Command_1.CommandError(`Couldn't find project root folder`, SpfxProjectUpgradeCommand.ERROR_NO_PROJECT_ROOT_FOLDER));
-            return;
-        }
-        this.toVersion = args.options.toVersion ? args.options.toVersion : this.supportedVersions[this.supportedVersions.length - 1];
-        // uncomment to support upgrading to preview releases
-        if (!args.options.toVersion &&
-            !args.options.preview &&
-            (0, semver_1.prerelease)(this.toVersion)) {
-            // no version and no preview specified while the current version to
-            // upgrade to is a prerelease so let's grab the first non-preview version
-            // since we're supporting only one preview version, it's sufficient for
-            // us to take second to last version
-            this.toVersion = this.supportedVersions[this.supportedVersions.length - 2];
-        }
-        this.packageManager = args.options.packageManager || 'npm';
-        this.shell = args.options.shell || 'bash';
-        if (this.supportedVersions.indexOf(this.toVersion) < 0) {
-            cb(new Command_1.CommandError(`CLI for Microsoft 365 doesn't support upgrading SharePoint Framework projects to version ${this.toVersion}. Supported versions are ${this.supportedVersions.join(', ')}`, SpfxProjectUpgradeCommand.ERROR_UNSUPPORTED_TO_VERSION));
-            return;
-        }
-        this.projectVersion = this.getProjectVersion();
-        if (!this.projectVersion) {
-            cb(new Command_1.CommandError(`Unable to determine the version of the current SharePoint Framework project`, SpfxProjectUpgradeCommand.ERROR_NO_VERSION));
-            return;
-        }
-        const pos = this.supportedVersions.indexOf(this.projectVersion);
-        if (pos < 0) {
-            cb(new Command_1.CommandError(`CLI for Microsoft 365 doesn't support upgrading projects built using SharePoint Framework v${this.projectVersion}`, SpfxProjectUpgradeCommand.ERROR_UNSUPPORTED_FROM_VERSION));
-            return;
-        }
-        const posTo = this.supportedVersions.indexOf(this.toVersion);
-        if (pos > posTo) {
-            cb(new Command_1.CommandError('You cannot downgrade a project', SpfxProjectUpgradeCommand.ERROR_NO_DOWNGRADE));
-            return;
-        }
-        if (pos === posTo) {
-            logger.log(`Project doesn't need to be upgraded`);
-            cb();
-            return;
-        }
-        if (this.verbose) {
-            logger.logToStderr('Collecting project...');
-        }
-        const project = this.getProject(this.projectRootPath);
-        if (this.debug) {
-            logger.logToStderr('Collected project');
-            logger.logToStderr(project);
-        }
-        // reverse the list of versions to upgrade to, so that most recent findings
-        // will end up on top already. Saves us reversing a larger array later
-        const versionsToUpgradeTo = this.supportedVersions.slice(pos + 1, posTo + 1).reverse();
-        try {
-            versionsToUpgradeTo.forEach(v => {
-                const rules = require(`./project-upgrade/upgrade-${v}`);
-                rules.forEach(r => {
-                    r.visit(project, this.allFindings);
-                });
-            });
-        }
-        catch (e) {
-            cb(new Command_1.CommandError(e.message));
-            return;
-        }
-        if (this.packageManager === 'npm') {
-            const npmDedupeRule = new FN017001_MISC_npm_dedupe_1.FN017001_MISC_npm_dedupe();
-            npmDedupeRule.visit(project, this.allFindings);
-        }
-        // dedupe
-        const findings = this.allFindings.filter((f, i) => {
-            const firstFindingPos = this.allFindings.findIndex(f1 => f1.id === f.id);
-            return i === firstFindingPos;
-        });
-        // remove superseded findings
-        findings
-            // get findings that supersede other findings
-            .filter(f => f.supersedes.length > 0)
-            .forEach(f => {
-            f.supersedes.forEach(s => {
-                // find the superseded finding
-                const i = findings.findIndex(f1 => f1.id === s);
-                if (i > -1) {
-                    // ...and remove it from findings
-                    findings.splice(i, 1);
-                }
-            });
-        });
-        // remove findings without title
-        findings
-            .forEach(f => {
-            if (!f.title) {
-                // find the finding
-                const i = findings.findIndex(f1 => f1.id === f.id);
-                if (i > -1) {
-                    // ...and remove it from findings
-                    findings.splice(i, 1);
-                }
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            this.projectRootPath = this.getProjectRoot(process.cwd());
+            if (this.projectRootPath === null) {
+                throw new Command_1.CommandError(`Couldn't find project root folder`, SpfxProjectUpgradeCommand.ERROR_NO_PROJECT_ROOT_FOLDER);
             }
-        });
-        // flatten
-        const findingsToReport = [].concat.apply([], findings.map(f => {
-            return f.occurrences.map(o => {
-                return {
-                    description: f.description,
-                    id: f.id,
-                    file: o.file,
-                    position: o.position,
-                    resolution: o.resolution,
-                    resolutionType: f.resolutionType,
-                    severity: f.severity,
-                    title: f.title
-                };
-            });
-        }));
-        // replace package operation tokens with command for the specific package manager
-        findingsToReport.forEach(f => {
-            // matches must be in this particular order to avoid false matches, eg.
-            // uninstallDev contains install
-            if (f.resolution.startsWith('uninstallDev')) {
-                f.resolution = f.resolution.replace('uninstallDev', utils_1.packageManager.getPackageManagerCommand('uninstallDev', this.packageManager));
-                return;
+            this.toVersion = args.options.toVersion ? args.options.toVersion : this.supportedVersions[this.supportedVersions.length - 1];
+            // uncomment to support upgrading to preview releases
+            if (!args.options.toVersion &&
+                !args.options.preview &&
+                (0, semver_1.prerelease)(this.toVersion)) {
+                // no version and no preview specified while the current version to
+                // upgrade to is a prerelease so let's grab the first non-preview version
+                // since we're supporting only one preview version, it's sufficient for
+                // us to take second to last version
+                this.toVersion = this.supportedVersions[this.supportedVersions.length - 2];
             }
-            if (f.resolution.startsWith('installDev')) {
-                f.resolution = f.resolution.replace('installDev', utils_1.packageManager.getPackageManagerCommand('installDev', this.packageManager));
-                return;
+            this.packageManager = args.options.packageManager || 'npm';
+            this.shell = args.options.shell || 'bash';
+            if (this.supportedVersions.indexOf(this.toVersion) < 0) {
+                throw new Command_1.CommandError(`CLI for Microsoft 365 doesn't support upgrading SharePoint Framework projects to version ${this.toVersion}. Supported versions are ${this.supportedVersions.join(', ')}`, SpfxProjectUpgradeCommand.ERROR_UNSUPPORTED_TO_VERSION);
             }
-            if (f.resolution.startsWith('uninstall')) {
-                f.resolution = f.resolution.replace('uninstall', utils_1.packageManager.getPackageManagerCommand('uninstall', this.packageManager));
-                return;
+            this.projectVersion = this.getProjectVersion();
+            if (!this.projectVersion) {
+                throw new Command_1.CommandError(`Unable to determine the version of the current SharePoint Framework project`, SpfxProjectUpgradeCommand.ERROR_NO_VERSION);
             }
-            if (f.resolution.startsWith('install')) {
-                f.resolution = f.resolution.replace('install', utils_1.packageManager.getPackageManagerCommand('install', this.packageManager));
-                return;
+            const pos = this.supportedVersions.indexOf(this.projectVersion);
+            if (pos < 0) {
+                throw new Command_1.CommandError(`CLI for Microsoft 365 doesn't support upgrading projects built using SharePoint Framework v${this.projectVersion}`, SpfxProjectUpgradeCommand.ERROR_UNSUPPORTED_FROM_VERSION);
             }
-            // copy support for multiple shells
-            if (f.resolution.startsWith('copy_cmd')) {
-                f.resolution = f.resolution.replace('copy_cmd', utils_1.fsUtil.getCopyCommand('copyCommand', this.shell));
-                f.resolution = f.resolution.replace('DestinationParam', utils_1.fsUtil.getCopyCommand('copyDestinationParam', this.shell));
-                return;
+            const posTo = this.supportedVersions.indexOf(this.toVersion);
+            if (pos > posTo) {
+                throw new Command_1.CommandError('You cannot downgrade a project', SpfxProjectUpgradeCommand.ERROR_NO_DOWNGRADE);
             }
-            // createdir support for multiple shells
-            if (f.resolution.startsWith('create_dir_cmd')) {
-                f.resolution = f.resolution.replace('create_dir_cmd', utils_1.fsUtil.getDirectoryCommand('createDirectoryCommand', this.shell));
-                f.resolution = f.resolution.replace('NameParam', utils_1.fsUtil.getDirectoryCommand('createDirectoryNameParam', this.shell));
-                f.resolution = f.resolution.replace('PathParam', utils_1.fsUtil.getDirectoryCommand('createDirectoryPathParam', this.shell));
-                f.resolution = f.resolution.replace('ItemTypeParam', utils_1.fsUtil.getDirectoryCommand('createDirectoryItemTypeParam', this.shell));
+            if (pos === posTo) {
+                logger.log(`Project doesn't need to be upgraded`);
                 return;
             }
-            // 'Add' support for multiple shells
-            if (f.resolution.startsWith('add_cmd')) {
-                const pathStart = f.resolution.indexOf('[BEFOREPATH]') + '[BEFOREPATH]'.length;
-                const pathEnd = f.resolution.indexOf('[AFTERPATH]');
-                const filePath = f.resolution.substring(pathStart, pathEnd);
-                const contentStart = f.resolution.indexOf('[BEFORECONTENT]') + '[BEFORECONTENT]'.length;
-                const contentEnd = f.resolution.indexOf('[AFTERCONTENT]');
-                const fileContent = f.resolution.substring(contentStart, contentEnd);
-                f.resolution = utils_1.fsUtil.getAddCommand('addFileCommand', this.shell);
-                f.resolution = f.resolution.replace('[FILECONTENT]', fileContent);
-                f.resolution = f.resolution.replace('[FILEPATH]', filePath);
-                f.resolution = f.resolution.replace('[BEFOREPATH]', ' ');
-                f.resolution = f.resolution.replace('[AFTERPATH]', ' ');
-                f.resolution = f.resolution.replace('[BEFORECONTENT]', ' ');
-                f.resolution = f.resolution.replace('[AFTERCONTENT]', ' ');
-                return;
+            if (this.verbose) {
+                logger.logToStderr('Collecting project...');
             }
-            // 'Remove' support for multiple shells
-            if (f.resolution.startsWith('remove_cmd')) {
-                f.resolution = f.resolution.replace('remove_cmd', utils_1.fsUtil.getRemoveCommand('removeFileCommand', this.shell));
-                return;
+            const project = this.getProject(this.projectRootPath);
+            if (this.debug) {
+                logger.logToStderr('Collected project');
+                logger.logToStderr(project);
+            }
+            // reverse the list of versions to upgrade to, so that most recent findings
+            // will end up on top already. Saves us reversing a larger array later
+            const versionsToUpgradeTo = this.supportedVersions.slice(pos + 1, posTo + 1).reverse();
+            try {
+                versionsToUpgradeTo.forEach(v => {
+                    const rules = require(`./project-upgrade/upgrade-${v}`);
+                    rules.forEach(r => {
+                        r.visit(project, this.allFindings);
+                    });
+                });
+            }
+            catch (e) {
+                throw new Command_1.CommandError(e.message);
+            }
+            if (this.packageManager === 'npm') {
+                const npmDedupeRule = new FN017001_MISC_npm_dedupe_1.FN017001_MISC_npm_dedupe();
+                npmDedupeRule.visit(project, this.allFindings);
+            }
+            // dedupe
+            const findings = this.allFindings.filter((f, i) => {
+                const firstFindingPos = this.allFindings.findIndex(f1 => f1.id === f.id);
+                return i === firstFindingPos;
+            });
+            // remove superseded findings
+            findings
+                // get findings that supersede other findings
+                .filter(f => f.supersedes.length > 0)
+                .forEach(f => {
+                f.supersedes.forEach(s => {
+                    // find the superseded finding
+                    const i = findings.findIndex(f1 => f1.id === s);
+                    if (i > -1) {
+                        // ...and remove it from findings
+                        findings.splice(i, 1);
+                    }
+                });
+            });
+            // remove findings without title
+            findings
+                .forEach(f => {
+                if (!f.title) {
+                    // find the finding
+                    const i = findings.findIndex(f1 => f1.id === f.id);
+                    if (i > -1) {
+                        // ...and remove it from findings
+                        findings.splice(i, 1);
+                    }
+                }
+            });
+            // flatten
+            const findingsToReport = [].concat.apply([], findings.map(f => {
+                return f.occurrences.map(o => {
+                    return {
+                        description: f.description,
+                        id: f.id,
+                        file: o.file,
+                        position: o.position,
+                        resolution: o.resolution,
+                        resolutionType: f.resolutionType,
+                        severity: f.severity,
+                        title: f.title
+                    };
+                });
+            }));
+            // replace package operation tokens with command for the specific package manager
+            findingsToReport.forEach(f => {
+                // matches must be in this particular order to avoid false matches, eg.
+                // uninstallDev contains install
+                if (f.resolution.startsWith('uninstallDev')) {
+                    f.resolution = f.resolution.replace('uninstallDev', utils_1.packageManager.getPackageManagerCommand('uninstallDev', this.packageManager));
+                    return;
+                }
+                if (f.resolution.startsWith('installDev')) {
+                    f.resolution = f.resolution.replace('installDev', utils_1.packageManager.getPackageManagerCommand('installDev', this.packageManager));
+                    return;
+                }
+                if (f.resolution.startsWith('uninstall')) {
+                    f.resolution = f.resolution.replace('uninstall', utils_1.packageManager.getPackageManagerCommand('uninstall', this.packageManager));
+                    return;
+                }
+                if (f.resolution.startsWith('install')) {
+                    f.resolution = f.resolution.replace('install', utils_1.packageManager.getPackageManagerCommand('install', this.packageManager));
+                    return;
+                }
+                // copy support for multiple shells
+                if (f.resolution.startsWith('copy_cmd')) {
+                    f.resolution = f.resolution.replace('copy_cmd', utils_1.fsUtil.getCopyCommand('copyCommand', this.shell));
+                    f.resolution = f.resolution.replace('DestinationParam', utils_1.fsUtil.getCopyCommand('copyDestinationParam', this.shell));
+                    return;
+                }
+                // createdir support for multiple shells
+                if (f.resolution.startsWith('create_dir_cmd')) {
+                    f.resolution = f.resolution.replace('create_dir_cmd', utils_1.fsUtil.getDirectoryCommand('createDirectoryCommand', this.shell));
+                    f.resolution = f.resolution.replace('NameParam', utils_1.fsUtil.getDirectoryCommand('createDirectoryNameParam', this.shell));
+                    f.resolution = f.resolution.replace('PathParam', utils_1.fsUtil.getDirectoryCommand('createDirectoryPathParam', this.shell));
+                    f.resolution = f.resolution.replace('ItemTypeParam', utils_1.fsUtil.getDirectoryCommand('createDirectoryItemTypeParam', this.shell));
+                    return;
+                }
+                // 'Add' support for multiple shells
+                if (f.resolution.startsWith('add_cmd')) {
+                    const pathStart = f.resolution.indexOf('[BEFOREPATH]') + '[BEFOREPATH]'.length;
+                    const pathEnd = f.resolution.indexOf('[AFTERPATH]');
+                    const filePath = f.resolution.substring(pathStart, pathEnd);
+                    const contentStart = f.resolution.indexOf('[BEFORECONTENT]') + '[BEFORECONTENT]'.length;
+                    const contentEnd = f.resolution.indexOf('[AFTERCONTENT]');
+                    const fileContent = f.resolution.substring(contentStart, contentEnd);
+                    f.resolution = utils_1.fsUtil.getAddCommand('addFileCommand', this.shell);
+                    f.resolution = f.resolution.replace('[FILECONTENT]', fileContent);
+                    f.resolution = f.resolution.replace('[FILEPATH]', filePath);
+                    f.resolution = f.resolution.replace('[BEFOREPATH]', ' ');
+                    f.resolution = f.resolution.replace('[AFTERPATH]', ' ');
+                    f.resolution = f.resolution.replace('[BEFORECONTENT]', ' ');
+                    f.resolution = f.resolution.replace('[AFTERCONTENT]', ' ');
+                    return;
+                }
+                // 'Remove' support for multiple shells
+                if (f.resolution.startsWith('remove_cmd')) {
+                    f.resolution = f.resolution.replace('remove_cmd', utils_1.fsUtil.getRemoveCommand('removeFileCommand', this.shell));
+                    return;
+                }
+            });
+            switch (args.options.output) {
+                case 'text':
+                    logger.log(this.getTextReport(findingsToReport));
+                    break;
+                case 'json':
+                    logger.log(findingsToReport);
+                    break;
+                case 'tour':
+                    this.writeReportTourFolder(this.getTourReport(findingsToReport, project));
+                    break;
+                case 'md':
+                    logger.log(this.getMdReport(findingsToReport));
+                    break;
+                default:
+                    logger.log(findingsToReport);
             }
         });
-        switch (args.options.output) {
-            case 'text':
-                logger.log(this.getTextReport(findingsToReport));
-                break;
-            case 'json':
-                logger.log(findingsToReport);
-                break;
-            case 'tour':
-                this.writeReportTourFolder(this.getTourReport(findingsToReport, project));
-                break;
-            case 'md':
-                logger.log(this.getMdReport(findingsToReport));
-                break;
-            default:
-                logger.log(findingsToReport);
-        }
-        cb();
     }
     writeReportTourFolder(findingsToReport) {
         const toursFolder = path.join(this.projectRootPath, '.tours');

package/dist/m365/spfx/commands/spfx-doctor.js

@@ -18,7 +18,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const chalk = require("chalk");
 const child_process = require("child_process");
 const semver_1 = require("semver");
-const Command_1 = require("../../../Command");
 const AnonymousCommand_1 = require("../../base/AnonymousCommand");
 const commands_1 = require("../commands");
 /**
@@ -442,58 +441,54 @@ class SpfxDoctorCommand extends AnonymousCommand_1.default {
     get description() {
         return 'Verifies environment configuration for using the specific version of the SharePoint Framework';
     }
-    commandAction(logger, args, cb) {
-        if (!args.options.output) {
-            args.options.output = 'text';
-        }
-        logger.log(' ');
-        logger.log('CLI for Microsoft 365 SharePoint Framework doctor');
-        logger.log('Verifying configuration of your system for working with the SharePoint Framework');
-        logger.log(' ');
-        let spfxVersion = '';
-        let prerequisites;
-        const fixes = [];
-        this
-            .getSharePointFrameworkVersion(logger)
-            .then((_spfxVersion) => {
-            if (!_spfxVersion) {
-                logger.log(this.getStatus(CheckStatus.Failure, `SharePoint Framework`));
-                return Promise.reject(`SharePoint Framework not found`);
-            }
-            spfxVersion = _spfxVersion;
-            prerequisites = this.versions[spfxVersion];
-            if (!prerequisites) {
-                logger.log(this.getStatus(CheckStatus.Failure, `SharePoint Framework v${spfxVersion}`));
-                return Promise.reject(`spfx doctor doesn't support SPFx v${spfxVersion} at this moment`);
-            }
-            logger.log(this.getStatus(CheckStatus.Success, `SharePoint Framework v${spfxVersion}`));
-            return Promise.resolve();
-        })
-            .then(_ => this.checkSharePointCompatibility(spfxVersion, prerequisites, args, fixes, logger))
-            .then(_ => this.checkNodeVersion(prerequisites, fixes, logger))
-            .then(_ => this.checkYo(prerequisites, fixes, logger))
-            .then(_ => this.checkGulp(fixes, logger))
-            .then(_ => this.checkGulpCli(prerequisites, fixes, logger))
-            .then(_ => this.checkTypeScript(fixes, logger))
-            .then(_ => {
-            if (fixes.length > 0) {
-                logger.log(' ');
-                logger.log('Recommended fixes:');
-                logger.log(' ');
-                fixes.forEach(f => logger.log(`- ${f}`));
-                logger.log(' ');
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            if (!args.options.output) {
+                args.options.output = 'text';
             }
-            cb();
-        })
-            .catch((error) => {
             logger.log(' ');
-            if (fixes.length > 0) {
-                logger.log('Recommended fixes:');
-                logger.log(' ');
-                fixes.forEach(f => logger.log(`- ${f}`));
+            logger.log('CLI for Microsoft 365 SharePoint Framework doctor');
+            logger.log('Verifying configuration of your system for working with the SharePoint Framework');
+            logger.log(' ');
+            let spfxVersion = '';
+            let prerequisites;
+            const fixes = [];
+            try {
+                spfxVersion = yield this.getSharePointFrameworkVersion(logger);
+                if (!spfxVersion) {
+                    logger.log(this.getStatus(CheckStatus.Failure, `SharePoint Framework`));
+                    throw `SharePoint Framework not found`;
+                }
+                prerequisites = this.versions[spfxVersion];
+                if (!prerequisites) {
+                    logger.log(this.getStatus(CheckStatus.Failure, `SharePoint Framework v${spfxVersion}`));
+                    throw `spfx doctor doesn't support SPFx v${spfxVersion} at this moment`;
+                }
+                logger.log(this.getStatus(CheckStatus.Success, `SharePoint Framework v${spfxVersion}`));
+                yield this.checkSharePointCompatibility(spfxVersion, prerequisites, args, fixes, logger);
+                yield this.checkNodeVersion(prerequisites, fixes, logger);
+                yield this.checkYo(prerequisites, fixes, logger);
+                yield this.checkGulp(fixes, logger);
+                yield this.checkGulpCli(prerequisites, fixes, logger);
+                yield this.checkTypeScript(fixes, logger);
+                if (fixes.length > 0) {
+                    logger.log(' ');
+                    logger.log('Recommended fixes:');
+                    logger.log(' ');
+                    fixes.forEach(f => logger.log(`- ${f}`));
+                    logger.log(' ');
+                }
+            }
+            catch (err) {
                 logger.log(' ');
+                if (fixes.length > 0) {
+                    logger.log('Recommended fixes:');
+                    logger.log(' ');
+                    fixes.forEach(f => logger.log(`- ${f}`));
+                    logger.log(' ');
+                }
+                this.handleRejectedPromise(err);
             }
-            cb(new Command_1.CommandError(error));
         });
     }
     checkSharePointCompatibility(spfxVersion, prerequisites, args, fixes, logger) {

package/dist/m365/spo/commands/app/app-add.js

@@ -35,40 +35,39 @@ class SpoAppAddCommand extends SpoAppBaseCommand_1.SpoAppBaseCommand {
     get description() {
         return 'Adds an app to the specified SharePoint Online app catalog';
     }
-    commandAction(logger, args, cb) {
-        const scope = (args.options.scope) ? args.options.scope.toLowerCase() : 'tenant';
-        const overwrite = args.options.overwrite || false;
-        utils_1.spo
-            .getSpoUrl(logger, this.debug)
-            .then((spoUrl) => {
-            return this.getAppCatalogSiteUrl(logger, spoUrl, args);
-        })
-            .then((appCatalogUrl) => {
-            const fullPath = path.resolve(args.options.filePath);
-            if (this.verbose) {
-                logger.logToStderr(`Adding app '${fullPath}' to app catalog...`);
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            const scope = (args.options.scope) ? args.options.scope.toLowerCase() : 'tenant';
+            const overwrite = args.options.overwrite || false;
+            try {
+                const spoUrl = yield utils_1.spo.getSpoUrl(logger, this.debug);
+                const appCatalogUrl = yield this.getAppCatalogSiteUrl(logger, spoUrl, args);
+                const fullPath = path.resolve(args.options.filePath);
+                if (this.verbose) {
+                    logger.logToStderr(`Adding app '${fullPath}' to app catalog...`);
+                }
+                const fileName = path.basename(fullPath);
+                const requestOptions = {
+                    url: `${appCatalogUrl}/_api/web/${scope}appcatalog/Add(overwrite=${(overwrite.toString().toLowerCase())}, url='${fileName}')`,
+                    headers: {
+                        accept: 'application/json;odata=nometadata',
+                        binaryStringRequestBody: 'true'
+                    },
+                    data: fs.readFileSync(fullPath)
+                };
+                const res = yield request_1.default.post(requestOptions);
+                const json = JSON.parse(res);
+                if (args.options.output === 'json') {
+                    logger.log(json);
+                }
+                else {
+                    logger.log(json.UniqueId);
+                }
             }
-            const fileName = path.basename(fullPath);
-            const requestOptions = {
-                url: `${appCatalogUrl}/_api/web/${scope}appcatalog/Add(overwrite=${(overwrite.toString().toLowerCase())}, url='${fileName}')`,
-                headers: {
-                    accept: 'application/json;odata=nometadata',
-                    binaryStringRequestBody: 'true'
-                },
-                data: fs.readFileSync(fullPath)
-            };
-            return request_1.default.post(requestOptions);
-        })
-            .then((res) => {
-            const json = JSON.parse(res);
-            if (args.options.output === 'json') {
-                logger.log(json);
+            catch (err) {
+                this.handleRejectedODataPromise(err);
             }
-            else {
-                logger.log(json.UniqueId);
-            }
-            cb();
-        }, (rawRes) => this.handleRejectedODataPromise(rawRes, logger, cb));
+        });
     }
 }
 _SpoAppAddCommand_instances = new WeakSet(), _SpoAppAddCommand_initTelemetry = function _SpoAppAddCommand_initTelemetry() {

package/dist/m365/spo/commands/app/app-deploy.js

@@ -34,54 +34,53 @@ class SpoAppDeployCommand extends SpoAppBaseCommand_1.SpoAppBaseCommand {
     get description() {
         return 'Deploys the specified app in the specified app catalog';
     }
-    commandAction(logger, args, cb) {
-        let appId = '';
-        const scope = (args.options.scope) ? args.options.scope.toLowerCase() : 'tenant';
-        let appCatalogUrl = '';
-        utils_1.spo
-            .getSpoUrl(logger, this.debug)
-            .then((spoUrl) => {
-            return this.getAppCatalogSiteUrl(logger, spoUrl, args);
-        })
-            .then((_appCatalogUrl) => {
-            appCatalogUrl = _appCatalogUrl;
-            if (args.options.id) {
-                if (this.verbose) {
-                    logger.logToStderr(`Using the specified app id ${args.options.id}`);
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            let appId = '';
+            const scope = (args.options.scope) ? args.options.scope.toLowerCase() : 'tenant';
+            let appCatalogUrl = '';
+            try {
+                const spoUrl = yield utils_1.spo.getSpoUrl(logger, this.debug);
+                appCatalogUrl = yield this.getAppCatalogSiteUrl(logger, spoUrl, args);
+                let res;
+                if (args.options.id) {
+                    if (this.verbose) {
+                        logger.logToStderr(`Using the specified app id ${args.options.id}`);
+                    }
+                    res = { UniqueId: args.options.id };
                 }
-                return Promise.resolve({ UniqueId: args.options.id });
-            }
-            else {
+                else {
+                    if (this.verbose) {
+                        logger.logToStderr(`Looking up app id for app named ${args.options.name}...`);
+                    }
+                    const requestOptions = {
+                        url: `${appCatalogUrl}/_api/web/getfolderbyserverrelativeurl('AppCatalog')/files('${args.options.name}')?$select=UniqueId`,
+                        headers: {
+                            accept: 'application/json;odata=nometadata'
+                        },
+                        responseType: 'json'
+                    };
+                    res = yield request_1.default.get(requestOptions);
+                }
+                appId = res.UniqueId;
                 if (this.verbose) {
-                    logger.logToStderr(`Looking up app id for app named ${args.options.name}...`);
+                    logger.logToStderr(`Deploying app...`);
                 }
                 const requestOptions = {
-                    url: `${appCatalogUrl}/_api/web/getfolderbyserverrelativeurl('AppCatalog')/files('${args.options.name}')?$select=UniqueId`,
+                    url: `${appCatalogUrl}/_api/web/${scope}appcatalog/AvailableApps/GetById('${appId}')/deploy`,
                     headers: {
-                        accept: 'application/json;odata=nometadata'
+                        accept: 'application/json;odata=nometadata',
+                        'content-type': 'application/json;odata=nometadata;charset=utf-8'
                     },
+                    data: { 'skipFeatureDeployment': args.options.skipFeatureDeployment || false },
                     responseType: 'json'
                 };
-                return request_1.default.get(requestOptions);
+                yield request_1.default.post(requestOptions);
             }
-        })
-            .then((res) => {
-            appId = res.UniqueId;
-            if (this.verbose) {
-                logger.logToStderr(`Deploying app...`);
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
             }
-            const requestOptions = {
-                url: `${appCatalogUrl}/_api/web/${scope}appcatalog/AvailableApps/GetById('${appId}')/deploy`,
-                headers: {
-                    accept: 'application/json;odata=nometadata',
-                    'content-type': 'application/json;odata=nometadata;charset=utf-8'
-                },
-                data: { 'skipFeatureDeployment': args.options.skipFeatureDeployment || false },
-                responseType: 'json'
-            };
-            return request_1.default.post(requestOptions);
-        })
-            .then(_ => cb(), (rawRes) => this.handleRejectedODataJsonPromise(rawRes, logger, cb));
+        });
     }
 }
 _SpoAppDeployCommand_instances = new WeakSet(), _SpoAppDeployCommand_initTelemetry = function _SpoAppDeployCommand_initTelemetry() {