@pnp/cli-microsoft365 7.1.0-beta.d53f0d9 → 7.2.0-beta.194fd07

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/aad/commands.js

@@ -1,5 +1,9 @@
 const prefix = 'aad';
 export default {
+    ADMINISTRATIVEUNIT_ADD: `${prefix} administrativeunit add`,
+    ADMINISTRATIVEUNIT_GET: `${prefix} administrativeunit get`,
+    ADMINISTRATIVEUNIT_LIST: `${prefix} administrativeunit list`,
+    ADMINISTRATIVEUNIT_REMOVE: `${prefix} administrativeunit remove`,
     APP_ADD: `${prefix} app add`,
     APP_GET: `${prefix} app get`,
     APP_LIST: `${prefix} app list`,
@@ -14,6 +18,7 @@ export default {
     GROUP_GET: `${prefix} group get`,
     GROUP_LIST: `${prefix} group list`,
     GROUP_REMOVE: `${prefix} group remove`,
+    GROUP_USER_LIST: `${prefix} group user list`,
     GROUPSETTING_ADD: `${prefix} groupsetting add`,
     GROUPSETTING_GET: `${prefix} groupsetting get`,
     GROUPSETTING_LIST: `${prefix} groupsetting list`,

package/dist/m365/base/AppCommand.js

@@ -8,6 +8,7 @@ import fs from 'fs';
 import { Cli } from '../../cli/Cli.js';
 import Command, { CommandError } from '../../Command.js';
 import { validation } from '../../utils/validation.js';
+import { formatting } from '../../utils/formatting.js';
 class AppCommand extends Command {
     get resource() {
         return 'https://graph.microsoft.com';
@@ -52,18 +53,8 @@ class AppCommand extends Command {
             return super.action(logger, args);
         }
         if (this.m365rcJson.apps.length > 1) {
-            const result = await Cli.prompt({
-                message: `Multiple Azure AD apps found in ${m365rcJsonPath}. Which app would you like to use?`,
-                type: 'list',
-                choices: this.m365rcJson.apps.map((app, i) => {
-                    return {
-                        name: `${app.name} (${app.appId})`,
-                        value: i
-                    };
-                }),
-                default: 0,
-                name: 'appIdIndex'
-            });
+            const resultAsKeyValuePair = formatting.convertArrayToHashTable('appIdIndex', this.m365rcJson.apps);
+            const result = await Cli.handleMultipleResultsFound(`Multiple Azure AD apps found in ${m365rcJsonPath}.`, resultAsKeyValuePair);
             this.appId = this.m365rcJson.apps[result.appIdIndex].appId;
             await super.action(logger, args);
         }

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

@@ -14,11 +14,14 @@ class CliDoctorCommand extends Command {
     }
     async commandAction(logger) {
         const roles = [];
-        const scopes = [];
+        const scopes = new Map();
         Object.keys(auth.service.accessTokens).forEach(resource => {
             const accessToken = auth.service.accessTokens[resource].accessToken;
             this.getRolesFromAccessToken(accessToken).forEach(role => roles.push(role));
-            this.getScopesFromAccessToken(accessToken).forEach(scope => scopes.push(scope));
+            const [res, scp] = this.getScopesFromAccessToken(accessToken);
+            if (res !== "") {
+                scopes.set(res, scp);
+            }
         });
         const diagnosticInfo = {
             os: {
@@ -34,7 +37,7 @@ class CliDoctorCommand extends Command {
             cliEnvironment: process.env.CLIMICROSOFT365_ENV ? process.env.CLIMICROSOFT365_ENV : '',
             cliConfig: Cli.getInstance().config.all,
             roles: roles,
-            scopes: scopes
+            scopes: Object.fromEntries(scopes)
         };
         await logger.log(diagnosticInfo);
     }
@@ -55,20 +58,22 @@ class CliDoctorCommand extends Command {
         return roles;
     }
     getScopesFromAccessToken(accessToken) {
+        let resource = "";
         let scopes = [];
         if (!accessToken || accessToken.length === 0) {
-            return scopes;
+            return [resource, scopes];
         }
         const chunks = accessToken.split('.');
         if (chunks.length !== 3) {
-            return scopes;
+            return [resource, scopes];
         }
         const tokenString = Buffer.from(chunks[1], 'base64').toString();
         const token = JSON.parse(tokenString);
         if (token.scp?.length > 0) {
+            resource = token.aud.replace(/(-my|-admin).sharepoint.com/, '.sharepoint.com');
             scopes = token.scp.split(' ');
         }
-        return scopes;
+        return [resource, scopes];
     }
 }
 export default new CliDoctorCommand();

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;
@@ -44,11 +45,13 @@ class SetupCommand extends AnonymousCommand {
             await this.configureSettings(settings, true, logger);
             return;
         }
+        // stop the spinner. Fixes #5598
+        Cli.getInstance().spinner.stop();
         await logger.logToStderr(`Welcome to the CLI for Microsoft 365 setup!`);
         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',
@@ -94,6 +97,8 @@ class SetupCommand extends AnonymousCommand {
             await logger.logToStderr('');
             await logger.logToStderr('Configuring settings...');
             await logger.logToStderr('');
+            // start the spinner. Fixes #5598
+            Cli.getInstance().spinner.start();
             await this.configureSettings(settings, false, logger);
             if (!this.verbose) {
                 await logger.logToStderr('');

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

@@ -4,7 +4,9 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
     return kind === "m" ? f : kind === "a" ? f.call(receiver) : f ? f.value : state.get(receiver);
 };
 var _SpoSiteDesignGetCommand_instances, _SpoSiteDesignGetCommand_initTelemetry, _SpoSiteDesignGetCommand_initOptions, _SpoSiteDesignGetCommand_initValidators, _SpoSiteDesignGetCommand_initOptionSets;
+import { Cli } from '../../../../cli/Cli.js';
 import request from '../../../../request.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';
@@ -24,9 +26,9 @@ class SpoSiteDesignGetCommand extends SpoCommand {
         __classPrivateFieldGet(this, _SpoSiteDesignGetCommand_instances, "m", _SpoSiteDesignGetCommand_initValidators).call(this);
         __classPrivateFieldGet(this, _SpoSiteDesignGetCommand_instances, "m", _SpoSiteDesignGetCommand_initOptionSets).call(this);
     }
-    getSiteDesignId(args, spoUrl) {
+    async getSiteDesignId(args, spoUrl) {
         if (args.options.id) {
-            return Promise.resolve(args.options.id);
+            return args.options.id;
         }
         const requestOptions = {
             url: `${spoUrl}/_api/Microsoft.Sharepoint.Utilities.WebTemplateExtensions.SiteScriptUtility.GetSiteDesigns`,
@@ -35,18 +37,17 @@ class SpoSiteDesignGetCommand extends SpoCommand {
             },
             responseType: 'json'
         };
-        return request
-            .post(requestOptions)
-            .then(response => {
-            const matchingSiteDesigns = response.value.filter(x => x.Title === args.options.title);
-            if (matchingSiteDesigns.length === 0) {
-                return Promise.reject(`The specified site design does not exist`);
-            }
-            if (matchingSiteDesigns.length > 1) {
-                return Promise.reject(`Multiple site designs with title ${args.options.title} found: ${matchingSiteDesigns.map(x => x.Id).join(', ')}`);
-            }
-            return Promise.resolve(matchingSiteDesigns[0].Id);
-        });
+        const response = await request.post(requestOptions);
+        const matchingSiteDesigns = response.value.filter(x => x.Title === args.options.title);
+        if (matchingSiteDesigns.length === 0) {
+            throw `The specified site design does not exist`;
+        }
+        if (matchingSiteDesigns.length > 1) {
+            const resultAsKeyValuePair = formatting.convertArrayToHashTable('Id', matchingSiteDesigns);
+            const result = await Cli.handleMultipleResultsFound(`Multiple site designs with title '${args.options.title}' found.`, resultAsKeyValuePair);
+            return result.Id;
+        }
+        return matchingSiteDesigns[0].Id;
     }
     async commandAction(logger, args) {
         try {

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

@@ -25,28 +25,8 @@ class SpoSiteDesignRemoveCommand extends SpoCommand {
         __classPrivateFieldGet(this, _SpoSiteDesignRemoveCommand_instances, "m", _SpoSiteDesignRemoveCommand_initValidators).call(this);
     }
     async commandAction(logger, args) {
-        const removeSiteDesign = async () => {
-            try {
-                const spoUrl = await spo.getSpoUrl(logger, this.debug);
-                const requestDigest = await spo.getRequestDigest(spoUrl);
-                const requestOptions = {
-                    url: `${spoUrl}/_api/Microsoft.Sharepoint.Utilities.WebTemplateExtensions.SiteScriptUtility.DeleteSiteDesign`,
-                    headers: {
-                        'X-RequestDigest': requestDigest.FormDigestValue,
-                        'content-type': 'application/json;charset=utf-8',
-                        accept: 'application/json;odata=nometadata'
-                    },
-                    data: { id: args.options.id },
-                    responseType: 'json'
-                };
-                await request.post(requestOptions);
-            }
-            catch (err) {
-                this.handleRejectedODataJsonPromise(err);
-            }
-        };
         if (args.options.force) {
-            await removeSiteDesign();
+            await this.removeSiteDesign(logger, args.options.id);
         }
         else {
             const result = await Cli.prompt({
@@ -56,10 +36,30 @@ class SpoSiteDesignRemoveCommand extends SpoCommand {
                 message: `Are you sure you want to remove the site design ${args.options.id}?`
             });
             if (result.continue) {
-                await removeSiteDesign();
+                await this.removeSiteDesign(logger, args.options.id);
             }
         }
     }
+    async removeSiteDesign(logger, id) {
+        try {
+            const spoUrl = await spo.getSpoUrl(logger, this.debug);
+            const requestDigest = await spo.getRequestDigest(spoUrl);
+            const requestOptions = {
+                url: `${spoUrl}/_api/Microsoft.Sharepoint.Utilities.WebTemplateExtensions.SiteScriptUtility.DeleteSiteDesign`,
+                headers: {
+                    'X-RequestDigest': requestDigest.FormDigestValue,
+                    'content-type': 'application/json;charset=utf-8',
+                    accept: 'application/json;odata=nometadata'
+                },
+                data: { id: id },
+                responseType: 'json'
+            };
+            await request.post(requestOptions);
+        }
+        catch (err) {
+            this.handleRejectedODataJsonPromise(err);
+        }
+    }
 }
 _SpoSiteDesignRemoveCommand_instances = new WeakSet(), _SpoSiteDesignRemoveCommand_initTelemetry = function _SpoSiteDesignRemoveCommand_initTelemetry() {
     this.telemetry.push((args) => {

package/dist/m365/teams/commands/app/app-update.js

@@ -11,6 +11,7 @@ import { formatting } from '../../../../utils/formatting.js';
 import { validation } from '../../../../utils/validation.js';
 import GraphCommand from '../../../base/GraphCommand.js';
 import commands from '../../commands.js';
+import { Cli } from '../../../../cli/Cli.js';
 class TeamsAppUpdateCommand extends GraphCommand {
     get name() {
         return commands.APP_UPDATE;
@@ -29,7 +30,7 @@ class TeamsAppUpdateCommand extends GraphCommand {
     async commandAction(logger, args) {
         const { filePath } = args.options;
         try {
-            const appId = await this.getAppId(args);
+            const appId = await this.getAppId(args.options);
             const fullPath = path.resolve(filePath);
             if (this.verbose) {
                 await logger.logToStderr(`Updating app with id '${appId}' and file '${fullPath}' in the app catalog...`);
@@ -47,12 +48,12 @@ class TeamsAppUpdateCommand extends GraphCommand {
             this.handleRejectedODataJsonPromise(err);
         }
     }
-    async getAppId(args) {
-        if (args.options.id) {
-            return args.options.id;
+    async getAppId(options) {
+        if (options.id) {
+            return options.id;
         }
         const requestOptions = {
-            url: `${this.resource}/v1.0/appCatalogs/teamsApps?$filter=displayName eq '${formatting.encodeQueryParameter(args.options.name)}'`,
+            url: `${this.resource}/v1.0/appCatalogs/teamsApps?$filter=displayName eq '${formatting.encodeQueryParameter(options.name)}'`,
             headers: {
                 accept: 'application/json;odata.metadata=none'
             },
@@ -64,7 +65,9 @@ class TeamsAppUpdateCommand extends GraphCommand {
             throw `The specified Teams app does not exist`;
         }
         if (response.value.length > 1) {
-            throw `Multiple Teams apps with name ${args.options.name} found. Please choose one of these ids: ${response.value.map(x => x.id).join(', ')}`;
+            const resultAsKeyValuePair = formatting.convertArrayToHashTable('id', response.value);
+            const result = await Cli.handleMultipleResultsFound(`Multiple Teams apps with name ${options.name} found.`, resultAsKeyValuePair);
+            return result.id;
         }
         return app.id;
     }

package/dist/m365/todo/commands/task/task-list.js

@@ -26,9 +26,9 @@ class TodoTaskListCommand extends GraphCommand {
         __classPrivateFieldGet(this, _TodoTaskListCommand_instances, "m", _TodoTaskListCommand_initOptions).call(this);
         __classPrivateFieldGet(this, _TodoTaskListCommand_instances, "m", _TodoTaskListCommand_initOptionSets).call(this);
     }
-    getTodoListId(args) {
+    async getTodoListId(args) {
         if (args.options.listId) {
-            return Promise.resolve(args.options.listId);
+            return args.options.listId;
         }
         const requestOptions = {
             url: `${this.resource}/v1.0/me/todo/lists?$filter=displayName eq '${escape(args.options.listName)}'`,
@@ -37,14 +37,12 @@ class TodoTaskListCommand extends GraphCommand {
             },
             responseType: 'json'
         };
-        return request.get(requestOptions)
-            .then(response => {
-            const taskList = response.value[0];
-            if (!taskList) {
-                return Promise.reject(`The specified task list does not exist`);
-            }
-            return Promise.resolve(taskList.id);
-        });
+        const response = await request.get(requestOptions);
+        const taskList = response.value[0];
+        if (!taskList) {
+            throw `The specified task list does not exist`;
+        }
+        return taskList.id;
     }
     async commandAction(logger, args) {
         try {

package/dist/m365/todo/commands/task/task-remove.js

@@ -23,42 +23,8 @@ class TodoTaskRemoveCommand extends GraphCommand {
         __classPrivateFieldGet(this, _TodoTaskRemoveCommand_instances, "m", _TodoTaskRemoveCommand_initOptionSets).call(this);
     }
     async commandAction(logger, args) {
-        const getToDoListId = async () => {
-            if (args.options.listName) {
-                // Search list by its name
-                const requestOptions = {
-                    url: `${this.resource}/v1.0/me/todo/lists?$filter=displayName eq '${escape(args.options.listName)}'`,
-                    headers: {
-                        accept: "application/json;odata.metadata=none"
-                    },
-                    responseType: 'json'
-                };
-                const response = await request.get(requestOptions);
-                return response.value && response.value.length === 1 ? response.value[0].id : undefined;
-            }
-            return Promise.resolve(args.options.listId);
-        };
-        const removeToDoTask = async () => {
-            try {
-                const toDoListId = await getToDoListId();
-                if (!toDoListId) {
-                    throw `The list ${args.options.listName} cannot be found`;
-                }
-                const requestOptions = {
-                    url: `${this.resource}/v1.0/me/todo/lists/${toDoListId}/tasks/${args.options.id}`,
-                    headers: {
-                        accept: "application/json;odata.metadata=none"
-                    },
-                    responseType: 'json'
-                };
-                await request.delete(requestOptions);
-            }
-            catch (err) {
-                this.handleRejectedODataJsonPromise(err);
-            }
-        };
         if (args.options.force) {
-            await removeToDoTask();
+            await this.removeToDoTask(args.options);
         }
         else {
             const result = await Cli.prompt({
@@ -68,10 +34,44 @@ class TodoTaskRemoveCommand extends GraphCommand {
                 message: `Are you sure you want to remove the task ${args.options.id} from list ${args.options.listId || args.options.listName}?`
             });
             if (result.continue) {
-                await removeToDoTask();
+                await this.removeToDoTask(args.options);
             }
         }
     }
+    async getToDoListId(options) {
+        if (options.listName) {
+            // Search list by its name
+            const requestOptions = {
+                url: `${this.resource}/v1.0/me/todo/lists?$filter=displayName eq '${escape(options.listName)}'`,
+                headers: {
+                    accept: "application/json;odata.metadata=none"
+                },
+                responseType: 'json'
+            };
+            const response = await request.get(requestOptions);
+            return response.value && response.value.length === 1 ? response.value[0].id : undefined;
+        }
+        return options.listId;
+    }
+    async removeToDoTask(options) {
+        try {
+            const toDoListId = await this.getToDoListId(options);
+            if (!toDoListId) {
+                throw `The list ${options.listName} cannot be found`;
+            }
+            const requestOptions = {
+                url: `${this.resource}/v1.0/me/todo/lists/${toDoListId}/tasks/${options.id}`,
+                headers: {
+                    accept: "application/json;odata.metadata=none"
+                },
+                responseType: 'json'
+            };
+            await request.delete(requestOptions);
+        }
+        catch (err) {
+            this.handleRejectedODataJsonPromise(err);
+        }
+    }
 }
 _TodoTaskRemoveCommand_instances = new WeakSet(), _TodoTaskRemoveCommand_initTelemetry = function _TodoTaskRemoveCommand_initTelemetry() {
     this.telemetry.push((args) => {

package/dist/m365/todo/commands/task/task-set.js

@@ -28,7 +28,7 @@ class TodoTaskSetCommand extends GraphCommand {
         const endpoint = `${this.resource}/v1.0`;
         const data = this.mapRequestBody(args.options);
         try {
-            const listId = await this.getTodoListId(args);
+            const listId = await this.getTodoListId(args.options);
             const requestOptions = {
                 url: `${endpoint}/me/todo/lists/${listId}/tasks/${formatting.encodeQueryParameter(args.options.id)}`,
                 headers: {
@@ -45,25 +45,23 @@ class TodoTaskSetCommand extends GraphCommand {
             this.handleRejectedODataJsonPromise(err);
         }
     }
-    getTodoListId(args) {
-        if (args.options.listId) {
-            return Promise.resolve(args.options.listId);
+    async getTodoListId(options) {
+        if (options.listId) {
+            return options.listId;
         }
         const requestOptions = {
-            url: `${this.resource}/v1.0/me/todo/lists?$filter=displayName eq '${escape(args.options.listName)}'`,
+            url: `${this.resource}/v1.0/me/todo/lists?$filter=displayName eq '${escape(options.listName)}'`,
             headers: {
                 accept: 'application/json;odata.metadata=none'
             },
             responseType: 'json'
         };
-        return request.get(requestOptions)
-            .then(response => {
-            const taskList = response.value[0];
-            if (!taskList) {
-                return Promise.reject(`The specified task list does not exist`);
-            }
-            return Promise.resolve(taskList.id);
-        });
+        const response = await request.get(requestOptions);
+        const taskList = response.value[0];
+        if (!taskList) {
+            throw `The specified task list does not exist`;
+        }
+        return taskList.id;
     }
     getDateTimeTimeZone(dateTime) {
         return {