Make EnsureSourceMappingExistsAsync private and call it from InitializeRepository

Co-authored-by: premun <[email protected]>

Author: Copilot

src/Microsoft.DotNet.Darc/Darc/Operations/VirtualMonoRepo/AddRepoOperation.cs

@@ -44,13 +44,6 @@ protected override async Task ExecuteInternalAsync(
         }
 
         var sourceMappingsPath = _vmrInfo.VmrPath / VmrInfo.DefaultRelativeSourceMappingsPath;
-        
-        // Ensure source mapping exists (will add if not present and stage the file)
-        await _vmrInitializer.EnsureSourceMappingExistsAsync(
-            repoName,
-            defaultRemote: null, // Will default to https://github.com/dotnet/{repoName}
-            sourceMappingsPath,
-            cancellationToken);
 
         await _vmrInitializer.InitializeRepository(
             repoName,

src/Microsoft.DotNet.Darc/DarcLib/VirtualMonoRepo/IVmrInitializer.cs

@@ -11,20 +11,6 @@ namespace Microsoft.DotNet.DarcLib.VirtualMonoRepo;
 
 public interface IVmrInitializer
 {
-    /// <summary>
-    /// Adds a new source mapping to the source-mappings.json file if it doesn't already exist.
-    /// </summary>
-    /// <param name="repoName">Name of the repository</param>
-    /// <param name="defaultRemote">Default remote URL for the repository</param>
-    /// <param name="sourceMappingsPath">Path to the source-mappings.json file</param>
-    /// <param name="cancellationToken">Cancellation token</param>
-    /// <returns>True if a new mapping was added, false if it already existed</returns>
-    Task<bool> EnsureSourceMappingExistsAsync(
-        string repoName,
-        string? defaultRemote,
-        LocalPath sourceMappingsPath,
-        CancellationToken cancellationToken);
-
     /// <summary>
     /// Initializes new repo that hasn't been synchronized into the VMR yet.
     /// </summary>

src/Microsoft.DotNet.Darc/DarcLib/VirtualMonoRepo/VmrInitializer.cs

@@ -84,7 +84,7 @@ public VmrInitializer(
         _sourceManifest = sourceManifest;
     }
 
-    public async Task<bool> EnsureSourceMappingExistsAsync(
+    private async Task<bool> EnsureSourceMappingExistsAsync(
         string repoName,
         string? defaultRemote,
         LocalPath sourceMappingsPath,
@@ -147,6 +147,9 @@ public async Task InitializeRepository(
         CodeFlowParameters codeFlowParameters,
         CancellationToken cancellationToken)
     {
+        // Ensure source mapping exists before initializing
+        await EnsureSourceMappingExistsAsync(mappingName, defaultRemote: null, sourceMappingsPath, cancellationToken);
+
         await _dependencyTracker.RefreshMetadataAsync(sourceMappingsPath);
         var mapping = _dependencyTracker.GetMapping(mappingName);