@pnp/cli-microsoft365 7.10.0-beta.ebb7426 → 7.11.0-beta.10b9b79

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

@@ -34,11 +34,11 @@ class SpoGroupMemberAddCommand extends SpoCommand {
         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);

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-admin-list.js

@@ -0,0 +1,144 @@
+var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (receiver, state, kind, f) {
+    if (kind === "a" && !f) throw new TypeError("Private accessor was defined without a getter");
+    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 _SpoSiteAdminListCommand_instances, _SpoSiteAdminListCommand_initTelemetry, _SpoSiteAdminListCommand_initOptions, _SpoSiteAdminListCommand_initValidators, _SpoSiteAdminListCommand_initTypes;
+import request from '../../../../request.js';
+import { spo } from '../../../../utils/spo.js';
+import { validation } from '../../../../utils/validation.js';
+import SpoCommand from '../../../base/SpoCommand.js';
+import commands from '../../commands.js';
+import { ListPrincipalType } from '../list/ListPrincipalType.js';
+class SpoSiteAdminListCommand extends SpoCommand {
+    get name() {
+        return commands.SITE_ADMIN_LIST;
+    }
+    get description() {
+        return 'Lists all administrators of a specific SharePoint site';
+    }
+    constructor() {
+        super();
+        _SpoSiteAdminListCommand_instances.add(this);
+        __classPrivateFieldGet(this, _SpoSiteAdminListCommand_instances, "m", _SpoSiteAdminListCommand_initTelemetry).call(this);
+        __classPrivateFieldGet(this, _SpoSiteAdminListCommand_instances, "m", _SpoSiteAdminListCommand_initOptions).call(this);
+        __classPrivateFieldGet(this, _SpoSiteAdminListCommand_instances, "m", _SpoSiteAdminListCommand_initValidators).call(this);
+        __classPrivateFieldGet(this, _SpoSiteAdminListCommand_instances, "m", _SpoSiteAdminListCommand_initTypes).call(this);
+    }
+    async commandAction(logger, args) {
+        try {
+            if (args.options.asAdmin) {
+                await this.callActionAsAdmin(logger, args);
+                return;
+            }
+            await this.callAction(logger, args);
+        }
+        catch (err) {
+            this.handleRejectedODataJsonPromise(err);
+        }
+    }
+    async callActionAsAdmin(logger, args) {
+        if (this.verbose) {
+            await logger.logToStderr('Retrieving site administrators as an administrator...');
+        }
+        const adminUrl = await spo.getSpoAdminUrl(logger, this.debug);
+        const siteId = await this.getSiteId(args.options.siteUrl, logger);
+        const requestOptions = {
+            url: `${adminUrl}/_api/SPO.Tenant/GetSiteAdministrators?siteId='${siteId}'`,
+            headers: {
+                accept: 'application/json;odata=nometadata',
+                'content-type': 'application/json;charset=utf-8'
+            }
+        };
+        const response = await request.post(requestOptions);
+        const responseContent = JSON.parse(response);
+        const primaryAdminLoginName = await this.getPrimaryAdminLoginNameFromAdmin(adminUrl, siteId);
+        const mappedResult = responseContent.value.map((u) => ({
+            Id: null,
+            Email: u.email,
+            LoginName: u.loginName,
+            Title: u.name,
+            PrincipalType: null,
+            PrincipalTypeString: null,
+            IsPrimaryAdmin: u.loginName === primaryAdminLoginName
+        }));
+        await logger.log(mappedResult);
+    }
+    async getSiteId(siteUrl, logger) {
+        const siteGraphId = await spo.getSiteId(siteUrl, logger, this.verbose);
+        const match = siteGraphId.match(/,([a-f0-9\-]{36}),/i);
+        if (!match) {
+            throw `Site with URL ${siteUrl} not found`;
+        }
+        return match[1];
+    }
+    async getPrimaryAdminLoginNameFromAdmin(adminUrl, siteId) {
+        const requestOptions = {
+            url: `${adminUrl}/_api/SPO.Tenant/sites('${siteId}')?$select=OwnerLoginName`,
+            headers: {
+                accept: 'application/json;odata=nometadata',
+                'content-type': 'application/json;charset=utf-8'
+            }
+        };
+        const response = await request.get(requestOptions);
+        const responseContent = JSON.parse(response);
+        return responseContent.OwnerLoginName;
+    }
+    async callAction(logger, args) {
+        if (this.verbose) {
+            await logger.logToStderr('Retrieving site administrators...');
+        }
+        const requestOptions = {
+            url: `${args.options.siteUrl}/_api/web/siteusers?$filter=IsSiteAdmin eq true`,
+            method: 'GET',
+            headers: {
+                'accept': 'application/json;odata=nometadata'
+            },
+            responseType: 'json'
+        };
+        const responseContent = await request.get(requestOptions);
+        const primaryOwnerLogin = await this.getPrimaryOwnerLoginFromSite(args.options.siteUrl);
+        const mappedResult = responseContent.value.map((u) => ({
+            Id: u.Id,
+            LoginName: u.LoginName,
+            Title: u.Title,
+            PrincipalType: u.PrincipalType,
+            PrincipalTypeString: ListPrincipalType[u.PrincipalType],
+            Email: u.Email,
+            IsPrimaryAdmin: u.LoginName === primaryOwnerLogin
+        }));
+        await logger.log(mappedResult);
+    }
+    async getPrimaryOwnerLoginFromSite(siteUrl) {
+        const requestOptions = {
+            url: `${siteUrl}/_api/site/owner`,
+            method: 'GET',
+            headers: {
+                'accept': 'application/json;odata=nometadata'
+            },
+            responseType: 'json'
+        };
+        const responseContent = await request.get(requestOptions);
+        return responseContent?.LoginName ?? null;
+    }
+}
+_SpoSiteAdminListCommand_instances = new WeakSet(), _SpoSiteAdminListCommand_initTelemetry = function _SpoSiteAdminListCommand_initTelemetry() {
+    this.telemetry.push((args) => {
+        Object.assign(this.telemetryProperties, {
+            siteUrl: typeof args.options.siteUrl !== 'undefined',
+            asAdmin: !!args.options.asAdmin
+        });
+    });
+}, _SpoSiteAdminListCommand_initOptions = function _SpoSiteAdminListCommand_initOptions() {
+    this.options.unshift({
+        option: '-u, --siteUrl <siteUrl>'
+    }, {
+        option: '--asAdmin'
+    });
+}, _SpoSiteAdminListCommand_initValidators = function _SpoSiteAdminListCommand_initValidators() {
+    this.validators.push(async (args) => validation.isValidSharePointUrl(args.options.siteUrl));
+}, _SpoSiteAdminListCommand_initTypes = function _SpoSiteAdminListCommand_initTypes() {
+    this.types.string.push('siteUrl');
+};
+export default new SpoSiteAdminListCommand();
+//# sourceMappingURL=site-admin-list.js.map

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) {

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

@@ -44,7 +44,7 @@ class SpoTenantCommandSetSetCommand extends SpoCommand {
     }
     async getListItemById(logger, webUrl, listServerRelativeUrl, id) {
         if (this.verbose) {
-            logger.logToStderr(`Getting the list item by id ${id}`);
+            await logger.logToStderr(`Getting the list item by id ${id}`);
         }
         const reqOptions = {
             url: `${webUrl}/_api/web/GetList('${formatting.encodeQueryParameter(listServerRelativeUrl)}')/Items(${id})`,
@@ -57,7 +57,7 @@ class SpoTenantCommandSetSetCommand extends SpoCommand {
     }
     async updateTenantWideExtension(appCatalogUrl, options, listServerRelativeUrl, logger) {
         if (this.verbose) {
-            logger.logToStderr('Updating tenant wide extension to the TenantWideExtensions list');
+            await logger.logToStderr('Updating tenant wide extension to the TenantWideExtensions list');
         }
         const formValues = [];
         if (options.newTitle !== undefined) {

package/dist/m365/spo/commands/user/user-ensure.js

@@ -27,7 +27,7 @@ class SpoUserEnsureCommand extends SpoCommand {
     async commandAction(logger, args) {
         if (args.options.aadId) {
             args.options.entraId = args.options.aadId;
-            this.warn(logger, `Option 'aadId' is deprecated. Please use 'entraId' instead`);
+            await this.warn(logger, `Option 'aadId' is deprecated. Please use 'entraId' instead`);
         }
         if (this.verbose) {
             await logger.logToStderr(`Ensuring user ${args.options.entraId || args.options.userName} at site ${args.options.webUrl}`);

package/dist/m365/spo/commands.js

@@ -239,6 +239,7 @@ export default {
     SERVICEPRINCIPAL_SET: `${prefix} serviceprincipal set`,
     SET: `${prefix} set`,
     SITE_ADD: `${prefix} site add`,
+    SITE_ADMIN_LIST: `${prefix} site admin list`,
     SITE_APPCATALOG_ADD: `${prefix} site appcatalog add`,
     SITE_APPCATALOG_LIST: `${prefix} site appcatalog list`,
     SITE_APPCATALOG_REMOVE: `${prefix} site appcatalog remove`,

package/dist/m365/teams/commands/chat/chat-member-add.js

@@ -27,7 +27,7 @@ class TeamsChatMemberAddCommand extends GraphCommand {
     async commandAction(logger, args) {
         try {
             if (this.verbose) {
-                logger.logToStderr(`Adding member ${args.options.userId || args.options.userName} to chat with id ${args.options.chatId}...`);
+                await logger.logToStderr(`Adding member ${args.options.userId || args.options.userName} to chat with id ${args.options.chatId}...`);
             }
             const chatMemberAddOptions = {
                 url: `${this.resource}/v1.0/chats/${args.options.chatId}/members`,

package/dist/m365/teams/commands/meeting/meeting-list.js

@@ -90,7 +90,7 @@ class TeamsMeetingListCommand extends GraphCommand {
         let result = [];
         for (let i = 0; i < meetingUrls.length; i += 20) {
             if (this.verbose) {
-                logger.logToStderr(`Retrieving meetings ${i + 1}-${Math.min(i + 20, meetingUrls.length)}...`);
+                await logger.logToStderr(`Retrieving meetings ${i + 1}-${Math.min(i + 20, meetingUrls.length)}...`);
             }
             const batch = meetingUrls.slice(i, i + 20);
             const requestOptions = {

package/dist/m365/viva/commands/engage/engage-community-get.js

@@ -22,7 +22,7 @@ class VivaEngageCommunityGetCommand extends GraphCommand {
     }
     async commandAction(logger, args) {
         if (this.verbose) {
-            logger.logToStderr(`Getting the information of Viva Engage community with id '${args.options.id}'...`);
+            await logger.logToStderr(`Getting the information of Viva Engage community with id '${args.options.id}'...`);
         }
         const requestOptions = {
             url: `${this.resource}/beta/employeeExperience/communities/${args.options.id}`,

package/dist/request.js

@@ -14,39 +14,39 @@ class Request {
         }
         this._debug = debug;
         if (this._debug) {
-            this.req.interceptors.request.use(config => {
+            this.req.interceptors.request.use(async (config) => {
                 if (this._logger) {
-                    this._logger.logToStderr('Request:');
+                    await this._logger.logToStderr('Request:');
                     const properties = ['url', 'method', 'headers', 'responseType', 'decompress'];
                     if (config.responseType !== 'stream') {
                         properties.push('data');
                     }
-                    this._logger.logToStderr(JSON.stringify(formatting.filterObject(config, properties), null, 2));
+                    await this._logger.logToStderr(JSON.stringify(formatting.filterObject(config, properties), null, 2));
                 }
                 return config;
             });
             // since we're stubbing requests, response interceptor is never called in
             // tests, so let's exclude it from coverage
             /* c8 ignore next 26 */
-            this.req.interceptors.response.use((response) => {
+            this.req.interceptors.response.use(async (response) => {
                 if (this._logger) {
-                    this._logger.logToStderr('Response:');
+                    await this._logger.logToStderr('Response:');
                     const properties = ['status', 'statusText', 'headers'];
                     if (response.headers['content-type'] &&
                         response.headers['content-type'].indexOf('json') > -1) {
                         properties.push('data');
                     }
-                    this._logger.logToStderr(JSON.stringify({
+                    await this._logger.logToStderr(JSON.stringify({
                         url: response.config.url,
                         ...formatting.filterObject(response, properties)
                     }, null, 2));
                 }
                 return response;
-            }, (error) => {
+            }, async (error) => {
                 if (this._logger) {
                     const properties = ['status', 'statusText', 'headers'];
-                    this._logger.logToStderr('Request error:');
-                    this._logger.logToStderr(JSON.stringify({
+                    await this._logger.logToStderr('Request error:');
+                    await this._logger.logToStderr(JSON.stringify({
                         url: error.config?.url,
                         ...formatting.filterObject(error.response, properties),
                         error: error.error
@@ -169,7 +169,7 @@ class Request {
                     timings.api.push(Number(end - start));
                     _resolve((options.responseType === 'stream' || options.fullResponse) ? res : res.data);
                 }
-            }, (error) => {
+            }, async (error) => {
                 if (error && error.response &&
                     (error.response.status === 429 ||
                         error.response.status === 503)) {
@@ -178,11 +178,10 @@ class Request {
                         retryAfter = 10;
                     }
                     if (this._debug) {
-                        this._logger.log(`Request throttled. Waiting ${retryAfter}sec before retrying...`);
+                        await this._logger.log(`Request throttled. Waiting ${retryAfter}sec before retrying...`);
                     }
-                    setTimeout(() => {
-                        this.execute(options, resolve || _resolve, reject || _reject);
-                    }, retryAfter * 1000);
+                    // eslint-disable-next-line @typescript-eslint/no-floating-promises
+                    setTimeout(async () => { this.execute(options, resolve || _resolve, reject || _reject); }, retryAfter * 1000);
                 }
                 else {
                     if (reject) {

package/dist/utils/spo.js

@@ -1173,7 +1173,7 @@ export const spo = {
     */
     async getFileAsListItemByUrl(absoluteListUrl, url, logger, verbose) {
         if (verbose && logger) {
-            logger.logToStderr(`Getting the file properties with url ${url}`);
+            await logger.logToStderr(`Getting the file properties with url ${url}`);
         }
         const serverRelativePath = urlUtil.getServerRelativePath(absoluteListUrl, url);
         const requestUrl = `${absoluteListUrl}/_api/web/GetFileByServerRelativePath(DecodedUrl=@f)?$expand=ListItemAllFields&@f='${formatting.encodeQueryParameter(serverRelativePath)}'`;
@@ -1206,7 +1206,7 @@ export const spo = {
         const serverRelativeSiteMatch = absoluteListUrl.match(new RegExp('/sites/[^/]+'));
         const webUrl = `${parsedUrl.protocol}//${parsedUrl.host}${serverRelativeSiteMatch ?? ''}`;
         if (verbose && logger) {
-            logger.logToStderr(`Getting list id...`);
+            await logger.logToStderr(`Getting list id...`);
         }
         const listRequestOptions = {
             url: `${absoluteListUrl}?$select=Id`,
@@ -1218,7 +1218,7 @@ export const spo = {
         const list = await request.get(listRequestOptions);
         const listId = list.Id;
         if (verbose && logger) {
-            logger.logToStderr(`Getting request digest for systemUpdate request`);
+            await logger.logToStderr(`Getting request digest for systemUpdate request`);
         }
         const res = await spo.getRequestDigest(webUrl);
         const formDigestValue = res.FormDigestValue;
@@ -1361,7 +1361,7 @@ export const spo = {
      */
     async getSiteId(webUrl, logger, verbose) {
         if (verbose && logger) {
-            logger.logToStderr(`Getting site id for URL: ${webUrl}...`);
+            await logger.logToStderr(`Getting site id for URL: ${webUrl}...`);
         }
         const url = new URL(webUrl);
         const requestOptions = {
@@ -1393,7 +1393,7 @@ export const spo = {
         };
         const response = await request.post(requestOptions);
         if (verbose) {
-            logger.logToStderr('Attempt to get _ObjectIdentity_ key values');
+            await logger.logToStderr('Attempt to get _ObjectIdentity_ key values');
         }
         const json = JSON.parse(response);
         const contents = json.find(x => { return x.ErrorInfo; });

package/dist/utils/validation.js

@@ -332,6 +332,12 @@ export const validation = {
     isValidISODuration(duration) {
         const durationRegEx = new RegExp(/^P(?!$)((\d+Y)|(\d+\.\d+Y$))?((\d+M)|(\d+\.\d+M$))?((\d+W)|(\d+\.\d+W$))?((\d+D)|(\d+\.\d+D$))?(T(?=\d)((\d+H)|(\d+\.\d+H$))?((\d+M)|(\d+\.\d+M$))?(\d+(\.\d+)?S)?)??$/);
         return durationRegEx.test(duration);
+    },
+    isValidPermission(permissions) {
+        const invalidPermissions = permissions
+            .split(' ')
+            .filter(permission => permission.indexOf('/') < 0);
+        return invalidPermissions.length > 0 ? invalidPermissions : true;
     }
 };
 //# sourceMappingURL=validation.js.map

package/docs/docs/cmd/entra/app/app-permission-remove.mdx

@@ -14,19 +14,19 @@ m365 entra app permission remove [options]
 
 ```md definition-list
 `-i, --appId [appId]`
-: Client ID of the Microsoft Entra app to add the API permissions to. Specify either `appId`, `appName` or `appObjectId`.
+: Client ID of the Microsoft Entra app to remove the API permissions from. Specify either `appId`, `appName` or `appObjectId`.
 
 `--appObjectId [appObjectId]`
-: Object ID of the Microsoft Entra app to add the API permissions to. Specify either `appId`, `appName` or `appObjectId`.
+: Object ID of the Microsoft Entra app to remove the API permissions from. Specify either `appId`, `appName` or `appObjectId`.
 
 `-n, --appName [appName]`
 : Display name of the Entra app to remove the API permissions from. Specify either `appId`, `appName` or `appObjectId`.
 
 `-a, --applicationPermissions [applicationPermissions]`
-: Space-separated list of application permissions to add. Specify at least `applicationPermissions` or `delegatedPermissions`.
+: Space-separated list of application permissions to remove. Specify at least `applicationPermissions` or `delegatedPermissions`.
 
 `-d, --delegatedPermissions [delegatedPermissions]`
-: Space-separated list of delegated permissions to add. Specify at least `applicationPermissions` or `delegatedPermissions`.
+: Space-separated list of delegated permissions to remove. Specify at least `applicationPermissions` or `delegatedPermissions`.
 
 `--revokeAdminConsent`
 : When specified, revokes the admin consent for the specified permissions as well.

package/docs/docs/cmd/spo/site/site-admin-list.mdx

@@ -0,0 +1,115 @@
+import Global from '/docs/cmd/_global.mdx';
+import Tabs from '@theme/Tabs';
+import TabItem from '@theme/TabItem';
+
+# spo site admin list
+
+Lists all administrators of a specific SharePoint site
+
+## Usage
+
+```sh
+m365 spo site admin list [options]
+```
+
+## Options
+
+```md definition-list
+`-u, --siteUrl <siteUrl>`
+: The URL of the SharePoint site
+
+`--asAdmin`
+: List admins as admin for sites you don't have permission to
+```
+
+<Global />
+
+## Remarks
+
+:::info
+
+To use this command with the `--asAdmin` mode, you must have permission to access the tenant admin site.
+
+Without this parameter, you must have site collection admin permissions for the requested site.
+
+In `--asAdmin` mode, the Id, PrincipalType, and PrincipalTypeString properties are not exported.
+
+:::
+
+## Examples
+
+Lists all admins of a SharePoint site
+
+```sh
+m365 spo site admin list --siteUrl https://contoso.sharepoint.com
+```
+
+Lists all admins of a SharePoint site as admin
+
+```sh
+m365 spo site admin list --siteUrl https://contoso.sharepoint.com --asAdmin
+```
+
+## Response
+
+<Tabs>
+  <TabItem value="JSON">
+
+  ```json
+  [
+    {
+      "Id": 6,
+      "LoginName": "i:0#.f|membership|user@contoso.com",
+      "Title": "User Example",
+      "PrincipalType": 1,
+      "PrincipalTypeString": "User",
+      "Email": "user@contoso.com",
+      "IsPrimaryAdmin": true
+    }
+  ]
+  ```
+
+  </TabItem>
+  <TabItem value="Text">
+
+  ```text
+  Email               : user@contoso.com
+  Id                  : 6
+  LoginName           : i:0#.f|membership|user@contoso.com
+  PrincipalType       : 1
+  PrincipalTypeString : User
+  Title               : User Example
+  IsPrimaryAdmin      : true
+  ```
+
+  </TabItem>
+  <TabItem value="CSV">
+
+  ```csv
+  Id,LoginName,Title,PrincipalType,PrincipalTypeString,Email,IsPrimaryAdmin
+  6,i:0#.f|membership|user@contoso.com,User Example,1,User,user@contoso.com,1
+  ```
+
+  </TabItem>
+  <TabItem value="Markdown">
+
+  ```md
+  # spo site admin list --siteUrl "https://contoso.sharepoint.com/sites/Test"
+
+  Date: 20/03/2024
+
+  ## User
+
+  Property | Value
+  ---------|-------
+  Id | 6
+  LoginName | i:0#.f\|membership\|user@contoso.com
+  Title | User Example
+  PrincipalType | 1
+  PrincipalTypeString | User
+  Email | user@contoso.com
+  IsPrimaryAdmin | true
+  ```
+
+  </TabItem>
+</Tabs>