refactor: contextual implementation for rune range validation

token/token.go

@@ -654,37 +654,41 @@ func IsNeedQuoted(value string) bool {
 // shouldQuoteRune returns true if the rune should be quoted.
 // excludes all runes in the Basic Multilingual Plane and all Emoticons
 func shouldQuoteRune(r rune) bool {
-	if r < 0x1000 { // Basic Multilingual Plane
-		return false
-	}
-	if unicode.IsLetter(r) { // Letters in any language
+	// Basic Multilingual Plane, Letters and Emoji
+	if r < 0x1000 || unicode.IsLetter(r) || isEmoticon(r) {
 		return false
 	}
+	return true
+}
+
+// uses the following list to identify the emoticon range
+// https://unicode.org/emoji/charts/full-emoji-list.html
+func isEmoticon(r rune) bool {
 	if r > 0x1F600 && r < 0x1F64F { // Emoticons
-		return false
+		return true
 	}
 	if r > 0x1F300 && r < 0x1F5FF { // Misc Symbols and Pictographs
-		return false
+		return true
 	}
 	if r > 0x1F680 && r < 0x1F6FF { // Transport and Map
-		return false
+		return true
 	}
 	if r > 0x2600 && r < 0x26FF { // Misc symbols
-		return false
+		return true
 	}
 	if r > 0x2700 && r < 0x27BF { // Dingbats
-		return false
+		return true
 	}
 	if r > 0xFE00 && r < 0xFE0F { // Variation Selectors
-		return false
+		return true
 	}
 	if r > 0x1F900 && r < 0x1F9FF { // Supplemental Symbols and Pictographs
-		return false
+		return true
 	}
 	if r > 0x1F1E6 && r < 0x1F1FF { // Flags
-		return false
+		return true
 	}
-	return true
+	return false
 }
 
 // LiteralBlockHeader detect literal block scalar header