@pnp/cli-microsoft365 7.11.0-beta.c513557 → 8.0.0-beta.b0e3dab

package/dist/m365/pa/commands/app/app-owner-set.js

@@ -26,7 +26,7 @@ class PaAppOwnerSetCommand extends PowerAppsCommand {
     }
     async commandAction(logger, args) {
         if (this.verbose) {
-            logger.logToStderr(`Setting new owner ${args.options.userId || args.options.userName} for Power Apps app ${args.options.appName}...`);
+            await logger.logToStderr(`Setting new owner ${args.options.userId || args.options.userName} for Power Apps app ${args.options.appName}...`);
         }
         try {
             const userId = await this.getUserId(args.options);

package/dist/m365/pp/commands/solution/solution-publish.js

@@ -50,7 +50,7 @@ class PpSolutionPublishCommand extends PowerPlatformCommand {
                 await request.post(requestOptions);
             }
             else {
-                request.post(requestOptions);
+                void request.post(requestOptions);
             }
         }
         catch (err) {

package/dist/m365/purview/commands/threatassessment/threatassessment-list.js

@@ -26,7 +26,7 @@ class PurviewThreatAssessmentListCommand extends GraphCommand {
     }
     async commandAction(logger, args) {
         if (this.verbose) {
-            logger.logToStderr('Retrieving a list of threat assessments');
+            await logger.logToStderr('Retrieving a list of threat assessments');
         }
         try {
             const filter = this.getFilterQuery(args.options);

package/dist/m365/spfx/commands/project/project-azuredevops-pipeline-add.js

@@ -37,7 +37,7 @@ class SpfxProjectAzureDevOpsPipelineAddCommand extends BaseProjectCommand {
         const packageJson = fs.readFileSync(solutionPackageJsonFile, 'utf-8');
         const solutionName = JSON.parse(packageJson).name;
         if (this.debug) {
-            logger.logToStderr(`Adding Azure DevOps pipeline in the current SPFx project`);
+            await logger.logToStderr(`Adding Azure DevOps pipeline in the current SPFx project`);
         }
         try {
             this.updatePipeline(solutionName, pipeline, args.options);

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

@@ -81,7 +81,7 @@ class SpfxProjectExternalizeCommand extends BaseProjectCommand {
             this.allEditSuggestions.push(...rulesResults.map(x => x.suggestions).reduce((x, y) => [...x, ...y]));
             //removing duplicates
             this.allFindings = this.allFindings.filter((x, i) => this.allFindings.findIndex(y => y.key === x.key) === i);
-            this.writeReport(this.allFindings, this.allEditSuggestions, logger, args.options);
+            await this.writeReport(this.allFindings, this.allEditSuggestions, logger, args.options);
         }
         catch (err) {
             throw new CommandError(err);

package/dist/m365/spfx/commands/project/project-github-workflow-add.js

@@ -40,7 +40,7 @@ class SpfxProjectGithubWorkflowAddCommand extends BaseProjectCommand {
         const packageJson = fs.readFileSync(solutionPackageJsonFile, 'utf-8');
         const solutionName = JSON.parse(packageJson).name;
         if (this.debug) {
-            logger.logToStderr(`Adding GitHub workflow in the current SPFx project`);
+            await logger.logToStderr(`Adding GitHub workflow in the current SPFx project`);
         }
         try {
             this.updateWorkflow(solutionName, workflow, args.options);

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

@@ -713,7 +713,7 @@ class SpfxDoctorCommand extends BaseProjectCommand {
     }
     async checkNodeVersion(prerequisites) {
         const nodeVersion = this.getNodeVersion();
-        this.checkStatus('Node', nodeVersion, prerequisites.node);
+        await this.checkStatus('Node', nodeVersion, prerequisites.node);
     }
     async checkSharePointFrameworkVersion(spfxVersionRequested) {
         let spfxVersionDetected = await this.getSPFxVersionFromYoRcFile();
@@ -725,7 +725,7 @@ class SpfxDoctorCommand extends BaseProjectCommand {
             fix: `npm i -g @microsoft/generator-sharepoint@${spfxVersionRequested}`
         };
         if (spfxVersionDetected) {
-            this.checkStatus(`SharePoint Framework`, spfxVersionDetected, versionCheck);
+            await this.checkStatus(`SharePoint Framework`, spfxVersionDetected, versionCheck);
         }
         else {
             const message = `SharePoint Framework v${spfxVersionRequested} not found`;
@@ -742,7 +742,7 @@ class SpfxDoctorCommand extends BaseProjectCommand {
     async checkYo(prerequisites) {
         const yoVersion = await this.getPackageVersion('yo', PackageSearchMode.GlobalOnly, HandlePromise.Continue);
         if (yoVersion) {
-            this.checkStatus('yo', yoVersion, prerequisites.yo);
+            await this.checkStatus('yo', yoVersion, prerequisites.yo);
         }
         else {
             const message = 'yo not found';
@@ -758,7 +758,7 @@ class SpfxDoctorCommand extends BaseProjectCommand {
     async checkGulpCli(prerequisites) {
         const gulpCliVersion = await this.getPackageVersion('gulp-cli', PackageSearchMode.GlobalOnly, HandlePromise.Continue);
         if (gulpCliVersion) {
-            this.checkStatus('gulp-cli', gulpCliVersion, prerequisites.gulpCli);
+            await this.checkStatus('gulp-cli', gulpCliVersion, prerequisites.gulpCli);
         }
         else {
             const message = 'gulp-cli not found';

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

@@ -3,7 +3,7 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
     if (typeof state === "function" ? receiver !== state || !f : !state.has(receiver)) throw new TypeError("Cannot read private member from an object whose class did not declare it");
     return kind === "m" ? f : kind === "a" ? f.call(receiver) : f ? f.value : state.get(receiver);
 };
-var _SpoCdnGetCommand_instances, _SpoCdnGetCommand_initTelemetry, _SpoCdnGetCommand_initOptions, _SpoCdnGetCommand_initValidators;
+var _SpoCdnGetCommand_instances, _SpoCdnGetCommand_initTelemetry, _SpoCdnGetCommand_initOptions, _SpoCdnGetCommand_initValidators, _SpoCdnGetCommand_initTypes;
 import config from '../../../../config.js';
 import request from '../../../../request.js';
 import { spo } from '../../../../utils/spo.js';
@@ -19,9 +19,11 @@ class SpoCdnGetCommand extends SpoCommand {
     constructor() {
         super();
         _SpoCdnGetCommand_instances.add(this);
+        this.validTypes = ['Public', 'Private'];
         __classPrivateFieldGet(this, _SpoCdnGetCommand_instances, "m", _SpoCdnGetCommand_initTelemetry).call(this);
         __classPrivateFieldGet(this, _SpoCdnGetCommand_instances, "m", _SpoCdnGetCommand_initOptions).call(this);
         __classPrivateFieldGet(this, _SpoCdnGetCommand_instances, "m", _SpoCdnGetCommand_initValidators).call(this);
+        __classPrivateFieldGet(this, _SpoCdnGetCommand_instances, "m", _SpoCdnGetCommand_initTypes).call(this);
     }
     async commandAction(logger, args) {
         const cdnTypeString = args.options.type || 'Public';
@@ -48,15 +50,11 @@ class SpoCdnGetCommand extends SpoCommand {
             if (response.ErrorInfo) {
                 throw response.ErrorInfo.ErrorMessage;
             }
-            else {
-                const result = json[json.length - 1];
-                if (this.verbose) {
-                    await logger.logToStderr(`${(cdnType === 0 ? 'Public' : 'Private')} CDN at ${spoAdminUrl} is ${(result === true ? 'enabled' : 'disabled')}`);
-                }
-                else {
-                    await logger.logToStderr(result);
-                }
+            const result = json[json.length - 1];
+            if (this.verbose) {
+                await logger.logToStderr(`${cdnType === 0 ? 'Public' : 'Private'} CDN at ${spoAdminUrl} is ${result === true ? 'enabled' : 'disabled'}`);
             }
+            await logger.log(result);
         }
         catch (err) {
             this.handleRejectedPromise(err);
@@ -72,18 +70,17 @@ _SpoCdnGetCommand_instances = new WeakSet(), _SpoCdnGetCommand_initTelemetry = f
 }, _SpoCdnGetCommand_initOptions = function _SpoCdnGetCommand_initOptions() {
     this.options.unshift({
         option: '-t, --type [type]',
-        autocomplete: ['Public', 'Private']
+        autocomplete: this.validTypes
     });
 }, _SpoCdnGetCommand_initValidators = function _SpoCdnGetCommand_initValidators() {
     this.validators.push(async (args) => {
-        if (args.options.type) {
-            if (args.options.type !== 'Public' &&
-                args.options.type !== 'Private') {
-                return `${args.options.type} is not a valid CDN type. Allowed values are Public|Private`;
-            }
+        if (args.options.type && !this.validTypes.includes(args.options.type)) {
+            return `'${args.options.type}' is not a valid CDN type. Allowed values are: ${this.validTypes.join(', ')}.`;
         }
         return true;
     });
+}, _SpoCdnGetCommand_initTypes = function _SpoCdnGetCommand_initTypes() {
+    this.types.string.push('type');
 };
 export default new SpoCdnGetCommand();
 //# sourceMappingURL=cdn-get.js.map

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

@@ -19,6 +19,7 @@ class SpoCdnSetCommand extends SpoCommand {
     constructor() {
         super();
         _SpoCdnSetCommand_instances.add(this);
+        this.validTypes = ['Public', 'Private', 'Both'];
         __classPrivateFieldGet(this, _SpoCdnSetCommand_instances, "m", _SpoCdnSetCommand_initTelemetry).call(this);
         __classPrivateFieldGet(this, _SpoCdnSetCommand_instances, "m", _SpoCdnSetCommand_initOptions).call(this);
         __classPrivateFieldGet(this, _SpoCdnSetCommand_instances, "m", _SpoCdnSetCommand_initTypes).call(this);
@@ -104,7 +105,7 @@ _SpoCdnSetCommand_instances = new WeakSet(), _SpoCdnSetCommand_initTelemetry = f
         Object.assign(this.telemetryProperties, {
             cdnType: args.options.type || 'Public',
             enabled: args.options.enabled,
-            noDefaultOrigins: (!(!args.options.noDefaultOrigins)).toString()
+            noDefaultOrigins: !!args.options.noDefaultOrigins
         });
     });
 }, _SpoCdnSetCommand_initOptions = function _SpoCdnSetCommand_initOptions() {
@@ -113,18 +114,19 @@ _SpoCdnSetCommand_instances = new WeakSet(), _SpoCdnSetCommand_initTelemetry = f
         autocomplete: ['true', 'false']
     }, {
         option: '-t, --type [type]',
-        autocomplete: ['Public', 'Private', 'Both']
+        autocomplete: this.validTypes
     }, {
         option: '--noDefaultOrigins'
     });
 }, _SpoCdnSetCommand_initTypes = function _SpoCdnSetCommand_initTypes() {
-    this.types.boolean.push('enabled');
+    this.types.boolean.push('enabled', 'noDefaultOrigins');
+    this.types.string.push('type');
 }, _SpoCdnSetCommand_initValidators = function _SpoCdnSetCommand_initValidators() {
     this.validators.push(async (args) => {
         if (args.options.type) {
             if (args.options.type !== 'Public' && args.options.type !== 'Both' &&
                 args.options.type !== 'Private') {
-                return `${args.options.type} is not a valid CDN type. Allowed values are Public|Private|Both`;
+                return `${args.options.type} is not a valid CDN type. Allowed values are ${this.validTypes.join(', ')}.`;
             }
         }
         return true;

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

@@ -60,7 +60,7 @@ class SpoCommandSetGetCommand extends SpoCommand {
                 else {
                     const resultAsKeyValuePair = formatting.convertArrayToHashTable('Id', commandSets);
                     const commandSet = await cli.handleMultipleResultsFound(`Multiple command sets with title '${args.options.title}' found.`, resultAsKeyValuePair);
-                    logger.log(commandSet);
+                    await logger.log(commandSet);
                 }
             }
         }

package/dist/m365/spo/commands/file/file-retentionlabel-remove.js

@@ -54,7 +54,7 @@ class SpoFileRetentionLabelRemoveCommand extends SpoCommand {
     }
     async getFileProperties(logger, args) {
         if (this.verbose) {
-            logger.logToStderr(`Retrieving list and item information for file '${args.options.fileId || args.options.fileUrl}' in site at ${args.options.webUrl}...`);
+            await logger.logToStderr(`Retrieving list and item information for file '${args.options.fileId || args.options.fileUrl}' in site at ${args.options.webUrl}...`);
         }
         let requestUrl = `${args.options.webUrl}/_api/web/`;
         if (args.options.fileId) {

package/dist/m365/spo/commands/group/group-member-add.js

@@ -3,12 +3,10 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
     if (typeof state === "function" ? receiver !== state || !f : !state.has(receiver)) throw new TypeError("Cannot read private member from an object whose class did not declare it");
     return kind === "m" ? f : kind === "a" ? f.call(receiver) : f ? f.value : state.get(receiver);
 };
-var _SpoGroupMemberAddCommand_instances, _SpoGroupMemberAddCommand_initTelemetry, _SpoGroupMemberAddCommand_initOptions, _SpoGroupMemberAddCommand_initValidators, _SpoGroupMemberAddCommand_initOptionSets;
+var _SpoGroupMemberAddCommand_instances, _SpoGroupMemberAddCommand_initTelemetry, _SpoGroupMemberAddCommand_initOptions, _SpoGroupMemberAddCommand_initValidators, _SpoGroupMemberAddCommand_initOptionSets, _SpoGroupMemberAddCommand_initTypes;
 import request from '../../../../request.js';
 import { entraGroup } from '../../../../utils/entraGroup.js';
-import { entraUser } from '../../../../utils/entraUser.js';
 import { formatting } from '../../../../utils/formatting.js';
-import { spo } from '../../../../utils/spo.js';
 import { validation } from '../../../../utils/validation.js';
 import SpoCommand from '../../../base/SpoCommand.js';
 import commands from '../../commands.js';
@@ -20,7 +18,7 @@ class SpoGroupMemberAddCommand extends SpoCommand {
         return 'Add members to a SharePoint Group';
     }
     defaultProperties() {
-        return ['DisplayName', 'Email'];
+        return ['Title', 'UserPrincipalName'];
     }
     constructor() {
         super();
@@ -29,110 +27,100 @@ class SpoGroupMemberAddCommand extends SpoCommand {
         __classPrivateFieldGet(this, _SpoGroupMemberAddCommand_instances, "m", _SpoGroupMemberAddCommand_initOptions).call(this);
         __classPrivateFieldGet(this, _SpoGroupMemberAddCommand_instances, "m", _SpoGroupMemberAddCommand_initValidators).call(this);
         __classPrivateFieldGet(this, _SpoGroupMemberAddCommand_instances, "m", _SpoGroupMemberAddCommand_initOptionSets).call(this);
+        __classPrivateFieldGet(this, _SpoGroupMemberAddCommand_instances, "m", _SpoGroupMemberAddCommand_initTypes).call(this);
     }
     async commandAction(logger, args) {
         try {
             if (args.options.aadGroupIds) {
                 args.options.entraGroupIds = args.options.aadGroupIds;
-                this.warn(logger, `Option 'aadGroupIds' is deprecated. Please use 'entraGroupIds' instead`);
+                await this.warn(logger, `Option 'aadGroupIds' is deprecated. Please use 'entraGroupIds' instead.`);
             }
             if (args.options.aadGroupNames) {
                 args.options.entraGroupNames = args.options.aadGroupNames;
-                this.warn(logger, `Option 'aadGroupNames' is deprecated. Please use 'entraGroupNames' instead`);
+                await this.warn(logger, `Option 'aadGroupNames' is deprecated. Please use 'entraGroupNames' instead.`);
             }
-            const groupId = await this.getGroupId(args, logger);
-            const resolvedUsernameList = await this.getValidUsers(args, logger);
+            const loginNames = await this.getLoginNames(logger, args.options);
+            let apiUrl = `${args.options.webUrl}/_api/web/SiteGroups`;
+            if (args.options.groupId) {
+                apiUrl += `/GetById(${args.options.groupId})`;
+            }
+            else {
+                apiUrl += `/GetByName('${formatting.encodeQueryParameter(args.options.groupName)}')`;
+            }
+            apiUrl += '/users';
             if (this.verbose) {
-                await logger.logToStderr(`Adding resource(s) to SharePoint Group ${args.options.groupId || args.options.groupName}`);
-            }
-            const requestOptions = {
-                url: `${args.options.webUrl}/_api/SP.Web.ShareObject`,
-                headers: {
-                    accept: 'application/json;odata=nometadata',
-                    'content-type': 'application/json;odata=nometadata'
-                },
-                responseType: 'json',
-                data: {
-                    url: args.options.webUrl,
-                    peoplePickerInput: this.getFormattedUserList(resolvedUsernameList),
-                    roleValue: `group:${groupId}`
-                }
-            };
-            const sharingResult = await request.post(requestOptions);
-            if (sharingResult.ErrorMessage !== null) {
-                throw sharingResult.ErrorMessage;
+                await logger.logToStderr('Adding members to group...');
+            }
+            const result = [];
+            for (const loginName of loginNames) {
+                const requestOptions = {
+                    url: apiUrl,
+                    headers: {
+                        accept: 'application/json;odata=nometadata'
+                    },
+                    responseType: 'json',
+                    data: {
+                        LoginName: loginName
+                    }
+                };
+                const response = await request.post(requestOptions);
+                result.push(response);
             }
-            await logger.log(sharingResult.UsersAddedToGroup);
+            await logger.log(result);
         }
         catch (err) {
             this.handleRejectedODataJsonPromise(err);
         }
     }
-    async getGroupId(args, logger) {
-        if (this.verbose) {
-            await logger.logToStderr(`Getting group Id for SharePoint Group ${args.options.groupId ? args.options.groupId : args.options.groupName}`);
-        }
-        const getGroupMethod = args.options.groupName ?
-            `GetByName('${formatting.encodeQueryParameter(args.options.groupName)}')` :
-            `GetById('${args.options.groupId}')`;
-        const requestOptions = {
-            url: `${args.options.webUrl}/_api/web/sitegroups/${getGroupMethod}?$select=Id`,
-            headers: {
-                'accept': 'application/json;odata=nometadata'
-            },
-            responseType: 'json'
-        };
-        const response = await request.get(requestOptions);
-        return response.Id;
-    }
-    async getValidUsers(args, logger) {
-        if (this.verbose) {
-            await logger.logToStderr('Checking if the specified users and groups exist');
+    async getLoginNames(logger, options) {
+        const loginNames = [];
+        if (options.userNames || options.emails) {
+            loginNames.push(...formatting.splitAndTrim(options.userNames || options.emails).map(u => `i:0#.f|membership|${u}`));
         }
-        const validUserNames = [];
-        const identifiers = args.options.userNames ?? args.options.emails ?? args.options.entraGroupIds ?? args.options.entraGroupNames ?? args.options.userIds.toString();
-        await Promise.all(identifiers.split(',').map(async (identifier) => {
-            const trimmedIdentifier = identifier.trim();
-            try {
-                if (args.options.userIds) {
-                    if (this.verbose) {
-                        await logger.logToStderr(`Getting AAD ID of user with ID ${trimmedIdentifier}`);
-                    }
-                    const spoUserAzureId = await spo.getUserAzureIdBySpoId(args.options.webUrl, trimmedIdentifier);
-                    validUserNames.push(spoUserAzureId);
-                }
-                else if (args.options.userNames) {
-                    validUserNames.push(trimmedIdentifier);
-                }
-                else if (args.options.entraGroupIds) {
-                    validUserNames.push(trimmedIdentifier);
-                }
-                else if (args.options.entraGroupNames) {
-                    if (this.verbose) {
-                        await logger.logToStderr(`Getting ID of Microsoft Entra group ${trimmedIdentifier}`);
-                    }
-                    const groupId = await entraGroup.getGroupIdByDisplayName(trimmedIdentifier);
-                    validUserNames.push(groupId);
+        else if (options.entraGroupIds || options.entraGroupNames) {
+            if (this.verbose) {
+                await logger.logToStderr(`Resolving ${(options.entraGroupIds || options.entraGroupNames).length} group(s)...`);
+            }
+            const groups = [];
+            if (options.entraGroupIds) {
+                const groupIds = formatting.splitAndTrim(options.entraGroupIds);
+                for (const groupId of groupIds) {
+                    const group = await entraGroup.getGroupById(groupId);
+                    groups.push(group);
                 }
-                else {
-                    if (this.verbose) {
-                        await logger.logToStderr(`Getting Microsoft Entra ID for user ${trimmedIdentifier}`);
-                    }
-                    const upn = await entraUser.getUserIdByEmail(trimmedIdentifier);
-                    validUserNames.push(upn);
+            }
+            else {
+                const groupNames = formatting.splitAndTrim(options.entraGroupNames);
+                for (const groupName of groupNames) {
+                    const group = await entraGroup.getGroupByDisplayName(groupName);
+                    groups.push(group);
                 }
             }
-            catch (err) {
-                throw `Resource '${trimmedIdentifier}' does not exist.`;
+            // Check if group is M365 group or security group
+            loginNames.push(...groups.map(g => g.mailEnabled ? `c:0o.c|federateddirectoryclaimprovider|${g.id}` : `c:0t.c|tenant|${g.id}`));
+        }
+        else if (options.userIds) {
+            const userIds = formatting.splitAndTrim(options.userIds);
+            if (this.verbose) {
+                await logger.logToStderr(`Resolving ${userIds.length} user(s)...`);
             }
-        }));
-        return validUserNames;
+            for (const userId of userIds) {
+                const loginName = await this.getUserLoginNameById(options.webUrl, parseInt(userId));
+                loginNames.push(loginName);
+            }
+        }
+        return loginNames;
     }
-    getFormattedUserList(activeUserList) {
-        const generatedPeoplePicker = JSON.stringify(activeUserList.map(singleUsername => {
-            return { Key: singleUsername.trim() };
-        }));
-        return generatedPeoplePicker;
+    async getUserLoginNameById(webUrl, userId) {
+        const requestOptions = {
+            url: `${webUrl}/_api/web/SiteUsers/GetById(${userId})?$select=LoginName`,
+            headers: {
+                accept: 'application/json;odata=nometadata'
+            },
+            responseType: 'json'
+        };
+        const user = await request.get(requestOptions);
+        return user.LoginName;
     }
 }
 _SpoGroupMemberAddCommand_instances = new WeakSet(), _SpoGroupMemberAddCommand_initTelemetry = function _SpoGroupMemberAddCommand_initTelemetry() {
@@ -177,29 +165,45 @@ _SpoGroupMemberAddCommand_instances = new WeakSet(), _SpoGroupMemberAddCommand_i
         if (isValidSharePointUrl !== true) {
             return isValidSharePointUrl;
         }
-        if (args.options.groupId && isNaN(args.options.groupId)) {
-            return `Specified groupId ${args.options.groupId} is not a number`;
+        if (args.options.groupId && !validation.isValidPositiveInteger(args.options.groupId)) {
+            return `Specified groupId ${args.options.groupId} is not a positive number.`;
         }
-        const userIdReg = new RegExp(/^[0-9,]*$/);
-        if (args.options.userIds && !userIdReg.test(args.options.userIds)) {
-            return `${args.options.userIds} is not a number or a comma separated value`;
+        if (args.options.userIds) {
+            const isValidArray = validation.isValidPositiveIntegerArray(args.options.userIds);
+            if (isValidArray !== true) {
+                return `Option 'userIds' contains one or more invalid numbers: ${isValidArray}.`;
+            }
         }
-        if (args.options.userNames && args.options.userNames.split(',').some(e => !validation.isValidUserPrincipalName(e))) {
-            return `${args.options.userNames} contains one or more invalid usernames`;
+        if (args.options.userNames) {
+            const isValidArray = validation.isValidUserPrincipalNameArray(args.options.userNames);
+            if (isValidArray !== true) {
+                return `Option 'userNames' contains one or more invalid UPNs: ${isValidArray}.`;
+            }
         }
-        if (args.options.emails && args.options.emails.split(',').some(e => !validation.isValidUserPrincipalName(e))) {
-            return `${args.options.emails} contains one or more invalid email addresses`;
+        if (args.options.emails) {
+            const isValidArray = validation.isValidUserPrincipalNameArray(args.options.emails);
+            if (isValidArray !== true) {
+                return `Option 'emails' contains one or more invalid UPNs: ${isValidArray}.`;
+            }
         }
-        if (args.options.entraGroupIds && args.options.entraGroupIds.split(',').some(e => !validation.isValidGuid(e))) {
-            return `${args.options.entraGroupIds} contains one or more invalid GUIDs`;
+        if (args.options.entraGroupIds) {
+            const isValidArray = validation.isValidGuidArray(args.options.entraGroupIds);
+            if (isValidArray !== true) {
+                return `Option 'entraGroupIds' contains one or more invalid GUIDs: ${isValidArray}.`;
+            }
         }
-        if (args.options.aadGroupIds && args.options.aadGroupIds.split(',').some(e => !validation.isValidGuid(e))) {
-            return `${args.options.aadGroupIds} contains one or more invalid GUIDs`;
+        if (args.options.aadGroupIds) {
+            const isValidArray = validation.isValidGuidArray(args.options.aadGroupIds);
+            if (isValidArray !== true) {
+                return `Option 'aadGroupIds' contains one or more invalid GUIDs: ${isValidArray}.`;
+            }
         }
         return true;
     });
 }, _SpoGroupMemberAddCommand_initOptionSets = function _SpoGroupMemberAddCommand_initOptionSets() {
     this.optionSets.push({ options: ['groupId', 'groupName'] }, { options: ['userNames', 'emails', 'userIds', 'entraGroupIds', 'aadGroupIds', 'entraGroupNames', 'aadGroupNames'] });
+}, _SpoGroupMemberAddCommand_initTypes = function _SpoGroupMemberAddCommand_initTypes() {
+    this.types.string.push('webUrl', 'groupName', 'userNames', 'emails', 'userIds', 'entraGroupIds', 'aadGroupIds', 'entraGroupNames', 'aadGroupNames');
 };
 export default new SpoGroupMemberAddCommand();
 //# sourceMappingURL=group-member-add.js.map

package/dist/m365/spo/commands/group/group-member-remove.js

@@ -47,11 +47,11 @@ class SpoGroupMemberRemoveCommand extends SpoCommand {
     async commandAction(logger, args) {
         if (args.options.aadGroupId) {
             args.options.entraGroupId = args.options.aadGroupId;
-            this.warn(logger, `Option 'aadGroupId' is deprecated. Please use 'entraGroupId' instead`);
+            await this.warn(logger, `Option 'aadGroupId' is deprecated. Please use 'entraGroupId' instead`);
         }
         if (args.options.aadGroupName) {
             args.options.entraGroupName = args.options.aadGroupName;
-            this.warn(logger, `Option 'aadGroupName' is deprecated. Please use 'entraGroupName' instead`);
+            await this.warn(logger, `Option 'aadGroupName' is deprecated. Please use 'entraGroupName' instead`);
         }
         if (args.options.force) {
             if (this.debug) {

package/dist/m365/spo/commands/list/list-retentionlabel-ensure.js

@@ -38,7 +38,7 @@ class SpoListRetentionLabelEnsureCommand extends SpoCommand {
     }
     async getListServerRelativeUrl(args, logger) {
         if (this.verbose) {
-            logger.logToStderr('Getting the list server relative URL');
+            await logger.logToStderr('Getting the list server relative URL');
         }
         if (args.options.listUrl) {
             return urlUtil.getServerRelativePath(args.options.webUrl, args.options.listUrl);

package/dist/m365/spo/commands/listitem/listitem-batch-remove.js

@@ -34,7 +34,7 @@ class SpoListItemBatchRemoveCommand extends SpoCommand {
         const removeListItems = async () => {
             try {
                 if (this.verbose) {
-                    logger.logToStderr('Removing the listitems from SharePoint...');
+                    await logger.logToStderr('Removing the listitems from SharePoint...');
                 }
                 let idsToRemove = [];
                 if (args.options.filePath) {

package/dist/m365/spo/commands/listitem/listitem-retentionlabel-ensure.js

@@ -46,7 +46,7 @@ class SpoListItemRetentionLabelEnsureCommand extends SpoCommand {
             return options.name;
         }
         if (this.verbose) {
-            logger.logToStderr(`Retrieving the name of the retention label based on the Id '${options.id}'...`);
+            await logger.logToStderr(`Retrieving the name of the retention label based on the Id '${options.id}'...`);
         }
         const requestUrl = `${options.webUrl}/_api/SP.CompliancePolicy.SPPolicyStoreProxy.GetAvailableTagsForSite(siteUrl=@a1)?@a1='${formatting.encodeQueryParameter(options.webUrl)}'`;
         const labels = await odata.getAllItems(requestUrl);
@@ -87,7 +87,7 @@ class SpoListItemRetentionLabelEnsureCommand extends SpoCommand {
         const serverRelativePath = urlUtil.getServerRelativePath(options.webUrl, response.RootFolder.ServerRelativeUrl);
         const listAbsoluteUrl = urlUtil.urlCombine(tenantUrl, serverRelativePath);
         if (this.verbose) {
-            logger.logToStderr(`List absolute URL found: '${listAbsoluteUrl}'`);
+            await logger.logToStderr(`List absolute URL found: '${listAbsoluteUrl}'`);
         }
         return listAbsoluteUrl;
     }

package/dist/m365/spo/commands/listitem/listitem-retentionlabel-remove.js

@@ -56,7 +56,7 @@ class SpoListItemRetentionLabelRemoveCommand extends SpoCommand {
             return urlUtil.urlCombine(tenantUrl, serverRelativePath);
         }
         if (this.verbose) {
-            logger.logToStderr(`Retrieving list absolute URL...`);
+            await logger.logToStderr(`Retrieving list absolute URL...`);
         }
         let requestUrl = `${options.webUrl}/_api/web`;
         if (options.listId) {
@@ -76,7 +76,7 @@ class SpoListItemRetentionLabelRemoveCommand extends SpoCommand {
         const serverRelativePath = urlUtil.getServerRelativePath(options.webUrl, response.RootFolder.ServerRelativeUrl);
         const listAbsoluteUrl = urlUtil.urlCombine(tenantUrl, serverRelativePath);
         if (this.verbose) {
-            logger.logToStderr(`List absolute URL found: '${listAbsoluteUrl}'`);
+            await logger.logToStderr(`List absolute URL found: '${listAbsoluteUrl}'`);
         }
         return listAbsoluteUrl;
     }

package/dist/m365/spo/commands/site/site-commsite-enable.js

@@ -26,7 +26,7 @@ class SpoSiteCommSiteEnableCommand extends SpoCommand {
     async commandAction(logger, args) {
         const designPackageId = this.getDesignPackageId(args.options);
         if (this.verbose) {
-            logger.logToStderr(`Enabling communication site with design package '${designPackageId}' at '${args.options.url}'...`);
+            await logger.logToStderr(`Enabling communication site with design package '${designPackageId}' at '${args.options.url}'...`);
         }
         try {
             const requestOptions = {

package/dist/m365/spo/commands/spo-search.js

@@ -39,7 +39,7 @@ class SpoSearchCommand extends SpoCommand {
                 await logger.logToStderr(`Executing search query '${args.options.queryText}' on site at ${webUrl}...`);
             }
             const results = await this.executeSearchQuery(logger, args, webUrl, []);
-            this.printResults(logger, args, results);
+            await this.printResults(logger, args, results);
         }
         catch (err) {
             this.handleRejectedODataJsonPromise(err);

package/dist/m365/spo/commands/tenant/tenant-applicationcustomizer-set.js

@@ -76,7 +76,7 @@ class SpoTenantApplicationCustomizerSetCommand extends SpoCommand {
     }
     async getComponentManifest(appCatalogUrl, clientSideComponentId, logger) {
         if (this.verbose) {
-            logger.logToStderr('Retrieving component manifest item from the ComponentManifests list on the app catalog site so that we get the solution id');
+            await logger.logToStderr('Retrieving component manifest item from the ComponentManifests list on the app catalog site so that we get the solution id');
         }
         const camlQuery = `<View><ViewFields><FieldRef Name='ClientComponentId'></FieldRef><FieldRef Name='SolutionId'></FieldRef><FieldRef Name='ClientComponentManifest'></FieldRef></ViewFields><Query><Where><Eq><FieldRef Name='ClientComponentId' /><Value Type='Guid'>${clientSideComponentId}</Value></Eq></Where></Query></View>`;
         const commandOptions = {
@@ -89,7 +89,7 @@ class SpoTenantApplicationCustomizerSetCommand extends SpoCommand {
         };
         const output = await cli.executeCommandWithOutput(spoListItemListCommand, { options: { ...commandOptions, _: [] } });
         if (this.verbose) {
-            logger.logToStderr(output.stderr);
+            await logger.logToStderr(output.stderr);
         }
         const outputParsed = JSON.parse(output.stdout);
         if (outputParsed.length === 0) {
@@ -99,7 +99,7 @@ class SpoTenantApplicationCustomizerSetCommand extends SpoCommand {
     }
     async getSolutionFromAppCatalog(appCatalogUrl, solutionId, logger) {
         if (this.verbose) {
-            logger.logToStderr(`Retrieving solution with id ${solutionId} from the application catalog`);
+            await logger.logToStderr(`Retrieving solution with id ${solutionId} from the application catalog`);
         }
         const camlQuery = `<View><ViewFields><FieldRef Name='SkipFeatureDeployment'></FieldRef><FieldRef Name='ContainsTenantWideExtension'></FieldRef></ViewFields><Query><Where><Eq><FieldRef Name='AppProductID' /><Value Type='Guid'>${solutionId}</Value></Eq></Where></Query></View>`;
         const commandOptions = {
@@ -112,7 +112,7 @@ class SpoTenantApplicationCustomizerSetCommand extends SpoCommand {
         };
         const output = await cli.executeCommandWithOutput(spoListItemListCommand, { options: { ...commandOptions, _: [] } });
         if (this.verbose) {
-            logger.logToStderr(output.stderr);
+            await logger.logToStderr(output.stderr);
         }
         const outputParsed = JSON.parse(output.stdout);
         if (outputParsed.length === 0) {