Add HostedFeatureLifecycleService when AddOpenFeature is called

kylejuliandev · kylejuliandev · commit 3b609ad5a657 · 2025-07-24T19:04:32.000+01:00
* Update Samples App to show how you can work with OpenFeature and
  Dependency Injection

Signed-off-by: Kyle Julian &lt;38759683+kylejuliandev@users.noreply.github.com&gt;
diff --git a/samples/AspNetCore/Program.cs b/samples/AspNetCore/Program.cs
@@ -31,7 +31,7 @@
         .WithFlagEvaluationMetadata("boolean", s => s.GetBool("boolean"))
         .Build();
 
-    featureBuilder.AddHostedFeatureLifecycle()
+    featureBuilder
         .AddHook(sp => new LoggingHook(sp.GetRequiredService<ILogger<LoggingHook>>()))
         .AddHook(_ => new MetricsHook(metricsHookOptions))
         .AddHook<TraceEnricherHook>()
diff --git a/samples/AspNetCore/Samples.AspNetCore.csproj b/samples/AspNetCore/Samples.AspNetCore.csproj
@@ -5,7 +5,6 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <ProjectReference Include="..\..\src\OpenFeature.DependencyInjection\OpenFeature.DependencyInjection.csproj" />
     <ProjectReference Include="..\..\src\OpenFeature.Hosting\OpenFeature.Hosting.csproj" />
     <ProjectReference Include="..\..\src\OpenFeature\OpenFeature.csproj" />
   </ItemGroup>
diff --git a/src/OpenFeature.Hosting/DependencyInjection/OpenFeatureServiceCollectionExtensions.cs b/src/OpenFeature.Hosting/DependencyInjection/OpenFeatureServiceCollectionExtensions.cs
@@ -3,6 +3,7 @@
 using Microsoft.Extensions.Options;
 using OpenFeature.DependencyInjection;
 using OpenFeature.DependencyInjection.Internal;
+using OpenFeature.Hosting;
 
 namespace OpenFeature;
 
@@ -54,6 +55,9 @@ public static IServiceCollection AddOpenFeature(this IServiceCollection services
         }
 
         builder.AddPolicyBasedClient();
+
+        builder.Services.AddHostedService<HostedFeatureLifecycleService>();
+
         return services;
     }
 }
diff --git a/src/OpenFeature.Hosting/OpenFeatureBuilderExtensions.cs b/src/OpenFeature.Hosting/OpenFeatureBuilderExtensions.cs
@@ -17,6 +17,7 @@ public static partial class OpenFeatureBuilderExtensions
     /// <param name="builder">The <see cref="OpenFeatureBuilder"/> instance.</param>
     /// <param name="configureOptions">An optional action to configure <see cref="FeatureLifecycleStateOptions"/>.</param>
     /// <returns>The <see cref="OpenFeatureBuilder"/> instance.</returns>
+    [Obsolete("Calling AddHostedFeatureLifecycle() is no longer necessary. OpenFeature will inject this automatically when you call AddOpenFeature().")]
     public static OpenFeatureBuilder AddHostedFeatureLifecycle(this OpenFeatureBuilder builder, Action<FeatureLifecycleStateOptions>? configureOptions = null)
     {
         if (configureOptions is not null)

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`using Microsoft.Extensions.Options;`
`4`	`4`	`using OpenFeature.DependencyInjection;`
`5`	`5`	`using OpenFeature.DependencyInjection.Internal;`
	`6`	`+using OpenFeature.Hosting;`
`6`	`7`
`7`	`8`	`namespace OpenFeature;`
`8`	`9`
`@@ -54,6 +55,9 @@ public static IServiceCollection AddOpenFeature(this IServiceCollection services`
`54`	`55`	`}`
`55`	`56`
`56`	`57`	`builder.AddPolicyBasedClient();`
	`58`	`+`
	`59`	`+ builder.Services.AddHostedService<HostedFeatureLifecycleService>();`
	`60`	`+`
`57`	`61`	`return services;`
`58`	`62`	`}`
`59`	`63`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ public static partial class OpenFeatureBuilderExtensions`
`17`	`17`	`/// <param name="builder">The <see cref="OpenFeatureBuilder"/> instance.</param>`
`18`	`18`	`/// <param name="configureOptions">An optional action to configure <see cref="FeatureLifecycleStateOptions"/>.</param>`
`19`	`19`	`/// <returns>The <see cref="OpenFeatureBuilder"/> instance.</returns>`
	`20`	`+ [Obsolete("Calling AddHostedFeatureLifecycle() is no longer necessary. OpenFeature will inject this automatically when you call AddOpenFeature().")]`
`20`	`21`	`public static OpenFeatureBuilder AddHostedFeatureLifecycle(this OpenFeatureBuilder builder, Action<FeatureLifecycleStateOptions>? configureOptions = null)`
`21`	`22`	`{`
`22`	`23`	`if (configureOptions is not null)`