fix: ensure Vocab exceptions are sorted by length

This ensures that, e.g., "Vale Server" will appear before "Vale" -- effectively
making our regex use greedy alternation.

check/capitalization.go

@@ -46,9 +46,7 @@ func NewCapitalization(cfg *config.Config, generic baseCheck) (Capitalization, e
 		func() string { return "" },
 		true)
 
-	for term := range cfg.AcceptedTokens {
-		rule.Exceptions = append(rule.Exceptions, term)
-	}
+	rule.Exceptions = updateExceptions(rule.Exceptions, cfg.AcceptedTokens)
 
 	regex = fmt.Sprintf(regex, strings.Join(rule.Exceptions, "|"))
 	if len(rule.Exceptions) > 0 {

check/conditional.go

@@ -38,9 +38,7 @@ func NewConditional(cfg *config.Config, generic baseCheck) (Conditional, error)
 		return rule, readStructureError(err, path)
 	}
 
-	for term := range cfg.AcceptedTokens {
-		rule.Exceptions = append(rule.Exceptions, term)
-	}
+	rule.Exceptions = updateExceptions(rule.Exceptions, cfg.AcceptedTokens)
 	rule.exceptRe = regexp.MustCompile(strings.Join(rule.Exceptions, "|"))
 
 	re, err = regexp.Compile(rule.Second)

check/definition.go

@@ -2,6 +2,7 @@ package check
 
 import (
 	"fmt"
+	"sort"
 	"strconv"
 	"strings"
 
@@ -247,3 +248,16 @@ func matchToken(expected, observed string, ignorecase bool) bool {
 	}
 	return r.MatchString(observed)
 }
+
+func updateExceptions(previous []string, current map[string]struct{}) []string {
+	for term := range current {
+		previous = append(previous, term)
+	}
+
+	// NOTE: This is required to ensure that we have greedy alternation.
+	sort.Slice(previous, func(p, q int) bool {
+		return len(previous[p]) > len(previous[q])
+	})
+
+	return previous
+}

fixtures/misc/filesystem/projects/ini/styles/Vocab/Basic/accept.txt

@@ -1,3 +1,3 @@
-Vale Server
 Vale
+Vale Server
 JavaScript