Merge pull request #1379 from microsoft/feature/clean-output

- adds a clean ouput argument

CHANGELOG.md

@@ -9,6 +9,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
+- Adds a `--clean-output` argument to clean the target directory before generation #1357
+
 ### Changed
 
 ## [0.0.18] - 2022-03-14

docs/using.md

@@ -14,6 +14,7 @@ Kiota accepts the following parameters during the generation.
 ```shell
 kiota [--backing-store | -b]
       [--class-name | -c]
+      [--clean-output | -co]
       [--deserializer | -ds]
       [--language | -l]
       [--loglevel | -ll]
@@ -37,6 +38,14 @@ kiota --backing-store
 
 The class name to use for the core client class. Defaults to `ApiClient`.
 
+### `--clean-output (-co)`
+
+Delete the output directory before generating the client. Defaults to false.
+
+```shell
+kiota --clean-output
+```
+
 #### Accepted values
 
 The provided name MUST be a valid class name for the target language.

src/Kiota.Builder/GenerationConfiguration.cs

@@ -28,5 +28,6 @@ public class GenerationConfiguration {
         private static readonly HashSet<GenerationLanguage> BarreledLanguagesWithConstantFileName = new () {
             //TODO: add typescript once we have a barrel writer for it
         };
+        public bool CleanOutput { get; set;}
     }
 }

src/Kiota.Builder/KiotaBuilder.cs

@@ -29,6 +29,14 @@ public KiotaBuilder(ILogger<KiotaBuilder> logger, GenerationConfiguration config
         this.logger = logger;
         this.config = config;
     }
+    private void CleanOutputDirectory()
+    {
+        if(config.CleanOutput && Directory.Exists(config.OutputPath))
+        {
+            logger.LogInformation("Cleaning output directory {path}", config.OutputPath);
+            Directory.Delete(config.OutputPath, true);
+        }
+    }
 
     public async Task GenerateSDK(CancellationToken cancellationToken)
     {
@@ -37,6 +45,7 @@ public async Task GenerateSDK(CancellationToken cancellationToken)
         string inputPath = config.OpenAPIFilePath;
 
         try {
+            CleanOutputDirectory();
             // doing this verification at the begining to give immediate feedback to the user
             Directory.CreateDirectory(config.OutputPath);
         } catch (Exception ex) {

src/kiota/KiotaHost.cs

@@ -45,6 +45,9 @@ public RootCommand GetRootCommand()
             var deserializerOption = new Option<List<string>>("--deserializer", () => new List<string> {"Microsoft.Kiota.Serialization.Json.JsonParseNodeFactory"}, "The fully qualified class names for deserializers. Accepts multiple values.");
             deserializerOption.AddAlias("--ds");
 
+            var cleanOutputOption = new Option<bool>("--clean-output", () => false, "Removes all files from the output directory before generating the code files.");
+            cleanOutputOption.AddAlias("-co");
+
             var command = new RootCommand {
                 outputOption,
                 languageOption,
@@ -55,15 +58,16 @@ public RootCommand GetRootCommand()
                 namespaceOption,
                 serializerOption,
                 deserializerOption,
+                cleanOutputOption,
             };
-            command.SetHandler<string, GenerationLanguage, string, bool, string, LogLevel, string, List<string>, List<string>, CancellationToken>(HandleCommandCall, outputOption, languageOption, descriptionOption, backingStoreOption, classOption, logLevelOption, namespaceOption, serializerOption, deserializerOption);
+            command.SetHandler<string, GenerationLanguage, string, bool, string, LogLevel, string, List<string>, List<string>, bool, CancellationToken>(HandleCommandCall, outputOption, languageOption, descriptionOption, backingStoreOption, classOption, logLevelOption, namespaceOption, serializerOption, deserializerOption, cleanOutputOption);
             return command;
         }
         private void AssignIfNotNullOrEmpty(string input, Action<GenerationConfiguration, string> assignment) {
             if (!string.IsNullOrEmpty(input))
                 assignment.Invoke(Configuration, input);
         }
-        private async Task<int> HandleCommandCall(string output, GenerationLanguage language, string openapi, bool backingstore, string classname, LogLevel loglevel, string namespacename, List<string> serializer, List<string> deserializer, CancellationToken cancellationToken) {
+        private async Task<int> HandleCommandCall(string output, GenerationLanguage language, string openapi, bool backingstore, string classname, LogLevel loglevel, string namespacename, List<string> serializer, List<string> deserializer, bool cleanOutput, CancellationToken cancellationToken) {
             AssignIfNotNullOrEmpty(output, (c, s) => c.OutputPath = s);
             AssignIfNotNullOrEmpty(openapi, (c, s) => c.OpenAPIFilePath = s);
             AssignIfNotNullOrEmpty(classname, (c, s) => c.ClientClassName = s);
@@ -81,6 +85,7 @@ private async Task<int> HandleCommandCall(string output, GenerationLanguage lang
 
             Configuration.OpenAPIFilePath = GetAbsolutePath(Configuration.OpenAPIFilePath);
             Configuration.OutputPath = GetAbsolutePath(Configuration.OutputPath);
+            Configuration.CleanOutput = cleanOutput;
 
             var logger = LoggerFactory.Create((builder) => {
                 builder