feat: bot name is now @all-contributors

Author: jakebolam

.env.example

@@ -1,4 +1,4 @@
-# The name that the app will respond to (change this for development). e.g. @AllContributorsBot
+# The name that the app will respond to (change this for development). e.g. @all-contributors
 BOT_NAME=AllContributorsDev
 # The ID of your GitHub App
 APP_ID=

README.md

@@ -18,7 +18,7 @@ A bot for automatically adding all-contributors. 🤖
 ## Usage
 
 ### Adding contributions
-1. Comment on Issue/PR etc with text: `@AllContributorBot please add jakebolam for infrastructure, testing and code` (Can also use the short terms, full key coming soon)
+1. Comment on Issue/PR etc with text: `@all-contributors please add jakebolam for infrastructure, testing and code` (Can also use the short terms, full key coming soon)
 2. Bot will look for `.all-contributorsrc` if not found, comments on pr to run setup
 3. If user exists, add new contribution, if not add user and add contribution
 

privacy-policy.md

@@ -16,7 +16,7 @@ When installing the the app you grant it access to the following three scopes
 
 2. **Read & write access to [issues](https://developer.github.com/v3/apps/permissions/#permission-on-issues)**
 
-   The app responds to comments on issues/pull-reuqests when a user types `@AllContributorsBot` the write perimission is required for this behaviour.
+   The app responds to comments on issues/pull-reuqests when a user types `@all-contributors` the write perimission is required for this behaviour.
 
 3. **Read & write access to [contents](https://developer.github.com/v3/apps/permissions/#permission-on-contents)**
 

serverless.yml

@@ -11,7 +11,7 @@ custom:
     prod: '23186'
   botName:
     dev: 'AllContributorsDevBot'
-    prod: 'AllContributorsBot'
+    prod: 'all-contributors'
   logLevel:
     dev: debug
     prod: info

src/utils/isMessageForBot.js

@@ -1,7 +1,10 @@
 const { GIHUB_BOT_NAME } = require('./settings')
 
 function isMessageForBot(message) {
-    const isMessageForBot = message.includes(`@${GIHUB_BOT_NAME}`)
+    const isMessageForBot =
+        message.includes(`@${GIHUB_BOT_NAME}`) ||
+        message.includes(`@allcontributors[bot]`) ||
+        message.includes(`@AllContributorsBot`) // TODO: phase this one out
     return isMessageForBot
 }
 

test/utils/isMessageForBot.test.js

@@ -12,9 +12,17 @@ describe('isMessageForBot', () => {
 
         expect(
             isMessageForBot(
-                `@AllContributorsBotTest please add jakebolam for doc, infra and code`,
+                `@allcontributors[bot] please add jakebolam for doc, infra and code`,
             ),
         ).toBe(true)
+
+        // TODO: Legacy phase this one out
+        expect(
+            isMessageForBot(
+                `@AllContributorsBot please add jakebolam for doc, infra and code`,
+            ),
+        ).toBe(true)
+
     })
 
     test('Not for us', () => {