fix: 🐛 crossplatform glob expr (#2441)

IWANABETHATGUY · web-flow · commit 5bf9e84081bc · 2024-10-15T13:09:57.000Z
&lt;!-- Thank you for contributing! --&gt;

### Description

&lt;!-- Please insert your description here and provide especially info
about the "what" this PR is solving --&gt;
diff --git a/scripts/snap-diff/aggregate-reason.ts b/scripts/snap-diff/aggregate-reason.ts
@@ -44,32 +44,35 @@ export function extractReason(source: string) {
   return ret
 }
 
-const esbuildTestDir = path.join(
-  import.meta.dirname,
-  '../../crates/rolldown/tests/esbuild',
-)
-
 const workspaceDir = path.join(import.meta.dirname, '../..')
+
 export type AggregateReasonEntries = [string, string[]][]
+
 export function aggregateReason(): AggregateReasonEntries {
-  const entries = fg.globSync([`${esbuildTestDir}/**/diff.md`], { dot: false })
+  const entries = fg.globSync(['crates/rolldown/tests/esbuild/**/diff.md'], {
+    dot: false,
+    cwd: workspaceDir,
+  })
   // a map for each directory to its diff reasons
-  let reasonMap: Record<string, string[]> = {}
-  let reverseMap: Record<string, string[]> = {}
+  let reasonToCaseDirMap: Record<string, string[]> = {}
   for (let entry of entries) {
-    let content = fs.readFileSync(entry, 'utf-8')
+    const entryAbPath = path.resolve(workspaceDir, entry)
+    let content = fs.readFileSync(entryAbPath, 'utf-8')
     let reasons = extractReason(content)
-    let dirname = path.relative(workspaceDir, path.dirname(entry))
+    let dirname = path.relative(workspaceDir, path.dirname(entryAbPath))
+    const posixPath = dirname.replaceAll('\\', '/')
 
-    reasonMap[dirname] = reasons
     for (let reason of reasons) {
-      if (!reverseMap[reason]) {
-        reverseMap[reason] = []
+      if (!reasonToCaseDirMap[reason]) {
+        reasonToCaseDirMap[reason] = []
       }
-      reverseMap[reason].push(dirname)
+      reasonToCaseDirMap[reason].push(posixPath)
     }
   }
-  let reverseMapEntries = Object.entries(reverseMap)
+  let reverseMapEntries = Object.entries(reasonToCaseDirMap)
+  for (let [_, dirs] of reverseMapEntries) {
+    dirs.sort()
+  }
   reverseMapEntries.sort((a, b) => {
     return b[1].length - a[1].length
   })
