refactor: allow type tokens with spaces in lenient mode

Author: lppedd

src/main/kotlin/com/github/lppedd/cc/parser/ConventionalCommitParser.kt

@@ -54,6 +54,10 @@ internal fun parseConventionalCommit(message: String, lenient: Boolean = false):
     return ParseResult.Error("The commit type is missing or invalid")
   }
 
+  if (!lenient && type.any(Char::isWhitespace)) {
+    return ParseResult.Error("The commit type '$type' is invalid")
+  }
+
   var scope: String? = null
 
   if (reader.consumeIf(CCToken.Type.SCOPE_OPEN_PAREN)) {

src/main/kotlin/com/github/lppedd/cc/parser/specConventionalCommit.flex

@@ -93,7 +93,7 @@ FooterType  = [^\s:][^:\r\n]*
 
 <TYPE> {
       // The commit type must be at the beginning of the line, and cannot start with a '!'
-      ^[^!(:\s][^(:\s]*\!? {
+      ^[^!(:\r\n][^(:\r\n]*\!? {
         if (yycharat(yylength() - 1) == '!') {
           yypushback(1);
         }

src/test/kotlin/com/github/lppedd/cc/parser/ConventionalCommitParserTest.kt

@@ -7,7 +7,11 @@ import kotlin.contracts.contract
 class ConventionalCommitParserTest {
   @Test
   fun `fail incomplete`() {
-    var result = parseConventionalCommit("f!x")
+    var result = parseConventionalCommit("fi x:")
+    assertError(result)
+    assertEquals("The commit type 'fi x' is invalid", result.message)
+
+    result = parseConventionalCommit("f!x")
     assertError(result)
     assertEquals("The ':' separator is missing after the type/scope", result.message)
 
@@ -56,13 +60,21 @@ class ConventionalCommitParserTest {
 
   @Test
   fun `parse one liner`() {
-    val result = parseConventionalCommit("fix: foo")
+    var result = parseConventionalCommit("fix: foo")
     assertSuccess(result)
 
-    val message = result.message
+    var message = result.message
     assertEquals("fix", message.type)
     assertNull(message.scope)
     assertEquals("foo", message.subject.trim())
+
+    result = parseConventionalCommit("bui  (npm):foo", lenient = true)
+    assertSuccess(result)
+
+    message = result.message
+    assertEquals("bui  ", message.type)
+    assertEquals("npm", message.scope)
+    assertEquals("foo", message.subject)
   }
 
   @Test