Connect controllers without listing every context: 3,000 controllers start 43% faster

src/core/outlet_observer.ts

@@ -222,7 +222,9 @@ export class OutletObserver implements AttributeObserverDelegate, SelectorObserv
 
   private get dependentContexts() {
     const identifiers = this.dependentControllerIdentifiers
-    return this.router.contexts.filter((context) => identifiers.includes(context.identifier))
+    return this.router.modules
+      .filter((module) => identifiers.includes(module.identifier))
+      .reduce((contexts, module) => contexts.concat(module.contexts), [] as Context[])
   }
 
   private hasOutlet(element: Element, outletName: string) {

src/tests/modules/core/application_tests.ts

@@ -45,6 +45,23 @@ export default class ApplicationTests extends ApplicationTestCase {
     this.assert.ok(this.controllers[0] instanceof BController)
   }
 
+  "test connecting controllers does not list the contexts of every controller"() {
+    const router = this.application.router
+    const contexts = Object.getOwnPropertyDescriptor(Object.getPrototypeOf(router), "contexts")!.get!
+    let contextsReads = 0
+    Object.defineProperty(router, "contexts", {
+      configurable: true,
+      get() {
+        contextsReads++
+        return contexts.call(this)
+      },
+    })
+
+    this.application.load(this.definitions)
+
+    this.assert.equal(contextsReads, 0)
+  }
+
   get controllers() {
     return this.application.controllers as LogController[]
   }