update components to address review comments

Author: js07

components/slack/actions/update-profile/update-profile.mjs

@@ -72,6 +72,7 @@ export default {
       && !this.phone
       && !this.pronouns
       && !this.title
+      && !this.email
     ) {
       throw new ConfigurationError("Please provide at least one value to update");
     }
@@ -83,6 +84,7 @@ export default {
         phone: this.phone,
         pronouns: this.pronouns,
         title: this.title,
+        email: this.email,
       },
       user: this.user,
     });

components/slack_v2/actions/common/build-blocks.mjs

@@ -94,7 +94,7 @@ export default {
     if (!this.passArrayOrConfigure) {
       return props;
     }
-    if (this.passArrayOrConfigure == "array") {
+    if (this.passArrayOrConfigure === "array") {
       props.blocks = {
         type: common.props.slack.propDefinitions.blocks.type,
         label: common.props.slack.propDefinitions.blocks.label,
@@ -159,7 +159,7 @@ export default {
   },
   async run() {
     let blocks = [];
-    if (this.passArrayOrConfigure == "array") {
+    if (this.passArrayOrConfigure === "array") {
       blocks = this.blocks;
     } else {
       for (let i = 1; i <= 5; i++) {

components/slack_v2/actions/find-message/find-message.mjs

@@ -39,7 +39,7 @@ export default {
     sortDirection: {
       type: "string",
       label: "Sort Direction",
-      description: "Sort ascending (asc) or descending (desc)`",
+      description: "Sort ascending (asc) or descending (desc)",
       options: [
         "desc",
         "asc",

components/slack_v2/actions/invite-user-to-channel/invite-user-to-channel.mjs

@@ -35,7 +35,7 @@ export default {
       $.export("$summary", `Successfully invited user ${this.user} to channel with ID ${this.conversation}`);
       return response;
     } catch (error) {
-      if (error.includes("already_in_channel")) {
+      if (`${error}`.includes("already_in_channel")) {
         $.export("$summary", `The user ${this.user} is already in the channel`);
         return;
       }

components/slack_v2/actions/kick-user/kick-user.mjs

@@ -46,7 +46,7 @@ export default {
       $.export("$summary", `Successfully kicked user ${this.user} from channel with ID ${this.conversation}`);
       return response;
     } catch (error) {
-      if (error.includes("not_in_channel")) {
+      if (`${error}`.includes("not_in_channel")) {
         $.export("$summary", `The user ${this.user} is not in the channel`);
         return;
       }

components/slack_v2/actions/list-users/list-users.mjs

@@ -48,7 +48,7 @@ export default {
       users.push(...members);
       params.cursor = nextCursor;
       page++;
-    } while (params.cursor && page < this.numPages);
+    } while (params.cursor && page < (this.numPages ?? 1));
 
     $.export("$summary", `Successfully retrieved ${users.length} user${users.length === 1
       ? ""

components/slack_v2/actions/send-message-advanced/send-message-advanced.mjs

@@ -6,7 +6,7 @@ export default {
   ...buildBlocks,
   key: "slack_v2-send-message-advanced",
   name: "Send Message (Advanced)",
-  description: "Customize advanced setttings and send a message to a channel, group or user. See [postMessage](https://api.slack.com/methods/chat.postMessage) or [scheduleMessage](https://api.slack.com/methods/chat.scheduleMessage) docs here",
+  description: "Customize advanced settings and send a message to a channel, group or user. See [postMessage](https://api.slack.com/methods/chat.postMessage) or [scheduleMessage](https://api.slack.com/methods/chat.scheduleMessage) docs here",
   version: "0.1.0",
   annotations: {
     destructiveHint: false,

components/slack_v2/actions/update-group-members/update-group-members.mjs

@@ -2,7 +2,7 @@ import slack from "../../slack_v2.app.mjs";
 
 export default {
   key: "slack_v2-update-group-members",
-  name: "Update Groups Members",
+  name: "Update Group Members",
   description: "Update the list of users for a User Group. [See the documentation](https://api.slack.com/methods/usergroups.users.update)",
   version: "0.0.10",
   annotations: {

components/slack_v2/actions/upload-file/upload-file.mjs

@@ -80,7 +80,6 @@ export default {
       method: "POST",
       headers: {
         ...formData.getHeaders(),
-        Authorization: `Bearer ${this.slack.getToken()}`,
       },
     });
 

components/slack_v2/slack_v2.app.mjs

@@ -897,6 +897,8 @@ export default {
     },
     assistantSearch(args = {}) {
       args.count ||= constants.LIMIT;
+      // Uses apiCall directly since assistant.search.context is not exposed as
+      // a method on WebClient
       return this.sdk().apiCall("assistant.search.context", {
         ...args,
       });