Use service registry for render phases

src/GameKit.RenderOrchestration/DefaultRenderManager.cs

@@ -1,4 +1,5 @@
 using GameKit.App;
+using GameKit.DependencyInjection;
 using GameKit.Gpu;
 
 namespace GameKit.RenderOrchestration;
@@ -12,16 +13,16 @@ public class DefaultRenderManager<TRenderContext> : IRenderManager
 {
     private readonly GpuMemorySystem _gpuMemorySystem;
     private readonly IRenderContextProvider<TRenderContext> _renderContextProvider;
-    private readonly RenderPhaseRegistry<TRenderContext> _renderPhaseRegistry;
+    private readonly ServiceRegistry<IRenderPhase<TRenderContext>> _renderPhases;
 
     internal DefaultRenderManager(
         GpuMemorySystem gpuMemorySystem,
         IRenderContextProvider<TRenderContext> renderContextProvider,
-        RenderPhaseRegistry<TRenderContext> renderPhaseRegistry)
+        ServiceRegistry<IRenderPhase<TRenderContext>> renderPhases)
     {
         _gpuMemorySystem = gpuMemorySystem;
         _renderContextProvider = renderContextProvider;
-        _renderPhaseRegistry = renderPhaseRegistry;
+        _renderPhases = renderPhases;
     }
 
     /// <summary>
@@ -36,7 +37,10 @@ public void Execute()
 
         using (renderContext)
         {
-            _renderPhaseRegistry.Render(renderContext);
+            foreach (IRenderPhase<TRenderContext> renderPhase in _renderPhases)
+            {
+                renderPhase.Render(renderContext);
+            }
 
             // submit all pending changes before renderContext is disposed
             _gpuMemorySystem.Submit();

src/GameKit.RenderOrchestration/GameKitAppBuilderExtensions.cs

@@ -8,25 +8,12 @@ public static class GameKitAppBuilderExtensions
 {
     public static GameKitAppBuilder UseDefaultRenderManager<TRenderContext>(this GameKitAppBuilder builder) where TRenderContext: IRenderContext
     {
-        RenderPhaseRegistry<TRenderContext> renderPhaseRegistry = new();
-        builder.OnActivated((instance, _) =>
-        {
-            if (instance is IRenderPhase<TRenderContext> renderPhase)
-            {
-                renderPhaseRegistry.Register(renderPhase);
-            }
-        });
-        builder.OnDisposing((instance, _) =>
-        {
-            if (instance is IRenderPhase<TRenderContext> renderPhase)
-            {
-                renderPhaseRegistry.Unregister(renderPhase);
-            }
-        });
+        builder.AddRegistry<IRenderPhase<TRenderContext>>(
+            static (left, right) => left.Order.CompareTo(right.Order));
         builder.AddSingleton<IRenderManager>(sp => new DefaultRenderManager<TRenderContext>(
             sp.GetRequiredService<GpuMemorySystem>(),
             sp.GetRequiredService<IRenderContextProvider<TRenderContext>>(),
-            renderPhaseRegistry));
+            sp.GetRequiredService<ServiceRegistry<IRenderPhase<TRenderContext>>>()));
         return builder;
     }