bug/issue 1397 handle wildcard exports for sub export condition types (…

…#1398)

packages/cli/package.json

@@ -81,6 +81,7 @@
     "redux-thunk": "^3.1.0",
     "rehype-autolink-headings": "^4.0.0",
     "rehype-slug": "^3.0.0",
+    "signal-utils": "^0.21.1",
     "simpledotcss": "^1.0.0",
     "tslib": "^2.8.1"
   }

packages/cli/src/lib/walker-package-ranger.js

@@ -115,9 +115,11 @@ function patternRoot(pattern) {
  * "./icons/*": "./icons/*" - https://unpkg.com/browse/@spectrum-web-components/[email protected]/package.json
  * "./components/*": "./dist/components/*.js" - https://unpkg.com/browse/@uswds/[email protected]/package.json
  * "./src/components/*": "./src/components/* /index.js - https://unpkg.com/browse/@uswds/[email protected]/package.json
+ *  "./*": { "default": "./dist/*.ts.js" } - https://unpkg.com/browse/[email protected]/package.json
  */
 async function walkExportPatterns(dependency, sub, subValue, resolvedRoot) {
   // find the "deepest" segment we can start from to avoid unnecessary file scanning / crawling
+  const rootSubOffset = patternRoot(sub);
   const rootSubValueOffset = patternRoot(subValue);
 
   // ideally we can use fs.glob when it comes out of experimental
@@ -135,12 +137,13 @@ async function walkExportPatterns(dependency, sub, subValue, resolvedRoot) {
         walkDirectoryForExportPatterns(new URL(`./${file}/`, directoryUrl));
       } else if (regexPattern.test(filePathUrl.href)) {
         const relativePath = filePathUrl.href.replace(resolvedRoot, '');
-        // naive way to offset a subValue pattern to the sub pattern
-        // ex. "./js/*": "./packages/*/src/index.js",
-        // https://unpkg.com/browse/@uswds/[email protected]/package.json
-        const rootSubRelativePath = relativePath.replace(rootSubValueOffset, '');
+        // naive way to offset a subValue pattern to the sub pattern when dealing with wildcards
+        // ex. "./js/*": "./packages/*/src/index.js" -> /js/<package-name>/src/index.js
+        const rootSubRelativePath = sub.endsWith('*')
+          ? `./${relativePath}`.replace(subValue.split('*')[0], '').replace(subValue.split('*')[1], '')
+          : relativePath.replace(rootSubValueOffset, '');
 
-        updateImportMap(`${dependency}/${rootSubRelativePath}`, relativePath, resolvedRoot);
+        updateImportMap(`${dependency}${rootSubOffset}/${rootSubRelativePath}`, relativePath, resolvedRoot);
       }
     });
   }
@@ -182,7 +185,11 @@ async function walkPackageForExports(dependency, packageJson, resolvedRoot) {
         for (const condition of SUPPORTED_EXPORT_CONDITIONS) {
           if (exports[sub][condition]) {
             matched = true;
-            trackExportConditions(dependency, exports, sub, condition, resolvedRoot);
+            if (sub.indexOf('*') >= 0) {
+              await walkExportPatterns(dependency, sub, exports[sub][condition], resolvedRoot);
+            } else {
+              trackExportConditions(dependency, exports, sub, condition, resolvedRoot);
+            }
             break;
           }
         }

packages/cli/test/cases/develop.default/develop.default.spec.js

@@ -122,7 +122,7 @@ describe('Develop Greenwood With: ', function() {
         Object.keys(expectedImportMap).forEach((key) => {
           expect(expectedImportMap[key].startsWith('/~/')).to.equal(true);
 
-          // truncate full location to avoid difference between local dev and CI workspace paths
+          // truncate full location to avoid differences between local dev and CI workspace paths
           // e.g. passes on my machine
           const actualSubPath = importMap[key].slice(importMap[key].indexOf('/node_modules/'));
           const expectedSubPath = expectedImportMap[key].slice(expectedImportMap[key].indexOf('/node_modules/'));