should not walk primitive types

Author: hauner

openapi-parser/src/main/java/io/openapiparser/OpenApiBundler.java

@@ -8,6 +8,7 @@
 import io.openapiprocessor.jsonschema.schema.Keyword;
 import io.openapiprocessor.jsonschema.schema.Keywords;
 import io.openapiprocessor.jsonschema.schema.*;
+import io.openapiprocessor.jsonschema.support.Types;
 import org.checkerframework.checker.nullness.qual.Nullable;
 
 import java.net.URI;
@@ -360,6 +361,10 @@ private void walkSchemaArray (Scope currentScope, Object value, JsonPointer loca
         int index = 0;
         for (Object item : items) {
             JsonPointer itemLocation = location.append (index);
+            if (!Types.isObject (item)) {
+                continue;
+            }
+
             walkSchema (currentScope, item, itemLocation);
             index++;
         }

openapi-parser/src/test/kotlin/io/openapiparser/OpenApiBundlerSpec.kt

@@ -5,12 +5,14 @@
 
 package io.openapiparser
 
+import io.kotest.assertions.throwables.shouldNotThrowAny
 import io.kotest.core.spec.style.FreeSpec
 import io.kotest.datatest.withData
 import io.kotest.matchers.booleans.shouldBeFalse
 import io.kotest.matchers.booleans.shouldBeTrue
 import io.kotest.matchers.nulls.shouldNotBeNull
 import io.kotest.matchers.shouldBe
+import io.openapiprocessor.jackson.JacksonConverter
 import io.openapiprocessor.jsonschema.support.Types.asObject
 import io.openapiprocessor.jsonschema.reader.UriReader
 import io.openapiprocessor.jsonschema.schema.JsonPointer.from
@@ -58,6 +60,38 @@ class OpenApiBundlerSpec : FreeSpec({
 
     data class Data(val api: String, val bundle: (result: ResolverResult) -> Bucket)
 
+    "do not walk array on simple type" - {
+        val context = Context(Scope.empty(), ReferenceRegistry())
+
+        val converter = JacksonConverter()
+        val document = converter.convert("""
+                    openapi: 3.0.2
+                    info:
+                      title: do not walk tags
+                      version: 1.0.0
+        
+                    paths:
+                      /foo:
+                        get:
+                          tags:
+                            - a tag
+                          responses:
+                            '204':
+                              description: none
+                    """.trimIndent()
+                )
+
+        val documents = DocumentStore()
+        val scope = Scope.empty()
+        documents.addId(scope.documentUri, document)
+
+        val bucket = Bucket.createBucket(scope, document)
+
+        shouldNotThrowAny {
+            OpenApiBundler(context, documents, bucket!!).bundle()
+        }
+    }
+
     withData(
         mapOf(
             $$"bundle handles $ref loop, 30" to Data("/bundle-ref-loop/openapi30.yaml", ::bundle30),