Adds support for special case : with test

Signed-off-by: Aminu Oluwaseun Joshua <[email protected]>

Author: seun-ja

crates/manifest/src/schema/v2.rs

@@ -796,6 +796,21 @@ mod tests {
         }
     }
 
+    #[test]
+    fn test_valid_kebab_ids() {
+        for valid in [
+            "default",
+            "mixed-case-words",
+            "letters1-then2-numbers345",
+            "gpt-oss:20b",
+            "gpt-4",
+        ] {
+            if let Err(err) = KebabId::try_from(valid.to_string()) {
+                panic!("{valid:?} should be value: {err:?}");
+            }
+        }
+    }
+
     #[test]
     fn test_invalid_snake_ids() {
         for invalid in [

crates/manifest/tests/ui/invalid/bad_kebab_component_ref.err

@@ -2,5 +2,5 @@ TOML parse error at line 7, column 13
   |
 7 | component = "1-2-3"
   |             ^^^^^^^
-'-'-separated words must start with an ASCII letter; got '1'
+'-'-separated words must start with an ASCII letter or digit; got '1'
 

crates/manifest/tests/ui/invalid/bad_variable_name.err

@@ -2,4 +2,4 @@ TOML parse error at line 7, column 1
   |
 7 | "bad+name" = { required = true }
   | ^^^^^^^^^^
-'_'-separated words may only contain alphanumeric ASCII; got '+'
+'_'-separated words may only contain alphanumeric ASCII (plus ':' inside words); got '+'

crates/serde/src/id.rs

@@ -44,23 +44,23 @@ impl<const DELIM: char, const LOWER: bool> TryFrom<String> for Id<DELIM, LOWER>
                 ));
             }
         }
-        for word in id.split(DELIM) {
+        for (i, word) in id.split(DELIM).enumerate() {
             if word.is_empty() {
                 return Err(format!("{DELIM:?}-separated words must not be empty"));
             }
             let mut chars = word.chars();
             let first = chars.next().unwrap();
-            if !first.is_ascii_alphabetic() {
+            if !(first.is_ascii_alphabetic() || (i > 0 && first.is_ascii_digit())) {
                 return Err(format!(
-                    "{DELIM:?}-separated words must start with an ASCII letter; got {first:?}"
+                    "{DELIM:?}-separated words must start with an ASCII letter or digit; got {first:?}"
                 ));
             }
             let word_is_uppercase = first.is_ascii_uppercase();
             for ch in chars {
-                if ch.is_ascii_digit() {
+                if ch.is_ascii_digit() || ch == ':' {
                 } else if !ch.is_ascii_alphanumeric() {
                     return Err(format!(
-                        "{DELIM:?}-separated words may only contain alphanumeric ASCII; got {ch:?}"
+                        "{DELIM:?}-separated words may only contain alphanumeric ASCII (plus ':' inside words); got {ch:?}"
                     ));
                 } else if ch.is_ascii_uppercase() != word_is_uppercase {
                     return Err(format!("{DELIM:?}-separated words must be all lowercase or all UPPERCASE; got {word:?}"));