Replaced Error with SlackbotError

src/server/models/vo/slack-bot-error.js

@@ -3,7 +3,9 @@
  */
 class SlackbotError extends Error {
 
-  constructor(method, to, popupMessage, mainMessage) {
+  constructor({
+    method, to, popupMessage, mainMessage,
+  } = {}) {
     super();
     this.method = method;
     this.to = to;

src/server/service/slack-command-handler/create-page-service.js

@@ -1,8 +1,8 @@
-const { markdownSectionBlock } = require('@growi/slack');
 const logger = require('@alias/logger')('growi:service:CreatePageService');
 const { reshapeContentsBody } = require('@growi/slack');
 const mongoose = require('mongoose');
 const pathUtils = require('growi-commons').pathUtils;
+const SlackbotError = require('../../models/vo/slack-bot-error');
 
 class CreatePageService {
 
@@ -31,13 +31,13 @@ class CreatePageService {
       });
     }
     catch (err) {
-      client.chat.postMessage({
-        channel: payload.user.id,
-        blocks: [
-          markdownSectionBlock(`Cannot create new page to existed path\n *Contents* :memo:\n ${reshapedContentsBody}`)],
+      logger.error('Failed to create page in GROWI.', err);
+      throw new SlackbotError({
+        method: 'postMessage',
+        to: 'dm',
+        popupMessage: 'Cannot create new page to existed path.',
+        mainMessage: `Cannot create new page to existed path\n *Contents* :memo:\n ${reshapedContentsBody}`,
       });
-      logger.error('Failed to create page in GROWI.');
-      throw err;
     }
   }
 

src/server/service/slack-command-handler/create.js

@@ -1,5 +1,4 @@
 const { markdownSectionBlock, inputSectionBlock } = require('@growi/slack');
-const logger = require('@alias/logger')('growi:service:SlackCommandHandler:create');
 
 module.exports = (crowi) => {
   const CreatePageService = require('./create-page-service');
@@ -8,38 +7,32 @@ module.exports = (crowi) => {
   const handler = new BaseSlackCommandHandler();
 
   handler.handleCommand = async(client, body) => {
-    try {
-      await client.views.open({
-        trigger_id: body.trigger_id,
+    await client.views.open({
+      trigger_id: body.trigger_id,
 
-        view: {
-          type: 'modal',
-          callback_id: 'create:createPage',
-          title: {
-            type: 'plain_text',
-            text: 'Create Page',
-          },
-          submit: {
-            type: 'plain_text',
-            text: 'Submit',
-          },
-          close: {
-            type: 'plain_text',
-            text: 'Cancel',
-          },
-          blocks: [
-            markdownSectionBlock('Create new page.'),
-            inputSectionBlock('path', 'Path', 'path_input', false, '/path'),
-            inputSectionBlock('contents', 'Contents', 'contents_input', true, 'Input with Markdown...'),
-          ],
-          private_metadata: JSON.stringify({ channelId: body.channel_id }),
+      view: {
+        type: 'modal',
+        callback_id: 'create:createPage',
+        title: {
+          type: 'plain_text',
+          text: 'Create Page',
         },
-      });
-    }
-    catch (err) {
-      logger.error(err);
-      throw new Error('Failed to create page.');
-    }
+        submit: {
+          type: 'plain_text',
+          text: 'Submit',
+        },
+        close: {
+          type: 'plain_text',
+          text: 'Cancel',
+        },
+        blocks: [
+          markdownSectionBlock('Create new page.'),
+          inputSectionBlock('path', 'Path', 'path_input', false, '/path'),
+          inputSectionBlock('contents', 'Contents', 'contents_input', true, 'Input with Markdown...'),
+        ],
+        private_metadata: JSON.stringify({ channelId: body.channel_id }),
+      },
+    });
   };
 
   handler.handleBlockActions = async function(client, payload, handlerMethodName) {

src/server/service/slack-command-handler/search.js

@@ -3,6 +3,7 @@ const logger = require('@alias/logger')('growi:service:SlackCommandHandler:searc
 const { markdownSectionBlock, divider } = require('@growi/slack');
 const { formatDistanceStrict } = require('date-fns');
 const axios = require('axios');
+const SlackbotError = require('../../models/vo/slack-bot-error');
 
 const PAGINGLIMIT = 10;
 
@@ -17,15 +18,12 @@ module.exports = (crowi) => {
     }
     catch (err) {
       logger.error('Failed to get search results.', err);
-      await client.chat.postEphemeral({
-        channel: body.channel_id,
-        user: body.user_id,
-        text: 'Failed To Search',
-        blocks: [
-          markdownSectionBlock('*Failed to search.*\n Hint\n `/growi search [keyword]`'),
-        ],
+      throw new SlackbotError({
+        method: 'postEphemeral',
+        to: 'channel',
+        popupMessage: 'Failed To Search',
+        mainMessage: '*Failed to search.*\n Hint\n `/growi search [keyword]`',
       });
-      throw new Error('Failed to search.');
     }
 
     const appUrl = crowi.appService.getSiteUrl();
@@ -140,26 +138,12 @@ module.exports = (crowi) => {
     }
     blocks.push(actionBlocks);
 
-    try {
-      await client.chat.postEphemeral({
-        channel: body.channel_id,
-        user: body.user_id,
-        text: 'Successed To Search',
-        blocks,
-      });
-    }
-    catch (err) {
-      logger.error('Failed to post ephemeral message.', err);
-      await client.chat.postEphemeral({
-        channel: body.channel_id,
-        user: body.user_id,
-        text: 'Failed to post ephemeral message.',
-        blocks: [
-          markdownSectionBlock(err.toString()),
-        ],
-      });
-      throw new Error(err.message);
-    }
+    await client.chat.postEphemeral({
+      channel: body.channel_id,
+      user: body.user_id,
+      text: 'Successed To Search',
+      blocks,
+    });
   };
 
   handler.handleBlockActions = async function(client, payload, handlerMethodName) {
@@ -210,15 +194,12 @@ module.exports = (crowi) => {
     }
     catch (err) {
       logger.error('Failed to get search results.', err);
-      await client.chat.postEphemeral({
-        channel: body.channel_id,
-        user: body.user_id,
-        text: 'Failed To Search',
-        blocks: [
-          markdownSectionBlock('*Failed to search.*\n Hint\n `/growi search [keyword]`'),
-        ],
+      throw new SlackbotError({
+        method: 'postEphemeral',
+        to: 'channel',
+        popupMessage: 'Failed To Search',
+        mainMessage: '*Failed to search.*\n Hint\n `/growi search [keyword]`',
       });
-      throw new Error('/growi command:search: Failed to search');
     }
 
     const appUrl = crowi.appService.getSiteUrl();
@@ -333,26 +314,12 @@ module.exports = (crowi) => {
     }
     blocks.push(actionBlocks);
 
-    try {
-      await client.chat.postEphemeral({
-        channel: body.channel_id,
-        user: body.user_id,
-        text: 'Successed To Search',
-        blocks,
-      });
-    }
-    catch (err) {
-      logger.error('Failed to post ephemeral message.', err);
-      await client.chat.postEphemeral({
-        channel: body.channel_id,
-        user: body.user_id,
-        text: 'Failed to post ephemeral message.',
-        blocks: [
-          markdownSectionBlock(err.toString()),
-        ],
-      });
-      throw new Error(err.message);
-    }
+    await client.chat.postEphemeral({
+      channel: body.channel_id,
+      user: body.user_id,
+      text: 'Successed To Search',
+      blocks,
+    });
   };
 
   handler.dismissSearchResults = async function(client, payload) {

src/server/service/slack-command-handler/togetter.js

@@ -4,6 +4,7 @@ const {
 const { parse, format } = require('date-fns');
 const axios = require('axios');
 const logger = require('@alias/logger')('growi:service:SlackBotService:togetter');
+const SlackbotError = require('../../models/vo/slack-bot-error');
 
 module.exports = (crowi) => {
   const CreatePageService = require('./create-page-service');
@@ -54,16 +55,8 @@ module.exports = (crowi) => {
       await this.togetterCreatePageAndSendPreview(client, payload, path, channel, contentsBody);
     }
     catch (err) {
-      logger.error(err);
-      // upcoming GW-6853 will change: just throw Error() here and handle in slackbot.js
-      await client.chat.postMessage({
-        channel: payload.user.id,
-        text: err.message,
-        blocks: [
-          markdownSectionBlock(err.message),
-        ],
-      });
-      return;
+      logger.err('Error occured by togetter.');
+      throw err;
     }
   };
 
@@ -75,7 +68,12 @@ module.exports = (crowi) => {
     oldest = oldest.trim();
     latest = latest.trim();
     if (!path) {
-      throw new Error('Page path is required.');
+      throw new SlackbotError({
+        method: 'postMessage',
+        to: 'dm',
+        popupMessage: 'Page path is required.',
+        mainMessage: 'Page path is required.',
+      });
     }
     /**
      * RegExp for datetime yyyy/MM/dd-HH:mm
@@ -84,17 +82,32 @@ module.exports = (crowi) => {
     const regexpDatetime = new RegExp(/^[12]\d\d\d\/(0[1-9]|1[012])\/(0[1-9]|[12][0-9]|3[01])-([01][0-9]|2[0123]):[0-5][0-9]$/);
 
     if (!regexpDatetime.test(oldest)) {
-      throw new Error('Datetime format for oldest must be yyyy/MM/dd-HH:mm');
+      throw new SlackbotError({
+        method: 'postMessage',
+        to: 'dm',
+        popupMessage: 'Datetime format for oldest must be yyyy/MM/dd-HH:mm',
+        mainMessage: 'Datetime format for oldest must be yyyy/MM/dd-HH:mm',
+      });
     }
     if (!regexpDatetime.test(latest)) {
-      throw new Error('Datetime format for latest must be yyyy/MM/dd-HH:mm');
+      throw new SlackbotError({
+        method: 'postMessage',
+        to: 'dm',
+        popupMessage: 'Datetime format for latest must be yyyy/MM/dd-HH:mm',
+        mainMessage: 'Datetime format for latest must be yyyy/MM/dd-HH:mm',
+      });
     }
     oldest = parse(oldest, 'yyyy/MM/dd-HH:mm', new Date()).getTime() / 1000 + grwTzoffset;
     // + 60s in order to include messages between hh:mm.00s and hh:mm.59s
     latest = parse(latest, 'yyyy/MM/dd-HH:mm', new Date()).getTime() / 1000 + grwTzoffset + 60;
 
     if (oldest > latest) {
-      throw new Error('Oldest datetime must be older than the latest date time.');
+      throw new SlackbotError({
+        method: 'postMessage',
+        to: 'dm',
+        popupMessage: 'Oldest datetime must be older than the latest date time.',
+        mainMessage: 'Oldest datetime must be older than the latest date time.',
+      });
     }
 
     return { path, oldest, latest };
@@ -111,7 +124,12 @@ module.exports = (crowi) => {
 
     // return if no message found
     if (!result.messages.length) {
-      throw new Error('No message found from togetter command. Try again.');
+      throw new SlackbotError({
+        method: 'postMessage',
+        to: 'dm',
+        popupMessage: 'No message found from togetter command. Try different datetime.',
+        mainMessage: 'No message found from togetter command. Try different datetime.',
+      });
     }
     return result;
   };
@@ -164,7 +182,13 @@ module.exports = (crowi) => {
       });
     }
     catch (err) {
-      throw new Error('Error occurred while creating a page.');
+      logger.error('Error occurred while creating a page.', err);
+      throw new SlackbotError({
+        method: 'postMessage',
+        to: 'dm',
+        popupMessage: 'Error occurred while creating a page.',
+        mainMessage: 'Error occurred while creating a page.',
+      });
     }
   };
 

src/server/service/slackbot.js

@@ -92,7 +92,6 @@ class SlackBotService extends S2sMessageHandlable {
       await handler.handleBlockActions(client, payload, handlerMethodName);
     }
     catch (err) {
-      // response
       throw err;
     }
     return;
@@ -108,7 +107,6 @@ class SlackBotService extends S2sMessageHandlable {
       await handler.handleBlockActions(client, payload, handlerMethodName);
     }
     catch (err) {
-      // response
       throw err;
     }
     return;