fix(core): normalise windows backslashes

Closes #126

Author: JamieMason

src/main.rs

@@ -31,6 +31,8 @@ mod instance_state;
 mod logger;
 mod package_json;
 mod packages;
+#[cfg(test)]
+mod packages_test;
 mod rcfile;
 #[cfg(test)]
 mod rcfile_test;

src/packages.rs

@@ -156,6 +156,24 @@ impl Packages {
   }
 }
 
+/// Normalize a source pattern by:
+/// 1. Converting Windows backslashes to forward slashes for glob compatibility
+/// 2. Ensuring pattern ends with /package.json
+///
+/// Examples:
+/// - "projects\\apps\\*" -> "projects/apps/*/package.json"
+/// - "projects/libs/*" -> "projects/libs/*/package.json"
+/// - "package.json" -> "package.json"
+/// - "apps\\*/package.json" -> "apps/*/package.json"
+pub fn normalize_pattern(pattern: String) -> String {
+  let normalized = pattern.replace('\\', "/");
+  if normalized.contains("package.json") {
+    normalized
+  } else {
+    format!("{normalized}/package.json")
+  }
+}
+
 /// Resolve every source glob pattern into their absolute file paths of
 /// package.json files
 fn get_file_paths(config: &Config) -> Vec<PathBuf> {
@@ -214,18 +232,7 @@ fn get_source_patterns(config: &Config) -> Vec<String> {
           patterns
         })
     })
-    .map(|patterns| {
-      patterns
-        .into_iter()
-        .map(|pattern| {
-          if pattern.contains("package.json") {
-            pattern
-          } else {
-            format!("{pattern}/package.json")
-          }
-        })
-        .collect()
-    })
+    .map(|patterns| patterns.into_iter().map(normalize_pattern).collect())
     .or_else(get_default_patterns)
     .unwrap()
 }

src/packages_test.rs

@@ -0,0 +1,33 @@
+use crate::packages::normalize_pattern;
+
+#[test]
+fn normalizes_backslashes_to_forward_slashes() {
+  let cases = vec![
+    // Windows-style backslashes
+    ("projects\\apps\\*", "projects/apps/*/package.json"),
+    ("projects\\libs\\lib1", "projects/libs/lib1/package.json"),
+    ("apps\\*\\src", "apps/*/src/package.json"),
+    // Mixed slashes
+    ("projects\\mixed/pkg1", "projects/mixed/pkg1/package.json"),
+    ("apps/test\\utils", "apps/test/utils/package.json"),
+    // Already normalized (forward slashes)
+    ("projects/apps/*", "projects/apps/*/package.json"),
+    ("packages/*", "packages/*/package.json"),
+    // Already includes package.json with backslashes
+    ("apps\\*/package.json", "apps/*/package.json"),
+    ("projects\\libs\\*\\package.json", "projects/libs/*/package.json"),
+    // Already includes package.json with forward slashes
+    ("apps/*/package.json", "apps/*/package.json"),
+    ("packages/*/package.json", "packages/*/package.json"),
+    // Just package.json
+    ("package.json", "package.json"),
+    // Complex patterns
+    ("**\\*\\package.json", "**/*/package.json"),
+    ("src\\**\\tests", "src/**/tests/package.json"),
+  ];
+
+  for (input, expected) in cases {
+    let result = normalize_pattern(input.to_string());
+    assert_eq!(result, expected, "normalize_pattern({input:?}) should return {expected:?}");
+  }
+}