feature #62800 [HttpKernel] Add support for bundles as compiler passes (yceruto)

nicolas-grekas · nicolas-grekas · commit 7862a339e4b2 · 2025-12-19T08:46:45.000+01:00
This PR was merged into the 8.1 branch. Discussion ---------- [HttpKernel] Add support for bundles as compiler passes | Q | A | ------------- | --- | Branch? | 8.1 | Bug fix? | no | New feature? | yes | Deprecations? | no | Issues | Fix #62470 | License | MIT Adds a bundle-level compiler pass to hook into the container compilation phase: ```php class MyBundle extends AbstractBundle implements CompilerPassInterface { public function process(ContainerBuilder $container): void { // ... } } ``` Commits ------- b029905 [HttpKernel] Add support for bundles as compiler passes
diff --git a/src/Symfony/Component/HttpKernel/CHANGELOG.md b/src/Symfony/Component/HttpKernel/CHANGELOG.md
@@ -1,6 +1,11 @@
 CHANGELOG
 =========
 
+8.1
+---
+
+ * Add support for bundles as compiler pass
+
 8.0
 ---
 
diff --git a/src/Symfony/Component/HttpKernel/Kernel.php b/src/Symfony/Component/HttpKernel/Kernel.php
@@ -616,6 +616,10 @@ protected function prepareContainer(ContainerBuilder $container): void
             if ($this->debug) {
                 $container->addObjectResource($bundle);
             }
+
+            if ($bundle instanceof CompilerPassInterface) {
+                $container->addCompilerPass($bundle, PassConfig::TYPE_BEFORE_OPTIMIZATION, -10000);
+            }
         }
 
         foreach ($this->bundles as $bundle) {
diff --git a/src/Symfony/Component/HttpKernel/Tests/Bundle/BundleTest.php b/src/Symfony/Component/HttpKernel/Tests/Bundle/BundleTest.php
@@ -12,7 +12,10 @@
 namespace Symfony\Component\HttpKernel\Tests\Bundle;
 
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\Config\Loader\LoaderInterface;
 use Symfony\Component\HttpKernel\Bundle\Bundle;
+use Symfony\Component\HttpKernel\Kernel;
+use Symfony\Component\HttpKernel\Tests\Fixtures\BundleCompilerPass\BundleAsCompilerPassBundle;
 use Symfony\Component\HttpKernel\Tests\Fixtures\ExtensionPresentBundle\ExtensionPresentBundle;
 
 class BundleTest extends TestCase
@@ -43,6 +46,29 @@ public function testBundleNameCanBeExplicitlyProvided()
         $this->assertSame('Symfony\Component\HttpKernel\Tests\Bundle', $bundle->getNamespace());
         $this->assertSame('ExplicitlyNamedBundle', $bundle->getName());
     }
+
+    public function testBundleAsCompilerPass()
+    {
+        $kernel = new class('test', true) extends Kernel {
+            public function registerBundles(): iterable
+            {
+                yield new BundleAsCompilerPassBundle();
+            }
+
+            public function registerContainerConfiguration(LoaderInterface $loader): void
+            {
+            }
+
+            public function getProjectDir(): string
+            {
+                return sys_get_temp_dir().'/bundle_as_compiler_pass';
+            }
+        };
+
+        $kernel->boot();
+
+        $this->assertTrue($kernel->getContainer()->has('foo'));
+    }
 }
 
 class NamedBundle extends Bundle
diff --git a/src/Symfony/Component/HttpKernel/Tests/Fixtures/BundleCompilerPass/BundleAsCompilerPassBundle.php b/src/Symfony/Component/HttpKernel/Tests/Fixtures/BundleCompilerPass/BundleAsCompilerPassBundle.php
@@ -0,0 +1,25 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\HttpKernel\Tests\Fixtures\BundleCompilerPass;
+
+use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\HttpKernel\Bundle\AbstractBundle;
+
+class BundleAsCompilerPassBundle extends AbstractBundle implements CompilerPassInterface
+{
+    public function process(ContainerBuilder $container): void
+    {
+        $container->register('foo', \stdClass::class)
+            ->setPublic(true);
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -616,6 +616,10 @@ protected function prepareContainer(ContainerBuilder $container): void`
`616`	`616`	`if ($this->debug) {`
`617`	`617`	`$container->addObjectResource($bundle);`
`618`	`618`	`}`
	`619`	`+`
	`620`	`+ if ($bundle instanceof CompilerPassInterface) {`
	`621`	`+ $container->addCompilerPass($bundle, PassConfig::TYPE_BEFORE_OPTIMIZATION, -10000);`
	`622`	`+ }`
`619`	`623`	`}`
`620`	`624`
`621`	`625`	`foreach ($this->bundles as $bundle) {`