Merge branch release/2.0.0

Author: irvinesunday

src/Microsoft.OpenApi.Hidi/OpenApiService.cs

@@ -60,8 +60,10 @@ public static async Task TransformOpenApiDocument(HidiOptions options, ILogger l
                 if (options.Output == null)
                 {
 #pragma warning disable CA1308 // Normalize strings to uppercase
-                    var inputExtension = string.Concat(".", options.OpenApiFormat?.GetDisplayName().ToLowerInvariant())
-                        ?? GetInputPathExtension(options.OpenApi, options.Csdl);
+                    var extension = options.OpenApiFormat?.GetDisplayName().ToLowerInvariant();
+                    var inputExtension = !string.IsNullOrEmpty(extension) ? string.Concat(".", extension) 
+                        : GetInputPathExtension(options.OpenApi, options.Csdl);
+
 #pragma warning restore CA1308 // Normalize strings to uppercase
                     options.Output = new($"./output{inputExtension}");
                 };

src/Microsoft.OpenApi/Interfaces/IEffective.cs

@@ -26,8 +26,9 @@ internal interface IOpenApiVersionService
         /// </summary>
         /// <typeparam name="T">Type of element to load</typeparam>
         /// <param name="node">document fragment node</param>
+        /// <param name="doc">A host document instance.</param>
         /// <returns>Instance of OpenAPIElement</returns>
-        T LoadElement<T>(ParseNode node) where T : IOpenApiElement;
+        T LoadElement<T>(ParseNode node, OpenApiDocument doc = null) where T : IOpenApiElement;
 
         /// <summary>
         /// Converts a generic RootNode instance into a strongly typed OpenApiDocument

src/Microsoft.OpenApi/Interfaces/IOpenApiVersionService.cs

@@ -12,7 +12,7 @@ namespace Microsoft.OpenApi.Models
     /// <summary>
     /// Callback Object: A map of possible out-of band callbacks related to the parent operation.
     /// </summary>
-    public class OpenApiCallback : IOpenApiReferenceable, IOpenApiExtensible, IEffective<OpenApiCallback>
+    public class OpenApiCallback : IOpenApiReferenceable, IOpenApiExtensible
     {
         /// <summary>
         /// A Path Item Object used to define a callback request and expected responses.
@@ -61,10 +61,7 @@ public void AddPathItem(RuntimeExpression expression, OpenApiPathItem pathItem)
             Utils.CheckArgumentNull(expression);
             Utils.CheckArgumentNull(pathItem);
 
-            if (PathItems == null)
-            {
-                PathItems = new();
-            }
+            PathItems ??= new();
 
             PathItems.Add(expression, pathItem);
         }
@@ -102,40 +99,9 @@ private void SerializeInternal(IOpenApiWriter writer,
             Utils.CheckArgumentNull(writer);
 
             var target = this;
-            
-            if (Reference != null)
-            {
-                if (!writer.GetSettings().ShouldInlineReference(Reference))
-                {
-                    callback(writer, Reference);
-                    return;
-                }
-                else
-                {
-                    target = GetEffective(Reference.HostDocument);
-                }
-            }
-            
             action(writer, target);
         }
 
-        /// <summary>
-        /// Returns an effective OpenApiCallback object based on the presence of a $ref
-        /// </summary>
-        /// <param name="doc">The host OpenApiDocument that contains the reference.</param>
-        /// <returns>OpenApiCallback</returns>
-        public OpenApiCallback GetEffective(OpenApiDocument doc)
-        {
-            if (Reference != null)
-            {
-                return doc.ResolveReferenceTo<OpenApiCallback>(Reference);
-            }
-            else
-            {
-                return this;
-            }
-        }
-
         /// <summary>
         /// Serialize to OpenAPI V31 document without using reference.
         /// </summary>

src/Microsoft.OpenApi/Models/OpenApiCallback.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation. All rights reserved.
+// Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT license.
 
 using System;
@@ -459,6 +459,16 @@ public IEnumerable<OpenApiError> ResolveReferences()
             return resolver.Errors;
         }
 
+        /// <summary>
+        /// Walks the OpenApiDocument and sets the host document for all referenceable objects
+        /// </summary>
+        public void SetHostDocument()
+        {
+            var resolver = new HostDocumentResolver(this);
+            var walker = new OpenApiWalker(resolver);
+            walker.Walk(this);
+        }
+
         /// <summary>
         /// Load the referenced <see cref="IOpenApiReferenceable"/> object from a <see cref="OpenApiReference"/> object
         /// </summary>
@@ -682,6 +692,12 @@ public JsonSchema FindSubschema(Json.Pointer.JsonPointer pointer, EvaluationOpti
         {
             throw new NotImplementedException();
         }
+
+        internal JsonSchema ResolveJsonSchemaReference(Uri reference)
+        {
+            var referencePath = string.Concat("https://registry", reference.OriginalString.Split('#').Last());
+            return (JsonSchema)SchemaRegistry.Global.Get(new Uri(referencePath));
+        }
     }
 
     internal class FindSchemaReferences : OpenApiVisitorBase

src/Microsoft.OpenApi/Models/OpenApiDocument.cs

@@ -13,7 +13,7 @@ namespace Microsoft.OpenApi.Models
     /// <summary>
     /// Example Object.
     /// </summary>
-    public class OpenApiExample : IOpenApiReferenceable, IOpenApiExtensible, IEffective<OpenApiExample>
+    public class OpenApiExample : IOpenApiReferenceable, IOpenApiExtensible
     {
         /// <summary>
         /// Short description for the example.
@@ -101,39 +101,9 @@ internal virtual void SerializeInternal(IOpenApiWriter writer, Action<IOpenApiWr
             Utils.CheckArgumentNull(writer);
 
             var target = this;
-
-            if (Reference != null)
-            {
-                if (!writer.GetSettings().ShouldInlineReference(Reference))
-                {
-                    callback(writer, Reference);
-                    return;
-                }
-                else
-                {
-                    target = GetEffective(Reference.HostDocument);
-                }
-            }
             action(writer, target);
         }
 
-        /// <summary>
-        /// Returns an effective OpenApiExample object based on the presence of a $ref
-        /// </summary>
-        /// <param name="doc">The host OpenApiDocument that contains the reference.</param>
-        /// <returns>OpenApiExample</returns>
-        public OpenApiExample GetEffective(OpenApiDocument doc)
-        {
-            if (Reference != null)
-            {
-                return doc.ResolveReferenceTo<OpenApiExample>(this.Reference);
-            }
-            else
-            {
-                return this;
-            }
-        }
-
         /// <summary>
         /// Serialize to OpenAPI V31 example without using reference.
         /// </summary>

src/Microsoft.OpenApi/Models/OpenApiExample.cs

@@ -16,7 +16,7 @@ namespace Microsoft.OpenApi.Models
     /// Header Object.
     /// The Header Object follows the structure of the Parameter Object.
     /// </summary>
-    public class OpenApiHeader : IOpenApiReferenceable, IOpenApiExtensible, IEffective<OpenApiHeader>
+    public class OpenApiHeader : IOpenApiReferenceable, IOpenApiExtensible
     {
         private JsonSchema _schema;
 
@@ -145,40 +145,9 @@ private void SerializeInternal(IOpenApiWriter writer, Action<IOpenApiWriter, IOp
             Utils.CheckArgumentNull(writer);;
 
             var target = this;
-
-            if (Reference != null)
-            {
-                if (!writer.GetSettings().ShouldInlineReference(Reference))
-                {
-                    callback(writer, Reference);
-                    return;
-                }
-                else
-                {
-                    target = GetEffective(Reference.HostDocument);
-                }
-            }
-
             action(writer, target);
         }
 
-        /// <summary>
-        /// Returns an effective OpenApiHeader object based on the presence of a $ref
-        /// </summary>
-        /// <param name="doc">The host OpenApiDocument that contains the reference.</param>
-        /// <returns>OpenApiHeader</returns>
-        public OpenApiHeader GetEffective(OpenApiDocument doc)
-        {
-            if (Reference != null)
-            {
-                return doc.ResolveReferenceTo<OpenApiHeader>(Reference);
-            }
-            else
-            {
-                return this;
-            }
-        }
-
         /// <summary>
         /// Serialize to OpenAPI V31 document without using reference.
         /// </summary>
@@ -244,24 +213,11 @@ internal virtual void SerializeInternalWithoutReference(IOpenApiWriter writer, O
         /// <summary>
         /// Serialize <see cref="OpenApiHeader"/> to Open Api v2.0
         /// </summary>
-        public void SerializeAsV2(IOpenApiWriter writer)
+        public virtual void SerializeAsV2(IOpenApiWriter writer)
         {
-            Utils.CheckArgumentNull(writer);;
+            Utils.CheckArgumentNull(writer);
 
             var target = this;
-
-            if (Reference != null)
-            {
-                if (!writer.GetSettings().ShouldInlineReference(Reference))
-                {
-                    Reference.SerializeAsV2(writer);
-                    return;
-                }
-                else
-                {
-                    target = GetEffective(Reference.HostDocument);
-                }
-            }
             target.SerializeAsV2WithoutReference(writer);
         }
 

src/Microsoft.OpenApi/Models/OpenApiHeader.cs

@@ -11,7 +11,7 @@ namespace Microsoft.OpenApi.Models
     /// <summary>
     /// Link Object.
     /// </summary>
-    public class OpenApiLink : IOpenApiReferenceable, IOpenApiExtensible, IEffective<OpenApiLink>
+    public class OpenApiLink : IOpenApiReferenceable, IOpenApiExtensible
     {
         /// <summary>
         /// A relative or absolute reference to an OAS operation.
@@ -106,39 +106,9 @@ private void SerializeInternal(IOpenApiWriter writer, Action<IOpenApiWriter, IOp
             Utils.CheckArgumentNull(writer);
 
             var target = this;
-
-            if (Reference != null)
-            {
-                if (!writer.GetSettings().ShouldInlineReference(Reference))
-                {
-                    callback(writer, Reference);
-                    return;
-                }
-                else
-                {
-                    target = GetEffective(Reference.HostDocument);
-                }
-            }
             action(writer, target);
         }
 
-        /// <summary>
-        /// Returns an effective OpenApiLink object based on the presence of a $ref
-        /// </summary>
-        /// <param name="doc">The host OpenApiDocument that contains the reference.</param>
-        /// <returns>OpenApiLink</returns>
-        public OpenApiLink GetEffective(OpenApiDocument doc)
-        {
-            if (Reference != null)
-            {
-                return doc.ResolveReferenceTo<OpenApiLink>(Reference);
-            }
-            else
-            {
-                return this;
-            }
-        }
-
         /// <summary>
         /// Serialize to OpenAPI V31 document without using reference.
         /// </summary>

src/Microsoft.OpenApi/Models/OpenApiLink.cs

@@ -5,6 +5,7 @@
 using System.Collections.Generic;
 using System.Linq;
 using Microsoft.OpenApi.Interfaces;
+using Microsoft.OpenApi.Models.References;
 using Microsoft.OpenApi.Writers;
 
 namespace Microsoft.OpenApi.Models
@@ -256,18 +257,13 @@ public void SerializeAsV2(IOpenApiWriter writer)
                 }
                 else if (RequestBody.Reference != null)
                 {
+                    var hostDocument = RequestBody.Reference.HostDocument;
                     parameters.Add(
-                        new()
-                        {
-                            UnresolvedReference = true,
-                            Reference = RequestBody.Reference
-                        });
+                        new OpenApiParameterReference(RequestBody.Reference.Id, hostDocument));
 
-                    if (RequestBody.Reference.HostDocument != null)
-                    {
-                        var effectiveRequestBody = RequestBody.GetEffective(RequestBody.Reference.HostDocument);
-                        if (effectiveRequestBody != null)
-                            consumes = effectiveRequestBody.Content.Keys.Distinct().ToList();
+                    if (hostDocument != null)
+                    {                        
+                        consumes = RequestBody.Content.Keys.Distinct().ToList();
                     }
                 }
 
@@ -291,7 +287,7 @@ public void SerializeAsV2(IOpenApiWriter writer)
                     .Concat(
                         Responses
                         .Where(static r => r.Value.Reference is {HostDocument: not null})
-                        .SelectMany(static r => r.Value.GetEffective(r.Value.Reference.HostDocument)?.Content?.Keys))
+                        .SelectMany(static r => r.Value.Content?.Keys))
                     .Distinct()
                     .ToList();