[AssetMapper] Put importmap in polyfill so it can be hosted locally easily

Author: weaverryan

CHANGELOG.md

@@ -31,6 +31,7 @@ CHANGELOG
  * Deprecate the `framework.asset_mapper.provider` config option
  * Add `--exclude` option to the `cache:pool:clear` command
  * Add parameters deprecations to the output of `debug:container` command
+ * Change `framework.asset_mapper.importmap_polyfill` from a URL to the name of an item in the importmap
 
 6.3
 ---

DependencyInjection/Configuration.php

@@ -924,8 +924,8 @@ private function addAssetMapperSection(ArrayNodeDefinition $rootNode, callable $
                             ->defaultValue('%kernel.project_dir%/importmap.php')
                         ->end()
                         ->scalarNode('importmap_polyfill')
-                            ->info('URL of the ES Module Polyfill to use, false to disable. Defaults to using a CDN URL.')
-                            ->defaultValue(null)
+                            ->info('The importmap name that will be used to load the polyfill. Set to false to disable.')
+                            ->defaultValue('es-module-shims')
                         ->end()
                         ->arrayNode('importmap_script_attributes')
                             ->info('Key-value pair of attributes to add to script tags output for the importmap.')

DependencyInjection/FrameworkExtension.php

@@ -33,7 +33,6 @@
 use Symfony\Component\Asset\PackageInterface;
 use Symfony\Component\AssetMapper\AssetMapper;
 use Symfony\Component\AssetMapper\Compiler\AssetCompilerInterface;
-use Symfony\Component\AssetMapper\ImportMap\ImportMapManager;
 use Symfony\Component\BrowserKit\AbstractBrowser;
 use Symfony\Component\Cache\Adapter\AdapterInterface;
 use Symfony\Component\Cache\Adapter\ArrayAdapter;
@@ -1378,7 +1377,7 @@ private function registerAssetMapperConfiguration(array $config, ContainerBuilde
 
         $container
             ->getDefinition('asset_mapper.importmap.renderer')
-            ->replaceArgument(3, $config['importmap_polyfill'] ?? ImportMapManager::POLYFILL_URL)
+            ->replaceArgument(3, $config['importmap_polyfill'])
             ->replaceArgument(4, $config['importmap_script_attributes'])
         ;
     }

Tests/DependencyInjection/ConfigurationTest.php

@@ -132,7 +132,7 @@ public function testAssetMapperCanBeEnabled()
             'missing_import_mode' => 'warn',
             'extensions' => [],
             'importmap_path' => '%kernel.project_dir%/importmap.php',
-            'importmap_polyfill' => null,
+            'importmap_polyfill' => 'es-module-shims',
             'vendor_dir' => '%kernel.project_dir%/assets/vendor',
             'importmap_script_attributes' => [],
         ];
@@ -668,7 +668,7 @@ protected static function getBundleDefaultConfig()
                 'missing_import_mode' => 'warn',
                 'extensions' => [],
                 'importmap_path' => '%kernel.project_dir%/importmap.php',
-                'importmap_polyfill' => null,
+                'importmap_polyfill' => 'es-module-shims',
                 'vendor_dir' => '%kernel.project_dir%/assets/vendor',
                 'importmap_script_attributes' => [],
             ],