Ignore non-method fields in Path Object Item (#4934)

* path object summary/desc fix * fix(openapi): ignore non method objects under paths --------- Co-authored-by: Arda TANRIKULU <ardatanrikulu@gmail.com>
ardatan · Mar 31, 2023 · cc754db · cc754db
1 parent 870de5d
commit cc754db
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/.changeset/happy-melons-vanish.md b/.changeset/happy-melons-vanish.md
@@ -0,0 +1,5 @@
+---
+'@omnigraph/openapi': patch
+---
+
+Ignore non object path fields
diff --git a/packages/loaders/openapi/src/getJSONSchemaOptionsFromOpenAPIOptions.ts b/packages/loaders/openapi/src/getJSONSchemaOptionsFromOpenAPIOptions.ts
@@ -122,7 +122,7 @@ export async function getJSONSchemaOptionsFromOpenAPIOptions(
     const pathObj = oasOrSwagger.paths[relativePath];
     const pathParameters = pathObj.parameters;
     for (const method in pathObj) {
-      if (method === 'parameters' || method === 'servers') {
+      if (method === 'parameters' || method === 'summary' || method === 'description' || method === 'servers') {
         continue;
       }
       const methodObj = pathObj[method as OpenAPIV2.HttpMethods] as

diff --git a/packages/loaders/openapi/tests/fixtures/basket.json b/packages/loaders/openapi/tests/fixtures/basket.json
@@ -6,6 +6,13 @@
   },
   "paths": {
     "/basket": {
+      "summary": "Get basket",
+      "description": "Get basket",
+      "servers": [
+        {
+          "url": "https://api.example.com"
+        }
+      ],
       "post": {
         "requestBody": {
           "content": {