Refactor anonymous enum processor (#20)

* Reimplement AnonymousEnumProcessor as AnonymousEnumParameterProcessor
* Allow empty TypeNameExtensions.CreateTypeName(lastPropertySpan)

Author: trejjam

src/Apple.AppStoreConnect.OpenApiDocument.Generator/Extensions/TypeNameExtensions.cs

@@ -17,11 +17,15 @@ ReadOnlySpan<char> lastPropertySpan
         ].AsSpan();
 
         typePrefix.CopyTo(titleSpan);
-        lastPropertySpan.CopyTo(titleSpan[componentNameLength..]);
 
-        if (char.IsLower(titleSpan[componentNameLength]))
+        if (titleSpan.Length > typePrefix.Length)
         {
-            titleSpan[componentNameLength] = char.ToUpperInvariant(titleSpan[componentNameLength]);
+            lastPropertySpan.CopyTo(titleSpan[componentNameLength..]);
+
+            if (char.IsLower(titleSpan[componentNameLength]))
+            {
+                titleSpan[componentNameLength] = char.ToUpperInvariant(titleSpan[componentNameLength]);
+            }
         }
 
         return titleSpan;

src/Apple.AppStoreConnect.OpenApiDocument.Generator/JsonIterator.cs

@@ -15,7 +15,7 @@ public static void ProcessJson(
     {
         var path = new Stack<PathItem>();
         ReadOnlySpan<byte> lastProperty = null;
-        var context = new TransposeContext();
+        using var context = new TransposeContext();
 
         while (jsonReader.Read())
         {
@@ -119,7 +119,7 @@ TransposeContext context
         path,
         lastProperty,
         ref jsonReader, jsonWriter
-    ) || AnonymousEnumProcessor.TryProcessItem(
+    ) || AnonymousEnumParameterProcessor.TryProcessItem(
         path,
         lastProperty,
         ref jsonReader, jsonWriter,
@@ -139,12 +139,7 @@ private static bool TryWriteAdditional(
         TransposeContext context
     )
     {
-        var response = AnonymousEnumProcessor.TryWriteAdditional(
-            pathItem,
-            path,
-            jsonWriter,
-            context
-        );
+        var response = false;
 
         response = AnonymousComplexPropertyProcessor.TryWriteAdditional(
             pathItem,

src/Apple.AppStoreConnect.OpenApiDocument.Generator/Processors/AnonymousComplexPropertyProcessor.cs

@@ -201,7 +201,7 @@ TransposeContext context
             })
         )
         {
-            foreach (var renamedComponentValue in context.RenamedComponentValues)
+            foreach (var renamedComponentValue in context.NewComponents)
             {
                 jsonWriter.WritePropertyName(renamedComponentValue.Key);
                 renamedComponentValue.Value.WriteTo(jsonWriter);

src/Apple.AppStoreConnect.OpenApiDocument.Generator/Processors/AnonymousEnumProcessor.cs renamed to src/Apple.AppStoreConnect.OpenApiDocument.Generator/Processors/AnonymousEnumParameterProcessor.cs

@@ -7,7 +7,7 @@
 
 namespace Apple.AppStoreConnect.OpenApiDocument.Generator.Processors;
 
-public static class AnonymousEnumProcessor
+public static class AnonymousEnumParameterProcessor
 {
     public static bool TryProcessItem(
         IReadOnlyCollection<PathItem> path,
@@ -37,9 +37,8 @@ TransposeContext context
 
             if (
                 path.ElementAt(0).Properties.TryGetValue("name", out var name)
-                && path.ElementAt(2).Properties.TryGetValue("tags", out var tag)
                 && path.ElementAt(2).Properties.TryGetValue("operationId", out var operationId)
-                && TryReadInner(path, tag, name, operationId, ref jsonReaderClone, jsonWriter, context)
+                && TryReadInner(path, name, operationId, ref jsonReaderClone, jsonWriter, context)
             )
             {
                 var innerTokenStartIndex = jsonReaderClone.TokenStartIndex;
@@ -56,52 +55,9 @@ TransposeContext context
         return false;
     }
 
-    public static bool TryWriteAdditional(
-        PathItem? pathItem,
-        IReadOnlyCollection<PathItem> path,
-        Utf8JsonWriter jsonWriter,
-        TransposeContext context
-    )
-    {
-        if (
-            pathItem == new PathItem(JsonTokenType.StartObject, "schemas")
-            && path.SequenceEqual(new PathItem[]
-            {
-                new(JsonTokenType.StartObject, "components"),
-                new(JsonTokenType.StartObject, null),
-            })
-        )
-        {
-            foreach (var enumComponentValue in context.EnumComponentValues)
-            {
-                jsonWriter.WritePropertyName(enumComponentValue.Key);
-                jsonWriter.WriteStartObject();
-
-                jsonWriter.WritePropertyName("type"u8);
-                jsonWriter.WriteStringValue("string"u8);
-
-                jsonWriter.WritePropertyName("enum"u8);
-                jsonWriter.WriteStartArray();
-
-                foreach (var enumValue in enumComponentValue.Value)
-                {
-                    jsonWriter.WriteStringValue(enumValue);
-                }
-
-                jsonWriter.WriteEndArray();
-
-                jsonWriter.WriteEndObject();
-            }
-
-            return true;
-        }
-
-        return false;
-    }
-
     private static bool TryReadInner(
         IReadOnlyCollection<PathItem> parentPath,
-        string tag, string name, string operationId,
+        string name, string operationId,
         ref Utf8JsonReader jsonReader, Utf8JsonWriter jsonWriter,
         TransposeContext context
     )
@@ -232,10 +188,15 @@ TransposeContext context
             return false;
         }
 
-        var reference = context.GetEnumComponentReference(
+        var componentPrefix = GetComponentPrefix(
             parentPath,
-            tag, name.AsSpan(),
-            operationId.AsSpan(),
+            name.AsSpan(),
+            operationId.AsSpan()
+        );
+
+        var reference = context.GetEnumComponentReference(
+            componentPrefix,
+            ReadOnlySpan<char>.Empty,
             enumValues
         );
 
@@ -254,4 +215,131 @@ TransposeContext context
 
         return true;
     }
+
+    private static ReadOnlySpan<char> GetComponentPrefix(
+        IReadOnlyCollection<PathItem> parentPath,
+        ReadOnlySpan<char> parameterName,
+        ReadOnlySpan<char> operationId
+    )
+    {
+        var componentName = GetComponentName(parameterName);
+
+        var urlPath = parentPath.ElementAt(3).PropertyName!.ToArray().AsSpan();
+        if (urlPath[0] == '/')
+        {
+            urlPath = urlPath[1..];
+        }
+
+        var slashIndex = urlPath.IndexOf('/');
+        var urlVersion = urlPath[..slashIndex];
+
+        if (char.IsLower(urlVersion[0]))
+        {
+            urlVersion[0] = char.ToUpperInvariant(urlVersion[0]);
+        }
+
+        if (operationId.IndexOf('-') is var dashIndex and > 0)
+        {
+            var operationGroupName = operationId[..dashIndex];
+            dashIndex++;
+            operationId = operationId[dashIndex..];
+            dashIndex = operationId.IndexOf('-');
+
+            ReadOnlySpan<char> operationSuffix;
+            if (dashIndex > 0)
+            {
+                operationSuffix = operationId[(dashIndex + 1)..];
+                operationId = operationId[..dashIndex];
+            }
+            else
+            {
+                operationSuffix = operationId;
+                operationId = default;
+            }
+
+            if (
+                operationGroupName.Length > 0
+                && char.IsLower(operationGroupName[0])
+            )
+            {
+                var operationGroupNameSpan = operationGroupName.ToArray().AsSpan();
+                operationGroupNameSpan[0] = char.ToUpperInvariant(operationGroupNameSpan[0]);
+
+                operationGroupName = operationGroupNameSpan;
+            }
+
+            if (
+                operationId.Length > 0
+                && char.IsLower(operationId[0])
+            )
+            {
+                var operationIdSpan = operationId.ToArray().AsSpan();
+                operationIdSpan[0] = char.ToUpperInvariant(operationIdSpan[0]);
+
+                operationId = operationIdSpan;
+            }
+
+            var i = 0;
+            Span<char> operationName = new char[operationSuffix.Length];
+            while (operationSuffix.IndexOf('_') is var underscoreIndex and > 0)
+            {
+                var token = operationSuffix[..underscoreIndex];
+                token.CopyTo(operationName[i..]);
+
+                if (char.IsLower(operationName[i]))
+                {
+                    operationName[i] = char.ToUpperInvariant(operationName[i]);
+                }
+
+                i += token.Length;
+                operationSuffix = operationSuffix[(underscoreIndex + 1)..];
+            }
+
+            operationSuffix.CopyTo(operationName[i..]);
+            if (char.IsLower(operationName[i]))
+            {
+                operationName[i] = char.ToUpperInvariant(operationName[i]);
+            }
+
+            i += operationSuffix.Length;
+            operationName = operationName[..i];
+
+            return
+                $"{operationGroupName.ToString()}{urlVersion.ToString()}Operation{operationId.ToString()}{operationName.ToString()}{componentName}"
+                    .AsSpan();
+        }
+
+        return $"{urlVersion.ToString()}Operation{operationId.ToString()}{componentName}".AsSpan();
+    }
+
+    private static string GetComponentName(
+        ReadOnlySpan<char> parameterName
+    )
+    {
+        var openingBracket = parameterName.IndexOf('[');
+
+        if (openingBracket > 0)
+        {
+            openingBracket++;
+            parameterName = parameterName[openingBracket..];
+
+            var closingBracket = parameterName.IndexOf(']');
+
+            if (closingBracket > 0)
+            {
+                parameterName = parameterName[..closingBracket];
+            }
+        }
+
+        if (char.IsLower(parameterName[0]))
+        {
+            var upper = new char[parameterName.Length].AsSpan();
+
+            parameterName.CopyTo(upper);
+            upper[0] = char.ToUpperInvariant(parameterName[0]);
+            parameterName = upper;
+        }
+
+        return $"Parameter{parameterName.ToString()}";
+    }
 }