Turbopack: support character class ranges in regex

turbopack/crates/turbo-tasks-fs/src/glob.rs

@@ -237,6 +237,7 @@ mod tests {
     #[case::alternatives_empty1("react{,-dom}", "react")]
     #[case::alternatives_empty2("react{,-dom}", "react-dom")]
     #[case::alternatives_chars("[abc]", "b")]
+    #[case::character_range("[a-z].js", "b.js")]
     fn glob_match(#[case] glob: &str, #[case] path: &str) {
         let glob = Glob::parse(RcStr::from(glob), GlobOptions::default()).unwrap();
 
@@ -245,6 +246,13 @@ mod tests {
         assert!(glob.matches(path));
     }
 
+    #[test]
+    fn glob_rejects_invalid_character_range() {
+        let error = Glob::parse(rcstr!("[z-a]"), GlobOptions::default()).unwrap_err();
+
+        assert!(format!("{error:#}").contains("invalid character range"));
+    }
+
     #[rstest]
     #[case::early_end("*.raw", "hello.raw.js")]
     #[case::early_end(

turbopack/crates/turbo-tasks-fs/src/globset.rs

@@ -585,7 +585,7 @@ impl<'a> Parser<'a> {
                     if in_range {
                         // invariant: in_range is only set when there is
                         // already at least one character seen.
-                        if let Some(kind) = add_to_last_range(ranges.last_mut().unwrap(), '-') {
+                        if let Some(kind) = add_to_last_range(ranges.last_mut().unwrap(), c) {
                             return Err(self.error(kind));
                         }
                     } else {
@@ -653,6 +653,7 @@ mod tests {
         "(?:a|b|c(?:/)?)(?:/h(?:/.*)?)?"
     )]
     #[case::classes("[abc]/d/**", "[abc]/d/.*", "[abc](?:/d(?:/.*)?)?")]
+    #[case::ranges("[a-z]/d/**", "[a-z]/d/.*", "[a-z](?:/d(?:/.*)?)?")]
     fn glob_regex_mapping(
         #[case] glob: &str,
         #[case] glob_regex: &str,