[AssetMapper] Change default importmap "provider" to JsDelivr+esm

Author: weaverryan

DependencyInjection/Configuration.php

@@ -902,8 +902,8 @@ private function addAssetMapperSection(ArrayNodeDefinition $rootNode, callable $
                             ->defaultValue('%kernel.project_dir%/assets/vendor')
                         ->end()
                         ->scalarNode('provider')
-                            ->info('The provider (CDN) to use', class_exists(ImportMapManager::class) ? sprintf(' (e.g.: "%s").', implode('", "', ImportMapManager::PROVIDERS)) : '.')
-                            ->defaultValue('jspm')
+                            ->info('The provider (CDN) to use'.class_exists(ImportMapManager::class) ? sprintf(' (e.g.: "%s").', implode('", "', ImportMapManager::PROVIDERS)) : '.')
+                            ->defaultValue('jsdelivr.esm')
                         ->end()
                     ->end()
                 ->end()

DependencyInjection/FrameworkExtension.php

@@ -1314,7 +1314,17 @@ private function registerAssetMapperConfiguration(array $config, ContainerBuilde
             ->getDefinition('asset_mapper.importmap.manager')
             ->replaceArgument(2, $config['importmap_path'])
             ->replaceArgument(3, $config['vendor_dir'])
-            ->replaceArgument(4, $config['provider'])
+        ;
+
+        $importMapProviderId = 'asset_mapper.importmap.provider.jspm_provider';
+        if (ImportMapManager::PROVIDER_JSDELIVR_ESM === $config['provider']) {
+            $importMapProviderId = 'asset_mapper.importmap.provider.js_delivr_esm_provider';
+        }
+        $container->setAlias('asset_mapper.importmap.provider', new Alias($importMapProviderId));
+
+        $container
+            ->getDefinition('asset_mapper.importmap.provider.jspm_provider')
+            ->replaceArgument(0, $config['provider'])
         ;
 
         $container

Resources/config/asset_mapper.php

@@ -29,6 +29,8 @@
 use Symfony\Component\AssetMapper\Factory\MappedAssetFactory;
 use Symfony\Component\AssetMapper\ImportMap\ImportMapManager;
 use Symfony\Component\AssetMapper\ImportMap\ImportMapRenderer;
+use Symfony\Component\AssetMapper\ImportMap\Providers\JsDelivrEsmImportMapProvider;
+use Symfony\Component\AssetMapper\ImportMap\Providers\JspmImportMapProvider;
 use Symfony\Component\AssetMapper\MapperAwareAssetPackage;
 use Symfony\Component\AssetMapper\Path\PublicAssetsPathResolver;
 use Symfony\Component\HttpKernel\Event\RequestEvent;
@@ -136,10 +138,18 @@
                 service('asset_mapper.public_assets_path_resolver'),
                 abstract_arg('importmap.php path'),
                 abstract_arg('vendor directory'),
-                abstract_arg('provider'),
+                // dynamic alias to the chosen provider
+                service('asset_mapper.importmap.provider'),
             ])
         ->alias(ImportMapManager::class, 'asset_mapper.importmap.manager')
 
+        ->set('asset_mapper.importmap.provider.jspm_provider', JspmImportMapProvider::class)
+            ->args([
+                abstract_arg('provider'),
+            ])
+
+        ->set('asset_mapper.importmap.provider.js_delivr_esm_provider', JsDelivrEsmImportMapProvider::class)
+
         ->set('asset_mapper.importmap.renderer', ImportMapRenderer::class)
             ->args([
                 service('asset_mapper.importmap.manager'),

Tests/DependencyInjection/ConfigurationTest.php

@@ -112,7 +112,7 @@ public function testAssetMapperCanBeEnabled()
             'importmap_path' => '%kernel.project_dir%/importmap.php',
             'importmap_polyfill' => null,
             'vendor_dir' => '%kernel.project_dir%/assets/vendor',
-            'provider' => 'jspm',
+            'provider' => 'jsdelivr.esm',
             'importmap_script_attributes' => [],
         ];
 
@@ -624,7 +624,7 @@ protected static function getBundleDefaultConfig()
                 'importmap_path' => '%kernel.project_dir%/importmap.php',
                 'importmap_polyfill' => null,
                 'vendor_dir' => '%kernel.project_dir%/assets/vendor',
-                'provider' => 'jspm',
+                'provider' => 'jsdelivr.esm',
                 'importmap_script_attributes' => [],
             ],
             'cache' => [