fix: absolute paths in Node env (#1134)

amzn · Jun 28, 2024 · 8720c42 · 8720c42
1 parent 19514e6
commit 8720c42
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/.changeset/tricky-snakes-appear.md b/.changeset/tricky-snakes-appear.md
@@ -0,0 +1,5 @@
+---
+'style-dictionary': patch
+---
+
+Fix scenario of passing absolute paths in Node env, do not remove leading slash in absolute paths.
diff --git a/lib/utils/combineJSON.js b/lib/utils/combineJSON.js
@@ -72,8 +72,11 @@ export default async function combineJSON(
     files = files.concat(new_files);
   }
 
-  // adjust for browser env glob results have leading slash
-  files = files.map((f) => f.replace(/^\//, ''));
+  if (typeof window === 'object') {
+    // adjust for browser env glob results have leading slash
+    // make sure we dont remove these in Node, that would break absolute paths!!
+    files = files.map((f) => f.replace(/^\//, ''));
+  }
 
   for (let i = 0; i < files.length; i++) {
     const filePath = files[i];