fix: Remove redundant directory creation and use HttpClient from DownloadOptions (#1669)

- FileSystemModuleEstimatedTimeProvider: Remove duplicate directoryInfo.Create()
  call that was redundant since directory is already created in the if block above
- Downloader: Pass HttpClient from DownloadOptions to HttpOptions so that
  custom HttpClient instances are actually used during downloads

Fixes #1588, #1589

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

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

Author: thomhurst

src/ModularPipelines/Context/Downloader.cs

@@ -60,6 +60,7 @@ public async Task<HttpResponseMessage> DownloadResponseAsync(DownloadOptions opt
         var response = await _http.SendAsync(new HttpOptions(request)
         {
             LoggingType = options.LoggingType,
+            HttpClient = options.HttpClient,
         }, cancellationToken).ConfigureAwait(false);
 
         return response.EnsureSuccessStatusCode();

src/ModularPipelines/Engine/FileSystemModuleEstimatedTimeProvider.cs

@@ -30,8 +30,6 @@ public async Task<IEnumerable<SubModuleEstimation>> GetSubModuleEstimatedTimesAs
             directoryInfo.Create();
         }
 
-        directoryInfo.Create();
-
         var paths = directoryInfo
             .EnumerateFiles("*.txt", SearchOption.TopDirectoryOnly)
             .Where(x => x.Name.StartsWith($"Mod-{moduleType.FullName}"))