@toptal/davinci-ci 1.13.4-alpha-fx-upgrade-stylelint-rules.1335 → 1.13.4-alpha-fix-testid-eslint-rule-bug.1335

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@toptal/davinci-ci",
-  "version": "1.13.4-alpha-fx-upgrade-stylelint-rules.1335+72109f9",
+  "version": "1.13.4-alpha-fix-testid-eslint-rule-bug.1335+94c355e",
   "description": "Continuos integrations tools for frontend projects",
   "publishConfig": {
     "access": "public"
@@ -32,9 +32,9 @@
   "dependencies": {
     "@commitlint/cli": "^12.1.4",
     "@commitlint/config-conventional": "^12.1.4",
-    "@toptal/davinci-cli-shared": "1.4.2-alpha-fx-upgrade-stylelint-rules.1335+72109f9",
+    "@toptal/davinci-cli-shared": "1.4.2-alpha-fix-testid-eslint-rule-bug.1335+94c355e",
     "danger": "^10.7.1",
     "markdown-table": "^2.0.0"
   },
-  "gitHead": "72109f93ee9cf82099b24934b322929fe3e9b883"
+  "gitHead": "94c355e157c825baf75e8de94c43766d1c4cd8e1"
 }

package/src/configs/danger/toptal/config.js

@@ -1,6 +1,12 @@
 const WHITELISTED_USERS = ['dependabot-preview[bot]', 'dependabot[bot]', 'toptal-devbot']
 const VALID_PR_CODE_REGEX = /\[[A-Z]{1,4}-.*]\s/
-const VALID_COMMIT_MESSAGE_REGEX = /^[A-Z]\w[^.\n]*[^.\s](\n\n(.|\n)*)*$/
+// Valid commit titles:
+// "Regular commit message"
+// "Commit with sentence. But does not end with a full-stop"
+// "[FOO-1234] Commit with prefix"
+// "[FOO] Prefix numbers can be omitted"
+// "[FOO][BAR] Prefixes can couple"
+const VALID_COMMIT_TITLE_REGEX = /^((\[[A-Z]+(-\d+)?])+ )?[A-Z]\w[^\n]*[^.]$/
 const MAX_COMMIT_LINE_LENGTH = 79
 const ENGINEERING_DOCS_LINK =
   'https://toptal-core.atlassian.net/wiki/spaces/ENG/pages/210665897/Commit+Message+Quality'
@@ -14,7 +20,7 @@ module.exports = {
   ENGINEERING_DOCS_LINK,
   MAX_COMMIT_LINE_LENGTH,
   MISSING_TICKET_CODE_ERROR_MESSAGE,
-  VALID_COMMIT_MESSAGE_REGEX,
+  VALID_COMMIT_TITLE_REGEX,
   VALID_PR_CODE_REGEX,
   WHITELISTED_USERS
 }

package/src/configs/danger/toptal/plugins/toptal-commits/index.js

@@ -1,6 +1,6 @@
 const {
   MAX_COMMIT_LINE_LENGTH,
-  VALID_COMMIT_MESSAGE_REGEX,
+  VALID_COMMIT_TITLE_REGEX,
   WHITELISTED_USERS
 } = require('../../config')
 
@@ -8,14 +8,20 @@ const validateCommitMessage = message => {
   const errors = []
   const [title = '', body = ''] = message.split(/\n\n/)
   const bodyLines = body.split(/\n/)
-  const isFormatValid = VALID_COMMIT_MESSAGE_REGEX.test(message)
+  const isTitleFormatValid = VALID_COMMIT_TITLE_REGEX.test(title)
   const isTitleLengthValid = title.length <= MAX_COMMIT_LINE_LENGTH
   const isBodyLinesLengthValid = bodyLines.every(
     line => line.length <= MAX_COMMIT_LINE_LENGTH
   )
 
-  if (!isFormatValid) {
-    errors.push('- format of commit title or body is not correct')
+  if (!isTitleFormatValid) {
+    errors.push(`- format of commit title is not correct ([read more](https://toptal-core.atlassian.net/wiki/spaces/ENG/pages/210665897/Commit+Message+Quality)): 
+  - Title should start with capital letter
+  - Title should not end with a full-stop (i.e .)\n
+  Example of a valid commit title:
+    - Regular commit message
+    - [FOO-1234] Commit with prefix
+    - [FOO][BAR] Coupled prefixes`)
   }
 
   if (!isTitleLengthValid) {

package/src/configs/danger/toptal/plugins/toptal-pr-title/index.js

@@ -1,5 +1,5 @@
 const {
-  VALID_COMMIT_MESSAGE_REGEX,
+  VALID_COMMIT_TITLE_REGEX,
   VALID_PR_CODE_REGEX,
   WHITELISTED_USERS,
   DEFAULT_PR_TITLE_ERROR_MESSAGE,
@@ -47,7 +47,7 @@ const toptalPRTitle = () => {
   }
 
   const titleText = splitTitle[splitTitle.length - 1]
-  const isTitleFormatValid = VALID_COMMIT_MESSAGE_REGEX.test(titleText)
+  const isTitleFormatValid = VALID_COMMIT_TITLE_REGEX.test(titleText)
 
   if (!isTitleFormatValid) {
     showError()