@hubot-friends/hubot-slack 3.7.0 → 3.7.2

package/.github/issue_template.md

@@ -11,8 +11,8 @@ Describe your issue here.
 - [ ] discussion
 
 ### Requirements (place an `x` in each of the `[ ]`)
-* [ ] I've read and understood the [Contributing guidelines](./contributing.md) and have done my best effort to follow them.
-* [ ] I've read and agree to the [Code of Conduct](./CODE_OF_CONDUCT.md).
+* [ ] I've read and understood the [Contributing guidelines](./.github/contributing.md) and have done my best effort to follow them.
+* [ ] I've read and agree to the [Code of Conduct](./.github/CODE_OF_CONDUCT.md).
 * [ ] I've searched for any related issues and avoided creating a duplicate issue.
 
 ---

package/.github/pull_request_template.md

@@ -4,4 +4,4 @@ Describe the goal of this PR. Mention any related Issue numbers.
 
 ### Requirements (place an `x` in each `[ ]`)
 
-* [ ] I've read and understood the [Contributing Guidelines](./contributing.md) and have done my best effort to follow them.
+* [ ] I've read and understood the [Contributing Guidelines](./.github/contributing.md) and have done my best effort to follow them.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hubot-friends/hubot-slack",
-  "version": "3.7.0",
+  "version": "3.7.2",
   "description": "A new Slack adapter for Hubot",
   "homepage": "https://github.com/hubot-friends/hubot-slack#readme",
   "main": "./index.mjs",
@@ -8,7 +8,7 @@
     "test": "node --test",
     "test:watch": "node --test --watch",
     "test:integration": "node --test --env-file=.env",
-    "start:local": "node --watch --env-file=.env node_modules/hubot/bin/hubot --adapter ../../../index.mjs --name gbot"
+    "start:local": "node --watch --env-file=.env node_modules/hubot/bin/hubot --adapter ../../../index.mjs --name jbot"
   },
   "keywords": [
     "hubot",
@@ -32,7 +32,7 @@
     "node": ">= 18"
   },
   "peerDependencies": {
-    "hubot": "^12.0.0"
+    "hubot": "^13.0.0"
   },
   "devDependencies": {
     "pino-pretty": "^10.0.1"

package/src/Bot.mjs

@@ -78,12 +78,19 @@ class SlackClient {
       return;
     }
     this.robot.logger.debug(`SlackClient#send() room: ${room}, message: ${message}`);
-    const messageOptions = {
+    let messageOptions = {
       channel: room,
       text: typeof message === "string" ? message : message.text,
       thread_ts, // Include thread_ts if it's defined
-      ...message, // Spread other properties from the message if it's an object
     };
+    if (typeof message === "object") {
+      Object.assign(messageOptions, message);
+      // Ensure channel and text are not overwritten
+      messageOptions.channel = room;
+      if (message.text == null) {
+        messageOptions.text = '';
+      }
+    }
   
     try {
       const result = await this.web.chat.postMessage(messageOptions);
@@ -179,10 +186,9 @@ class SlackBot extends Adapter {
     super(robot);
     this.options = options;
     this.robot.logger.info(`hubot-slack adapter v${pkg.version}`);
-    this.socket = new SocketModeClient({ appToken: options.appToken, ...options.socketModeOptions });
-    this.web = new WebClient(options.botToken, { agent: robot.config?.agent ?? undefined, maxRequestConcurrency: 1, logLevel: 'error'});
-    this.client = new SlackClient(this.options, this.robot, this.socket, this.web);
-    this.seenMessages = new Set();
+    this.socket = options.socket ?? new SocketModeClient({ appToken: options.appToken, ...options.socketModeOptions });
+    const web = options.web ?? new WebClient(options.botToken, { agent: robot.config?.agent ?? undefined, maxRequestConcurrency: 1, logLevel: 'error'});
+    this.client = new SlackClient(this.options, this.robot, this.socket, web);
   }
 
   async run() {
@@ -489,14 +495,6 @@ class SlackBot extends Adapter {
     this.robot.logger.debug(`Text = ${message.body.event.text}`);
     this.robot.logger.debug(`Event subtype = ${message.body.event?.subtype}`);
 
-    // Ignore messages we've already seen
-    if (this.seenMessages.has(message.body.event.client_msg_id)) {
-      this.robot.logger.debug(`Ignoring message ${message.body.event.client_msg_id}`);
-      return;
-    }
-
-    this.seenMessages.add(message.body.event.client_msg_id);
-
     try {
       switch (message.event.type) {
         case "member_joined_channel":
@@ -542,9 +540,6 @@ class SlackBot extends Adapter {
       }
     } catch (e) {
       this.robot.logger.error(e);
-    } finally {
-      this.robot.logger.debug(`Removing message ${message.body.event.client_msg_id}`);
-      this.seenMessages.delete(message.body.event.client_msg_id);
     }
   }
 

package/test/Bot.mjs

@@ -4,13 +4,14 @@ import { SlackBot } from '../src/Bot.mjs'
 import hubotSlackMock from '../index.mjs'
 import { loadBot } from 'hubot'
 import { SlackTextMessage, ReactionMessage, FileSharedMessage } from '../src/Message.mjs'
+import { EventEmitter } from 'node:events'
 
 describe('Adapter', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
-    
+
   it('Should initialize with a robot', () => {
     assert.deepEqual(slackbot.robot, stubs.robot)
   })
@@ -20,7 +21,7 @@ describe('Adapter', () => {
     process.env.HUBOT_SLACK_BOT_TOKEN = 'xoxb-faketoken'
 
     const loadedRobot = loadBot(hubotSlackMock, false, 'Hubot')
-    await loadedRobot.loadAdapter()    
+    await loadedRobot.loadAdapter()
 
     assert.ok(loadedRobot.hearReaction instanceof Function)
     assert.deepEqual(loadedRobot.hearReaction.length, 3)
@@ -34,7 +35,7 @@ describe('Adapter', () => {
 describe('Connect', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
 
   it('Should connect successfully', (t, done) => {
@@ -49,11 +50,11 @@ describe('Connect', () => {
 describe('Authenticate', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
 
   it('Should authenticate successfully', async () => {
-    const {logger} = slackbot.robot
+    const { logger } = slackbot.robot
     const start = {
       self: {
         id: stubs.self.id,
@@ -79,7 +80,7 @@ describe('Authenticate', () => {
 describe('Socket', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
     slackbot.socket.disconnect = mock.fn(() => {
       slackbot.socket.shuttingDown = true
       slackbot.socket.emit('close')
@@ -96,7 +97,7 @@ describe('Socket', () => {
 
   it('Should log socket close event when we call socket.disconnect()', async () => {
     const { logger } = slackbot.robot
-    
+
     slackbot.socket.autoReconnectEnabled = true
 
     await slackbot.run()
@@ -138,11 +139,11 @@ describe('Socket', () => {
 describe('Logger', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
 
   it('It should log invalid botToken error', (t, done) => {
-    const {logger} = slackbot.robot
+    const { logger } = slackbot.robot
     logger.error = message => {
       assert.deepEqual(message, 'Invalid botToken provided, please follow the upgrade instructions')
       done()
@@ -153,7 +154,7 @@ describe('Logger', () => {
   })
 
   it('It should log invalid appToken error', (t, done) => {
-    const {logger} = slackbot.robot
+    const { logger } = slackbot.robot
     logger.error = message => {
       assert.deepEqual(message, 'Invalid appToken provided, please follow the upgrade instructions')
       done()
@@ -167,12 +168,12 @@ describe('Logger', () => {
 describe('Disable Sync', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
 
   it('Should sync users by default', () => {
     slackbot.run()
-    assert.deepEqual(Object.keys(slackbot.robot.brain.data.users), ['1','2','3','4'])
+    assert.deepEqual(Object.keys(slackbot.robot.brain.data.users), ['1', '2', '3', '4'])
   })
 
   it('Should not sync users when disabled', () => {
@@ -184,18 +185,74 @@ describe('Disable Sync', () => {
 
 describe('Send Messages', () => {
   let stubs, slackbot
+  let bot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    bot = new SlackBot({
+      alias: '!', logger: { info() { }, debug() { }, error(e) { throw e } }
+    }, {
+      appToken: '',
+      socket: new EventEmitter(),
+      web: {
+          users: {
+            list: () => Promise.resolve(stubs.responseUsersList)
+          },
+          conversations: {
+            open: ({ users }) => {
+              const user = users.split(',')[0]
+              return Promise.resolve({ ok: true, channel: { id: `D${user.substring(1)}` } })
+            }
+          },
+          chat: {
+            postMessage: (options) => {
+              stubs._topic = options.topic
+              return Promise.resolve({ ok: true })
+            }
+          }
+        }
+      })
+
+    bot.self = {
+      user_id: '1234'
+    };
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
 
-  it('Should send a message', () => {
-    slackbot.client.send = (envelope, message) => {
-      stubs._sendCount++
-      stubs._msg = message
+  it('Send a message and message options has thread_ts and text, but not indexed properties due to the message being spread out to it', async () => {
+    const envelope = {
+      room: 'D1234',
+      message: {
+        room: 'D1234',
+        thread_ts: '1234567890.123456'
+      }
     }
-    slackbot.send({room: stubs.channel.id}, 'message')
-    assert.deepEqual(stubs._sendCount, 1)
-    assert.deepEqual(stubs._msg, 'message')
+
+    bot.client.web = {
+      chat: {
+        async postMessage(options) {
+          assert.deepEqual(options[0], undefined)
+          assert.deepEqual(options.channel, 'D1234')
+          assert.deepEqual(options.text, 'test message')
+          assert.deepEqual(options.thread_ts, '1234567890.123456')
+          return Promise.resolve({ ok: true })
+        }
+      }
+    }
+    await bot.send(envelope, 'test message')
+  })
+
+  it('Should send a message that is an object', async () => {
+    const envelope = { room: 'D1234' }
+    bot.client.web = {
+      chat: {
+        async postMessage(options) {
+          assert.deepEqual(options.channel, 'D1234')
+          assert.deepEqual(options.text, 'test message')
+          assert.deepEqual(options.thread_ts, '1234567890.123456')
+          return Promise.resolve({ ok: true })
+        }
+      }
+    }
+    await bot.send(envelope, { text: 'test message', thread_ts: '1234567890.123456' })
   })
 
   it('Should send multiple messages', () => {
@@ -203,7 +260,7 @@ describe('Send Messages', () => {
       stubs._sendCount++
     }
 
-    slackbot.send({room: stubs.channel.id}, 'one', 'two', 'three')
+    slackbot.send({ room: stubs.channel.id }, 'one', 'two', 'three')
     assert.deepEqual(stubs._sendCount, 3)
   })
 
@@ -211,12 +268,12 @@ describe('Send Messages', () => {
     slackbot.client.send = (envelope, message) => {
       stubs._sendCount++
     }
-    slackbot.send({room: stubs.channel.id}, 'Hello', '', '', 'world!')
+    slackbot.send({ room: stubs.channel.id }, 'Hello', '', '', 'world!')
     assert.deepEqual(stubs._sendCount, 2)
   })
 
   it('Should not fail for inexistant user', () => {
-    assert.doesNotThrow(() => slackbot.send({room: 'U987'}, 'Hello'))
+    assert.doesNotThrow(() => slackbot.send({ room: 'U987' }, 'Hello'))
   })
 
   it('Should open a DM channel if needed', () => {
@@ -224,7 +281,7 @@ describe('Send Messages', () => {
     slackbot.client.send = (envelope, message) => {
       stubs._dmmsg = message
     }
-    slackbot.send({room: stubs.user.id}, msg)
+    slackbot.send({ room: stubs.user.id }, msg)
     assert.deepEqual(stubs._dmmsg, msg)
   })
 
@@ -233,17 +290,17 @@ describe('Send Messages', () => {
       stubs._dmmsg = message
       stubs._room = envelope.room
     }
-    slackbot.send({room: stubs.user.id}, 'message')
+    slackbot.send({ room: stubs.user.id }, 'message')
     assert.deepEqual(stubs._dmmsg, 'message')
     assert.deepEqual(stubs._room, stubs.user.id)
   })
 
-  it('Should send a message with a callback', function(t, done) {
+  it('Should send a message with a callback', function (t, done) {
     slackbot.client.send = (envelope, message) => {
       stubs._msg = message
       stubs._sendCount++
     }
-    slackbot.send({room: stubs.channel.id}, 'message with a callback', () => {
+    slackbot.send({ room: stubs.channel.id }, 'message with a callback', () => {
       assert.ok(true)
       done()
     })
@@ -293,7 +350,7 @@ describe('Send Messages', () => {
 describe('Reply to Messages', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
 
   it('Should mention the user in a reply sent in a channel', () => {
@@ -301,7 +358,7 @@ describe('Reply to Messages', () => {
       stubs._sendCount++
       stubs._msg = message
     }
-    slackbot.reply({user: stubs.user, room: stubs.channel.id}, 'message')
+    slackbot.reply({ user: stubs.user, room: stubs.channel.id }, 'message')
     assert.deepEqual(stubs._sendCount, 1)
     assert.deepEqual(stubs._msg, `<@${stubs.user.id}>: message`)
   })
@@ -311,7 +368,7 @@ describe('Reply to Messages', () => {
       stubs._sendCount++
       stubs._msg = message
     }
-    slackbot.reply({user: stubs.user, room: stubs.channel.id}, 'one', 'two', 'three')
+    slackbot.reply({ user: stubs.user, room: stubs.channel.id }, 'one', 'two', 'three')
     assert.deepEqual(stubs._sendCount, 3)
     assert.deepEqual(stubs._msg, `<@${stubs.user.id}>: three`)
   })
@@ -321,7 +378,7 @@ describe('Reply to Messages', () => {
       stubs._sendCount++
       stubs._msg = message
     }
-    slackbot.reply({user: stubs.user, room: stubs.channel.id}, '')
+    slackbot.reply({ user: stubs.user, room: stubs.channel.id }, '')
     assert.deepEqual(stubs._sendCount, 0)
   })
 
@@ -330,17 +387,17 @@ describe('Reply to Messages', () => {
       stubs._sendCount++
       stubs._dmmsg = message
     }
-    slackbot.reply({user: stubs.user, room: stubs.DM.id }, 'message')
+    slackbot.reply({ user: stubs.user, room: stubs.DM.id }, 'message')
     assert.deepEqual(stubs._sendCount, 1)
     assert.deepEqual(stubs._dmmsg, 'message')
   })
 
-  it('Should call the callback', function(t, done) {
+  it('Should call the callback', function (t, done) {
     slackbot.client.send = (envelope, message) => {
       stubs._sendCount++
       stubs._msg = message
     }
-    slackbot.reply({user: stubs.user, room: stubs.channel.id}, 'message', () => {
+    slackbot.reply({ user: stubs.user, room: stubs.channel.id }, 'message', () => {
       assert.ok(true)
       done()
     })
@@ -352,21 +409,21 @@ describe('Reply to Messages', () => {
 describe('Setting the channel topic', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
 
   it('Should set the topic in channels', async () => {
     let wasCalled = false
-    stubs.receiveMock.onTopic = function(topic) {
+    stubs.receiveMock.onTopic = function (topic) {
       assert.deepEqual(topic, 'channel')
       wasCalled = true
     }
-    await slackbot.setTopic({room: stubs.channel.id}, 'channel')
+    await slackbot.setTopic({ room: stubs.channel.id }, 'channel')
     assert.deepEqual(wasCalled, true)
   })
 
   it('Should NOT set the topic in DMs', async () => {
-    await slackbot.setTopic({room: 'D1232'}, 'DM')
+    await slackbot.setTopic({ room: 'D1232' }, 'DM')
     assert.equal(stubs._topic, undefined)
   })
 })
@@ -374,7 +431,7 @@ describe('Setting the channel topic', () => {
 describe('Receiving an error event', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
   it('Should propagate that error', () => {
     let hit = false
@@ -383,13 +440,13 @@ describe('Receiving an error event', () => {
       hit = true
     })
     assert.ok(!hit)
-    slackbot.error({msg: 'ohno', code: -2})
+    slackbot.error({ msg: 'ohno', code: -2 })
     assert.ok(hit)
   })
 
   it('Should handle rate limit errors', () => {
-    const {logger} = slackbot.robot
-    slackbot.error({msg: 'ratelimit', code: -1})
+    const { logger } = slackbot.robot
+    slackbot.error({ msg: 'ratelimit', code: -1 })
     assert.ok(logger.logs["error"].length > 0)
   })
 })
@@ -397,48 +454,48 @@ describe('Receiving an error event', () => {
 describe('Handling incoming messages', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
 
-  it('Should handle regular messages as hoped and dreamed', function(t, done) {
-    stubs.receiveMock.onReceived = function(msg) {
+  it('Should handle regular messages as hoped and dreamed', function (t, done) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.deepEqual(msg.text, 'foo')
       done()
     }
-    slackbot.eventHandler({body: { event: { text: 'foo', type: 'message', user: stubs.user.id }}, event: { text: 'foo', type: 'message', user: stubs.user.id, channel: stubs.channel.id }})
+    slackbot.eventHandler({ body: { event: { text: 'foo', type: 'message', user: stubs.user.id } }, event: { text: 'foo', type: 'message', user: stubs.user.id, channel: stubs.channel.id } })
   })
 
-  it('Should prepend our name to a name-lacking message addressed to us in a DM', function(t, done) {
+  it('Should prepend our name to a name-lacking message addressed to us in a DM', function (t, done) {
     const bot_name = slackbot.robot.name
-    stubs.receiveMock.onReceived = function(msg) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.deepEqual(msg.text, `@${bot_name} foo`)
       done()
     }
-    slackbot.eventHandler({body: { event: { text: 'foo', type: 'message', user: stubs.user.id, channel_type: 'im' }}, event: { text: 'foo', type: 'message', user: stubs.user.id, channel_type: 'im', channel:stubs.DM.id }})
+    slackbot.eventHandler({ body: { event: { text: 'foo', type: 'message', user: stubs.user.id, channel_type: 'im' } }, event: { text: 'foo', type: 'message', user: stubs.user.id, channel_type: 'im', channel: stubs.DM.id } })
   })
 
-  it('Should preprend our alias to a name-lacking message addressed to us in a DM', function(t, done) {
-   const bot = new SlackBot({alias: '!', logger: {info(){}, debug(){}}}, {appToken: ''})
-   bot.self = {
-    user_id: '1234'
-   }
-   const text = bot.replaceBotIdWithName({
+  it('Should preprend our alias to a name-lacking message addressed to us in a DM', function (t, done) {
+    const bot = new SlackBot({ alias: '!', logger: { info() { }, debug() { } } }, { appToken: '', socket: new EventEmitter() })
+    bot.self = {
+      user_id: '1234'
+    }
+    const text = bot.replaceBotIdWithName({
       text: '<@1234> foo',
-   })
+    })
     assert.deepEqual(text, '! foo')
     done()
   })
 
-  it('Should NOT prepend our name to a name-containing message addressed to us in a DM', function(t, done) {
+  it('Should NOT prepend our name to a name-containing message addressed to us in a DM', function (t, done) {
     const bot_name = slackbot.robot.name
-    stubs.receiveMock.onReceived = function(msg) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.deepEqual(msg.text, `@${bot_name} foo`)
       done()
     }
-    slackbot.eventHandler({body: { event: { text: `@${bot_name} foo`, type: 'message', user: stubs.user.id }}, event: { text: 'foo', type: 'message', user: stubs.user.id, channel:stubs.DM.id }})
+    slackbot.eventHandler({ body: { event: { text: `@${bot_name} foo`, type: 'message', user: stubs.user.id } }, event: { text: 'foo', type: 'message', user: stubs.user.id, channel: stubs.DM.id } })
   })
 
-  it('Should return a message object with raw text and message', function(t, done) {
+  it('Should return a message object with raw text and message', function (t, done) {
     //the shape of this data is an RTM message event passed through SlackClient#messageWrapper
     //see: https://api.slack.com/events/message
     const messageData = {
@@ -457,7 +514,7 @@ describe('Handling incoming messages', () => {
         channel: stubs.channel.id,
       }
     }
-    stubs.receiveMock.onReceived = function(msg) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.deepEqual((msg instanceof SlackTextMessage), true)
       assert.deepEqual(msg.text, "foo http://www.example.com bar")
       assert.deepEqual(msg.rawText, "foo <http://www.example.com> bar")
@@ -468,7 +525,7 @@ describe('Handling incoming messages', () => {
   })
 
   it('Should handle member_joined_channel events as envisioned', () => {
-    stubs.receiveMock.onReceived = function(msg) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.deepEqual(msg.constructor.name, "EnterMessage")
       assert.deepEqual(msg.ts, stubs.event_timestamp)
       assert.deepEqual(msg.user.id, stubs.user.id)
@@ -493,7 +550,7 @@ describe('Handling incoming messages', () => {
   })
 
   it('Should handle member_left_channel events as envisioned', () => {
-    stubs.receiveMock.onReceived = function(msg) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.deepEqual(msg.constructor.name, "LeaveMessage")
       assert.deepEqual(msg.ts, stubs.event_timestamp)
       assert.deepEqual(msg.user.id, stubs.user.id)
@@ -533,7 +590,7 @@ describe('Handling incoming messages', () => {
           },
           reaction: 'thumbsup',
           event_ts: '1360782804.083113'
-    
+
         }
       },
       event: {
@@ -552,7 +609,7 @@ describe('Handling incoming messages', () => {
       }
     }
 
-    stubs.receiveMock.onReceived = function(msg) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.deepEqual((msg instanceof ReactionMessage), true)
       assert.deepEqual(msg.user.id, stubs.user.id)
       assert.deepEqual(msg.user.room, stubs.channel.id)
@@ -580,7 +637,7 @@ describe('Handling incoming messages', () => {
           },
           reaction: 'thumbsup',
           event_ts: '1360782804.083113'
-    
+
         }
       },
       event: {
@@ -598,7 +655,7 @@ describe('Handling incoming messages', () => {
         event_ts: '1360782804.083113'
       }
     }
-    stubs.receiveMock.onReceived = function(msg) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.deepEqual((msg instanceof ReactionMessage), true)
       assert.deepEqual(msg.user.id, stubs.user.id)
       assert.deepEqual(msg.user.room, stubs.channel.id)
@@ -611,7 +668,7 @@ describe('Handling incoming messages', () => {
   })
 
   it('Should ignore messages it sent itself', (t, done) => {
-    stubs.receiveMock.onReceived = function(msg) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.fail('Should not have received a message')
     }
 
@@ -622,7 +679,7 @@ describe('Handling incoming messages', () => {
           text: 'Ignore me',
           user: stubs.self.id,
           channel: stubs.channel.id,
-          ts: stubs.event_timestamp    
+          ts: stubs.event_timestamp
         }
       },
       event: {
@@ -630,14 +687,14 @@ describe('Handling incoming messages', () => {
         text: 'Ignore me',
         user: stubs.self.id,
         channel: stubs.channel.id,
-        ts: stubs.event_timestamp    
+        ts: stubs.event_timestamp
       }
     })
     done()
   })
 
-  it('Should handle empty users as envisioned', function(t, done){
-    stubs.receiveMock.onReceived = function(msg) {
+  it('Should handle empty users as envisioned', function (t, done) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.fail('Should not have received a message')
     }
     slackbot.eventHandler({
@@ -647,7 +704,7 @@ describe('Handling incoming messages', () => {
           text: 'Foo',
           user: '',
           channel: stubs.channel.id,
-          ts: stubs.event_timestamp    
+          ts: stubs.event_timestamp
         }
       },
       event: {
@@ -655,7 +712,7 @@ describe('Handling incoming messages', () => {
         text: 'Foo',
         user: '',
         channel: stubs.channel.id,
-        ts: stubs.event_timestamp    
+        ts: stubs.event_timestamp
       }
     })
     done()
@@ -684,7 +741,7 @@ describe('Handling incoming messages', () => {
         event_ts: stubs.event_timestamp
       }
     }
-    stubs.receiveMock.onReceived = function(msg) {
+    stubs.receiveMock.onReceived = function (msg) {
       assert.deepEqual((msg instanceof FileSharedMessage), true)
       assert.deepEqual(msg.user.id, stubs.user.id)
       assert.deepEqual(msg.user.room, stubs.channel.id)
@@ -693,13 +750,13 @@ describe('Handling incoming messages', () => {
     slackbot.eventHandler(fileMessage)
   })
 })
-    
+
 describe('Robot.fileShared', () => {
   let stubs, slackbot, fileSharedMessage
   const handleFileShared = msg => `${msg.file_id} shared`
 
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
     const user = { id: stubs.user.id, room: stubs.channel.id }
     fileSharedMessage = new FileSharedMessage(user, "F2147483862", '1360782804.083113')
   })
@@ -708,15 +765,15 @@ describe('Robot.fileShared', () => {
     slackbot.robot.fileShared(handleFileShared)
     const listener = slackbot.robot.listeners.shift()
     assert.ok(listener.matcher(fileSharedMessage))
-    assert.deepEqual(listener.options, {id: null})
+    assert.deepEqual(listener.options, { id: null })
     assert.deepEqual(listener.callback(fileSharedMessage), 'F2147483862 shared')
   })
-    
+
   it('Should register a Listener with opts and callback', () => {
-    slackbot.robot.fileShared({id: 'foobar'}, handleFileShared)
+    slackbot.robot.fileShared({ id: 'foobar' }, handleFileShared)
     const listener = slackbot.robot.listeners.shift()
     assert.ok(listener.matcher(fileSharedMessage))
-    assert.deepEqual(listener.options, {id: 'foobar'})
+    assert.deepEqual(listener.options, { id: 'foobar' })
     assert.deepEqual(listener.callback(fileSharedMessage), 'F2147483862 shared')
   })
 
@@ -725,16 +782,16 @@ describe('Robot.fileShared', () => {
     slackbot.robot.fileShared(matcher, handleFileShared)
     const listener = slackbot.robot.listeners.shift()
     assert.ok(listener.matcher(fileSharedMessage))
-    assert.deepEqual(listener.options, {id: null})
+    assert.deepEqual(listener.options, { id: null })
     assert.deepEqual(listener.callback(fileSharedMessage), 'F2147483862 shared')
   })
 
   it('Should register a Listener with matcher, opts, and callback', () => {
     const matcher = msg => msg.file_id === 'F2147483862'
-    slackbot.robot.fileShared(matcher, {id: 'foobar'}, handleFileShared)
+    slackbot.robot.fileShared(matcher, { id: 'foobar' }, handleFileShared)
     const listener = slackbot.robot.listeners.shift()
     assert.ok(listener.matcher(fileSharedMessage))
-    assert.deepEqual(listener.options, {id: 'foobar'})
+    assert.deepEqual(listener.options, { id: 'foobar' })
     assert.deepEqual(listener.callback(fileSharedMessage), 'F2147483862 shared')
   })
 
@@ -750,7 +807,7 @@ describe('Robot.hearReaction', () => {
   let stubs, slackbot, reactionMessage
   const handleReaction = msg => `${msg.reaction} handled`
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
     const user = { id: stubs.user.id, room: stubs.channel.id }
     const item = {
       type: 'message', channel: stubs.channel.id, ts: '1360782804.083113'
@@ -764,15 +821,15 @@ describe('Robot.hearReaction', () => {
     slackbot.robot.hearReaction(handleReaction)
     const listener = slackbot.robot.listeners.shift()
     assert.ok(listener.matcher(reactionMessage))
-    assert.deepEqual(listener.options, {id: null})
+    assert.deepEqual(listener.options, { id: null })
     assert.deepEqual(listener.callback(reactionMessage), 'thumbsup handled')
   })
 
   it('Should register a Listener with opts and callback', () => {
-    slackbot.robot.hearReaction({id: 'foobar'}, handleReaction)
+    slackbot.robot.hearReaction({ id: 'foobar' }, handleReaction)
     const listener = slackbot.robot.listeners.shift()
     assert.ok(listener.matcher(reactionMessage))
-    assert.deepEqual(listener.options, {id: 'foobar'})
+    assert.deepEqual(listener.options, { id: 'foobar' })
     assert.deepEqual(listener.callback(reactionMessage), 'thumbsup handled')
   })
 
@@ -781,16 +838,16 @@ describe('Robot.hearReaction', () => {
     slackbot.robot.hearReaction(matcher, handleReaction)
     const listener = slackbot.robot.listeners.shift()
     assert.ok(listener.matcher(reactionMessage))
-    assert.deepEqual(listener.options, {id: null})
+    assert.deepEqual(listener.options, { id: null })
     assert.deepEqual(listener.callback(reactionMessage), 'thumbsup handled')
   })
 
   it('Should register a Listener with matcher, opts, and callback', () => {
     const matcher = msg => (msg.type === 'removed') || (msg.reaction === 'thumbsup')
-    slackbot.robot.hearReaction(matcher, {id: 'foobar'}, handleReaction)
+    slackbot.robot.hearReaction(matcher, { id: 'foobar' }, handleReaction)
     const listener = slackbot.robot.listeners.shift()
     assert.ok(listener.matcher(reactionMessage))
-    assert.deepEqual(listener.options, {id: 'foobar'})
+    assert.deepEqual(listener.options, { id: 'foobar' })
     assert.deepEqual(listener.callback(reactionMessage), 'thumbsup handled')
   })
 
@@ -805,7 +862,7 @@ describe('Robot.hearReaction', () => {
 describe('Users data', () => {
   let stubs, slackbot
   beforeEach(async () => {
-    ({stubs, slackbot} = (await import('./Stubs.mjs')).default())
+    ({ stubs, slackbot } = (await import('./Stubs.mjs')).default())
   })
   it('Should load users data from web api', () => {
     slackbot.usersLoaded(null, stubs.responseUsersList)
@@ -826,7 +883,7 @@ describe('Users data', () => {
 
   it('Should merge with user data which is stored by other program', () => {
     const originalUser =
-      {something: 'something'}
+      { something: 'something' }
 
     slackbot.robot.brain.userForId(stubs.user.id, originalUser)
     slackbot.usersLoaded(null, stubs.responseUsersList)

package/configuration/Configure.mjs

@@ -1,8 +0,0 @@
-// If you want to use this, you have to `npm install proxy-agent` and uncomment the import statement below
-// import { ProxyAgent  } from 'proxy-agent'
-export default async robot => {
-    robot.logger.info(`This is an example showing how to configure a proxy for the Slack Adapter's Web Client to use.`)
-    robot.config = {
-        // agent: new ProxyAgent(),
-    }
-}