core: compressImportMap deletes empty scopes; adjust types and tests

Author: lzxb

packages/core/src/utils/import-map.test.ts

@@ -1638,8 +1638,7 @@ describe('compressImportMap', () => {
 
         const result = compressImportMap(importMap);
         assert.deepEqual(result, {
-            imports: { vue: '/a/vue.final.mjs' },
-            scopes: {}
+            imports: { vue: '/a/vue.final.mjs' }
         });
     });
 
@@ -1672,8 +1671,7 @@ describe('compressImportMap', () => {
             imports: {
                 vue: '/shared/vue.final.mjs',
                 lodash: '/a/lodash.final.mjs'
-            },
-            scopes: {}
+            }
         };
         assert.deepEqual(result, expected);
     });
@@ -1689,8 +1687,7 @@ describe('compressImportMap', () => {
 
         const result = compressImportMap(importMap);
         assert.deepEqual(result, {
-            imports: { vue: '/shared/vue.final.mjs' },
-            scopes: {}
+            imports: { vue: '/shared/vue.final.mjs' }
         });
     });
 
@@ -1704,8 +1701,7 @@ describe('compressImportMap', () => {
         };
         const result = compressImportMap(importMap);
         assert.deepEqual(result, {
-            imports: { vue: '/x/vue.final.mjs' },
-            scopes: {}
+            imports: { vue: '/x/vue.final.mjs' }
         });
     });
 

packages/core/src/utils/import-map.ts

@@ -126,10 +126,8 @@ export function fixImportMapNestedScopes(
     return importMap;
 }
 
-export function compressImportMap(
-    importMap: Required<ImportMap>
-): Required<ImportMap> {
-    const compressed: Required<ImportMap> = {
+export function compressImportMap(importMap: Required<ImportMap>): ImportMap {
+    const compressed: ImportMap = {
         imports: { ...importMap.imports },
         scopes: {}
     };
@@ -179,6 +177,9 @@ export function compressImportMap(
         }
     });
 
+    if (Object.keys(compressed.scopes ?? {}).length === 0) {
+        Reflect.deleteProperty(compressed, 'scopes');
+    }
     return compressed;
 }
 
@@ -196,9 +197,7 @@ export function createImportMap({
     };
 }
 
-export function createClientImportMap(
-    options: GetImportMapOptions
-): Required<ImportMap> {
+export function createClientImportMap(options: GetImportMapOptions): ImportMap {
     const base = createImportMap(options);
     const fixed = fixImportMapNestedScopes(base);
     return compressImportMap(fixed);