Normalize directory separators in WorkspaceManagement

microsoft · Jul 3, 2024 · 71619b8 · 71619b8
1 parent c87c7cc
commit 71619b8
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 10 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -27,6 +27,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Emit `[GeneratedCode]` attribute for C# types. [#4907](https://github.com/microsoft/kiota/issues/4907)
 - Fixes error property disambiguation when the property has the same name as class [#4893](https://github.com/microsoft/kiota/issues/)
 - Fixes missing imports for `UntypedNode` for method parameter and return value scenarios. [#4925](https://github.com/microsoft/kiota/issues/4925)
+- Normalize path separators in lock and workspace files to use `/` as the path separator. [#4228](https://github.com/microsoft/kiota/issues/4228)
 
 ## [1.15.0] - 2024-06-06
 

diff --git a/src/Kiota.Builder/Configuration/GenerationConfiguration.cs b/src/Kiota.Builder/Configuration/GenerationConfiguration.cs
@@ -197,7 +197,7 @@ private string NormalizeDescriptionLocation(string targetDirectory)
             !OpenAPIFilePath.StartsWith("http", StringComparison.OrdinalIgnoreCase) &&
             Path.IsPathRooted(OpenAPIFilePath) &&
             Path.GetFullPath(OpenAPIFilePath).StartsWith(Path.GetFullPath(targetDirectory), StringComparison.Ordinal))
-            return "./" + Path.GetRelativePath(targetDirectory, OpenAPIFilePath);
+            return "./" + Path.GetRelativePath(targetDirectory, OpenAPIFilePath).NormalizePathSeparators();
         return OpenAPIFilePath;
     }
     public bool IsPluginConfiguration => PluginTypes.Count != 0;

diff --git a/src/Kiota.Builder/Extensions/StringExtensions.cs b/src/Kiota.Builder/Extensions/StringExtensions.cs
@@ -299,4 +299,10 @@ public static string GetFileExtension(this string path)
         if (string.IsNullOrEmpty(path)) return string.Empty;
         return Path.GetExtension(path).TrimStart('.');
     }
+    public static string NormalizePathSeparators(this string path)
+    {
+        if (string.IsNullOrEmpty(path)) return string.Empty;
+        if (Path.DirectorySeparatorChar != '/') return path.Replace(Path.DirectorySeparatorChar, '/');
+        return path;
+    }
 }
diff --git a/src/Kiota.Builder/Lock/LockManagementService.cs b/src/Kiota.Builder/Lock/LockManagementService.cs
@@ -7,6 +7,7 @@
 using System.Text.Json;
 using System.Threading;
 using System.Threading.Tasks;
+using Kiota.Builder.Extensions;
 
 namespace Kiota.Builder.Lock;
 
@@ -83,12 +84,7 @@ private static string GetRelativeDescriptionPath(string descriptionPath, string
     {
         if (IsDescriptionLocal(descriptionPath) &&
             Path.GetDirectoryName(lockFilePath) is string lockFileDirectoryPath)
-        {
-            var relativePath = Path.GetRelativePath(lockFileDirectoryPath, descriptionPath);
-            if (Path.DirectorySeparatorChar != '/')
-                relativePath = relativePath.Replace(Path.DirectorySeparatorChar, '/');
-            return relativePath;
-        }
+            return Path.GetRelativePath(lockFileDirectoryPath, descriptionPath).NormalizePathSeparators();
         return descriptionPath;
     }
     /// <inheritdoc/>

diff --git a/src/Kiota.Builder/WorkspaceManagement/BaseApiConsumerConfiguration.cs b/src/Kiota.Builder/WorkspaceManagement/BaseApiConsumerConfiguration.cs
@@ -3,6 +3,7 @@
 using System.IO;
 using System.Linq;
 using Kiota.Builder.Configuration;
+using Kiota.Builder.Extensions;
 using Microsoft.OpenApi.ApiManifest;
 
 namespace Kiota.Builder.WorkspaceManagement;
@@ -41,12 +42,12 @@ private protected BaseApiConsumerConfiguration(GenerationConfiguration config)
     public void NormalizeOutputPath(string targetDirectory)
     {
         if (Path.IsPathRooted(OutputPath))
-            OutputPath = "./" + Path.GetRelativePath(targetDirectory, OutputPath);
+            OutputPath = "./" + Path.GetRelativePath(targetDirectory, OutputPath).NormalizePathSeparators();
     }
     public void NormalizeDescriptionLocation(string targetDirectory)
     {
         if (Path.IsPathRooted(DescriptionLocation) && Path.GetFullPath(DescriptionLocation).StartsWith(Path.GetFullPath(targetDirectory), StringComparison.Ordinal) && !DescriptionLocation.StartsWith("http", StringComparison.OrdinalIgnoreCase))
-            DescriptionLocation = "./" + Path.GetRelativePath(targetDirectory, DescriptionLocation);
+            DescriptionLocation = "./" + Path.GetRelativePath(targetDirectory, DescriptionLocation).NormalizePathSeparators();
     }
     protected void CloneBase(BaseApiConsumerConfiguration target)
     {

diff --git a/src/Kiota.Builder/WorkspaceManagement/WorkspaceManagementService.cs b/src/Kiota.Builder/WorkspaceManagement/WorkspaceManagementService.cs
@@ -331,7 +331,7 @@ public async Task<IEnumerable<string>> MigrateFromLockFileAsync(string clientNam
         }
         var generationConfiguration = new GenerationConfiguration();
         lockInfo.UpdateGenerationConfigurationFromLock(generationConfiguration);
-        generationConfiguration.OutputPath = "./" + Path.GetRelativePath(WorkingDirectory, lockFileDirectory);
+        generationConfiguration.OutputPath = "./" + Path.GetRelativePath(WorkingDirectory, lockFileDirectory).NormalizePathSeparators();
         if (!string.IsNullOrEmpty(clientName))
         {
             generationConfiguration.ClientClassName = clientName;