Normalize globby paths (#1865)

* Normalize globby paths * Create young-socks-learn.md
jpmorganchase · Jun 22, 2022 · ad9a85c · ad9a85c
1 parent f8ef4e7
commit ad9a85c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/.changeset/young-socks-learn.md b/.changeset/young-socks-learn.md
@@ -0,0 +1,5 @@
+---
+"modular-scripts": patch
+---
+
+Fix files not added when using a template with `modular add` on Windows
diff --git a/packages/modular-scripts/src/addPackage.ts b/packages/modular-scripts/src/addPackage.ts
@@ -216,13 +216,13 @@ async function addPackage({
   const packageFilePaths = getAllFiles(packagePath);
 
   // If we get our package locally we need to whitelist files like yarn publish does
-  const packageWhitelist = globby.sync(
-    modularTemplatePackageJson.files || ['*'],
-    {
+  const packageWhitelist = globby
+    .sync(modularTemplatePackageJson.files || ['*'], {
       cwd: packagePath,
       absolute: true,
-    },
-  );
+    })
+    .map((filePath) => path.normalize(filePath));
+
   for (const packageFilePath of packageFilePaths) {
     if (!packageWhitelist.includes(packageFilePath)) {
       fs.removeSync(packageFilePath);