fix(#233): change registration order to ensure composed providers are available in the request

Author: francescovallone

packages/serinus/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 2.1.7
+
+**Released on:** 14-03-2026
+
+### Fixes
+
+- Fix a bug that prevented composed providers to be correctly registered in the route context. This bug caused the composed providers to not be available in the route context, which could lead to issues with dependency injection and route handling. This fix ensures that composed providers are correctly registered and available in the route context, allowing for proper dependency injection and route handling when using composed providers in Serinus applications. (#233)[https://github.com/francescovallone/serinus/issues/233]
+
 ## 2.1.6
 
 **Released on:** 12-03-2026

packages/serinus/lib/src/containers/serinus_container.dart

@@ -75,8 +75,8 @@ class SerinusContainer {
       await modulesContainer.registerModules(entrypoint);
       modulesContainer.addEntrypointToInternalCoreModule(internalCoreModule);
     }
-    routesResolver?.resolve();
     await modulesContainer.finalize(entrypoint);
+    routesResolver?.resolve();
     inspector.inspectModules(modulesContainer.scopes);
     await emitHook<OnApplicationBootstrap>();
   }

packages/serinus/test/containers/composed_module_test.dart

@@ -1,3 +1,5 @@
+import 'dart:io';
+
 import 'package:mocktail/mocktail.dart';
 import 'package:serinus/serinus.dart';
 import 'package:test/test.dart';
@@ -12,6 +14,33 @@ class _MockAdapter extends Mock implements HttpAdapter {
   String get name => 'http';
 }
 
+class _MockIncomingMessage extends Mock implements IncomingMessage {}
+
+class _MockHttpHeaders extends Mock implements HttpHeaders {}
+
+Request _buildRequest() {
+  final incoming = _MockIncomingMessage();
+  when(() => incoming.queryParameters).thenReturn({});
+  when(() => incoming.headers).thenReturn(SerinusHeaders(_MockHttpHeaders()));
+  when(() => incoming.contentType).thenReturn(ContentType.json);
+  when(() => incoming.method).thenReturn('GET');
+  when(() => incoming.path).thenReturn('/composed');
+  when(() => incoming.uri).thenReturn(Uri.parse('http://localhost/composed'));
+  when(() => incoming.id).thenReturn('req-composed');
+  when(() => incoming.host).thenReturn('localhost');
+  when(() => incoming.hostname).thenReturn('localhost');
+  when(() => incoming.port).thenReturn(3000);
+  when(() => incoming.cookies).thenReturn(const []);
+  when(() => incoming.segments).thenReturn(const []);
+  when(() => incoming.clientInfo).thenReturn(null);
+  when(() => incoming.contentLength).thenReturn(0);
+  when(() => incoming.isWebSocket).thenReturn(false);
+  when(() => incoming.webSocketKey).thenReturn('');
+  when(() => incoming.body()).thenReturn('');
+  when(() => incoming.json()).thenReturn(null);
+  return Request(incoming);
+}
+
 // Define a provider type that will be produced by a composed provider
 class ProducedProvider extends Provider {
   ProducedProvider();
@@ -364,4 +393,37 @@ void main() {
       },
     );
   });
+
+  group('ComposedProvider in RequestContext', () {
+    test('composed providers are available in RequestContext', () async {
+      final composedA = Module.composed<Module>(
+        (CompositionContext ctx) async => ProducedModuleA(),
+        inject: const [],
+      );
+      final parent = ParentModule([composedA]);
+
+      final container = ModulesContainer(
+        ApplicationConfig(serverAdapter: _MockAdapter()),
+      );
+      await container.registerModules(parent);
+      await container.finalize(parent);
+
+      final scope = container.getScopeByProvider(ProducedProvider);
+      final providers = <Type, Provider>{
+        for (final provider in scope.unifiedProviders)
+          provider.runtimeType: provider,
+      };
+
+      final context = RequestContext<dynamic>.withBody(
+        _buildRequest(),
+        null,
+        providers,
+        scope.unifiedValues,
+        const <Type, Object>{},
+      );
+
+      expect(context.canUse<ProducedProvider>(), isTrue);
+      expect(context.use<ProducedProvider>().hello(), equals('produced'));
+    });
+  });
 }