@pnp/cli-microsoft365 7.0.0 → 7.1.0-beta.129c6c7

package/.devcontainer/Dockerfile

@@ -6,7 +6,6 @@ LABEL name="CLI for Microsoft 365 Development" \
   maintainers="Waldek Mastykarz <waldek@mastykarz.nl>, \
   Garry Trinder <garry.trinder@live.com>, \
   Albert-Jan Schot <appie@digiwijs.nl>, \
-  Rabia Williams <rabiawilliams@gmail.com>, \
   Patrick Lamber <patrick@nubo.eu>, \
   Arjun Menon <arjun.umenon@gmail.com>, \
   Adam Wojcik <adam.wojcik.it@gmail.com>, \

package/.eslintrc.cjs

@@ -136,6 +136,7 @@ module.exports = {
   "rules": {
     "cli-microsoft365/correct-command-class-name": ["error", sortedDictionary, capitalized],
     "cli-microsoft365/correct-command-name": "error",
+    "cli-microsoft365/no-by-server-relative-url-usage": "error",
     "indent": "off",
     "@typescript-eslint/indent": [
       "error",

package/Dockerfile

@@ -8,7 +8,6 @@ LABEL name="m365pnp/cli-microsoft365:${CLI_VERSION}" \
   maintainers="Waldek Mastykarz <waldek@mastykarz.nl>, \
   Garry Trinder <garry.trinder@live.com>, \
   Albert-Jan Schot <appie@digiwijs.nl>, \
-  Rabia Williams <rabiawilliams@gmail.com>, \
   Patrick Lamber <patrick@nubo.eu>, \
   Arjun Menon <arjun.umenon@gmail.com>, \
   Adam Wojcik <adam.wojcik.it@gmail.com>, \

package/dist/Auth.js

@@ -341,7 +341,7 @@ export class Auth {
         const cli = Cli.getInstance();
         cli.spinner.text = response.message;
         cli.spinner.spinner = {
-            frames: ['🌶️  ']
+            frames: ['🌶️']
         };
         // don't show spinner if running tests
         /* c8 ignore next 3 */

package/dist/Command.js

@@ -12,6 +12,7 @@ import { settingsNames } from './settingsNames.js';
 import { telemetry } from './telemetry.js';
 import { accessToken } from './utils/accessToken.js';
 import { md } from './utils/md.js';
+import { prompt } from './utils/prompt.js';
 export class CommandError {
     constructor(message, code) {
         this.message = message;
@@ -75,7 +76,6 @@ class Command {
     }
     async validateRequiredOptions(args, command) {
         const shouldPrompt = Cli.getInstance().getSettingWithDefaultValue(settingsNames.prompt, true);
-        let inquirer;
         let prompted = false;
         for (let i = 0; i < command.options.length; i++) {
             if (!command.options[i].required ||
@@ -87,19 +87,17 @@ class Command {
             }
             if (!prompted) {
                 prompted = true;
-                Cli.log('Provide values for the following parameters:');
+                Cli.error('🌶️  Provide values for the following parameters:');
             }
-            if (!inquirer) {
-                inquirer = await import('inquirer');
-            }
-            const missingRequireOptionValue = await inquirer.default
-                .prompt({
+            const missingRequireOptionValue = await prompt.forInput({
                 name: 'missingRequireOptionValue',
                 message: `${command.options[i].name}: `
-            })
-                .then(result => result.missingRequireOptionValue);
+            }).then(result => result.missingRequireOptionValue);
             args.options[command.options[i].name] = missingRequireOptionValue;
         }
+        if (prompted) {
+            Cli.error('');
+        }
         this.processOptions(args.options);
         return true;
     }
@@ -108,7 +106,6 @@ class Command {
         if (!optionsSets || optionsSets.length === 0) {
             return true;
         }
-        let inquirer;
         const shouldPrompt = Cli.getInstance().getSettingWithDefaultValue(settingsNames.prompt, true);
         const argsOptions = Object.keys(args.options);
         for (const optionSet of optionsSets.sort(opt => opt.runsWhen ? 0 : 1)) {
@@ -120,50 +117,43 @@ class Command {
                 if (!shouldPrompt) {
                     return `Specify one of the following options: ${optionSet.options.join(', ')}.`;
                 }
-                await this.promptForOptionSetNameAndValue(args, optionSet, inquirer);
+                await this.promptForOptionSetNameAndValue(args, optionSet);
             }
             if (commonOptions.length > 1) {
                 if (!shouldPrompt) {
                     return `Specify one of the following options: ${optionSet.options.join(', ')}, but not multiple.`;
                 }
-                await this.promptForSpecificOption(args, commonOptions, inquirer);
+                await this.promptForSpecificOption(args, commonOptions);
             }
         }
         return true;
     }
-    async promptForOptionSetNameAndValue(args, optionSet, inquirer) {
-        if (!inquirer) {
-            inquirer = await import('inquirer');
-        }
-        Cli.log(`Please specify one of the following options:`);
-        const resultOptionName = await inquirer.default.prompt({
+    async promptForOptionSetNameAndValue(args, optionSet) {
+        Cli.error(`🌶️  Please specify one of the following options:`);
+        const resultOptionName = await prompt.forInput({
             type: 'list',
             name: 'missingRequiredOptionName',
             message: `Option to use:`,
             choices: optionSet.options
         });
         const missingRequiredOptionName = resultOptionName.missingRequiredOptionName;
-        const resultOptionValue = await inquirer.default
-            .prompt({
+        const resultOptionValue = await prompt.forInput({
             name: 'missingRequiredOptionValue',
             message: `${missingRequiredOptionName}:`
         });
         args.options[missingRequiredOptionName] = resultOptionValue.missingRequiredOptionValue;
-        Cli.log();
+        Cli.error('');
     }
-    async promptForSpecificOption(args, commonOptions, inquirer) {
-        if (!inquirer) {
-            inquirer = await import('inquirer');
-        }
-        Cli.log(`Multiple options for an option set specified. Please specify the correct option that you wish to use.`);
-        const requiredOptionNameResult = await inquirer.default.prompt({
+    async promptForSpecificOption(args, commonOptions) {
+        Cli.error(`🌶️  Multiple options for an option set specified. Please specify the correct option that you wish to use.`);
+        const requiredOptionNameResult = await prompt.forInput({
             type: 'list',
             name: 'missingRequiredOptionName',
             message: `Option to use:`,
             choices: commonOptions
         });
         commonOptions.filter(y => y !== requiredOptionNameResult.missingRequiredOptionName).map(optionName => args.options[optionName] = undefined);
-        Cli.log();
+        Cli.error('');
     }
     async validateOutput(args) {
         if (args.options.output &&

package/dist/appInsights.js

@@ -9,7 +9,6 @@ import fs from 'fs';
 import path from 'path';
 import url from 'url';
 import { app } from './utils/app.js';
-import { pid } from './utils/pid.js';
 const __dirname = url.fileURLToPath(new URL('.', import.meta.url));
 const config = appInsights.setup('6b908c80-d09f-4cf6-8274-e54349a0149a');
 config.setInternalLogging(false, false);
@@ -18,17 +17,14 @@ config.setInternalLogging(false, false);
 // in the telemetry
 const version = `${app.packageJson().version}${fs.existsSync(path.join(__dirname, `..${path.sep}src`)) ? '-dev' : ''}`;
 const env = process.env.CLIMICROSOFT365_ENV !== undefined ? process.env.CLIMICROSOFT365_ENV : '';
-appInsights.default.defaultClient.commonProperties = {
+const appInsightsClient = appInsights.default.defaultClient;
+appInsightsClient.commonProperties = {
     version: version,
     node: process.version,
-    shell: pid.getProcessName(process.ppid) || '',
     env: env,
     ci: Boolean(process.env.CI).toString()
 };
-appInsights.default.defaultClient.context.tags['ai.session.id'] =
-    crypto.randomBytes(24).toString('base64');
-appInsights.default.defaultClient.context.tags['ai.cloud.roleInstance'] =
-    crypto.createHash('sha256').update(appInsights.default.defaultClient.context.tags['ai.cloud.roleInstance']).digest('hex');
-delete appInsights.default.defaultClient.context.tags['ai.cloud.role'];
-export default appInsights.default.defaultClient;
+appInsightsClient.context.tags['ai.cloud.roleInstance'] = crypto.createHash('sha256').update(appInsightsClient.context.tags['ai.cloud.roleInstance']).digest('hex');
+appInsightsClient.context.tags['ai.cloud.role'];
+export default appInsightsClient;
 //# sourceMappingURL=appInsights.js.map

package/dist/cli/Cli.js

@@ -15,6 +15,7 @@ import { formatting } from '../utils/formatting.js';
 import { fsUtil } from '../utils/fsUtil.js';
 import { md } from '../utils/md.js';
 import { validation } from '../utils/validation.js';
+import { prompt } from '../utils/prompt.js';
 export class Cli {
     get config() {
         if (!this._config) {
@@ -31,11 +32,15 @@ export class Cli {
             return configuredValue;
         }
     }
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
     constructor() {
         this.commands = [];
         this.commandsFolder = '';
-        this.spinner = ora('Running command...');
+        const options = {
+            text: 'Running command...',
+            /* c8 ignore next 1 */
+            stream: this.getSettingWithDefaultValue('errorOutput', 'stderr') === 'stderr' ? process.stderr : process.stdout
+        };
+        this.spinner = ora(options);
     }
     static getInstance() {
         if (!Cli.instance) {
@@ -799,14 +804,13 @@ export class Cli {
         }
     }
     static async prompt(options, answers) {
-        const inquirer = await import('inquirer');
         const cli = Cli.getInstance();
         const spinnerSpinning = cli.spinner.isSpinning;
         /* c8 ignore next 3 */
         if (spinnerSpinning) {
             cli.spinner.stop();
         }
-        const response = await inquirer.default.prompt(options, answers);
+        const response = await prompt.forInput(options, answers);
         // Restart the spinner if it was running before the prompt
         /* c8 ignore next 3 */
         if (spinnerSpinning) {
@@ -823,6 +827,7 @@ export class Cli {
             type: 'list',
             name: 'select',
             default: 0,
+            prefix: '🌶️  ',
             message: `${message} Please choose one:`,
             choices: Object.keys(values)
         });

package/dist/m365/aad/commands/m365group/m365group-remove.js

@@ -10,16 +10,21 @@ import { aadGroup } from '../../../../utils/aadGroup.js';
 import { validation } from '../../../../utils/validation.js';
 import GraphCommand from '../../../base/GraphCommand.js';
 import commands from '../../commands.js';
+import config from '../../../../config.js';
+import { formatting } from '../../../../utils/formatting.js';
+import { spo } from '../../../../utils/spo.js';
+import { setTimeout } from 'timers/promises';
 class AadM365GroupRemoveCommand extends GraphCommand {
     get name() {
         return commands.M365GROUP_REMOVE;
     }
     get description() {
-        return 'Removes an Microsoft 365 Group';
+        return 'Removes a Microsoft 365 Group';
     }
     constructor() {
         super();
         _AadM365GroupRemoveCommand_instances.add(this);
+        this.intervalInMs = 5000;
         __classPrivateFieldGet(this, _AadM365GroupRemoveCommand_instances, "m", _AadM365GroupRemoveCommand_initTelemetry).call(this);
         __classPrivateFieldGet(this, _AadM365GroupRemoveCommand_instances, "m", _AadM365GroupRemoveCommand_initOptions).call(this);
         __classPrivateFieldGet(this, _AadM365GroupRemoveCommand_instances, "m", _AadM365GroupRemoveCommand_initValidators).call(this);
@@ -34,21 +39,13 @@ class AadM365GroupRemoveCommand extends GraphCommand {
                 if (!isUnifiedGroup) {
                     throw Error(`Specified group with id '${args.options.id}' is not a Microsoft 365 group.`);
                 }
-                const requestOptions = {
-                    url: `${this.resource}/v1.0/groups/${args.options.id}`,
-                    headers: {
-                        'accept': 'application/json;odata.metadata=none'
-                    }
-                };
-                await request.delete(requestOptions);
+                const siteUrl = await this.getM365GroupSiteUrl(logger, args.options.id);
+                const spoAdminUrl = await spo.getSpoAdminUrl(logger, this.debug);
+                // Delete the Microsoft 365 group site. This operation will also delete the group.
+                await this.deleteM365GroupSite(logger, siteUrl, spoAdminUrl);
                 if (args.options.skipRecycleBin) {
-                    const requestOptions2 = {
-                        url: `${this.resource}/v1.0/directory/deletedItems/${args.options.id}`,
-                        headers: {
-                            'accept': 'application/json;odata.metadata=none'
-                        }
-                    };
-                    await request.delete(requestOptions2);
+                    await this.deleteM365GroupFromRecycleBin(logger, args.options.id);
+                    await this.deleteSiteFromRecycleBin(logger, siteUrl, spoAdminUrl);
                 }
             }
             catch (err) {
@@ -70,6 +67,102 @@ class AadM365GroupRemoveCommand extends GraphCommand {
             }
         }
     }
+    async getM365GroupSiteUrl(logger, id) {
+        if (this.verbose) {
+            await logger.logToStderr(`Getting the site URL of Microsoft 365 Group: ${id}...`);
+        }
+        const requestOptions = {
+            url: `${this.resource}/v1.0/groups/${id}/drive?$select=webUrl`,
+            headers: {
+                accept: 'application/json;odata.metadata=none'
+            },
+            responseType: 'json'
+        };
+        const res = await request.get(requestOptions);
+        // Extract the base URL by removing everything after the last '/' character in the URL.
+        const baseUrl = res.webUrl.substring(0, res.webUrl.lastIndexOf('/'));
+        return baseUrl;
+    }
+    async deleteM365GroupSite(logger, url, spoAdminUrl) {
+        if (this.verbose) {
+            await logger.logToStderr(`Deleting the group site: '${url}'...`);
+        }
+        const requestOptions = {
+            url: `${spoAdminUrl}/_api/GroupSiteManager/Delete?siteUrl='${url}'`,
+            headers: {
+                accept: 'application/json;odata=nometadata'
+            },
+            responseType: 'json'
+        };
+        await request.post(requestOptions);
+    }
+    async deleteM365GroupFromRecycleBin(logger, id) {
+        for (let retries = 0; retries < AadM365GroupRemoveCommand.maxRetries; retries++) {
+            if (await this.isM365GroupInDeletedItemsList(id)) {
+                await this.removeM365GroupPermanently(logger, id);
+                return;
+            }
+            else {
+                if (this.verbose) {
+                    await logger.logToStderr(`Group has not been deleted yet. Waiting for ${this.intervalInMs / 1000} seconds before the next attempt. ${AadM365GroupRemoveCommand.maxRetries - retries} attempts remaining...`);
+                }
+                await setTimeout(this.intervalInMs);
+            }
+        }
+        await logger.logToStderr(`Group has been successfully deleted, but it couldn't be permanently removed from the recycle bin after all retries. It will still appear in the deleted groups list.`);
+    }
+    async removeM365GroupPermanently(logger, id) {
+        if (this.verbose) {
+            await logger.logToStderr(`Group has been deleted and is now available in the deleted groups list. Removing permanently...`);
+        }
+        const requestOptions = {
+            url: `${this.resource}/v1.0/directory/deletedItems/${id}`,
+            headers: {
+                accept: 'application/json;odata.metadata=none'
+            }
+        };
+        await request.delete(requestOptions);
+    }
+    async isM365GroupInDeletedItemsList(id) {
+        const requestOptions = {
+            url: `${this.resource}/v1.0/directory/deletedItems/${id}`,
+            headers: {
+                accept: 'application/json;odata.metadata=none'
+            },
+            responseType: 'json'
+        };
+        try {
+            const response = await request.get(requestOptions);
+            return Boolean(response && response.id);
+        }
+        catch (error) {
+            if (error.response && error.response.status === 404) {
+                return false;
+            }
+            else {
+                throw error;
+            }
+        }
+    }
+    async deleteSiteFromRecycleBin(logger, url, spoAdminUrl) {
+        if (this.verbose) {
+            await logger.logToStderr(`Deleting the M365 group site '${url}' from the recycle bin...`);
+        }
+        const res = await spo.ensureFormDigest(spoAdminUrl, logger, undefined, this.debug);
+        const requestOptions = {
+            url: `${spoAdminUrl}/_vti_bin/client.svc/ProcessQuery`,
+            headers: {
+                'X-RequestDigest': res.FormDigestValue
+            },
+            data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><ObjectPath Id="185" ObjectPathId="184" /><Query Id="186" ObjectPathId="184"><Query SelectAllProperties="false"><Properties><Property Name="IsComplete" ScalarProperty="true" /><Property Name="PollingInterval" ScalarProperty="true" /></Properties></Query></Query></Actions><ObjectPaths><Method Id="184" ParentId="175" Name="RemoveDeletedSite"><Parameters><Parameter Type="String">${formatting.escapeXml(url)}</Parameter></Parameters></Method><Constructor Id="175" TypeId="{268004ae-ef6b-4e9b-8425-127220d84719}" /></ObjectPaths></Request>`
+        };
+        const processQuery = await request.post(requestOptions);
+        const json = JSON.parse(processQuery);
+        const response = json[0];
+        if (response.ErrorInfo) {
+            throw response.ErrorInfo.ErrorMessage;
+        }
+    }
 }
 _AadM365GroupRemoveCommand_instances = new WeakSet(), _AadM365GroupRemoveCommand_initTelemetry = function _AadM365GroupRemoveCommand_initTelemetry() {
     this.telemetry.push((args) => {
@@ -94,5 +187,6 @@ _AadM365GroupRemoveCommand_instances = new WeakSet(), _AadM365GroupRemoveCommand
         return true;
     });
 };
+AadM365GroupRemoveCommand.maxRetries = 10;
 export default new AadM365GroupRemoveCommand();
 //# sourceMappingURL=m365group-remove.js.map

package/dist/m365/commands/setup.js

@@ -12,6 +12,7 @@ import { pid } from '../../utils/pid.js';
 import AnonymousCommand from '../base/AnonymousCommand.js';
 import commands from './commands.js';
 import { interactivePreset, powerShellPreset, scriptingPreset } from './setupPresets.js';
+import { prompt } from '../../utils/prompt.js';
 class SetupCommand extends AnonymousCommand {
     get name() {
         return commands.SETUP;
@@ -48,7 +49,7 @@ class SetupCommand extends AnonymousCommand {
         await logger.logToStderr(`This command will guide you through the process of configuring the CLI for your needs.`);
         await logger.logToStderr(`Please, answer the following questions and we'll define a set of settings to best match how you intend to use the CLI.`);
         await logger.logToStderr('');
-        const preferences = await Cli.prompt([
+        const preferences = await prompt.forInput([
             {
                 type: 'list',
                 name: 'usageMode',

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

@@ -43,7 +43,7 @@ class SpoAppDeployCommand extends SpoAppBaseCommand {
                     await 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`,
+                    url: `${appCatalogUrl}/_api/web/GetFolderByServerRelativePath(DecodedUrl='AppCatalog')/files('${args.options.name}')?$select=UniqueId`,
                     headers: {
                         accept: 'application/json;odata=nometadata'
                     },

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

@@ -39,7 +39,7 @@ class SpoAppGetCommand extends SpoAppBaseCommand {
                     await logger.logToStderr(`Looking up app id for app named ${args.options.name}...`);
                 }
                 const requestOptions = {
-                    url: `${appCatalogSiteUrl}/_api/web/getfolderbyserverrelativeurl('AppCatalog')/files('${args.options.name}')?$select=UniqueId`,
+                    url: `${appCatalogSiteUrl}/_api/web/GetFolderByServerRelativePath(DecodedUrl='AppCatalog')/files('${args.options.name}')?$select=UniqueId`,
                     headers: {
                         accept: 'application/json;odata=nometadata'
                     },

package/dist/m365/spo/commands/app/app-teamspackage-download.js

@@ -88,7 +88,7 @@ class SpoAppTeamsPackageDownloadCommand extends SpoAppBaseCommand {
             url += `GetList('${serverRelativeAppCatalogListUrl}')/GetItemById(${args.options.appItemId})?$expand=File&$select=File/Name`;
         }
         else if (args.options.appName) {
-            url += `getfolderbyserverrelativeurl('${appCatalogListName}')/files('${formatting.encodeQueryParameter(args.options.appName)}')/ListItemAllFields?$select=Id`;
+            url += `GetFolderByServerRelativePath(DecodedUrl='${appCatalogListName}')/files('${formatting.encodeQueryParameter(args.options.appName)}')/ListItemAllFields?$select=Id`;
         }
         const requestOptions = {
             url,

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

@@ -38,7 +38,7 @@ class SpoAppPageAddCommand extends SpoCommand {
             const page = await request.post(createPageRequestOptions);
             const pageUrl = page.value;
             let requestOptions = {
-                url: `${args.options.webUrl}/_api/web/getfilebyserverrelativeurl('${urlUtil.getServerRelativeSiteUrl(args.options.webUrl)}/${pageUrl}')?$expand=ListItemAllFields`,
+                url: `${args.options.webUrl}/_api/web/GetFileByServerRelativePath(DecodedUrl='${urlUtil.getServerRelativeSiteUrl(args.options.webUrl)}/${pageUrl}')?$expand=ListItemAllFields`,
                 headers: {
                     'content-type': 'application/json;charset=utf-8',
                     accept: 'application/json;odata=nometadata'

package/dist/m365/spo/commands/feature/feature-disable.js

@@ -5,6 +5,7 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 };
 var _SpoFeatureDisableCommand_instances, _SpoFeatureDisableCommand_initTelemetry, _SpoFeatureDisableCommand_initOptions, _SpoFeatureDisableCommand_initValidators, _SpoFeatureDisableCommand_initTypes;
 import request from "../../../../request.js";
+import { validation } from '../../../../utils/validation.js';
 import SpoCommand from "../../../base/SpoCommand.js";
 import commands from "../../commands.js";
 class SpoFeatureDisableCommand extends SpoCommand {
@@ -24,13 +25,10 @@ class SpoFeatureDisableCommand extends SpoCommand {
     }
     async commandAction(logger, args) {
         let scope = args.options.scope;
-        let force = args.options.force;
+        const force = !!args.options.force;
         if (!scope) {
             scope = "web";
         }
-        if (!force) {
-            force = false;
-        }
         if (this.verbose) {
             await logger.logToStderr(`Disabling feature '${args.options.id}' on scope '${scope}' for url '${args.options.webUrl}' (force='${force}')...`);
         }
@@ -39,7 +37,8 @@ class SpoFeatureDisableCommand extends SpoCommand {
             url: url,
             headers: {
                 accept: 'application/json;odata=nometadata'
-            }
+            },
+            responseType: 'json'
         };
         try {
             await request.post(requestOptions);
@@ -69,9 +68,12 @@ _SpoFeatureDisableCommand_instances = new WeakSet(), _SpoFeatureDisableCommand_i
     });
 }, _SpoFeatureDisableCommand_initValidators = function _SpoFeatureDisableCommand_initValidators() {
     this.validators.push(async (args) => {
+        if (!validation.isValidGuid(args.options.id)) {
+            return `${args.options.id} is not a valid GUID for option id.`;
+        }
         if (args.options.scope) {
             if (['site', 'web'].indexOf(args.options.scope.toLowerCase()) < 0) {
-                return `${args.options.scope} is not a valid Feature scope. Allowed values are Site|Web`;
+                return `${args.options.scope} is not a valid Feature scope. Allowed values are: Site, Web.`;
             }
         }
         return true;

package/dist/m365/spo/commands/feature/feature-enable.js

@@ -25,13 +25,10 @@ class SpoFeatureEnableCommand extends SpoCommand {
     }
     async commandAction(logger, args) {
         let scope = args.options.scope;
-        let force = args.options.force;
+        const force = !!args.options.force;
         if (!scope) {
             scope = "web";
         }
-        if (!force) {
-            force = false;
-        }
         if (this.verbose) {
             await logger.logToStderr(`Enabling feature '${args.options.id}' on scope '${scope}' for url '${args.options.webUrl}' (force='${force}')...`);
         }
@@ -40,7 +37,8 @@ class SpoFeatureEnableCommand extends SpoCommand {
             url: url,
             headers: {
                 accept: 'application/json;odata=nometadata'
-            }
+            },
+            responseType: 'json'
         };
         try {
             await request.post(requestOptions);
@@ -70,6 +68,9 @@ _SpoFeatureEnableCommand_instances = new WeakSet(), _SpoFeatureEnableCommand_ini
     });
 }, _SpoFeatureEnableCommand_initValidators = function _SpoFeatureEnableCommand_initValidators() {
     this.validators.push(async (args) => {
+        if (!validation.isValidGuid(args.options.id)) {
+            return `${args.options.id} is not a valid GUID for option id.`;
+        }
         if (args.options.scope) {
             if (['site', 'web'].indexOf(args.options.scope.toLowerCase()) < 0) {
                 return `${args.options.scope} is not a valid Feature scope. Allowed values are Site|Web`;

package/dist/m365/spo/commands/page/Page.js

@@ -12,7 +12,7 @@ export class Page {
         }
         const pageName = this.getPageNameWithExtension(name);
         const requestOptions = {
-            url: `${webUrl}/_api/web/getfilebyserverrelativeurl('${urlUtil.getServerRelativeSiteUrl(webUrl)}/SitePages/${formatting.encodeQueryParameter(pageName)}')?$expand=ListItemAllFields/ClientSideApplicationId`,
+            url: `${webUrl}/_api/web/GetFileByServerRelativePath(DecodedUrl='${urlUtil.getServerRelativeSiteUrl(webUrl)}/SitePages/${formatting.encodeQueryParameter(pageName)}')?$expand=ListItemAllFields/ClientSideApplicationId`,
             headers: {
                 'content-type': 'application/json;charset=utf-8',
                 accept: 'application/json;odata=nometadata'

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

@@ -52,7 +52,7 @@ class SpoPageAddCommand extends SpoCommand {
             const reqDigest = await spo.getRequestDigest(args.options.webUrl);
             requestDigest = reqDigest.FormDigestValue;
             let requestOptions = {
-                url: `${args.options.webUrl}/_api/web/getfolderbyserverrelativeurl('${serverRelativeSiteUrl}/sitepages')/files/AddTemplateFile`,
+                url: `${args.options.webUrl}/_api/web/GetFolderByServerRelativePath(DecodedUrl='${serverRelativeSiteUrl}/sitepages')/files/AddTemplateFile`,
                 headers: {
                     'X-RequestDigest': requestDigest,
                     'content-type': 'application/json;odata=nometadata',

package/dist/m365/spo/commands/page/page-get.js

@@ -36,7 +36,7 @@ class SpoPageGetCommand extends SpoCommand {
         }
         try {
             let requestOptions = {
-                url: `${args.options.webUrl}/_api/web/getfilebyserverrelativeurl('${urlUtil.getServerRelativeSiteUrl(args.options.webUrl)}/SitePages/${formatting.encodeQueryParameter(pageName)}')?$expand=ListItemAllFields/ClientSideApplicationId,ListItemAllFields/PageLayoutType,ListItemAllFields/CommentsDisabled`,
+                url: `${args.options.webUrl}/_api/web/GetFileByServerRelativePath(DecodedUrl='${urlUtil.getServerRelativeSiteUrl(args.options.webUrl)}/SitePages/${formatting.encodeQueryParameter(pageName)}')?$expand=ListItemAllFields/ClientSideApplicationId,ListItemAllFields/PageLayoutType,ListItemAllFields/CommentsDisabled`,
                 headers: {
                     'content-type': 'application/json;charset=utf-8',
                     accept: 'application/json;odata=nometadata'

package/dist/m365/spo/commands/page/page-header-set.js

@@ -300,7 +300,7 @@ class SpoPageHeaderSetCommand extends SpoCommand {
             await logger.logToStderr(`Retrieving information about the header image...`);
         }
         const requestOptions = {
-            url: `${siteUrl}/_api/web/getfilebyserverrelativeurl('${formatting.encodeQueryParameter(imageUrl)}')?$select=ListId,UniqueId`,
+            url: `${siteUrl}/_api/web/GetFileByServerRelativePath(DecodedUrl='${formatting.encodeQueryParameter(imageUrl)}')?$select=ListId,UniqueId`,
             headers: {
                 accept: 'application/json;odata=nometadata'
             },

package/dist/m365/spo/commands/page/page-remove.js

@@ -54,7 +54,7 @@ class SpoPageRemoveCommand extends SpoCommand {
             }
             const requestOptions = {
                 url: `${args.options
-                    .webUrl}/_api/web/getfilebyserverrelativeurl('${urlUtil.getServerRelativeSiteUrl(args.options.webUrl)}/sitepages/${pageName}')`,
+                    .webUrl}/_api/web/GetFileByServerRelativePath(DecodedUrl='${urlUtil.getServerRelativeSiteUrl(args.options.webUrl)}/sitepages/${pageName}')`,
                 headers: {
                     'X-RequestDigest': requestDigest,
                     'X-HTTP-Method': 'DELETE',

package/dist/m365/spo/commands/page/page-set.js

@@ -199,7 +199,7 @@ class SpoPageSetCommand extends SpoCommand {
             }
             if (typeof args.options.commentsEnabled !== 'undefined') {
                 const requestOptions = {
-                    url: `${args.options.webUrl}/_api/web/getfilebyserverrelativeurl('${serverRelativeFileUrl}')/ListItemAllFields/SetCommentsDisabled(${args.options.commentsEnabled === false})`,
+                    url: `${args.options.webUrl}/_api/web/GetFileByServerRelativePath(DecodedUrl='${serverRelativeFileUrl}')/ListItemAllFields/SetCommentsDisabled(${args.options.commentsEnabled === false})`,
                     headers: {
                         'X-RequestDigest': requestDigest,
                         'content-type': 'application/json;odata=nometadata',
@@ -239,7 +239,7 @@ class SpoPageSetCommand extends SpoCommand {
             }
             else {
                 requestOptions = {
-                    url: `${args.options.webUrl}/_api/web/getfilebyserverrelativeurl('${serverRelativeFileUrl}')/CheckIn(comment=@a1,checkintype=@a2)?@a1='${formatting.encodeQueryParameter(args.options.publishMessage || '')}'&@a2=1`,
+                    url: `${args.options.webUrl}/_api/web/GetFileByServerRelativePath(DecodedUrl='${serverRelativeFileUrl}')/CheckIn(comment=@a1,checkintype=@a2)?@a1='${formatting.encodeQueryParameter(args.options.publishMessage || '')}'&@a2=1`,
                     headers: {
                         'X-RequestDigest': requestDigest,
                         'content-type': 'application/json;odata=nometadata',

package/dist/m365/spo/commands/page/page-text-add.js

@@ -79,7 +79,7 @@ class SpoPageTextAddCommand extends SpoCommand {
         }
         const requestOptions = {
             url: `${args.options
-                .webUrl}/_api/web/getfilebyserverrelativeurl('${urlUtil.getServerRelativeSiteUrl(args.options.webUrl)}/sitepages/${pageName}')/ListItemAllFields`,
+                .webUrl}/_api/web/GetFileByServerRelativePath(DecodedUrl='${urlUtil.getServerRelativeSiteUrl(args.options.webUrl)}/sitepages/${pageName}')/ListItemAllFields`,
             headers: {
                 'X-RequestDigest': requestDigest,
                 'content-type': 'application/json;odata=nometadata',