Fix fetching eng/common files from the VMR (#4855)

Resolves #4849

Author: premun

src/Microsoft.DotNet.Darc/DarcLib/GitHubClient.cs

@@ -1099,61 +1099,6 @@ private async Task<TreeResponse> GetTreeForPathAsync(string owner, string repo,
         }
     }
 
-    private async Task GetCommitMapForPathAsync(
-        string repoUri,
-        string branch,
-        string assetsProducedInCommit,
-        List<GitFile> files,
-        string pullRequestBaseBranch,
-        string path = "eng/common/")
-    {
-        if (path.EndsWith("/"))
-        {
-            path = path.Substring(0, path.Length - 1);
-        }
-
-        _logger.LogInformation(
-            $"Getting the contents of file/files in '{path}' of repo '{repoUri}' at commit '{assetsProducedInCommit}'");
-
-        (string owner, string repo) = ParseRepoUri(repoUri);
-        List<GitHubContent>? contents;
-
-        using (HttpResponseMessage response = await ExecuteRemoteGitCommandAsync(
-                   HttpMethod.Get,
-                   $"https://github.com/{owner}/{repo}",
-                   $"repos/{owner}/{repo}/contents/{path}?ref={assetsProducedInCommit}",
-                   _logger))
-        {
-            contents = JsonConvert.DeserializeObject<List<GitHubContent>>(await response.Content.ReadAsStringAsync());
-        }
-
-        foreach (GitHubContent content in contents!)
-        {
-            if (content.Type == GitHubContentType.File)
-            {
-                if (!DependencyFileManager.DependencyFiles.Contains(content.Path))
-                {
-                    string fileContent = await GetFileContentsAsync(owner, repo, content.Path);
-                    var gitCommit = new GitFile(content.Path, fileContent);
-                    files.Add(gitCommit);
-                }
-            }
-            else
-            {
-                await GetCommitMapForPathAsync(
-                    repoUri,
-                    branch,
-                    assetsProducedInCommit,
-                    files,
-                    pullRequestBaseBranch,
-                    content.Path);
-            }
-        }
-
-        _logger.LogInformation(
-            $"Getting the contents of file/files in '{path}' of repo '{repoUri}' at commit '{assetsProducedInCommit}' succeeded!");
-    }
-
     /// <summary>
     ///     Parse out the owner and repo from a repository url
     /// </summary>

src/Microsoft.DotNet.Darc/DarcLib/Local.cs

@@ -102,7 +102,14 @@ public async Task UpdateDependenciesAsync(List<DependencyDetail> dependencies, I
                 // so we need to strip the src/arcade prefix from the file paths.
                 if (repoIsVmr)
                 {
-                    engCommonFiles = engCommonFiles.Select(f => new GitFile(f.FilePath.Replace("src/arcade/", null), f.Content)).ToList();
+                    engCommonFiles = engCommonFiles
+                        .Select(f => new GitFile(
+                            f.FilePath.Replace("src/arcade/", null),
+                            f.Content,
+                            f.ContentEncoding,
+                            f.Mode,
+                            f.Operation))
+                        .ToList();
                 }
 
                 filesToUpdate.AddRange(engCommonFiles);