fix: fix bot-me search limitations (#12925)

OfficeDev · Dec 17, 2024 · 4574dd1 · 4574dd1
1 parent e3e79e0
commit 4574dd1
Show file tree

Hide file tree

Showing 6 changed files with 72 additions and 0 deletions.
diff --git a/templates/js/m365-message-extension/src/searchApp.js b/templates/js/m365-message-extension/src/searchApp.js
@@ -13,6 +13,18 @@ class SearchApp extends TeamsActivityHandler {
   // Search.
   async handleTeamsMessagingExtensionQuery(context, query) {
     const searchQuery = query.parameters[0].value;
+
+    // Due to npmjs search limitations, do not search if input length < 2
+    if (searchQuery.length < 2) {
+      return {
+        composeExtension: {
+          type: "result",
+          attachmentLayout: "list",
+          attachments: [],
+        },
+      };
+    }
+
     const response = await axios.get(
       `http://registry.npmjs.com/-/v1/search?${querystring.stringify({
         text: searchQuery,

diff --git a/templates/js/message-extension-copilot/src/searchApp.js b/templates/js/message-extension-copilot/src/searchApp.js
@@ -13,6 +13,18 @@ class SearchApp extends TeamsActivityHandler {
   // Search.
   async handleTeamsMessagingExtensionQuery(context, query) {
     const searchQuery = query.parameters[0].value;
+
+    // Due to npmjs search limitations, do not search if input length < 2
+    if (searchQuery.length < 2) {
+      return {
+        composeExtension: {
+          type: "result",
+          attachmentLayout: "list",
+          attachments: [],
+        },
+      };
+    }
+
     const response = await axios.get(
       `http://registry.npmjs.com/-/v1/search?${querystring.stringify({
         text: searchQuery,

diff --git a/templates/js/message-extension/src/teamsBot.js b/templates/js/message-extension/src/teamsBot.js
@@ -31,6 +31,18 @@ class TeamsBot extends TeamsActivityHandler {
   // Search.
   async handleTeamsMessagingExtensionQuery(context, query) {
     const searchQuery = query.parameters[0].value;
+
+    // Due to npmjs search limitations, do not search if input length < 2
+    if (searchQuery.length < 2) {
+      return {
+        composeExtension: {
+          type: "result",
+          attachmentLayout: "list",
+          attachments: [],
+        },
+      };
+    }
+
     const response = await axios.get(
       `http://registry.npmjs.com/-/v1/search?${querystring.stringify({
         text: searchQuery,

diff --git a/templates/ts/m365-message-extension/src/searchApp.ts b/templates/ts/m365-message-extension/src/searchApp.ts
@@ -21,6 +21,18 @@ export class SearchApp extends TeamsActivityHandler {
     query: MessagingExtensionQuery
   ): Promise<MessagingExtensionResponse> {
     const searchQuery = query.parameters[0].value;
+
+    // Due to npmjs search limitations, do not search if input length < 2
+    if (searchQuery.length < 2) {
+      return {
+        composeExtension: {
+          type: "result",
+          attachmentLayout: "list",
+          attachments: [],
+        },
+      };
+    }
+
     const response = await axios.get(
       `http://registry.npmjs.com/-/v1/search?${querystring.stringify({
         text: searchQuery,

diff --git a/templates/ts/message-extension-copilot/src/searchApp.ts b/templates/ts/message-extension-copilot/src/searchApp.ts
@@ -21,6 +21,18 @@ export class SearchApp extends TeamsActivityHandler {
     query: MessagingExtensionQuery
   ): Promise<MessagingExtensionResponse> {
     const searchQuery = query.parameters[0].value;
+
+    // Due to npmjs search limitations, do not search if input length < 2
+    if (searchQuery.length < 2) {
+      return {
+        composeExtension: {
+          type: "result",
+          attachmentLayout: "list",
+          attachments: [],
+        },
+      };
+    }
+
     const response = await axios.get(
       `http://registry.npmjs.com/-/v1/search?${querystring.stringify({
         text: searchQuery,

diff --git a/templates/ts/message-extension/src/teamsBot.ts b/templates/ts/message-extension/src/teamsBot.ts
@@ -46,6 +46,18 @@ export class TeamsBot extends TeamsActivityHandler {
     query: MessagingExtensionQuery
   ): Promise<MessagingExtensionResponse> {
     const searchQuery = query.parameters[0].value;
+
+    // Due to npmjs search limitations, do not search if input length < 2
+    if (searchQuery.length < 2) {
+      return {
+        composeExtension: {
+          type: "result",
+          attachmentLayout: "list",
+          attachments: [],
+        },
+      };
+    }
+
     const response = await axios.get(
       `http://registry.npmjs.com/-/v1/search?${querystring.stringify({
         text: searchQuery,