Resolve conflicts

Author: vmapetr

Diff for: BuildConfigGen/EnsureUpdateModeVerifier.cs

@@ -14,6 +14,7 @@ internal class EnsureUpdateModeVerifier
         private List<string> VerifyErrors = new List<string>();
         internal Dictionary<string, string> CopiedFilesToCheck = new Dictionary<string, string>();
         internal Dictionary<string, string> RedirectedToTempl = new Dictionary<string, string>();
+        private HashSet<string> tempsToKeep = new HashSet<string>();
 
         public EnsureUpdateModeVerifier(bool verifyOnly)
         {
@@ -35,22 +36,38 @@ public IEnumerable<string> GetVerifyErrors(bool skipContentCheck)
                     string? sourceFile;
                     string procesed = "";
 
+                    string? tempToKeep = null;
                     if (RedirectedToTempl.TryGetValue(r.Key, out sourceFile))
                     {
-                        procesed = "(processed) ";
+                        procesed = $"(processed temp={sourceFile}) ";
+                        tempToKeep = sourceFile;
                     }
                     else
                     {
                         sourceFile = r.Value;
                     }
 
-                    if (Helpers.FilesEqual(sourceFile, r.Key))
+                    FileInfo fi = new FileInfo(r.Key);
+
+                    if (fi.Name.Equals("resources.resjson", StringComparison.OrdinalIgnoreCase))
                     {
-                        // if overwrite and content match, everything is good!  Verification passed.
+                        // resources.resjson is generated by make.js and does not need to be verified
+                        // it can differ between configs if configs have different inputs (causes verifier to fail);
                     }
                     else
                     {
-                        yield return $"Content doesn't match {r.Value} {procesed}to {r.Key} (overwrite=true).  Dest file doesn't match {procesed}source.";
+                        if (Helpers.FilesEqual(sourceFile, r.Key))
+                        {
+                            // if overwrite and content match, everything is good!  Verification passed.
+                        }
+                        else
+                        {
+                            if (tempToKeep != null)
+                            {
+                                this.tempsToKeep.Add(tempToKeep!);
+                            }
+                            yield return $"Content doesn't match {r.Value} {procesed}to {r.Key} (overwrite=true).  Dest file doesn't match source.";
+                        }
                     }
                 }
             }
@@ -62,9 +79,12 @@ public void CleanupTempFiles()
             foreach (var f in RedirectedToTempl.Values)
             {
                 count++;
-                if (File.Exists(f))
+                if (!tempsToKeep.Contains(f))
                 {
-                    File.Delete(f);
+                    if (File.Exists(f))
+                    {
+                        File.Delete(f);
+                    }
                 }
             }
 
@@ -137,10 +157,12 @@ internal void WriteAllText(string path, string contents, bool suppressValidation
                 {
                     string? tempFilePath;
 
-                    if (!RedirectedToTempl.TryGetValue(path, out tempFilePath))
+                    string normalizedPath = NormalizeFile(path);
+
+                    if (!RedirectedToTempl.TryGetValue(normalizedPath, out tempFilePath))
                     {
                         tempFilePath = Path.Combine(Path.GetTempPath(), Path.GetRandomFileName());
-                        RedirectedToTempl.Add(path, tempFilePath);
+                        RedirectedToTempl.Add(normalizedPath, tempFilePath);
                     }
 
                     //Console.WriteLine($"writing to tempFilePath={tempFilePath}");
@@ -240,11 +262,21 @@ private string ResolveFile(string filePath)
             filePath = NormalizeFile(filePath);
 
             string? sourceFile = null, tempFile = null;
-            if (CopiedFilesToCheck.TryGetValue(filePath, out sourceFile))
+
+            if (RedirectedToTempl.TryGetValue(filePath, out tempFile))
+            {
+                // We'll get here if we're reading a file that was written to in WriteAllText
+            }
+            else
             {
-                if (RedirectedToTempl.TryGetValue(sourceFile, out tempFile))
+                if (CopiedFilesToCheck.TryGetValue(filePath, out sourceFile))
                 {
-                    // do nothing
+                    // We'll get here if we're reading a file that was copied
+
+                    if (RedirectedToTempl.TryGetValue(sourceFile, out tempFile))
+                    {
+                        // We'll get here if we're reading a file that was copied and then written to in WriteAllText
+                    }
                 }
             }
 

Diff for: BuildConfigGen/Program.cs

@@ -51,7 +51,8 @@ public record ConfigRecord(string name, string constMappingKey, bool isDefault,
             public static readonly ConfigRecord Node20_229_13 = new ConfigRecord(name: nameof(Node20_229_13), constMappingKey: "Node20_229_13", isDefault: false, isNode: true, nodePackageVersion: "^20.11.0", isWif: false, nodeHandler: "Node20_1", preprocessorVariableName: "NODE20", enableBuildConfigOverrides: true, deprecated: false, shouldUpdateTypescript: true, overriddenDirectoryName: "Node20", writeNpmrc: true);
             public static readonly ConfigRecord Node20_229_14 = new ConfigRecord(name: nameof(Node20_229_14), constMappingKey: "Node20_229_14", isDefault: false, isNode: true, nodePackageVersion: "^20.3.1", isWif: false, nodeHandler: "Node20_1", preprocessorVariableName: "NODE20", enableBuildConfigOverrides: true, deprecated: false, shouldUpdateTypescript: true, overriddenDirectoryName: "Node20", writeNpmrc: true);
             public static readonly ConfigRecord WorkloadIdentityFederation = new ConfigRecord(name: nameof(WorkloadIdentityFederation), constMappingKey: "WorkloadIdentityFederation", isDefault: false, isNode: true, nodePackageVersion: "^16.11.39", isWif: true, nodeHandler: "Node16", preprocessorVariableName: "WORKLOADIDENTITYFEDERATION", enableBuildConfigOverrides: true, deprecated: false, shouldUpdateTypescript: false, writeNpmrc: false);
-            public static ConfigRecord[] Configs = { Default, Node16, Node16_225, Node20, Node20_228, Node20_229_1, Node20_229_2, Node20_229_3, Node20_229_4, Node20_229_5, Node20_229_6, Node20_229_7, Node20_229_8, Node20_229_9, Node20_229_10, Node20_229_11, Node20_229_12, Node20_229_13, Node20_229_14, WorkloadIdentityFederation };
+            public static readonly ConfigRecord wif_242 = new ConfigRecord(name: nameof(wif_242), constMappingKey: "wif_242", isDefault: false, isNode: true, nodePackageVersion: "^20.3.1", isWif: true, nodeHandler: "Node20_1", preprocessorVariableName: "WIF", enableBuildConfigOverrides: true, deprecated: false, shouldUpdateTypescript: true, overriddenDirectoryName: "Wif", writeNpmrc: true);
+            public static ConfigRecord[] Configs = { Default, Node16, Node16_225, Node20, Node20_228, Node20_229_1, Node20_229_2, Node20_229_3, Node20_229_4, Node20_229_5, Node20_229_6, Node20_229_7, Node20_229_8, Node20_229_9, Node20_229_10, Node20_229_11, Node20_229_12, Node20_229_13, Node20_229_14, WorkloadIdentityFederation, wif_242 };
         }
 
         static List<string> notSyncronizedDependencies = [];
@@ -98,7 +99,6 @@ private static void MainInner(string? task, string? configs, int? currentSprint,
             else
             {
                 NotNullOrThrow(task, "Task is required");
-                NotNullOrThrow(configs, "Configs is required");
             }
 
             string currentDir = Environment.CurrentDirectory;
@@ -142,7 +142,30 @@ private static void MainInner(string? task, string? configs, int? currentSprint,
                 // 3. Ideally default windows exception will occur and errors reported to WER/watson.  I'm not sure this is happening, perhaps DragonFruit is handling the exception
                 foreach (var t in task!.Split(',', '|'))
                 {
-                    MainUpdateTask(t, configs!, writeUpdates, currentSprint, debugConfGen);
+                    // If config weren't passed, constract specific config for each task, otherwise we might have problems if first task has 2 configs, but the other 1,0 or diffrenet ones
+                    if (configs == null)
+                    {
+                        var tasks = MakeOptionsReader.ReadMakeOptions(gitRootPath);
+                        if (tasks.ContainsKey(t))
+                        {
+                            var taskMakeOptions = tasks[t];
+                            var taskConfigs = string.Join('|', taskMakeOptions.Configs);
+                            MainUpdateTask(t, taskConfigs!, writeUpdates, currentSprint, debugConfGen);
+                        }
+                        else
+                        {
+                            Console.WriteLine($"Config was not found for the task: {t}, skipping");
+                        }
+                    }
+                    else
+                    {
+                        // If configs was passed as arguments, just execute it
+                        MainUpdateTask(t, configs!, writeUpdates, currentSprint, debugConfGen);
+                    }
+
+                    if (configs != null)
+                    {
+                    }
                 }
             }
 
@@ -421,10 +444,11 @@ private static void MainUpdateTaskInner(
 
                     HandlePreprocessingInTarget(taskOutput, config, validateAndWriteChanges: true, out _);
 
+                    WriteWIFInputTaskJson(taskOutput, config, "task.json", isLoc: false);
+                    WriteWIFInputTaskJson(taskOutput, config, "task.loc.json", isLoc: true);
                     WriteTaskJson(taskOutput, configTaskVersionMapping, config, "task.json");
                     WriteTaskJson(taskOutput, configTaskVersionMapping, config, "task.loc.json");
                 }
-
                 WriteInputTaskJson(taskTargetPath, configTaskVersionMapping, "task.json");
                 WriteInputTaskJson(taskTargetPath, configTaskVersionMapping, "task.loc.json");
 
@@ -654,6 +678,24 @@ private static void WriteTaskJson(string taskPath, Dictionary<Config.ConfigRecor
             ensureUpdateModeVerifier!.WriteAllText(outputTaskPath, outputTaskNode.ToJsonString(jso), suppressValidationErrorIfTargetPathDoesntExist: false);
         }
 
+        private static void WriteWIFInputTaskJson(string taskPath, Config.ConfigRecord config, string fileName, bool isLoc)
+        {
+            if (!config.isWif)
+            {
+                return;
+            }
+
+            string taskJsonOverridePath = Path.Combine(taskPath, isLoc ? "taskJsonOverride.loc.json" : "taskJsonOverride.json");
+            JsonNode inputTaskNode = JsonNode.Parse(ensureUpdateModeVerifier!.FileReadAllText(taskJsonOverridePath))!;
+            var clonedArray = JsonNode.Parse(inputTaskNode["inputs"]!.ToJsonString())!.AsArray();
+
+            string outputTaskPath = Path.Combine(taskPath, fileName);
+            JsonNode outputTaskNode = JsonNode.Parse(ensureUpdateModeVerifier!.FileReadAllText(outputTaskPath))!;
+            outputTaskNode["inputs"] = clonedArray;
+
+            ensureUpdateModeVerifier!.WriteAllText(outputTaskPath, outputTaskNode.ToJsonString(jso), suppressValidationErrorIfTargetPathDoesntExist: false);
+        }
+
         private static void WriteNodePackageJson(string taskOutputNode, string nodeVersion, bool shouldUpdateTypescript)
         {
             string outputNodePackagePath = Path.Combine(taskOutputNode, "package.json");

Diff for: BuildConfigGen/Properties/launchSettings.json

@@ -2,7 +2,7 @@
   "profiles": {
     "BuildConfigGen": {
       "commandName": "Project",
-      "commandLineArgs": "--write-updates --configs \"Node20\" --task MavenV3"
+      "commandLineArgs": " --task PipAuthenticateV1   "
     }
   }
 }