@esmx/core 3.0.0-rc.62 → 3.0.0-rc.64

package/src/utils/import-map.ts

@@ -4,7 +4,6 @@ import type { ImportMap, ScopesMap, SpecifierMap } from '@esmx/import';
 
 export interface ImportMapManifest {
     name: string;
-    imports: Record<string, string>;
     exports: Record<
         string,
         {
@@ -36,13 +35,6 @@ export function buildImportsMap(
         });
     });
 
-    manifests.forEach((manifest) => {
-        Object.entries(manifest.imports).forEach(([name, identifier]) => {
-            const fullName = `${manifest.name}/${name}`;
-            imports[fullName] = imports[identifier] ?? identifier;
-        });
-    });
-
     pathWithoutIndex(imports);
 
     return imports;
@@ -80,12 +72,64 @@ export function buildScopesMap(
 
     return scopes;
 }
+/**
+ * Fixes Chrome's nested scope resolution bug in import maps.
+ *
+ * Chrome has a bug where nested scopes in import maps are not resolved correctly.
+ * For example, when you have both "/shared-modules/" and "/shared-modules/vue2/" scopes,
+ * Chrome fails to properly apply the more specific nested scope.
+ *
+ * This function works around the bug by:
+ * 1. Sorting scopes by path depth (shallow paths first, deeper paths last)
+ * 2. Manually applying scopes to matching imports in the correct order
+ *
+ * @example
+ * Problematic import map that fails in Chrome:
+ * ```json
+ * {
+ *   "scopes": {
+ *     "/shared-modules/": {
+ *       "vue": "/shared-modules/vue.d8c7a640.final.mjs"
+ *     },
+ *     "/shared-modules/vue2/": {
+ *       "vue": "/shared-modules/vue2.9b4efaf3.final.mjs"
+ *     }
+ *   }
+ * }
+ * ```
+ *
+ * @see https://github.com/guybedford/es-module-shims/issues/529
+ * @see https://issues.chromium.org/issues/453147451
+ */
+export function fixNestedScopesResolution(
+    importMap: Required<ImportMap>
+): Required<ImportMap> {
+    Object.entries(importMap.scopes)
+        .sort(([pathA], [pathB]) => {
+            const depthA = pathA.split('/').length;
+            const depthB = pathB.split('/').length;
+            return depthA - depthB;
+        })
+        .forEach(([scopePath, scopeMappings]) => {
+            Object.values(importMap.imports).forEach((importPath) => {
+                if (importPath.startsWith(scopePath)) {
+                    importMap.scopes[importPath] = {
+                        ...importMap.scopes[importPath],
+                        ...scopeMappings
+                    };
+                }
+            });
+            Reflect.deleteProperty(importMap.scopes, scopePath);
+        });
+
+    return importMap;
+}
 
 export function getImportMap({
     manifests,
     getFile,
     getScope
-}: GetImportMapOptions): ImportMap {
+}: GetImportMapOptions): Required<ImportMap> {
     const imports = buildImportsMap(manifests, getFile);
 
     const scopes = buildScopesMap(imports, manifests, getScope);