Fix file path handling on Windows

Co-authored-by: Josh Faigan <[email protected]>

Author: aswamy

.changeset/khaki-hornets-care.md

@@ -0,0 +1,5 @@
+---
+'@shopify/theme-check-node': patch
+---
+
+[Bug fix] Fix file URI handling for Windows platforms

packages/theme-check-node/src/index.ts

@@ -153,9 +153,19 @@ export async function getTheme(config: Config): Promise<Theme> {
   // as mentioned in the documentation of node-glob
 
   // the path is normalised and '\' are replaced with '/' and then passed to the glob function
-  const normalizedGlob = path
+  let normalizedGlob = path
     .normalize(path.join(config.rootUri.replace(/^file:/, ''), '**/*.{liquid,json}'))
     .replace(/\\/g, '/');
+
+  // The glob is always an absoluate path, so windows paths should always
+  // start with a drive letter (with few exceptions we can ignore).
+  // More info: https://learn.microsoft.com/en-us/dotnet/standard/io/file-path-formats
+  //
+  // If an absoluate path starts with a slash on Windows, we strip it
+  if (process.platform === 'win32' && normalizedGlob.match(/^\/[a-zA-Z]:/)) {
+    normalizedGlob = normalizedGlob.slice(1);
+  }
+
   const paths = await asyncGlob(normalizedGlob).then((result) =>
     // Global ignored paths should not be part of the theme
     result.filter((filePath) => !isIgnored(filePath, config)),

packages/theme-check-node/src/test/test-helpers.ts

@@ -80,10 +80,10 @@ export async function makeTempWorkspace(structure: Tree): Promise<Workspace> {
 
   await createFiles(structure, [root]);
 
-  const rootUri = pathUtils.normalize('file:' + root);
+  const rootUri = pathUtils.normalize('file://' + root);
 
   return {
-    rootUri: 'file:' + root,
+    rootUri: 'file://' + root,
     uri: (relativePath) => pathUtils.join(rootUri, ...relativePath.split('/')),
     clean: async () => fs.rm(root, { recursive: true, force: true }),
   };