refactor: Extract duplicate dotnet format logic into helper method (#1464)

* chore: Add .worktrees/ to .gitignore for parallel development

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: Extract duplicate dotnet format logic into helper method

Consolidates 4 nearly identical DotNetFormatOptions configurations into a
single reusable RunDotNetFormat helper method with parameters for
whitespaceOnly and verifyNoChanges variations.

Fixes #1460

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: thomhurst

src/ModularPipelines.Build/Modules/CodeFormattedNicelyModule.cs

@@ -44,21 +44,10 @@ public Task<SkipDecision> ShouldSkip(IPipelineContext context)
 
         try
         {
-            await context.DotNet().Format(new DotNetFormatOptions
-            {
-                Arguments = ["whitespace"],
-                WorkingDirectory = context.Git().RootDirectory,
-                VerifyNoChanges = true,
-                Severity = "info",
-            }, cancellationToken);
+            await RunDotNetFormat(context, whitespaceOnly: true, verifyNoChanges: true, cancellationToken);
 
             // The code hasn't been formatted nicely!
-            return await context.DotNet().Format(new DotNetFormatOptions
-            {
-                WorkingDirectory = context.Git().RootDirectory,
-                VerifyNoChanges = true,
-                Severity = "info",
-            }, cancellationToken);
+            return await RunDotNetFormat(context, whitespaceOnly: false, verifyNoChanges: true, cancellationToken);
         }
         catch (Exception)
         {
@@ -70,20 +59,8 @@ await context.DotNet().Format(new DotNetFormatOptions
 
             ArgumentNullException.ThrowIfNull(_githubSettings.Value.StandardToken);
 
-            await context.DotNet().Format(new DotNetFormatOptions
-            {
-                WorkingDirectory = context.Git().RootDirectory,
-                VerifyNoChanges = false,
-                Severity = "info",
-            }, cancellationToken);
-
-            await context.DotNet().Format(new DotNetFormatOptions
-            {
-                Arguments = ["whitespace"],
-                WorkingDirectory = context.Git().RootDirectory,
-                VerifyNoChanges = false,
-                Severity = "info",
-            }, cancellationToken);
+            await RunDotNetFormat(context, whitespaceOnly: false, verifyNoChanges: false, cancellationToken);
+            await RunDotNetFormat(context, whitespaceOnly: true, verifyNoChanges: false, cancellationToken);
 
             var branchTriggeringPullRequest = context.GitHub().EnvironmentVariables.HeadRef!;
 
@@ -98,4 +75,21 @@ await GitHelpers.CommitAndPush(context, branchTriggeringPullRequest, DotnetForma
             throw new Exception("Formatting code. This run will abort. Another run will trigger with the formatted code.");
         }
     }
+
+    private static Task<CommandResult> RunDotNetFormat(
+        IModuleContext context,
+        bool whitespaceOnly,
+        bool verifyNoChanges,
+        CancellationToken cancellationToken)
+    {
+        var options = new DotNetFormatOptions
+        {
+            Arguments = whitespaceOnly ? ["whitespace"] : null,
+            WorkingDirectory = context.Git().RootDirectory,
+            VerifyNoChanges = verifyNoChanges,
+            Severity = "info",
+        };
+
+        return context.DotNet().Format(options, cancellationToken);
+    }
 }