refactor: require a subject separator even in lenient mode

Author: lppedd

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

@@ -58,35 +58,24 @@ internal fun parseConventionalCommit(message: String, lenient: Boolean = false):
     return ParseResult.Error("The commit type '$type' is invalid")
   }
 
+  // These messages are valid:
+  //  'build(  ): updated dev dependencies'
+  //  'build(): updated dev dependencies'
   var scope: String? = null
 
   if (reader.consumeIf(CCToken.Type.SCOPE_OPEN_PAREN)) {
-    scope = reader.consume(CCToken.Type.SCOPE)
-    val hasCloseParen = reader.consumeIf(CCToken.Type.SCOPE_CLOSE_PAREN)
+    scope = reader.consume(CCToken.Type.SCOPE) ?: ""
 
-    if (scope.isNullOrBlank()) {
-      if (!lenient && !hasCloseParen) {
-        return ParseResult.Error("The commit scope is missing or invalid")
-      }
-
-      // These messages are valid:
-      //  'build(  ): updated dev dependencies'
-      //  'build(): updated dev dependencies'
-      scope = scope ?: ""
-    }
-
-    // In lenient mode this message without closing parenthesis is valid:
-    //  'build(np  '
-    if (!lenient && !hasCloseParen) {
+    if (!reader.consumeIf(CCToken.Type.SCOPE_CLOSE_PAREN)) {
       return ParseResult.Error("The commit scope is missing the closing parenthesis")
     }
   }
 
   val isBreakingChange = reader.consume(CCToken.Type.BREAKING_CHANGE) != null
 
-  // In lenient mode this message without a subject separator is valid:
-  //  'build(npm)'
-  if (!reader.consumeIf(CCToken.Type.SEPARATOR) && !lenient) {
+  // The ':' subject separator is required to recognize a Conventional Commits message,
+  // even in lenient mode
+  if (!reader.consumeIf(CCToken.Type.SEPARATOR)) {
     return ParseResult.Error("The ':' separator is missing after the type/scope")
   }
 

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

@@ -41,15 +41,7 @@ class ConventionalCommitParserTest {
     assertEquals("fix", message.type)
     assertEquals("  ", message.subject)
 
-    result = parseConventionalCommit("build(np:  ", lenient = true)
-    assertSuccess(result)
-
-    message = result.message
-    assertEquals("build", message.type)
-    assertEquals("np:  ", message.scope)
-    assertEquals("", message.subject)
-
-    result = parseConventionalCommit("build(npm)", lenient = true)
+    result = parseConventionalCommit("build(npm):", lenient = true)
     assertSuccess(result)
 
     message = result.message