fix: allow slack usernames with spaces (#12)

cypress/e2e/utils/parse-channel.cy.js

@@ -0,0 +1,32 @@
+/// <reference types="cypress" />
+const { getChannelAndPeople } = require('../../../src/utils')
+
+it('parses channel name with usernames', () => {
+  const { channel, people } = getChannelAndPeople(
+    '#channel-name @user1 @user2 @user3',
+  )
+  expect(channel, 'channel').to.equal('#channel-name')
+  expect(people, 'people').to.deep.equal(['@user1', '@user2', '@user3'])
+})
+
+it('parses channel name with usernames with spaces', () => {
+  const { channel, people } = getChannelAndPeople(
+    '#channel-name-1 @john doe @mary ann',
+  )
+  expect(channel, 'channel').to.equal('#channel-name-1')
+  expect(people, 'people').to.deep.equal(['@john doe', '@mary ann'])
+})
+
+it('ignores multiple spaces', () => {
+  const { channel, people } = getChannelAndPeople(
+    '  #channel-name-1     @john doe    @mary ann  ',
+  )
+  expect(channel, 'channel').to.equal('#channel-name-1')
+  expect(people, 'people').to.deep.equal(['@john doe', '@mary ann'])
+})
+
+it('does not care about casing', () => {
+  const { channel, people } = getChannelAndPeople('  #channel-NAME-1     @Mary')
+  expect(channel, 'channel').to.equal('#channel-NAME-1')
+  expect(people, 'people').to.deep.equal(['@Mary'])
+})

cypress/expected.json

@@ -5,5 +5,9 @@
   "fails in the subfolder sub": "fail",
   "finds the right channel by the filename end": "pass",
   "finds the right channel by minimatch": "pass",
-  "has single target shortcut": "pass"
+  "has single target shortcut": "pass",
+  "parses channel name with usernames": "pass",
+  "parses channel name with usernames with spaces": "pass",
+  "ignores multiple spaces": "pass",
+  "does not care about casing": "pass"
 }

src/utils.js

@@ -33,7 +33,10 @@ function getChannelAndPeople(s) {
   if (typeof s !== 'string') {
     throw new Error(`expected a string, got "${s}"`)
   }
-  const parts = s.split(' ')
+  const parts = s
+    .split(/\s(?=@)/g)
+    .map((s) => s.trim())
+    .filter(Boolean)
   const channel = parts.find((s) => s.startsWith('#'))
   const people = parts.filter((s) => s.startsWith('@'))
   return { channel, people }