feat(parse): Make scope syntax less restrictive

aexvir · kodiakhq[bot] · commit 93098de466a7 · 2020-03-12T11:31:55.000Z
diff --git a/parse_commit.go b/parse_commit.go
@@ -10,7 +10,7 @@ import (
 var (
 	referenceFormatRegex   = regexp.MustCompile(`Refs:?[^\r\n]*`)
 	referenceIDFormatRegex = regexp.MustCompile(`\#([0-9]+)`)
-	expectedFormatRegex    = regexp.MustCompile(`(?s)^(?P<category>\w+?)?(?P<scope>\(\S+\))?(?P<breaking>!?)?: (?P<heading>[^\n\r]+)?([\n\r]{2}(?P<body>.*))?`)
+	expectedFormatRegex    = regexp.MustCompile(`(?s)^(?P<category>\w+?)?(?P<scope>\([^\)]+\))?(?P<breaking>!?)?: (?P<heading>[^\n\r]+)?([\n\r]{2}(?P<body>.*))?`)
 )
 
 // GetIssueNumbers converts the matches from the reference regular expression to integers
diff --git a/parse_commit_test.go b/parse_commit_test.go
@@ -21,11 +21,13 @@ func TestParseCommitMessage(t *testing.T) {
 		"chore: test\n\nsomething more here\nRefs: #12 and #13":             Commit{Category: "chore", Scope: "", Heading: "test", Body: "something more here", Issues: []int{12, 13}},
 		"chore: add something\n":                                            Commit{Category: "chore", Heading: "add something"},
 		"chore(ci): added new CI stuff\n":                                   Commit{Category: "chore", Scope: "ci", Heading: "added new CI stuff"},
+                "chore(a b c d): added new CI stuff\n":                              Commit{Category: "chore", Scope: "a b c d", Heading: "added new CI stuff"},
+                "chore(><(((*>): added new CI stuff\n":                              Commit{Category: "chore", Scope: "><(((*>", Heading: "added new CI stuff"},
 		"feat: added a new feature\n":                                       Commit{Category: "feat", Heading: "added a new feature"},
 		"fix!: breaking change\n":                                           Commit{Category: "fix", Breaking: true, Heading: "breaking change"},
 		"fix(security)!: breaking\n":                                        Commit{Category: "fix", Scope: "security", Breaking: true, Heading: "breaking"},
 		"fix!!: breaking\n":                                                 Commit{Heading: "fix!!: breaking\n"},
-		"fix(security)(stuff): should break\n":                              Commit{Category: "fix", Scope: "security(stuff)", Heading: "should break"},
+		"fix(security)(stuff): should break\n":                              Commit{Heading: "fix(security)(stuff): should break\n"},
 		"chore:really close\n":                                              Commit{Heading: "chore:really close\n"},
 		"perf(): nope\n":                                                    Commit{Heading: "perf(): nope\n"},
 		"chore(: bad\n":                                                     Commit{Heading: "chore(: bad\n"},

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ import (`
`10`	`10`	`var (`
`11`	`11`	referenceFormatRegex = regexp.MustCompile(`Refs:?[^\r\n]*`)
`12`	`12`	referenceIDFormatRegex = regexp.MustCompile(`\#([0-9]+)`)
`13`		- expectedFormatRegex = regexp.MustCompile(`(?s)^(?P<category>\w+?)?(?P<scope>\(\S+\))?(?P<breaking>!?)?: (?P<heading>[^\n\r]+)?([\n\r]{2}(?P<body>.*))?`)
	`13`	+ expectedFormatRegex = regexp.MustCompile(`(?s)^(?P<category>\w+?)?(?P<scope>\([^\)]+\))?(?P<breaking>!?)?: (?P<heading>[^\n\r]+)?([\n\r]{2}(?P<body>.*))?`)
`14`	`14`	`)`
`15`	`15`
`16`	`16`	`// GetIssueNumbers converts the matches from the reference regular expression to integers`