feat: add Action overloads to ConfigureFunctionsWebApplication for FunctionsApplicationBuilder by MO2k4 · Pull Request #3349 · Azure/azure-functions-dotnet-worker

MO2k4 · 2026-03-16T20:39:35Z

Summary

Adds Action<IFunctionsWorkerApplicationBuilder> and Action<HostBuilderContext, IFunctionsWorkerApplicationBuilder> overloads to FunctionsApplicationBuilderAspNetCoreExtensions, mirroring the existing IHostBuilder API in FunctionsHostBuilderExtensions
Both overloads delegate through FunctionsApplicationBuilder.HostBuilder — zero duplicated logic; idempotency and ASP.NET Core integration guards continue to run through the single existing implementation
Includes null guards (ArgumentNullException.ThrowIfNull) on the callback parameters

Test Plan

3 new tests added: action callback, context+action callback, idempotency (service descriptor count + callback invocation count)
All 26 tests pass (dotnet test test/extensions/Worker.Extensions.Http.AspNetCore.Tests/)
Build clean: 0 errors, 0 warnings on net6.0 and net8.0

…nctionsApplicationBuilder Adds Action<IFunctionsWorkerApplicationBuilder> and Action<HostBuilderContext, IFunctionsWorkerApplicationBuilder> overloads to FunctionsApplicationBuilderAspNetCoreExtensions, mirroring the existing IHostBuilder API. Fixes Azure#3052

kshyju

Thanks for your contribution @MO2k4 Change looks good to me. One minor suggestion about including exception information in the summary document. I noticed the existing method does not throw and handle null input gracefully. But it is okay for this new method to throw when argument is null.

kshyju · 2026-03-17T23:42:28Z

...ons/Worker.Extensions.Http.AspNetCore/src/FunctionsApplicationBuilderAspNetCoreExtensions.cs

+    /// <returns>The <see cref="FunctionsApplicationBuilder"/> for chaining.</returns>
+    public static FunctionsApplicationBuilder ConfigureFunctionsWebApplication(this FunctionsApplicationBuilder builder, Action<IFunctionsWorkerApplicationBuilder> configureWorker)
+    {
+        ArgumentNullException.ThrowIfNull(configureWorker);


Could we update the method summary to document what exception is being thrown? (Use <exception cref="ArgumentNullException">)

Thanks for the hint, i've added it

Copilot

Pull request overview

Adds ConfigureFunctionsWebApplication overloads on FunctionsApplicationBuilder that accept worker configuration callbacks, enabling scenarios (e.g., F# / .NET Aspire) that need to imperatively configure the worker pipeline without reflecting into the internal HostBuilder.

Changes:

Added Action<IFunctionsWorkerApplicationBuilder> and Action<HostBuilderContext, IFunctionsWorkerApplicationBuilder> overloads to FunctionsApplicationBuilderAspNetCoreExtensions.
Delegated both overloads through FunctionsApplicationBuilder.HostBuilder to reuse the existing IHostBuilder implementation (including idempotency/guards).
Added new unit tests covering callback invocation, context propagation, and idempotent registrations.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
extensions/Worker.Extensions.Http.AspNetCore/src/FunctionsApplicationBuilderAspNetCoreExtensions.cs	Adds the new overloads and delegates to the existing host-builder-based implementation.
test/extensions/Worker.Extensions.Http.AspNetCore.Tests/FunctionsApplicationBuilderAspNetCoreExtensionsTests.cs	Adds tests validating the new overloads and idempotent service registration behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Adds Action-based overloads for ConfigureFunctionsWebApplication on FunctionsApplicationBuilder to mirror the existing IHostBuilder API, along with tests validating callbacks and idempotent registrations.

Changes:

Added Action<IFunctionsWorkerApplicationBuilder> and Action<HostBuilderContext, IFunctionsWorkerApplicationBuilder> overloads that delegate to builder.HostBuilder.
Added tests covering action callback, context callback, and idempotency of ASP.NET Core integration service registrations.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
test/extensions/Worker.Extensions.Http.AspNetCore.Tests/FunctionsApplicationBuilderAspNetCoreExtensionsTests.cs	Adds new unit tests for the new overloads and idempotent registrations.
extensions/Worker.Extensions.Http.AspNetCore/src/FunctionsApplicationBuilderAspNetCoreExtensions.cs	Introduces new overloads with null guards that delegate to existing host-builder integration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

MO2k4 · 2026-03-19T16:54:12Z

...ker.Extensions.Http.AspNetCore.Tests/FunctionsApplicationBuilderAspNetCoreExtensionsTests.cs

+        private static void VerifyRegistrationOfCustomMiddleware(IHost host)
+        {
+            Assert.NotNull(host.Services.GetService<TestMiddleware>());
+        }


Since the middleware is directly registered before this assertion this looks like a false positive to me.

underTest.ConfigureFunctionsWebApplication(builder => builder.UseMiddleware());

...ker.Extensions.Http.AspNetCore.Tests/FunctionsApplicationBuilderAspNetCoreExtensionsTests.cs

MO2k4 · 2026-03-18T19:28:00Z

@kshyju I've updated the release notes as well, if you want to take a look

kshyju · 2026-03-19T15:21:16Z

release_notes.md

+
+### Microsoft.Azure.Functions.Worker.Extensions.Http.AspNetCore <version>
+
+- Add `Action<IApplicationBuilder>` overloads to `ConfigureFunctionsWebApplication` for `FunctionsApplicationBuilder` (#3052)


This release notes file is for the worker packages. Since you are making changes to the aspnetcore extension package, the changelog entry should go to https://github.com/Azure/azure-functions-dotnet-worker/blob/main/extensions/Worker.Extensions.Http.AspNetCore/release_notes.md file.

Ah sorry i have not seen this

kshyju · 2026-03-19T15:22:34Z

@kshyju I've updated the release notes as well, if you want to take a look

Thanks! Can you also review the feedback from GHCP review and address them?

Azure#3052)

- Strengthen idempotency test with GetServices<T>() to detect duplicate registrations post-build - Fix TestMiddleware to call next(context) instead of short-circuiting

MO2k4 · 2026-03-19T16:55:09Z

@kshyju I've checked the three comments by ghcp and adapted two, the first looks like a false positive to me, because it is handled directly before the assertion.

kshyju reviewed Mar 17, 2026

View reviewed changes

kshyju requested a review from Copilot March 17, 2026 23:44

Copilot started reviewing on behalf of kshyju March 17, 2026 23:45 View session

Copilot AI reviewed Mar 17, 2026

View reviewed changes

mr feedback: Add exception documentation for new overloads

c46a9c6

kshyju requested a review from Copilot March 18, 2026 14:24

Copilot AI reviewed Mar 18, 2026

View reviewed changes

Copilot started reviewing on behalf of kshyju March 18, 2026 14:49 View session

kshyju reviewed Mar 19, 2026

View reviewed changes

docs: add release notes for ConfigureFunctionsWebApplication overloads (

a32c44f

Azure#3052)

MO2k4 force-pushed the feature/issue-3052-configure-functions-web-application-overloads branch from 066e041 to a32c44f Compare March 19, 2026 15:31

fix: address Copilot review feedback on test quality

6973fe6

- Strengthen idempotency test with GetServices<T>() to detect duplicate registrations post-build - Fix TestMiddleware to call next(context) instead of short-circuiting

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add Action overloads to ConfigureFunctionsWebApplication for FunctionsApplicationBuilder#3349

feat: add Action overloads to ConfigureFunctionsWebApplication for FunctionsApplicationBuilder#3349
MO2k4 wants to merge 4 commits intoAzure:mainfrom
MO2k4:feature/issue-3052-configure-functions-web-application-overloads

MO2k4 commented Mar 16, 2026

Uh oh!

kshyju left a comment •

edited

Loading

Uh oh!

kshyju Mar 17, 2026

Uh oh!

MO2k4 Mar 18, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

MO2k4 Mar 19, 2026

Uh oh!

Uh oh!

Uh oh!

MO2k4 commented Mar 18, 2026

Uh oh!

kshyju Mar 19, 2026

Uh oh!

MO2k4 Mar 19, 2026

Uh oh!

kshyju commented Mar 19, 2026

Uh oh!

MO2k4 commented Mar 19, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants


		### Microsoft.Azure.Functions.Worker.Extensions.Http.AspNetCore <version>

		- Add `Action<IApplicationBuilder>` overloads to `ConfigureFunctionsWebApplication` for `FunctionsApplicationBuilder` (#3052) No newline at end of file

Conversation

MO2k4 commented Mar 16, 2026

Summary

Test Plan

Uh oh!

kshyju left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kshyju Mar 17, 2026

Choose a reason for hiding this comment

Uh oh!

MO2k4 Mar 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

MO2k4 Mar 19, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

MO2k4 commented Mar 18, 2026

Uh oh!

kshyju Mar 19, 2026

Choose a reason for hiding this comment

Uh oh!

MO2k4 Mar 19, 2026

Choose a reason for hiding this comment

Uh oh!

kshyju commented Mar 19, 2026

Uh oh!

MO2k4 commented Mar 19, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

kshyju left a comment •

edited

Loading