TfsWorkItemTypeValidatorTool: fix default excluded WITs handling

Author: satano

appsettings.json

@@ -181,14 +181,8 @@
             },
             "TfsWorkItemTypeValidatorTool": {
                 "Enabled": true,
-                "ExcludeWorkItemtypes": [
-                    "Code Review Request",
-                    "Code Review Response",
-                    "Feedback Request",
-                    "Feedback Response",
-                    "Shared Parameter",
-                    "Shared Steps"
-                ],
+                "ExcludeWorkItemtypes": [],
+                "ExcludeDefaultWorkItemTypes": true,
                 "SourceFieldMappings": [],
                 "FixedTargetFields": []
             }

src/MigrationTools.Clients.TfsObjectModel/ServiceCollectionExtensions.cs

@@ -27,6 +27,7 @@ public static void AddMigrationToolServicesForClientTfs_Tools(this IServiceColle
             context.AddSingleton<TfsTeamSettingsTool>().AddMigrationToolsOptions<TfsTeamSettingsToolOptions>(configuration);
             context.AddSingleton<TfsChangeSetMappingTool>().AddMigrationToolsOptions<TfsChangeSetMappingToolOptions>(configuration);
             context.AddSingleton<TfsWorkItemTypeValidatorTool>().AddMigrationToolsOptions<TfsWorkItemTypeValidatorToolOptions, TfsWorkItemTypeValidatorToolOptionsValidator>(configuration);
+            context.PostConfigure<TfsWorkItemTypeValidatorToolOptions>(options => options.Normalize());
         }
 
         public static void AddMigrationToolServicesForClientTfs_Processors(this IServiceCollection context)

src/MigrationTools.Clients.TfsObjectModel/Tools/TfsWorkItemTypeValidatorTool.cs

@@ -27,7 +27,6 @@ public TfsWorkItemTypeValidatorTool(
             : base(options, services, logger, telemetry)
         {
             _witMappingTool = witMappingTool ?? throw new ArgumentNullException(nameof(witMappingTool));
-            Options.Normalize();
         }
 
         public bool ValidateReflectedWorkItemIdField(

src/MigrationTools.Clients.TfsObjectModel/Tools/TfsWorkItemTypeValidatorToolOptions.cs

@@ -14,6 +14,14 @@ public class TfsWorkItemTypeValidatorToolOptions : ToolOptions
 
         private static readonly StringComparer _normalizedComparer = StringComparer.OrdinalIgnoreCase;
         private bool _isNormalized = false;
+        private static string[] _defaultExcludedWorkItemTypes = [
+            "Code Review Request",
+            "Code Review Response",
+            "Feedback Request",
+            "Feedback Response",
+            "Shared Parameter",
+            "Shared Steps"
+        ];
 
         /// <summary>
         /// List of work item types which will be validated. If this list is empty, all work item types will be validated.
@@ -26,6 +34,13 @@ public class TfsWorkItemTypeValidatorToolOptions : ToolOptions
         /// </summary>
         public List<string> ExcludeWorkItemtypes { get; set; } = [];
 
+        /// <summary>
+        /// If <see langword="true"/>, some work item types will be automatically added to <see cref="ExcludeWorkItemtypes"/> list.
+        /// Work item types excluded by default are: Code Review Request, Code Review Response, Feedback Request,
+        /// Feedback Response, Shared Parameter, Shared Steps.
+        /// </summary>
+        public bool ExcludeDefaultWorkItemTypes { get; set; } = true;
+
         /// <summary>
         /// Field reference name mappings. Key is work item type name, value is dictionary of mapping source filed name to
         /// target field name. Target field name can be empty string to indicate that this field will not be validated in target.
@@ -90,6 +105,17 @@ public void Normalize()
 
             IncludeWorkItemtypes ??= [];
             ExcludeWorkItemtypes ??= [];
+            if (ExcludeDefaultWorkItemTypes)
+            {
+                foreach (string defaultExcludedWit in _defaultExcludedWorkItemTypes)
+                {
+                    if (!ExcludeWorkItemtypes.Contains(defaultExcludedWit, _normalizedComparer))
+                    {
+                        ExcludeWorkItemtypes.Add(defaultExcludedWit);
+                    }
+                }
+            }
+
             FixedTargetFields = newFixedFields;
             SourceFieldMappings = newMappings;
             _isNormalized = true;

src/MigrationTools.Clients.TfsObjectModel/Tools/TfsWorkItemTypeValidatorToolOptionsValidator.cs

@@ -12,7 +12,9 @@ public ValidateOptionsResult Validate(string name, TfsWorkItemTypeValidatorToolO
             if ((includedCount > 0) && (excludedCount > 0))
             {
                 const string msg = $"'{nameof(options.IncludeWorkItemtypes)}' and '{nameof(options.ExcludeWorkItemtypes)}'"
-                    + $" cannot be set both at the same time.";
+                    + $" cannot be set both at the same time."
+                    + $" If '{nameof(options.IncludeWorkItemtypes)}' list is not empty,"
+                    + $" '{nameof(options.ExcludeDefaultWorkItemTypes)}' must be set to 'false'.";
                 return ValidateOptionsResult.Fail(msg);
             }
             return ValidateOptionsResult.Success;