Fix #1185 - Add additionalProperties when type object has default value and no props

Signed-off-by: Ricardo Zanini <[email protected]>

Author: ricardozanini

client/deployment/src/main/java/io/quarkiverse/openapi/generator/deployment/codegen/OpenApiGeneratorCodeGenBase.java

@@ -473,23 +473,18 @@ private <T> Optional<T> getConfigKeyValues(final Config config, final Path openA
             Class<T> propertyType) {
 
         Optional<String> possibleConfigKey = getConfigKeyValue(config, openApiFilePath);
-        if (possibleConfigKey.isPresent()) {
-            return getValuesByConfigKey(config, CodegenConfig.getSpecConfigNameByConfigKey(possibleConfigKey.get(), configName),
-                    propertyType, configName);
-        }
+        return possibleConfigKey
+                .flatMap(s -> getValuesByConfigKey(config, CodegenConfig.getSpecConfigNameByConfigKey(s, configName),
+                        propertyType, configName));
 
-        return Optional.empty();
     }
 
     private <K, V> Optional<Map<K, V>> getConfigKeyValues(final SmallRyeConfig config, final Path openApiFilePath,
             CodegenConfig.ConfigName configName,
             Class<K> kClass, Class<V> vClass) {
 
         Optional<String> possibleConfigKey = getConfigKeyValue(config, openApiFilePath);
-        if (possibleConfigKey.isPresent()) {
-            return getValuesByConfigKey(config, configName, kClass, vClass, possibleConfigKey.get());
-        }
+        return possibleConfigKey.flatMap(s -> getValuesByConfigKey(config, configName, kClass, vClass, s));
 
-        return Optional.empty();
     }
 }

client/deployment/src/main/java/io/quarkiverse/openapi/generator/deployment/codegen/OpenApiGeneratorOutputPaths.java

@@ -2,7 +2,6 @@
 
 import java.nio.file.Path;
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.Collection;
 import java.util.Iterator;
 import java.util.List;
@@ -12,7 +11,7 @@ public class OpenApiGeneratorOutputPaths {
     public static final String OPENAPI_PATH = "open-api";
     public static final String STREAM_PATH = "open-api-stream";
 
-    private static final Collection<String> rootPaths = Arrays.asList(STREAM_PATH);
+    private static final Collection<String> rootPaths = List.of(STREAM_PATH);
 
     public static Path getRelativePath(Path path) {
         List<String> paths = new ArrayList<>();

client/deployment/src/main/java/io/quarkiverse/openapi/generator/deployment/wrapper/QuarkusJavaClientCodegen.java

@@ -148,6 +148,20 @@ public CodegenModel fromModel(String name, Schema model) {
         return codegenModel;
     }
 
+    @Override
+    public CodegenProperty fromProperty(String name, Schema p, boolean required, boolean schemaIsFromAdditionalProperties) {
+        if (p != null && p.getType() != null) {
+            // Property is a `type: object` without `additionalProperties: true`, without `properties`, but has `default` values set!
+            // In this peculiar situation, the template will try to initialize a Java Object with such values, and it will fail to compile.
+            // See https://github.com/quarkiverse/quarkus-openapi-generator/issues/1185 for more context.
+            if (p.getType().equals("object") && p.getDefault() != null && p.getAdditionalProperties() == null
+                    && p.getItems() == null) {
+                p.setAdditionalProperties(true);
+            }
+        }
+        return super.fromProperty(name, p, required, schemaIsFromAdditionalProperties);
+    }
+
     private void warnIfDuplicated(CodegenModel m) {
         Set<String> propertyNames = new TreeSet<>();
         for (CodegenProperty element : m.allVars) {

client/deployment/src/test/java/io/quarkiverse/openapi/generator/deployment/wrapper/OpenApiClientGeneratorWrapperTest.java

@@ -718,6 +718,30 @@ void verifyDynamicUrlAnnotation() throws Exception {
         }
     }
 
+    @Test
+    void verifyAllowAdditionalPropertiesIfNotPresent() throws Exception {
+        List<File> generatedFiles = createGeneratorWrapperReactive("issue-1185.json")
+                .generate("org.issue1185")
+                .stream()
+                .filter(file -> file.getPath().endsWith("PictureDescriptionLocal.java")).toList();
+
+        assertThat(generatedFiles).isNotEmpty();
+        // generationConfig
+        for (File file : generatedFiles) {
+            try {
+                Optional<FieldDeclaration> var = StaticJavaParser.parse(file)
+                        .findAll(FieldDeclaration.class).stream()
+                        .filter(c -> c.getVariable(0)
+                                .getNameAsString().equals("generationConfig"))
+                        .findFirst();
+                assertThat(var).isPresent();
+                assertThat(var.get().getElementType().asString()).contains("Map<String,Object>");
+            } catch (FileNotFoundException e) {
+                throw new RuntimeException(e.getMessage());
+            }
+        }
+    }
+
     private List<File> generateRestClientFiles() throws URISyntaxException {
         OpenApiClientGeneratorWrapper generatorWrapper = createGeneratorWrapper("simple-openapi.json").withCircuitBreakerConfig(
                 Map.of("org.openapitools.client.api.DefaultApi", List.of("opThatDoesNotExist", "byeMethodGet")));

client/deployment/src/test/resources/openapi/issue-1185.json

@@ -0,0 +1,89 @@
+{
+  "openapi": "3.1.0",
+  "info": {
+    "title": "Docling Serve – PictureDescriptionLocal",
+    "version": "0.13.0"
+  },
+  "paths": {
+    "/v1alpha/convert/file": {
+      "post": {
+        "summary": "Process File with Local Picture Description",
+        "operationId": "processFileWithPictureDescriptionLocal",
+        "tags": ["Docling"],
+        "requestBody": {
+          "required": true,
+          "content": {
+            "multipart/form-data": {
+              "schema": {
+                "type": "object",
+                "properties": {
+                  "files": {
+                    "type": "array",
+                    "items": {
+                      "type": "string",
+                      "format": "binary"
+                    }
+                  },
+                  "picture_description_local": {
+                    "$ref": "#/components/schemas/PictureDescriptionLocal"
+                  }
+                },
+                "required": ["files", "picture_description_local"]
+              }
+            }
+          }
+        },
+        "responses": {
+          "200": {
+            "description": "Successful Response",
+            "content": {
+              "application/json": {
+                "schema": {
+                  "type": "object"
+                }
+              }
+            }
+          }
+        }
+      }
+    }
+  },
+  "components": {
+    "schemas": {
+      "PictureDescriptionLocal": {
+        "type": "object",
+        "title": "PictureDescriptionLocal",
+        "description": "Options for running a local vision-language model in the picture description. Parameters refer to a model hosted on Hugging Face.",
+        "properties": {
+          "repo_id": {
+            "type": "string",
+            "title": "Repo Id",
+            "description": "Repository ID on the Hugging Face Hub."
+          },
+          "prompt": {
+            "type": "string",
+            "title": "Prompt",
+            "description": "Prompt used when calling the vision-language model.",
+            "default": "Describe this image in a few sentences."
+          },
+          "generation_config": {
+            "type": "object",
+            "title": "Generation Config",
+            "description": "Config from Transformers’ GenerationConfig (e.g. max_new_tokens, do_sample).",
+            "default": {
+              "max_new_tokens": 200,
+              "do_sample": false
+            },
+            "examples": [
+              {
+                "do_sample": false,
+                "max_new_tokens": 200
+              }
+            ]
+          }
+        },
+        "required": ["repo_id"]
+      }
+    }
+  }
+}

docs/modules/ROOT/pages/includes/quarkus-openapi-generator-moqu-wiremock.adoc

@@ -15,7 +15,7 @@ endif::add-copy-button-to-config-props[]
 
 [.description]
 --
-Path to the Moqu (relative to the project).
+Path to the Moqu OpenAPI files, relative to the `src/main/resources` directory.
 
 
 ifdef::add-copy-button-to-env-var[]

docs/modules/ROOT/pages/includes/quarkus-openapi-generator-moqu-wiremock_quarkus.openapi-generator.adoc

@@ -15,7 +15,7 @@ endif::add-copy-button-to-config-props[]
 
 [.description]
 --
-Path to the Moqu (relative to the project).
+Path to the Moqu OpenAPI files, relative to the `src/main/resources` directory.
 
 
 ifdef::add-copy-button-to-env-var[]